Project

General

Profile

Evolution #25627

[-50] Modifier l'ajout d'urls à une app

Added by Renaud Dussol over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Haut
Assigned To:
Target version:
Start date:
10/19/2018
Due date:
% Done:

100%

Distribution:

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

ajoutnouvellesurls.png View (32 KB) Renaud Dussol, 11/06/2018 04:43 PM

Associated revisions

Revision 3ca5dd88 (diff)
Added by Renaud Dussol over 1 year ago

Fixes #25627 : Modifier l'ajout d'urls à une app

History

#1 Updated by Renaud Dussol over 1 year 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 1 year 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 1 year 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 1 year 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 1 year ago

Je créerai une autre demande pour la 2eme partie (liste des urls orphelines)

#6 Updated by Renaud Dussol over 1 year ago

  • Status changed from En attente d'informations to Résolu
  • % Done changed from 0 to 100

#7 Updated by Arnaud FORNEROT over 1 year ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF