Suite [7549]: Introduction de la fonction ajax_retour utilisés par TOUS les...

Suite [7549]: Introduction de la fonction ajax_retour utilisés par TOUS les scripts envoyant une réponse Ajax. Cette fonction est en fait le bout de code figurant auparavant dans index.php qui n'a ainsi plus besoin de distinguer les deux formes de retour et est donc plus court.

Cette simplification a été rendue possible en inversant le chantier prévu lors de la [7310]: les scripts en réponse Ajax utilisent echo, via ajax_retour. Un fichier index.php comportant l'unique echo de tout le code PHP de l'espace privé reste l'objectif final, mais il est préférable d'unifier d'abord ainsi, et de basculer lorsque chaque script de exec/ ne comportera plus qu'un seul echo.

ecrire/exec/articles_forum.php

@@ -49,7 +49,7 @@ function exec_articles_forum_dist()
 
 	$droit= acces_rubrique($id_rubrique);
 
-	if (_request('var_ajaxcharset') AND $droit) return $mess;
+	if (_request('var_ajaxcharset') AND $droit) ajax_retour($mess);
 
  	pipeline('exec_init',array('args'=>array('exec'=>'articles_forum','id_article'=>$id_article),'data'=>''));
 

ecrire/exec/auteurs.php

@@ -58,7 +58,7 @@ while ($auteur = spip_fetch_array($result)) {
 
  $res = auteurs_tranches($auteurs, $debut, $lettre, $tri, $visiteurs, $max_par_page, $nombre_auteurs);
 
-  if (_request('var_ajaxcharset')) return $res;
+ if (_request('var_ajaxcharset')) ajax_retour($res);
 
   pipeline('exec_init',array('args'=>array('exec'=>'auteurs'),'data'=>''));
 
@@ -225,12 +225,10 @@ function auteurs_href($clic, $args='', $att='')
 {
 	$h = generer_url_ecrire('auteurs', $args);
 	$a = 'auteurs';
-	if ($_COOKIE['spip_accepte_ajax'] != 1 )
-		$evt = '';
-        else 
-		$evt = "\nonclick='return AjaxSqueeze(\"$h\",\n\t\"$a\")'";
+	if ($_COOKIE['spip_accepte_ajax'] == 1 )
+		$att .= ("\nonclick=" . ajax_action_declencheur("\"$h\"",$a));
 
-	return "<a$att href='$h#$a'$evt>$clic</a>";
+	return "<a href='$h#$a'$att>$clic</a>";
 }
 
 // http://doc.spip.org/@requete_auteurs

ecrire/exec/calendrier.php

@@ -44,7 +44,7 @@ function exec_calendrier_dist()
 
   $r = http_calendrier_init('', $type, '','',generer_url_ecrire('calendrier', ($type ? "type=$type" : '')) . "#$ancre");
 
-  if (_request('var_ajaxcharset')) return $r;
+  if (_request('var_ajaxcharset')) ajax_retour($r);
 
   debut_page($titre, "accueil", "calendrier");
   echo "\n<div>&nbsp;</div>\n<div id='", $ancre, "'>",$r,'</div>';

ecrire/exec/controle_forum.php

@@ -197,7 +197,7 @@ function controle_un_forum($row) {
 function exec_controle_forum_dist()
 {
 
-  global $type, $debut, $debut_id_forum, $id_rubrique, $connect_statut, $connect_toutes_rubriques;
+	global $type, $debut, $debut_id_forum, $id_rubrique, $connect_statut, $connect_toutes_rubriques;
 
 	$debut= intval($debut);
 	$id_rubrique = intval($id_rubrique);
@@ -230,7 +230,7 @@ function exec_controle_forum_dist()
 		   (!$id_rubrique OR !acces_rubrique($id_rubrique))));
 
 	if (_request('var_ajaxcharset') AND !$droit) {
-		return $mess;
+		ajax_retour($mess);
 	} else {
 
 		debut_page(_T('titre_page_forum_suivi'), "forum", "forum-controle");

ecrire/exec/dater.php

@@ -29,7 +29,7 @@ function exec_dater_dist()
 	$date = $row["date"];
 	$date_redac = $row["date_redac"];
 
-	$f = charger_fonction('dater', 'inc');
-	return $f($id_article, 'ajax', $statut_article, $date, $date_redac);
+	$dater = charger_fonction('dater', 'inc');
+	ajax_retour($dater($id_article, 'ajax', $statut_article, $date, $date_redac));
 }
 ?>

ecrire/exec/discuter.php

@@ -16,7 +16,7 @@ function exec_discuter_dist()
 {
 	$debut = _request('debut');
 	$id_article = _request('id_article');
-	$f = charger_fonction('discuter', 'inc');
-	return $f($id_article, 'ajax', $debut);
+	$discuter = charger_fonction('discuter', 'inc');
+	ajax_retour($discuter($id_article, 'ajax', $debut));
 }
 ?>

ecrire/exec/editer_auteurs.php

@@ -22,7 +22,7 @@ function exec_editer_auteurs_dist()
 		minipres(_T('info_acces_interdit'));
 	}
 
