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

New Issue

Closed

opened 2020-08-05 23:55:24 +02:00 by Darks · 2 comments

Darks commented

2020-08-05 23:55:24 +02:00

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 2020-08-05 23:55:24 +02:00

Darks referenced this issue from a commit

2020-09-23 23:59:16 +02:00

Refact some forms (#70)

Lephenixnoir commented

2021-07-08 11:07:10 +02:00

Owner

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 2021-07-08 11:07:14 +02:00

Lephenixnoir referenced this issue from a commit

2021-07-08 15:12:21 +02:00

clean up account forms (factor + syntax) (#70)

Lephenixnoir commented

2021-07-08 15:13:57 +02:00

Owner

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

2021-07-08 15:13:57 +02:00