Suite à ce topic pb : Ré-éditer un commentaire j'ai créé le plugin CommentEditor :
[extension by Criss] CommentEditor
Il permet d'éditer un commentaire déjà posté. Un admin peut éditer tous les commentaires (1 par 1, pas d'edit de masse) et changer l'auteur si nécesaire. L'auteur du commentaire peut éditer son propre commentaire.
Un lien "Edit" est rajouté à droite de la date du commentaire dans la page "Comments" (au moins) mais il n'est pas disponible pour les utilisateurs, seulement pour l'admin. Si l'utilisateur connait l'adresse du lien il peut éditer son commentaire tout de même.
Hors ligne
Salut,
J'ai installé le plugin sur le thème montblanc que j'utilise et ça fonctionne au poil. J'ai juste une petite question, pourquoi, au moment de l'édition nous sommes ramené que la page d'accueil pour faire la modif ? ne pourrions nous pas resté sur la page de l'image dont nous éditons le commentaire ?
J'ai bien vue que c'était une première version et c'est déjà super d'avoir ce niveau de possibilité et mon commentaire n'ai là que pour potentiellement alimenter les tablettes dédié aux évolutions potentiel ;o))
Encore bravo pour plugin fort sympa et que je vais suivre en utilisateur et testeur.
Bonne journée.
Seb
Hors ligne
Pourquoi ramené sur l'index ? Parce que c'était le plus simple à gérer. :P
Je rappelle que je ne sais pas utiliser jQuery donc le faire directement sur le commentaire c'est pas pour de suite.
Quant à garder la page je vais voir si c'est facile. Je vois déjà pointer un conflit sur le paramètre "action" :P
Hors ligne
Merci pour ta réponse je me doutais que c'était pour faire au plus simple ;o)) donc pour pour moi pas de panique c'est déjà super comme ça et comme nous ne sommes pas censé en corriger 200 par jour il reste installé et utilisable ;o))
Bonne journée.
Seb
Hors ligne
En cas d'erreur un bloc message est affiché sur l'accueil.
En cas de confirmation que la modification est prise en compte, un bloc message est affiché sur l'accueil.
Pour vous donner une idée du bloc, il suffit de choiri un lien comme galerie/index.php?action=update_comment
(j'ai honteusement récupéré les images infos.png et errors.png présents en admin...)
J'envisage de rendre ceci configurable.
Hors ligne
Il manque le fichier maintain.inc.php qui permet de gérer installation/désinstallation/activation/désactivation du plugin.
Hors ligne
Bonjour,
Excellente idée ce plugin, je me suis empressé de l'installer et de l'activer ;-)
J'ai donc testé, et après édition d'un commentaire, voici les lignes qui s'affichent en haut de page :
===============================================================
Notice: Undefined variable: userList in /homez.52/montagne/www/ventoux-photos/plugins/extension_296/classes/ce_comment.class.php on line 53
Warning: array_change_key_case() [function.array-change-key-case]: The argument should be an array in /homez.52/montagne/www/ventoux-photos/plugins/extension_296/classes/ce_comment.class.php on line 53
Warning: array_key_exists() [function.array-key-exists]: The second argument should be either an array or an object in /homez.52/montagne/www/ventoux-photos/plugins/extension_296/classes/ce_comment.class.php on line 53
Warning: Cannot modify header information - headers already sent by (output started at /homez.52/montagne/www/ventoux-photos/plugins/extension_296/classes/ce_comment.class.php:53) in /homez.52/montagne/www/ventoux-photos/include/page_header.php on line 98
===============================================================
La modification est bien prise en compte :-)
Autre petit soucis, un utilisateur enregistré ayant posté un commentaire ne peut pas le modifier contrairement à ce qui est indiqué dans la description.
Quoiqu'il en soit, je laisse le plugin en place, car à priori, ça n'empêche pas le bon focntionnement de cet excellent script de galerie.
Grand bravo à l'équipe ;-)
Cdlt
Laurent
Hors ligne
laurentg a écrit:
Bonjour,
Excellente idée ce plugin, je me suis empressé de l'installer et de l'activer ;-)
J'ai donc testé, et après édition d'un commentaire, voici les lignes qui s'affichent en haut de page :
===============================================================
Notice: Undefined variable: userList in /homez.52/montagne/www/ventoux-photos/plugins/extension_296/classes/ce_comment.class.php on line 53
Warning: array_change_key_case() [function.array-change-key-case]: The argument should be an array in /homez.52/montagne/www/ventoux-photos/plugins/extension_296/classes/ce_comment.class.php on line 53
Warning: array_key_exists() [function.array-key-exists]: The second argument should be either an array or an object in /homez.52/montagne/www/ventoux-photos/plugins/extension_296/classes/ce_comment.class.php on line 53
Warning: Cannot modify header information - headers already sent by (output started at /homez.52/montagne/www/ventoux-photos/plugins/extension_296/classes/ce_comment.class.php:53) in /homez.52/montagne/www/ventoux-photos/include/page_header.php on line 98
===============================================================
La modification est bien prise en compte :-)
VDigital a aussi vu ce soucis, j'ai oublié un use case de test. Je tâche de corriger ASAP.
Autre petit soucis, un utilisateur enregistré ayant posté un commentaire ne peut pas le modifier contrairement à ce qui est indiqué dans la description.
Quoiqu'il en soit, je laisse le plugin en place, car à priori, ça n'empêche pas le bon focntionnement de cet excellent script de galerie.
Grand bravo à l'équipe ;-)
Cdlt
Laurent
Je sais bien que l'auteur "semble ne pas pouvoir" éditer son commentaire, en fait juste il n'a pas le lien ;)
C'est ce que j'ai mis dans le premier post de ce topic : si l'utilisateur connait l'adresse du lien il peut éditer son commentaire tout de même.
C'est une limitation du moment et je ne sais pas comment la contourner...
Hors ligne
nicolas a écrit:
Il manque le fichier maintain.inc.php qui permet de gérer installation/désinstallation/activation/désactivation du plugin.
Je sais mais ne faisant aucune action spécifique (ni modification de BdD ou autre) j'estimais qu'il n'était pas nécessaire.
Après je peux en mettre un s'il faut, ça ne me pose aucun soucis.
Hors ligne
Il faut voir ce qu'il y a dans la log php mais je pense que cela ne pose pas de problème.
(Cela me donne une idée que je vais aller expliquer à l'équipe.)
Hors ligne
Je pensais qu'il était facultatif puisque non créé dans le PersonalPlugin :P
Hors ligne
Sans maintain.inc.php, on ne peut pas désactiver le plugin sans le supprimer par exemple. Je dis ça sans tester; ni regarder le code.
Hors ligne
Criss a écrit:
La modification est bien prise en compte :-)
VDigital a aussi vu ce soucis, j'ai oublié un use case de test. Je tâche de corriger ASAP.
Tu gères tes tests à la main ou via un framework de tests ? Si oui il serait peut-être intéressant de publier les tests dans svn en faisant un répertoire src et un répertoire test par exemple.
Hors ligne
nicolas a écrit:
Tu gères tes tests à la main ou via un framework de tests ? Si oui il serait peut-être intéressant de publier les tests dans svn en faisant un répertoire src et un répertoire test par exemple.
Euh non je teste tout à la mimine, je n'ai pas poussé le vice à me créer un framework de test. :P
Je me donne juste une forme de checklist.
Version 1.0.c publiée. Le bug n'était pas vraiment lié au fait que ce soit un guest qui poste mais juste une type de nom de variable. Au passage il est fortement recommandé de mettre à jour si le plugin est activé et que vous souhaitez accéder à la partie admin.php?page=configuration§ion=comments (joli timeout sinon...)
Hors ligne
Criss a écrit:
nicolas a écrit:
Tu gères tes tests à la main ou via un framework de tests ? Si oui il serait peut-être intéressant de publier les tests dans svn en faisant un répertoire src et un répertoire test par exemple.
Euh non je teste tout à la mimine, je n'ai pas poussé le vice à me créer un framework de test. :P
Je me donne juste une forme de checklist.
Ok. C'était juste pour savoir et pour tenter de convaincre mes petits camarades du bon usage d'un framework de tests genre simpletest.
Tiens deux nouveaus tests :
- index.php?action=edit_comment&comment_id=UNE_CHAINE_QUELCONQUE (provoque une erreur moche !!)
- index.php?action=edit_comment&comment_id=UN_ID_NON_EXISTANT (pas d'erreur mais si on soumet le formulaire, on n'a une erreur moche !)
Hors ligne