Anomalie #1125
ferméGalette 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.
Fichiers
Mis à jour par Johan Cwiklinski il y a presque 7 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 presque 7 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 presque 7 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 presque 7 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 environ 5 ans
- Fichier 0001-Store-fields-category-names-in-english-in-the-databa.patch 0001-Store-fields-category-names-in-english-in-the-databa.patch ajouté
- Fichier 0002-Update-fields-category-names-in-the-database-on-next.patch 0002-Update-fields-category-names-in-the-database-on-next.patch ajouté
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 environ 5 ans
- Statut changé de In Progress à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit b217ba27194a560650919a151f382f30e6293f7e.
Mis à jour par Johan Cwiklinski il y a environ 5 ans
J'avais presque bon au départ donc :) Merci pour le corectif
Mis à jour par Johan Cwiklinski il y a environ 5 ans
- Statut changé de Résolu à Fermé
- Version cible mis à 0.9.3
Mis à jour par Roland Telle il y a environ 5 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 environ 5 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 environ 5 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 environ 5 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 environ 5 ans
- Fichier 0001-Fix-typo-in-next-upgrade-script-fixes-1125.patch 0001-Fix-typo-in-next-upgrade-script-fixes-1125.patch ajouté
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 environ 5 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 environ 5 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 environ 5 ans
- Fichier 0001-Fix-translation-in-template-closes-1125.patch 0001-Fix-translation-in-template-closes-1125.patch ajouté
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 environ 5 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 environ 5 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 environ 5 ans
C'est réglé, on peut fermer.
Merci à Johan et Guillaume ;-)
Mis à jour par Guillaume AGNIERAY il y a environ 5 ans
- Statut changé de In Progress à Résolu
- Assigné à changé de Guillaume AGNIERAY à Johan Cwiklinski
- % réalisé changé de 80 à 100
- 0001-Fix-typo-in-next-upgrade-script-fixes-1125.patch
- 0001-Fix-translation-in-template-closes-1125.patch
Merci ! :)
Mis à jour par Alain Paris il y a environ 5 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 environ 5 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 environ 5 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 environ 5 ans
- Fichier 0001-Forgotten-translation-in-template-closes-1125.patch 0001-Forgotten-translation-in-template-closes-1125.patch ajouté
- Statut changé de Nouveau à 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.
Mis à jour par Anonyme il y a environ 5 ans
- Statut changé de In Progress à Résolu
Appliqué par commit 130eac336cbe71fc995e86123f93b76933ce476b.
Mis à jour par Roland Telle il y a environ 5 ans
C'est parfait, on peut fermer ce ticket. Merci
Mis à jour par Johan Cwiklinski il y a environ 5 ans
- Statut changé de Résolu à Fermé
Me semble bon aussi maintenant ; merci :)