Yaron Koren has submitted this change and it was merged.
Change subject: New file, ReplaceText.hooks.php
......................................................................
New file, ReplaceText.hooks.php
Change-Id: I65bbe8bfed07cc406bba02ca5d34a335d937a956
---
A ReplaceText.hooks.php
M ReplaceText.php
2 files changed, 23 insertions(+), 23 deletions(-)
Approvals:
Yaron Koren: Checked; Looks good to me, approved
diff --git a/ReplaceText.hooks.php b/ReplaceText.hooks.php
new file mode 100644
index 0000000..4edf513
--- /dev/null
+++ b/ReplaceText.hooks.php
@@ -0,0 +1,21 @@
+<?php
+/**
+ */
+
+class ReplaceTextHooks {
+
+ public static function addToAdminLinks( ALTree &$adminLinksTree ) {
+ $generalSection = $adminLinksTree->getSection( wfMessage(
'adminlinks_general' )->text() );
+ $extensionsRow = $generalSection->getRow( 'extensions' );
+
+ if ( is_null( $extensionsRow ) ) {
+ $extensionsRow = new ALRow( 'extensions' );
+ $generalSection->addRow( $extensionsRow );
+ }
+
+ $extensionsRow->addItem( ALItem::newFromSpecialPage(
'ReplaceText' ) );
+
+ return true;
+ }
+
+}
diff --git a/ReplaceText.php b/ReplaceText.php
index 568e4d2..9915319 100644
--- a/ReplaceText.php
+++ b/ReplaceText.php
@@ -41,31 +41,10 @@
$wgAvailableRights[] = 'replacetext';
$wgGroupPermissions['sysop']['replacetext'] = true;
-$wgHooks['AdminLinks'][] = 'rtAddToAdminLinks';
+$wgHooks['AdminLinks'][] = 'ReplaceTextHooks::addToAdminLinks';
$wgSpecialPages['ReplaceText'] = 'SpecialReplaceText';
+$wgAutoloadClasses['ReplaceTextHooks'] = $rtgIP . 'ReplaceText.hooks.php';
$wgAutoloadClasses['SpecialReplaceText'] = $rtgIP . 'SpecialReplaceText.php';
$wgAutoloadClasses['ReplaceTextJob'] = $rtgIP . 'ReplaceTextJob.php';
$wgAutoloadClasses['ReplaceTextSearch'] = $rtgIP . 'ReplaceTextSearch.php';
-
-/**
- * This function should really go into a "ReplaceText_body.php" file.
- *
- * Handler for 'AdminLinks' hook in the AdminLinks extension
- *
- * @param $admin_links_tree ALTree
- * @return bool
- */
-function rtAddToAdminLinks( ALTree &$admin_links_tree ) {
- $general_section = $admin_links_tree->getSection( wfMessage(
'adminlinks_general' )->text() );
- $extensions_row = $general_section->getRow( 'extensions' );
-
- if ( is_null( $extensions_row ) ) {
- $extensions_row = new ALRow( 'extensions' );
- $general_section->addRow( $extensions_row );
- }
-
- $extensions_row->addItem( ALItem::newFromSpecialPage( 'ReplaceText' ) );
-
- return true;
-}
--
To view, visit https://gerrit.wikimedia.org/r/231159
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I65bbe8bfed07cc406bba02ca5d34a335d937a956
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ReplaceText
Gerrit-Branch: master
Gerrit-Owner: Yaron Koren <[email protected]>
Gerrit-Reviewer: Yaron Koren <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
