backoffice: allow table fields in management tables (#4245) #833
lguerin
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/4245-do-not-allow-table-fields-in-listings"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
85c7e8f019to14b03e0829WIP: backoffice: never allow table fields in management tables (#4245)to WIP: backoffice: allow table fields in management tables (#4245)WIP: backoffice: allow table fields in management tables (#4245)to backoffice: allow table fields in management tables (#4245)Il y avait possibilité de configurer les champs tableau pour apparaitre dans les tableaux de traitement mais uniquement en passant par l'ajout de la colonne depuis le vue tableau de traitement, pas de possibilité de les configurer comme devant s'y afficher par défaut depuis la définition du champ en elle-même. Comme cette limitation était vraiment devenue bête maintenant qu'on peut juste définir une vue par défaut, et comme ces champs sont dépréciés et que ça ne changera rien pour personne, ce patch supprime la particularité.
En passant, j'ajoute un attribut can_include_in_listing qui 1/ évite d'avoir à répéter la liste des options possibles, 2/ est plus propre à écrire qu'un hasattr(field, 'get_view_value').
(à un moment j'ai hésité à juste totalement interdire la présence en colonne mais il y a du code pour le formatage de ces champs dans les CSV/ODS qu'il aurait fallu aussi supprimer, et si jamais c'était utilisé ça aurait êté bête, ce patch est sans risque).