Summary: FlaggedRevsHooks::maybeMakeEditor() and
                    checkAutoPromote() seem to use variables defined
                    locally in editSpacingCheck()
           Product: MediaWiki extensions
           Version: any
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: major
          Priority: Normal
         Component: FlaggedRevs

Actually I'm not sure if I'm not wrong, because I thought something like this
should cause a crash, so maybe it's ok, but...

                $pass = self::editSpacingCheck(
                # Make a key to store the results
                if ( !$pass ) {
                    $wgMemc->set( $APSkipKey, 'true',
                        3600 * 24 * $spacing * ( $benchmarks - $needed - 1 ) );
                    return true;
                } else {
                    $wgMemc->set( $sTestKey, 'true', 7 * 24 * 3600 );

$spacing, $benchmarks and $needed seem to be local variables in
self::editSpacingCheck(). As far as I can see there are no such variables in
the current scope. Or are there?

Configure bugmail:
------- You are receiving this mail because: -------
You are on the CC list for the bug.

Wikibugs-l mailing list

Reply via email to