Cleanup for r54264 "* Don't suppress warnings while developing, wasting my time for...
authorBrion Vibber <brion@users.mediawiki.org>
Wed, 19 Aug 2009 18:22:31 +0000 (18:22 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Wed, 19 Aug 2009 18:22:31 +0000 (18:22 +0000)
Remove the warning suppression entirely; if you're seeing any warnings come up here it's probably because you forgot an isset() :) Please find and fix any such items.

skins/Vector.php

index 99f6d53..6461ec3 100644 (file)
@@ -366,15 +366,11 @@ class VectorTemplate extends QuickTemplate {
         * Outputs the entire contents of the XHTML page
         */
        public function execute() {
-               global $wgRequest, $wgOut, $wgContLang, $wgDevelopmentWarnings;
+               global $wgRequest, $wgOut, $wgContLang;
 
                $this->skin = $this->data['skin'];
                $action = $wgRequest->getText( 'action' );
 
-               // Suppress warnings to prevent notices about missing indexes in
-               // $this->data (is this really the best way to handle this?)
-               $wgDevelopmentWarnings && wfSuppressWarnings();
-
                // Build additional attributes for navigation urls
                $nav = $this->skin->buildNavigationUrls();
                foreach ( $nav as $section => $links ) {
@@ -578,8 +574,6 @@ class VectorTemplate extends QuickTemplate {
        </body>
 </html>
 <?php
-               // We're done with abusing arrays now...
-               $wgDevelopmentWarnings && wfRestoreWarnings();
        }
 
        /**