-	$f = charger_fonction('editer_auteurs', 'inc');
-	return $f($id_article, 'ajax', _request('cherche_auteur'), _request('ids'));
+	$editer_auteurs = charger_fonction('editer_auteurs', 'inc');
+	ajax_retour($editer_auteurs($id_article, 'ajax', _request('cherche_auteur'), _request('ids')));
 }
 ?>

ecrire/exec/editer_mot.php

@@ -42,7 +42,7 @@ function exec_editer_mot_dist()
 
 	$ch = _request('cherche_mot');
 	$id_groupe = _request('select_groupe');
-	$f = charger_fonction('editer_mot', 'inc');
-	return $f($objet, $id_objet, $ch, $id_groupe, 'ajax'); 
+	$editer_mot = charger_fonction('editer_mot', 'inc');
+	ajax_retour($editer_mot($objet, $id_objet, $ch, $id_groupe, 'ajax')); 
 }
 ?>

ecrire/exec/gadgets.php

@@ -18,6 +18,6 @@ function exec_gadgets_dist()
 	$gadget = _request('gadget');
 	$gadgets = charger_fonction('gadgets', 'inc');
 
-	return $gadgets($id_rubrique, $gadget);
+	ajax_retour($gadgets($id_rubrique, $gadget));
 }
 ?>

ecrire/exec/grouper_mots.php

@@ -16,8 +16,8 @@ function exec_grouper_mots_dist()
 {
 	$id_groupe = intval(_request('id_groupe'));
 	$cpt = spip_fetch_array(spip_query("SELECT COUNT(*) AS n FROM spip_mots WHERE id_groupe=$id_groupe"));
-	if (! ($cpt = $cpt['n'])) return '' ;
-	$f = charger_fonction('grouper_mots', 'inc');
-	return $f($id_groupe, $cpt);
+	if (! ($cpt = $cpt['n'])) ajax_retour('') ;
+	$grouper_mots = charger_fonction('grouper_mots', 'inc');
+	ajax_retour($grouper_mots($id_groupe, $cpt));
 }
 ?>

ecrire/exec/iconifier.php

@@ -46,7 +46,7 @@ function exec_iconifier_dist()
 		minipres(_T('info_acces_interdit'));
 	}
 
-	$f = charger_fonction('iconifier', 'inc');
-	return $f($type, $id, $script);
+	$iconifier = charger_fonction('iconifier', 'inc');
+	ajax_retour($iconifier($type, $id, $script));
 }
 ?>

ecrire/exec/informer.php

@@ -16,13 +16,12 @@ if (!defined("_ECRIRE_INC_VERSION")) return;
 
 function exec_informer_dist()
 {
-
 	$id = intval(_request('id'));
 	$col = intval(_request('col'));
 	$exclus = intval(_request('exclus'));
 
-	$f = charger_fonction('informer', 'inc');
-	return $f($id, $col, $exclus, _request('rac'), _request('type'));
+	$informer = charger_fonction('informer', 'inc');
+	ajax_retour($informer($id, $col, $exclus, _request('rac'), _request('type')));
 }
 
 ?>

