Evolution #25627
[-50] Modifier l'ajout d'urls à une app
100%
Description
Grâce à ta dernière intervention on a le libellé qui est bien sauvegardé
Cependant ce la pose quand même quelques pb :
- la meme chose se passe sur la description
- le gestionnaire peut modifier le libelle mais pas d'autres infos et c'est un peu perturbant
Je pense que le pb vient du controleur : sur AppControler,
if ($url->libelle != $post['url_libelle'][$id]) {
568 $url->libelle = $post['url_libelle'][$id];
569 $bChanged = true;
570 }
571
572 if ($url->description != $post['url_description'][$id]) {
573 $url->description = $post['url_description'][$id];
574 $bChanged = true;
575 }
A mon avis il faudrait faire un test sur la présence du post :
if ($post['url_libelle'][$id] && $url->libelle != $post['url_libelle'][$id]) {
...
if ($post['url_description'][$id] && $url->description != $post['url_description'][$id]) {
Je pense que l'on ne perdra plus le libellé
Dans ce cas on pourrait simplement afficher l'url et son libellé (mais pas dans un input) et ajouter un message en rouge :
NOUVELLE URL : cliquez d'abord sur "Appliquer" pour l'enregistrer. Vous pourrez ensuite modifier ses informations
Associated revisions
Fixes #25627 : Modifier l'ajout d'urls à une app
History
#1 Updated by Renaud Dussol over 4 years ago
Il faut faire ce test sur la présence du $post pour tous les champs car cela peut poser problème partout
Dans le cas des urls ajoutées, si les éléments ne sont pas présents en input, le test dans le controleur retournera toujours VRAI et la valeur sera updatée (avec potentiellement une valeur fausse)
Après au niveau ergonomie, il faut voir ce que l'on met dans le formulaire au moment de l'ajout (sachant qu'on ne pourra pas tout mettre, par exemple cela me semble excessivement compliqué d'ajouter en javscript à la volée l'ajout de l'icône)
Dans tous les cas je pense qu'il faut ajouter un petit message explicite à l'utilisateur pour l'avertir que s'il veut modifier d'autres infos liées à l'url il faut qu'il enregistre d'abord ("Appliquer")
- soit on fait un formulaire le plus complet possible : libellé, description, vue de l'icone (mais pas modif), catégorie, etc...
Pas trop trop compliqué, j'ai pu faire afficher l'icone et la description mais cela donne l'impression que l'on peut tout modifier... et si des choses changent il faudra revenir sur ce bout de JS
- Soit au contraire aucune info n'est modifiable (on enregistre juste l'url_id) et on affiche en rouge "ATTENTION, NOUVELLE URL, veuillez d'abord enregistrer pour modifier les infos"
- soit un entre-deux (proche de ce qu'on a actuellement) : on met juste le libellé modifiable (c'est ce qu'on change le + souvent) / éventuellement la description et on ajoute le message précédent
#2 Updated by Renaud Dussol over 4 years ago
- File ajoutnouvellesurls.png View added
- Status changed from Nouveau to En attente d'informations
- Assigned To changed from Renaud Dussol to Christophe LEON
J'ajoute une copie écran de ce à quoi ça pourrait ressembler
Je t'assigne la demande : si tu peux me dire ce que tu en penses ?
(tout ceci est parce que je vais bientôt déployer le app manager à plusieurs collègues, d'où le fameux rôle "gestionnaire", mais je souhaite simplifier au maximum, et je pense qu'ils vont beaucoup utiliser l'ajout manuel d'URLS car ils n'auront pas le motif)
Je souhaiterais aussi faire une autre modif, mais je ne sais pas si c'est possible/facile : que le passage par la liste des urls orphelines se fasse par la "vraie" liste (que l'on puisse voir les urls obsolètes, etc...)
#3 Updated by Christophe LEON over 4 years ago
Ok pour la capture et donc ta proposition.
Pour l'autre point il faut que je regarde un peu en détails.
#4 Updated by Renaud Dussol over 4 years ago
L'idée serait de ne plus surcharger app/edit par un modal de la liste des orphelines reconstruit, mais d'aller direct sur la liste des orphelines :
- soit par un frame (mais j'ai peur qu'il y ait des pb, je n'aime pas bcp ça)
- soit par un appel direct en passant l'id de l'app en param (mais cela peut poser pb dans le cas d'une création ou l'app n'a pas encore d'id, et d'autre part il faudrait forcer enregistrement des éventuelles modifs déjà faites sur l'app lors du clic sur "Ajouter" --- Cela dit pourquoi pas ?)
- soit par un jeu d'inclusion de template dans un autre : ce serait l'idéal mais je ne sais pas si twig le permet ?
#5 Updated by Renaud Dussol over 4 years ago
Je créerai une autre demande pour la 2eme partie (liste des urls orphelines)
#6 Updated by Renaud Dussol over 4 years ago
- Status changed from En attente d'informations to Résolu
- % Done changed from 0 to 100
Appliqué par commit 3ca5dd8852b0a4f73cc80d02c842570ee25dec91.
#7 Updated by Arnaud FORNEROT over 4 years ago
- Status changed from Résolu to Fermé