divers peits bugs de l'API de modification (Franz & ESJ)

ecrire/action/editer_article.php

@@ -140,8 +140,8 @@ function revisions_articles ($id_article, $c=false) {
 	if ($r) {
 		spip_query("UPDATE spip_articles SET date_modif=NOW() WHERE id_article="._q($id_article));
 	}
-	
-	return $r;
+
+	return ''; // pas d'erreur
 }
 
 
@@ -159,6 +159,7 @@ function instituer_article($id_article, $c) {
 	$row = spip_fetch_array($s);
 	$id_rubrique = $row['id_rubrique'];
 	$statut_ancien = $statut = $row['statut'];
+	$champs = array();
 
 	$s = _request('statut', $c);
 	if ($s AND _request('statut', $c) != $statut) {
@@ -263,6 +264,8 @@ function instituer_article($id_article, $c) {
 			'data' => $data
 		)
 	);
+
+	return ''; // pas d'erreur
 }
 
 
@@ -308,6 +311,8 @@ function article_referent ($id_article, $c) {
 	else {
 		spip_query("UPDATE spip_articles SET id_trad = $id_lier WHERE id_article = $id_article");
 	}
+
+	return ''; // pas d'erreur
 }
 
 

ecrire/action/editer_breve.php

@@ -74,7 +74,6 @@ function insert_breve($id_rubrique) {
 
 
 // Enregistre une revision de breve
-// $new indique si c'est un INSERT
 // $c est un contenu (par defaut on prend le contenu via _request())
 // http://doc.spip.org/@revisions_breves
 function revisions_breves ($id_breve, $c=false) {

ecrire/inc/modifier.php

@@ -113,19 +113,19 @@ function revision_document($id_document, $c=false) {
 		),
 		$c);
 
+	return ''; // pas d'erreur
 }
 
 
 // http://doc.spip.org/@revision_auteur
 function revision_auteur($id_auteur, $c=false) {
 
-	return modifier_contenu('auteur', $id_auteur,
+	modifier_contenu('auteur', $id_auteur,
 		array(
 			'champs' => array('nom', 'bio', 'pgp', 'nom_site', 'lien_site', 'email'),
 			'nonvide' => array('nom' => _T('ecrire:item_nouvel_auteur'))
 		),
 		$c);
-
 }
 
 // Quand on edite un forum, on tient a conserver l'original
@@ -134,7 +134,7 @@ function conserver_original($id_forum) {
 	$s = spip_query("SELECT id_forum FROM spip_forum WHERE id_parent="._q($id_forum)." AND statut='original'");
 
 	if (spip_num_rows($s))
-		return true;
+		return ''; // pas d'erreur
 
 	// recopier le forum
 	$t = spip_fetch_array(
@@ -145,17 +145,17 @@ function conserver_original($id_forum) {
 	AND spip_query("INSERT spip_forum (date_heure,titre,texte,auteur,email_auteur,nom_site,url_site,ip,id_auteur,idx,id_thread) VALUES (".join(',',array_map('_q', $t)).")")) {
 		$id_copie = spip_insert_id();
 		spip_query("UPDATE spip_forum SET id_parent="._q($id_forum).", statut='original' WHERE id_forum=$id_copie");
-		return true;
+		return ''; // pas d'erreur
 	}
 
-	return false;
+		return '&erreur';
 }
 
 // http://doc.spip.org/@revision_auteur
 function revision_forum($id_forum, $c=false) {
 
-	if (!conserver_original($id_forum)) {
-		spip_log("erreur de sauvegarde de l'original");
+	if ($err = conserver_original($id_forum)) {
+		spip_log("erreur de sauvegarde de l'original, $err");
 		return;
 	}
 
@@ -171,8 +171,6 @@ function revision_forum($id_forum, $c=false) {
 	if ($r) {
 		spip_query("UPDATE spip_forum SET ip="._q($GLOBALS['ip']).", id_auteur="._q($GLOBALS['auteur_session']['id_auteur'])." WHERE id_forum="._q($id_forum));
 	}
-
-	return $r;
 }
 
 

ecrire/inc/revisions.php

@@ -452,6 +452,8 @@ function ajouter_version($id_article, $champs, $titre_version = "", $id_auteur)
 
 	spip_release_lock($lock);
 
+	spip_log("creation version $id_version_new de l'article $id_article $titre_version");
+
 	return $id_version_new;
 }
 
@@ -548,7 +550,6 @@ function enregistrer_premiere_revision($x) {
 			}
 		}
 	}
-
 	return $x;
 }
 
@@ -567,7 +568,7 @@ function enregistrer_nouvelle_revision($x) {
 		if (count($champs))
 			ajouter_version($x['args']['id_objet'], $champs, '', $GLOBALS['auteur_session']['id_auteur']);
 	}
-	
+
 	return $x;
 }
 

ecrire/inc/utils.php

@@ -98,6 +98,7 @@ function include_spip($f, $include = true) {
 // utilisee dans le script pipeline precompile
 // http://doc.spip.org/@minipipe
 function minipipe($fonc,$val){
+
 	// fonction
 	if (function_exists($fonc))
 		$val = call_user_func($fonc, $val);
@@ -147,8 +148,9 @@ function pipeline($action,$val) {
 	}
 	// si le flux est une table qui encapsule donnees et autres
 	// on ne ressort du pipe que les donnees
-	if (is_array($val) && isset($val['data']))
-			$val = $val['data'];
+	// array_key_exists pour php 4.1.0
+	if (is_array($val) && in_array('data', array_keys($val)))
+		$val = $val['data'];
 	return $val;
 }