Fork me on GitHub






nicemen
Corriger petit Bug xLanguage + ajout fonction pour afficher URL page en cours !
#1
Semi pro
Inscrit: 19/08/2006 03:45
De Paris
Messages: 520
Le module xlanguage fonctionne parfaitement sous Xoops 2.5.8.1 il faut penser à installer le contenu du dossier Extra.

Une fois mis en service le module à un léger bug, lorsqu'on clique sur drapeau il ne recharge pas la page courante ce qui est un peu embêtant pour les utilisateurs.

Pour corriger le problème j'ai procéder comme ceci :

Dans le fichier Class/theme.php à la ligne 298 j'a ajouté ça
'page_url' => htmlspecialchars("http://".$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI'], ENT_QUOTES),
c'est le code smarty qui va permettre de générer l'url de la page en cours
<{$page_url}>
utile aussi pour les métas Facebook, Twitter, ...

Ensuite dans le module xlanguage :

modules/xlanguage/templates/blocks/xlanguage_block.html

il faut remplacer la ligne N°4 :
<a href="<{$block.url}><{$lang.name}>" title="<{$lang.desc}>">

par celle-ci :
<a href="<{$page_url}>?lang=<{$lang.name}>" title="<{$lang.desc}>">


Et voilà c'est tout ! maintenant vous pouvez changer de langue sans changer de page.

Posté le : 19/01/2017 08:21

Version Bootstrap
4.5.3
Version font-awesome
5.15.1
Version Xoops
2.5.10
Partager Twitter Partagez cette article sur GG+
Rapport Haut
Cesagonchu
Re: Corriger petit Bug xLanguage + ajout fonction pour afficher URL page en cours !
#2
Admin Frxoops
Inscrit: 16/03/2009 16:40
De Tende (06)
Messages: 3528
Merci pour ton aide, je t'ai envoyé un message privé afin de faire avancer les choses

Posté le : 19/01/2017 12:26

Open in new window

Xoopseuse Mai 2012 | Responsable équipe internationale | Recherche avancée en bas de page !
Partager Twitter Partagez cette article sur GG+
Rapport Haut
Cesagonchu
Re: Corriger petit Bug xLanguage + ajout fonction pour afficher URL page en cours !
#3
Admin Frxoops
Inscrit: 16/03/2009 16:40
De Tende (06)
Messages: 3528
Tu n'as pas répondu à mon message privé nicemen.

Posté le : 25/01/2017 10:57

Open in new window

Xoopseuse Mai 2012 | Responsable équipe internationale | Recherche avancée en bas de page !
Partager Twitter Partagez cette article sur GG+
Rapport Haut
Cesagonchu
Re: Corriger petit Bug xLanguage + ajout fonction pour afficher URL page en cours !
#4
Admin Frxoops
Inscrit: 16/03/2009 16:40
De Tende (06)
Messages: 3528
Suite à ta proposition de solution et afin de ne pas faire d'erreur lorsque je la proposerai en ton nom, peux-tu me faire passer par courriel ton fichier class/theme.php de Xoops ainsi que ton fichier corrigé /xlanguage/templates/blocks/xlanguage_block.html du module xLanguage ?

Posté le : 17/02/2017 10:37

Open in new window

Xoopseuse Mai 2012 | Responsable équipe internationale | Recherche avancée en bas de page !
Partager Twitter Partagez cette article sur GG+
Rapport Haut
mage
Re: Corriger petit Bug xLanguage + ajout fonction pour afficher URL page en cours !
#5
Semi pro
Inscrit: 08/03/2006 21:08
Messages: 1176
Pour le fichier theme.php, je sais ce qu'il faut faire pour intégrer le code donc pas besoin du fichier. Le code à un problème car si le site est en https, cela ne fonctionne pas... C'est pas un gros problème à contourner mais on va voir ce que Richard pense de cette demande, peut être qu'il a une autre solution. Il faut poster la demande sur Github pour qu'elle soit traitée rapidement.

Pourquoi une modification du module ne suffit pas?

Posté le : 17/02/2017 12:07

monxoops.fr
Mage sur github
Partager Twitter Partagez cette article sur GG+
Rapport Haut
mage
Re: Corriger petit Bug xLanguage + ajout fonction pour afficher URL page en cours !
#6
Semi pro
Inscrit: 08/03/2006 21:08
Messages: 1176
Dans le module c'est pas bon, désolé!

Pas besoin de poster une demande de modification de xoops! La solution est très simple avec une variable qui existe déjà!

xoops_requesturi


Il faut juste modifier le code dans xlanguage:

<a href="<{$xoops_requesturi}>?lang=<{$lang.name}>" title="<{$lang.desc}>">


Tout simple et ça fonctionne!

Donc problème de module et pas de xoops!


Posté le : 17/02/2017 13:50

monxoops.fr
Mage sur github
Partager Twitter Partagez cette article sur GG+
Rapport Haut
Cesagonchu
Re: Corriger petit Bug xLanguage + ajout fonction pour afficher URL page en cours !
#7
Admin Frxoops
Inscrit: 16/03/2009 16:40
De Tende (06)
Messages: 3528
Ok, merci, peux-tu faire un pull request ici ?

C'est l'endroit où xLanguage est le plus à jour.

Posté le : 17/02/2017 13:58

Open in new window

Xoopseuse Mai 2012 | Responsable équipe internationale | Recherche avancée en bas de page !
Partager Twitter Partagez cette article sur GG+
Rapport Haut
mage
Re: Corriger petit Bug xLanguage + ajout fonction pour afficher URL page en cours !
#8
Semi pro
Inscrit: 08/03/2006 21:08
Messages: 1176
Je n'ai pas téléchargé le module... Je vais faire directement depuis le site!

Posté le : 17/02/2017 14:11

monxoops.fr
Mage sur github
Partager Twitter Partagez cette article sur GG+
Rapport Haut
nicemen
Re: Corriger petit Bug xLanguage + ajout fonction pour afficher URL page en cours !
#9
Semi pro
Inscrit: 19/08/2006 03:45
De Paris
Messages: 520
Merci je ne connaissais pas ce smarty mais, il n'affiche pas l'URL absolue.

Et pour les méta Facebook ou Twitter par exemple & bien ça ne fonctionne pas, il faut une URL absolue.

Posté le : 22/02/2017 23:14

Version Bootstrap
4.5.3
Version font-awesome
5.15.1
Version Xoops
2.5.10
Partager Twitter Partagez cette article sur GG+
Rapport Haut

 Haut   Précédent   Suivant



Vous pouvez voir les sujets.
Vous ne pouvez pas débuter de nouveaux sujets.
Vous ne pouvez pas répondre aux contributions.
Vous ne pouvez pas éditer vos contributions.
Vous ne pouvez pas effacez vos contributions.
Vous ne pouvez pas ajouter de nouveaux sondages.
Vous ne pouvez pas voter en sondage.
Vous ne pouvez pas attacher des fichiers à vos contributions.
Vous ne pouvez pas poster sans approbation.
Propulsé avec XOOPS | Graphisme adapté par Tatane, Grosdunord, Montuy337513

47 Personne(s) en ligne (38 Personne(s) connectée(s) sur Forum) | Utilisateur(s): 0 | Invité(s): 47 | Plus ...