THELIA Forum

Welcome to the THELIA support and discusssion forum

Announcement

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

Offline

#1 Site multilangue

(24-04-2017 14:06:34)

Bonjour,

Je constate un dysfonctionnement au niveau des urls sur un site en multilangue.
exemple sur le site
president-electronics.fr

je prends un produit qui existe dans plusieurs langues
en français https://president-electronics.fr/Access … ro-DNC-518
lien vers l'anglais https://president-electronics.fr/Access … lang=en_US
lorsque l'on clique sur le lien anglais, la réécriture renvoie sur la page d'accueil

Je ne vois pas ou chercher, ou se trouve la partie que s'occupe de la reecriture ?

Merci

Offline

#2 Re: Site multilangue

(25-04-2017 07:12:21)

Salut,

Comme il semble que tu aies un domaine par langue, ton URL vers l'anglais ne devrait-elle pas être celle là ?
https://president-electronics.com/Acces … ro-DNC-518

Offline

#3 Re: Site multilangue

(25-04-2017 07:43:54)

informatiqueprog wrote:

Comme il semble que tu aies un domaine par langue, ton URL vers l'anglais ne devrait-elle pas être celle là ?
https://president-electronics.com/Acces … ro-DNC-518

Oui après réécriture cela devrait donner le lien que tu donnes

les liens vers les autres langues sont généré par

{url path="{navigate to="current"}" lang={$LOCALE}}

ce qui donne sur la page française vers l'anglais
https://president-electronics.fr/Access … lang=en_US
mais lorsque l'on clique sur le lien cela renvoie vers https://president-electronics.com
cela ne réécrit pas l'url
bug ? je ne trouve pas la partie du code qui doit réécrire

Offline

#4 Re: Site multilangue

(25-04-2017 08:25:16)

Pour 

{url path="{navigate to="current"}" lang={$LOCALE}}

Le code est ici
https://github.com/thelia/thelia/blob/m … or.php#L56

Offline

#5 Re: Site multilangue

(25-04-2017 11:12:50)

informatiqueprog wrote:

Pour 

{url path="{navigate to="current"}" lang={$LOCALE}}

Le code est ici
https://github.com/thelia/thelia/blob/m … or.php#L56


Merci !
je vais regarder comment corriger

Offline

#6 Re: Site multilangue

(25-04-2017 12:43:02)

je recherche la partie de code qui s'occupe de traduire
https://president-electronics.fr/Access … lang=en_US
en
https://president-electronics.com/Acces … ro-DNC-518

Ce n'est pas dans UrlGenerator.php
Je continue a chercher
Merci

Offline

#7 Re: Site multilangue

(25-04-2017 13:38:58)

Regarde ici : https://github.com/thelia/thelia/blob/m … r.php#L159

OpenStudio Toulouse

Offline

#8 Re: Site multilangue

(25-04-2017 15:02:10)

roadster31 wrote:

Merci RoadSter31 ! je vais regarder de ce cote
Le peu que je teste en locale tout est correct, la reecriture se fait bien

Offline

#9 Re: Site multilangue

(25-04-2017 15:33:23)

Je ne sais pas ce que j'ai fait mais
la reecriture ne marche pas
lorsque j'essaie d'acceder a la page anglaise ça part directement sur le domaine de la langue sans passer par la fonction matchrequest()
je vais desactiver quelques modules...

Offline

#10 Re: Site multilangue

(26-04-2017 08:13:49)

Bonjour,

Je ne trouve pas ou est realise la reecriture de l'url
president-electronics.fr/Accessoires/CB/Micros/Micro-DNC-518?lang=en_US
en
president-electronics.com/Accessories/CB/Microphones/Micro-DNC-518

j'ai mis des points d'arret dans UrlGenerator.php et RewritingRouter.php mais apparemment la traduction ne se fait pas la.
Je ne trouve pas ou cette derniere s'effectue

Offline

#11 Re: Site multilangue

(26-04-2017 08:52:54)

Quelle est ta valeur pour la variable de conf "one_domain_foreach_lang" ?

J'ai peut-être une piste, regarde...
https://github.com/thelia/thelia/blob/m … e.php#L140

Offline

#12 Re: Site multilangue

(26-04-2017 12:02:43)

informatiqueprog wrote:

Quelle est ta valeur pour la variable de conf "one_domain_foreach_lang" ?

J'ai peut-être une piste, regarde...
https://github.com/thelia/thelia/blob/m … e.php#L140

one_domain_foreach_lang = 1

c'est bien la ligne 140 qui me retourne le nouveau domaine sans prendre en compte le reste

c'est un peu normal, vu que j'ai des domaines differents, c'est sense etre sur des serveurs differents
pour mon cas j'ai un seul serveur avec des domaines qui pointent sur la même ressource Thelia

j'ai teste en remplacant le domaine fr par com
remplace:
president-electronics.fr/Accessoires/CB/Micros/Micro-DNC-518?lang=en_US
par :
president-electronics.com/Accessoires/CB/Micros/Micro-DNC-518?lang=en_US

j'obtiens bien la traduction president-electronics.com/Accessories/CB/Microphones/Micro-DNC-518

Last edited by zzuutt (26-04-2017 12:31:08)

Offline

#13 Re: Site multilangue

(26-04-2017 15:54:38)

Merci InformatiqueProg !
J'ai modifie sur mon site maintenant la traduction se fait
le visiteur est renvoye vers la page qu'il consulte mais avec la nouvelle langue

Offline

#14 Re: Site multilangue

(26-04-2017 16:15:50)

De rien (tu as oublié roadster31 wink )

Tu as résolu ton problème en modifiant le fichier "ParamInitMiddleware.php" ?

Peux-tu faire une ISSUE ou PR (avec un lien sur ce post) ?

Offline

#15 Re: Site multilangue

(26-04-2017 16:26:31)

Pardon Merci Roadster31 !

informatiqueprog wrote:

De rien (tu as oublié roadster31 wink )

Tu as résolu ton problème en modifiant le fichier "ParamInitMiddleware.php" ?

Peux-tu faire une ISSUE ou PR (avec un lien sur ce post) ?

Non j'ai modifie directement dans le template. - je remplace le domaine par celui de la langue ciblee (c'est pas beau mais ca marche)

Je suis en train de voir comment modifier directement dans "ParamInitMiddleware.php"
Je pense rajouter une variable de config 'sameServerSameThelia'

Offline

#16 Re: Site multilangue

(28-04-2017 11:27:55)

Bonjour,

Je viens de faire une PR pour ajouter la reecriture des url si le domaine est sur le meme serveur, meme Thelia.
Mon probleme sur GitHub
j'ai clone Thelia sur Github desktop
J'ai realise les modifs
mais comment je cree la PR
si je fais synchroniser il les transfert sur la branche Thelia pas sur mon compte
y aurait il une commande ? ou je dois tout refaire ?

Merci

Offline

#17 Re: Site multilangue

(28-04-2017 11:32:46)

Normalement, tu dois créer une nouvelle branche avant d'attaquer tes modifications
Maintenant, je ne sais pas ce qui est plus simple...

Offline

#18 Re: Site multilangue

(28-04-2017 11:38:22)

Voici la PR
https://github.com/thelia/thelia/pull/2381

Last edited by zzuutt (29-04-2017 13:26:48)