Development #77141
wcs/admin/workflows, nettoyage hasattr obsolètes
0%
Description
Il y a très très longtemps fill_admin_form était défini sur les classes des actions de workflow, sans héritage. Il ne fallait alors pas d'icône pour arriver sur une page de modification qui n'aurait pas trouvé quoi afficher et donc 9f367ecfd2075720389ea72dbd76911a9c25057e (juillet 2006).
Il y a très longtemps il y a eu une classe de base et les hasattr() auraient pu être retirés, ça n'a pas été fait. (février 2011, #255).
Il y a longtemps il y a eu l'ajout d'actions globales et les hasattr() inutiles ont été copiés/collés. (décembre 2015, #3659)
Là on pourrait virer les hasattr().
Révisions associées
Historique
Mis à jour par Robot Gitea il y a presqu'un an
- Statut changé de Nouveau à Solution proposée
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/272
- Titre : backoffice: remove check for fill_admin_form, always defined (#77141)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/272/files
Mis à jour par Robot Gitea il y a presqu'un an
- Statut changé de Solution proposée à Solution validée
Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 12 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/272
- Titre : backoffice: remove check for fill_admin_form, always defined (#77141)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/272/files
Mis à jour par Transition automatique il y a 12 mois
- Statut changé de Résolu (à déployer) à Solution déployée
backoffice: remove check for fill_admin_form, always defined (#77141)