[BUG] Traduction message succès Newsletter (Page 1) / Développement / Thelia 2

Announcement

Rejoignez la communauté sur le Discord Thelia : https://discord.gg/YgwpYEE3y3

yoan1005
Membre

Registered: 21-11-2014
Posts: 29

Offline

#1 [BUG] Traduction message succès Newsletter

(03-07-2015 08:19:04)

Bonjour,

Vous avez peut être remarqué sur la version 2.0.4 (et peut être les autres), lorsque que vous inscrivez à la newsletter, le message de succès est toujours :

Thanks for signing up ! We’ll keep you posted whenever we have any new updates.

Embêtant pour un site en version FR... Et après minte recherche dans les fichiers tpl, les fichiers de traductions aucunes traces de cette phrase.

En analysant le code j'ai réussi à remonté à la source, la phrase est mise (sans traduction) dans le fichier NewsletterController.php dans local/modules/Front/Controller, un fichier qu'on est pas trop censé toucher..

Bref pensez à corriger ce bug dans les prochaines MAJ

A+

MrGuillou
Administrateur

Registered: 16-01-2015
Posts: 297

Offline

#2 Re: [BUG] Traduction message succès Newsletter

(03-07-2015 08:26:30)

Bonjour,

Ceci est corrigé à partir de la version 2.1 de Thelia
https://github.com/thelia/thelia/blob/2 … er.php#L85

Développeur Web chez OpenStudio
Thelia Module, Thelia School, My GitHub, My Twitter

yoan1005
Membre

Registered: 21-11-2014
Posts: 29

Offline

#3 Re: [BUG] Traduction message succès Newsletter

(03-07-2015 08:27:47)

Parfait

neywen
Membre

Registered: 06-08-2014
Posts: 39

Offline

#4 Re: [BUG] Traduction message succès Newsletter

(07-02-2016 10:24:53)

Bonjour,

je suis sous Thelia 2.2.1 et je tombe sur le même bug.
J'ai remplacé le template Front par default, par un template custom "myCustomTemplate",

et je remarque que pour que la traduction se fasse bien, il est nécessaire de remplacer, dans le NewsletterController :

                "message" => ($errorMessage) ? $errorMessage : $this->getTranslator()->trans(
                    "Thanks for signing up! We'll keep you posted whenever we have any new updates.",
                )

par

                "message" => ($errorMessage) ? $errorMessage : $this->getTranslator()->trans(
                    "Thanks for signing up! We'll keep you posted whenever we have any new updates.",
                    [],
                    "fo.myCustomTemplate"
                )

ce qui n'est évidemment pas propre. C'est un fixe temporaire. Je suis peut-être passé à côté de quelque chose ?
loïc b.

roadster31
Core Dev

Registered: 19-09-2009
Posts: 9,454
Toulouse
Website

Offline

#5 Re: [BUG] Traduction message succès Newsletter

(07-02-2016 11:44:15)

La correction adéquate est :

"message" => ($errorMessage) ? $errorMessage : $this->getTranslator()->trans(
"Thanks for signing up! We'll keep you posted whenever we have any new updates.",
[],
Front::MESSAGE_DOMAIN
)

En effet, ce message appartient au domaine de traduction du module Front, et pas à celui de ton template. Il faut ajouter ensuite la traduction dans le module Front.

OpenStudio Toulouse

neywen
Membre

Registered: 06-08-2014
Posts: 39

Offline

#6 Re: [BUG] Traduction message succès Newsletter

(07-02-2016 11:46:57)

of course. Merci pour le tip, je vais corriger.
l.