Project

General

Profile

Actions

Anomalie #1125

closed

Galette 0.9rc3. Traduction.

Added by Roland Telle over 6 years ago. Updated almost 5 years ago.

Status:
Fermé
Priority:
Normal
Category:
I18n
Target version:
Start date:
12/17/2017
Due date:
% Done:

100%

Estimated time:
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.


Files

Actions #1

Updated by Johan Cwiklinski over 6 years ago

  • Category set to I18n
  • Status changed from Nouveau to In Progress
  • Assignee set to 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à ?

Actions #2

Updated by Johan Cwiklinski over 6 years ago

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.

Actions #3

Updated by Roland Telle over 6 years ago

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.

Actions #4

Updated by Johan Cwiklinski over 6 years ago

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.

Updated by Guillaume AGNIERAY almost 5 years ago

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 ;)

Actions #6

Updated by Guillaume AGNIERAY almost 5 years ago

  • Status changed from In Progress to Résolu
  • % Done changed from 0 to 100
Actions #7

Updated by Johan Cwiklinski almost 5 years ago

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

Actions #8

Updated by Johan Cwiklinski almost 5 years ago

  • Status changed from Résolu to Fermé
  • Target version set to 0.9.3
Actions #9

Updated by Roland Telle almost 5 years ago

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.

Actions #10

Updated by Johan Cwiklinski almost 5 years ago

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.

Actions #11

Updated by Roland Telle almost 5 years ago

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.

Actions #12

Updated by Roland Telle almost 5 years ago

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).

Actions #13

Updated by Guillaume AGNIERAY almost 5 years ago

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.

Actions #14

Updated by Roland Telle almost 5 years ago

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 ?

Actions #15

Updated by Guillaume AGNIERAY almost 5 years ago

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 ;)

Actions #16

Updated by Guillaume AGNIERAY almost 5 years ago

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 :)

Actions #17

Updated by Johan Cwiklinski almost 5 years ago

  • Status changed from Fermé to In Progress
  • Assignee changed from Johan Cwiklinski to Guillaume AGNIERAY
  • % Done changed from 100 to 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 ;)

Actions #18

Updated by Johan Cwiklinski almost 5 years ago

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

Actions #19

Updated by Roland Telle almost 5 years ago

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

Actions #20

Updated by Guillaume AGNIERAY almost 5 years ago

  • Status changed from In Progress to Résolu
  • Assignee changed from Guillaume AGNIERAY to Johan Cwiklinski
  • % Done changed from 80 to 100
Les 2 derniers patchs peuvent être appliqués pour clore définitivement cette anomalie.

Merci ! :)

Actions #21

Updated by Johan Cwiklinski almost 5 years ago

  • Status changed from Résolu to Fermé
Actions #22

Updated by Alain Paris almost 5 years ago

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.

Actions #23

Updated by Roland Telle almost 5 years ago

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 ;-)

Actions #24

Updated by Johan Cwiklinski almost 5 years ago

  • Status changed from Fermé to 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.

Actions #25

Updated by Guillaume AGNIERAY almost 5 years ago

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.

Actions #26

Updated by Anonymous almost 5 years ago

  • Status changed from In Progress to Résolu
Actions #27

Updated by Roland Telle almost 5 years ago

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

Actions #28

Updated by Johan Cwiklinski almost 5 years ago

  • Status changed from Résolu to Fermé

Me semble bon aussi maintenant ; merci :)

Actions

Also available in: Atom PDF