passing $_POST['code'] to check() is not always a string about securimage HOT 4 CLOSED

Admittedly, the documentation on the site is in need of an update, and the current state of things is to use the examples and code as docs (bad!). But this does seem like a fair warning.

The phpDoc comments for the check() function reads:

    /**
     * Checks a given code against the correct value from the session and/or database.
     *
     * @param string $code      The captcha code to check
     * @param string $captchaId The ID of the captcha being checked
     * @param bool   $alwaysDelete True to delete captcha data after a failed guess.
     *   Useful if the form post results in a full page reload and the old captcha
     *   ID is never used again. Not so useful on Ajax forms.  (default false)
     *
     * <code>
     *     $code = $_POST['code'];
     *     $img  = new Securimage();
     *     if ($img->check($code) == true) {
     *         $captcha_valid = true;
     *     } else {
     *         $captcha_valid = false;
     *     }
     * </code>
     *
     * @return bool true if the given code was correct, false if not.
     */

Admittedly, the 'string' parameter hint is subtle as well.

I'm not against type checking the $code parameter explicitly to prevent this warning from strtolower. Would a trigger_error to the effect of The $code parameter passed to Securimage::check() must be a string, $type given be sufficient in your opinion?

from securimage.

My error log is full of this error:
The $code parameter passed to Securimage::check() must be a string, NULL given in /home/xxxx/public_html/securimage/securimage.php on line 1278

from securimage.

My error log is full of this error:
The $code parameter passed to Securimage::check() must be a string, NULL given in /home/xxxx/public_html/securimage/securimage.php on line 1278

Yes I get this too.

from securimage.

I solved this problem. The problem is solved when I send the values sent from the form with GET instead of POST and receive them with GET.

from securimage.