]> scripts.mit.edu Git - autoinstalls/mediawiki.git/blobdiff - maintenance/FiveUpgrade.inc
MediaWiki 1.11.0
[autoinstalls/mediawiki.git] / maintenance / FiveUpgrade.inc
index 4c8fcbc797f7f531d4430fc2e1ac3ffa4ef072f8..9a882bc60a97ab9fdde51e259f7dafdf9829f327 100644 (file)
@@ -11,24 +11,23 @@ define( 'MW_UPGRADE_CALLBACK', null  ); // for self-documentation only
 
 class FiveUpgrade {
        function FiveUpgrade() {
-               global $wgDatabase;
                $this->conversionTables = $this->prepareWindows1252();
-               
+
                $this->dbw =& $this->newConnection();
                $this->dbr =& $this->streamConnection();
-               
+
                $this->cleanupSwaps = array();
                $this->emailAuth = false; # don't preauthenticate emails
                $this->maxLag    = 10; # if slaves are lagged more than 10 secs, wait
        }
-       
+
        function doing( $step ) {
                return is_null( $this->step ) || $step == $this->step;
        }
-       
+
        function upgrade( $step ) {
                $this->step = $step;
-               
+
                $tables = array(
                        'page',
                        'links',
@@ -49,13 +48,13 @@ class FiveUpgrade {
                                $this->$method();
                        }
                }
-               
+
                if( $this->doing( 'cleanup' ) ) {
                        $this->upgradeCleanup();
                }
        }
-       
-       
+
+
        /**
         * Open a connection to the master server with the admin rights.
         * @return Database
@@ -64,10 +63,10 @@ class FiveUpgrade {
        function &newConnection() {
                global $wgDBadminuser, $wgDBadminpassword;
                global $wgDBserver, $wgDBname;
-               $db =& new Database( $wgDBserver, $wgDBadminuser, $wgDBadminpassword, $wgDBname );
+               $db = new Database( $wgDBserver, $wgDBadminuser, $wgDBadminpassword, $wgDBname );
                return $db;
        }
-       
+
        /**
         * Open a second connection to the master server, with buffering off.
         * This will let us stream large datasets in and write in chunks on the
@@ -83,7 +82,7 @@ class FiveUpgrade {
                $db->query( "SET net_write_timeout=$timeout" );
                return $db;
        }
-       
+
        /**
         * Prepare a conversion array for converting Windows Code Page 1252 to
         * UTF-8. This should provide proper conversion of text that was miscoded
@@ -98,7 +97,7 @@ class FiveUpgrade {
                # http://www.unicode.org/Public/MAPPINGS/VENDORS/MICSFT/WINDOWS/CP1252.TXT
                static $cp1252 = array(
                        0x80 => 0x20AC, #EURO SIGN
-                       0x81 => UNICODE_REPLACEMENT,
+                       0x81 => 0xFFFD, #REPLACEMENT CHARACTER (no mapping)
                        0x82 => 0x201A, #SINGLE LOW-9 QUOTATION MARK
                        0x83 => 0x0192, #LATIN SMALL LETTER F WITH HOOK
                        0x84 => 0x201E, #DOUBLE LOW-9 QUOTATION MARK
@@ -110,10 +109,10 @@ class FiveUpgrade {
                        0x8A => 0x0160, #LATIN CAPITAL LETTER S WITH CARON
                        0x8B => 0x2039, #SINGLE LEFT-POINTING ANGLE QUOTATION MARK
                        0x8C => 0x0152, #LATIN CAPITAL LIGATURE OE
-                       0x8D => UNICODE_REPLACEMENT,
+                       0x8D => 0xFFFD, #REPLACEMENT CHARACTER (no mapping)
                        0x8E => 0x017D, #LATIN CAPITAL LETTER Z WITH CARON
-                       0x8F => UNICODE_REPLACEMENT,
-                       0x90 => UNICODE_REPLACEMENT,
+                       0x8F => 0xFFFD, #REPLACEMENT CHARACTER (no mapping)
+                       0x90 => 0xFFFD, #REPLACEMENT CHARACTER (no mapping)
                        0x91 => 0x2018, #LEFT SINGLE QUOTATION MARK
                        0x92 => 0x2019, #RIGHT SINGLE QUOTATION MARK
                        0x93 => 0x201C, #LEFT DOUBLE QUOTATION MARK
@@ -126,7 +125,7 @@ class FiveUpgrade {
                        0x9A => 0x0161, #LATIN SMALL LETTER S WITH CARON
                        0x9B => 0x203A, #SINGLE RIGHT-POINTING ANGLE QUOTATION MARK
                        0x9C => 0x0153, #LATIN SMALL LIGATURE OE
-                       0x9D => UNICODE_REPLACEMENT,
+                       0x9D => 0xFFFD, #REPLACEMENT CHARACTER (no mapping)
                        0x9E => 0x017E, #LATIN SMALL LETTER Z WITH CARON
                        0x9F => 0x0178, #LATIN CAPITAL LETTER Y WITH DIAERESIS
                        );
@@ -137,7 +136,7 @@ class FiveUpgrade {
                }
                return $pairs;
        }
-       
+
        /**
         * Convert from 8-bit Windows-1252 to UTF-8 if necessary.
         * @param string $text
@@ -152,17 +151,17 @@ class FiveUpgrade {
                                ? strtr( $text, $this->conversionTables )
                                : $text );
        }
-       
+
        /**
         * Dump timestamp and message to output
         * @param string $message
         * @access private
         */
        function log( $message ) {
-               echo wfTimestamp( TS_DB ) . ': ' . $message . "\n";
+               echo wfWikiID() . ' ' . wfTimestamp( TS_DB ) . ': ' . $message . "\n";
                flush();
        }
-       
+
        /**
         * Initialize the chunked-insert system.
         * Rows will be inserted in chunks of the given number, rather
@@ -187,7 +186,7 @@ class FiveUpgrade {
                $this->chunkTable = $table;
                $this->chunkFunction = $fname;
        }
-       
+
        /**
         * Chunked inserts: perform an insert if we've reached the chunk limit.
         * Prints a progress report with estimated completion time.
@@ -201,22 +200,22 @@ class FiveUpgrade {
        function addChunk( &$chunk, $key = null ) {
                if( count( $chunk ) >= $this->chunkSize ) {
                        $this->insertChunk( $chunk );
-               
+
                        $this->chunkCount += count( $chunk );
                        $now = wfTime();
                        $delta = $now - $this->chunkStartTime;
                        $rate = $this->chunkCount / $delta;
-                       
+
                        if( is_null( $key ) ) {
                                $completed = $this->chunkCount;
                        } else {
                                $completed = $key;
                        }
                        $portion = $completed / $this->chunkFinal;
-                       
+
                        $estimatedTotalTime = $delta / $portion;
                        $eta = $this->chunkStartTime + $estimatedTotalTime;
-                       
+
                        printf( "%s: %6.2f%% done on %s; ETA %s [%d/%d] %.2f/sec\n",
                                wfTimestamp( TS_DB, intval( $now ) ),
                                $portion * 100.0,
@@ -226,11 +225,11 @@ class FiveUpgrade {
                                $this->chunkFinal,
                                $rate );
                        flush();
-                       
+
                        $chunk = array();
                }
        }
-       
+
        /**
         * Chunked inserts: perform an insert unconditionally, at the end, and log.
         * @param array &$chunk -- This will be emptied if an insert is done.
@@ -243,7 +242,7 @@ class FiveUpgrade {
                }
                $this->log( "100.00% done on $this->chunkTable (last chunk $n rows)." );
        }
-       
+
        /**
         * Chunked inserts: perform an insert.
         * @param array &$chunk -- This will be emptied if an insert is done.
@@ -254,8 +253,8 @@ class FiveUpgrade {
                wfWaitForSlaves( $this->maxLag );
                $this->dbw->insert( $this->chunkTable, $chunk, $this->chunkFunction, $this->chunkOptions );
        }
-       
-       
+
+
        /**
         * Copy and transcode a table to table_temp.
         * @param string $name Base name of the source table
@@ -271,21 +270,20 @@ class FiveUpgrade {
         */
        function copyTable( $name, $tabledef, $fields, $callback = null ) {
                $fname = 'FiveUpgrade::copyTable';
-               
+
                $name_temp = $name . '_temp';
                $this->log( "Migrating $name table to $name_temp..." );
 
-               $table      = $this->dbw->tableName( $name );
                $table_temp = $this->dbw->tableName( $name_temp );
-               
+
                // Create temporary table; we're going to copy everything in there,
                // then at the end rename the final tables into place.
                $def = str_replace( '$1', $table_temp, $tabledef );
                $this->dbw->query( $def, $fname );
-               
+
                $numRecords = $this->dbw->selectField( $name, 'COUNT(*)', '', $fname );
                $this->setChunkScale( 100, $numRecords, $name_temp, $fname );
-               
+
                // Pull all records from the second, streaming database connection.
                $sourceFields = array_keys( array_filter( $fields,
                        create_function( '$x', 'return $x !== MW_UPGRADE_NULL;' ) ) );
@@ -293,7 +291,7 @@ class FiveUpgrade {
                        $sourceFields,
                        '',
                        $fname );
-               
+
                $add = array();
                while( $row = $this->dbr->fetchObject( $result ) ) {
                        $copy = array();
@@ -316,26 +314,26 @@ class FiveUpgrade {
                }
                $this->lastChunk( $add );
                $this->dbr->freeResult( $result );
-               
+
                $this->log( "Done converting $name." );
                $this->cleanupSwaps[] = $name;
        }
-       
+
        function upgradePage() {
                $fname = "FiveUpgrade::upgradePage";
                $chunksize = 100;
-               
+
                if( $this->dbw->tableExists( 'page' ) ) {
                        $this->log( 'Page table already exists; aborting.' );
                        die( -1 );
                }
-               
+
                $this->log( "Checking cur table for unique title index and applying if necessary" );
                checkDupes( true );
 
                $this->log( "...converting from cur/old to page/revision/text DB structure." );
-               
-               extract( $this->dbw->tableNames( 'cur', 'old', 'page', 'revision', 'text' ) );
+
+               list ($cur, $old, $page, $revision, $text) = $this->dbw->tableNamesN( 'cur', 'old', 'page', 'revision', 'text' );
 
                $this->log( "Creating page and revision tables..." );
                $this->dbw->query("CREATE TABLE $page (
@@ -366,7 +364,7 @@ class FiveUpgrade {
                        rev_timestamp char(14) binary NOT NULL default '',
                        rev_minor_edit tinyint(1) unsigned NOT NULL default '0',
                        rev_deleted tinyint(1) unsigned NOT NULL default '0',
-  
+
                        PRIMARY KEY rev_page_id (rev_page, rev_id),
                        UNIQUE INDEX rev_id (rev_id),
                        INDEX rev_timestamp (rev_timestamp),
@@ -375,7 +373,7 @@ class FiveUpgrade {
                        INDEX usertext_timestamp (rev_user_text,rev_timestamp)
                        ) TYPE=InnoDB", $fname );
 
-               $maxold = IntVal( $this->dbw->selectField( 'old', 'max(old_id)', '', $fname ) );
+               $maxold = intval( $this->dbw->selectField( 'old', 'max(old_id)', '', $fname ) );
                $this->log( "Last old record is {$maxold}" );
 
                global $wgLegacySchemaConversion;
@@ -395,7 +393,7 @@ class FiveUpgrade {
 
                $maxcur = $this->dbw->selectField( 'cur', 'max(cur_id)', '', $fname );
                $this->log( "Last cur entry is $maxcur" );
-               
+
                /**
                 * Copy placeholder records for each page's current version into old
                 * Don't do any conversion here; text records are converted at runtime
@@ -424,7 +422,7 @@ class FiveUpgrade {
                }
                $this->lastChunk( $add );
                $this->dbr->freeResult( $result );
-               
+
                /**
                 * Copy revision metadata from old into revision.
                 * We'll also do UTF-8 conversion of usernames and comments.
@@ -434,7 +432,7 @@ class FiveUpgrade {
                #$countold = $this->dbw->selectField( 'old', 'count(old_id)', '', $fname );
                $countold = $this->dbw->selectField( 'old', 'max(old_id)', '', $fname );
                $this->setChunkScale( $chunksize, $countold, 'revision', $fname );
-               
+
                $this->log( "......Setting up revision table." );
                $result = $this->dbr->query(
                        "SELECT old_id, cur_id, old_comment, old_user, old_user_text,
@@ -457,7 +455,7 @@ class FiveUpgrade {
                }
                $this->lastChunk( $add );
                $this->dbr->freeResult( $result );
-               
+
 
                /**
                 * Copy page metadata from cur into page.
@@ -490,37 +488,38 @@ class FiveUpgrade {
                }
                $this->lastChunk( $add );
                $this->dbr->freeResult( $result );
-               
+
                $this->log( "...done with cur/old -> page/revision." );
        }
-       
+
        function upgradeLinks() {
                $fname = 'FiveUpgrade::upgradeLinks';
                $chunksize = 200;
-               extract( $this->dbw->tableNames( 'links', 'brokenlinks', 'pagelinks', 'cur' ) );
-               
+               list ($links, $brokenlinks, $pagelinks, $cur) = $this->dbw->tableNamesN( 'links', 'brokenlinks', 'pagelinks', 'cur' );
+
                $this->log( 'Checking for interwiki table change in case of bogus items...' );
                if( $this->dbw->fieldExists( 'interwiki', 'iw_trans' ) ) {
                        $this->log( 'interwiki has iw_trans.' );
                } else {
+                       global $IP;
                        $this->log( 'adding iw_trans...' );
-                       dbsource( 'maintenance/archives/patch-interwiki-trans.sql', $this->dbw );
+                       dbsource( $IP . '/maintenance/archives/patch-interwiki-trans.sql', $this->dbw );
                        $this->log( 'added iw_trans.' );
                }
-               
+
                $this->log( 'Creating pagelinks table...' );
                $this->dbw->query( "
 CREATE TABLE $pagelinks (
   -- Key to the page_id of the page containing the link.
   pl_from int(8) unsigned NOT NULL default '0',
-  
+
   -- Key to page_namespace/page_title of the target page.
   -- The target page may or may not exist, and due to renames
   -- and deletions may refer to different page records as time
   -- goes by.
   pl_namespace int NOT NULL default '0',
   pl_title varchar(255) binary NOT NULL default '',
-  
+
   UNIQUE KEY pl_from(pl_from,pl_namespace,pl_title),
   KEY (pl_namespace,pl_title)
 
@@ -546,7 +545,7 @@ CREATE TABLE $pagelinks (
                } else {
                        $this->log( 'no links!' );
                }
-               
+
                $this->log( 'Importing brokenlinks -> pagelinks' );
                $nbrokenlinks = $this->dbw->selectField( 'brokenlinks', 'count(*)', '', $fname );
                if( $nbrokenlinks ) {
@@ -572,10 +571,10 @@ CREATE TABLE $pagelinks (
                } else {
                        $this->log( 'no brokenlinks!' );
                }
-               
+
                $this->log( 'Done with links.' );
        }
-       
+
        function upgradeUser() {
                // Apply unique index, if necessary:
                $duper = new UserDupes( $this->dbw );
@@ -587,7 +586,7 @@ CREATE TABLE $pagelinks (
                                $this->log( "WARNING: Duplicate user accounts, may explode!" );
                        }
                }
-               
+
                $tabledef = <<<END
 CREATE TABLE $1 (
   user_id int(5) unsigned NOT NULL auto_increment,
@@ -625,14 +624,14 @@ END;
                $this->copyTable( 'user', $tabledef, $fields,
                        array( &$this, 'userCallback' ) );
        }
-       
+
        function userCallback( $row, $copy ) {
                $now = $this->dbw->timestamp();
                $copy['user_touched'] = $now;
                $copy['user_email_authenticated'] = $this->emailAuth ? $now : null;
                return $copy;
        }
-       
+
        function upgradeImage() {
                $tabledef = <<<END
 CREATE TABLE $1 (
@@ -649,7 +648,7 @@ CREATE TABLE $1 (
   img_user int(5) unsigned NOT NULL default '0',
   img_user_text varchar(255) binary NOT NULL default '',
   img_timestamp char(14) binary NOT NULL default '',
-  
+
   PRIMARY KEY img_name (img_name),
   INDEX img_size (img_size),
   INDEX img_timestamp (img_timestamp)
@@ -672,13 +671,13 @@ END;
                $this->copyTable( 'image', $tabledef, $fields,
                        array( &$this, 'imageCallback' ) );
        }
-       
+
        function imageCallback( $row, $copy ) {
                global $options;
                if( !isset( $options['noimage'] ) ) {
                        // Fill in the new image info fields
                        $info = $this->imageInfo( $row->img_name );
-                       
+
                        $copy['img_width'     ] = $info['width'];
                        $copy['img_height'    ] = $info['height'];
                        $copy['img_metadata'  ] = ""; // loaded on-demand
@@ -687,17 +686,14 @@ END;
                        $copy['img_major_mime'] = $info['major'];
                        $copy['img_minor_mime'] = $info['minor'];
                }
-               
+
                // If doing UTF8 conversion the file must be renamed
                $this->renameFile( $row->img_name, 'wfImageDir' );
-               
+
                return $copy;
        }
-       
-       function imageInfo( $name, $subdirCallback='wfImageDir', $basename = null ) {
-               if( is_null( $basename ) ) $basename = $name;
-               $dir = call_user_func( $subdirCallback, $basename );
-               $filename = $dir . '/' . $name;
+
+       function imageInfo( $filename ) {
                $info = array(
                        'width'  => 0,
                        'height' => 0,
@@ -705,33 +701,26 @@ END;
                        'media'  => '',
                        'major'  => '',
                        'minor'  => '' );
-               
+
                $magic =& wfGetMimeMagic();
                $mime = $magic->guessMimeType( $filename, true );
                list( $info['major'], $info['minor'] ) = explode( '/', $mime );
-               
+
                $info['media'] = $magic->getMediaType( $filename, $mime );
-               
-               # Height and width
-               $gis = false;
-               if( $mime == 'image/svg' ) {
-                       $gis = wfGetSVGsize( $this->imagePath );
-               } elseif( $magic->isPHPImageType( $mime ) ) {
-                       $gis = getimagesize( $filename );
-               } else {
-                       $this->log( "Surprising mime type: $mime" );
-               }
-               if( $gis ) {
-                       $info['width' ] = $gis[0];
-                       $info['height'] = $gis[1];
-               }
-               if( isset( $gis['bits'] ) ) {
+
+               $image = UnregisteredLocalFile::newFromPath( $filename, $mime );
+
+               $info['width'] = $image->getWidth();
+               $info['height'] = $image->getHeight();
+
+               $gis = $image->getImageSize();
+               if ( isset( $gis['bits'] ) ) {
                        $info['bits'] = $gis['bits'];
                }
-               
+
                return $info;
        }
-       
+
 
        /**
         * Truncate a table.
@@ -740,9 +729,9 @@ END;
        function clearTable( $table ) {
                print "Clearing $table...\n";
                $tableName = $this->db->tableName( $table );
-               $this->db->query( 'TRUNCATE $tableName' );
+               $this->db->query( "TRUNCATE $tableName" );
        }
-       
+
        /**
         * Rename a given image or archived image file to the converted filename,
         * leaving a symlink for URL compatibility.
@@ -757,15 +746,15 @@ END;
                        // No need to rename; another field triggered this row.
                        return false;
                }
-               
+
                if( is_null( $basename ) ) $basename = $oldname;
                $ubasename = $this->conv( $basename );
                $oldpath = call_user_func( $subdirCallback, $basename ) . '/' . $oldname;
                $newpath = call_user_func( $subdirCallback, $ubasename ) . '/' . $newname;
-               
+
                $this->log( "$oldpath -> $newpath" );
                if( rename( $oldpath, $newpath ) ) {
-                       $relpath = $this->relativize( $newpath, dirname( $oldpath ) );
+                       $relpath = wfRelativePath( $newpath, dirname( $oldpath ) );
                        if( !symlink( $relpath, $oldpath ) ) {
                                $this->log( "... symlink failed!" );
                        }
@@ -775,49 +764,17 @@ END;
                        return false;
                }
        }
-       
-       /**
-        * Generate a relative path name to the given file.
-        * Assumes Unix-style paths, separators, and semantics.
-        *
-        * @param string $path Absolute destination path including target filename
-        * @param string $from Absolute source path, directory only
-        * @return string
-        * @access private
-        * @static
-        */
-       function relativize( $path, $from ) {
-               $pieces  = explode( '/', dirname( $path ) );
-               $against = explode( '/', $from );
-               
-               // Trim off common prefix
-               while( count( $pieces ) && count( $against )
-                       && $pieces[0] == $against[0] ) {
-                       array_shift( $pieces );
-                       array_shift( $against );
-               }
-               
-               // relative dots to bump us to the parent
-               while( count( $against ) ) {
-                       array_unshift( $pieces, '..' );
-                       array_shift( $against );
-               }
-               
-               array_push( $pieces, basename( $path ) );
-               
-               return implode( '/', $pieces );
-       }
-       
+
        function upgradeOldImage() {
                $tabledef = <<<END
 CREATE TABLE $1 (
   -- Base filename: key to image.img_name
   oi_name varchar(255) binary NOT NULL default '',
-  
+
   -- Filename of the archived file.
   -- This is generally a timestamp and '!' prepended to the base name.
   oi_archive_name varchar(255) binary NOT NULL default '',
-  
+
   -- Other fields as in image...
   oi_size int(8) unsigned NOT NULL default 0,
   oi_width int(5) NOT NULL default 0,
@@ -846,7 +803,7 @@ END;
                $this->copyTable( 'oldimage', $tabledef, $fields,
                        array( &$this, 'oldimageCallback' ) );
        }
-       
+
        function oldimageCallback( $row, $copy ) {
                global $options;
                if( !isset( $options['noimage'] ) ) {
@@ -856,37 +813,37 @@ END;
                        $copy['oi_height'] = $info['height'];
                        $copy['oi_bits'  ] = $info['bits'  ];
                }
-               
+
                // If doing UTF8 conversion the file must be renamed
                $this->renameFile( $row->oi_archive_name, 'wfImageArchiveDir', $row->oi_name );
-               
+
                return $copy;
        }
-       
+
 
        function upgradeWatchlist() {
                $fname = 'FiveUpgrade::upgradeWatchlist';
                $chunksize = 100;
-               
-               extract( $this->dbw->tableNames( 'watchlist', 'watchlist_temp' ) );
-               
+
+               list ($watchlist, $watchlist_temp) = $this->dbw->tableNamesN( 'watchlist', 'watchlist_temp' );
+
                $this->log( 'Migrating watchlist table to watchlist_temp...' );
                $this->dbw->query(
 "CREATE TABLE $watchlist_temp (
   -- Key to user_id
   wl_user int(5) unsigned NOT NULL,
-  
+
   -- Key to page_namespace/page_title
   -- Note that users may watch patches which do not exist yet,
   -- or existed in the past but have been deleted.
   wl_namespace int NOT NULL default '0',
   wl_title varchar(255) binary NOT NULL default '',
-  
+
   -- Timestamp when user was last sent a notification e-mail;
   -- cleared when the user visits the page.
   -- FIXME: add proper null support etc
   wl_notificationtimestamp varchar(14) binary NOT NULL default '0',
-  
+
   UNIQUE KEY (wl_user, wl_namespace, wl_title),
   KEY namespace_title (wl_namespace,wl_title)
 
@@ -895,7 +852,7 @@ END;
                // Fix encoding for Latin-1 upgrades, add some fields,
                // and double article to article+talk pairs
                $numwatched = $this->dbw->selectField( 'watchlist', 'count(*)', '', $fname );
-               
+
                $this->setChunkScale( $chunksize, $numwatched * 2, 'watchlist_temp', $fname );
                $result = $this->dbr->select( 'watchlist',
                        array(
@@ -904,17 +861,16 @@ END;
                                'wl_title' ),
                        '',
                        $fname );
-               
+
                $add = array();
                while( $row = $this->dbr->fetchObject( $result ) ) {
-                       $now = $this->dbw->timestamp();
                        $add[] = array(
                                'wl_user'      =>                        $row->wl_user,
                                'wl_namespace' => Namespace::getSubject( $row->wl_namespace ),
                                'wl_title'     =>           $this->conv( $row->wl_title ),
                                'wl_notificationtimestamp' =>            '0' );
                        $this->addChunk( $add );
-                       
+
                        $add[] = array(
                                'wl_user'      =>                        $row->wl_user,
                                'wl_namespace' =>    Namespace::getTalk( $row->wl_namespace ),
@@ -924,34 +880,34 @@ END;
                }
                $this->lastChunk( $add );
                $this->dbr->freeResult( $result );
-               
+
                $this->log( 'Done converting watchlist.' );
                $this->cleanupSwaps[] = 'watchlist';
        }
 
        function upgradeLogging() {
-               $tabledef = <<<END
+               $tabledef = <<<ENDS
 CREATE TABLE $1 (
   -- Symbolic keys for the general log type and the action type
   -- within the log. The output format will be controlled by the
   -- action field, but only the type controls categorization.
   log_type char(10) NOT NULL default '',
   log_action char(10) NOT NULL default '',
-  
+
   -- Timestamp. Duh.
   log_timestamp char(14) NOT NULL default '19700101000000',
-  
+
   -- The user who performed this action; key to user_id
   log_user int unsigned NOT NULL default 0,
-  
+
   -- Key to the page affected. Where a user is the target,
   -- this will point to the user page.
   log_namespace int NOT NULL default 0,
   log_title varchar(255) binary NOT NULL default '',
-  
+
   -- Freeform text. Interpreted as edit history comments.
   log_comment varchar(255) NOT NULL default '',
-  
+
   -- LF separated list of miscellaneous parameters
   log_params blob NOT NULL default '',
 
@@ -960,7 +916,7 @@ CREATE TABLE $1 (
   KEY page_time (log_namespace, log_title, log_timestamp)
 
 ) TYPE=InnoDB
-END;
+ENDS;
                $fields = array(
                        'log_type'      => MW_UPGRADE_COPY,
                        'log_action'    => MW_UPGRADE_COPY,
@@ -974,27 +930,27 @@ END;
        }
 
        function upgradeArchive() {
-               $tabledef = <<<END
+               $tabledef = <<<ENDS
 CREATE TABLE $1 (
   ar_namespace int NOT NULL default '0',
   ar_title varchar(255) binary NOT NULL default '',
   ar_text mediumblob NOT NULL default '',
-  
+
   ar_comment tinyblob NOT NULL default '',
   ar_user int(5) unsigned NOT NULL default '0',
   ar_user_text varchar(255) binary NOT NULL,
   ar_timestamp char(14) binary NOT NULL default '',
   ar_minor_edit tinyint(1) NOT NULL default '0',
-  
+
   ar_flags tinyblob NOT NULL default '',
 
   ar_rev_id int(8) unsigned,
   ar_text_id int(8) unsigned,
-  
+
   KEY name_title_timestamp (ar_namespace,ar_title,ar_timestamp)
 
 ) TYPE=InnoDB
-END;
+ENDS;
                $fields = array(
                        'ar_namespace'  => MW_UPGRADE_COPY,
                        'ar_title'      => MW_UPGRADE_ENCODE,
@@ -1009,25 +965,25 @@ END;
                        'ar_text_id'    => MW_UPGRADE_NULL );
                $this->copyTable( 'archive', $tabledef, $fields );
        }
-       
+
        function upgradeImagelinks() {
                global $wgUseLatin1;
                if( $wgUseLatin1 ) {
-                       $tabledef = <<<END
+                       $tabledef = <<<ENDS
 CREATE TABLE $1 (
   -- Key to page_id of the page containing the image / media link.
   il_from int(8) unsigned NOT NULL default '0',
-  
+
   -- Filename of target image.
   -- This is also the page_title of the file's description page;
   -- all such pages are in namespace 6 (NS_IMAGE).
   il_to varchar(255) binary NOT NULL default '',
-  
+
   UNIQUE KEY il_from(il_from,il_to),
   KEY (il_to)
 
 ) TYPE=InnoDB
-END;
+ENDS;
                        $fields = array(
                                'il_from' => MW_UPGRADE_COPY,
                                'il_to'   => MW_UPGRADE_ENCODE );
@@ -1038,18 +994,18 @@ END;
        function upgradeCategorylinks() {
                global $wgUseLatin1;
                if( $wgUseLatin1 ) {
-                       $tabledef = <<<END
+                       $tabledef = <<<ENDS
 CREATE TABLE $1 (
   cl_from int(8) unsigned NOT NULL default '0',
   cl_to varchar(255) binary NOT NULL default '',
   cl_sortkey varchar(86) binary NOT NULL default '',
   cl_timestamp timestamp NOT NULL,
-  
+
   UNIQUE KEY cl_from(cl_from,cl_to),
   KEY cl_sortkey(cl_to,cl_sortkey),
   KEY cl_timestamp(cl_to,cl_timestamp)
 ) TYPE=InnoDB
-END;
+ENDS;
                        $fields = array(
                                'cl_from'      => MW_UPGRADE_COPY,
                                'cl_to'        => MW_UPGRADE_ENCODE,
@@ -1058,11 +1014,11 @@ END;
                        $this->copyTable( 'categorylinks', $tabledef, $fields );
                }
        }
-       
+
        function upgradeIpblocks() {
                global $wgUseLatin1;
                if( $wgUseLatin1 ) {
-                       $tabledef = <<<END
+                       $tabledef = <<<ENDS
 CREATE TABLE $1 (
   ipb_id int(8) NOT NULL auto_increment,
   ipb_address varchar(40) binary NOT NULL default '',
@@ -1078,7 +1034,7 @@ CREATE TABLE $1 (
   INDEX ipb_user (ipb_user)
 
 ) TYPE=InnoDB
-END;
+ENDS;
                        $fields = array(
                                'ipb_id'        => MW_UPGRADE_COPY,
                                'ipb_address'   => MW_UPGRADE_COPY,
@@ -1091,39 +1047,39 @@ END;
                        $this->copyTable( 'ipblocks', $tabledef, $fields );
                }
        }
-       
+
        function upgradeRecentchanges() {
                // There's a format change in the namespace field
-               $tabledef = <<<END
+               $tabledef = <<<ENDS
 CREATE TABLE $1 (
   rc_id int(8) NOT NULL auto_increment,
   rc_timestamp varchar(14) binary NOT NULL default '',
   rc_cur_time varchar(14) binary NOT NULL default '',
-  
+
   rc_user int(10) unsigned NOT NULL default '0',
   rc_user_text varchar(255) binary NOT NULL default '',
-  
+
   rc_namespace int NOT NULL default '0',
   rc_title varchar(255) binary NOT NULL default '',
-  
+
   rc_comment varchar(255) binary NOT NULL default '',
   rc_minor tinyint(3) unsigned NOT NULL default '0',
-  
+
   rc_bot tinyint(3) unsigned NOT NULL default '0',
   rc_new tinyint(3) unsigned NOT NULL default '0',
-  
+
   rc_cur_id int(10) unsigned NOT NULL default '0',
   rc_this_oldid int(10) unsigned NOT NULL default '0',
   rc_last_oldid int(10) unsigned NOT NULL default '0',
-  
+
   rc_type tinyint(3) unsigned NOT NULL default '0',
   rc_moved_to_ns tinyint(3) unsigned NOT NULL default '0',
   rc_moved_to_title varchar(255) binary NOT NULL default '',
-  
+
   rc_patrolled tinyint(3) unsigned NOT NULL default '0',
-  
+
   rc_ip char(15) NOT NULL default '',
-  
+
   PRIMARY KEY rc_id (rc_id),
   INDEX rc_timestamp (rc_timestamp),
   INDEX rc_namespace_title (rc_namespace, rc_title),
@@ -1132,7 +1088,7 @@ CREATE TABLE $1 (
   INDEX rc_ip (rc_ip)
 
 ) TYPE=InnoDB
-END;
+ENDS;
                $fields = array(
                        'rc_id'             => MW_UPGRADE_COPY,
                        'rc_timestamp'      => MW_UPGRADE_COPY,
@@ -1155,25 +1111,25 @@ END;
                        'rc_ip'             => MW_UPGRADE_COPY );
                $this->copyTable( 'recentchanges', $tabledef, $fields );
        }
-       
+
        function upgradeQuerycache() {
                // There's a format change in the namespace field
-               $tabledef = <<<END
+               $tabledef = <<<ENDS
 CREATE TABLE $1 (
   -- A key name, generally the base name of of the special page.
   qc_type char(32) NOT NULL,
-  
+
   -- Some sort of stored value. Sizes, counts...
   qc_value int(5) unsigned NOT NULL default '0',
-  
+
   -- Target namespace+title
   qc_namespace int NOT NULL default '0',
   qc_title char(255) binary NOT NULL default '',
-  
+
   KEY (qc_type,qc_value)
 
 ) TYPE=InnoDB
-END;
+ENDS;
                $fields = array(
                        'qc_type'      => MW_UPGRADE_COPY,
                        'qc_value'     => MW_UPGRADE_COPY,
@@ -1181,7 +1137,7 @@ END;
                        'qc_title'     => MW_UPGRADE_ENCODE );
                $this->copyTable( 'querycache', $tabledef, $fields );
        }
-       
+
        /**
         * Rename all our temporary tables into final place.
         * We've left things in place so a read-only wiki can continue running
@@ -1189,12 +1145,12 @@ END;
         */
        function upgradeCleanup() {
                $this->renameTable( 'old', 'text' );
-               
+
                foreach( $this->cleanupSwaps as $table ) {
                        $this->swap( $table );
                }
        }
-       
+
        function renameTable( $from, $to ) {
                $this->log( "Renaming $from to $to..." );
 
@@ -1202,12 +1158,12 @@ END;
                $totable   = $this->dbw->tableName( $to );
                $this->dbw->query( "ALTER TABLE $fromtable RENAME TO $totable" );
        }
-       
+
        function swap( $base ) {
                $this->renameTable( $base, "{$base}_old" );
                $this->renameTable( "{$base}_temp", $base );
        }
-       
+
 }
 
-?>
\ No newline at end of file
+?>