Projet

Général

Profil

Actions

Anomalie #1125

fermé

Galette 0.9rc3. Traduction.

Ajouté par Roland Telle il y a plus de 6 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
I18n
Version cible:
Début:
17/12/2017
Echéance:
% réalisé:

100%

Temps estimé:
Version utilisée:

Description

Dans la page Configuration des champs, les titres des modules repliables sont en français avec la mention (not translated), dans les versions française et anglaise de Galette.


Fichiers

Mis à jour par Johan Cwiklinski il y a plus de 6 ans

  • Catégorie mis à I18n
  • Statut changé de Nouveau à In Progress
  • Assigné à mis à Johan Cwiklinski

En effet, c'est étrange...

Il s'agit d'une nouvelle installation je présume ? Sur une mise à jour, le problème est-il identique ?

Dans la base qui pose le souci, les chaînes doivent-être en français (table galette_fields_categories) ; est-ce le cas de bases plus anciennes, ou le texte est-il en anglais dans ces cas là ?

Mis à jour par Johan Cwiklinski il y a plus de 6 ans

J'ai regardé de plus près ; les chaînes en question devraient être stockées en anglais dans la base ; leur traduction est assurée à l'affichage.

Pour corriger ça :

diff --git a/galette/includes/fields_defs/members_fields_cats.php b/galette/includes/fields_defs/members_fields_cats.php
index e465762..c4301e4 100644
--- a/galette/includes/fields_defs/members_fields_cats.php
+++ b/galette/includes/fields_defs/members_fields_cats.php
@@ -41,19 +41,19 @@ $members_fields_cats = array(
     array(
         'id'         => 1,
         'table_name' => Adherent::TABLE,
-        'category'   => _T("Identity:"),
+        'category'   => "Identity",
         'position'   => 1
     ),
     array(
         'id'         => 2,
         'table_name' => Adherent::TABLE,
-        'category'   => _T("Galette-related data:"),
+        'category'   => "Galette-related data",
         'position'   => 3
     ),
     array(
         'id'         => 3,
         'table_name' => Adherent::TABLE,
-        'category'   => _T("Contact information:"),
+        'category'   => "Contact information",
         'position'   => 2
     )
 );

Ça devrait résoudre le problème pour les nouvelles installations. Si c'était OK en 0.8, il n'y a pas de soucis particulier ; la mise à jour n'affectera rien. Il faudra envisager dans le script de migration de la 0.9.1 d'ajouter un correctif pour les utilisateurs des RC.

Mis à jour par Roland Telle il y a plus de 6 ans

Il s'agit de l'avant dernière nightly et le problème est identique sur une galette neuve (local) ou une mise à jour (production) à partir d'une v8.3.3 où le problème n'apparaissait pas.

Le correctif, hélas, est sans effet.

Mis à jour par Johan Cwiklinski il y a plus de 6 ans

Roland Telle a écrit :

Il s'agit de l'avant dernière nightly et le problème est identique sur une galette neuve (local) ou une mise à jour (production) à partir d'une v8.3.3 où le problème n'apparaissait pas.

Le correctif, hélas, est sans effet.

J'ai testé sur un autre poste sans le patch, pour essayer de corriger le souci ce mdi... Et je ne l'ai pas reproduit :-/ Ça me fait penser que le soucis est ailleurs ; mais je ne sais pas trop où. Bon, c'est mineur, si je ne parviens pas à le corriger tout de suite, ça peut attendre un peu.

Mis à jour par Guillaume AGNIERAY il y a plus de 4 ans

Johan Cwiklinski a écrit :

J'ai testé sur un autre poste sans le patch, pour essayer de corriger le souci ce mdi... Et je ne l'ai pas reproduit :-/ Ça me fait penser que le soucis est ailleurs ; mais je ne sais pas trop où. Bon, c'est mineur, si je ne parviens pas à le corriger tout de suite, ça peut attendre un peu.

Testé sur une galette neuve sur les branches master et develop : problème confirmé.

Roland Telle a écrit :

Le correctif, hélas, est sans effet.

Car il manque les ":" à la fin de chaque chaîne et qu'elles ne sont pas reconnues lors de la traduction.

