Fixed timestamp comparison in fixUserRegistration

* Also optimized batching logic slightly

Change-Id: I0cedb5b39f996b7baec067890f230cff51468e4b

diff --git a/maintenance/fixUserRegistration.php b/maintenance/fixUserRegistration.php
index 0120f35..43f4b6c 100644 (file)
--- a/maintenance/fixUserRegistration.php
+++ b/maintenance/fixUserRegistration.php
@@ -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 );
        }
 }