Projet

Général

Profil

Development #38095

quixote3, ne pas laisser les id ajoutés automatiquement aux widgets

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
29 novembre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

quixote3 ajoute automatiquement un attribut id aux champs,

        # add 'id' attribute if not already present
        if 'id' not in kwargs:
            kwargs['id'] = name

mais on le fait déjà (différemment), lors du rendu, avec une construction qui est basiquement :

    attrs = {'id': 'form_' + self.name}
    if self.attrs:
        attrs.update(self.attrs)

et notre id se trouve du coup écrasé.

Et la correspondance posée sur le libellée (<label for="form_...">) n'est plus correcte.


Fichiers

Révisions associées

Révision 1678d3b6 (diff)
Ajouté par Frédéric Péters il y a plus de 4 ans

form: don't let quixote3 assign a default id to widgets (#38095)

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

misc: add note about quixote3/id removal (#38095)

Historique

#1

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

#2

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

  • Statut changé de Solution proposée à Solution validée

Fichtre. Je serais surpris de savoir combien de temps ça t'a pris de trouver l'origine du pépin.

Sans savoir si c'est vraiment pertinent, on pourrait imaginer un petit commentaire genre « # remove automatic quixote id », ou pas.

#3

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

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

(pas particulièrement long, il n'y a pas non plus des masses de changement dans quixote3)

commit 1678d3b6a29299c3773fd7074eb407bf3a5b4c4a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Nov 29 18:34:01 2019 +0100

    form: don't let quixote3 assign a default id to widgets (#38095)

(+ une ligne de commentaire, dans un second commit)

#4

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

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

Formats disponibles : Atom PDF