Voici des patchs avec les chaînes valides pour la trad et un update des valeurs correspondantes dans le script de migration de la 0.9.3.
Si ces correctifs sont validés, je ferai l'ajout dans le changelog ;)

Mis à jour par Guillaume AGNIERAY il y a plus de 4 ans

  • Statut changé de In Progress à Résolu
  • % réalisé changé de 0 à 100

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

J'avais presque bon au départ donc :) Merci pour le corectif

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de Résolu à Fermé
  • Version cible mis à 0.9.3

Mis à jour par Roland Telle il y a plus de 4 ans

Roland Telle a écrit :

Dans la page Configuration des champs, les titres des modules repliables sont en français avec la mention (not translated), dans les versions française et anglaise de Galette.

Le problème n'est pas résolu chez moi avec la dernière nightly. Dans les trois langues, les expressions dites not translated sont en français.

Dans la fiche adhérent, les termes Identity:, Contact information: et Galette-related data: sont, dans les trois langues, en anglais sans la mention not translated
Seul Champs supplémentaires : est traduit correctement dans chaque langue.

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

Si la base de données de ta nighly était déjà a jour, les modifs en base ne sont pas appliquées (normal) ; il faut que tu lances les requêtes ajoutées dans ton phpmyadmin.

Mis à jour par Roland Telle il y a plus de 4 ans

Johan Cwiklinski a écrit :

Si la base de données de ta nighly était déjà a jour, les modifs en base ne sont pas appliquées (normal) ; il faut que tu lances les requêtes ajoutées dans ton phpmyadmin.

Tu as raison. Ca marche pour la page Configuration des champs sous réserve de corriger la requête en remplaçant indentity: par identity: ;-)

En revanche, je confirme :

Dans la fiche adhérent, les termes Identity: , Contact information: et Galette-related data: sont, dans les trois langues, en anglais sans la mention not translated. Seul Champs supplémentaires : est traduit correctement dans chaque langue.

Mis à jour par Roland Telle il y a plus de 4 ans

Roland Telle a écrit :

Roland Telle a écrit :

Dans la fiche adhérent, les termes Identity:, Contact information: et Galette-related data: sont, dans les trois langues, en anglais sans la mention not translated
Seul Champs supplémentaires : est traduit correctement dans chaque langue.

Et pour régler le problème, il faut supprimer le patch _0001-Store-fields-category-names-in-english-in-the-databa.patch _ (revenir en arrière).

Mis à jour par Guillaume AGNIERAY il y a plus de 4 ans

Roland Telle a écrit :

Johan Cwiklinski a écrit :

Si la base de données de ta nighly était déjà a jour, les modifs en base ne sont pas appliquées (normal) ; il faut que tu lances les requêtes ajoutées dans ton phpmyadmin.

Tu as raison. Ca marche pour la page Configuration des champs sous réserve de corriger la requête en remplaçant indentity: par identity: ;-)

Oups! Voici le correctif pour la branche develop. Merci :)

Roland Telle a écrit :

Roland Telle a écrit :

Dans la fiche adhérent, les termes Identity:, Contact information: et Galette-related data: sont, dans les trois langues, en anglais sans la mention not translated
Seul Champs supplémentaires : est traduit correctement dans chaque langue.

Et pour régler le problème, il faut supprimer le patch _0001-Store-fields-category-names-in-english-in-the-databa.patch _ (revenir en arrière).

Ce patch est indispensable pour que le problème ne se produise pas sur les nouvelles installations (testé et confirmé).

Pour les installations existantes directement concernées par le problème, seul le script de maj de la bdd est en mesure d'apporter la correction nécessaire (testé et confirmé). Sur ces installations, l'autre patch à lui tout seul n'a aucun effet.

Mis à jour par Roland Telle il y a plus de 4 ans

Guillaume AGNIERAY a écrit :

Ce patch est indispensable pour que le problème ne se produise pas sur les nouvelles installations (testé et confirmé).

Ok.

Pour les installations existantes directement concernées par le problème,

C'est mon cas.

seul le script de maj de la bdd est en mesure d'apporter la correction nécessaire (testé et confirmé). Sur ces installations, l'autre patch à lui tout seul n'a aucun effet.

