Question

I can’t find anything wrong with this… This should work right?

function ConfirmedNumber()
{
$rs = mysql_query("CALL ConfirmedNumber('" , $_SESSION['UserID'] . "',@Active)");
    while ($row = mysql_fetch_assoc($rs))
    {
        if ($row['Active'] = 1)
        {
        return true;
        }
    }
return false;
}

Assuming the Stored procedure returns a single row with the value ‘1’ in it then I can call the function like this right?

if (ConfirmedNumber())
{
//do some stuff.
}

Answer 1

To expand on my comment:

if ($row['Active'] = 1) should be if ($row['Active'] == 1) to work correctly.

If you want to avoid accidentally doing this in future, you could write your if statements like this:

if (1 == $row['Active'])

This way, you can’t accidentally use = as PHP will throw a Fatal Error. You can read more about Comparison Operators at PHP.net

---

Comment below with the full answer:

The call to the stored proc… line $rs = mysql_query(“CALL ConfirmedNumber(‘” . $_SESSION[‘UserID’] . “‘,@Active)”); had a comma instead of the period in the initial post.