Disturbing edit item form

[ghost]

The edit item form is disturbing, there at least 2 issues:

• when you edit without changes, the item status is still changed to waiting moderation.
• when the image is not changed, you always got the message "The item failed to be saved"
• Edit: even if the image is changed, you got the failed message (because saveImage() will return null)

I'm working on it !

[FlorianLB]

Pour le 1er point on peut pas y faire grand chose, le workflow de validation est comme ça, on a pas de retour sur quels champs ont étés modifiés. A mon avis faudrait rework tout ça, mais c'est un gros gros chantier.

Pour le 2eme point c'est vachement plus gênant effectivement...

[ghost]

Pour le 1er point, je pensait utiliser http://docs.jelix.org/fr/manuel-1.3/jforms/apres-submit#connaitre-les-valeurs-qui-ont-ete-modifiees et ça éviterai aux admins du travail supplémentaire, je vais quand même regarder si c'est possible. Oui pour le 2eme, c'est très troublant en tant qu'utilisateur :p

[foxmask]

pour le premier on peut c'est une option qu'on utilise avec parcimonie http://jelix.localhost/articles/fr/manuel-1.3/jforms/apres-submit "Connaitre les valeurs qui ont été modifiées"

[FlorianLB]

Oui c'est clairement ce qu'il faut utiliser mais quitte à l'utiliser, il faudrait que ce soit tout le workflow qui en tire partie. Du genre : on ne voie dans la partie admin que les champs modifiés avec anciennes/nouvelles valeurs. Mais ça nécessite de changer le schéma en base.

On aurait une table du genre _modificationitems : item_id / old_value / new_value / date Et dans l'admin on appliquerait ou pas les changements. Mais bon, c'est un gros gros chantier du coup.

[ghost]

Ah oui carrément ! Mais dans un premier temps, est-ce que on peut juste faire en sorte de voir si il y a eu des valeurs modifiées dans le formulaire et si c'est pas le cas, simplement afficher un message ('Item non modifié') ?
Sinon, pour la gestion des erreurs côté serveur lors de l'upload de l'image (puisqu'elle aura déjà passé le $form->check()), je pense un message du genre (Les changements ont été appliqué etc mais il y a eu un problème lors de l'enregistrement de l'image, les admins en ont été informés) et les admins seront informés via les logs (a priori ça voudrait dire qu'il y a eu une erreur php) Ducoup je vais faire 2 patch, un pour le plus urgent, et l'autre pour les controls modifiés :)

[FlorianLB]

Je pense pouvoir faire le rework cette semaine donc te préoccupe pas trop des controls modifiés.

Le 1 mai 2012 18:51, catsoup < reply@reply.github.com

a écrit :

Ah oui carrément ! Mais dans un premier temps, est-ce que on peut juste faire en sorte de voir si il y a eu des valeurs modifiées dans le formulaire et si c'est pas le cas, simplement afficher un message ('Item non modifié') ? Sinon, pour la gestion des erreurs côté serveur lors de l'upload de l'image (puisqu'elle aura déjà passé le $form->check()), je pense un message du genre (Les changements ont été appliqué etc mais il y a eu un problème lors de l'enregistrement de l'image, les admins en ont été informés) et les admins seront informés via les logs (a priori ça voudrait dire qu'il y a eu une erreur php) Ducoup je vais faire 2 patch, un pour le plus urgent, et l'autre pour les controls modifiés :)

---

Reply to this email directly or view it on GitHub: https://github.com/foxmask/Booster/issues/46#issuecomment-5442633

[foxmask]

AMHA @catsoup tu peux reprendre le cours de ce ticket parce que @FlorianLB ne semble plus s'occuper du projet :/

[ghost]

ok :s

[ghost]

@foxmask, désolé pas trop de temps récemment, un déménagement + nouveau projet sur Symfony2 TT au taf, enfin, bientôt je m'y remet, peut être en faisant une migration vers la 1.5 si c'est possible côté prod ?

[foxmask]

@catsoup t'inquiete pas - rien presse.