Boucle infinie avec les messages d'erreurs localisés

Je prends en charge.

Normalement je sors quelque chose dans l'après-midi.

fix by using an internal "last-chance fallback" locale, by default it is en_EN

Voilà le patch.

Ca règle le problème évoqué ce matin.

Et ça m'a donné de nouvelles perspectives pour améliorer mon patch du ticket #95.

On verra ça début janvier.

++ Julien

Ça ne rêgle pas tout, en particulier dans le cas où la locale n'existe pas du tout (dans un autre contexte que celui du bug). Je n'ai donc pas pris en compte ton patch sauf au niveau de $this->_cacheSuffix.

Pour info, voici la correction que j'ai appliqué dans le trunk (en plus de tests unitaires) :

Index: lib/jelix/core/jSelector.class.php
===================================================================
--- lib/jelix/core/jSelector.class.php  (révision 714)
+++ lib/jelix/core/jSelector.class.php  (copie de travail)
@@ -460,7 +460,7 @@
         $this->charset = $charset;
         $this->_dirname =  'locales/' .$locale.'/';
         $this->_suffix = '.'.$charset.'.properties';
-        $this->_cacheSuffix = '.'.$charset.'.php';
+        $this->_cacheSuffix = '.'.$locale.'.'.$charset.'.php';
         $this->_compilerPath=JELIX_LIB_CORE_PATH.'jLocalesCompiler.class.php';

 #if ENABLE_PHP_JELIX
@@ -489,6 +489,8 @@
     protected function _createPath(){
         global $gJConfig;
         if(!isset($gJConfig->_modulesPathList[$this->module])){
+            if ($this->module == 'jelix')
+                throw new Exception('jelix module is not enabled !!');
             throw new jExceptionSelector('jelix~errors.selector.module.unknow', $this->toString());
         }

@@ -503,7 +505,15 @@
         $this->_path = $gJConfig->_modulesPathList[$this->module].$this->_dirname.$this->resource.$this->_suffix;

         if (!is_readable ($this->_path)){
-            throw new jExceptionSelector('jelix~errors.selector.invalid.target', array($this->toString(), "locale"));
+            // to avoid infinite loop in a specific lang, we should check if we don't
+            // try to retrieve the same message as the one we use for the exception below,
+            // and if it is this message, it means that the error message doesn't exist
+            // in the specific lang, so we retrieve it in en_EN language
+            if($this->toString() == 'jelix~errors.selector.invalid.target')
+                $l = 'en_EN';
+            else
+                $l=null;
+            throw new jExceptionSelector('jelix~errors.selector.invalid.target', array($this->toString(), "locale"),1,$l);
         }
         $this->_where = 'modules/';
     }
Index: lib/jelix/core/jException.lib.php
===================================================================
--- lib/jelix/core/jException.lib.php   (révision 714)
+++ lib/jelix/core/jException.lib.php   (copie de travail)
@@ -119,14 +119,15 @@
      * @param string $localekey a locale key
      * @param array $localeParams parameters for the message (for sprintf)
      * @param integer $code error code (can be provided by the localized message)
+     * @param string $lang
      */
-    public function __construct($localekey, $localeParams=array(), $code = 1) {
+    public function __construct($localekey, $localeParams=array(), $code = 1, $lang=null) {

         $this->localeKey = $localekey;
         $this->localeParams = $localeParams;

         try{
-            $message = jLocale::get($localekey, $localeParams);
+            $message = jLocale::get($localekey, $localeParams, $lang);
         }catch(Exception $e){
             $message = $e->getMessage();
         }