followup r88898 -- move BeforeInitialize hook to where it can be
authorMark A. Hershberger <mah@users.mediawiki.org>
Sun, 18 Dec 2011 01:46:25 +0000 (01:46 +0000)
committerMark A. Hershberger <mah@users.mediawiki.org>
Sun, 18 Dec 2011 01:46:25 +0000 (01:46 +0000)
called in a way that matches the documented parameters

As it was, when an extension tried to use BeforeInitialize with the
documented parameters this resulted in an exception:

    Detected bug in an extension! Hook HookFunctionName has invalid
    call signature; Parameter 2 to HookFunctionName() expected to be a
    reference, value given

includes/Wiki.php

index 996302e..8b62139 100644 (file)
@@ -146,9 +146,6 @@ class MediaWiki {
                        $output->setPrintable();
                }
 
-               wfRunHooks( 'BeforeInitialize',
-                       array( &$title, null, &$output, &$user, $request, $this ) );
-
                // Invalid titles. Bug 21776: The interwikis must redirect even if the page name is empty.
                if ( is_null( $title ) || ( $title->getDBkey() == '' && $title->getInterwiki() == '' ) ||
                        $title->isSpecial( 'Badtitle' ) )
@@ -270,6 +267,8 @@ class MediaWiki {
                                wfProfileOut( __METHOD__ );
                                throw new MWException( "Shouldn't happen: MediaWiki::initializeArticle() returned neither an object nor a URL" );
                        }
+                       wfRunHooks( 'BeforeInitialize',
+                               array( &$title, &$article, &$output, &$user, $request, $this ) );
                }
 
                if ( $pageView ) {