CtrlAltSuppr.com / ravie de l'intégration de signatux

Melody Nelson · 10/02/08 01:43:48

bonjour,

je viens d'intégrer signatux à un site multilingue géré avec Plume CMS... ce n'est pas en soi, une "vraie intégration" au sein du CMS, mais l'utilisateur final ne verra pas la différence

j'ai fait quelques bidouilles "amateur" pour faire suivre la langue d'affichage du livre d'or dans les différents cas de figure (message d'erreur, redirection) afin que les utilisateurs restent toujours sur l'interface du livre d'or dans la langue du site qu'il consulte (je me sers également de la langue du livre d'or pour afficher automatiquement les rubriques et liens du CMS)

le livre d'or n'est pas encore "officiellement" lancé, je fais encore quelques tests, voilà les adresses :
http://bilin-village.org/livre.php?sgtux_langue=en
http://bilin-village.org/livre.php?sgtux_langue=fr

j'en profite pour signaler quelques petites choses :
- la modification du template dans l'admin n'est pas prise en compte, j'ai contourné en faisant mes modifs directement dans phpmyadmin
- je n'ai pas réussi à supprimer le lien "Mail" de l'affichage du livre d'or choisissant "non" dans l'admin pour "Links Web and Mail always on"... j'ai donc retouché directement le template

et enfin une suggestion : les messages d'erreurs et messages affichés après acceptation d'un nouveau message s'affichent dans la langue de l'interface d'amin... c'est un peu gênant quand on propose un livre d'or en plusieurs langues car dans l'idéal, il faudrait que ces messages soient personnalisés en fonction de la langue d'affichage du livre (peu importe la langue de l'admin)

merci pour ce livre d'or, vraiment bien conçu autant en admin qu'en utilisation

Malcolm · 10/02/08 12:17:11

merci d'avoir donné un tel retour d'info, ça fera plaisir au créateur du script, je sais qu'il apprécie beaucoup. Et puis dire qu'on est content, c'est bien aussi, c'est facile de dire quand on n'est pas content, mais des avis positifs, il faut en exprimer aussi ...

Ju · 11/02/08 19:44:19

Hello, merci pour tes retours smile .

Pour le pb de la modification des templates, c'est une erreur b^ete de ma part (dsl).

Dans valide.php, ligne 895, remplace :

Code:

    // Injection SQL
    $signatux->sql_query('UPDATE '.$signatux->config['SQL']['Table'].'_extras SET valeur="'.$template.'" WHERE nom="template"');

Par :

Code:

    // Injection SQL
    $signatux->sql_query('UPDATE '.$signatux->config['SQL']['Table'].'_extras SET valeur="'.$template.'" WHERE nom="Template"');

Pour ce qui est de la langue dans les pages de redirections, puisque tu n'as pas l'air d'avoir peur de bidouiller, je te propose cette série de modifications (je dois avouer que je n'ai pas testé, mais ça devrait ^etre cohérent).

Dans admin/class/admin.class.php, ligne 37, remplace :

Code:

        // Sélection de la langue
        if(file_exists(SGTUX_ROOT.'langues/'.$this->config['Divers']['Langue'].'/'.$this->config['Divers']['Langue'].'.admin.inc.php'))
            require_once(SGTUX_ROOT.'langues/'.$this->config['Divers']['Langue'].'/'.$this->config['Divers']['Langue'].'.admin.inc.php');
        else
            require_once(SGTUX_ROOT.'langues/fr/fr.admin.inc.php');

Par :

Code:

        // Sélection de la langue
        if(isset($_COOKIE['sgtux_langue']) || isset($_GET['sgtux_langue']))
            $langue = htmlentities((isset($_GET['sgtux_langue'])) ? $_GET['sgtux_langue'] : $_COOKIE['sgtux_langue']);
        else
            $langue = $this->config['Divers']['Langue'];

        if(file_exists(SGTUX_ROOT.'langues/'.$langue.'/'.$langue.'.admin.inc.php'))
            require_once(SGTUX_ROOT.'langues/'.$langue.'/'.$langue.'.admin.inc.php');
        else
            require_once(SGTUX_ROOT.'langues/fr/fr.admin.inc.php');

