Merge "Fixed timestamp comparison in fixUserRegistration"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Tue, 17 Mar 2015 18:50:50 +0000 (18:50 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Tue, 17 Mar 2015 18:50:50 +0000 (18:50 +0000)
maintenance/fixUserRegistration.php

index 0120f35..43f4b6c 100644 (file)
@@ -66,7 +66,7 @@ class FixUserRegistration extends Maintenance {
                                        __METHOD__
                                );
                                // Update
-                               if ( $timestamp !== false ) {
+                               if ( $timestamp !== null ) {
                                        $dbw->update(
                                                'user',
                                                array( 'user_registration' => $timestamp ),
@@ -83,7 +83,7 @@ class FixUserRegistration extends Maintenance {
                        $this->output( "Waiting for slaves..." );
                        wfWaitForSlaves();
                        $this->output( " done.\n" );
-               } while ( $res->numRows() );
+               } while ( $res->numRows() >= $this->mBatchSize );
        }
 }