From: Niklas Laxström <nikerabbit@users.mediawiki.org>
Date: Fri, 19 Aug 2011 09:02:35 +0000 (+0000)
Subject: Fu r94907 - also update localisation cache, which accesses the fallback variable... 
X-Git-Tag: 1.31.0-rc.0~28177
X-Git-Url: http://git.cyclocoop.org/%7B%24admin_url%7Dmes_infos.php?a=commitdiff_plain;h=cd47443fcfb2ef677e27b35fab1c2749e9b3378f;p=lhc%2Fweb%2Fwiklou.git

Fu r94907 - also update localisation cache, which accesses the fallback variable directly
Fixes bug 30456
---

diff --git a/includes/LocalisationCache.php b/includes/LocalisationCache.php
index 8d8c03f872..b20db3298a 100644
--- a/includes/LocalisationCache.php
+++ b/includes/LocalisationCache.php
@@ -1,6 +1,6 @@
 <?php
 
-define( 'MW_LC_VERSION', 1 );
+define( 'MW_LC_VERSION', 2 );
 
 /**
  * Class for caching the contents of localisation files, Messages*.php
@@ -485,7 +485,6 @@ class LocalisationCache {
 	 * and save it to the persistent cache store and the process cache
 	 */
 	public function recache( $code ) {
-		static $recursionGuard = array();
 		global $wgExtensionMessagesFiles, $wgExtensionAliasesFiles;
 		wfProfileIn( __METHOD__ );
 
@@ -522,27 +521,27 @@ class LocalisationCache {
 			$coreData['fallback'] = $code === 'en' ? false : 'en';
 		}
 
-		if ( $coreData['fallback'] !== false ) {
-			# Guard against circular references
-			if ( isset( $recursionGuard[$code] ) ) {
-				throw new MWException( "Error: Circular fallback reference in language code $code" );
+		if ( $coreData['fallback'] === false ) {
+			$coreData['fallbackSequence'] = array();
+		} else {
+			$coreData['fallbackSequence'] = array_map( 'trim', explode( ',', $coreData['fallback'] ) );
+			$len = count( $coreData['fallbackSequence'] );
+			# Ensure that the sequence ends at en
+			if ( $coreData['fallbackSequence'][$len - 1] !== 'en' ) {
+				$coreData['fallbackSequence'][] = 'en';
 			}
-			$recursionGuard[$code] = true;
 
 			# Load the fallback localisation item by item and merge it
-			$deps = array_merge( $deps, $this->getItem( $coreData['fallback'], 'deps' ) );
-			foreach ( self::$allKeys as $key ) {
-				if ( is_null( $coreData[$key] ) || $this->isMergeableKey( $key ) ) {
-					$fallbackValue = $this->getItem( $coreData['fallback'], $key );
-					$this->mergeItem( $key, $coreData[$key], $fallbackValue );
+			foreach ( $coreData['fallbackSequence'] as $fallback ) {
+				$deps = array_merge( $deps, $this->getItem( $fallback, 'deps' ) );
+				foreach ( self::$allKeys as $key ) {
+					if ( is_null( $coreData[$key] ) || $this->isMergeableKey( $key ) ) {
+						$fallbackValue = $this->getItem( $fallback, $key );
+						$this->mergeItem( $key, $coreData[$key], $fallbackValue );
+					}
 				}
 			}
-			$fallbackSequence = $this->getItem( $coreData['fallback'], 'fallbackSequence' );
-			array_unshift( $fallbackSequence, $coreData['fallback'] );
-			$coreData['fallbackSequence'] = $fallbackSequence;
-			unset( $recursionGuard[$code] );
-		} else {
-			$coreData['fallbackSequence'] = array();
+			
 		}
 		$codeSequence = array_merge( array( $code ), $coreData['fallbackSequence'] );