This application is not used any more and exists only for history. Post new tickets on the Github account.
Cette application n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Ticket #1203 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

les plugins jlocale n'accepte pas tous les paramètres pour jLocale::get

Reported by: Vincentv Owned by:
Priority: normal Milestone: Jelix 1.3 beta 1
Component: jelix:plugins:tpl Version: 1.1.7
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed: yes
Hosting Provider: Php version:

Description

les plugins jtpl jlocale ne permettent pas d'indiquer la lang et le charset

Attachments

tpl.jlocale.diff (7.9 KB) - added by Vincentv 3 years ago.
tpl.jlocale-0.2.diff (9.6 KB) - added by Vincentv 3 years ago.

Change History

Changed 3 years ago by Vincentv

comment:1 Changed 3 years ago by foxmask

Actuellement, sauf erreur, on ne peut pas y toucher car c'est hérité de la config (charset+lang dans defautlconfig.ini.php)

personnellement le seul intérêt à cela serait de jongler avec des charsets d'une page à l'autre. Pour la langue comme elle peut être spécifiée sur l'url et ensuite récupérée par jLocale, je ne vois pas l'utilité.

Ou alors autre possibilité, avoir une page traduite en plusieurs langue en même temps. Mais cette fois-ci c'est le charset qui devient inutile car on ne peut pas mixer plusieurs charset sur la même page.

Quel est le but en fait ? :)

comment:2 Changed 3 years ago by Vincentv

Pour avoir plusieurs traduction sur la même page principalement ;). Par exemple si tu veux faire une page (ou une barre) pour sélectionner une langue avec le libelle localisé dans la langue correspondant (anglais en anglais, finnois en finnois, etc)

le charset est la uniquement parce que jLocale::get prend le charset en dernier paramètre. Jelix supporte encore les charset autre qu'UTF-8 tant que le ticket #982 n'est pas traité :D

comment:3 follow-up: ↓ 4 Changed 3 years ago by laurentj

Pour la langue, je peux comprendre. Mais pour le charset, il y aura un souci : tu ne peux pas avoir dans une même page plusieurs phrases écrite dans des charsets différents. Le navigateur interprète la page selon le charset indiqué dans l'entête http ou à défaut, dans la balise meta.

Cela ne sert donc à rien de vouloir indiquer un charset différent du charset par défaut. non ?

comment:4 in reply to: ↑ 3 Changed 3 years ago by Vincentv

Oui ca sert a rien, je l'avais juste mis car jLocale::get prenait un charset. je refait le patch

Changed 3 years ago by Vincentv

comment:5 Changed 3 years ago by laurentj

  • review set to review-

Il y a un problème si on veut rajouter le code langue. En effet, les plugins en questions acceptent un nombre indéfini d'arguments, qui servent de valeurs à insérer dans la chaine.

On peut donc écrire

{jlocale 'cle', array($val1, $va2, $val3,...)}
ou
{jlocale 'cle', $val1, $va2, $val3,...}

Il faut donc garder cette compatibilité.

Je propose donc qu'on ne puisse indiquer une clé de locale, uniquement dans le cas où on reçoit un tableau en deuxième paramètre. Si ce n'est pas un tableau, il faut traiter les arguments comme à l'origine.

comment:6 Changed 3 years ago by laurentj

  • Milestone changed from Jelix 1.2 to Jelix 1.3

après discussion sur IRC, on pousse à la 1.3, afin de permettre de casser l'API.

comment:7 Changed 3 years ago by laurentj

  • Status changed from new to closed
  • Type changed from bug to enhancement
  • Resolution set to fixed

comment:8 Changed 3 years ago by laurentj

  • Documentation needed set
Note: See TracTickets for help on using tickets.