Development #59122
Utiliser un TextField pour Role.name
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
Révisions associées
Historique
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é
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-a2_rbac-use-a-TextField-to-store-role-name-59122.patch 0001-a2_rbac-use-a-TextField-to-store-role-name-59122.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
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)
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
a2_rbac: use a TextField to store role name (#59122)