Afficher tous les messages de retour

Ticket #44 (closed) : afficher tous les messages de retour, même en cas de message personnalisé.

J'en ai profité pour séparer chaque message par un saut de ligne afin de mieux les distinguer (2 <br> quoi). Incidemment, les traitements n'ont plus de visibilité sur les messages des traitements précédents dans $retours['message_ok'], ça me semble inutile. Ils ont juste à ajouter leur message de retour éventuel, sans se soucier d'avoir à mettre un saut de ligne ou autre si jamais il y avait d'autres messages en amont.

Exemple avec un message personnalisé et 2 traitements activés : « enregistrer les données » et « s'inscrire à des listes de diffusion ».

Mon super message de retour personnalisé.

Merci. Vos réponses ont bien été enregistrées.

Vous allez recevoir un email sur l’adresse prout@pouet.com pour confirmer votre inscription.

formulaires/formidable.php

@@ -383,6 +383,19 @@ function formulaires_formidable_traiter_dist($id, $valeurs = array(), $id_formul
 		$retours['redirect'] = $url_redirect;
 	}
 
+	// Message de retour personnalisé toujours en 1er
+	$messages_ok = array();
+	if ($formulaire['message_retour']) {
+		$messages_ok[] = _T_ou_typo(
+			formidable_raccourcis_arobases_2_valeurs_champs(
+				$formulaire['message_retour'],
+				$saisies,
+				false,
+				''
+			)
+		);
+	}
+
 	// les traitements deja faits se notent ici
 	// pour etre sur de ne pas etre appeles 2 fois
 	// ainsi si un traitement A a besoin d'un traitement B,
@@ -413,6 +426,11 @@ function formulaires_formidable_traiter_dist($id, $valeurs = array(), $id_formul
 							),
 							$retours
 						);
+						// Collecter les messages de retour afin de les formater correctement par la suite
+						if (!empty($retours['message_ok'])) {
+							$messages_ok[] = $retours['message_ok'];
+							$retours['message_ok'] = ''; // Inutile de les exposer aux autres traitements
+						}
 					} else {
 						// traitement introuvable, ne pas retenter
 						$retours['traitements'][$type_traitement] = true;
@@ -426,21 +444,14 @@ function formulaires_formidable_traiter_dist($id, $valeurs = array(), $id_formul
 				. 'Traitements attendus :'.implode(',', array_keys($traitements))."\n"
 				. 'Traitements realises :'.implode(',', array_keys($retours['traitements']))."\n";
 		}
-
-		// Si on a personnalisé le message de retour, c'est lui qui est affiché uniquement
-		if ($formulaire['message_retour']) {
-			$retours['message_ok'] = _T_ou_typo(
-				formidable_raccourcis_arobases_2_valeurs_champs(
-					$formulaire['message_retour'],
-					$saisies,
-					false,
-					''
-				)
-			);
-		}
 	} else {
 		$retours['message_erreur'] = _T('formidable:retour_aucun_traitement');
 	}
+	// Réinsérer le message de retour en le formatant un minimum :
+	// saut de lignes pour différencier chaque message (perso et traitements)
+	if (count($messages_ok) > 0) {
+		$retours['message_ok'] = implode('<br><br>', $messages_ok);
+	}
 	if (isset($retours['fichiers'])) {// traitement particuliers si fichiers
 		if ($erreurs_fichiers = formidable_produire_messages_erreurs_fichiers($retours['fichiers'])) {
 			// Inspecter les fichiers pour voir s'il y a des erreurs