Bug #23942
Évaluation de la première page d'une demande initiée depuis welco
0%
Description
Page 1, une condition : form_submission_channel != "counter" and is_in_backoffice, Page 2, whatever.
On initie une demande depuis welco interface guichet, on arrive sur un écran qui affiche Page 2 mais reprend les champs de la page 1.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 6 ans
- Fichier 0001-misc-restore-backoffice-drafts-to-the-correct-page-2.patch 0001-misc-restore-backoffice-drafts-to-the-correct-page-2.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Deux points ici, 1) un brouillon directement posé dans le db via l'API avait page_no == 0, et comme page_no était positionné, il n'était pas recalculé. Et 2) l'évaluation de la page de démarrage se faisait lors de l'ouverture de ce brouillon sans connaitre les données de la demande, et donc avec form_submission_channel pas/mal défini.
Mis à jour par Thomas Noël il y a presque 6 ans
int(data.get('page_no', '0') or '0') != 0:
ne m'est pas super lisible, data.has_key('page_no') and int(data['page_no']) != 0
est plus clair pour moi, mais bon, question de goût.
Dans le test, ces lignes peuvent être supprimées (reste de copier coller de test_backoffice_submission):
resp = app.get('/backoffice/') assert not 'Submission' in resp.body app.get('/backoffice/submission/', status=403)
Ack avec ces adaptations, ou pas (surtout la première, tu choisis)
Mis à jour par Frédéric Péters il y a presque 6 ans
- Statut changé de En cours à Résolu (à déployer)
J'ai fait 'page_no' in data plutôt que le .has_key.
commit 3d79ca4cbc1363900075964fac15a2150985dab7 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sun May 20 19:20:31 2018 +0200 misc: restore backoffice drafts to the correct page (#23942)
Mis à jour par Frédéric Péters il y a presque 6 ans
Christophe, oui, si on considère que data ne peut pas avoir un page_no explicitement positionné mais à None; ce qui est aussi une situation que le code posé par Thomas ignore. Sur des idées d'aventure, j'ai décidé de l'ignorer aussi.
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
misc: restore backoffice drafts to the correct page (#23942)