Follow-up r76434: Run hook for Modern skin too per suggestion of Jack Phoenix

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 5084728..6bbc2ba 100644 (file)
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -201,7 +201,8 @@ LocalSettings.php. The specific bugs are listed below in the general notes.
 * Special:Search: Add CSS classes to 'none found' and 'create link' messages
 * Add CSS classes (including namespace and pagename) to the enhanced recent
   changes/watchlist entries
-* (bug 22463) Add hook 'SkinGetPoweredBy' to make 'powered by' icon customizable
+* (bug 22463) Add hook 'SkinGetPoweredBy' to make 'powered by' icon/text
+  customizable
 * Added CSS print pagination to the print stylesheets
 
 === Bug fixes in 1.17 ===

diff --git a/docs/hooks.txt b/docs/hooks.txt
index 0bcf00d..d97718a 100644 (file)
--- a/docs/hooks.txt
+++ b/docs/hooks.txt
@@ -1462,7 +1462,8 @@ $type: 'normal' or 'history' for old/diff views
 &$forContent: overridable flag if copyright footer is shown in content language. 
 
 'SkinGetPoweredBy'
-&$text: additional 'powered by' icons in HTML
+&$text: additional 'powered by' icons in HTML.
+Note: Modern skin does not use the MediaWiki icon but plain text instead
 
 'SkinSubPageSubtitle': At the beginning of Skin::subPageSubtitle()
 &$subpages: Subpage links HTML

diff --git a/skins/Modern.php b/skins/Modern.php
index 899c53e..f3605f7 100644 (file)
--- a/skins/Modern.php
+++ b/skins/Modern.php
@@ -27,7 +27,9 @@ class SkinModern extends SkinTemplate {
         */
        function getPoweredBy() {
                global  $wgVersion;
-               return "<div class='mw_poweredby'>Powered by MediaWiki $wgVersion</div>";
+               $text = "<div class='mw_poweredby'>Powered by MediaWiki $wgVersion</div>";
+               wfRunHooks( 'SkinGetPoweredBy', array( &$text ) );
+               return $text;
        }
 
        function setupSkinUserCss( OutputPage $out ){