Code style fixes in Language::isValidCode()
authorlwelling <lwelling@wikimedia.org>
Fri, 3 May 2013 17:26:14 +0000 (13:26 -0400)
committerGerrit Code Review <gerrit@wikimedia.org>
Sat, 4 May 2013 04:44:15 +0000 (04:44 +0000)
Fix issues from https://gerrit.wikimedia.org/r/#/c/62123

Change-Id: I97e826bb9aa9e3a14d07aab7cf02a09ef2c84ef7

languages/Language.php

index 84e7f37..2fd4bb7 100644 (file)
@@ -327,18 +327,18 @@ class Language {
         */
        public static function isValidCode( $code ) {
                static $cache = array();
-               if( isset( $cache[$code] ) ) {
+               if ( isset( $cache[$code] ) ) {
                        return $cache[$code];
                }
                // People think language codes are html safe, so enforce it.
                // Ideally we should only allow a-zA-Z0-9-
                // but, .+ and other chars are often used for {{int:}} hacks
                // see bugs 37564, 37587, 36938
-               $return =
+               $cache[$code] =
                        strcspn( $code, ":/\\\000&<>'\"" ) === strlen( $code )
                        && !preg_match( Title::getTitleInvalidRegex(), $code );
-               $cache[ $code ] = $return;
-               return $return;
+
+               return $cache[$code];
        }
 
        /**