From c06afd56b3d995bb579fbbdb0e26aa98a46908b5 Mon Sep 17 00:00:00 2001 From: Andrew Garrett Date: Sun, 30 Nov 2008 03:15:22 +0000 Subject: [PATCH] Revert "Follow up on r43982. Reduce dirname(__FILE__) calls in core and extensions." Uses $dir in extension files, and assumes that it remains unchanged in require_once( 'maintenance/commandLine.inc' ). In fact, it is likely that '$dir' will be set when setting up command-line, as some extensions will use the same var. Recommended fix: Use $CentralAuth_dir, $EmailPage_dir, etc. --- img_auth.php | 5 ++--- includes/GlobalFunctions.php | 7 +++---- languages/classes/LanguageKk.deps.php | 5 ++--- languages/classes/LanguageKk.php | 5 ++--- languages/classes/LanguageKu.deps.php | 5 ++--- languages/classes/LanguageSr.deps.php | 5 ++--- languages/classes/LanguageSr.php | 7 +++---- languages/classes/LanguageZh.deps.php | 5 ++--- languages/classes/LanguageZh.php | 5 ++--- 9 files changed, 20 insertions(+), 29 deletions(-) diff --git a/img_auth.php b/img_auth.php index f1214fed36..4b625e3972 100644 --- a/img_auth.php +++ b/img_auth.php @@ -13,10 +13,9 @@ */ define( 'MW_NO_OUTPUT_COMPRESSION', 1 ); -$dir = dirname(__FILE__) . '/'; -require_once( $dir . 'includes/WebStart.php' ); +require_once( dirname( __FILE__ ) . '/includes/WebStart.php' ); wfProfileIn( 'img_auth.php' ); -require_once( $dir . 'includes/StreamFile.php' ); +require_once( dirname( __FILE__ ) . '/includes/StreamFile.php' ); $perms = User::getGroupPermissions( array( '*' ) ); if ( in_array( 'read', $perms, true ) ) { diff --git a/includes/GlobalFunctions.php b/includes/GlobalFunctions.php index fa2cc577b3..44304409dc 100644 --- a/includes/GlobalFunctions.php +++ b/includes/GlobalFunctions.php @@ -8,10 +8,9 @@ if ( !defined( 'MEDIAWIKI' ) ) { * Global functions used everywhere */ -$dir = dirname(__FILE__) . '/'; -require_once $dir . 'LogPage.php'; -require_once $dir . 'normal/UtfNormalUtil.php'; -require_once $dir . 'XmlFunctions.php'; +require_once dirname(__FILE__) . '/LogPage.php'; +require_once dirname(__FILE__) . '/normal/UtfNormalUtil.php'; +require_once dirname(__FILE__) . '/XmlFunctions.php'; // Hide compatibility functions from Doxygen /// @cond diff --git a/languages/classes/LanguageKk.deps.php b/languages/classes/LanguageKk.deps.php index 9a76de0453..aa7361bbc0 100644 --- a/languages/classes/LanguageKk.deps.php +++ b/languages/classes/LanguageKk.deps.php @@ -5,6 +5,5 @@ // changed on a subsequent page view. // see http://lists.wikimedia.org/pipermail/wikitech-l/2006-January/021311.html -$dir = dirname(__FILE__) . '/'; -require_once( $dir . '../LanguageConverter.php' ); -require_once( $dir . 'LanguageKk_cyrl.php' ); +require_once( dirname(__FILE__).'/../LanguageConverter.php' ); +require_once( dirname(__FILE__).'/LanguageKk_cyrl.php' ); diff --git a/languages/classes/LanguageKk.php b/languages/classes/LanguageKk.php index c862c3b456..40beca67e5 100644 --- a/languages/classes/LanguageKk.php +++ b/languages/classes/LanguageKk.php @@ -1,8 +1,7 @@