Faire hériter les modèles de formulaire #70

Closed
opened 2 years ago by Darks · 2 comments
Darks commented 2 years ago
Owner

En particulier ceux des comptes. Je suis sûr qu'on peut trouver un modèle qui permettre de rendre le truc un peu plus DRY, plutôt que de tout réécrire pour chaque formulaire…

Avec un poil d'héritage, des classes à la con et de la surcharge comme il faut, ça doit le faire.

En particulier ceux des comptes. Je suis sûr qu'on peut trouver un modèle qui permettre de rendre le truc un peu plus DRY, plutôt que de tout réécrire pour chaque formulaire… Avec un poil d'héritage, des classes à la con et de la surcharge comme il faut, ça doit le faire.
Darks added the
performance
label 2 years ago
Darks referenced this issue from a commit 2 years ago

Comme exemple facile pour illustrer ce que Darks demande, dans cc5f4e481b on fait hériter DeleteAccountForm et AdminDeleteAccountForm d'un formulaire commun qui est DeleteAccountBaseForm. L'intérêt c'est que les champs (transfert des posts vers un invité, confirmation, et soumission) sont partagés. Le DeleteAccountForm ajoute un champ supplémentaire demandant le mot de passe.

Ça doit pas être trop compliqué, je ferai une passe dessus.

Comme exemple facile pour illustrer ce que Darks demande, dans https://gitea.planet-casio.com/devs/PCv5/commit/cc5f4e481b534271408e5e560263208a47339c3c on fait hériter `DeleteAccountForm` et `AdminDeleteAccountForm` d'un formulaire commun qui est `DeleteAccountBaseForm`. L'intérêt c'est que les champs (transfert des posts vers un invité, confirmation, et soumission) sont partagés. Le `DeleteAccountForm` ajoute un champ supplémentaire demandant le mot de passe. Ça doit pas être trop compliqué, je ferai une passe dessus.
Lephenixnoir self-assigned this 1 year ago

J'ai factorisé le formulaire de mise à jour de compte (probablement le plus gros). Je me suis permis d'alléger un peu la syntaxe parce que le fichier est assez long et un peu dur à lire.

Sur les autres formulaires il n'y a pas grand-chose à dire, ils sont tous assez directs et avec peu de redondance. Je ne voyais pas d'interêt de factoriser de la poussière donc je considère que c'est bon. ^^

J'ai factorisé le formulaire de mise à jour de compte (probablement le plus gros). Je me suis permis d'alléger un peu la syntaxe parce que le fichier est assez long et un peu dur à lire. Sur les autres formulaires il n'y a pas grand-chose à dire, ils sont tous assez directs et avec peu de redondance. Je ne voyais pas d'interêt de factoriser de la poussière donc je considère que c'est bon. ^^
Lephenixnoir closed this issue 1 year ago
Sign in to join this conversation.
Loading…
There is no content yet.