From fff1dce6dbe016e6fd7613d01becee807f14b30e Mon Sep 17 00:00:00 2001
From: "Committo,Ergo:sum" <esj@rezo.net>
Date: Mon, 3 Aug 2009 13:35:24 +0000
Subject: [PATCH] =?UTF-8?q?Un=20peu=20de=20rigueur=20dans=20les=20specs=20?=
 =?UTF-8?q?du=20d=C3=A9busqueur:=20il=20ne=20rend=20pas=20de=20r=C3=A9sult?=
 =?UTF-8?q?at,=20c'est=20aux=20appelants=20de=20retourner=20ce=20qu'il=20f?=
 =?UTF-8?q?aut=20selon=20les=20situations.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 ecrire/balise/formulaire_ecrire_auteur.php |  3 ++-
 ecrire/balise/formulaire_signature.php     |  3 ++-
 ecrire/balise/formulaire_site.php          |  3 ++-
 ecrire/inc/filtres.php                     |  3 ++-
 ecrire/inc/utils.php                       |  2 +-
 ecrire/public/compiler.php                 |  6 +++---
 ecrire/public/composer.php                 |  8 +++++---
 ecrire/public/phraser_html.php             | 11 ++---------
 8 files changed, 19 insertions(+), 20 deletions(-)

diff --git a/ecrire/balise/formulaire_ecrire_auteur.php b/ecrire/balise/formulaire_ecrire_auteur.php
index e3a4615c5e..cac9483582 100644
--- a/ecrire/balise/formulaire_ecrire_auteur.php
+++ b/ecrire/balise/formulaire_ecrire_auteur.php
@@ -33,7 +33,8 @@ function balise_FORMULAIRE_ECRIRE_AUTEUR_stat($args, $context_compil) {
 				array ('champ' => 'FORMULAIRE_ECRIRE_AUTEUR',
 					'motif' => 'AUTEURS/ARTICLES'));
 
-		return denoncer_inclure_dynamique($msg, $context_compil);
+		denoncer_inclure_dynamique($msg, $context_compil);
+		return '';
 	}
 	// Si on est dans un contexte article, 
 	// sortir tous les mails des auteurs de l'article
diff --git a/ecrire/balise/formulaire_signature.php b/ecrire/balise/formulaire_signature.php
index d68c84ef41..84a3246451 100644
--- a/ecrire/balise/formulaire_signature.php
+++ b/ecrire/balise/formulaire_signature.php
@@ -38,7 +38,8 @@ function balise_FORMULAIRE_SIGNATURE_stat($args, $context_compil) {
 		$msg = array('zbug_champ_hors_motif',
 				array ('champ' => 'FORMULAIRE_SIGNATURE',
 				       'motif' => 'ARTICLES'));
-		return denoncer_inclure_dynamique($msg, $context_compil);
+		denoncer_inclure_dynamique($msg, $context_compil);
+		return '';
 	}
 	// article sans petition => pas de balise
 	else if (!$args[1])
diff --git a/ecrire/balise/formulaire_site.php b/ecrire/balise/formulaire_site.php
index 5085a4b283..a92f8f4bd9 100644
--- a/ecrire/balise/formulaire_site.php
+++ b/ecrire/balise/formulaire_site.php
@@ -28,7 +28,8 @@ function balise_FORMULAIRE_SITE_stat($args, $context_compil) {
 		$msg = array('zbug_champ_hors_motif',
 				array ('champ' => 'FORMULAIRE_SITE',
 					'motif' => 'RUBRIQUES'));
-		return denoncer_inclure_dynamique($msg, $context_compil);
+		denoncer_inclure_dynamique($msg, $context_compil);
+		return '';
 	}
 	// Verifier que les visisteurs sont autorises a proposer un site
 
diff --git a/ecrire/inc/filtres.php b/ecrire/inc/filtres.php
index 0a0ba4a878..978c94cea7 100644
--- a/ecrire/inc/filtres.php
+++ b/ecrire/inc/filtres.php
@@ -224,7 +224,8 @@ function filtrer($filtre) {
 	}
 	else {
 		// le filtre n'existe pas, on provoque une erreur
-	  erreur_squelette(_T('zbug_erreur_filtre', array('filtre'=>texte_script($filtre))));
+		erreur_squelette(_T('zbug_erreur_filtre', array('filtre'=>texte_script($filtre))));
+		return '';
 	}
 }
 
