Development #23535
Gestion "sous-page automatique"
0%
Description
L'idée c'est de pouvoir créer une page, mettons /fiche-usager/, et de définir dans le paramétrage de celle-ci que sous /fiche-usager/ il faut "quelque chose" (a priori dans la même boite de dialogue que celle où on édite aujourd'hui le slug); ce "quelque chose" se trouverait ensuite posé dans le contexte, à disposition pour tout le monde. (et il pourrait y avoir un traitement particulier sur le <user_id> pour poser concerned_user).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 6 ans
- Lié à Development #19756: Personnalisation accrue du portail agent pour en faire aussi la page d'entrée des agents d'accueil ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-general-add-sub-slug-to-create-variable-pages-23535.patch 0001-general-add-sub-slug-to-create-variable-pages-23535.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ça ajoute un attribut "sub slug" aux pages. (traduction à discuter)
Il prend alors une regex avec un groupe nommé, genre (?P<blah>[a-z]+). Et l'info se retrouve ensuite dans le contexte de rendu.
Exemple, une page /plop/ avec sub slug à (?P<blah>[a-z]+), aller sur /plop/whatever/ mettra dans le contexte "blah" avec comme valeur "whatever".
Au-dessus de ça il y a quatre lignes pour reconnaitre user_id et name_id, et dans ces situations taper l'utilisateur ainsi trouvé dans le contexte dans "selected_user" (pour compatibilité avec ce qu'on avait déjà).
La page peut elle-même contenir des sous-pages, genre "toto" comme sous-page de "plop", aller sur /plop/whatever/toto/ fera le rendu de la page "toto", avec également whatever dans le contexte. (l'idée étant ainsi de pouvoir faire des sous-pages de la fiche usager.
Le comportement actuel sur un accès à une page demandant un sub slug quand l'info manque de l'url, exemple /plop/, c'est simplement une 404. (j'imaginais à un moment altérer le comportement de redirect_url, mais ça fait un truc bizarre, je me dis que mon option préférée s'il fallait autre chose qu'une 404, ce serait une redirection vers la page parente.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Lié à Development #8521: Autoriser plusieurs pages à avoir le même slug ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
Le comportement actuel sur un accès à une page demandant un sub slug quand l'info manque de l'url, exemple /plop/, c'est simplement une 404. (j'imaginais à un moment altérer le comportement de redirect_url, mais ça fait un truc bizarre, je me dis que mon option préférée s'il fallait autre chose qu'une 404, ce serait une redirection vers la page parente.
Pour avoir continué à hésiter, je viens de pousser une branche avec comme comportement la redirection vers la page parente; ça me semble le plus pratique.
Mis à jour par Thomas Noël il y a plus de 5 ans
Il y a aussi l'éventuelle possibilité d'une info vide genre /plop//toto, mais le parts = [x for x in request.path_info.strip('/').split('/') if x] mange les // et on va se retrouver avec toto comme valeur pour le sub slug, pas bon. A priori ça serait bien de retirer le "if x" dans la gestion de parts, c'est à dire avoir juste parts = request.path_info.strip('/').split('/') mais je n'ai pas encore bien mesuré l'impact, ie à quoi sert le "if x".
Mis à jour par Thomas Noël il y a plus de 5 ans
Le "if x" a toujours été là... pour moi il n'est pas très pertinent... ou bien je rate une idée ?
Mis à jour par Frédéric Péters il y a plus de 5 ans
Le "if x" a toujours été là... pour moi il n'est pas très pertinent... ou bien je rate une idée ?
Peut-être bien qu'on a des liens avec par erreur // et ça permet d'assurer quand même que la page soit servie. De mon côté je ne toucherais pas à ça et continuerait à considérer que // comme / (je viens de vérifier et côté filesystem ça se passe comme ça).
Mis à jour par Thomas Noël il y a plus de 5 ans
Ok, le //// à l'intérieur ne compte pas (dans les fs je savais, pas sur http, on découvre tous les jours, tant mieux). Laissons donc ainsi.
Je serais pour afficher le sub slug dans combo/manager/templates/combo/page_view.html s'il existe, pour ne pas avoir à cliquer pour le voir.
Pas trop d'idée non plus pour un help_text, mais ça serait bien, histoire qu'on ne nous demande pas "c'est quoi ce truc ?".
Donc en dehors de ces petits trucs d'UI, ça me semble tout bien (mais je dois relire wip/guichet dans son ensemble)
Mis à jour par Frédéric Péters il y a plus de 5 ans
Je serais pour afficher le sub slug dans combo/manager/templates/combo/page_view.html s'il existe, pour ne pas avoir à cliquer pour le voir.
Fait.
Pas trop d'idée non plus pour un help_text, mais ça serait bien, histoire qu'on ne nous demande pas "c'est quoi ce truc ?".
J'ai tapé un truc un peu technique quand même, "Regular expression to create variadic subpages. Matching named groups are exposed as context variables." mais ça gagnerait surtout à une page de documentation détaillant ça.
Mis à jour par Thomas Noël il y a plus de 5 ans
Ici on devrait faire des try/except pour ne pas crasher si l'utilisateur n'existe pas (User.DoesNotExist) , ou si le ctx['user_id'] ne représente pas un nombre (ValueError) :
def modify_global_context(request, ctx):
if 'user_id' in ctx:
ctx['selected_user'] = User.objects.get(id=ctx['user_id'])
if 'name_id' in ctx and UserSAMLIdentifier:
ctx['selected_user'] = UserSAMLIdentifier.objects.get(name_id=ctx['name_id']).user
deviendrait quelque chose comme ça :
def modify_global_context(request, ctx):
if 'user_id' in ctx:
try:
ctx['selected_user'] = User.objects.get(id=ctx['user_id'])
except (User.DoesNotExist, ValueError):
pass
if 'name_id' in ctx and UserSAMLIdentifier:
try:
ctx['selected_user'] = UserSAMLIdentifier.objects.get(name_id=ctx['name_id']).user
except UserSAMLIdentifier.DoesNotExist:
pass
Bon, le ValueError pourrait être géré via une regex correct slug, mais au moins les DoesNotExist
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Ack, avec ou sans cet ajout en fin de tests/test_public.py::test_sub_slug si tu veux :
# bad user id => no selected_user
page2.sub_slug = '(?P<user_id>[0-9a-z]+)'
page2.save()
resp = app.get('/users/9999999/', status=200)
assert 'XXYY' in resp.text
resp = app.get('/users/abc/', status=200)
assert 'XXYY' in resp.text
# custom behaviour for <name_id>, it will add the SAML user to context
with mock.patch('combo.public.views.UserSAMLIdentifier') as user_saml:
class DoesNotExist(Exception):
pass
user_saml.DoesNotExist = DoesNotExist
def side_effect(*args, **kwargs):
name_id = kwargs['name_id']
if name_id == 'foo':
raise user_saml.DoesNotExist
return mock.Mock(user=john_doe)
mocked_objects = mock.Mock()
mocked_objects.get = mock.Mock(side_effect=side_effect)
user_saml.objects = mocked_objects
page2.sub_slug = '(?P<name_id>[0-9a-z.]+)'
page2.save()
resp = app.get('/users/john.doe/', status=200)
assert 'XXjohn.doeYY' in resp.text
# unknown name id => no selected_user
resp = app.get('/users/foo/', status=200)
assert 'XXYY' in resp.text
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
J'ai ajouté ces tests et poussé,
commit cae8d0cda594b7fefae762b5ae56dd10612d89e3
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Aug 8 22:18:36 2018 +0200
general: add "sub slug" to create variable pages (#23535)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
general: add "sub slug" to create variable pages (#23535)