From dead5afa5be805a4fa08482e5a7abfd7ed175d7c Mon Sep 17 00:00:00 2001 From: "Mark A. Hershberger" Date: Tue, 5 Jan 2010 09:07:12 +0000 Subject: [PATCH] Coding Convention cleanup, removed extraneous references to $_SERVER --- languages/LanguageConverter.php | 976 ++++++++++++++++++-------------- 1 file changed, 539 insertions(+), 437 deletions(-) diff --git a/languages/LanguageConverter.php b/languages/LanguageConverter.php index b54fcbc4a8..28cfea93cc 100644 --- a/languages/LanguageConverter.php +++ b/languages/LanguageConverter.php @@ -9,27 +9,28 @@ */ /** - * base class for language convert + * Base class for language conversion. * @ingroup Language * * @author Zhengzhu Feng * @maintainers fdcn , shinjiman , PhiLiP */ class LanguageConverter { - var $mPreferredVariant=''; + var $mPreferredVariant = ''; var $mMainLanguageCode; var $mVariants, $mVariantFallbacks, $mVariantNames; var $mTablesLoaded = false; var $mTables; var $mNamespaceTables; - var $mDoTitleConvert=true, $mDoContentConvert=true; - var $mManualLevel; // 'bidirectional' 'unidirectional' 'disable' for each variants + var $mDoTitleConvert = true, $mDoContentConvert = true; + // 'bidirectional' 'unidirectional' 'disable' for each variant + var $mManualLevel; var $mTitleFromFlag = false; var $mCacheKey; var $mLangObj; var $mMarkup; var $mFlags; - var $mDescCodeSep = ':',$mDescVarSep = ';'; + var $mDescCodeSep = ':', $mDescVarSep = ';'; var $mUcfirst = false; var $mTitleOriginal = ''; var $mTitleDisplay = ''; @@ -39,18 +40,20 @@ class LanguageConverter { /** * Constructor * + * @param $langobj The Language Object * @param string $maincode the main language code of this language * @param array $variants the supported variants of this language * @param array $variantfallback the fallback language of each variant * @param array $markup array defining the markup used for manual conversion - * @param array $flags array defining the custom strings that maps to the flags + * @param array $flags array defining the custom strings that maps to the + * flags * @param array $manualLevel limit for supported variants * @public */ function __construct( $langobj, $maincode, - $variants=array(), - $variantfallbacks=array(), - $markup=array(), + $variants = array(), + $variantfallbacks = array(), + $markup = array(), $flags = array(), $manualLevel = array() ) { $this->mLangObj = $langobj; @@ -58,41 +61,45 @@ class LanguageConverter { global $wgDisabledVariants; $this->mVariants = array(); - foreach( $variants as $variant ) { - if( !in_array( $variant, $wgDisabledVariants ) ) + foreach ( $variants as $variant ) { + if ( !in_array( $variant, $wgDisabledVariants ) ) { $this->mVariants[] = $variant; + } } $this->mVariantFallbacks = $variantfallbacks; global $wgLanguageNames; $this->mVariantNames = $wgLanguageNames; $this->mCacheKey = wfMemcKey( 'conversiontables', $maincode ); $m = array( - 'begin'=>'-{', - 'flagsep'=>'|', - 'unidsep'=>'=>', //for unidirectional conversion - 'codesep'=>':', - 'varsep'=>';', - 'end'=>'}-' + 'begin' => '-{', + 'flagsep' => '|', + 'unidsep' => '=>', // for unidirectional conversion + 'codesep' => ':', + 'varsep' => ';', + 'end' => '}-' ); - $this->mMarkup = array_merge($m, $markup); - $f = array( + $this->mMarkup = array_merge( $m, $markup ); + $f = array( // 'S' show converted text // '+' add rules for alltext // 'E' the gave flags is error // these flags above are reserved for program - 'A'=>'A', // add rule for convert code (all text convert) - 'T'=>'T', // title convert - 'R'=>'R', // raw content - 'D'=>'D', // convert description (subclass implement) - '-'=>'-', // remove convert (not implement) - 'H'=>'H', // add rule for convert code (but no display in placed code ) - 'N'=>'N' // current variant name + 'A' => 'A', // add rule for convert code (all text convert) + 'T' => 'T', // title convert + 'R' => 'R', // raw content + 'D' => 'D', // convert description (subclass implement) + '-' => '-', // remove convert (not implement) + 'H' => 'H', // add rule for convert code + // (but no display in placed code ) + 'N' => 'N' // current variant name ); - $this->mFlags = array_merge($f, $flags); - foreach( $this->mVariants as $v) { - $this->mManualLevel[$v]=array_key_exists($v,$manualLevel) - ?$manualLevel[$v] - :'bidirectional'; + $this->mFlags = array_merge( $f, $flags ); + foreach ( $this->mVariants as $v ) { + if ( array_key_exists( $v, $manualLevel ) ) { + $this->mManualLevel[$v] = $manualLevel[$v]; + } else { + $this->mManualLevel[$v] = 'bidirectional'; + } $this->mNamespaceTables[$v] = array(); $this->mFlags[$v] = $v; } @@ -106,127 +113,128 @@ class LanguageConverter { } /** - * in case some variant is not defined in the markup, we need - * to have some fallback. for example, in zh, normally people + * In case some variant is not defined in the markup, we need + * to have some fallback. For example, in zh, normally people * will define zh-hans and zh-hant, but less so for zh-sg or zh-hk. * when zh-sg is preferred but not defined, we will pick zh-hans - * in this case. right now this is only used by zh. + * in this case. Right now this is only used by zh. * - * @param string $v the language code of the variant - * @return string array the code of the fallback language or false if there is no fallback + * @param string $v The language code of the variant + * @return string array The code of the fallback language or false if there + * is no fallback * @public */ - function getVariantFallbacks($v) { - if( isset( $this->mVariantFallbacks[$v] ) ) { + function getVariantFallbacks( $v ) { + if ( isset( $this->mVariantFallbacks[$v] ) ) { return $this->mVariantFallbacks[$v]; } return $this->mMainLanguageCode; } /** - * get preferred language variants. + * Get preferred language variants. * @param boolean $fromUser Get it from $wgUser's preferences * @param boolean $fromHeader Get it from Accept-Language * @return string the preferred language code * @public */ function getPreferredVariant( $fromUser = true, $fromHeader = false ) { - global $wgUser, $wgRequest, $wgVariantArticlePath, $wgDefaultLanguageVariant, $wgOut; + global $wgUser, $wgRequest, $wgVariantArticlePath, + $wgDefaultLanguageVariant, $wgOut; // bug 21974, don't return $this->mPreferredVariant if $fromUser = false - if( $fromUser && $this->mPreferredVariant ) + if ( $fromUser && $this->mPreferredVariant ) { return $this->mPreferredVariant; + } - // figure out user lang without constructing wgLang to avoid infinite recursion - if( $fromUser ) + // figure out user lang without constructing wgLang to avoid + // infinite recursion + if ( $fromUser ) { $defaultUserLang = $wgUser->getOption( 'language' ); - else + } else { $defaultUserLang = $this->mMainLanguageCode; + } + $userLang = $wgRequest->getVal( 'uselang', $defaultUserLang ); - // see if interface language is same as content, if not, prevent conversion - if( ! in_array( $userLang, $this->mVariants ) ){ - $this->mPreferredVariant = $this->mMainLanguageCode; // no conversion + // see if interface language is same as content, if not, prevent + // conversion + + if ( ! in_array( $userLang, $this->mVariants ) ) { + // no conversion + $this->mPreferredVariant = $this->mMainLanguageCode; return $this->mPreferredVariant; } // see if the preference is set in the request $req = $wgRequest->getText( 'variant' ); - if( in_array( $req, $this->mVariants ) ) { + if ( in_array( $req, $this->mVariants ) ) { $this->mPreferredVariant = $req; - return $req; - } - - // check the syntax /code/ArticleTitle - if($wgVariantArticlePath!=false && isset($_SERVER['SCRIPT_NAME'])){ - // Note: SCRIPT_NAME probably won't hold the correct value if PHP is run as CGI - // (it will hold path to php.cgi binary), and might not exist on some very old PHP installations - $scriptBase = basename( $_SERVER['SCRIPT_NAME'] ); - if(in_array($scriptBase,$this->mVariants)){ - $this->mPreferredVariant = $scriptBase; - return $this->mPreferredVariant; - } + return $this->mPreferredVariant; } // get language variant preference from logged in users // Don't call this on stub objects because that causes infinite // recursion during initialisation - if( $fromUser && $wgUser->isLoggedIn() ) { - $this->mPreferredVariant = $wgUser->getOption('variant'); + if ( $fromUser && $wgUser->isLoggedIn() ) { + $this->mPreferredVariant = $wgUser->getOption( 'variant' ); return $this->mPreferredVariant; } // see if default variant is globaly set - if($wgDefaultLanguageVariant != false && in_array( $wgDefaultLanguageVariant, $this->mVariants )){ + if ( $wgDefaultLanguageVariant != false + && in_array( $wgDefaultLanguageVariant, $this->mVariants ) ) { $this->mPreferredVariant = $wgDefaultLanguageVariant; return $this->mPreferredVariant; } - if( !$this->mPreferredVariant ) { + if ( !$this->mPreferredVariant ) { // see if some supported language variant is set in the // http header, but we don't set the mPreferredVariant // variable in case this is called before the user's // preference is loaded - if( $fromHeader && array_key_exists( 'HTTP_ACCEPT_LANGUAGE', $_SERVER ) ) { - $acceptLanguage = strtolower( $_SERVER['HTTP_ACCEPT_LANGUAGE'] ); + + $acceptLanguage = $wgRequest->getHeader( 'Accept-Language' ); + if ( $fromHeader && $acceptLanguage ) { // explode by comma - $result = explode(',', $acceptLanguage); + $result = explode( ',', strtolower( $acceptLanguage ) ); $languages = array(); - foreach( $result as $elem ) { + foreach ( $result as $elem ) { // if $elem likes 'zh-cn;q=0.9' - if(($posi = strpos( $elem, ';' )) !== false ) { + if ( ( $posi = strpos( $elem, ';' ) ) !== false ) { // get the real language code likes 'zh-cn' $languages[] = substr( $elem, 0, $posi ); - } - else { + } else { $languages[] = $elem; } } $fallback_languages = array(); - foreach( $languages as $language ) { + foreach ( $languages as $language ) { // strip whitespace $language = trim( $language ); - if( in_array( $language, $this->mVariants ) ) { + if ( in_array( $language, $this->mVariants ) ) { return $language; - } - else { + } else { // To see if there are fallbacks of current language. // We record these fallback variants, and process // them later. $fallbacks = $this->getVariantFallbacks( $language ); - if( is_string( $fallbacks ) ) + if ( is_string( $fallbacks ) ) { $fallback_languages[] = $fallbacks; - elseif( is_array( $fallbacks ) ) - $fallback_languages = array_merge( $fallback_languages, $fallbacks ); + } elseif ( is_array( $fallbacks ) ) { + $fallback_languages = + array_merge( $fallback_languages, + $fallbacks ); + } } } // process fallback languages now $fallback_languages = array_unique( $fallback_languages ); - foreach( $fallback_languages as $language ) { - if( in_array( $language, $this->mVariants ) ) { + foreach ( $fallback_languages as $language ) { + if ( in_array( $language, $this->mVariants ) ) { return $language; } } @@ -236,9 +244,9 @@ class LanguageConverter { } /** - * caption convert, base on preg_replace_callback + * Caption convert, base on preg_replace_callback. * - * to convert text in "title" or "alt", like 'texttranslate($text, $toVariant); + if ( !strpos( $text, '://' ) ) { + $text = $this->translate( $text, $toVariant ); + } return " $title=\"$text\""; } /** - * dictionary-based conversion + * Dictionary-based conversion. * * @param string $text the text to be converted * @param string $toVariant the target language code * @return string the converted text * @private */ - function autoConvert($text, $toVariant=false) { - $fname="LanguageConverter::autoConvert"; + function autoConvert( $text, $toVariant = false ) { + $fname = 'LanguageConverter::autoConvert'; wfProfileIn( $fname ); - if(!$this->mTablesLoaded) + if ( !$this->mTablesLoaded ) { $this->loadTables(); + } - if(!$toVariant) + if ( !$toVariant ) { $toVariant = $this->getPreferredVariant(); - if(!in_array($toVariant, $this->mVariants)) + } + if ( !in_array( $toVariant, $this->mVariants ) ) { return $text; + } /* we convert everything except: 1. html markups (anything between < and >) @@ -281,10 +293,11 @@ class LanguageConverter { 3. place holders created by the parser */ global $wgParser; - if (isset($wgParser) && $wgParser->UniqPrefix()!=''){ + if ( isset( $wgParser ) && $wgParser->UniqPrefix() != '' ) { $marker = '|' . $wgParser->UniqPrefix() . '[\-a-zA-Z0-9]+'; - } else - $marker = ""; + } else { + $marker = ''; + } // this one is needed when the text is inside an html markup $htmlfix = '|<[^>]+$|^[^<>]*>'; @@ -296,35 +309,39 @@ class LanguageConverter { // disable conversion of 
 ...
$prefix = '.*?<\/pre>|'; - $reg = '/'.$codefix . $scriptfix . $prefix . '<[^>]+>|&[a-zA-Z#][a-z0-9]+;' . $marker . $htmlfix . '/s'; + $reg = '/' . $codefix . $scriptfix . $prefix . + '<[^>]+>|&[a-zA-Z#][a-z0-9]+;' . $marker . $htmlfix . '/s'; - $matches = preg_split($reg, $text, -1, PREG_SPLIT_OFFSET_CAPTURE); + $matches = preg_split( $reg, $text, - 1, PREG_SPLIT_OFFSET_CAPTURE ); - $m = array_shift($matches); + $m = array_shift( $matches ); - $ret = $this->translate($m[0], $toVariant); - $mstart = $m[1]+strlen($m[0]); + $ret = $this->translate( $m[0], $toVariant ); + $mstart = $m[1] + strlen( $m[0] ); - // enable convertsion of 'xxxxtranslate( $trtext, $toVariant ); $trtext = StringUtils::explode( $trtextmark, $trtext ); - foreach( $trtext as $t ) { - $ret .= array_shift($notrtext); + foreach ( $trtext as $t ) { + $ret .= array_shift( $notrtext ); $ret .= $t; } wfProfileOut( $fname ); @@ -332,8 +349,9 @@ class LanguageConverter { } /** - * Translate a string to a variant - * Doesn't process markup or do any of that other stuff, for that use convert() + * Translate a string to a variant. + * Doesn't process markup or do any of that other stuff, for that use + * convert(). * * @param string $text Text to convert * @param string $variant Variant language code @@ -344,31 +362,33 @@ class LanguageConverter { wfProfileIn( __METHOD__ ); // If $text is empty or only includes spaces, do nothing // Otherwise translate it - if( trim($text) ) { - if( !$this->mTablesLoaded ) + if ( trim( $text ) ) { + if ( !$this->mTablesLoaded ) { $this->loadTables(); - $text = $this->mTables[$variant]->replace( $text ); + } + $text = $this->mTables[$variant]->replace( $text ); } wfProfileOut( __METHOD__ ); return $text; } /** - * convert text to all supported variants + * Convert text to all supported variants. * * @param string $text the text to be converted * @return array of string * @public */ - function autoConvertToAllVariants($text) { - $fname="LanguageConverter::autoConvertToAllVariants"; + function autoConvertToAllVariants( $text ) { + $fname = 'LanguageConverter::autoConvertToAllVariants'; wfProfileIn( $fname ); - if( !$this->mTablesLoaded ) + if ( !$this->mTablesLoaded ) { $this->loadTables(); + } $ret = array(); - foreach($this->mVariants as $variant) { - $ret[$variant] = $this->translate($text, $variant); + foreach ( $this->mVariants as $variant ) { + $ret[$variant] = $this->translate( $text, $variant ); } wfProfileOut( $fname ); @@ -376,30 +396,34 @@ class LanguageConverter { } /** - * convert link text to all supported variants + * Convert link text to all supported variants. * * @param string $text the text to be converted * @return array of string * @public */ - function convertLinkToAllVariants($text) { - if( !$this->mTablesLoaded ) + function convertLinkToAllVariants( $text ) { + if ( !$this->mTablesLoaded ) { $this->loadTables(); + } $ret = array(); - $tarray = explode($this->mMarkup['begin'], $text); - $tfirst = array_shift($tarray); + $tarray = explode( $this->mMarkup['begin'], $text ); + $tfirst = array_shift( $tarray ); - foreach($this->mVariants as $variant) - $ret[$variant] = $this->translate($tfirst,$variant); + foreach ( $this->mVariants as $variant ) { + $ret[$variant] = $this->translate( $tfirst, $variant ); + } - foreach($tarray as $txt) { - $marked = explode($this->mMarkup['end'], $txt, 2); + foreach ( $tarray as $txt ) { + $marked = explode( $this->mMarkup['end'], $txt, 2 ); - foreach($this->mVariants as $variant){ - $ret[$variant] .= $this->mMarkup['begin'].$marked[0].$this->mMarkup['end']; - if(array_key_exists(1, $marked)) - $ret[$variant] .= $this->translate($marked[1],$variant); + foreach ( $this->mVariants as $variant ) { + $ret[$variant] .= $this->mMarkup['begin'] . $marked[0] . + $this->mMarkup['end']; + if ( array_key_exists( 1, $marked ) ) { + $ret[$variant] .= $this->translate( $marked[1], $variant ); + } } } @@ -408,52 +432,57 @@ class LanguageConverter { } /** - * prepare manual conversion table + * Prepare manual conversion table. * @private */ - function applyManualConv( $convRule ){ - // use syntax -{T|zh:TitleZh;zh-tw:TitleTw}- for custom conversion in title + function applyManualConv( $convRule ) { + // use syntax -{T|zh:TitleZh;zh-tw:TitleTw}- for custom + // conversion in title $title = $convRule->getTitle(); - if( $title ){ + if ( $title ) { $this->mTitleFromFlag = true; - $this->mTitleDisplay = $title; + $this->mTitleDisplay = $title; } - //apply manual conversion table to global table + // apply manual conversion table to global table $convTable = $convRule->getConvTable(); $action = $convRule->getRulesAction(); - foreach( $convTable as $variant => $pair ) { - if( !in_array( $variant, $this->mVariants ) )continue; - if( $action == 'add' ) { - foreach( $pair as $from => $to ) { + foreach ( $convTable as $variant => $pair ) { + if ( !in_array( $variant, $this->mVariants ) ) { + continue; + } + + if ( $action == 'add' ) { + foreach ( $pair as $from => $to ) { // to ensure that $from and $to not be left blank // so $this->translate() could always return a string - if ( $from || $to ) + if ( $from || $to ) { // more efficient than array_merge(), about 2.5 times. $this->mTables[$variant]->setPair( $from, $to ); + } } - } - elseif ( $action == 'remove' ) { + } elseif ( $action == 'remove' ) { $this->mTables[$variant]->removeArray( $pair ); } } } /** - * Convert text using a parser object for context + * Convert text using a parser object for context. * @public */ function parserConvert( $text, &$parser ) { global $wgDisableLangConversion; /* don't do anything if this is the conversion table */ - if ( $parser->getTitle()->getNamespace() == NS_MEDIAWIKI && - strpos($parser->mTitle->getText(), "Conversiontable") !== false ) - { + if ( $parser->getTitle()->getNamespace() == NS_MEDIAWIKI + && strpos( $parser->mTitle->getText(), + 'Conversiontable' ) !== false ) { return $text; } - if ( $wgDisableLangConversion ) + if ( $wgDisableLangConversion ) { return $text; + } $text = $this->convert( $text ); @@ -464,18 +493,20 @@ class LanguageConverter { } /** - * convert namespace + * Convert namespace. * @param string $title the title included namespace * @return array of string * @private */ function convertNamespace( $title, $variant ) { $splittitle = explode( ':', $title ); - if (count($splittitle) < 2) + if ( count( $splittitle ) < 2 ) { return $title; - if ( isset( $this->mNamespaceTables[$variant][$splittitle[0]] ) ) + } + if ( isset( $this->mNamespaceTables[$variant][$splittitle[0]] ) ) { $splittitle[0] = $this->mNamespaceTables[$variant][$splittitle[0]]; - $ret = implode(':', $splittitle ); + } + $ret = implode( ':', $splittitle ); return $ret; } @@ -484,7 +515,7 @@ class LanguageConverter { * store the default converted title to $this->mTitleDisplay. * @private */ - function preConvertTitle( $text, $variant ){ + function preConvertTitle( $text, $variant ) { $this->mTitleOriginal = $text; $text = $this->convertNamespace( $text, $variant ); @@ -492,34 +523,34 @@ class LanguageConverter { } /** - * convert title + * Convert title. * @private */ - function convertTitle(){ + function convertTitle() { global $wgDisableTitleConversion, $wgUser, $wgRequest; $isredir = $wgRequest->getText( 'redirect', 'yes' ); $action = $wgRequest->getText( 'action' ); $linkconvert = $wgRequest->getText( 'linkconvert', 'yes' ); // check for the global variable, __NOTC__ magic word, and user setting - if( $wgDisableTitleConversion || !$this->mDoTitleConvert || - $wgUser->getOption('noconvertlink') == 1 ) { + if ( $wgDisableTitleConversion || !$this->mDoTitleConvert || + $wgUser->getOption( 'noconvertlink' ) == 1 ) { $this->mTitleDisplay = $this->mTitleOriginal; - } - - // check for GET params - elseif ( $isredir == 'no' || $action == 'edit' || $linkconvert == 'no' ) { + } elseif ( $isredir == 'no' + || $action == 'edit' + || $linkconvert == 'no' ) { + // check for GET params $this->mTitleDisplay = $this->mTitleOriginal; } } /** - * convert text to different variants of a language. the automatic - * conversion is done in autoConvert(). here we parse the text + * Convert text to different variants of a language. The automatic + * conversion is done in autoConvert(). Here we parse the text * marked with -{}-, which specifies special conversions of the - * text that can not be accomplished in autoConvert() + * text that can not be accomplished in autoConvert(). * - * syntax of the markup: + * Syntax of the markup: * -{code1:text1;code2:text2;...}- or * -{flags|code1:text1;code2:text2;...}- or * -{text}- in which case no conversion should take place for text @@ -532,17 +563,19 @@ class LanguageConverter { function convert( $text, $isTitle = false ) { $mw =& MagicWord::get( 'notitleconvert' ); - if( $mw->matchAndRemove( $text ) ) + if ( $mw->matchAndRemove( $text ) ) { $this->mDoTitleConvert = false; + } $mw =& MagicWord::get( 'nocontentconvert' ); - if( $mw->matchAndRemove( $text ) ) { + if ( $mw->matchAndRemove( $text ) ) { $this->mDoContentConvert = false; } // no conversion if redirecting $mw =& MagicWord::get( 'redirect' ); - if( $mw->matchStart( $text ) ) + if ( $mw->matchStart( $text ) ) { return $text; + } $plang = $this->getPreferredVariant(); @@ -559,75 +592,87 @@ class LanguageConverter { $marked = explode( $this->mMarkup['begin'], $txt, 2 ); - if( $this->mDoContentConvert ) - // Bug 19620: should convert a string immediately after a new rule added. + if ( $this->mDoContentConvert ) { + // Bug 19620: should convert a string immediately after a + // new rule added. $text .= $this->autoConvert( $marked[0], $plang ); - else + } else { $text .= $marked[0]; + } if ( array_key_exists( 1, $marked ) ) { - $crule = new ConverterRule($marked[1], $this); + $crule = new ConverterRule( $marked[1], $this ); $crule->parse( $plang ); $text .= $crule->getDisplay(); $this->applyManualConv( $crule ); - } - else + } else { $text .= $this->mMarkup['end']; - + } } // Remove the last delimiter (wasn't real) - $text = substr( $text, 0, -strlen( $this->mMarkup['end'] ) ); + $text = substr( $text, 0, - strlen( $this->mMarkup['end'] ) ); return $text; } /** - * if a language supports multiple variants, it is + * If a language supports multiple variants, it is * possible that non-existing link in one variant - * actually exists in another variant. this function + * actually exists in another variant. This function * tries to find it. See e.g. LanguageZh.php * * @param string $link the name of the link * @param mixed $nt the title object of the link * @param boolean $ignoreOtherCond: to disable other conditions when - * we need to transclude a template or update a category's link + * we need to transclude a template or update a category's link * @return null the input parameters may be modified upon return * @public */ function findVariantLink( &$link, &$nt, $ignoreOtherCond = false ) { # If the article has already existed, there is no need to # check it again, otherwise it may cause a fault. - if ( is_object( $nt ) && $nt->exists() ) + if ( is_object( $nt ) && $nt->exists() ) { return; + } - global $wgDisableLangConversion, $wgDisableTitleConversion, $wgRequest, $wgUser; + global $wgDisableLangConversion, $wgDisableTitleConversion, $wgRequest, + $wgUser; $isredir = $wgRequest->getText( 'redirect', 'yes' ); $action = $wgRequest->getText( 'action' ); $linkconvert = $wgRequest->getText( 'linkconvert', 'yes' ); - $disableLinkConversion = $wgDisableLangConversion || $wgDisableTitleConversion; + $disableLinkConversion = $wgDisableLangConversion + || $wgDisableTitleConversion; $linkBatch = new LinkBatch(); - $ns=NS_MAIN; + $ns = NS_MAIN; - if ( $disableLinkConversion || ( !$ignoreOtherCond && ( $isredir == 'no' || $action == 'edit' - || $action == 'submit' || $linkconvert == 'no' || $wgUser->getOption('noconvertlink') == 1 ) ) ) + if ( $disableLinkConversion || + ( !$ignoreOtherCond && + ( $isredir == 'no' + || $action == 'edit' + || $action == 'submit' + || $linkconvert == 'no' + || $wgUser->getOption( 'noconvertlink' ) == 1 ) ) ) { return; + } - if ( is_object( $nt ) ) + if ( is_object( $nt ) ) { $ns = $nt->getNamespace(); + } - $variants = $this->autoConvertToAllVariants($link); - if($variants == false) //give up + $variants = $this->autoConvertToAllVariants( $link ); + if ( $variants == false ) { // give up return; + } $titles = array(); - foreach( $variants as $v ) { - if($v != $link){ + foreach ( $variants as $v ) { + if ( $v != $link ) { $varnt = Title::newFromText( $v, $ns ); - if(!is_null($varnt)){ - $linkBatch->addObj($varnt); - $titles[]=$varnt; + if ( !is_null( $varnt ) ) { + $linkBatch->addObj( $varnt ); + $titles[] = $varnt; } } } @@ -635,8 +680,8 @@ class LanguageConverter { // fetch all variants in single query $linkBatch->execute(); - foreach( $titles as $varnt ) { - if( $varnt->getArticleID() > 0 ) { + foreach ( $titles as $varnt ) { + if ( $varnt->getArticleID() > 0 ) { $nt = $varnt; $link = $varnt->getText(); break; @@ -645,7 +690,7 @@ class LanguageConverter { } /** - * returns language specific hash options + * Returns language specific hash options. * * @public */ @@ -655,7 +700,7 @@ class LanguageConverter { } /** - * get title text as defined in the body of the article text + * Get title text as defined in the body of the article text. * * @public */ @@ -664,103 +709,109 @@ class LanguageConverter { } /** - * Load default conversion tables - * This method must be implemented in derived class + * Load default conversion tables. + * This method must be implemented in derived class. * * @private */ function loadDefaultTables() { - $name = get_class($this); - wfDie("Must implement loadDefaultTables() method in class $name"); + $name = get_class( $this ); + wfDie( "Must implement loadDefaultTables() method in class $name" ); } /** - * load conversion tables either from the cache or the disk + * Load conversion tables either from the cache or the disk. * @private */ - function loadTables($fromcache=true) { + function loadTables( $fromcache = true ) { global $wgMemc; - if( $this->mTablesLoaded ) + if ( $this->mTablesLoaded ) { return; + } wfProfileIn( __METHOD__ ); $this->mTablesLoaded = true; $this->mTables = false; - if($fromcache) { - wfProfileIn( __METHOD__.'-cache' ); + if ( $fromcache ) { + wfProfileIn( __METHOD__ . '-cache' ); $this->mTables = $wgMemc->get( $this->mCacheKey ); - wfProfileOut( __METHOD__.'-cache' ); + wfProfileOut( __METHOD__ . '-cache' ); } - if ( !$this->mTables || !isset( $this->mTables[self::CACHE_VERSION_KEY] ) ) { - wfProfileIn( __METHOD__.'-recache' ); + if ( !$this->mTables + || !isset( $this->mTables[self::CACHE_VERSION_KEY] ) ) { + wfProfileIn( __METHOD__ . '-recache' ); // not in cache, or we need a fresh reload. // we will first load the default tables // then update them using things in MediaWiki:Zhconversiontable/* $this->loadDefaultTables(); - foreach($this->mVariants as $var) { - $cached = $this->parseCachedTable($var); - $this->mTables[$var]->mergeArray($cached); + foreach ( $this->mVariants as $var ) { + $cached = $this->parseCachedTable( $var ); + $this->mTables[$var]->mergeArray( $cached ); } $this->postLoadTables(); $this->mTables[self::CACHE_VERSION_KEY] = true; - $wgMemc->set($this->mCacheKey, $this->mTables, 43200); - wfProfileOut( __METHOD__.'-recache' ); + $wgMemc->set( $this->mCacheKey, $this->mTables, 43200 ); + wfProfileOut( __METHOD__ . '-recache' ); } wfProfileOut( __METHOD__ ); } /** - * Hook for post processig after conversion tables are loaded + * Hook for post processig after conversion tables are loaded. * */ - function postLoadTables() {} + function postLoadTables() { } /** - * Reload the conversion tables + * Reload the conversion tables. * * @private */ function reloadTables() { - if($this->mTables) - unset($this->mTables); + if ( $this->mTables ) { + unset( $this->mTables ); + } $this->mTablesLoaded = false; - $this->loadTables(false); + $this->loadTables( false ); } /** - * parse the conversion table stored in the cache + * Parse the conversion table stored in the cache. * - * the tables should be in blocks of the following form: + * The tables should be in blocks of the following form: * -{ * word => word ; * word => word ; * ... * }- * - * to make the tables more manageable, subpages are allowed - * and will be parsed recursively if $recursive=true + * To make the tables more manageable, subpages are allowed + * and will be parsed recursively if $recursive == true. * */ - function parseCachedTable($code, $subpage='', $recursive=true) { + function parseCachedTable( $code, $subpage = '', $recursive = true ) { global $wgMessageCache; static $parsed = array(); - if(!is_object($wgMessageCache)) + if ( !is_object( $wgMessageCache ) ) { return array(); + } - $key = 'Conversiontable/'.$code; - if($subpage) + $key = 'Conversiontable/' . $code; + if ( $subpage ) { $key .= '/' . $subpage; - - if(array_key_exists($key, $parsed)) + } + if ( array_key_exists( $key, $parsed ) ) { return array(); + } if ( strpos( $code, '/' ) === false ) { $txt = $wgMessageCache->get( 'Conversiontable', true, $code ); } else { - $title = Title::makeTitleSafe( NS_MEDIAWIKI, "Conversiontable/$code" ); + $title = Title::makeTitleSafe( NS_MEDIAWIKI, + "Conversiontable/$code" ); if ( $title && $title->exists() ) { $article = new Article( $title ); $txt = $article->getContents(); @@ -771,57 +822,61 @@ class LanguageConverter { // get all subpage links of the form // [[MediaWiki:conversiontable/zh-xx/...|...]] - $linkhead = $this->mLangObj->getNsText(NS_MEDIAWIKI) . ':Conversiontable'; - $subs = explode('[[', $txt); + $linkhead = $this->mLangObj->getNsText( NS_MEDIAWIKI ) . + ':Conversiontable'; + $subs = explode( '[[', $txt ); $sublinks = array(); - foreach( $subs as $sub ) { - $link = explode(']]', $sub, 2); - if(count($link) != 2) + foreach ( $subs as $sub ) { + $link = explode( ']]', $sub, 2 ); + if ( count( $link ) != 2 ) { continue; - $b = explode('|', $link[0]); - $b = explode('/', trim($b[0]), 3); - if(count($b)==3) + } + $b = explode( '|', $link[0] ); + $b = explode( '/', trim( $b[0] ), 3 ); + if ( count( $b ) == 3 ) { $sublink = $b[2]; - else + } else { $sublink = ''; + } - if($b[0] == $linkhead && $b[1] == $code) { + if ( $b[0] == $linkhead && $b[1] == $code ) { $sublinks[] = $sublink; } } // parse the mappings in this page - $blocks = explode($this->mMarkup['begin'], $txt); - array_shift($blocks); + $blocks = explode( $this->mMarkup['begin'], $txt ); + array_shift( $blocks ); $ret = array(); - foreach($blocks as $block) { - $mappings = explode($this->mMarkup['end'], $block, 2); - $stripped = str_replace(array("'", '"', '*','#'), '', $mappings[0]); + foreach ( $blocks as $block ) { + $mappings = explode( $this->mMarkup['end'], $block, 2 ); + $stripped = str_replace( array( "'", '"', '*', '#' ), '', + $mappings[0] ); $table = explode( ';', $stripped ); - foreach( $table as $t ) { + foreach ( $table as $t ) { $m = explode( '=>', $t ); - if( count( $m ) != 2) + if ( count( $m ) != 2 ) continue; // trim any trailling comments starting with '//' - $tt = explode('//', $m[1], 2); - $ret[trim($m[0])] = trim($tt[0]); + $tt = explode( '//', $m[1], 2 ); + $ret[trim( $m[0] )] = trim( $tt[0] ); } } $parsed[$key] = true; // recursively parse the subpages - if($recursive) { - foreach($sublinks as $link) { - $s = $this->parseCachedTable($code, $link, $recursive); - $ret = array_merge($ret, $s); + if ( $recursive ) { + foreach ( $sublinks as $link ) { + $s = $this->parseCachedTable( $code, $link, $recursive ); + $ret = array_merge( $ret, $s ); } } - if ($this->mUcfirst) { - foreach ($ret as $k => $v) { - $ret[Language::ucfirst($k)] = Language::ucfirst($v); + if ( $this->mUcfirst ) { + foreach ( $ret as $k => $v ) { + $ret[Language::ucfirst( $k )] = Language::ucfirst( $v ); } } return $ret; @@ -829,42 +884,45 @@ class LanguageConverter { /** * Enclose a string with the "no conversion" tag. This is used by - * various functions in the Parser + * various functions in the Parser. * * @param string $text text to be tagged for no conversion * @return string the tagged text * @public */ - function markNoConversion($text, $noParse=false) { + function markNoConversion( $text, $noParse = false ) { # don't mark if already marked - if(strpos($text, $this->mMarkup['begin']) || - strpos($text, $this->mMarkup['end'])) + if ( strpos( $text, $this->mMarkup['begin'] ) + || strpos( $text, $this->mMarkup['end'] ) ) { return $text; + } - $ret = $this->mMarkup['begin'] .'R|'. $text . $this->mMarkup['end']; + $ret = $this->mMarkup['begin'] . 'R|' . $text . $this->mMarkup['end']; return $ret; } /** - * convert the sorting key for category links. this should make different - * keys that are variants of each other map to the same key + * Convert the sorting key for category links. This should make different + * keys that are variants of each other map to the same key. */ function convertCategoryKey( $key ) { return $key; } + /** - * hook to refresh the cache of conversion tables when - * MediaWiki:conversiontable* is updated + * Hook to refresh the cache of conversion tables when + * MediaWiki:conversiontable* is updated. * @private */ - function OnArticleSaveComplete($article, $user, $text, $summary, $isminor, $iswatch, $section, $flags, $revision) { + function OnArticleSaveComplete( $article, $user, $text, $summary, $isminor, + $iswatch, $section, $flags, $revision ) { $titleobj = $article->getTitle(); - if($titleobj->getNamespace() == NS_MEDIAWIKI) { + if ( $titleobj->getNamespace() == NS_MEDIAWIKI ) { $title = $titleobj->getDBkey(); - $t = explode('/', $title, 3); - $c = count($t); - if( $c > 1 && $t[0] == 'Conversiontable' ) { - if(in_array($t[1], $this->mVariants)) { + $t = explode( '/', $title, 3 ); + $c = count( $t ); + if ( $c > 1 && $t[0] == 'Conversiontable' ) { + if ( in_array( $t[1], $this->mVariants ) ) { $this->reloadTables(); } } @@ -873,21 +931,21 @@ class LanguageConverter { } /** - * Armour rendered math against conversion - * Wrap math into rawoutput -{R| math }- syntax + * Armour rendered math against conversion. + * Wrap math into rawoutput -{R| math }- syntax. * @public */ - function armourMath($text){ - // we need to convert '-{' and '}-' to '-{' and '}-' - // to avoid a unwanted '}-' appeared after the math-image. - $text = strtr( $text, array('-{' => '-{', '}-' => '}-') ); + function armourMath( $text ) { + // we need to convert '-{' and '}-' to '-{' and '}-' + // to avoid a unwanted '}-' appeared after the math-image. + $text = strtr( $text, array( '-{' => '-{', '}-' => '}-' ) ); $ret = $this->mMarkup['begin'] . 'R|' . $text . $this->mMarkup['end']; return $ret; } } /** - * parser for rules of language conversion , parse rules in -{ }- tag + * Parser for rules of language conversion , parse rules in -{ }- tag. * @ingroup Language * @author fdcn , PhiLiP */ @@ -911,26 +969,30 @@ class ConverterRule { * @param object $converter a LanguageConverter object * @access public */ - function __construct( $text, $converter ){ + function __construct( $text, $converter ) { $this->mText = $text; $this->mConverter = $converter; - foreach( $converter->mVariants as $v ){ + foreach ( $converter->mVariants as $v ) { $this->mConvTable[$v] = array(); } } /** - * check if variants array in convert array + * Check if variants array in convert array. * * @param string $variant Variant language code * @return string Translated text * @public */ - function getTextInBidtable( $variants ){ - if( is_string( $variants ) ){ $variants = array( $variants ); } - if( !is_array( $variants ) ) return false; - foreach( $variants as $variant ){ - if( array_key_exists( $variant, $this->mBidtable ) ){ + function getTextInBidtable( $variants ) { + if ( is_string( $variants ) ) { + $variants = array( $variants ); + } + if ( !is_array( $variants ) ) { + return false; + } + foreach ( $variants as $variant ) { + if ( array_key_exists( $variant, $this->mBidtable ) ) { return $this->mBidtable[$variant]; } } @@ -941,9 +1003,9 @@ class ConverterRule { * Parse flags with syntax -{FLAG| ... }- * @private */ - function parseFlags(){ + function parseFlags() { $text = $this->mText; - if(strlen($text) < 2 ) { + if ( strlen( $text ) < 2 ) { $this->mFlags = array( 'R' ); $this->mRules = $text; return; @@ -954,60 +1016,68 @@ class ConverterRule { $validFlags = $this->mConverter->mFlags; $variants = $this->mConverter->mVariants; - $tt = explode($markup['flagsep'], $text, 2); - if(count($tt) == 2) { - $f = explode($markup['varsep'], $tt[0]); - foreach($f as $ff) { - $ff = trim($ff); - if(array_key_exists($ff, $validFlags) && - !in_array($validFlags[$ff], $flags)) + $tt = explode( $markup['flagsep'], $text, 2 ); + if ( count( $tt ) == 2 ) { + $f = explode( $markup['varsep'], $tt[0] ); + foreach ( $f as $ff ) { + $ff = trim( $ff ); + if ( array_key_exists( $ff, $validFlags ) + && !in_array( $validFlags[$ff], $flags ) ) { $flags[] = $validFlags[$ff]; + } } $rules = $tt[1]; } else { $rules = $text; } - //check flags - if( in_array('R',$flags) ){ - $flags = array('R');// remove other flags - } elseif ( in_array('N',$flags) ){ - $flags = array('N');// remove other flags - } elseif ( in_array('-',$flags) ){ - $flags = array('-');// remove other flags - } elseif (count($flags)==1 && $flags[0]=='T'){ - $flags[]='H'; - } elseif ( in_array('H',$flags) ){ + // check flags + if ( in_array( 'R', $flags ) ) { + $flags = array( 'R' );// remove other flags + } elseif ( in_array( 'N', $flags ) ) { + $flags = array( 'N' );// remove other flags + } elseif ( in_array( '-', $flags ) ) { + $flags = array( '-' );// remove other flags + } elseif ( count( $flags ) == 1 && $flags[0] == 'T' ) { + $flags[] = 'H'; + } elseif ( in_array( 'H', $flags ) ) { // replace A flag, and remove other flags except T - $temp=array('+','H'); - if(in_array('T',$flags)) $temp[] = 'T'; - if(in_array('D',$flags)) $temp[] = 'D'; + $temp = array( '+', 'H' ); + if ( in_array( 'T', $flags ) ) { + $temp[] = 'T'; + } + if ( in_array( 'D', $flags ) ) { + $temp[] = 'D'; + } $flags = $temp; } else { - if ( in_array('A',$flags) ) { - $flags[]='+'; - $flags[]='S'; + if ( in_array( 'A', $flags ) ) { + $flags[] = '+'; + $flags[] = 'S'; + } + if ( in_array( 'D', $flags ) ) { + $flags = array_diff( $flags, array( 'S' ) ); } - if ( in_array('D',$flags) ) - $flags=array_diff($flags,array('S')); $flags_temp = array(); - foreach ($variants as $variant) { + foreach ( $variants as $variant ) { // try to find flags like "zh-hans", "zh-hant" // allow syntaxes like "-{zh-hans;zh-hant|XXXX}-" - if ( in_array($variant, $flags) ) + if ( in_array( $variant, $flags ) ) $flags_temp[] = $variant; } - if ( count($flags_temp) !== 0 ) + if ( count( $flags_temp ) !== 0 ) { $flags = $flags_temp; + } + } + if ( count( $flags ) == 0 ) { + $flags = array( 'S' ); } - if ( count($flags) == 0 ) - $flags = array('S'); - $this->mRules=$rules; - $this->mFlags=$flags; + $this->mRules = $rules; + $this->mFlags = $flags; } /** - * generate conversion table + * Generate conversion table. * @private */ function parseRules() { @@ -1021,44 +1091,52 @@ class ConverterRule { // varsep_pattern for preg_split: // text should be splited by ";" only if a valid variant // name exist after the markup, for example: - // -{zh-hans:xxx;zh-hant:yyy;}- + // -{zh-hans:xxx;zh-hant:\ + // yyy;}- // we should split it as: // array( - // [0] => 'zh-hans:xxx' - // [1] => 'zh-hant:yyy' - // [2] => '' - // ) + // [0] => 'zh-hans:xxx' + // [1] => 'zh-hant:yyy' + // [2] => '' + // ) $varsep_pattern = '/' . $markup['varsep'] . '\s*' . '(?='; - foreach( $variants as $variant ) { - $varsep_pattern .= $variant . '\s*' . $markup['codesep'] . '|'; // zh-hans:xxx;zh-hant:yyy + foreach ( $variants as $variant ) { + // zh-hans:xxx;zh-hant:yyy + $varsep_pattern .= $variant . '\s*' . $markup['codesep'] . '|'; + // xxx=>zh-hans:yyy; xxx=>zh-hant:zzz $varsep_pattern .= '[^;]*?' . $markup['unidsep'] . '\s*' . $variant - . '\s*' . $markup['codesep'] . '|'; // xxx=>zh-hans:yyy; xxx=>zh-hant:zzz + . '\s*' . $markup['codesep'] . '|'; } $varsep_pattern .= '\s*$)/'; - $choice = preg_split($varsep_pattern, $rules); + $choice = preg_split( $varsep_pattern, $rules ); - foreach( $choice as $c ) { - $v = explode($markup['codesep'], $c, 2); - if( count($v) != 2 ) - continue;// syntax error, skip - $to = trim($v[1]); - $v = trim($v[0]); - $u = explode($markup['unidsep'], $v, 2); + foreach ( $choice as $c ) { + $v = explode( $markup['codesep'], $c, 2 ); + if ( count( $v ) != 2 ) { + // syntax error, skip + continue; + } + $to = trim( $v[1] ); + $v = trim( $v[0] ); + $u = explode( $markup['unidsep'], $v, 2 ); // if $to is empty, strtr() could return a wrong result - if( count($u) == 1 && $to && in_array( $v, $variants ) ) { + if ( count( $u ) == 1 && $to && in_array( $v, $variants ) ) { $bidtable[$v] = $to; - } else if(count($u) == 2){ - $from = trim($u[0]); - $v = trim($u[1]); - if( array_key_exists( $v, $unidtable ) && !is_array( $unidtable[$v] ) - && $to && in_array( $v, $variants ) ) - $unidtable[$v] = array( $from=>$to ); - elseif ( $to && in_array( $v, $variants ) ) + } elseif ( count( $u ) == 2 ) { + $from = trim( $u[0] ); + $v = trim( $u[1] ); + if ( array_key_exists( $v, $unidtable ) + && !is_array( $unidtable[$v] ) + && $to + && in_array( $v, $variants ) ) { + $unidtable[$v] = array( $from => $to ); + } elseif ( $to && in_array( $v, $variants ) ) { $unidtable[$v][$from] = $to; + } } // syntax error, pass - if ( !array_key_exists( $v, $this->mConverter->mVariantNames ) ){ + if ( !array_key_exists( $v, $this->mConverter->mVariantNames ) ) { $bidtable = array(); $unidtable = array(); break; @@ -1071,111 +1149,121 @@ class ConverterRule { /** * @private */ - function getRulesDesc(){ + function getRulesDesc() { $codesep = $this->mConverter->mDescCodeSep; $varsep = $this->mConverter->mDescVarSep; - $text=''; - foreach($this->mBidtable as $k => $v) - $text .= $this->mConverter->mVariantNames[$k]."$codesep$v$varsep"; - foreach($this->mUnidtable as $k => $a) - foreach($a as $from=>$to) - $text.=$from.'⇒'.$this->mConverter->mVariantNames[$k]."$codesep$to$varsep"; + $text = ''; + foreach ( $this->mBidtable as $k => $v ) { + $text .= $this->mConverter->mVariantNames[$k] . "$codesep$v$varsep"; + } + foreach ( $this->mUnidtable as $k => $a ) { + foreach ( $a as $from => $to ) { + $text .= $from . '⇒' . $this->mConverter->mVariantNames[$k] . + "$codesep$to$varsep"; + } + } return $text; } /** - * Parse rules conversion + * Parse rules conversion. * @private */ - function getRuleConvertedStr( $variant, $doConvert ){ + function getRuleConvertedStr( $variant, $doConvert ) { $bidtable = $this->mBidtable; $unidtable = $this->mUnidtable; - if( count($bidtable) + count($unidtable) == 0 ){ + if ( count( $bidtable ) + count( $unidtable ) == 0 ) { return $this->mRules; - } elseif ( $doConvert ){// the text converted + } elseif ( $doConvert ) { // the text converted // display current variant in bidirectional array - $disp = $this->getTextInBidtable($variant); + $disp = $this->getTextInBidtable( $variant ); // or display current variant in fallbacks - if(!$disp) + if ( !$disp ) { $disp = $this->getTextInBidtable( - $this->mConverter->getVariantFallbacks($variant)); + $this->mConverter->getVariantFallbacks( $variant ) ); + } // or display current variant in unidirectional array - if(!$disp && array_key_exists($variant,$unidtable)){ - $disp = array_values($unidtable[$variant]); + if ( !$disp && array_key_exists( $variant, $unidtable ) ) { + $disp = array_values( $unidtable[$variant] ); $disp = $disp[0]; } // or display frist text under disable manual convert - if(!$disp && $this->mConverter->mManualLevel[$variant]=='disable') { - if(count($bidtable)>0){ - $disp = array_values($bidtable); + if ( !$disp + && $this->mConverter->mManualLevel[$variant] == 'disable' ) { + if ( count( $bidtable ) > 0 ) { + $disp = array_values( $bidtable ); $disp = $disp[0]; } else { - $disp = array_values($unidtable); - $disp = array_values($disp[0]); + $disp = array_values( $unidtable ); + $disp = array_values( $disp[0] ); $disp = $disp[0]; } } return $disp; - } else {// no convert + } else { // no convert return $this->mRules; } } /** - * generate conversion table for all text + * Generate conversion table for all text. * @private */ - function generateConvTable(){ + function generateConvTable() { $flags = $this->mFlags; $bidtable = $this->mBidtable; $unidtable = $this->mUnidtable; $manLevel = $this->mConverter->mManualLevel; - $vmarked=array(); - foreach($this->mConverter->mVariants as $v) { + $vmarked = array(); + foreach ( $this->mConverter->mVariants as $v ) { /* for bidirectional array fill in the missing variants, if any, with fallbacks */ - if(!array_key_exists($v, $bidtable)) { - $variantFallbacks = $this->mConverter->getVariantFallbacks($v); - $vf = $this->getTextInBidtable($variantFallbacks); - if($vf) $bidtable[$v] = $vf; + if ( !array_key_exists( $v, $bidtable ) ) { + $variantFallbacks = + $this->mConverter->getVariantFallbacks( $v ); + $vf = $this->getTextInBidtable( $variantFallbacks ); + if ( $vf ) { + $bidtable[$v] = $vf; + } } - if(array_key_exists($v,$bidtable)){ - foreach($vmarked as $vo){ + if ( array_key_exists( $v, $bidtable ) ) { + foreach ( $vmarked as $vo ) { // use syntax: -{A|zh:WordZh;zh-tw:WordTw}- - // or -{H|zh:WordZh;zh-tw:WordTw}- or -{-|zh:WordZh;zh-tw:WordTw}- + // or -{H|zh:WordZh;zh-tw:WordTw}- + // or -{-|zh:WordZh;zh-tw:WordTw}- // to introduce a custom mapping between // words WordZh and WordTw in the whole text - if($manLevel[$v]=='bidirectional'){ - $this->mConvTable[$v][$bidtable[$vo]]=$bidtable[$v]; + if ( $manLevel[$v] == 'bidirectional' ) { + $this->mConvTable[$v][$bidtable[$vo]] = $bidtable[$v]; } - if($manLevel[$vo]=='bidirectional'){ - $this->mConvTable[$vo][$bidtable[$v]]=$bidtable[$vo]; + if ( $manLevel[$vo] == 'bidirectional' ) { + $this->mConvTable[$vo][$bidtable[$v]] = $bidtable[$vo]; } } - $vmarked[]=$v; + $vmarked[] = $v; } - /*for unidirectional array - fill to convert tables */ - $allow_unid = $manLevel[$v]=='bidirectional' - || $manLevel[$v]=='unidirectional'; - if( $allow_unid && array_key_exists( $v, $unidtable ) ){ + /*for unidirectional array fill to convert tables */ + if ( ( $manLevel[$v] == 'bidirectional' + || $manLevel[$v] == 'unidirectional' ) + && array_key_exists( $v, $unidtable ) ) { $ct = $this->mConvTable[$v]; - $this->mConvTable[$v] = array_merge($ct, $unidtable[$v]); + $this->mConvTable[$v] = array_merge( $ct, $unidtable[$v] ); } } } /** - * Parse rules and flags + * Parse rules and flags. * @public */ - function parse($variant){ - if(!$variant) + function parse( $variant = NULL ) { + if ( !$variant ) { $variant = $this->mConverter->getPreferredVariant(); + } $variants = $this->mConverter->mVariants; $this->parseFlags(); @@ -1183,70 +1271,82 @@ class ConverterRule { // convert to specified variant // syntax: -{zh-hans;zh-hant[;...]|}- - if( count( array_diff( $flags, $variants ) ) == 0 and count( $flags ) != 0 ) { - if ( in_array( $variant, $flags ) ) // check if current variant in flags + if ( count( array_diff( $flags, $variants ) ) == 0 + and count( $flags ) != 0 ) { + // check if current variant in flags + if ( in_array( $variant, $flags ) ) { // then convert to current language - $this->mRules = $this->mConverter->autoConvert( $this->mRules, $variant ); - else { // if current variant no in flags, + $this->mRules = $this->mConverter->autoConvert( $this->mRules, + $variant ); + } else { // if current variant no in flags, // then we check its fallback variants. - $variantFallbacks = $this->mConverter->getVariantFallbacks($variant); + $variantFallbacks = + $this->mConverter->getVariantFallbacks( $variant ); foreach ( $variantFallbacks as $variantFallback ) { // if current variant's fallback exist in flags if ( in_array( $variantFallback, $flags ) ) { // then convert to fallback language - $this->mRules = $this->mConverter->autoConvert( $this->mRules, $variantFallback ); + $this->mRules = + $this->mConverter->autoConvert( $this->mRules, + $variantFallback ); break; } } } - $this->mFlags = $flags = array('R'); + $this->mFlags = $flags = array( 'R' ); } - if( !in_array( 'R', $flags ) || !in_array( 'N', $flags ) ) { + if ( !in_array( 'R', $flags ) || !in_array( 'N', $flags ) ) { // decode => HTML entities modified by Sanitizer::removeHTMLtags - $this->mRules = str_replace('=>','=>',$this->mRules); + $this->mRules = str_replace( '=>', '=>', $this->mRules ); $this->parseRules(); } $rules = $this->mRules; - if( count( $this->mBidtable ) == 0 && count( $this->mUnidtable ) == 0 ){ - if(in_array('+',$flags) || in_array('-',$flags)) + if ( count( $this->mBidtable ) == 0 + && count( $this->mUnidtable ) == 0 ) { + if ( in_array( '+', $flags ) || in_array( '-', $flags ) ) { // fill all variants if text in -{A/H/-|text} without rules - foreach($this->mConverter->mVariants as $v) + foreach ( $this->mConverter->mVariants as $v ) { $this->mBidtable[$v] = $rules; - elseif (!in_array('N',$flags) && !in_array('T',$flags) ) - $this->mFlags = $flags = array('R'); + } + } elseif ( !in_array( 'N', $flags ) && !in_array( 'T', $flags ) ) { + $this->mFlags = $flags = array( 'R' ); + } } - if( in_array('R',$flags) ) { + if ( in_array( 'R', $flags ) ) { // if we don't do content convert, still strip the -{}- tags $this->mRuleDisplay = $rules; - } elseif ( in_array('N',$flags) ){ + } elseif ( in_array( 'N', $flags ) ) { // proces N flag: output current variant name - $this->mRuleDisplay = $this->mConverter->mVariantNames[trim($rules)]; - } elseif ( in_array('D',$flags) ){ + $this->mRuleDisplay = + $this->mConverter->mVariantNames[ trim( $rules ) ]; + } elseif ( in_array( 'D', $flags ) ) { // proces D flag: output rules description $this->mRuleDisplay = $this->getRulesDesc(); - } elseif ( in_array('H',$flags) || in_array('-',$flags) ) { + } elseif ( in_array( 'H', $flags ) || in_array( '-', $flags ) ) { // proces H,- flag or T only: output nothing $this->mRuleDisplay = ''; - } elseif ( in_array('S',$flags) ){ - $this->mRuleDisplay = $this->getRuleConvertedStr($variant, - $this->mConverter->mDoContentConvert); + } elseif ( in_array( 'S', $flags ) ) { + $this->mRuleDisplay = $this->getRuleConvertedStr( $variant, + $this->mConverter->mDoContentConvert ); } else { - $this->mRuleDisplay= $this->mManualCodeError; + $this->mRuleDisplay = $this->mManualCodeError; } // proces T flag - if ( in_array('T',$flags) ) { - $this->mRuleTitle = $this->getRuleConvertedStr($variant, - $this->mConverter->mDoTitleConvert); + if ( in_array( 'T', $flags ) ) { + $this->mRuleTitle = $this->getRuleConvertedStr( $variant, + $this->mConverter->mDoTitleConvert ); } - if (in_array('-', $flags)) - $this->mRulesAction='remove'; - if (in_array('+', $flags)) - $this->mRulesAction='add'; + if ( in_array( '-', $flags ) ) { + $this->mRulesAction = 'remove'; + } + if ( in_array( '+', $flags ) ) { + $this->mRulesAction = 'add'; + } $this->generateConvTable(); } @@ -1254,54 +1354,56 @@ class ConverterRule { /** * @public */ - function hasRules(){ + function hasRules() { // TODO: } /** - * get display text on markup -{...}- + * Get display text on markup -{...}- * @public */ - function getDisplay(){ + function getDisplay() { return $this->mRuleDisplay; } + /** - * get converted title + * Get converted title. * @public */ - function getTitle(){ + function getTitle() { return $this->mRuleTitle; } /** - * return how deal with conversion rules + * Return how deal with conversion rules. * @public */ - function getRulesAction(){ + function getRulesAction() { return $this->mRulesAction; } /** - * get conversion table ( bidirectional and unidirectional conversion table ) + * Get conversion table. ( bidirectional and unidirectional + * conversion table ) * @public */ - function getConvTable(){ + function getConvTable() { return $this->mConvTable; } /** - * get conversion rules string + * Get conversion rules string. * @public */ - function getRules(){ + function getRules() { return $this->mRules; } /** - * get conversion flags + * Get conversion flags. * @public */ - function getFlags(){ + function getFlags() { return $this->mFlags; } } -- 2.20.1