r114917 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r114916‎ | r114917 | r114918 >
Date:03:42, 16 April 2012
Author:werdna
Status:deferred
Tags:
Comment:
Add SQL patches to PageTriage.php. See ee mailing list.
Modified paths:
  • /trunk/extensions/PageTriage/PageTriage.hooks.php (modified) (history)
  • /trunk/extensions/PageTriage/PageTriage.php (modified) (history)

Diff [purge]

Index: trunk/extensions/PageTriage/PageTriage.hooks.php
@@ -156,7 +156,7 @@
157157 * @param &$pcache Set if you want to use the parser cache.
158158 * @return type description
159159 */
160 - public static function onArticleViewHeader( &$article, &$outputDone, &$pcache ) {
 160+ public static function onArticleViewFooter( &$article ) {
161161 global $wgUser, $wgPageTriageMarkPatrolledLinkExpiry, $wgOut, $wgRequest;
162162
163163 // the presence of rcid means this is coming from Special:NewPages,
Index: trunk/extensions/PageTriage/PageTriage.php
@@ -97,7 +97,7 @@
9898 $wgHooks['ArticleSaveComplete'][] = 'PageTriageHooks::onArticleSaveComplete';
9999 $wgHooks['UnitTestsList'][] = 'efPageTriageUnitTests'; // unit tests
100100 $wgHooks['GetPreferences'][] = 'PageTriageHooks::onGetPreferences';
101 -$wgHooks['ArticleViewHeader'][] = 'PageTriageHooks::onArticleViewHeader';
 101+$wgHooks['ArticleViewFooter'][] = 'PageTriageHooks::onArticleViewFooter';
102102 $wgHooks['ArticleDeleteComplete'][] = 'PageTriageHooks::onArticleDeleteComplete';
103103 $wgHooks['MarkPatrolledComplete'][] = 'PageTriageHooks::onMarkPatrolledComplete';
104104 $wgHooks['BeforePageDisplay'][] = 'PageTriageHooks::beforePageDisplay';
@@ -117,6 +117,9 @@
118118 $updater->addExtensionTable( 'pagetriage_log', $base . '/PageTriageLog.sql' );
119119
120120 $updater->modifyField( 'pagetriage_page', 'ptrp_triaged', "$base/PageTriageRename.sql", true );
 121+ $updater->addExtensionField( 'pagetriage_page', 'ptrp_created', "$base/PageTriageTimestamps.patch.sql", true );
 122+ $updater->addExtensionField( 'pagetriage_page', 'ptrp_tags_updated', "$base/PageTriageTimestamps.patch.sql", true );
 123+ $updater->addExtensionField( 'pagetriage_page', 'ptrp_deleted', "$base/PageTriagePageDeleted.sql", true );
121124
122125 return true;
123126 }

Follow-up revisions

RevisionCommit summaryAuthorDate
r114918Fix up accidentally-included other change in r114917werdna03:45, 16 April 2012

Status & tagging log