Le 7 mars 10 à 04:02, kent1@arscenic.info a écrit :
Author: kent1@arscenic.info
Date: 2010-03-07 16:02:14 +0100 (Sun, 07 Mar 2010)
New Revision: 35902
Added:
_plugins_/contact/formulaires/contact_champ_contact_texte.html
_plugins_/contact/prive/contenu/
_plugins_/contact/prive/contenu/message_contact.html
_plugins_/contact/prive/infos/
_plugins_/contact/prive/infos/message_contact.html
Removed:
_plugins_/contact/formulaires/contact_champ_texte.html
Modified:
_plugins_/contact/action/supprimer_message.php
_plugins_/contact/contact_options.php
_plugins_/contact/exec/contact_un_message.php
_plugins_/contact/formulaires/config_contact.html
_plugins_/contact/formulaires/contact.html
_plugins_/contact/formulaires/contact.php
_plugins_/contact/lang/contact_fr.php
_plugins_/contact/plugin.xml
_plugins_/contact/prive/style_prive_plugin_contact.html
Log:
Pas mal de modifications (un tag a été fait sur la version 0.61 pour pouvoir revenir en arrière facilement). Au menu :
- On change l'id du champs texte pour éviter d'avoir porte plume par défaut dessus dans les versions récentes (il s'applique à tous les champs de formulaire public avec l'id "texte") et pour éviter les doubles id "texte" dans les squelettes également au cas où
- Si on utilise des raccourcis SPIP dans les messages, le message enregistré en base est le message original (avec les raccourcis) puis on traite le message pour l'envoyer proprement par mail
- La date dans le message est correctement internationnalisée en utilisant les fonctions spip adéquates
- On passe la page de visualisation des messages du privé un peu plus SPIP style
- On supprime les documents attachés à la suppression des message
- On passe en version 0.62
Details: Connexion · GitLab
Bravo pour tout sauf le changement de texte en contact_texte qui fout un super merdier dans la base installés.
Si le but est de ne pas afficher la barre par défaut une fois le plugin configuré à minima, il n'est pas utile depuis hier de modifier quoique ce soit puisque c'est comme cela que ça fonctionne (pas de barre) en 2.1 qui devrait aussi fonctionner sur la 2 :
<input type="radio" name="barre" id="no_b" class="radio" value="no_barre"[ checked="checked"(#ENV{barre}|=={no_barre}|ou{[(#CONFIG{contact/barre}|non)]}|oui)] />
pierre
pj je fais un correctif à moins qu'il y une opposition farouche 
Le 7 mars 2010 19:35, Pierre Fiches <pierre.fiches@free.fr> a écrit :
Bravo pour tout sauf le changement de texte en contact_texte qui fout un super merdier dans la base installés.
Qu’est ce qu’un merdier dans la base installée? le seule changement sera peut être au niveau de la conf mais je vois pas le rapport avec la « base » … tu as des précisions?
Si le but est de ne pas afficher la barre par défaut une fois le plugin configuré à minima, il n’est pas utile depuis hier de modifier quoique ce soit puisque c’est comme cela que ça fonctionne (pas de barre) en 2.1 qui devrait aussi fonctionner sur la 2 :
<input type=« radio » name=« barre » id=« no_b » class=« radio » value=« no_barre »[ checked=« checked »(#ENV{barre}|=={no_barre}|ou{[(#CONFIG{contact/barre}|non)]}|oui)] />
il n’y a pas que cela …
c’est un id=« texte » du genre ultra utilisé facilement dans les squelettes pour d’autres trucs …
et donc il y a de fortes probabilités qu’il y ait 2 fois le même id dans les pages.
D’autre part, la configuration directe à l’installation doit être directement sans barre sans configuration, alors que porte-plume vise automatiquement tous les textareas avec l’id texte…
Pour la conf, la solution serait de faire une mise à jour de la meta avec les bons identifiants… pour la « base » je ne vois toujours pas où est le problème
pierre
kent1
pj je fais un correctif à moins qu’il y une opposition farouche
Le 7 mars 10 à 21:20, Quentin Drouet a écrit :
Le 7 mars 2010 19:35, Pierre Fiches <pierre.fiches@free.fr> a écrit :
Bravo pour tout sauf le changement de texte en contact_texte qui fout un super merdier dans la base installés.
Qu’est ce qu’un merdier dans la base installée? le seule changement sera peut être au niveau de la conf mais je vois pas le rapport avec la « base » … tu as des précisions?
Si le but est de ne pas afficher la barre par défaut une fois le plugin configuré à minima, il n’est pas utile depuis hier de modifier quoique ce soit puisque c’est comme cela que ça fonctionne (pas de barre) en 2.1 qui devrait aussi fonctionner sur la 2 :
<input type=« radio » name=« barre » id=« no_b » class=« radio » value=« no_barre »[ checked=« checked »(#ENV{barre}|=={no_barre}|ou{[(#CONFIG{contact/barre}|non)]}|oui)] />
il n’y a pas que cela …
c’est un id=« texte » du genre ultra utilisé facilement dans les squelettes pour d’autres trucs …
et donc il y a de fortes probabilités qu’il y ait 2 fois le même id dans les pages.
D’autre part, la configuration directe à l’installation doit être directement sans barre sans configuration, alors que porte-plume vise automatiquement tous les textareas avec l’id texte…
Pour la conf, la solution serait de faire une mise à jour de la meta avec les bons identifiants… pour la « base » je ne vois toujours pas où est le problème
par base installée j’entendais l’ensemble des sites qui utilisent le plugin.
Après réflexion plus appronfie ou pourrait changer un name mais il faut penser à gérer les modifications de meta en paralelle.
La gestion de la barre est possible sans aucune config avec un truc du genre $champs_mini_config dans formulaire/contact.php.
Mais ce dispositif est surtout la pour réduire le sad.
pierre