rub a écrit:

Pour la version optimisée, j'ai vu que tu as fait plus de switch_lang_back que switch_lang_to.
J'avais pensé qu'une erreur se serait lévée... mais non...
Ton test -//:---\spam que c'est bon

C'était volontaire, pour faire le test complet.

rub a écrit:

mais en fait non... les $lang, $lang_info sont corrects mais $user aura une langue à null.
J'ai donc rajouté un test sur la valeur du array_pop. J'ai fait aussi une version en testant le count. (laquelle prendre?)

Je préfère que la deuxième solution, vérifier le count avant d'extraire une donnée me parait plus correcte.

Sinon Bravo!
  Plus qu'a attendre une mise à jour.

Bonne soirée.

Dernière modification par cljosse (2010-02-18 20:19:22)

cljosse a écrit:

rub a écrit:

Pour la version optimisée, j'ai vu que tu as fait plus de switch_lang_back que switch_lang_to.
J'avais pensé qu'une erreur se serait lévée... mais non...
Ton test -//:---\spam que c'est bon

C'était volontaire, pour faire le test complet.

Je sais bien ;-)

cljosse a écrit:

rub a écrit:

mais en fait non... les $lang, $lang_info sont corrects mais $user aura une langue à null.
J'ai donc rajouté un test sur la valeur du array_pop. J'ai fait aussi une version en testant le count. (laquelle prendre?)

Je préfère que la deuxième solution, vérifier le count avant d'extraire une donnée me parait plus correcte.

Vendu, je commite ca asap.

cljosse a écrit:

Sinon Bravo!
  Plus qu'a attendre une mise à jour.

Bonne soirée.

Bravo à toi pour tous les tests et pour m'avoir aiguiller vers mes erreurs.
La vérification croisée du code, c'est ce qu'il a de mieux ;-)

Voila, c'est fait!

[Bugtracker] ticket 1142
[Subversion] r4907
[Subversion] r4908

cljosse a écrit:

Un dernier point  à éclaicir à propos du rechargement des langues.
Dans  switch_lang_to tu utilises :

           common.lang.php
           admin.lang.php
et
           local.lang.php
    Je suppose que cela devait servir à charger une langue locale.

Oui, c'est ca...
J'ai repris le même modèle que le chargement d'une langue lors de l'initialisation de la page.

cljosse a écrit:

Je ne trouve pas trace d'un fichier nommé "local.lang" ?

C'est normal, comme tous les fichiers "local*" sur Piwigo. (conf, css, ...)
On les utilise que s'ils sont présents...

cljosse a écrit:

Pourrait-on pas utiliser

load_language('local.lang', $Plug_in_Path, array('language'=>$language, 'no_fallback'=>true));
pour rechargé la langue?  En ajoutant un argument dans la fonction switch_lang_to ($language,$Plug_in_Path="")
En imposant 'local.lang.php' comme nom de fichier langue aux créateurs de plugins.

Ca, c'est une discussion plus avancée qui déborderdu cadre des 2 fonctions abordées ;-)
Je t'invite à ouvrir un nouveau topic (sur le forum anglais normalement) pour en discuter.
Mais, effectivement, ca pourrait être pratique comme méthode; mais bon, c'est possible de le faire avec le trigger.

cljosse a écrit:

Ce n'est qu'une remarque, après tout.
et je ne connais pas tout les tenants et aboutissements de la fonctions.

Cf mon commentaire au dessus, c'est un principe à généraliser, les fonctions swicths doivent simplement se calquer sur le même modèle que celui de l'init de la page.

Par contre, autre remarque, les fonctions swicth_lang sont dans un include destiné au mail (car besoin initial).
Il serait peut-être bon de le mettre dans un autre include:
  o soit le functions.inc.php avec le l10n
  o soit un nouveau fichier functions.lang.php

Il va y avoir du changement au niveau des traductions donc ca sera le moment de mettre en place de nouvelles choses!