diff --git a/ecrire/inc/utils.php b/ecrire/inc/utils.php
index fde43cd0ef..e8069d6f18 100644
--- a/ecrire/inc/utils.php
+++ b/ecrire/inc/utils.php
@@ -1652,7 +1652,7 @@ function exec_info_dist() {
 
 function erreur_squelette($message='', $lieu='', $quoi='') {
 	$debusquer = charger_fonction('debusquer', 'public');
-	return $debusquer($message, $lieu, $quoi);
+	$debusquer($message, $lieu, $quoi);
 }
 
 /**
diff --git a/ecrire/public/compiler.php b/ecrire/public/compiler.php
index 3b16f8c628..68e52c4aa3 100644
--- a/ecrire/public/compiler.php
+++ b/ecrire/public/compiler.php
@@ -61,9 +61,9 @@ function argumenter_inclure($params, $rejet_filtres, $p, &$boucles, $id_boucle,
 			$var = $val[0];
 			if ($var->type != 'texte') {
 			  if ($n OR $k) {
-			    $msg = array('zbug_parametres_inclus_incorrects',
+				$msg = array('zbug_parametres_inclus_incorrects',
 					 array('param' => $var->nom_champ));
-			    erreur_squelette($msg, $p);
+				erreur_squelette($msg, $p);
 			  } 
 			  $l[1] = calculer_liste($val, $p->descr, $boucles, $id_boucle);
 			  break;
@@ -910,7 +910,7 @@ function compiler_squelette($squelette, $boucles, $nom, $descr, $sourcefile, $co
 			if (!$rec) {
 				$msg = array('zbug_boucle_recursive_undef',
 					array('nom' => $boucle->param[0]));
-				return erreur_squelette($msg, $boucle);
+				erreur_squelette($msg, $boucle);
 			} else {
 				$rec->externe = $id;
 				$descr['id_mere'] = $id;
diff --git a/ecrire/public/composer.php b/ecrire/public/composer.php
index 5315010281..00c28610f7 100644
--- a/ecrire/public/composer.php
+++ b/ecrire/public/composer.php
@@ -327,7 +327,8 @@ function executer_balise_dynamique($nom, $args, $context_compil) {
 		else {
 			$msg =  "<span class='spip-debug-arg'>$nom</span> " .
 			  _T('zxml_inconnu_balise');
-			return denoncer_inclure_dynamique($msg, $context_compil);
+			denoncer_inclure_dynamique($msg, $context_compil);
+			return '';
 		}
 	}
 	// Y a-t-il une fonction de traitement des arguments ?
@@ -349,7 +350,8 @@ function executer_balise_dynamique($nom, $args, $context_compil) {
 
 			$msg =  "<span class='spip-debug-arg'>$nom</span> " .
 			  _T('zxml_inconnu_balise');
-			return denoncer_inclure_dynamique($msg, $context_compil);
+			denoncer_inclure_dynamique($msg, $context_compil);
+			return '';
 		}
 	}
 
@@ -367,7 +369,7 @@ function denoncer_inclure_dynamique($msg, $context_compil)
 	$p->id_boucle = $context_compil[2];
 	$p->ligne = $context_compil[3];
 	$p->lang = $context_compil[4];
-	return erreur_squelette($msg, $p);
+	erreur_squelette($msg, $p);
 }
 
 // http://doc.spip.org/@lister_objets_avec_logos
diff --git a/ecrire/public/phraser_html.php b/ecrire/public/phraser_html.php
index 831c1481d7..3a461813ba 100644
--- a/ecrire/public/phraser_html.php
+++ b/ecrire/public/phraser_html.php
@@ -540,9 +540,9 @@ function phraser_criteres($params, &$result) {
 			    $crit->cond = $m[5];
 			  }
 			  else {
-			    $msg = array('zbug_critere_inconnu', 
+			 	$msg = array('zbug_critere_inconnu', 
 					array('critere' => $param));
-			    erreur_squelette($msg, $result);
+				erreur_squelette($msg, $result);
 			  }
 			  if ((!preg_match(',^!?doublons *,', $param)) || $crit->not)
 			    $args[] = $crit;
@@ -597,13 +597,6 @@ function public_phraser_html_dist($texte, $id_parent, &$boucles, $descr, $ligne=
 					   $k - strlen(BALISE_BOUCLE)));
 		  $milieu = substr($milieu, $k);
 
-		  /* a adapter: si $n pointe sur $id_boucle ...
-		if (strpos($milieu, $s)) {
-			erreur_squelette(_T('zbug_erreur_boucle_syntaxe'),
-				$id_boucle . 
-				_T('zbug_balise_b_aval'));
-		}
-		  */
 		} else {
 		  $debut = substr($texte, 0, $n);
 		  $milieu = substr($texte, $n);
-- 
GitLab