Projet

Général

Profil

Bug #23942

Évaluation de la première page d'une demande initiée depuis welco

Ajouté par Frédéric Péters il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
20 mai 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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

Révision 3d79ca4c (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

misc: restore backoffice drafts to the correct page (#23942)

Historique

#2

Mis à jour par Frédéric Péters il y a presque 6 ans

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.

#3

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)

#5

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)
#6

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.

#7

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

Formats disponibles : Atom PDF