Uniformiser le comportement entre `message_ok` et `message_retour`
En écrivant https://contrib.spip.net/ecrire/?exec=article&id_article=5573
j'ai relu un peu le code. Et j'ai du mettre ce paragraphe
À noter qu’il y a une petite subtilité [2].
Les message_ok et scripts_ok sont automatiquement concaténés par formidable : le $retour['message_ok'] ne doit contenir le message de validation que du traitement courant
Les message_erreur doivent être cumulés par vous le cas échéant : il vous faut vérifier qu’il n’y a pas déjà un message d’erreur, et, le cas échéant ajouter le votre après le précédent.
@nicod disait sur discord
C'est bizarre le traitement différencié de
message_ok
(qu'on doit remplir) et demessage_erreur
(qu'on doit compléter)
et je suis d'accord avec lui
je propose d'uniformiser le comportement de message_erreur sur message_ok en v7.
La seule chose que ca casse c'est si jamais on décide de ne pas appliquer un traitement suite à l'échec d'un autre traitement. Ce qui n'est a priori jamais fait dans le code de la nature. Et sinon ca fera peut être un peu de code mort, avec des tests inutiles dans les traitements perso, mais c'est tout.