Project

General

Profile

Bug #37709

Affichage champ commentaire dans la page de validation quand il est seul seur la page

Added by Frédéric Péters 2 months ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
18 Nov 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

La case "Page de récapitulatif" a beau être cochée, quand le champ commentaire est l'unique champ de la page, celle-ci n'est pas reprise sur l'écran de validation.

0001-forms-consider-form-pages-with-only-inert-fields-in-.patch View (6.35 KB) Frédéric Péters, 08 Dec 2019 03:46 PM

Associated revisions

Revision fb0632d0 (diff)
Added by Frédéric Péters about 1 month ago

forms: consider form pages with only inert fields in validation pages (#37709)

History

#1 Updated by Frédéric Péters about 1 month ago

En variation, quand une page reprend uniquement des champs configurés pour ne pas être repris sur la page de validation, le titre de la page apparaissait quand même. Du coup, reprise de cette fonction, en deux temps, commencer par assembler les champs à considérer puis seulement faire l'affichage.

#2 Updated by Benjamin Dauvergne about 1 month ago

Je ne comprends pas quels noeuds ce code (tout à la fin) ferme :

            if visible_contents:
                form.widgets.append(HtmlWidget(htmltext('</div></div>')))

Toutes les <div> précédemment ouvertes sont fermées ou alors est-ce une interaction avec field.add_to_view_form() ?

#3 Updated by Frédéric Péters about 1 month ago

Ils ferment les <div> ouverts pour une page, en début de boucle,

                form.widgets.append(HtmlWidget(htmltext('<div class="page">')))
                form.widgets.append(HtmlWidget(htmltext('<div>')))

#4 Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Solution proposée to Solution validée

Ok problème d'yeux.

#5 Updated by Frédéric Péters about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)
commit fb0632d07f65acba8ed4bd5aaabf41a09137a7f0
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Dec 8 15:44:23 2019 +0100

    forms: consider form pages with only inert fields in validation pages (#37709)

#6 Updated by Frédéric Péters about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF