From a22f0c6f2fb579b2583a330a6964d2f00181816b Mon Sep 17 00:00:00 2001 From: =?utf8?q?Niklas=20Laxstr=C3=B6m?= Date: Thu, 8 Jul 2010 09:08:03 +0000 Subject: [PATCH] Simplify wfMsgGetKey --- includes/GlobalFunctions.php | 29 ++++++++--------------------- 1 file changed, 8 insertions(+), 21 deletions(-) diff --git a/includes/GlobalFunctions.php b/includes/GlobalFunctions.php index 314b37d3d8..5236e207ea 100644 --- a/includes/GlobalFunctions.php +++ b/includes/GlobalFunctions.php @@ -697,35 +697,22 @@ function wfMsgWeirdKey( $key ) { * behaves as a content language switch if it is a boolean. * @param $transform Boolean: whether to parse magic words, etc. * @return string - * @private */ function wfMsgGetKey( $key, $useDB, $langCode = false, $transform = true ) { global $wgContLang, $wgMessageCache; wfRunHooks('NormalizeMessageKey', array(&$key, &$useDB, &$langCode, &$transform)); - # If $wgMessageCache isn't initialised yet, try to return something sensible. - if( is_object( $wgMessageCache ) ) { - $message = $wgMessageCache->get( $key, $useDB, $langCode ); - if( $message === false ){ - $message = '<' . htmlspecialchars( $key ) . '>'; - } elseif ( $transform ) { - $message = $wgMessageCache->transform( $message ); - } - } else { - $lang = wfGetLangObj( $langCode ); - - # MessageCache::get() does this already, Language::getMessage() doesn't - # ISSUE: Should we try to handle "message/lang" here too? - $key = str_replace( ' ' , '_' , $wgContLang->lcfirst( $key ) ); - - if( is_object( $lang ) ) { - $message = $lang->getMessage( $key ); - } else { - $message = false; - } + if ( !is_object( $wgMessageCache ) ) { + throw new MWException( "Trying to get message before message cache is initialised" ); } + $message = $wgMessageCache->get( $key, $useDB, $langCode ); + if( $message === false ){ + $message = '<' . htmlspecialchars( $key ) . '>'; + } elseif ( $transform ) { + $message = $wgMessageCache->transform( $message ); + } return $message; } -- 2.20.1