Projet

Général

Profil

Development #26732

ne pas reprendre un champ titre s'il a le même contenu que le nom de la page, peu importe la présence de champs qu'on n'affiche pas

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

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Depuis #13529 on ne répète pas un titre placé en premier élément de page s'il est identique au nom de la page.

Mais si jamais la page commence par un champ pas affiché, genre un commentaire "les champs marqués * sont obligatoires", on peut malgré tout se trouver avec un enchainement immédiat nom de page champ titre, identiques.

Le code doit être autour de :

            if f.type == 'title' and on_page and fields[i-1] is on_page and on_page.label == f.label:
                # don't include first title of a page if that title has the
                # same text as the page.
                continue

Fichiers

Révisions associées

Révision dfe2cba4 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

misc: don't repeat title identical to page name on summary page (#26732)

Historique

#2

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

#3

Mis à jour par Thomas Noël il y a plus de 5 ans

Dans formdef.py plutôt que le if field.add_to_view_form is None ... (et les add_to_view_form=None associés), j'aurais préféré du bien gros explicite :

if field.type == 'comment' or field.type == 'page':
    continue

Comme ça viendrait juste avant ou après le if field.type == 'title' ... on reste dans la même "logique".

Ou pas, tu dis.

#4

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

  • Statut changé de Solution proposée à Résolu (à déployer)

Fait ainsi.

commit dfe2cba424f97888f70334621b1c2bf9c759fb72
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Oct 28 16:39:31 2018 +0100

    misc: don't repeat title identical to page name on summary page (#26732)

(et désolé j'avais lu ça comme un ack et j'ai poussé avant de vérifier)

#5

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