Dans le template (mysql, ou directement depuis la gestion des templates de l'admin si ça remarche ^^), change (première ligne) :

Code:

<form id="signawrite" action="<CHEMIN>valide.php" method="post"><div>

Par :

Code:

<form id="signawrite" action="<CHEMIN>valide.php<SGTUX_LANGUE>" method="post"><div>

Et dans class/affichage.class.php, ligne 328, remplace :

Code:

            // Chemin du fichier valide.php
            $tpl_panneau = str_replace('<CHEMIN>', SGTUX_ROOT, $tpl_panneau);

Par :

Code:

            // Chemin du fichier valide.php
            $tpl_panneau = str_replace('<CHEMIN>', SGTUX_ROOT, $tpl_panneau);
            if(isset($_GET['sgtux_lang']))
                $tpl_panneau = str_replace('<SGTUX_LANGUE>', '?sgtux_langue='.$_GET['sgtux_lang'], $tpl_panneau);
            else
                $tpl_panneau = str_replace('<SGTUX_LANGUE>', NULL, $tpl_panneau);

Enfin, pour ce qui est du lien mail, il y a confusion dans l'interprétation des options de l'admin.
« Links Web and Mail always on ... » est une option ambigue et un peu inutile : c'est une option spéciale maniaque. Si tu l'actives, tu auras les liens Mail et Web sur tous les messages. Pour les messages pour lesquels l'auteur n'a pas renseigné un des deux champs (ou les deux), ça affichera un petit message alerte lorsqu'on cliquera dessus. Ca permet juste d'obtenir exactement le m^eme esthétique pour tous les messages. Dans ton cas, tu devrais cocher « Non » à « Afficher un champ email » dans les options générales, la toute première.

Voilà, avec mes excuses pour les pb de code et en espérant que ça fonctionne,
et bravo pour l'intégration smile

EDIT/PS: Si tu testes mes modifs, pense à me dire si ça marche

Dernière modification par Ju (11/02/08 20:00:56)

Melody Nelson · 12/02/08 11:49:57

salut,

merci et merci (bis) pour ta réactivité

pour la modif du template, j'ai changé le code comme tu le proposes (si j'ai bien lu, c'était juste le nom du champ qui ne commençait pas par une majuscule)... après correction du code, je n'arrive toujours pas à mettre à jour le template depuis l'admin (mais pour moi, c'est pas dramatique)

je vais tester tes autres modifs plus tard et revenir donner des nouvelles

j'en profite pour faire part des modifs que j'ai faites pour adapter le livre à mes besoins multilingues, c'est amateur donc perfectible mais si ça pourrait être utile

dans signatux/class/affichage.class.php > ajout de cette ligne après de la ligne 329

Code:

$tpl_panneau = str_replace('<LANGUE_LIVRE>', '?sgtux_langue='.$_GET['sgtux_langue'], $tpl_panneau);

dans signatux/class/affichage.class.php > ligne 494 concernant les n° de pages :

avant :

Code:

$numeros_de_pages = $this->sgtux_generate_navigation(((is_null($chemin)) ? $this->page : $chemin).'sgtux=', $this->config['Page courante'], $this->config['Divers']['Messages par page']);
            $tpl_barreinfo = str_replace('[ELEMENT:NUMEROSDEPAGES /]', (($numeros_de_pages == 1) ? NULL : $numeros_de_pages), $tpl_barreinfo);

après :

Code:

$numeros_de_pages = $this->sgtux_generate_navigation(((is_null($chemin)) ? $this->page : $chemin).'sgtux_langue='.$_GET['sgtux_langue'].'&sgtux=', $this->config['Page courante'], $this->config['Divers']['Messages par page']);
            $tpl_barreinfo = str_replace('[ELEMENT:NUMEROSDEPAGES /]', (($numeros_de_pages == 1) ? NULL : $numeros_de_pages), $tpl_barreinfo);

