attention aux champs extra; desormais les fonctions qui creent le formulaire...

attention aux champs extra; desormais les fonctions qui creent le formulaire extra ou la liste des champs enregistres ne font plus echo eux-memes (a reporter dans les plugins) + il y avait un bug dans editer_rubriques, car ['champs_extra'] c'est la config des champs, pas les donnees envoyees par le redacteur

ecrire/action/editer_rubrique.php

@@ -101,9 +101,9 @@ function enregistre_modifier_naviguer($id_rubrique, $id_parent, $titre, $texte,
 
 	if ($id_parent == $id_rubrique) $parent = ''; // au fou
 
-	if (_request('champs_extra')) {
-			include_spip('inc/extra');
-			$extra = extra_recup_saisie("rubriques");
+	if ($GLOBALS['champs_extra']) {
+		include_spip('inc/extra');
+		$extra = extra_recup_saisie("rubriques");
 	}
 	else $extra = '';
 

ecrire/exec/articles.php

@@ -401,7 +401,7 @@ function afficher_corps_articles($virtuel, $chapo, $texte, $ps,  $extra)
 		
 		if ($champs_extra AND $extra) {
 			include_spip('inc/extra');
-			$res .= extra_affichage($extra, "articles", true);
+			$res .= extra_affichage($extra, "articles");
 		}
 	}
 	return $res;

ecrire/exec/articles_ortho.php

@@ -282,7 +282,7 @@ else {
 
 	if ($champs_extra AND $extra) {
 		include_spip('inc/extra');
-		extra_affichage($extra, "articles");
+		echo extra_affichage($extra, "articles");
 	}
 }
 

ecrire/exec/articles_versions.php

@@ -319,7 +319,7 @@ if ($id_version) {
 	
 		if ($champs_extra AND $extra) {
 			include_spip('inc/extra');
-			extra_affichage($extra, "articles");
+			echo extra_affichage($extra, "articles");
 		}
 	}
 }

ecrire/exec/breves_edit.php

