Development #10123
priorité email de l'utilisateur entre champ de profil et champ de formulaire
0%
Description
Je viens de relire def get_submitter_email(self, formdata): et on prend l'email du profil et en son absence l'email précisé dans le champ; d'une part je pensais que c'était l'inverse, d'autre part je pense que l'inverse a plus de sens. (pour permettre d'overrider le champ du profil).
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 7 ans
« l'inverse a plus de sens » : je vote pour ça aussi. À vue de nez, je pense que ça ne cassera rien dans l'existant (au contraire).
Mis à jour par Mikaël Ates il y a plus de 7 ans
Tout à fait d'accord. Si on a un champs email pré-rempli éditable il faut que ce soit la valeur du champs qui soit employée (par défaut).
Mis à jour par Frédéric Péters il y a plus de 7 ans
Je commente quand même le "(par défaut)". Il n'y aura pas d'option pour dire "non, prends en priorité le mail du profil" (mais il sera toujours temps de faire =form_user_email).
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Fichier 0001-misc-prefer-email-from-formdata-over-user-profile-10.patch 0001-misc-prefer-email-from-formdata-over-user-profile-10.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 7 ans
Ack.
Appréciable simplification au niveau du « return formdata.user.email », le code pré-existant n'était pas tout jeune !
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit 5772e64846d7bb7f723c2f2bd1a8b813d11b56ab
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Dec 21 15:16:50 2016 +0100
misc: prefer email from formdata over user profile (#10123)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: prefer email from formdata over user profile (#10123)