Thread

I’m wondering if anyone can help.  I’m not familiar with PHP, so I have no idea what I’m looking at here.  The upgrade is causing the following notice to appear at the top of my weblogs:

Notice: Undefined variable: now in /home/.../system/core/core.functions.php on line 1901

Line 1901 is:

$img_name = $now.’.jpg’;

. 

Can anyone offer a brief description on what this means?  Should I contact my web developer?
Thanks, Courtney

Hi Courtney,

That error is inside EE’s create_captcha() function. The short story is that it’s possible for the variable $now to not be set before it is used on line 1901. The variable is created inside a conditional that begins at line 1761:

if ((mt_rand() % 100) < $SESS->gc_probability)
        {
            ...blah blah blah…

            $now = ((float)$usec + (float)$sec);

            ...blah blah blah…
        }

If that conditional doesn’t evaluate to true, then $now is never set, and you get the PHP notice you pointed out. On a quick glance, this appears to be a bug.

My code looks the same as yours:

if ((mt_rand() % 100) < $SESS->gc_probability)
        {
            ...blah blah blah…

            $now = ((float)$usec + (float)$sec);

            ...blah blah blah…
        }

Any idea of a fix to remove that notice?
Courtney

Mr. Wilson nailed it.  It’s a known bug, already fixed for the next release (see bug tracker).

For a quick fix- you can open up system/core/core.functions.php and find the code above.  Add:

list($usec, $sec) = explode(" ", microtime());
$now = ((float)$usec + (float)$sec);

Above that conditional and you’ll be fine until the official fix comes out.

Robin & Mr. Wilson, Thank you both for your help!

😉  Just sorry you caught a bug.  Should be gone next build upgrade.

Otherwise- you all squared away and we good to close this one out?

Yep! Thanks again.

Good deal- is done!