Projet

Général

Profil

Development #59122

Utiliser un TextField pour Role.name

Ajouté par Benjamin Dauvergne il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
30 novembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Au niveau stockage de postgresql il n'y aucune différence entre un varchar(n) et text, ça ajoute simplement une contrainte lors des écritures; par contre je penche pour laisser une contrainte au niveau API et formulaire BO (ça peut être généraliser à d'autres champs). Il faudrait au passage créer un type spécifique pour ne pas avoir forcément TextArea comme widget (c'est la valeur par défaut pour un TextField) :

class A2TextField(TextField):
    def formfield(self, **kwargs):
        kwargs.setdefault('widget', forms.TextInput)
        return super().formfield(**kwargs)

Fichiers


Demandes liées

Lié à Authentic 2 - Bug #59093: DataError: value too long for type character varying(256)Fermé29 novembre 2021

Actions

Révisions associées

Révision 5ea73341 (diff)
Ajouté par Valentin Deniaud il y a plus de 2 ans

a2_rbac: use a TextField to store role name (#59122)

Historique

#2

Mis à jour par Valentin Deniaud il y a plus de 2 ans

  • Assigné à mis à Valentin Deniaud
#3

Mis à jour par Valentin Deniaud il y a plus de 2 ans

  • Lié à Bug #59093: DataError: value too long for type character varying(256) ajouté
#4

Mis à jour par Valentin Deniaud il y a plus de 2 ans

#5

Mis à jour par Paul Marillonnet il y a plus de 2 ans

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

Yes. Attention celui de ce patch ou de #58546 qui sera poussé en dernier devra subir une modification sur la migration. Conflit sur ces deux 0026 concurrentes (même remarque pour moi-même avec #58829, qui passera sans doute après ces deux là).

#6

Mis à jour par Valentin Deniaud il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 5ea73341ef456807358d31fa63f8522c4e241058
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Dec 8 10:36:06 2021 +0100

    a2_rbac: use a TextField to store role name (#59122)
#7

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

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

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF