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
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
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-misc-don-t-repeat-title-identical-to-page-name-on-su.patch 0001-misc-don-t-repeat-title-identical-to-page-name-on-su.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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.
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)
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: don't repeat title identical to page name on summary page (#26732)