Revert r45063 "Move doNothing return check above global declarations"

Change makes the code harder to read by hiding a control statement in the middle of a global declaraction chunk.

diff --git a/includes/OutputPage.php b/includes/OutputPage.php
index ab97478..f8dba71 100644 (file)
--- a/includes/OutputPage.php
+++ b/includes/OutputPage.php
@@ -798,13 +798,16 @@ class OutputPage {
         * the object, let's actually output it:
         */
        public function output() {
-               if( $this->mDoNothing ) return;
                global $wgUser, $wgOutputEncoding, $wgRequest;
                global $wgContLanguageCode, $wgDebugRedirects, $wgMimeType;
                global $wgJsMimeType, $wgUseAjax, $wgAjaxWatch;
                global $wgEnableMWSuggest, $wgUniversalEditButton;
                global $wgArticle, $wgTitle;
 
+               if( $this->mDoNothing ){
+                       return;
+               }
+
                wfProfileIn( __METHOD__ );
 
                if ( '' != $this->mRedirect ) {