Inconsistance des messages de retour dans les traitements supplémentaires
(Je remets ici pour avoir une trace)
Soit un plugin qui ajoute un traitement traiter/bidouiller.php.
Si ce traitement contient un message_ok
dans les retours, celui-ci est bien pris en compte sauf s'il y a un message de retour personnalisé dans la configuration du formulaire, à ce moment là il écrase tout et on perd les autres messages.
C'est chagrinant :
- D'une part c'est inconsistant : soit ils devraient être toujours pris en compte, soit jamais, mais pas d'entre deux
- Et je ne vois pas de raison de ne pas les prendre en compte. Je ne dis pas qu'il n'y en a pas, juste que j'en vois pas :p
@rastapopoulos a jeté un coup d'oeil, apparemment c'était bon à un moment, puis ça a été revert, mais le message de commit ne disait pas pourquoi.
Exemple concret : dans formidable_mailsubscribers je veux m'aligner un peu plus avec ce qui est fait dans #FORMULAIRE_NEWSLETTER_SUBSCRIBE. Ce dernier retourne 2 choses dans le message de retour :
- Si l'opt-in est activé, ça met un message de retour disant qu'un mail a été envoyé et nécessaire pour confirmer l'abonnement. C'est donc un retour important car le mail peut potentiellement finir dans les spams, ça indique à la personne qu'il faut vérifier.
- Et plus important encore, il y a une protection antispam : ça ajoute du javascript dans le message de retour qui lance l'action pour procéder à l'abonnement. Donc là c'est carrément bloquant, en l'état impossible de reprendre ce fonctionnement dans formidable_mailsubscribers.