ecrire/exec/instituer_auteur.php

@@ -20,7 +20,7 @@ function exec_instituer_auteur_dist()
 
 	$r = spip_fetch_array(spip_query("SELECT statut FROM spip_auteurs WHERE id_auteur=$id_auteur"));
 
-	$f = charger_fonction('instituer_auteur', 'inc');
-	return $f($id_auteur, $r['statut'] , $script);
+	$instituer_auteur = charger_fonction('instituer_auteur', 'inc');
+	ajax_retour($instituer_auteur($id_auteur, $r['statut'], $script));
 }
 ?>

ecrire/exec/legender.php

@@ -26,7 +26,7 @@ function exec_legender_dist()
 		minipres(_T('info_acces_interdit'));
 	}
 
-	$f = charger_fonction('legender', 'inc');
-	return $f($id_document, array(), $script, $type, $id, $ancre);
+	$legender = charger_fonction('legender', 'inc');
+	ajax_retour($legender($id_document, array(), $script, $type, $id, $ancre));
 }
 ?>

ecrire/exec/memoriser.php

@@ -40,7 +40,7 @@ function exec_memoriser_dist()
 	if ($flag_ob) {
 			$res = ob_get_contents();
 			ob_end_clean();
-			return $res;
+			ajax_retour($res);
 	}
 }
 ?>

ecrire/exec/petitionner.php

@@ -23,7 +23,7 @@ function exec_petitionner_dist()
 		minipres(_T('info_acces_interdit'));
 	}
 
-	$f = charger_fonction('petitionner', 'inc');
-	return $f($id_article, $script, "&id_article=$id_article", 'ajax');
+	$petitionner = charger_fonction('petitionner', 'inc');
+	ajax_retour($petitionner($id_article, $script, "&id_article=$id_article", 'ajax'));
 }
 ?>

ecrire/exec/plonger.php

@@ -24,7 +24,7 @@ function exec_plonger_dist()
 
 	include_spip('inc/texte');
 	include_spip('inc/mini_nav');
-	return mini_afficher_rubrique ($id, htmlentities($rac), array(), $col, $exclus);
+	ajax_retour(mini_afficher_rubrique($id, htmlentities($rac), array(), $col, $exclus));
 }
 
 ?>

ecrire/exec/poster.php

@@ -23,7 +23,7 @@ function exec_poster_dist()
 		minipres(_T('info_acces_interdit'));
 	}
 
-	$f = charger_fonction('poster', 'inc');
-	return $f($id_article, $script, "&id_article=$id_article", true);
+	$poster = charger_fonction('poster', 'inc');
+	ajax_retour($poster($id_article, $script, "&id_article=$id_article", true));
 }
 ?>

ecrire/exec/rechercher.php

@@ -95,12 +95,10 @@ function exec_rechercher_dist()
 			}
 				
 		}
-		if ($ret)
-			return $ret;
-		else
-			return "<div style='padding: 5px; color: red;'><b>"
+		if (!$ret)
+			$ret =  "<div style='padding: 5px; color: red;'><b>"
 			.htmlentities($type)
 			."</b> :  "._T('avis_aucun_resultat')."</div>";
-
+		ajax_retour($ret);
 }
 ?>

ecrire/exec/referencer_traduction.php

@@ -24,7 +24,7 @@ function exec_referencer_traduction_dist()
 
 	$row = spip_fetch_array(spip_query("SELECT id_trad, id_rubrique FROM spip_articles WHERE id_article=$id_article"));
 
-	$f = charger_fonction('referencer_traduction', 'inc');
-	return $f($id_article, 'ajax', $row['id_rubrique'], $row['id_trad']); 
+	$referencer_traduction = charger_fonction('referencer_traduction', 'inc');
+	ajax_retour($referencer_traduction($id_article, 'ajax', $row['id_rubrique'], $row['id_trad'])); 
 }
 ?>