From 113f49bb21487f813548fa4e75ecb568c433ecbf Mon Sep 17 00:00:00 2001 From: Reedy Date: Thu, 17 Oct 2013 00:32:40 +0000 Subject: [PATCH] Revert "Add new recentchanges field rc_source to replace rc_type" No feature flag, and also not deployed on WMF wikis. Asking for schema changes by the time we branch tomorrow is just unfair This reverts commit e478c230f6d3f1b7638d4443682eff0df7650def. Change-Id: I125c30e76fa04679cc04ae7b8728ae3a61f35194 --- RELEASE-NOTES-1.22 | 9 -- includes/RecentChange.php | 14 --- includes/installer/MysqlUpdater.php | 1 - includes/installer/PostgresUpdater.php | 3 - includes/installer/SqliteUpdater.php | 3 - maintenance/archives/patch-rc_source.sql | 16 --- maintenance/populateRecentChangesSource.php | 105 -------------------- maintenance/postgres/tables.sql | 1 - maintenance/rebuildrecentchanges.php | 1 - maintenance/tables.sql | 4 - 10 files changed, 157 deletions(-) delete mode 100644 maintenance/archives/patch-rc_source.sql delete mode 100644 maintenance/populateRecentChangesSource.php diff --git a/RELEASE-NOTES-1.22 b/RELEASE-NOTES-1.22 index dadedd2d93..3112e2681d 100644 --- a/RELEASE-NOTES-1.22 +++ b/RELEASE-NOTES-1.22 @@ -366,15 +366,6 @@ changes to languages because of Bugzilla reports. * (bug 46751) Made Buryat (Russia) (буряад) (bxr) fallback to Russian. === Other changes in 1.22 === -* The rc_type field in the recentchanges table has been superseded by a new - rc_source field. The rc_source field is a string representation of the - change type where rc_type was a numeric constant. This field is not yet - queried but will be in a future point release of 1.22. -** Utilize update.php to create and populate this new field. On larger wiki's - which do not wish to update recentchanges table in one large update please - review the sql and comments in maintenance/archives/patch-rc_source.sql. -** The rc_type field of recentchanges will be deprecated in a future point - release. * BREAKING CHANGE: Implementation of MediaWiki's JS and JSON value encoding has changed: ** MediaWiki no longer supports PHP installations in which the native JSON diff --git a/includes/RecentChange.php b/includes/RecentChange.php index 282890f726..980bd0a041 100644 --- a/includes/RecentChange.php +++ b/includes/RecentChange.php @@ -30,7 +30,6 @@ * rc_namespace namespace # * rc_title non-prefixed db key * rc_type is new entry, used to determine whether updating is necessary - * rc_source string representation of change source * rc_minor is minor * rc_cur_id page_id of associated page entry * rc_user user id who made the entry @@ -65,14 +64,6 @@ * @todo document functions and variables */ class RecentChange { - - // Constants for the rc_source field. Extensions may also have - // their own source constants. - const SRC_EDIT = 'mw.edit'; - const SRC_NEW = 'mw.new'; - const SRC_LOG = 'mw.log'; - const SRC_EXTERNAL = 'mw.external'; // obsolete - var $mAttribs = array(), $mExtra = array(); /** @@ -168,7 +159,6 @@ class RecentChange { 'rc_this_oldid', 'rc_last_oldid', 'rc_type', - 'rc_source', 'rc_patrolled', 'rc_ip', 'rc_old_len', @@ -499,7 +489,6 @@ class RecentChange { 'rc_namespace' => $title->getNamespace(), 'rc_title' => $title->getDBkey(), 'rc_type' => RC_EDIT, - 'rc_source' => self::SRC_EDIT, 'rc_minor' => $minor ? 1 : 0, 'rc_cur_id' => $title->getArticleID(), 'rc_user' => $user->getId(), @@ -559,7 +548,6 @@ class RecentChange { 'rc_namespace' => $title->getNamespace(), 'rc_title' => $title->getDBkey(), 'rc_type' => RC_NEW, - 'rc_source' => self::SRC_NEW, 'rc_minor' => $minor ? 1 : 0, 'rc_cur_id' => $title->getArticleID(), 'rc_user' => $user->getId(), @@ -669,7 +657,6 @@ class RecentChange { 'rc_namespace' => $target->getNamespace(), 'rc_title' => $target->getDBkey(), 'rc_type' => RC_LOG, - 'rc_source' => self::SRC_LOG, 'rc_minor' => 0, 'rc_cur_id' => $target->getArticleID(), 'rc_user' => $user->getId(), @@ -729,7 +716,6 @@ class RecentChange { 'rc_comment' => $row->rev_comment, 'rc_minor' => $row->rev_minor_edit ? 1 : 0, 'rc_type' => $row->page_is_new ? RC_NEW : RC_EDIT, - 'rc_source' => $row->page_is_new ? self::SRC_NEW : self::SRC_EDIT, 'rc_cur_id' => $row->page_id, 'rc_this_oldid' => $row->rev_id, 'rc_last_oldid' => isset( $row->rc_last_oldid ) ? $row->rc_last_oldid : 0, diff --git a/includes/installer/MysqlUpdater.php b/includes/installer/MysqlUpdater.php index 9220005c0b..02faf7c4bb 100644 --- a/includes/installer/MysqlUpdater.php +++ b/includes/installer/MysqlUpdater.php @@ -231,7 +231,6 @@ class MysqlUpdater extends DatabaseUpdater { // 1.22 array( 'doIwlinksIndexNonUnique' ), array( 'addIndex', 'iwlinks', 'iwl_prefix_from_title', 'patch-iwlinks-from-title-index.sql' ), - array( 'addField', 'recentchanges', 'rc_source', 'patch-rc_source.sql' ), ); } diff --git a/includes/installer/PostgresUpdater.php b/includes/installer/PostgresUpdater.php index 5868aa8f86..58a54c4219 100644 --- a/includes/installer/PostgresUpdater.php +++ b/includes/installer/PostgresUpdater.php @@ -353,9 +353,6 @@ class PostgresUpdater extends DatabaseUpdater { array( 'addInterwikiType' ), # end array( 'tsearchFixes' ), - - // 1.22 - array( 'addPgField', 'recentchanges', 'rc_source', "TEXT NOT NULL DEFAULT ''" ), ); } diff --git a/includes/installer/SqliteUpdater.php b/includes/installer/SqliteUpdater.php index 17c4f5455b..28d8d66b1b 100644 --- a/includes/installer/SqliteUpdater.php +++ b/includes/installer/SqliteUpdater.php @@ -109,9 +109,6 @@ class SqliteUpdater extends DatabaseUpdater { array( 'addIndex', 'page_props', 'pp_propname_page', 'patch-page_props-propname-page-index.sql' ), array( 'addIndex', 'image', 'img_media_mime', 'patch-img_media_mime-index.sql' ), array( 'addIndex', 'iwlinks', 'iwl_prefix_from_title', 'patch-iwlinks-from-title-index.sql' ), - - // 1.22 - array( 'addField', 'recentchanges', 'rc_source', 'patch-rc_source.sql' ), ); } diff --git a/maintenance/archives/patch-rc_source.sql b/maintenance/archives/patch-rc_source.sql deleted file mode 100644 index 7dedd7451b..0000000000 --- a/maintenance/archives/patch-rc_source.sql +++ /dev/null @@ -1,16 +0,0 @@ --- first step of migrating recentchanges rc_type to rc_source -ALTER TABLE /*$wgDBprefix*/recentchanges - ADD rc_source varbinary(16) NOT NULL default ''; - --- Populate rc_source field with the data from rc_type --- Large wiki's might prefer the PopulateRecentChangeSource maintenance --- script to batch updates into groups rather than all at once. -UPDATE /*$wgDBprefix*/recentchanges - SET rc_source = CASE - WHEN rc_type = 0 THEN 'mw.edit' - WHEN rc_type = 1 THEN 'mw.new' - WHEN rc_type = 3 THEN 'mw.log' - WHEN rc_type = 5 THEN 'mw.external' - ELSE '' - END -WHERE rc_source = ''; diff --git a/maintenance/populateRecentChangesSource.php b/maintenance/populateRecentChangesSource.php deleted file mode 100644 index 0e8e501252..0000000000 --- a/maintenance/populateRecentChangesSource.php +++ /dev/null @@ -1,105 +0,0 @@ -mDescription = "Populates rc_source field of the recentchanges table with the data in rc_type."; - $this->setBatchSize( 100 ); - } - - protected function doDBUpdates() { - $dbw = $this->getDB( DB_MASTER ); - if ( !$dbw->fieldExists( 'recentchanges', 'rc_source' ) ) { - $this->error( 'rc_source field in recentchanges table does not exist.' ); - } - - $start = $dbw->selectField( 'recentchanges', 'MIN(rc_id)', false, __METHOD__ ); - if ( !$start ) { - $this->output( "Nothing to do.\n" ); - return true; - } - $end = $dbw->selectField( 'recentchanges', 'MAX(rc_id)', false, __METHOD__ ); - $end += $this->mBatchSize - 1; - $blockStart = $start; - $blockEnd = $start + $this->mBatchSize - 1; - - $updatedValues = $this->buildUpdateCondition( $dbw ); - - while ( $blockEnd <= $end ) { - $cond = "rc_id BETWEEN $blockStart AND $blockEnd"; - - $dbw->update( - 'recentchanges', - array( $updatedValues ), - array( - "rc_source = ''", - "rc_id BETWEEN $blockStart AND $blockEnd" - ), - __METHOD__ - ); - - $this->output( "." ); - wfWaitForSlaves(); - - $blockStart += $this->mBatchSize; - $blockEnd += $this->mBatchSize; - } - - $this->output( "\nDone.\n" ); - } - - protected function getUpdateKey() { - return __CLASS__; - } - - protected function buildUpdateCondition( DatabaseBase $dbw ) { - $rcNew = $dbw->addQuotes( RC_NEW ); - $rcSrcNew = $dbw->addQuotes( RecentChange::SRC_NEW ); - $rcEdit = $dbw->addQuotes( RC_EDIT ); - $rcSrcEdit = $dbw->addQuotes( RecentChange::SRC_EDIT ); - $rcLog = $dbw->addQuotes( RC_LOG ); - $rcSrcLog = $dbw->addQuotes( RecentChange::SRC_LOG ); - $rcExternal = $dbw->addQuotes( RC_EXTERNAL ); - $rcSrcExternal = $dbw->addQuotes( RecentChange::SRC_EXTERNAL ); - - return "rc_source = CASE - WHEN rc_type = $rcNew THEN $rcSrcNew - WHEN rc_type = $rcEdit THEN $rcSrcEdit - WHEN rc_type = $rcLog THEN $rcSrcLog - WHEN rc_type = $rcExternal THEN $rcSrcExternal - ELSE '' - END"; - } -} - -$maintClass = "PopulateRecentChangesSource"; -require_once RUN_MAINTENANCE_IF_MAIN; diff --git a/maintenance/postgres/tables.sql b/maintenance/postgres/tables.sql index 5322254b98..4d44705c76 100644 --- a/maintenance/postgres/tables.sql +++ b/maintenance/postgres/tables.sql @@ -405,7 +405,6 @@ CREATE TABLE recentchanges ( rc_this_oldid INTEGER NOT NULL, rc_last_oldid INTEGER NOT NULL, rc_type SMALLINT NOT NULL DEFAULT 0, - rc_source TEXT NOT NULL, rc_patrolled SMALLINT NOT NULL DEFAULT 0, rc_ip CIDR, rc_old_len INTEGER, diff --git a/maintenance/rebuildrecentchanges.php b/maintenance/rebuildrecentchanges.php index 315176c12c..18348258b0 100644 --- a/maintenance/rebuildrecentchanges.php +++ b/maintenance/rebuildrecentchanges.php @@ -83,7 +83,6 @@ class RebuildRecentchanges extends Maintenance { 'rc_this_oldid' => 'rev_id', 'rc_last_oldid' => 0, // is this ok? 'rc_type' => $dbw->conditional( 'page_is_new != 0', RC_NEW, RC_EDIT ), - 'rc_source' => $dbw->conditional( 'page_is_new != 0', $dbw->addQuotes( RecentChange::SRC_NEW ), $dbw->addQuotes( RecentChange::SRC_EDIT ) ), 'rc_deleted' => 'rev_deleted' ), array( 'rev_timestamp > ' . $dbw->addQuotes( $dbw->timestamp( $cutoff ) ), diff --git a/maintenance/tables.sql b/maintenance/tables.sql index b3150914a3..df1bc06f84 100644 --- a/maintenance/tables.sql +++ b/maintenance/tables.sql @@ -1058,10 +1058,6 @@ CREATE TABLE /*_*/recentchanges ( -- The type of change entry (RC_EDIT,RC_NEW,RC_LOG,RC_EXTERNAL) rc_type tinyint unsigned NOT NULL default 0, - -- The source of the change entry (replaces rc_type) - -- default of '' is temporary, needed for initial migration - rc_source varchar(16) binary not null default '', - -- If the Recent Changes Patrol option is enabled, -- users may mark edits as having been reviewed to -- remove a warning flag on the RC list. -- 2.20.1