Merge "Allow Status::hasMessage to work with Message objects."

[lhc/web/wiklou.git] / maintenance / storage / fixBug20757.php

diff --git a/maintenance/storage/fixBug20757.php b/maintenance/storage/fixBug20757.php
index e832b4e..bfa269b 100644 (file)
--- a/maintenance/storage/fixBug20757.php
+++ b/maintenance/storage/fixBug20757.php
@@ -57,14 +57,9 @@ class FixBug20757 extends Maintenance {
 
                $totalRevs = $dbr->selectField( 'text', 'MAX(old_id)', false, __METHOD__ );
 
-               if ( $dbr->getType() == 'mysql'
-                       && version_compare( $dbr->getServerVersion(), '4.1.0', '>=' )
-               ) {
+               if ( $dbr->getType() == 'mysql' ) {
                        // In MySQL 4.1+, the binary field old_text has a non-working LOWER() function
                        $lowerLeft = 'LOWER(CONVERT(LEFT(old_text,22) USING latin1))';
-               } else {
-                       // No CONVERT() in MySQL 4.0
-                       $lowerLeft = 'LOWER(LEFT(old_text,22))';
                }
 
                while ( true ) {
@@ -307,8 +302,8 @@ class FixBug20757 extends Maintenance {
        /**
         * This is based on part of HistoryBlobStub::getText().
         * Determine if the text can be retrieved from the row in the normal way.
-        * @param $stub
-        * @param $secondaryRow
+        * @param array $stub
+        * @param stdClass $secondaryRow
         * @return bool
         */
        function isUnbrokenStub( $stub, $secondaryRow ) {
@@ -316,7 +311,10 @@ class FixBug20757 extends Maintenance {
                $text = $secondaryRow->old_text;
                if ( in_array( 'external', $flags ) ) {
                        $url = $text;
-                       @list( /* $proto */ , $path ) = explode( '://', $url, 2 );
+                       wfSuppressWarnings();
+                       list( /* $proto */ , $path ) = explode( '://', $url, 2 );
+                       wfRestoreWarnings();
+
                        if ( $path == "" ) {
                                return false;
                        }