jenkins-bot has submitted this change and it was merged.

Change subject: Move LoadExtensionSchemaUpdates to hooks file
......................................................................


Move LoadExtensionSchemaUpdates to hooks file

Change-Id: I2dc1acaad6fe38d714a2a6aeeb6d9d442456921b
---
M PageTriage.hooks.php
M PageTriage.php
2 files changed, 21 insertions(+), 21 deletions(-)

Approvals:
  Mattflaschen: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/PageTriage.hooks.php b/PageTriage.hooks.php
index 28d7339..a84e86b 100755
--- a/PageTriage.hooks.php
+++ b/PageTriage.hooks.php
@@ -728,4 +728,24 @@
 
                return true;
        }
+
+       /**
+        * @param $updater DatabaseUpdater
+        * @return bool
+        */
+       public static function onLoadExtensionSchemaUpdates( $updater = null ) {
+               $base = __DIR__ . "/sql";
+               // tables
+               $updater->addExtensionTable( 'pagetriage_tags', $base . 
'/PageTriageTags.sql' );
+               $updater->addExtensionTable( 'pagetriage_page_tags', $base . 
'/PageTriagePageTags.sql' );
+               $updater->addExtensionTable( 'pagetriage_page', $base . 
'/PageTriagePage.sql' );
+               $updater->addExtensionTable( 'pagetriage_log', $base . 
'/PageTriageLog.sql' );
+               // patches
+               $updater->addExtensionIndex(
+                       'pagetriage_page',
+                       'ptrp_reviewed_updated',
+                       $base . '/PageTriagePagePatch.sql'
+               );
+               return true;
+       }
 }
diff --git a/PageTriage.php b/PageTriage.php
index b62184a..f490b87 100644
--- a/PageTriage.php
+++ b/PageTriage.php
@@ -164,7 +164,7 @@
 $wgAPIModules['pagetriagetagging'] = 'ApiPageTriageTagging';
 
 // hooks
-$wgHooks['LoadExtensionSchemaUpdates'][] = 'efPageTriageSchemaUpdates';
+$wgHooks['LoadExtensionSchemaUpdates'][] = 
'PageTriageHooks::onLoadExtensionSchemaUpdates';
 $wgHooks['SpecialMovepageAfterMove'][] = 
'PageTriageHooks::onSpecialMovepageAfterMove';
 $wgHooks['NewRevisionFromEditComplete'][] = 
'PageTriageHooks::onNewRevisionFromEditComplete';
 $wgHooks['ArticleInsertComplete'][] = 
'PageTriageHooks::onArticleInsertComplete';
@@ -190,26 +190,6 @@
 $wgLogActionsHandlers['pagetriage-curation/tag'] = 'PageTriageLogFormatter';
 $wgLogActionsHandlers['pagetriage-curation/delete'] = 'PageTriageLogFormatter';
 $wgLogActionsHandlers['pagetriage-deletion/delete'] = 'PageTriageLogFormatter';
-
-/**
- * @param $updater DatabaseUpdater
- * @return bool
- */
-function efPageTriageSchemaUpdates( $updater = null ) {
-       $base = __DIR__ . "/sql";
-       // tables
-       $updater->addExtensionTable( 'pagetriage_tags', $base . 
'/PageTriageTags.sql' );
-       $updater->addExtensionTable( 'pagetriage_page_tags', $base . 
'/PageTriagePageTags.sql' );
-       $updater->addExtensionTable( 'pagetriage_page', $base . 
'/PageTriagePage.sql' );
-       $updater->addExtensionTable( 'pagetriage_log', $base . 
'/PageTriageLog.sql' );
-       // patches
-       $updater->addExtensionIndex(
-               'pagetriage_page',
-               'ptrp_reviewed_updated',
-               $base . '/PageTriagePagePatch.sql'
-       );
-       return true;
-}
 
 /**
  * UnitTestsList hook handler - adds unit test files to the unit tester

-- 
To view, visit https://gerrit.wikimedia.org/r/311187
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2dc1acaad6fe38d714a2a6aeeb6d9d442456921b
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/PageTriage
Gerrit-Branch: master
Gerrit-Owner: Reedy <re...@wikimedia.org>
Gerrit-Reviewer: Mattflaschen <mflasc...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to