@@ -167,7 +167,7 @@ if ($connect_statut=="0minirezo" OR $statut=="prop" OR $new == "oui") {
 
 	if ($champs_extra) {
 		include_spip('inc/extra');
-		extra_saisie($extra, 'breves', $id_rubrique);
+		echo extra_saisie($extra, 'breves', $id_rubrique);
 	}
 
 	if ($connect_statut=="0minirezo" AND acces_rubrique($id_rubrique)) {

ecrire/exec/breves_voir.php

@@ -183,7 +183,7 @@ function afficher_breves_voir($id_breve, $changer_lang, $cherche_mot, $select_gr
 	// afficher les extra
 	if ($champs_extra AND $extra) {
 		include_spip('inc/extra');
-		extra_affichage($extra, "breves");
+		echo extra_affichage($extra, "breves");
 	}
 
 	if ($connect_statut=="0minirezo" AND acces_rubrique($id_rubrique) AND ($statut=="prop" OR $statut=="prepa")){

ecrire/exec/mots_edit.php

@@ -263,7 +263,7 @@ if (acces_mots()){
 
 	if ($champs_extra) {
 		include_spip('inc/extra');
-		extra_saisie($extra, 'mots', $id_groupe);
+		echo extra_saisie($extra, 'mots', $id_groupe);
 	}
 
 	echo "<DIV align='right'><input type='submit' NAME='Valider' VALUE='"._T('bouton_enregistrer')."' CLASS='fondo'></div>";

ecrire/exec/naviguer.php

@@ -92,7 +92,7 @@ function exec_naviguer_dist()
 
 	  if ($extra) {
 		include_spip('inc/extra');
-		extra_affichage($extra, "rubriques");
+		echo extra_affichage($extra, "rubriques");
 	  }
 
 /// Mots-cles

ecrire/exec/rubriques_edit.php

@@ -148,7 +148,7 @@ function exec_rubriques_edit_dist()
 
 	if ($champs_extra) {
 		include_spip('inc/extra');
-		extra_saisie($extra, 'rubriques', $id_secteur);
+		$form .= extra_saisie($extra, 'rubriques', $id_secteur);
 	}
 
 	$form .= "<input type='hidden' name='new' value='"

ecrire/exec/sites.php

@@ -506,7 +506,7 @@ else if (preg_match(',^select: (.*),', trim($url_syndic), $regs)) {
 
 if ($champs_extra AND $extra) {
 		include_spip('inc/extra');
-		extra_affichage($extra, "sites");
+		echo extra_affichage($extra, "sites");
 	}
 
 fin_cadre_relief();

ecrire/exec/sites_edit.php

@@ -189,9 +189,9 @@ else {
 
 
 if ($champs_extra) {
-		include_spip('inc/extra');
-		extra_saisie($extra, 'sites', intval($id_secteur));
-	}
+	include_spip('inc/extra');
+	echo extra_saisie($extra, 'sites', intval($id_secteur));
+}
 
 
 echo "\n<div ALIGN='right'><INPUT TYPE='submit' NAME='Valider' VALUE='"._T('bouton_enregistrer')."' CLASS='fondo'></div>";

ecrire/inc/editer_article.php

@@ -141,7 +141,7 @@ function inc_editer_article($row, $lier_trad, $new, $champs_article) {
 		 ("\n<p><b>" . _T('info_post_scriptum') ."</b><br />" . "<textarea name='ps' class='forml' rows='5' cols='40' wrap=soft>" . $ps . "</textarea></p><p>\n") :
 		 '') .
 
-		(!$champs_extra ? '': extra_saisie($extra, 'articles', $id_secteur, false)) .
+		(!$champs_extra ? '': extra_saisie($extra, 'articles', $id_secteur)) .
 
 		"<div align='right'><input class='fondo' type='submit' value='" . _T('bouton_enregistrer') . "' /></div>";
 

ecrire/inc/extra.php

@@ -88,15 +88,11 @@ if (!defined("_ECRIRE_INC_VERSION")) return;
 
 // a partir de la liste des champs, generer la liste des input
 // http://doc.spip.org/@extra_saisie
-function extra_saisie($extra, $type, $ensemble='', $aff=true) {
+function extra_saisie($extra, $type, $ensemble='') {
 	if ($affiche = extra_form($extra, $type, $ensemble)) {
-	  if ($aff) {
-		debut_cadre_enfonce();
-		echo $affiche;
-		fin_cadre_enfonce();
-	  } else {
-	    return debut_cadre_enfonce('',true) . $affiche . fin_cadre_enfonce(true);
-	  }
+		return debut_cadre_enfonce('',true)
+			. $affiche
+			. fin_cadre_enfonce(true);
 	}
 }
 
@@ -322,7 +318,7 @@ function extra_champ_valide($type, $nom_champ) {
 
 // a partir de la liste des champs, generer l'affichage
 // http://doc.spip.org/@extra_affichage
-function extra_affichage($extra, $type, $return=false) {
+function extra_affichage($extra, $type) {
 	$extra = unserialize ($extra);
 	if (!is_array($extra)) return;
 	$champs = $GLOBALS['champs_extra'][$type];
@@ -361,11 +357,10 @@ function extra_affichage($extra, $type, $return=false) {
 			.interdire_scripts($contenu)."<br /></div>\n";
 	}
 
-	if ($affiche) {
-		$affiche = debut_cadre_enfonce('',true) . $affiche . fin_cadre_enfonce(true);
-		if ($return) return $affiche; else echo $affiche;
-	}
-
+	if ($affiche)
+		return debut_cadre_enfonce('',true)
+			. $affiche
+			. fin_cadre_enfonce(true);
 }
 
 ?>

ecrire/inc/legender_auteur.php

@@ -110,7 +110,7 @@ function legender_auteur_saisir($id_auteur, $auteur, $mode, $echec='', $redirect
 
 	if ($champs_extra) {
 		include_spip('inc/extra');
-		$corps .= extra_saisie($auteur['extra'], 'auteurs', $auteur['statut'],'', false);
+		$corps .= extra_saisie($auteur['extra'], 'auteurs', $auteur['statut']);
 	}
 
 //
@@ -276,7 +276,7 @@ function legender_auteur_voir($auteur, $redirect)
 
 	if ($champs_extra AND $extra) {
 		include_spip('inc/extra');
-		$res .= extra_affichage($extra, "auteurs", true);
+		$res .= extra_affichage($extra, "auteurs");
 	}
 
 	return $res;