Project

General

Profile

Anomalie #22779

[-50] L'ajout manuel d'URLS à une application entraîne la perte du libellé de l'URL

Added by Renaud Dussol over 5 years ago. Updated over 4 years ago.

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

100%

Distribution:

Description

Pour reproduire :

- Créer une nouvelle application ou éditer une application existante
- Dans l'onglet "Urls", cliquer sur "Ajouter" et ajouter des urls qui sont présentes dans la liste des urls orphelines
- On constate que les urls sont bien ajoutées mais que le libellé original de l'URL est perdu

Associated revisions

Revision 819512cd (diff)
Added by Christophe LEON over 4 years ago

fixes #22779 / L'ajout manuel d'URLS à une application entraîne la perte du libellé de l'URL

History

#1 Updated by Christophe LEON almost 5 years ago

  • Status changed from Nouveau to A étudier

#2 Updated by Christophe LEON over 4 years ago

  • Subject changed from L'ajout manuel d'URLS à une application entraîne la perte du libellé de l'URL to [-48] L'ajout manuel d'URLS à une application entraîne la perte du libellé de l'URL
  • Priority changed from Haut to Normal

#3 Updated by Renaud Dussol over 4 years ago

Je vais essayer de traiter cette demande

Je remarque que si on passe par urls orphelines --> créer, cela fonctionne bien

Dans AppController.php, le cas est prévu à createAppAction
// Si on précise des urls en paramètre, on va essayer de consctruire une nouvelle
// application avec celle-ci

Le pb survient uniquement si on passe par la création ou l'édition d'une application (c.à.d. par app/edit)
Car dans ce cas il surcharge la vue par la liste des urls orphelines (qui du coup d'ailleurs n'est pas à jour : pas de flag obsolete par exemple)

Cela doit être la fonction validUrl dans le JS qui , dans son "append" ne prend pas toutes les infos...

J'ai essayé de rajouter :
<td> ${$(this).attr('data-libelle')} </td>
sur le même modèle que data-url
Cela affiche bien le libellé dans le tableau, mais si j'enregistre cela ne conserve pas le libellé

Avant d'aller plus loin, je propose une chose :

- On supprime la fonction "Ajouter des urls" de l'édition d'une app

Et on la remplace par la mention : pour ajouter des urls existantes à cette application, veuillez les sélectionner dans la liste des urls orphelines (avec un lien dessus, dans lequel on pourra éventuellement passer en param l'ID de l'app)

Dans la listes des urls orphelines, on ajoute en plus du bouton "créer une app" un nouveau bouton "Ajouter à une app existante"

Ainsi les id des urls sont bien passés en paramètres et là il n'y a plus de problème

En plus on évite cette surcharge de edit.html.twig

#4 Updated by Renaud Dussol over 4 years ago

Sinon, autre possibilité :

on laisse le bouton "Ajouter des urls" dans app/edit, mais on en fait un simple lien vers urls/orphan avec l'ID de l'app en paramètre

Dans le template des ursl orphelines, si un ID d'url est passé en param, on remplace le bouton "Créer une app" par "valider"

Il faut que cet ID soit conservé lorsqu'on valide (on aura donc 2 params : un monoévalué qui sera l'ID de l'app et un multiévalué qui sera la liste des ID des urls que l'on aura choisies)

#5 Updated by Renaud Dussol over 4 years ago

  • Subject changed from [-48] L'ajout manuel d'URLS à une application entraîne la perte du libellé de l'URL to [-50] L'ajout manuel d'URLS à une application entraîne la perte du libellé de l'URL
  • Priority changed from Normal to Haut
  • % Done changed from 0 to 10

Il faudra faire aboutir cette demande rapidement car je pense que je vais retirer le motif au rôle gestionnaire, car trop destructif si mal utilisé

Du coup ils utiliseront sans doute plus souvent l'ajout manuel

Il faut qu'on en discute à la rentrée

#6 Updated by Anonymous over 4 years ago

  • Status changed from A étudier to Résolu
  • % Done changed from 10 to 100

#7 Updated by Christophe LEON over 4 years ago

  • Target version set to Envole 5.12

#8 Updated by Arnaud FORNEROT over 4 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF