Followup r87340: Post-hook swap bodytext into a new bodycontent key and append printf...

[lhc/web/wiklou.git] / includes / SpecialPageFactory.php

diff --git a/includes/SpecialPageFactory.php b/includes/SpecialPageFactory.php
index 94d40e1..ba9a918 100644 (file)
--- a/includes/SpecialPageFactory.php
+++ b/includes/SpecialPageFactory.php
@@ -126,6 +126,8 @@ class SpecialPageFactory {
                // Page tools
                'ComparePages'              => 'SpecialComparePages',
                'Export'                    => 'SpecialExport',
+               'Globalfileusage'           => 'SpecialGlobalFileUsage',
+               'Globaltemplateusage'       => 'SpecialGlobalTemplateUsage',
                'Import'                    => 'SpecialImport',
                'Undelete'                  => 'SpecialUndelete',
                'Whatlinkshere'             => 'SpecialWhatlinkshere',
@@ -262,12 +264,12 @@ class SpecialPageFactory {
        /**
         * Add a page to a certain display group for Special:SpecialPages
         *
-        * @param $page Mixed: SpecialPage or string
+        * @param $page SpecialPage|string
         * @param $group String
         */
        public static function setGroup( $page, $group ) {
                global $wgSpecialPageGroups;
-               $name = is_object( $page ) ? $page->mName : $page;
+               $name = is_object( $page ) ? $page->getName() : $page;
                $wgSpecialPageGroups[$name] = $group;
        }
 
@@ -275,25 +277,29 @@ class SpecialPageFactory {
         * Get the group that the special page belongs in on Special:SpecialPage
         *
         * @param $page SpecialPage
+        *
+        * @return string
         */
        public static function getGroup( &$page ) {
+               $name = $page->getName();
+
                global $wgSpecialPageGroups;
                static $specialPageGroupsCache = array();
-               if ( isset( $specialPageGroupsCache[$page->mName] ) ) {
-                       return $specialPageGroupsCache[$page->mName];
+               if ( isset( $specialPageGroupsCache[$name] ) ) {
+                       return $specialPageGroupsCache[$name];
                }
-               $msg = wfMessage( 'specialpages-specialpagegroup-' . strtolower( $page->mName ) );
+               $msg = wfMessage( 'specialpages-specialpagegroup-' . strtolower( $name ) );
                if ( !$msg->isBlank() ) {
                        $group = $msg->text();
                } else {
-                       $group = isset( $wgSpecialPageGroups[$page->mName] )
-                               ? $wgSpecialPageGroups[$page->mName]
+                       $group = isset( $wgSpecialPageGroups[$name] )
+                               ? $wgSpecialPageGroups[$name]
                                : '-';
                }
                if ( $group == '-' ) {
                        $group = 'other';
                }
-               $specialPageGroupsCache[$page->mName] = $group;
+               $specialPageGroupsCache[$name] = $group;
                return $group;
        }
 
@@ -469,40 +475,47 @@ class SpecialPageFactory {
        }
 
        /**
-        * Just like executePath() except it returns the HTML instead of outputting it
-        * Returns false if there was no such special page, or a title object if it was
+        * Just like executePath() but will override global variables and execute
+        * the page in "inclusion" mode. Returns true if the excution was successful
+        * or false if there was no such special page, or a title object if it was
         * a redirect.
         *
-        * Also saves the current $wgTitle, $wgOut, and $wgRequest variables so that
-        * the special page will get the context it'd expect on a normal request,
-        * and then restores them to their previous values after.
+        * Also saves the current $wgTitle, $wgOut, $wgRequest, $wgUser and $wgLang
+        * variables so that the special page will get the context it'd expect on a
+        * normal request, and then restores them to their previous values after.
         *
         * @param $title Title
+        * @param $context RequestContext
         *
         * @return String: HTML fragment
         */
-       static function capturePath( &$title ) {
-               global $wgOut, $wgTitle, $wgRequest;
+       static function capturePath( Title $title, RequestContext $context ) {
+               global $wgOut, $wgTitle, $wgRequest, $wgUser, $wgLang;
 
+               // Save current globals
                $oldTitle = $wgTitle;
                $oldOut = $wgOut;
                $oldRequest = $wgRequest;
+               $oldUser = $wgUser;
+               $oldLang = $wgLang;
 
-               // Don't want special pages interpreting ?feed=atom parameters.
-               $wgRequest = new FauxRequest( array() );
-
-               $context = new RequestContext;
-               $context->setTitle( $title );
-               $context->setRequest( $wgRequest );
+               // Set the globals to the current context
+               $wgTitle = $title;
                $wgOut = $context->getOutput();
+               $wgRequest = $context->getRequest();
+               $wgUser = $context->getUser();
+               $wgLang = $context->getLang();
 
+               // The useful part
                $ret = self::executePath( $title, $context, true );
-               if ( $ret === true ) {
-                       $ret = $wgOut->getHTML();
-               }
+
+               // And restore that globals
                $wgTitle = $oldTitle;
                $wgOut = $oldOut;
                $wgRequest = $oldRequest;
+               $wgUser = $oldUser;
+               $wgLang = $oldLang;
+
                return $ret;
        }