dans signatux/admin/class/admin.class.php > ligne 179

avant :

Code:

$redirection = str_replace('<REDIRECTIONAUTO>', '<meta http-equiv="refresh" content="3; URL='.$retour1.'" />', $redirection);

après :

Code:

$redirection = str_replace('<REDIRECTIONAUTO>', '<meta http-equiv="refresh" content="3; URL='.$retour1.'?sgtux_langue='.$_GET['sgtux_langue'].'" />', $redirection);

dans signatux/admin/class/admin.class.php > ligne 194

avant :

Code:

$redirection = str_replace('<CHOIX>', '<a href="'.$retour1.'">'.$this->langueAdmin['Retour'].'</a>', $redirection);

après :

Code:

$redirection = str_replace('<CHOIX>', '<a href="'.$retour1.'&sgtux_langue='.$_GET['sgtux_langue'].'">'.$this->langueAdmin['Retour'].'</a>', $redirection);

dans signatux/admin/class/admin.class.php > ligne 199 :

avant :

Code:

$redirection = str_replace('<CHOIX>', '<a href="'.$retour1.'">'.$this->langueAdmin['Cliquez ici si vous ne voulez pas attendre'].'</a>', $redirection);

après

Code:

$redirection = str_replace('<CHOIX>', '<a href="'.$retour1.'?sgtux_langue='.$_GET['sgtux_langue'].'">'.$this->langueAdmin['Cliquez ici si vous ne voulez pas attendre'].'</a>', $redirection);

dans le template, je n'ai pas gardé le modèle original mais de mémoire, j'ai apporté cette modif :

Code:

<form id="signawrite" action="<CHEMIN>valide.php<LANGUE_LIVRE>" method="post">

et pour ne plus afficher l'email avec le lien, celle-ci :

Code:

<div class="sgtux_liens">&nbsp;</div>

j'avais voulu faire plus "propre" pour ne supprimer que "Mail" dans la barre au dessus du message mais dès que j'intervenais sur le code, j'avais ensuite des problèmes d'affichage... j'ai essayé plein de combinaisons en veillant bien à ne pas de laisser de balises ouvertes non fermées mais rien à faire... même le changement du <a href=...> provoquait des problèmes

comme tu me le suggères, j'ai pensé à un moment à enlever directement l'email en choisissant "non" pour l'option "afficher un champ email" mais j'ai préféré conserver cette option active

en fait, je veux quand même garder les emails des "posteurs" pour éventuellement leur répondre en privé (soit pour échanger, soit pour inciter à l'auto-modération), mais je ne souhaite pas rendre leurs emails publics

et c'est ce qui me dérangeait avec le lien "Mail"... je ne voulais pas que les "posteurs" aient l'impression qu'ils ne sont pas anonymes et puissent éventuellement recevoir des mails "parasites"... vu que le site touche un sujet "sensible", je préfère éviter les dérapages qui peuvent nuire à la vie privée des personnes...

je ne sais pas si mon explication est super claire, mais si une option complémentaire du genre "email public : oui/non" s'activait quand tu choisis "afficher un champ email", ça aurait répondu à mon besoin

ps : le livre d'or est officiellement ouvert depuis dimanche soir, j'espère que ça marchera... on a envie de rendre le site plus "interactif", donc hâte de voir si le livre d'or est utilisé

Annonces

#1 10/02/08 01:43:48

ravie de l'intégration de signatux

#2 10/02/08 12:17:11

Re: ravie de l'intégration de signatux

#3 11/02/08 19:44:19

Re: ravie de l'intégration de signatux

Code:

Code:

Code:

Code:

Code:

Code:

Code:

Code:

#4 12/02/08 11:49:57

Re: ravie de l'intégration de signatux

Code:

Code:

Code:

Code:

Code:

Code:

Code:

Code:

Code:

Code:

Code:

Pied de page du Forum