Skip to content

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.