Anomalie #1125
closedGalette 0.9rc3. Traduction.
100%
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
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à ?
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.
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.
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
- File 0001-Store-fields-category-names-in-english-in-the-databa.patch 0001-Store-fields-category-names-in-english-in-the-databa.patch added
- File 0002-Update-fields-category-names-in-the-database-on-next.patch 0002-Update-fields-category-names-in-the-database-on-next.patch added
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 ;)
Updated by Guillaume AGNIERAY almost 5 years ago
- Status changed from In Progress to Résolu
- % Done changed from 0 to 100
Appliqué par commit b217ba27194a560650919a151f382f30e6293f7e.
Updated by Johan Cwiklinski almost 5 years ago
J'avais presque bon au départ donc :) Merci pour le corectif
Updated by Johan Cwiklinski almost 5 years ago
- Status changed from Résolu to Fermé
- Target version set to 0.9.3
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.
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.
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.
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).
Updated by Guillaume AGNIERAY almost 5 years ago
- File 0001-Fix-typo-in-next-upgrade-script-fixes-1125.patch 0001-Fix-typo-in-next-upgrade-script-fixes-1125.patch added
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.
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 ?
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 ;)
Updated by Guillaume AGNIERAY almost 5 years ago
- File 0001-Fix-translation-in-template-closes-1125.patch 0001-Fix-translation-in-template-closes-1125.patch added
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 :)
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 ;)
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
Updated by Roland Telle almost 5 years ago
C'est réglé, on peut fermer.
Merci à Johan et Guillaume ;-)
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
- 0001-Fix-typo-in-next-upgrade-script-fixes-1125.patch
- 0001-Fix-translation-in-template-closes-1125.patch
Merci ! :)
Updated by Johan Cwiklinski almost 5 years ago
- Status changed from Résolu to Fermé
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.
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 ;-)
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.
Updated by Guillaume AGNIERAY almost 5 years ago
- File 0001-Forgotten-translation-in-template-closes-1125.patch 0001-Forgotten-translation-in-template-closes-1125.patch added
- Status changed from Nouveau to In Progress
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.
Updated by Anonymous almost 5 years ago
- Status changed from In Progress to Résolu
Appliqué par commit 130eac336cbe71fc995e86123f93b76933ce476b.
Updated by Roland Telle almost 5 years ago
C'est parfait, on peut fermer ce ticket. Merci
Updated by Johan Cwiklinski almost 5 years ago
- Status changed from Résolu to Fermé
Me semble bon aussi maintenant ; merci :)