C'est quand même curieux car l'application du patch de mise à jour de la bdd a bien fonctionné pour la page Configuration des champs mais est resté sans effet pour la page Fiche adhérent où j'ai effectué, avec succès, le retour en arrière du patch 0001 (members_fields_cats.php).
Curieux non ?

Mis à jour par Guillaume AGNIERAY il y a plus de 4 ans

Roland Telle a écrit :

C'est quand même curieux car l'application du patch de mise à jour de la bdd a bien fonctionné pour la page Configuration des champs mais est resté sans effet pour la page Fiche adhérent où j'ai effectué, avec succès, le retour en arrière du patch 0001 (members_fields_cats.php).
Curieux non ?

Pardon, je n'avais pas lu attentivement ton précédent commentaire et n'avais pas vérifié la fiche adhérent.
Je confirme le problème sur cette page.
Certainement un autre update de la base à faire ; je vais regarder ;)

Mis à jour par Guillaume AGNIERAY il y a plus de 4 ans

Guillaume AGNIERAY a écrit :

Pardon, je n'avais pas lu attentivement ton précédent commentaire et n'avais pas vérifié la fiche adhérent.
Je confirme le problème sur cette page.
Certainement un autre update de la base à faire ; je vais regarder ;)

En fait non. C'était tout simplement les chaînes en question qui ne passaient pas par la fonction _T dans le template correspondant.
Ceci devrai corriger le problème sur la fiche adhérent :)

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de Fermé à In Progress
  • Assigné à changé de Johan Cwiklinski à Guillaume AGNIERAY
  • % réalisé changé de 100 à 80

Hum, autre chose... Avec les derniers commits, il n'y a plus aucune occurrence de la chaîne Identity:, elle sera donc supprimée à la prochaine mise à jour des chaînes...

Il faut l'ajouter dans galette/includes/i18n.inc.php, on a un petit hack à la fin pour les chaînes du genre ;)

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

Johan Cwiklinski a écrit :

Il faut l'ajouter dans galette/includes/i18n.inc.php, on a un petit hack à la fin pour les chaînes du genre ;)

Fait : 512b8d34

Mis à jour par Roland Telle il y a plus de 4 ans

C'est réglé, on peut fermer.
Merci à Johan et Guillaume ;-)

Mis à jour par Guillaume AGNIERAY il y a plus de 4 ans

  • Statut changé de In Progress à Résolu
  • Assigné à changé de Guillaume AGNIERAY à Johan Cwiklinski
  • % réalisé changé de 80 à 100
Les 2 derniers patchs peuvent être appliqués pour clore définitivement cette anomalie.

Merci ! :)

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de Résolu à Fermé

Mis à jour par Alain Paris il y a plus de 4 ans

Bonjour,
Je viens de réinstaller a zero une nightly (09/10/2019)et pour info sur la fiche Modification adhérent http://galette/webroot/member/edit/1 il subsiste les termes Identity:, Contact information: et Galette-related data: quelque soit la langue.

Mis à jour par Roland Telle il y a plus de 4 ans

Alain Paris a écrit :

Je viens de réinstaller a zero une nightly (09/10/2019) et pour info sur la fiche Modification adhérent http://galette/webroot/member/edit/1 il subsiste les termes Identity:, Contact information: et Galette-related data: quelque soit la langue.

Ainsi que sur la page Ajouter un adhérent ;-)

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de Fermé à Nouveau

Je me doutais que ce serait un merdier ce truc (ce pourquoi ce bug n'avait jamais été traité). J'essaierai de voir ça, quand j'aurai du temps.

Mis à jour par Guillaume AGNIERAY il y a plus de 4 ans

Il manquait encore une fonction _T dans un template.
J'ai cherché dans les autres templates pour voir si les catégories de champs étaient utilisées ailleurs.
Dans l'absolu c'est bon : elles ne sont utilisées que sur la vue d'un adhérent et le formulaire d'ajout/inscription.

Mis à jour par Anonyme il y a plus de 4 ans

  • Statut changé de In Progress à Résolu

Mis à jour par Roland Telle il y a plus de 4 ans

C'est parfait, on peut fermer ce ticket. Merci

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de Résolu à Fermé

Me semble bon aussi maintenant ; merci :)

Actions

Formats disponibles : Atom PDF