First: why the curly braces around every line? All the statements that are to be executed if an if clause evaluates to true should be in one set of braces:
if($something_is_true)
{
$this = 1;
$that = 2;
$theOtherThing = 'something else';
}
As far as not getting any output, I doubt it's a case of too many ifs. It's more likely some syntax and/or logic error that got introduced when you made the change (and which we can't identify without seeing the relative code in context).
Lastly, you might want to consider using switch/case controls instead of all those ifs. It tends to make the code easier to read and thus to maintain.
function iip($answer)
{
//time needed
switch($answer[1])
{
case "Flexible":
$iip_score = $iip_score - 500;
$pqasso_score = $pqasso_score + 1000;
$iso_score = $iso_score - 500;
break;
case "0-12 months":
$iip_score = $iip_score + 500;
$pqasso_score = $pqasso_score - 250;
$iso_score = $iso_score + 500;
break;
case "12 + months":
$iip_score = $iip_score + 1000;
$pqasso_score = $pqasso_score - 500;
$iso_score = $iso_score + 1000;
break;
default:
user_error("Invalid value '{$answer[1]}' received for Time Needed",
E_USER_WARNING);
}
// etc. for rest of function.......
}
