Development #64981
interdire certains slugs aux vues personnalisés
0%
Description
Genre "ics" qui clashe avec la vue existante.
Dans ensure_slug, prendre les _q_exports de CardPage et FormPage, ajouter "ics", mettre tout ça dans existing_slugs.
Fichiers
Révisions associées
Historique
Mis à jour par Corentin Séchet il y a presque 2 ans
- Fichier 0001-backoffice-prevent-custom-views-and-ics-CardPage-vie.patch 0001-backoffice-prevent-custom-views-and-ics-CardPage-vie.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
J'ai repris ce que tu avais déjà fait pour les FormPage, en ajoutant les _q_export de CardPage et ics. Il y avait déjà un test en place, je n'ai pas ajouté de test pour toutes les urls possibles, je ne sais pas si c'est nécessaire.
Mis à jour par Frédéric Péters il y a presque 2 ans
Ah trop bien je n'avais pas regardé le code...
Pour les tests sur un truc comme ça le plus facile c'est de regarder le commit qui a ajouté la vérification précéente (#53575) pour trouver le test qui lui était associé, (alternativement utiliser la vue "coverage report (native)" dans jenkins, qui pour les lignes de code peut pointer les tests).
Et ajouter une petite variation, ici ça pourrait être :
# check slug not created with view name
- resp = app.get('/backoffice/management/form-title/')
- resp.forms['listing-settings']['user-label'].checked = False
- resp = resp.forms['listing-settings'].submit()
- resp.forms['save-custom-view']['title'] = 'Export'
- resp.forms['save-custom-view']['visibility'] = 'any'
- resp = resp.forms['save-custom-view'].submit()
- assert resp.location.endswith('/x-export/')
- resp = resp.follow()
+ for view_title in ('Export', 'ics'):
+ resp = app.get('/backoffice/management/form-title/')
+ resp.forms['listing-settings']['user-label'].checked = False
+ resp = resp.forms['listing-settings'].submit()
+ resp.forms['save-custom-view']['title'] = view_title
+ resp.forms['save-custom-view']['visibility'] = 'any'
+ resp = resp.forms['save-custom-view'].submit()
+ assert resp.location.endswith('/x-%s/' % view_title.lower())
+ resp = resp.follow()
(pas testé).
Mis à jour par Corentin Séchet il y a presque 2 ans
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Corentin Séchet il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
Mis à jour par Corentin Séchet il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution validée
Mis à jour par Corentin Séchet il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3b5e97f94ae2548151ada966b21c032841aaecca
Author: Corentin Séchet <csechet@entrouvert.com>
Date: Mon May 9 16:22:16 2022 +0200
backoffice: prevent custom views and ics & CardPage views slug conflict (#64981)Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
backoffice: prevent custom views and ics & CardPage views slug conflict (#64981)