spip / SPIP

ecrire/action/editer_article.php

Indentation +323 added lines, -323 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -39,36 +39,36 @@ 
  *     Liste (identifiant de l'article, Texte d'erreur éventuel)
  */
 function action_editer_article_dist($arg = null) {
-	include_spip('inc/autoriser');
-	$err = '';
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
-
-	// si id_article n'est pas un nombre, c'est une creation
-	// mais on verifie qu'on a toutes les donnees qu'il faut.
-	if (!$id_article = intval($arg)) {
-		$id_parent = _request('id_parent');
-		if (!$id_parent) {
-			$err = _L("creation interdite d'un article sans rubrique");
-		} elseif (!autoriser('creerarticledans', 'rubrique', $id_parent)) {
-			$err = _T('info_creerdansrubrique_non_autorise');
-		} else {
-			$id_article = article_inserer($id_parent);
-		}
-	}
-
-	// Enregistre l'envoi dans la BD
-	if ($id_article > 0) {
-		$err = article_modifier($id_article);
-	}
-
-	if ($err) {
-		spip_log("echec editeur article: $err", _LOG_ERREUR);
-	}
-
-	return [$id_article, $err];
+    include_spip('inc/autoriser');
+    $err = '';
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
+
+    // si id_article n'est pas un nombre, c'est une creation
+    // mais on verifie qu'on a toutes les donnees qu'il faut.
+    if (!$id_article = intval($arg)) {
+        $id_parent = _request('id_parent');
+        if (!$id_parent) {
+            $err = _L("creation interdite d'un article sans rubrique");
+        } elseif (!autoriser('creerarticledans', 'rubrique', $id_parent)) {
+            $err = _T('info_creerdansrubrique_non_autorise');
+        } else {
+            $id_article = article_inserer($id_parent);
+        }
+    }
+
+    // Enregistre l'envoi dans la BD
+    if ($id_article > 0) {
+        $err = article_modifier($id_article);
+    }
+
+    if ($err) {
+        spip_log("echec editeur article: $err", _LOG_ERREUR);
+    }
+
+    return [$id_article, $err];
 }
 
 /**
@@ -90,50 +90,50 @@ 
  */
 function article_modifier($id_article, $set = null) {
 
-	// unifier $texte en cas de texte trop long
-	trop_longs_articles();
-
-	include_spip('inc/modifier');
-	include_spip('inc/filtres');
-	$c = collecter_requests(
-		// include list
-		objet_info('article', 'champs_editables'),
-		// exclude list
-		['date', 'statut', 'id_parent'],
-		// donnees eventuellement fournies
-		$set
-	);
-
-	// Si l'article est publie, invalider les caches et demander sa reindexation
-	$t = sql_getfetsel('statut', 'spip_articles', 'id_article=' . intval($id_article));
-	$invalideur = $indexation = false;
-	if ($t == 'publie') {
-		$invalideur = "id='article/$id_article'";
-		$indexation = true;
-	}
-
-	if (
-		$err = objet_modifier_champs(
-			'article',
-			$id_article,
-			[
-			'data' => $set,
-			'nonvide' => ['titre' => _T('info_nouvel_article') . ' ' . _T('info_numero_abbreviation') . $id_article],
-			'invalideur' => $invalideur,
-			'indexation' => $indexation,
-			'date_modif' => 'date_modif' // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
-			],
-			$c
-		)
-	) {
-		return $err;
-	}
-
-	// Modification de statut, changement de rubrique ?
-	$c = collecter_requests(['date', 'statut', 'id_parent'], [], $set);
-	$err = article_instituer($id_article, $c);
-
-	return $err;
+    // unifier $texte en cas de texte trop long
+    trop_longs_articles();
+
+    include_spip('inc/modifier');
+    include_spip('inc/filtres');
+    $c = collecter_requests(
+        // include list
+        objet_info('article', 'champs_editables'),
+        // exclude list
+        ['date', 'statut', 'id_parent'],
+        // donnees eventuellement fournies
+        $set
+    );
+
+    // Si l'article est publie, invalider les caches et demander sa reindexation
+    $t = sql_getfetsel('statut', 'spip_articles', 'id_article=' . intval($id_article));
+    $invalideur = $indexation = false;
+    if ($t == 'publie') {
+        $invalideur = "id='article/$id_article'";
+        $indexation = true;
+    }
+
+    if (
+        $err = objet_modifier_champs(
+            'article',
+            $id_article,
+            [
+            'data' => $set,
+            'nonvide' => ['titre' => _T('info_nouvel_article') . ' ' . _T('info_numero_abbreviation') . $id_article],
+            'invalideur' => $invalideur,
+            'indexation' => $indexation,
+            'date_modif' => 'date_modif' // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
+            ],
+            $c
+        )
+    ) {
+        return $err;
+    }
+
+    // Modification de statut, changement de rubrique ?
+    $c = collecter_requests(['date', 'statut', 'id_parent'], [], $set);
+    $err = article_instituer($id_article, $c);
+
+    return $err;
 }
 
 /**
@@ -169,98 +169,98 @@ 
  */
 function article_inserer($id_rubrique, $set = null) {
 
-	// Si id_rubrique vaut 0 ou n'est pas definie, creer l'article
-	// dans la premiere rubrique racine
-	if (!$id_rubrique = intval($id_rubrique)) {
-		$row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0', '', '0+titre,titre', '1');
-		$id_rubrique = $row['id_rubrique'];
-	} else {
-		$row = sql_fetsel('lang, id_secteur', 'spip_rubriques', "id_rubrique=$id_rubrique");
-	}
-
-	// eviter $id_secteur = NULL (erreur sqlite) si la requete precedente echoue
-	// cas de id_rubrique = -1 par exemple avec plugin "pages"
-	$id_secteur = $row['id_secteur'] ?? 0;
-	$lang_rub = $row['lang'] ?? '';
-
-	$lang = '';
-	$choisie = 'non';
-	// La langue a la creation : si les liens de traduction sont autorises
-	// dans les rubriques, on essaie avec la langue de l'auteur,
-	// ou a defaut celle de la rubrique
-	// Sinon c'est la langue de la rubrique qui est choisie + heritee
-	if (
-		!empty($GLOBALS['meta']['multi_objets']) and in_array(
-			'spip_articles',
-			explode(',', $GLOBALS['meta']['multi_objets'])
-		)
-	) {
-		lang_select($GLOBALS['visiteur_session']['lang']);
-		if (
-			in_array(
-				$GLOBALS['spip_lang'],
-				explode(',', $GLOBALS['meta']['langues_multilingue'])
-			)
-		) {
-			$lang = $GLOBALS['spip_lang'];
-			$choisie = 'oui';
-		}
-	}
-
-	if (!$lang) {
-		$choisie = 'non';
-		$lang = $lang_rub ?: $GLOBALS['meta']['langue_site'];
-	}
-
-	$champs = [
-		'id_rubrique' => $id_rubrique,
-		'id_secteur' => $id_secteur,
-		'statut' => 'prepa',
-		'date' => date('Y-m-d H:i:s'),
-		'lang' => $lang,
-		'langue_choisie' => $choisie
-	];
-
-	if ($set) {
-		$champs = array_merge($champs, $set);
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_insertion',
-		[
-			'args' => [
-				'table' => 'spip_articles',
-			],
-			'data' => $champs
-		]
-	);
-
-	$id_article = sql_insertq('spip_articles', $champs);
-
-	// controler si le serveur n'a pas renvoye une erreur
-	if ($id_article > 0) {
-		$id_auteur = ((is_null(_request('id_auteur')) and isset($GLOBALS['visiteur_session']['id_auteur'])) ?
-			$GLOBALS['visiteur_session']['id_auteur']
-			: _request('id_auteur'));
-		if ($id_auteur) {
-			include_spip('action/editer_auteur');
-			auteur_associer($id_auteur, ['article' => $id_article]);
-		}
-	}
-
-	pipeline(
-		'post_insertion',
-		[
-			'args' => [
-				'table' => 'spip_articles',
-				'id_objet' => $id_article
-			],
-			'data' => $champs
-		]
-	);
-
-	return $id_article;
+    // Si id_rubrique vaut 0 ou n'est pas definie, creer l'article
+    // dans la premiere rubrique racine
+    if (!$id_rubrique = intval($id_rubrique)) {
+        $row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0', '', '0+titre,titre', '1');
+        $id_rubrique = $row['id_rubrique'];
+    } else {
+        $row = sql_fetsel('lang, id_secteur', 'spip_rubriques', "id_rubrique=$id_rubrique");
+    }
+
+    // eviter $id_secteur = NULL (erreur sqlite) si la requete precedente echoue
+    // cas de id_rubrique = -1 par exemple avec plugin "pages"
+    $id_secteur = $row['id_secteur'] ?? 0;
+    $lang_rub = $row['lang'] ?? '';
+
+    $lang = '';
+    $choisie = 'non';
+    // La langue a la creation : si les liens de traduction sont autorises
+    // dans les rubriques, on essaie avec la langue de l'auteur,
+    // ou a defaut celle de la rubrique
+    // Sinon c'est la langue de la rubrique qui est choisie + heritee
+    if (
+        !empty($GLOBALS['meta']['multi_objets']) and in_array(
+            'spip_articles',
+            explode(',', $GLOBALS['meta']['multi_objets'])
+        )
+    ) {
+        lang_select($GLOBALS['visiteur_session']['lang']);
+        if (
+            in_array(
+                $GLOBALS['spip_lang'],
+                explode(',', $GLOBALS['meta']['langues_multilingue'])
+            )
+        ) {
+            $lang = $GLOBALS['spip_lang'];
+            $choisie = 'oui';
+        }
+    }
+
+    if (!$lang) {
+        $choisie = 'non';
+        $lang = $lang_rub ?: $GLOBALS['meta']['langue_site'];
+    }
+
+    $champs = [
+        'id_rubrique' => $id_rubrique,
+        'id_secteur' => $id_secteur,
+        'statut' => 'prepa',
+        'date' => date('Y-m-d H:i:s'),
+        'lang' => $lang,
+        'langue_choisie' => $choisie
+    ];
+
+    if ($set) {
+        $champs = array_merge($champs, $set);
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_insertion',
+        [
+            'args' => [
+                'table' => 'spip_articles',
+            ],
+            'data' => $champs
+        ]
+    );
+
+    $id_article = sql_insertq('spip_articles', $champs);
+
+    // controler si le serveur n'a pas renvoye une erreur
+    if ($id_article > 0) {
+        $id_auteur = ((is_null(_request('id_auteur')) and isset($GLOBALS['visiteur_session']['id_auteur'])) ?
+            $GLOBALS['visiteur_session']['id_auteur']
+            : _request('id_auteur'));
+        if ($id_auteur) {
+            include_spip('action/editer_auteur');
+            auteur_associer($id_auteur, ['article' => $id_article]);
+        }
+    }
+
+    pipeline(
+        'post_insertion',
+        [
+            'args' => [
+                'table' => 'spip_articles',
+                'id_objet' => $id_article
+            ],
+            'data' => $champs
+        ]
+    );
+
+    return $id_article;
 }
 
 
@@ -288,125 +288,125 @@ 
  */
 function article_instituer($id_article, $c, $calcul_rub = true) {
 
-	include_spip('inc/autoriser');
-	include_spip('inc/rubriques');
-	include_spip('inc/modifier');
-
-	$row = sql_fetsel('statut, date, id_rubrique', 'spip_articles', "id_article=$id_article");
-	$id_rubrique = $row['id_rubrique'];
-	$statut_ancien = $statut = $row['statut'];
-	$date_ancienne = $date = $row['date'];
-	$champs = [];
-
-	$d = $c['date'] ?? null;
-	$s = $c['statut'] ?? $statut;
-
-	// cf autorisations dans inc/instituer_article
-	if ($s != $statut or ($d and $d != $date)) {
-		if (autoriser('publierdans', 'rubrique', $id_rubrique)) {
-			$statut = $champs['statut'] = $s;
-		} elseif (autoriser('modifier', 'article', $id_article) and $s != 'publie') {
-			$statut = $champs['statut'] = $s;
-		} else {
-			spip_log("editer_article $id_article refus " . join(' ', $c));
-		}
-
-		// En cas de publication, fixer la date a "maintenant"
-		// sauf si $c commande autre chose
-		// ou si l'article est deja date dans le futur
-		// En cas de proposition d'un article (mais pas depublication), idem
-		if (
-			$champs['statut'] == 'publie'
-			or ($champs['statut'] == 'prop' and ($d or !in_array($statut_ancien, ['publie', 'prop'])))
-		) {
-			if ($d or strtotime($d = $date) > time()) {
-				$champs['date'] = $date = $d;
-			} else {
-				$champs['date'] = $date = date('Y-m-d H:i:s');
-			}
-		}
-	}
-
-	// Verifier que la rubrique demandee existe et est differente
-	// de la rubrique actuelle
-	if (
-		isset($c['id_parent'])
-		and $id_parent = $c['id_parent']
-		and $id_parent != $id_rubrique
-		and (sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . intval($id_parent)))
-	) {
-		$champs['id_rubrique'] = $id_parent;
-
-		// si l'article etait publie
-		// et que le demandeur n'est pas admin de la rubrique de destination
-		// repasser l'article en statut 'propose'.
-		if (
-			$statut == 'publie'
-			and !autoriser('publierdans', 'rubrique', $id_parent)
-		) {
-			$champs['statut'] = 'prop';
-		}
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_edition',
-		[
-			'args' => [
-				'table' => 'spip_articles',
-				'id_objet' => $id_article,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-				'date_ancienne' => $date_ancienne,
-			],
-			'data' => $champs
-		]
-	);
-
-	if (!(is_countable($champs) ? count($champs) : 0)) {
-		return '';
-	}
-
-	// Envoyer les modifs.
-	editer_article_heritage($id_article, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
-
-	// Invalider les caches
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='article/$id_article'");
-
-	if ($date) {
-		$t = strtotime($date);
-		$p = @$GLOBALS['meta']['date_prochain_postdate'];
-		if ($t > time() and (!$p or ($t < $p))) {
-			ecrire_meta('date_prochain_postdate', $t);
-		}
-	}
-
-	// Pipeline
-	pipeline(
-		'post_edition',
-		[
-			'args' => [
-				'table' => 'spip_articles',
-				'id_objet' => $id_article,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-				'date_ancienne' => $date_ancienne,
-			],
-			'data' => $champs
-		]
-	);
-
-	// Notifications
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			'instituerarticle',
-			$id_article,
-			['statut' => $statut, 'statut_ancien' => $statut_ancien, 'date' => $date, 'date_ancienne' => $date_ancienne]
-		);
-	}
-
-	return ''; // pas d'erreur
+    include_spip('inc/autoriser');
+    include_spip('inc/rubriques');
+    include_spip('inc/modifier');
+
+    $row = sql_fetsel('statut, date, id_rubrique', 'spip_articles', "id_article=$id_article");
+    $id_rubrique = $row['id_rubrique'];
+    $statut_ancien = $statut = $row['statut'];
+    $date_ancienne = $date = $row['date'];
+    $champs = [];
+
+    $d = $c['date'] ?? null;
+    $s = $c['statut'] ?? $statut;
+
+    // cf autorisations dans inc/instituer_article
+    if ($s != $statut or ($d and $d != $date)) {
+        if (autoriser('publierdans', 'rubrique', $id_rubrique)) {
+            $statut = $champs['statut'] = $s;
+        } elseif (autoriser('modifier', 'article', $id_article) and $s != 'publie') {
+            $statut = $champs['statut'] = $s;
+        } else {
+            spip_log("editer_article $id_article refus " . join(' ', $c));
+        }
+
+        // En cas de publication, fixer la date a "maintenant"
+        // sauf si $c commande autre chose
+        // ou si l'article est deja date dans le futur
+        // En cas de proposition d'un article (mais pas depublication), idem
+        if (
+            $champs['statut'] == 'publie'
+            or ($champs['statut'] == 'prop' and ($d or !in_array($statut_ancien, ['publie', 'prop'])))
+        ) {
+            if ($d or strtotime($d = $date) > time()) {
+                $champs['date'] = $date = $d;
+            } else {
+                $champs['date'] = $date = date('Y-m-d H:i:s');
+            }
+        }
+    }
+
+    // Verifier que la rubrique demandee existe et est differente
+    // de la rubrique actuelle
+    if (
+        isset($c['id_parent'])
+        and $id_parent = $c['id_parent']
+        and $id_parent != $id_rubrique
+        and (sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . intval($id_parent)))
+    ) {
+        $champs['id_rubrique'] = $id_parent;
+
+        // si l'article etait publie
+        // et que le demandeur n'est pas admin de la rubrique de destination
+        // repasser l'article en statut 'propose'.
+        if (
+            $statut == 'publie'
+            and !autoriser('publierdans', 'rubrique', $id_parent)
+        ) {
+            $champs['statut'] = 'prop';
+        }
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_edition',
+        [
+            'args' => [
+                'table' => 'spip_articles',
+                'id_objet' => $id_article,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+                'date_ancienne' => $date_ancienne,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    if (!(is_countable($champs) ? count($champs) : 0)) {
+        return '';
+    }
+
+    // Envoyer les modifs.
+    editer_article_heritage($id_article, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
+
+    // Invalider les caches
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='article/$id_article'");
+
+    if ($date) {
+        $t = strtotime($date);
+        $p = @$GLOBALS['meta']['date_prochain_postdate'];
+        if ($t > time() and (!$p or ($t < $p))) {
+            ecrire_meta('date_prochain_postdate', $t);
+        }
+    }
+
+    // Pipeline
+    pipeline(
+        'post_edition',
+        [
+            'args' => [
+                'table' => 'spip_articles',
+                'id_objet' => $id_article,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+                'date_ancienne' => $date_ancienne,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    // Notifications
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            'instituerarticle',
+            $id_article,
+            ['statut' => $statut, 'statut_ancien' => $statut_ancien, 'date' => $date, 'date_ancienne' => $date_ancienne]
+        );
+    }
+
+    return ''; // pas d'erreur
 }
 
 /**
@@ -431,37 +431,37 @@ 
  */
 function editer_article_heritage($id_article, $id_rubrique, $statut, $champs, $cond = true) {
 
-	// Si on deplace l'article
-	//  changer aussi son secteur et sa langue (si heritee)
-	if (isset($champs['id_rubrique'])) {
-		$row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
-
-		$langue = $row_rub['lang'];
-		$champs['id_secteur'] = $row_rub['id_secteur'];
-		if (
-			sql_fetsel(
-				'1',
-				'spip_articles',
-				'id_article=' . intval($id_article) . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
-			)
-		) {
-			$champs['lang'] = $langue;
-		}
-	}
-
-	if (!$champs) {
-		return;
-	}
-
-	sql_updateq('spip_articles', $champs, 'id_article=' . intval($id_article));
-
-	// Changer le statut des rubriques concernees
-
-	if ($cond) {
-		include_spip('inc/rubriques');
-		$postdate = ($GLOBALS['meta']['post_dates'] == 'non' and isset($champs['date']) and (strtotime($champs['date']) < time())) ? $champs['date'] : false;
-		calculer_rubriques_if($id_rubrique, $champs, ['statut_ancien' => $statut], $postdate);
-	}
+    // Si on deplace l'article
+    //  changer aussi son secteur et sa langue (si heritee)
+    if (isset($champs['id_rubrique'])) {
+        $row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
+
+        $langue = $row_rub['lang'];
+        $champs['id_secteur'] = $row_rub['id_secteur'];
+        if (
+            sql_fetsel(
+                '1',
+                'spip_articles',
+                'id_article=' . intval($id_article) . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
+            )
+        ) {
+            $champs['lang'] = $langue;
+        }
+    }
+
+    if (!$champs) {
+        return;
+    }
+
+    sql_updateq('spip_articles', $champs, 'id_article=' . intval($id_article));
+
+    // Changer le statut des rubriques concernees
+
+    if ($cond) {
+        include_spip('inc/rubriques');
+        $postdate = ($GLOBALS['meta']['post_dates'] == 'non' and isset($champs['date']) and (strtotime($champs['date']) < time())) ? $champs['date'] : false;
+        calculer_rubriques_if($id_rubrique, $champs, ['statut_ancien' => $statut], $postdate);
+    }
 }
 
 /**
@@ -470,10 +470,10 @@ 
  * @return void
  */
 function trop_longs_articles() {
-	if (is_array($plus = _request('texte_plus'))) {
-		foreach ($plus as $n => $t) {
-			$plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,", '', $t);
-		}
-		set_request('texte', join('', $plus) . _request('texte'));
-	}
+    if (is_array($plus = _request('texte_plus'))) {
+        foreach ($plus as $n => $t) {
+            $plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,", '', $t);
+        }
+        set_request('texte', join('', $plus) . _request('texte'));
+    }
 }

ecrire/action/editer_objet.php

Indentation +513 added lines, -513 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -33,36 +33,36 @@ 
  */
 function action_editer_objet_dist($id = null, $objet = null, $set = null) {
 
-	// appel direct depuis une url avec arg = "objet/id"
-	if (is_null($id) or is_null($objet)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-		[$objet, $id] = array_pad(explode('/', $arg, 2), 2, null);
-	}
-
-	// appel incorrect ou depuis une url erronnée interdit
-	if (is_null($id) or is_null($objet)) {
-		include_spip('inc/minipres');
-		echo minipres(_T('info_acces_interdit'));
-		die();
-	}
-
-	// si id n'est pas un nombre, c'est une creation
-	// mais on verifie qu'on a toutes les donnees qu'il faut.
-	if (!$id = intval($id)) {
-		// on ne sait pas si un parent existe mais on essaye
-		$id_parent = _request('id_parent');
-		$id = objet_inserer($objet, $id_parent);
-	}
-
-	if (!($id = intval($id)) > 0) {
-		return [$id, _L('echec enregistrement en base')];
-	}
-
-	// Enregistre l'envoi dans la BD
-	$err = objet_modifier($objet, $id, $set);
-
-	return [$id, $err];
+    // appel direct depuis une url avec arg = "objet/id"
+    if (is_null($id) or is_null($objet)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+        [$objet, $id] = array_pad(explode('/', $arg, 2), 2, null);
+    }
+
+    // appel incorrect ou depuis une url erronnée interdit
+    if (is_null($id) or is_null($objet)) {
+        include_spip('inc/minipres');
+        echo minipres(_T('info_acces_interdit'));
+        die();
+    }
+
+    // si id n'est pas un nombre, c'est une creation
+    // mais on verifie qu'on a toutes les donnees qu'il faut.
+    if (!$id = intval($id)) {
+        // on ne sait pas si un parent existe mais on essaye
+        $id_parent = _request('id_parent');
+        $id = objet_inserer($objet, $id_parent);
+    }
+
+    if (!($id = intval($id)) > 0) {
+        return [$id, _L('echec enregistrement en base')];
+    }
+
+    // Enregistre l'envoi dans la BD
+    $err = objet_modifier($objet, $id, $set);
+
+    return [$id, $err];
 }
 
 /**
@@ -75,85 +75,85 @@ 
  * @return mixed|string
  */
 function objet_modifier($objet, $id, $set = null) {
-	if (($t = objet_type($objet)) !== $objet) {
-		spip_log("objet_modifier: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
-		$objet = $t;
-	}
-	if (
-		include_spip('action/editer_' . $objet)
-		and function_exists($modifier = $objet . '_modifier')
-	) {
-		return $modifier($id, $set);
-	}
-
-	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_sql);
-	if (!$desc or !isset($desc['field'])) {
-		spip_log("Objet $objet inconnu dans objet_modifier", 'editer' . _LOG_ERREUR);
-
-		return _L("Erreur objet $objet inconnu");
-	}
-	include_spip('inc/modifier');
-
-	$champ_date = '';
-	if (isset($desc['date']) and $desc['date']) {
-		$champ_date = $desc['date'];
-	} elseif (isset($desc['field']['date'])) {
-		$champ_date = 'date';
-	}
-
-	$include_list = array_keys($desc['field']);
-	// on ne traite pas la cle primaire par defaut, notamment car
-	// sur une creation, id_x vaut 'oui', et serait enregistre en id_x=0 dans la base
-	$include_list = array_diff($include_list, [$desc['key']['PRIMARY KEY']]);
-
-	if (isset($desc['champs_editables']) and is_array($desc['champs_editables'])) {
-		$include_list = $desc['champs_editables'];
-	}
-	$c = collecter_requests(
-		// include list
-		$include_list,
-		// exclude list
-		[$champ_date, 'statut', 'id_parent', 'id_secteur'],
-		// donnees eventuellement fournies
-		$set
-	);
-
-	// Si l'objet est publie, invalider les caches et demander sa reindexation
-	if (objet_test_si_publie($objet, $id)) {
-		$invalideur = "id='$objet/$id'";
-		$indexation = true;
-	} else {
-		$invalideur = '';
-		$indexation = false;
-	}
-
-	if (
-		$err = objet_modifier_champs(
-			$objet,
-			$id,
-			[
-			'data' => $set,
-			'nonvide' => '',
-			'invalideur' => $invalideur,
-			'indexation' => $indexation,
-			// champ a mettre a date('Y-m-d H:i:s') s'il y a modif
-			'date_modif' => (isset($desc['field']['date_modif']) ? 'date_modif' : '')
-			],
-			$c
-		)
-	) {
-		return $err;
-	}
-
-	// Modification de statut, changement de rubrique ?
-	// FIXME: Ici lorsqu'un $set est passé, la fonction collecter_requests() retourne tout
-	//         le tableau $set hors liste d’exclusion, mais du coup on a possiblement des champs en trop.
-	$c = collecter_requests([$champ_date, 'statut', 'id_parent'], [], $set);
-	$err = objet_instituer($objet, $id, $c);
-
-	return $err;
+    if (($t = objet_type($objet)) !== $objet) {
+        spip_log("objet_modifier: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
+        $objet = $t;
+    }
+    if (
+        include_spip('action/editer_' . $objet)
+        and function_exists($modifier = $objet . '_modifier')
+    ) {
+        return $modifier($id, $set);
+    }
+
+    $table_sql = table_objet_sql($objet);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_sql);
+    if (!$desc or !isset($desc['field'])) {
+        spip_log("Objet $objet inconnu dans objet_modifier", 'editer' . _LOG_ERREUR);
+
+        return _L("Erreur objet $objet inconnu");
+    }
+    include_spip('inc/modifier');
+
+    $champ_date = '';
+    if (isset($desc['date']) and $desc['date']) {
+        $champ_date = $desc['date'];
+    } elseif (isset($desc['field']['date'])) {
+        $champ_date = 'date';
+    }
+
+    $include_list = array_keys($desc['field']);
+    // on ne traite pas la cle primaire par defaut, notamment car
+    // sur une creation, id_x vaut 'oui', et serait enregistre en id_x=0 dans la base
+    $include_list = array_diff($include_list, [$desc['key']['PRIMARY KEY']]);
+
+    if (isset($desc['champs_editables']) and is_array($desc['champs_editables'])) {
+        $include_list = $desc['champs_editables'];
+    }
+    $c = collecter_requests(
+        // include list
+        $include_list,
+        // exclude list
+        [$champ_date, 'statut', 'id_parent', 'id_secteur'],
+        // donnees eventuellement fournies
+        $set
+    );
+
+    // Si l'objet est publie, invalider les caches et demander sa reindexation
+    if (objet_test_si_publie($objet, $id)) {
+        $invalideur = "id='$objet/$id'";
+        $indexation = true;
+    } else {
+        $invalideur = '';
+        $indexation = false;
+    }
+
+    if (
+        $err = objet_modifier_champs(
+            $objet,
+            $id,
+            [
+            'data' => $set,
+            'nonvide' => '',
+            'invalideur' => $invalideur,
+            'indexation' => $indexation,
+            // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
+            'date_modif' => (isset($desc['field']['date_modif']) ? 'date_modif' : '')
+            ],
+            $c
+        )
+    ) {
+        return $err;
+    }
+
+    // Modification de statut, changement de rubrique ?
+    // FIXME: Ici lorsqu'un $set est passé, la fonction collecter_requests() retourne tout
+    //         le tableau $set hors liste d’exclusion, mais du coup on a possiblement des champs en trop.
+    $c = collecter_requests([$champ_date, 'statut', 'id_parent'], [], $set);
+    $err = objet_instituer($objet, $id, $c);
+
+    return $err;
 }
 
 /**
@@ -168,135 +168,135 @@ 
  * @return bool|int
  */
 function objet_inserer($objet, $id_parent = null, $set = null) {
-	$d = null;
-	if (($t = objet_type($objet)) !== $objet) {
-		spip_log("objet_inserer: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
-		$objet = $t;
-	}
-	if (
-		include_spip('action/editer_' . $objet)
-		and function_exists($inserer = $objet . '_inserer')
-	) {
-		return $inserer($id_parent, $set);
-	}
-
-	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_sql);
-	if (!$desc or !isset($desc['field'])) {
-		return 0;
-	}
-
-	$lang_rub = '';
-	$champs = [];
-	if (isset($desc['field']['id_rubrique'])) {
-		// Si id_rubrique vaut 0 ou n'est pas definie, creer l'objet
-		// dans la premiere rubrique racine
-		if (!$id_rubrique = intval($id_parent)) {
-			$row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0', '', '0+titre,titre', '1');
-			$id_rubrique = $row['id_rubrique'];
-		} else {
-			$row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique));
-		}
-
-		$champs['id_rubrique'] = $id_rubrique;
-		if (isset($desc['field']['id_secteur'])) {
-			$champs['id_secteur'] = $row['id_secteur'];
-		}
-		$lang_rub = $row['lang'];
-	}
-
-	// La langue a la creation : si les liens de traduction sont autorises
-	// dans les rubriques, on essaie avec la langue de l'auteur,
-	// ou a defaut celle de la rubrique
-	// Sinon c'est la langue de la rubrique qui est choisie + heritee
-	if (
-		isset($desc['field']['lang']) and !empty($GLOBALS['meta']['multi_objets']) and in_array(
-			$table_sql,
-			explode(',', $GLOBALS['meta']['multi_objets'])
-		)
-	) {
-		lang_select($GLOBALS['visiteur_session']['lang']);
-		if (
-			in_array(
-				$GLOBALS['spip_lang'],
-				explode(',', $GLOBALS['meta']['langues_multilingue'])
-			)
-		) {
-			$champs['lang'] = $GLOBALS['spip_lang'];
-			if (isset($desc['field']['langue_choisie'])) {
-				$champs['langue_choisie'] = 'oui';
-			}
-		}
-	} elseif (isset($desc['field']['lang']) and isset($desc['field']['langue_choisie'])) {
-		$champs['lang'] = ($lang_rub ?: $GLOBALS['meta']['langue_site']);
-		$champs['langue_choisie'] = 'non';
-	}
-
-	if (isset($desc['field']['statut'])) {
-		if (isset($desc['statut_textes_instituer'])) {
-			$cles_statut = array_keys($desc['statut_textes_instituer']);
-			$champs['statut'] = reset($cles_statut);
-		} else {
-			$champs['statut'] = 'prepa';
-		}
-	}
-
-
-	if ((isset($desc['date']) and $d = $desc['date']) or isset($desc['field'][$d = 'date'])) {
-		$champs[$d] = date('Y-m-d H:i:s');
-	}
-
-	if ($set) {
-		$champs = array_merge($champs, $set);
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_insertion',
-		[
-			'args' => [
-				'table' => $table_sql,
-				'id_parent' => $id_parent,
-			],
-			'data' => $champs
-		]
-	);
-
-	$id = sql_insertq($table_sql, $champs);
-
-	if ($id) {
-		// controler si le serveur n'a pas renvoye une erreur
-		// et associer l'auteur sinon
-		// si la table n'a pas deja un champ id_auteur
-		// et si le form n'a pas poste un id_auteur (meme vide, ce qui sert a annuler cette auto association)
-		if (
-			$id > 0
-			and !isset($desc['field']['id_auteur'])
-		) {
-			$id_auteur = ((is_null(_request('id_auteur')) and isset($GLOBALS['visiteur_session']['id_auteur'])) ?
-				$GLOBALS['visiteur_session']['id_auteur']
-				: _request('id_auteur'));
-			if ($id_auteur) {
-				include_spip('action/editer_auteur');
-				auteur_associer($id_auteur, [$objet => $id]);
-			}
-		}
-
-		pipeline(
-			'post_insertion',
-			[
-				'args' => [
-					'table' => $table_sql,
-					'id_parent' => $id_parent,
-					'id_objet' => $id,
-				],
-				'data' => $champs
-			]
-		);
-	}
-
-	return $id;
+    $d = null;
+    if (($t = objet_type($objet)) !== $objet) {
+        spip_log("objet_inserer: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
+        $objet = $t;
+    }
+    if (
+        include_spip('action/editer_' . $objet)
+        and function_exists($inserer = $objet . '_inserer')
+    ) {
+        return $inserer($id_parent, $set);
+    }
+
+    $table_sql = table_objet_sql($objet);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_sql);
+    if (!$desc or !isset($desc['field'])) {
+        return 0;
+    }
+
+    $lang_rub = '';
+    $champs = [];
+    if (isset($desc['field']['id_rubrique'])) {
+        // Si id_rubrique vaut 0 ou n'est pas definie, creer l'objet
+        // dans la premiere rubrique racine
+        if (!$id_rubrique = intval($id_parent)) {
+            $row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0', '', '0+titre,titre', '1');
+            $id_rubrique = $row['id_rubrique'];
+        } else {
+            $row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique));
+        }
+
+        $champs['id_rubrique'] = $id_rubrique;
+        if (isset($desc['field']['id_secteur'])) {
+            $champs['id_secteur'] = $row['id_secteur'];
+        }
+        $lang_rub = $row['lang'];
+    }
+
+    // La langue a la creation : si les liens de traduction sont autorises
+    // dans les rubriques, on essaie avec la langue de l'auteur,
+    // ou a defaut celle de la rubrique
+    // Sinon c'est la langue de la rubrique qui est choisie + heritee
+    if (
+        isset($desc['field']['lang']) and !empty($GLOBALS['meta']['multi_objets']) and in_array(
+            $table_sql,
+            explode(',', $GLOBALS['meta']['multi_objets'])
+        )
+    ) {
+        lang_select($GLOBALS['visiteur_session']['lang']);
+        if (
+            in_array(
+                $GLOBALS['spip_lang'],
+                explode(',', $GLOBALS['meta']['langues_multilingue'])
+            )
+        ) {
+            $champs['lang'] = $GLOBALS['spip_lang'];
+            if (isset($desc['field']['langue_choisie'])) {
+                $champs['langue_choisie'] = 'oui';
+            }
+        }
+    } elseif (isset($desc['field']['lang']) and isset($desc['field']['langue_choisie'])) {
+        $champs['lang'] = ($lang_rub ?: $GLOBALS['meta']['langue_site']);
+        $champs['langue_choisie'] = 'non';
+    }
+
+    if (isset($desc['field']['statut'])) {
+        if (isset($desc['statut_textes_instituer'])) {
+            $cles_statut = array_keys($desc['statut_textes_instituer']);
+            $champs['statut'] = reset($cles_statut);
+        } else {
+            $champs['statut'] = 'prepa';
+        }
+    }
+
+
+    if ((isset($desc['date']) and $d = $desc['date']) or isset($desc['field'][$d = 'date'])) {
+        $champs[$d] = date('Y-m-d H:i:s');
+    }
+
+    if ($set) {
+        $champs = array_merge($champs, $set);
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_insertion',
+        [
+            'args' => [
+                'table' => $table_sql,
+                'id_parent' => $id_parent,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    $id = sql_insertq($table_sql, $champs);
+
+    if ($id) {
+        // controler si le serveur n'a pas renvoye une erreur
+        // et associer l'auteur sinon
+        // si la table n'a pas deja un champ id_auteur
+        // et si le form n'a pas poste un id_auteur (meme vide, ce qui sert a annuler cette auto association)
+        if (
+            $id > 0
+            and !isset($desc['field']['id_auteur'])
+        ) {
+            $id_auteur = ((is_null(_request('id_auteur')) and isset($GLOBALS['visiteur_session']['id_auteur'])) ?
+                $GLOBALS['visiteur_session']['id_auteur']
+                : _request('id_auteur'));
+            if ($id_auteur) {
+                include_spip('action/editer_auteur');
+                auteur_associer($id_auteur, [$objet => $id]);
+            }
+        }
+
+        pipeline(
+            'post_insertion',
+            [
+                'args' => [
+                    'table' => $table_sql,
+                    'id_parent' => $id_parent,
+                    'id_objet' => $id,
+                ],
+                'data' => $champs
+            ]
+        );
+    }
+
+    return $id;
 }
 
 
@@ -313,138 +313,138 @@ 
  * @return string
  */
 function objet_instituer($objet, $id, $c, $calcul_rub = true) {
-	if (($t = objet_type($objet)) !== $objet) {
-		spip_log("objet_instituer: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
-		$objet = $t;
-	}
-	if (
-		include_spip('action/editer_' . $objet)
-		and function_exists($instituer = $objet . '_instituer')
-	) {
-		return $instituer($id, $c, $calcul_rub);
-	}
-
-	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_sql);
-	if (!$desc or !isset($desc['field'])) {
-		return _L("Impossible d'instituer $objet : non connu en base");
-	}
-
-	include_spip('inc/autoriser');
-	include_spip('inc/rubriques');
-	include_spip('inc/modifier');
-
-	$sel = [];
-	$sel[] = (isset($desc['field']['statut']) ? 'statut' : "'' as statut");
-
-	$champ_date = '';
-	if (isset($desc['date']) and $desc['date']) {
-		$champ_date = $desc['date'];
-	} elseif (isset($desc['field']['date'])) {
-		$champ_date = 'date';
-	}
-
-	$sel[] = ($champ_date ? "$champ_date as date" : "'' as date");
-	$sel[] = (isset($desc['field']['id_rubrique']) ? 'id_rubrique' : '0 as id_rubrique');
-
-	$row = sql_fetsel($sel, $table_sql, id_table_objet($objet) . '=' . intval($id));
-
-	$id_rubrique = $row['id_rubrique'];
-	$statut_ancien = $statut = $row['statut'];
-	$date_ancienne = $date = $row['date'];
-	$champs = [];
-
-	$d = ($date and isset($c[$champ_date])) ? $c[$champ_date] : null;
-	$s = (isset($desc['field']['statut']) and isset($c['statut'])) ? $c['statut'] : $statut;
-
-	// cf autorisations dans inc/instituer_objet
-	if ($s != $statut or ($d and $d != $date)) {
-		if (
-			$id_rubrique ?
-			autoriser('publierdans', 'rubrique', $id_rubrique)
-			:
-			autoriser('instituer', $objet, $id, null, ['statut' => $s])
-		) {
-			$statut = $champs['statut'] = $s;
-		} else {
-			if ($s != 'publie' and autoriser('modifier', $objet, $id)) {
-				$statut = $champs['statut'] = $s;
-			} else {
-				spip_log("editer_objet $objet #$id refus " . json_encode($c, JSON_THROW_ON_ERROR), 'editer' . _LOG_INFO_IMPORTANTE);
-			}
-		}
-
-		// En cas de publication, fixer la date a "maintenant"
-		// sauf si $c commande autre chose
-		// ou si l'objet est deja date dans le futur
-		// En cas de proposition d'un objet (mais pas depublication), idem
-		if ($champ_date) {
-			if (
-				$champs['statut'] == 'publie'
-				or ($champs['statut'] == 'prop' and !in_array($statut_ancien, ['publie', 'prop']))
-				or $d
-			) {
-				if ($d or strtotime($d = $date) > time()) {
-					$champs[$champ_date] = $date = $d;
-				} else {
-					$champs[$champ_date] = $date = date('Y-m-d H:i:s');
-				}
-			}
-		}
-	}
-
-	// Verifier que la rubrique demandee existe et est differente
-	// de la rubrique actuelle
-	if (
-		$id_rubrique
-		and isset($c['id_parent'])
-		and $id_parent = $c['id_parent']
-		and $id_parent != $id_rubrique
-		and (sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . intval($id_parent)))
-	) {
-		$champs['id_rubrique'] = $id_parent;
-
-		// si l'objet etait publie
-		// et que le demandeur n'est pas admin de la rubrique
-		// repasser l'objet en statut 'propose'.
-		if (
-			$statut == 'publie'
-			and !autoriser('publierdans', 'rubrique', $id_rubrique)
-		) {
-			$champs['statut'] = 'prop';
-		}
-	}
-
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_edition',
-		[
-			'args' => [
-				'table' => $table_sql,
-				'id_objet' => $id,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-				'date_ancienne' => $date_ancienne,
-				'id_parent_ancien' => $id_rubrique,
-			],
-			'data' => $champs
-		]
-	);
-
-	if (!(is_countable($champs) ? count($champs) : 0)) {
-		return '';
-	}
-
-	// Envoyer les modifs.
-	objet_editer_heritage($objet, $id, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
-
-	// Invalider les caches
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='$objet/$id'");
-
-	/*
+    if (($t = objet_type($objet)) !== $objet) {
+        spip_log("objet_instituer: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
+        $objet = $t;
+    }
+    if (
+        include_spip('action/editer_' . $objet)
+        and function_exists($instituer = $objet . '_instituer')
+    ) {
+        return $instituer($id, $c, $calcul_rub);
+    }
+
+    $table_sql = table_objet_sql($objet);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_sql);
+    if (!$desc or !isset($desc['field'])) {
+        return _L("Impossible d'instituer $objet : non connu en base");
+    }
+
+    include_spip('inc/autoriser');
+    include_spip('inc/rubriques');
+    include_spip('inc/modifier');
+
+    $sel = [];
+    $sel[] = (isset($desc['field']['statut']) ? 'statut' : "'' as statut");
+
+    $champ_date = '';
+    if (isset($desc['date']) and $desc['date']) {
+        $champ_date = $desc['date'];
+    } elseif (isset($desc['field']['date'])) {
+        $champ_date = 'date';
+    }
+
+    $sel[] = ($champ_date ? "$champ_date as date" : "'' as date");
+    $sel[] = (isset($desc['field']['id_rubrique']) ? 'id_rubrique' : '0 as id_rubrique');
+
+    $row = sql_fetsel($sel, $table_sql, id_table_objet($objet) . '=' . intval($id));
+
+    $id_rubrique = $row['id_rubrique'];
+    $statut_ancien = $statut = $row['statut'];
+    $date_ancienne = $date = $row['date'];
+    $champs = [];
+
+    $d = ($date and isset($c[$champ_date])) ? $c[$champ_date] : null;
+    $s = (isset($desc['field']['statut']) and isset($c['statut'])) ? $c['statut'] : $statut;
+
+    // cf autorisations dans inc/instituer_objet
+    if ($s != $statut or ($d and $d != $date)) {
+        if (
+            $id_rubrique ?
+            autoriser('publierdans', 'rubrique', $id_rubrique)
+            :
+            autoriser('instituer', $objet, $id, null, ['statut' => $s])
+        ) {
+            $statut = $champs['statut'] = $s;
+        } else {
+            if ($s != 'publie' and autoriser('modifier', $objet, $id)) {
+                $statut = $champs['statut'] = $s;
+            } else {
+                spip_log("editer_objet $objet #$id refus " . json_encode($c, JSON_THROW_ON_ERROR), 'editer' . _LOG_INFO_IMPORTANTE);
+            }
+        }
+
+        // En cas de publication, fixer la date a "maintenant"
+        // sauf si $c commande autre chose
+        // ou si l'objet est deja date dans le futur
+        // En cas de proposition d'un objet (mais pas depublication), idem
+        if ($champ_date) {
+            if (
+                $champs['statut'] == 'publie'
+                or ($champs['statut'] == 'prop' and !in_array($statut_ancien, ['publie', 'prop']))
+                or $d
+            ) {
+                if ($d or strtotime($d = $date) > time()) {
+                    $champs[$champ_date] = $date = $d;
+                } else {
+                    $champs[$champ_date] = $date = date('Y-m-d H:i:s');
+                }
+            }
+        }
+    }
+
+    // Verifier que la rubrique demandee existe et est differente
+    // de la rubrique actuelle
+    if (
+        $id_rubrique
+        and isset($c['id_parent'])
+        and $id_parent = $c['id_parent']
+        and $id_parent != $id_rubrique
+        and (sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . intval($id_parent)))
+    ) {
+        $champs['id_rubrique'] = $id_parent;
+
+        // si l'objet etait publie
+        // et que le demandeur n'est pas admin de la rubrique
+        // repasser l'objet en statut 'propose'.
+        if (
+            $statut == 'publie'
+            and !autoriser('publierdans', 'rubrique', $id_rubrique)
+        ) {
+            $champs['statut'] = 'prop';
+        }
+    }
+
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_edition',
+        [
+            'args' => [
+                'table' => $table_sql,
+                'id_objet' => $id,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+                'date_ancienne' => $date_ancienne,
+                'id_parent_ancien' => $id_rubrique,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    if (!(is_countable($champs) ? count($champs) : 0)) {
+        return '';
+    }
+
+    // Envoyer les modifs.
+    objet_editer_heritage($objet, $id, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
+
+    // Invalider les caches
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='$objet/$id'");
+
+    /*
 	if ($date) {
 		$t = strtotime($date);
 		$p = @$GLOBALS['meta']['date_prochain_postdate'];
@@ -453,32 +453,32 @@ 
 		}
 	}*/
 
-	// Pipeline
-	pipeline(
-		'post_edition',
-		[
-			'args' => [
-				'table' => $table_sql,
-				'id_objet' => $id,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-				'date_ancienne' => $date_ancienne,
-				'id_parent_ancien' => $id_rubrique,
-			],
-			'data' => $champs
-		]
-	);
-
-	// Notifications
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			"instituer$objet",
-			$id,
-			['statut' => $statut, 'statut_ancien' => $statut_ancien, 'date' => $date, 'date_ancienne' => $date_ancienne]
-		);
-	}
-
-	return ''; // pas d'erreur
+    // Pipeline
+    pipeline(
+        'post_edition',
+        [
+            'args' => [
+                'table' => $table_sql,
+                'id_objet' => $id,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+                'date_ancienne' => $date_ancienne,
+                'id_parent_ancien' => $id_rubrique,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    // Notifications
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            "instituer$objet",
+            $id,
+            ['statut' => $statut, 'statut_ancien' => $statut_ancien, 'date' => $date, 'date_ancienne' => $date_ancienne]
+        );
+    }
+
+    return ''; // pas d'erreur
 }
 
 /**
@@ -493,51 +493,51 @@ 
  * @return void
  */
 function objet_editer_heritage($objet, $id, $id_rubrique, $statut, $champs, $cond = true) {
-	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_sql);
-
-	// Si on deplace l'objet
-	// changer aussi son secteur et sa langue (si heritee)
-	if (isset($champs['id_rubrique'])) {
-		$row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
-		$langue = $row_rub['lang'];
-
-		if (isset($desc['field']['id_secteur'])) {
-			$champs['id_secteur'] = $row_rub['id_secteur'];
-		}
-
-		if (isset($desc['field']['lang']) and isset($desc['field']['langue_choisie'])) {
-			if (
-				sql_fetsel(
-					'1',
-					$table_sql,
-					id_table_objet($objet) . '=' . intval($id) . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
-				)
-			) {
-				$champs['lang'] = $langue;
-			}
-		}
-	}
-
-	if (!$champs) {
-		return;
-	}
-	sql_updateq($table_sql, $champs, id_table_objet($objet) . '=' . intval($id));
-
-	// Changer le statut des rubriques concernees
-	if ($cond) {
-		include_spip('inc/rubriques');
-		//$postdate = ($GLOBALS['meta']["post_dates"] == "non" AND isset($champs['date']) AND (strtotime($champs['date']) < time()))?$champs['date']:false;
-		$postdate = false;
-		// On rajoute les infos de l'objet
-		$infos = [
-			'objet' => $objet,
-			'id_objet' => $id,
-			'statut_ancien' => $statut,
-		];
-		calculer_rubriques_if($id_rubrique, $champs, $infos, $postdate);
-	}
+    $table_sql = table_objet_sql($objet);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_sql);
+
+    // Si on deplace l'objet
+    // changer aussi son secteur et sa langue (si heritee)
+    if (isset($champs['id_rubrique'])) {
+        $row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
+        $langue = $row_rub['lang'];
+
+        if (isset($desc['field']['id_secteur'])) {
+            $champs['id_secteur'] = $row_rub['id_secteur'];
+        }
+
+        if (isset($desc['field']['lang']) and isset($desc['field']['langue_choisie'])) {
+            if (
+                sql_fetsel(
+                    '1',
+                    $table_sql,
+                    id_table_objet($objet) . '=' . intval($id) . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
+                )
+            ) {
+                $champs['lang'] = $langue;
+            }
+        }
+    }
+
+    if (!$champs) {
+        return;
+    }
+    sql_updateq($table_sql, $champs, id_table_objet($objet) . '=' . intval($id));
+
+    // Changer le statut des rubriques concernees
+    if ($cond) {
+        include_spip('inc/rubriques');
+        //$postdate = ($GLOBALS['meta']["post_dates"] == "non" AND isset($champs['date']) AND (strtotime($champs['date']) < time()))?$champs['date']:false;
+        $postdate = false;
+        // On rajoute les infos de l'objet
+        $infos = [
+            'objet' => $objet,
+            'id_objet' => $id,
+            'statut_ancien' => $statut,
+        ];
+        calculer_rubriques_if($id_rubrique, $champs, $infos, $postdate);
+    }
 }
 
 
@@ -566,75 +566,75 @@ 
  *     string|int : valeur du champ demande pour l'objet demande
  */
 function objet_lire($objet, $valeur_id, $options = []) {
-	if (($t = objet_type($objet)) !== $objet) {
-		spip_log("objet_lire: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
-		$objet = $t;
-	}
-
-	// tableau du cache des descriptions et des id d'objet (au sens id_xxx).
-	// Les tableaux sont toujours indexés par le trio [objet][cle][valeur_cle]
-	static $descriptions = [];
-
-	// On détermine le nom du champ id de la table.
-	include_spip('base/objets');
-	$primary = id_table_objet($objet);
-
-	// On détermine l'id à utiliser.
-	$champ_id = (!empty($options['champ_id']) ? $options['champ_id'] : $primary);
-
-	// Si l'objet n'a pas encore été stocké, il faut récupérer sa description complète.
-	if (
-		!isset($descriptions[$objet][$champ_id][$valeur_id])
-		or (isset($options['force']) and $options['force'])
-	) {
-		// Il est possible pour un type d'objet de fournir une fonction de lecture de tous les champs d'un objet.
-		if (
-			include_spip('action/editer_' . $objet)
-			and function_exists($lire = "${objet}_lire_champs")
-		) {
-			$valeurs = $lire($objet, $valeur_id, $champ_id);
-		} else {
-			// On récupère la table SQL à partir du type d'objet.
-			$table = table_objet_sql($objet);
-
-			// La condition est appliquée sur le champ désigné par l'utilisateur.
-			$where = [
-				"${champ_id}=" . sql_quote($valeur_id)
-			];
-
-			// Acquisition de tous les champs de l'objet : si l'accès SQL retourne une erreur on renvoie un tableau vide.
-			$valeurs = sql_fetsel('*', $table, $where);
-		}
-
-		if (!$valeurs) {
-			$valeurs = false;
-		}
-
-		$descriptions[$objet][$champ_id][$valeur_id] = $valeurs;
-
-		if ($champ_id !== $primary and isset($valeurs[$primary])) {
-			$descriptions[$objet][$primary][$valeurs[$primary]] = $valeurs;
-			$descriptions[$objet][$champ_id][$valeur_id] = &$descriptions[$objet][$primary][$valeurs[$primary]];
-		}
-	}
-
-	$retour = $descriptions[$objet][$champ_id][$valeur_id];
-
-	// On ne retourne maintenant que les champs demandés.
-	// - on détermine les informations à renvoyer.
-	if ($retour and !empty($options['champs'])) {
-		$champs = $options['champs'];
-		// Extraction des seules informations demandées.
-		// -- si on demande une information unique on renvoie la valeur simple, sinon on renvoie un tableau.
-		// -- si une information n'est pas un champ valide elle n'est pas renvoyée sans renvoyer d'erreur.
-		if (is_array($champs)) {
-			// Tableau des informations valides
-			$retour = array_intersect_key($retour, array_flip($champs));
-		} else {
-			// Valeur unique demandée.
-			$retour = ($retour[$champs] ?? false);
-		}
-	}
-
-	return $retour;
+    if (($t = objet_type($objet)) !== $objet) {
+        spip_log("objet_lire: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
+        $objet = $t;
+    }
+
+    // tableau du cache des descriptions et des id d'objet (au sens id_xxx).
+    // Les tableaux sont toujours indexés par le trio [objet][cle][valeur_cle]
+    static $descriptions = [];
+
+    // On détermine le nom du champ id de la table.
+    include_spip('base/objets');
+    $primary = id_table_objet($objet);
+
+    // On détermine l'id à utiliser.
+    $champ_id = (!empty($options['champ_id']) ? $options['champ_id'] : $primary);
+
+    // Si l'objet n'a pas encore été stocké, il faut récupérer sa description complète.
+    if (
+        !isset($descriptions[$objet][$champ_id][$valeur_id])
+        or (isset($options['force']) and $options['force'])
+    ) {
+        // Il est possible pour un type d'objet de fournir une fonction de lecture de tous les champs d'un objet.
+        if (
+            include_spip('action/editer_' . $objet)
+            and function_exists($lire = "${objet}_lire_champs")
+        ) {
+            $valeurs = $lire($objet, $valeur_id, $champ_id);
+        } else {
+            // On récupère la table SQL à partir du type d'objet.
+            $table = table_objet_sql($objet);
+
+            // La condition est appliquée sur le champ désigné par l'utilisateur.
+            $where = [
+                "${champ_id}=" . sql_quote($valeur_id)
+            ];
+
+            // Acquisition de tous les champs de l'objet : si l'accès SQL retourne une erreur on renvoie un tableau vide.
+            $valeurs = sql_fetsel('*', $table, $where);
+        }
+
+        if (!$valeurs) {
+            $valeurs = false;
+        }
+
+        $descriptions[$objet][$champ_id][$valeur_id] = $valeurs;
+
+        if ($champ_id !== $primary and isset($valeurs[$primary])) {
+            $descriptions[$objet][$primary][$valeurs[$primary]] = $valeurs;
+            $descriptions[$objet][$champ_id][$valeur_id] = &$descriptions[$objet][$primary][$valeurs[$primary]];
+        }
+    }
+
+    $retour = $descriptions[$objet][$champ_id][$valeur_id];
+
+    // On ne retourne maintenant que les champs demandés.
+    // - on détermine les informations à renvoyer.
+    if ($retour and !empty($options['champs'])) {
+        $champs = $options['champs'];
+        // Extraction des seules informations demandées.
+        // -- si on demande une information unique on renvoie la valeur simple, sinon on renvoie un tableau.
+        // -- si une information n'est pas un champ valide elle n'est pas renvoyée sans renvoyer d'erreur.
+        if (is_array($champs)) {
+            // Tableau des informations valides
+            $retour = array_intersect_key($retour, array_flip($champs));
+        } else {
+            // Valeur unique demandée.
+            $retour = ($retour[$champs] ?? false);
+        }
+    }
+
+    return $retour;
 }

ecrire/action/editer_rubrique.php

Indentation +170 added lines, -170 removed lines

@@ -16,7 +16,7 @@ 
  * @package SPIP\Core\Rubriques\Edition
  */
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/rubriques');
@@ -38,34 +38,34 @@ 
  */
 function action_editer_rubrique_dist($arg = null) {
 
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
-
-	if (!$id_rubrique = intval($arg)) {
-		if ($arg != 'oui') {
-			include_spip('inc/headers');
-			redirige_url_ecrire();
-		}
-		$id_rubrique = rubrique_inserer(_request('id_parent'));
-	}
-
-	$err = rubrique_modifier($id_rubrique);
-
-	if (_request('redirect')) {
-		$redirect = parametre_url(
-			urldecode(_request('redirect')),
-			'id_rubrique',
-			$id_rubrique,
-			'&'
-		);
-
-		include_spip('inc/headers');
-		redirige_par_entete($redirect);
-	}
-
-	return [$id_rubrique, $err];
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
+
+    if (!$id_rubrique = intval($arg)) {
+        if ($arg != 'oui') {
+            include_spip('inc/headers');
+            redirige_url_ecrire();
+        }
+        $id_rubrique = rubrique_inserer(_request('id_parent'));
+    }
+
+    $err = rubrique_modifier($id_rubrique);
+
+    if (_request('redirect')) {
+        $redirect = parametre_url(
+            urldecode(_request('redirect')),
+            'id_rubrique',
+            $id_rubrique,
+            '&'
+        );
+
+        include_spip('inc/headers');
+        redirige_par_entete($redirect);
+    }
+
+    return [$id_rubrique, $err];
 }
 
 
@@ -80,42 +80,42 @@ 
  *     Identifiant de la rubrique crée
  */
 function rubrique_inserer($id_parent, $set = null) {
-	$champs = [
-		'titre' => _T('item_nouvelle_rubrique'),
-		'id_parent' => intval($id_parent),
-		'statut' => 'prepa'
-	];
-
-	if ($set) {
-		$champs = array_merge($champs, $set);
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_insertion',
-		[
-			'args' => [
-				'table' => 'spip_rubriques',
-			],
-			'data' => $champs
-		]
-	);
-
-	$id_rubrique = sql_insertq('spip_rubriques', $champs);
-	pipeline(
-		'post_insertion',
-		[
-			'args' => [
-				'table' => 'spip_rubriques',
-				'id_objet' => $id_rubrique
-			],
-			'data' => $champs
-		]
-	);
-	propager_les_secteurs();
-	calculer_langues_rubriques();
-
-	return $id_rubrique;
+    $champs = [
+        'titre' => _T('item_nouvelle_rubrique'),
+        'id_parent' => intval($id_parent),
+        'statut' => 'prepa'
+    ];
+
+    if ($set) {
+        $champs = array_merge($champs, $set);
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_insertion',
+        [
+            'args' => [
+                'table' => 'spip_rubriques',
+            ],
+            'data' => $champs
+        ]
+    );
+
+    $id_rubrique = sql_insertq('spip_rubriques', $champs);
+    pipeline(
+        'post_insertion',
+        [
+            'args' => [
+                'table' => 'spip_rubriques',
+                'id_objet' => $id_rubrique
+            ],
+            'data' => $champs
+        ]
+    );
+    propager_les_secteurs();
+    calculer_langues_rubriques();
+
+    return $id_rubrique;
 }
 
 /**
@@ -131,46 +131,46 @@ 
  *     - chaîne : Texte d'un message d'erreur
  */
 function rubrique_modifier($id_rubrique, $set = null) {
-	include_spip('inc/autoriser');
-	include_spip('inc/filtres');
-
-	include_spip('inc/modifier');
-	$c = collecter_requests(
-		// include list
-		objet_info('rubrique', 'champs_editables'),
-		// exclude list
-		['id_parent', 'confirme_deplace'],
-		// donnees eventuellement fournies
-		$set
-	);
-
-	if (
-		$err = objet_modifier_champs(
-			'rubrique',
-			$id_rubrique,
-			[
-			'data' => $set,
-			'nonvide' => ['titre' => _T('titre_nouvelle_rubrique') . ' ' . _T('info_numero_abbreviation') . $id_rubrique]
-			],
-			$c
-		)
-	) {
-		return $err;
-	}
-
-	$c = collecter_requests(['id_parent', 'confirme_deplace'], [], $set);
-	// Deplacer la rubrique
-	if (isset($c['id_parent'])) {
-		$err = rubrique_instituer($id_rubrique, $c);
-	}
-
-	// invalider les caches marques de cette rubrique
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='rubrique/$id_rubrique'");
-	// et celui de menu_rubriques
-	effacer_meta('date_calcul_rubriques');
-
-	return $err;
+    include_spip('inc/autoriser');
+    include_spip('inc/filtres');
+
+    include_spip('inc/modifier');
+    $c = collecter_requests(
+        // include list
+        objet_info('rubrique', 'champs_editables'),
+        // exclude list
+        ['id_parent', 'confirme_deplace'],
+        // donnees eventuellement fournies
+        $set
+    );
+
+    if (
+        $err = objet_modifier_champs(
+            'rubrique',
+            $id_rubrique,
+            [
+            'data' => $set,
+            'nonvide' => ['titre' => _T('titre_nouvelle_rubrique') . ' ' . _T('info_numero_abbreviation') . $id_rubrique]
+            ],
+            $c
+        )
+    ) {
+        return $err;
+    }
+
+    $c = collecter_requests(['id_parent', 'confirme_deplace'], [], $set);
+    // Deplacer la rubrique
+    if (isset($c['id_parent'])) {
+        $err = rubrique_instituer($id_rubrique, $c);
+    }
+
+    // invalider les caches marques de cette rubrique
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='rubrique/$id_rubrique'");
+    // et celui de menu_rubriques
+    effacer_meta('date_calcul_rubriques');
+
+    return $err;
 }
 
 /**
@@ -193,25 +193,25 @@ 
  *     false si la confirmation du déplacement n'est pas présente
  */
 function editer_rubrique_breves($id_rubrique, $id_parent, $c = []) {
-	if (!sql_countsel('spip_breves', "id_rubrique=$id_rubrique")) {
-		return true;
-	}
-
-	if (empty($c['confirme_deplace']) or $c['confirme_deplace'] != 'oui') {
-		return false;
-	}
-
-	if (
-		$id_secteur = sql_getfetsel(
-			'id_secteur',
-			'spip_rubriques',
-			"id_rubrique=$id_parent"
-		)
-	) {
-		sql_updateq('spip_breves', ['id_rubrique' => $id_secteur], "id_rubrique=$id_rubrique");
-	}
-
-	return true;
+    if (!sql_countsel('spip_breves', "id_rubrique=$id_rubrique")) {
+        return true;
+    }
+
+    if (empty($c['confirme_deplace']) or $c['confirme_deplace'] != 'oui') {
+        return false;
+    }
+
+    if (
+        $id_secteur = sql_getfetsel(
+            'id_secteur',
+            'spip_rubriques',
+            "id_rubrique=$id_parent"
+        )
+    ) {
+        sql_updateq('spip_breves', ['id_rubrique' => $id_secteur], "id_rubrique=$id_rubrique");
+    }
+
+    return true;
 }
 
 
@@ -233,50 +233,50 @@ 
  *     Chaîne : Texte du message d'erreur
  */
 function rubrique_instituer($id_rubrique, $c) {
-	// traitement de la rubrique parente
-	// interdiction de deplacer vers ou a partir d'une rubrique
-	// qu'on n'administre pas.
-
-	if (null !== ($id_parent = $c['id_parent'])) {
-		$id_parent = intval($id_parent);
-		$filles = calcul_branche_in($id_rubrique);
-		if (strpos(",$id_parent,", (string) ",$filles,") !== false) {
-			spip_log("La rubrique $id_rubrique ne peut etre fille de sa descendante $id_parent");
-		} else {
-			$s = sql_fetsel('id_parent, statut', 'spip_rubriques', "id_rubrique=$id_rubrique");
-			$old_parent = $s['id_parent'];
-
-			if (
-				!($id_parent != $old_parent
-				and autoriser('publierdans', 'rubrique', $id_parent)
-				and autoriser('creerrubriquedans', 'rubrique', $id_parent)
-				and autoriser('publierdans', 'rubrique', $old_parent)
-				)
-			) {
-				if ($s['statut'] != 'new') {
-					spip_log("deplacement de $id_rubrique vers $id_parent refuse a " . $GLOBALS['visiteur_session']['id_auteur'] . ' ' . $GLOBALS['visiteur_session']['statut']);
-				}
-			} elseif (editer_rubrique_breves($id_rubrique, $id_parent, $c)) {
-				$statut_ancien = $s['statut'];
-				sql_updateq('spip_rubriques', ['id_parent' => $id_parent], "id_rubrique=$id_rubrique");
-
-
-				propager_les_secteurs();
-
-				// Deplacement d'une rubrique publiee ==> chgt general de leur statut
-				if ($statut_ancien == 'publie') {
-					calculer_rubriques_if($old_parent, ['id_rubrique' => $id_parent], ['statut_ancien' => $statut_ancien]);
-				}
-				// Creation ou deplacement d'une rubrique non publiee
-				// invalider le cache de leur menu
-				elseif (!$statut_ancien || $old_parent != $id_parent) {
-					effacer_meta('date_calcul_rubriques');
-				}
-
-				calculer_langues_rubriques();
-			}
-		}
-	}
-
-	return ''; // pas d'erreur
+    // traitement de la rubrique parente
+    // interdiction de deplacer vers ou a partir d'une rubrique
+    // qu'on n'administre pas.
+
+    if (null !== ($id_parent = $c['id_parent'])) {
+        $id_parent = intval($id_parent);
+        $filles = calcul_branche_in($id_rubrique);
+        if (strpos(",$id_parent,", (string) ",$filles,") !== false) {
+            spip_log("La rubrique $id_rubrique ne peut etre fille de sa descendante $id_parent");
+        } else {
+            $s = sql_fetsel('id_parent, statut', 'spip_rubriques', "id_rubrique=$id_rubrique");
+            $old_parent = $s['id_parent'];
+
+            if (
+                !($id_parent != $old_parent
+                and autoriser('publierdans', 'rubrique', $id_parent)
+                and autoriser('creerrubriquedans', 'rubrique', $id_parent)
+                and autoriser('publierdans', 'rubrique', $old_parent)
+                )
+            ) {
+                if ($s['statut'] != 'new') {
+                    spip_log("deplacement de $id_rubrique vers $id_parent refuse a " . $GLOBALS['visiteur_session']['id_auteur'] . ' ' . $GLOBALS['visiteur_session']['statut']);
+                }
+            } elseif (editer_rubrique_breves($id_rubrique, $id_parent, $c)) {
+                $statut_ancien = $s['statut'];
+                sql_updateq('spip_rubriques', ['id_parent' => $id_parent], "id_rubrique=$id_rubrique");
+
+
+                propager_les_secteurs();
+
+                // Deplacement d'une rubrique publiee ==> chgt general de leur statut
+                if ($statut_ancien == 'publie') {
+                    calculer_rubriques_if($old_parent, ['id_rubrique' => $id_parent], ['statut_ancien' => $statut_ancien]);
+                }
+                // Creation ou deplacement d'une rubrique non publiee
+                // invalider le cache de leur menu
+                elseif (!$statut_ancien || $old_parent != $id_parent) {
+                    effacer_meta('date_calcul_rubriques');
+                }
+
+                calculer_langues_rubriques();
+            }
+        }
+    }
+
+    return ''; // pas d'erreur
 }

ecrire/action/editer_auteur.php

Indentation +279 added lines, -279 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -37,41 +37,41 @@ 
  */
 function action_editer_auteur_dist($arg = null) {
 
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
-
-
-	// si id_auteur n'est pas un nombre, c'est une creation
-	if (!$id_auteur = intval($arg)) {
-		if (($id_auteur = auteur_inserer()) > 0) {
-			# cf. GROS HACK
-			# recuperer l'eventuel logo charge avant la creation
-			# ils ont un id = 0-id_auteur de la session
-			$id_hack = 0 - $GLOBALS['visiteur_session']['id_auteur'];
-			$chercher_logo = charger_fonction('chercher_logo', 'inc');
-			foreach (['on', 'off'] as $type) {
-				if ($logo = $chercher_logo($id_hack, 'id_auteur', $type)) {
-					if ($logo = reset($logo)) {
-						rename($logo, str_replace($id_hack, $id_auteur, $logo));
-					}
-				}
-			}
-		}
-	}
-
-	// Enregistre l'envoi dans la BD
-	$err = '';
-	if ($id_auteur > 0) {
-		$err = auteur_modifier($id_auteur);
-	}
-
-	if ($err) {
-		spip_log("echec editeur auteur: $err", _LOG_ERREUR);
-	}
-
-	return [$id_auteur, $err];
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
+
+
+    // si id_auteur n'est pas un nombre, c'est une creation
+    if (!$id_auteur = intval($arg)) {
+        if (($id_auteur = auteur_inserer()) > 0) {
+            # cf. GROS HACK
+            # recuperer l'eventuel logo charge avant la creation
+            # ils ont un id = 0-id_auteur de la session
+            $id_hack = 0 - $GLOBALS['visiteur_session']['id_auteur'];
+            $chercher_logo = charger_fonction('chercher_logo', 'inc');
+            foreach (['on', 'off'] as $type) {
+                if ($logo = $chercher_logo($id_hack, 'id_auteur', $type)) {
+                    if ($logo = reset($logo)) {
+                        rename($logo, str_replace($id_hack, $id_auteur, $logo));
+                    }
+                }
+            }
+        }
+    }
+
+    // Enregistre l'envoi dans la BD
+    $err = '';
+    if ($id_auteur > 0) {
+        $err = auteur_modifier($id_auteur);
+    }
+
+    if ($err) {
+        spip_log("echec editeur auteur: $err", _LOG_ERREUR);
+    }
+
+    return [$id_auteur, $err];
 }
 
 /**
@@ -88,44 +88,44 @@ 
  */
 function auteur_inserer($source = null, $set = null) {
 
-	// Ce qu'on va demander comme modifications
-	$champs = [];
-	$champs['source'] = $source ?: 'spip';
-
-	$champs['login'] = '';
-	$champs['statut'] = '5poubelle';  // inutilisable tant qu'il n'a pas ete renseigne et institue
-	$champs['webmestre'] = 'non';
-	if (empty($champs['imessage'])) {
-		$champs['imessage'] = 'oui';
-	}
-
-	if ($set) {
-		$champs = array_merge($champs, $set);
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_insertion',
-		[
-			'args' => [
-				'table' => 'spip_auteurs',
-			],
-			'data' => $champs
-		]
-	);
-	$id_auteur = sql_insertq('spip_auteurs', $champs);
-	pipeline(
-		'post_insertion',
-		[
-			'args' => [
-				'table' => 'spip_auteurs',
-				'id_objet' => $id_auteur
-			],
-			'data' => $champs
-		]
-	);
-
-	return $id_auteur;
+    // Ce qu'on va demander comme modifications
+    $champs = [];
+    $champs['source'] = $source ?: 'spip';
+
+    $champs['login'] = '';
+    $champs['statut'] = '5poubelle';  // inutilisable tant qu'il n'a pas ete renseigne et institue
+    $champs['webmestre'] = 'non';
+    if (empty($champs['imessage'])) {
+        $champs['imessage'] = 'oui';
+    }
+
+    if ($set) {
+        $champs = array_merge($champs, $set);
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_insertion',
+        [
+            'args' => [
+                'table' => 'spip_auteurs',
+            ],
+            'data' => $champs
+        ]
+    );
+    $id_auteur = sql_insertq('spip_auteurs', $champs);
+    pipeline(
+        'post_insertion',
+        [
+            'args' => [
+                'table' => 'spip_auteurs',
+                'id_objet' => $id_auteur
+            ],
+            'data' => $champs
+        ]
+    );
+
+    return $id_auteur;
 }
 
 
@@ -150,70 +150,70 @@ 
  */
 function auteur_modifier($id_auteur, $set = null, $force_update = false) {
 
-	include_spip('inc/modifier');
-	include_spip('inc/filtres');
-	$c = collecter_requests(
-		// include list
-		objet_info('auteur', 'champs_editables'),
-		// exclude list
-		$force_update ? [] : ['webmestre', 'pass', 'login'],
-		// donnees eventuellement fournies
-		$set
-	);
-
-	if (
-		$err = objet_modifier_champs(
-			'auteur',
-			$id_auteur,
-			[
-			'data' => $set,
-			'nonvide' => ['nom' => _T('ecrire:item_nouvel_auteur')]
-			],
-			$c
-		)
-	) {
-		return $err;
-	}
-	$session = $c;
-
-	$err = '';
-	if (!$force_update) {
-		// Modification de statut, changement de rubrique ?
-		$c = collecter_requests(
-		// include list
-			[
-				'statut',
-				'new_login',
-				'new_pass',
-				'login',
-				'pass',
-				'webmestre',
-				'restreintes',
-				'id_parent'
-			],
-			// exclude list
-			[],
-			// donnees eventuellement fournies
-			$set
-		);
-		if (isset($c['new_login']) and !isset($c['login'])) {
-			$c['login'] = $c['new_login'];
-		}
-		if (isset($c['new_pass']) and !isset($c['pass'])) {
-			$c['pass'] = $c['new_pass'];
-		}
-		$err = auteur_instituer($id_auteur, $c);
-		$session = array_merge($session, $c);
-	}
-
-	// .. mettre a jour les sessions de cet auteur
-	include_spip('inc/session');
-	$session['id_auteur'] = $id_auteur;
-	unset($session['new_login']);
-	unset($session['new_pass']);
-	actualiser_sessions($session);
-
-	return $err;
+    include_spip('inc/modifier');
+    include_spip('inc/filtres');
+    $c = collecter_requests(
+        // include list
+        objet_info('auteur', 'champs_editables'),
+        // exclude list
+        $force_update ? [] : ['webmestre', 'pass', 'login'],
+        // donnees eventuellement fournies
+        $set
+    );
+
+    if (
+        $err = objet_modifier_champs(
+            'auteur',
+            $id_auteur,
+            [
+            'data' => $set,
+            'nonvide' => ['nom' => _T('ecrire:item_nouvel_auteur')]
+            ],
+            $c
+        )
+    ) {
+        return $err;
+    }
+    $session = $c;
+
+    $err = '';
+    if (!$force_update) {
+        // Modification de statut, changement de rubrique ?
+        $c = collecter_requests(
+        // include list
+            [
+                'statut',
+                'new_login',
+                'new_pass',
+                'login',
+                'pass',
+                'webmestre',
+                'restreintes',
+                'id_parent'
+            ],
+            // exclude list
+            [],
+            // donnees eventuellement fournies
+            $set
+        );
+        if (isset($c['new_login']) and !isset($c['login'])) {
+            $c['login'] = $c['new_login'];
+        }
+        if (isset($c['new_pass']) and !isset($c['pass'])) {
+            $c['pass'] = $c['new_pass'];
+        }
+        $err = auteur_instituer($id_auteur, $c);
+        $session = array_merge($session, $c);
+    }
+
+    // .. mettre a jour les sessions de cet auteur
+    include_spip('inc/session');
+    $session['id_auteur'] = $id_auteur;
+    unset($session['new_login']);
+    unset($session['new_pass']);
+    actualiser_sessions($session);
+
+    return $err;
 }
 
 /**
@@ -234,9 +234,9 @@ 
  * @return string
  */
 function auteur_associer($id_auteur, $objets, $qualif = null) {
-	include_spip('action/editer_liens');
+    include_spip('action/editer_liens');
 
-	return objet_associer(['auteur' => $id_auteur], $objets, $qualif);
+    return objet_associer(['auteur' => $id_auteur], $objets, $qualif);
 }
 
 /**
@@ -255,9 +255,9 @@ 
  * @return string
  */
 function auteur_dissocier($id_auteur, $objets) {
-	include_spip('action/editer_liens');
+    include_spip('action/editer_liens');
 
-	return objet_dissocier(['auteur' => $id_auteur], $objets);
+    return objet_dissocier(['auteur' => $id_auteur], $objets);
 }
 
 /**
@@ -278,9 +278,9 @@ 
  * @return bool|int
  */
 function auteur_qualifier($id_auteur, $objets, $qualif) {
-	include_spip('action/editer_liens');
+    include_spip('action/editer_liens');
 
-	return objet_qualifier_liens(['auteur' => $id_auteur], $objets, $qualif);
+    return objet_qualifier_liens(['auteur' => $id_auteur], $objets, $qualif);
 }
 
 
@@ -299,139 +299,139 @@ 
  * @return bool|string
  */
 function auteur_instituer($id_auteur, $c, $force_webmestre = false) {
-	if (!$id_auteur = intval($id_auteur)) {
-		return false;
-	}
-	$erreurs = []; // contiendra les differentes erreurs a traduire par _T()
-	$champs = [];
-
-	// les memoriser pour les faire passer dans le pipeline pre_edition
-	if (isset($c['login']) and strlen($c['login'])) {
-		$champs['login'] = $c['login'];
-	}
-	if (isset($c['pass']) and strlen($c['pass'])) {
-		$champs['pass'] = $c['pass'];
-	}
-
-	$statut = $statut_ancien = sql_getfetsel('statut', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
-
-	if (
-		isset($c['statut'])
-		and (autoriser('modifier', 'auteur', $id_auteur, null, ['statut' => $c['statut']]))
-	) {
-		$statut = $champs['statut'] = $c['statut'];
-	}
-
-	// Restreindre avant de declarer l'auteur
-	// (section critique sur les droits)
-	if (isset($c['id_parent']) and $c['id_parent']) {
-		if (is_array($c['restreintes'])) {
-			$c['restreintes'][] = $c['id_parent'];
-		} else {
-			$c['restreintes'] = [$c['id_parent']];
-		}
-	}
-
-	if (
-		isset($c['webmestre'])
-		and ($force_webmestre or autoriser('modifier', 'auteur', $id_auteur, null, ['webmestre' => '?']))
-	) {
-		$champs['webmestre'] = $c['webmestre'] == 'oui' ? 'oui' : 'non';
-	}
-
-	// si statut change et n'est pas 0minirezo, on force webmestre a non
-	if (isset($c['statut']) and $c['statut'] !== '0minirezo') {
-		$champs['webmestre'] = $c['webmestre'] = 'non';
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_edition',
-		[
-			'args' => [
-				'table' => 'spip_auteurs',
-				'id_objet' => $id_auteur,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-			],
-			'data' => $champs
-		]
-	);
-
-	if (
-		isset($c['restreintes']) and is_array($c['restreintes'])
-		and autoriser('modifier', 'auteur', $id_auteur, null, ['restreint' => $c['restreintes']])
-	) {
-		$rubriques = array_map('intval', $c['restreintes']);
-		$rubriques = array_unique($rubriques);
-		$rubriques = array_diff($rubriques, [0]);
-		auteur_dissocier($id_auteur, ['rubrique' => '*']);
-		auteur_associer($id_auteur, ['rubrique' => $rubriques]);
-	}
-
-	$flag_ecrire_acces = false;
-	// commencer par traiter les cas particuliers des logins et pass
-	// avant les autres ecritures en base
-	if (isset($champs['login']) or isset($champs['pass'])) {
-		$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
-		include_spip('inc/auth');
-		if (isset($champs['login']) and strlen($champs['login'])) {
-			if (!auth_modifier_login($auth_methode, $champs['login'], $id_auteur)) {
-				$erreurs[] = 'ecrire:impossible_modifier_login_auteur';
-			}
-		}
-		if (isset($champs['pass']) and strlen($champs['pass'])) {
-			$champs['login'] = sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
-			if (!auth_modifier_pass($auth_methode, $champs['login'], $champs['pass'], $id_auteur)) {
-				$erreurs[] = 'ecrire:impossible_modifier_pass_auteur';
-			}
-		}
-		unset($champs['login']);
-		unset($champs['pass']);
-		$flag_ecrire_acces = true;
-	}
-
-	if (!(is_countable($champs) ? count($champs) : 0)) {
-		return implode(' ', array_map('_T', $erreurs));
-	}
-	sql_updateq('spip_auteurs', $champs, 'id_auteur=' . $id_auteur);
-
-	// .. mettre a jour les fichiers .htpasswd et .htpasswd-admin
-	if (
-		$flag_ecrire_acces
-		or isset($champs['statut'])
-	) {
-		include_spip('inc/acces');
-		ecrire_acces();
-	}
-
-	// Invalider les caches
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='auteur/$id_auteur'");
-
-	// Pipeline
-	pipeline(
-		'post_edition',
-		[
-			'args' => [
-				'table' => 'spip_auteurs',
-				'id_objet' => $id_auteur,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-			],
-			'data' => $champs
-		]
-	);
-
-
-	// Notifications
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			'instituerauteur',
-			$id_auteur,
-			['statut' => $statut, 'statut_ancien' => $statut_ancien]
-		);
-	}
-
-	return implode(' ', array_map('_T', $erreurs));
+    if (!$id_auteur = intval($id_auteur)) {
+        return false;
+    }
+    $erreurs = []; // contiendra les differentes erreurs a traduire par _T()
+    $champs = [];
+
+    // les memoriser pour les faire passer dans le pipeline pre_edition
+    if (isset($c['login']) and strlen($c['login'])) {
+        $champs['login'] = $c['login'];
+    }
+    if (isset($c['pass']) and strlen($c['pass'])) {
+        $champs['pass'] = $c['pass'];
+    }
+
+    $statut = $statut_ancien = sql_getfetsel('statut', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
+
+    if (
+        isset($c['statut'])
+        and (autoriser('modifier', 'auteur', $id_auteur, null, ['statut' => $c['statut']]))
+    ) {
+        $statut = $champs['statut'] = $c['statut'];
+    }
+
+    // Restreindre avant de declarer l'auteur
+    // (section critique sur les droits)
+    if (isset($c['id_parent']) and $c['id_parent']) {
+        if (is_array($c['restreintes'])) {
+            $c['restreintes'][] = $c['id_parent'];
+        } else {
+            $c['restreintes'] = [$c['id_parent']];
+        }
+    }
+
+    if (
+        isset($c['webmestre'])
+        and ($force_webmestre or autoriser('modifier', 'auteur', $id_auteur, null, ['webmestre' => '?']))
+    ) {
+        $champs['webmestre'] = $c['webmestre'] == 'oui' ? 'oui' : 'non';
+    }
+
+    // si statut change et n'est pas 0minirezo, on force webmestre a non
+    if (isset($c['statut']) and $c['statut'] !== '0minirezo') {
+        $champs['webmestre'] = $c['webmestre'] = 'non';
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_edition',
+        [
+            'args' => [
+                'table' => 'spip_auteurs',
+                'id_objet' => $id_auteur,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    if (
+        isset($c['restreintes']) and is_array($c['restreintes'])
+        and autoriser('modifier', 'auteur', $id_auteur, null, ['restreint' => $c['restreintes']])
+    ) {
+        $rubriques = array_map('intval', $c['restreintes']);
+        $rubriques = array_unique($rubriques);
+        $rubriques = array_diff($rubriques, [0]);
+        auteur_dissocier($id_auteur, ['rubrique' => '*']);
+        auteur_associer($id_auteur, ['rubrique' => $rubriques]);
+    }
+
+    $flag_ecrire_acces = false;
+    // commencer par traiter les cas particuliers des logins et pass
+    // avant les autres ecritures en base
+    if (isset($champs['login']) or isset($champs['pass'])) {
+        $auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
+        include_spip('inc/auth');
+        if (isset($champs['login']) and strlen($champs['login'])) {
+            if (!auth_modifier_login($auth_methode, $champs['login'], $id_auteur)) {
+                $erreurs[] = 'ecrire:impossible_modifier_login_auteur';
+            }
+        }
+        if (isset($champs['pass']) and strlen($champs['pass'])) {
+            $champs['login'] = sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
+            if (!auth_modifier_pass($auth_methode, $champs['login'], $champs['pass'], $id_auteur)) {
+                $erreurs[] = 'ecrire:impossible_modifier_pass_auteur';
+            }
+        }
+        unset($champs['login']);
+        unset($champs['pass']);
+        $flag_ecrire_acces = true;
+    }
+
+    if (!(is_countable($champs) ? count($champs) : 0)) {
+        return implode(' ', array_map('_T', $erreurs));
+    }
+    sql_updateq('spip_auteurs', $champs, 'id_auteur=' . $id_auteur);
+
+    // .. mettre a jour les fichiers .htpasswd et .htpasswd-admin
+    if (
+        $flag_ecrire_acces
+        or isset($champs['statut'])
+    ) {
+        include_spip('inc/acces');
+        ecrire_acces();
+    }
+
+    // Invalider les caches
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='auteur/$id_auteur'");
+
+    // Pipeline
+    pipeline(
+        'post_edition',
+        [
+            'args' => [
+                'table' => 'spip_auteurs',
+                'id_objet' => $id_auteur,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+            ],
+            'data' => $champs
+        ]
+    );
+
+
+    // Notifications
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            'instituerauteur',
+            $id_auteur,
+            ['statut' => $statut, 'statut_ancien' => $statut_ancien]
+        );
+    }
+
+    return implode(' ', array_map('_T', $erreurs));
 }

ecrire/inc/filtres_selecteur_generique.php

Indentation +144 added lines, -144 removed lines

@@ -7,7 +7,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -28,51 +28,51 @@ 
  *     - afficher : tableau des objets à afficher (mais pas forcément sélectionnables)
  */
 function selecteur_lister_objets($includelist = [], $excludelist = []) {
-	static $liste_selecteurs, $liste_parents;
-
-	if (!$liste_selecteurs) {
-		$liste_selecteurs = find_all_in_path('formulaires/selecteur/', 'hierarchie-[\w]+[.]html$');
-	}
-	$objets_selectionner = [];
-	foreach ($liste_selecteurs as $fichier => $chemin) {
-		$objets_selectionner[] = preg_replace('/^hierarchie-([\w]+)[.]html$/', '$1', $fichier);
-	}
-
-	// S'il y a une whitelist on ne garde que ce qui est dedans
-	if (!empty($includelist)) {
-		$includelist = array_map('table_objet', $includelist);
-		$objets_selectionner = array_intersect($objets_selectionner, $includelist);
-	}
-	// On supprime ce qui est dans la liste d’exclusion
-	$excludelist = array_map('table_objet', $excludelist);
-	// On enlève toujours la racine
-	$excludelist[] = 'racine';
-	$objets_selectionner = array_diff($objets_selectionner, $excludelist);
-
-	// Ensuite on cherche ce qu'on doit afficher : au moins ceux qu'on peut sélectionner
-	$objets_afficher = $objets_selectionner;
-
-	// Il faut alors chercher d'éventuels parents obligatoires en plus :
-	// lister-trucs-bidules.html => on doit afficher des "trucs" pour trouver des "bidules"
-	if (!$liste_parents) {
-		$liste_parents = find_all_in_path('formulaires/selecteur/', 'lister-[\w]+-[\w]+[.]html$');
-	}
-	foreach ($liste_parents as $fichier => $chemin) {
-		preg_match('/^lister-([\w]+)-([\w]+)[.]html$/', $fichier, $captures);
-		$parent = $captures[1];
-		$type = $captures[2];
-		// Si le type fait partie de ce qu'on doit afficher alors on ajoute aussi le parent à l'affichage
-		if (in_array($type, $objets_afficher)) {
-			$objets_afficher[] = $parent;
-		}
-	}
-
-	$objets = [
-		'selectionner' => array_unique($objets_selectionner),
-		'afficher' => array_unique($objets_afficher),
-	];
-
-	return $objets;
+    static $liste_selecteurs, $liste_parents;
+
+    if (!$liste_selecteurs) {
+        $liste_selecteurs = find_all_in_path('formulaires/selecteur/', 'hierarchie-[\w]+[.]html$');
+    }
+    $objets_selectionner = [];
+    foreach ($liste_selecteurs as $fichier => $chemin) {
+        $objets_selectionner[] = preg_replace('/^hierarchie-([\w]+)[.]html$/', '$1', $fichier);
+    }
+
+    // S'il y a une whitelist on ne garde que ce qui est dedans
+    if (!empty($includelist)) {
+        $includelist = array_map('table_objet', $includelist);
+        $objets_selectionner = array_intersect($objets_selectionner, $includelist);
+    }
+    // On supprime ce qui est dans la liste d’exclusion
+    $excludelist = array_map('table_objet', $excludelist);
+    // On enlève toujours la racine
+    $excludelist[] = 'racine';
+    $objets_selectionner = array_diff($objets_selectionner, $excludelist);
+
+    // Ensuite on cherche ce qu'on doit afficher : au moins ceux qu'on peut sélectionner
+    $objets_afficher = $objets_selectionner;
+
+    // Il faut alors chercher d'éventuels parents obligatoires en plus :
+    // lister-trucs-bidules.html => on doit afficher des "trucs" pour trouver des "bidules"
+    if (!$liste_parents) {
+        $liste_parents = find_all_in_path('formulaires/selecteur/', 'lister-[\w]+-[\w]+[.]html$');
+    }
+    foreach ($liste_parents as $fichier => $chemin) {
+        preg_match('/^lister-([\w]+)-([\w]+)[.]html$/', $fichier, $captures);
+        $parent = $captures[1];
+        $type = $captures[2];
+        // Si le type fait partie de ce qu'on doit afficher alors on ajoute aussi le parent à l'affichage
+        if (in_array($type, $objets_afficher)) {
+            $objets_afficher[] = $parent;
+        }
+    }
+
+    $objets = [
+        'selectionner' => array_unique($objets_selectionner),
+        'afficher' => array_unique($objets_afficher),
+    ];
+
+    return $objets;
 }
 
 /**
@@ -99,31 +99,31 @@ 
  *     liste des couples (objets => id_objet) ou liste des identifiants d'un type d'objet.
  **/
 function picker_selected($selected, $type = '') {
-	$select = [];
-	$type = preg_replace(',\W,', '', $type);
-
-	if ($selected and !is_array($selected)) {
-		$selected = explode(',', $selected);
-	}
-
-	if (is_array($selected)) {
-		foreach ($selected as $value) {
-			// Si c'est le bon format déjà
-			if (preg_match('/^([\w]+)[|]([0-9]+)$/', $value, $captures)) {
-				$objet = $captures[1];
-				$id_objet = intval($captures[2]);
-
-				// Si on cherche un type et que c'est le bon, on renvoit un tableau que d'identifiants
-				if (is_string($type) and $type == $objet and ($id_objet or in_array($objet, ['racine', 'rubrique']))) {
-					$select[] = $id_objet;
-				} elseif (!$type and ($id_objet or in_array($objet, ['racine', 'rubrique']))) {
-					$select[] = ['objet' => $objet, 'id_objet' => $id_objet];
-				}
-			}
-		}
-	}
-
-	return $select;
+    $select = [];
+    $type = preg_replace(',\W,', '', $type);
+
+    if ($selected and !is_array($selected)) {
+        $selected = explode(',', $selected);
+    }
+
+    if (is_array($selected)) {
+        foreach ($selected as $value) {
+            // Si c'est le bon format déjà
+            if (preg_match('/^([\w]+)[|]([0-9]+)$/', $value, $captures)) {
+                $objet = $captures[1];
+                $id_objet = intval($captures[2]);
+
+                // Si on cherche un type et que c'est le bon, on renvoit un tableau que d'identifiants
+                if (is_string($type) and $type == $objet and ($id_objet or in_array($objet, ['racine', 'rubrique']))) {
+                    $select[] = $id_objet;
+                } elseif (!$type and ($id_objet or in_array($objet, ['racine', 'rubrique']))) {
+                    $select[] = ['objet' => $objet, 'id_objet' => $id_objet];
+                }
+            }
+        }
+    }
+
+    return $select;
 }
 
 /**
@@ -140,49 +140,49 @@ 
  *     Booléen indiquant si les articles sont sélectionnables
  */
 function picker_identifie_id_rapide($ref, $rubriques_ou_objets = false, $articles = false) {
-	include_spip('inc/json');
-	include_spip('inc/lien');
-
-	// On construit un tableau des objets sélectionnables suivant les paramètres
-	$objets = [];
-	if ($rubriques_ou_objets and is_array($rubriques_ou_objets)) {
-		$objets = $rubriques_ou_objets;
-	} else {
-		if ($rubriques_ou_objets) {
-			$objets[] = 'rubriques';
-		}
-		if ($articles) {
-			$objets[] = 'articles';
-		}
-	}
-
-	// si id numerique et un seul objet possible, pas d'ambiguite
-	if (is_numeric($ref) and count($objets) === 1) {
-		$ref = reset($objets) . $ref;
-	}
-
-	// Si la référence ne correspond à rien, c'est fini
-	if (!($match = typer_raccourci($ref))) {
-		return json_export(false);
-	}
-	// Sinon on récupère les infos utiles
-	[$type, , $id, , , , ] = array_pad($match, 7, null);
-
-	// On regarde si le type trouvé fait partie des objets sélectionnables
-	if (!in_array(table_objet($type), $objets)) {
-		return json_export(false);
-	}
-
-	// Maintenant que tout est bon, on cherche les informations sur cet objet
-	include_spip('inc/filtres');
-	if (!$titre = generer_objet_info($id, $type, 'titre')) {
-		return json_export(false);
-	}
-
-	// On simplifie le texte
-	$titre = attribut_html($titre);
-
-	return json_export(['type' => $type, 'id' => "$type|$id", 'titre' => $titre]);
+    include_spip('inc/json');
+    include_spip('inc/lien');
+
+    // On construit un tableau des objets sélectionnables suivant les paramètres
+    $objets = [];
+    if ($rubriques_ou_objets and is_array($rubriques_ou_objets)) {
+        $objets = $rubriques_ou_objets;
+    } else {
+        if ($rubriques_ou_objets) {
+            $objets[] = 'rubriques';
+        }
+        if ($articles) {
+            $objets[] = 'articles';
+        }
+    }
+
+    // si id numerique et un seul objet possible, pas d'ambiguite
+    if (is_numeric($ref) and count($objets) === 1) {
+        $ref = reset($objets) . $ref;
+    }
+
+    // Si la référence ne correspond à rien, c'est fini
+    if (!($match = typer_raccourci($ref))) {
+        return json_export(false);
+    }
+    // Sinon on récupère les infos utiles
+    [$type, , $id, , , , ] = array_pad($match, 7, null);
+
+    // On regarde si le type trouvé fait partie des objets sélectionnables
+    if (!in_array(table_objet($type), $objets)) {
+        return json_export(false);
+    }
+
+    // Maintenant que tout est bon, on cherche les informations sur cet objet
+    include_spip('inc/filtres');
+    if (!$titre = generer_objet_info($id, $type, 'titre')) {
+        return json_export(false);
+    }
+
+    // On simplifie le texte
+    $titre = attribut_html($titre);
+
+    return json_export(['type' => $type, 'id' => "$type|$id", 'titre' => $titre]);
 }
 
 /**
@@ -205,34 +205,34 @@ 
  *     Comme le filtre `oui` : espace (` `) si rubrique à afficher, chaîne vide sinon.
  */
 function test_enfants_rubrique($id_rubrique, $types = []) {
-	static $has_child = [];
-
-	if (!isset($has_child[$id_rubrique])) {
-		$types = (is_array($types) ? array_filter($types) : []);
-
-		// recuperer tous les freres et soeurs de la rubrique visee
-		$id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique));
-		$fratrie = sql_allfetsel('id_rubrique', 'spip_rubriques', 'id_parent=' . intval($id_parent));
-		$fratrie = array_column($fratrie, 'id_rubrique');
-		$has = sql_allfetsel('DISTINCT id_parent', 'spip_rubriques', sql_in('id_parent', $fratrie));
-		$has = array_column($has, 'id_parent');
-		$fratrie = array_diff($fratrie, $has);
-
-		while (count($fratrie) and is_array($types) and count($types)) {
-			$type = array_shift($types);
-			$h = sql_allfetsel('DISTINCT id_rubrique', table_objet_sql($type), sql_in('id_rubrique', $fratrie));
-			$h = array_column($h, 'id_rubrique');
-			$has = array_merge($has, $h);
-			$fratrie = array_diff($fratrie, $h);
-		}
-
-		if (count($has)) {
-			$has_child = $has_child + array_combine($has, array_pad([], count($has), true));
-		}
-		if (count($fratrie)) {
-			$has_child = $has_child + array_combine($fratrie, array_pad([], count($fratrie), false));
-		}
-	}
-
-	return $has_child[$id_rubrique] ? ' ' : '';
+    static $has_child = [];
+
+    if (!isset($has_child[$id_rubrique])) {
+        $types = (is_array($types) ? array_filter($types) : []);
+
+        // recuperer tous les freres et soeurs de la rubrique visee
+        $id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique));
+        $fratrie = sql_allfetsel('id_rubrique', 'spip_rubriques', 'id_parent=' . intval($id_parent));
+        $fratrie = array_column($fratrie, 'id_rubrique');
+        $has = sql_allfetsel('DISTINCT id_parent', 'spip_rubriques', sql_in('id_parent', $fratrie));
+        $has = array_column($has, 'id_parent');
+        $fratrie = array_diff($fratrie, $has);
+
+        while (count($fratrie) and is_array($types) and count($types)) {
+            $type = array_shift($types);
+            $h = sql_allfetsel('DISTINCT id_rubrique', table_objet_sql($type), sql_in('id_rubrique', $fratrie));
+            $h = array_column($h, 'id_rubrique');
+            $has = array_merge($has, $h);
+            $fratrie = array_diff($fratrie, $h);
+        }
+
+        if (count($has)) {
+            $has_child = $has_child + array_combine($has, array_pad([], count($has), true));
+        }
+        if (count($fratrie)) {
+            $has_child = $has_child + array_combine($fratrie, array_pad([], count($fratrie), false));
+        }
+    }
+
+    return $has_child[$id_rubrique] ? ' ' : '';
 }

ecrire/inc/modifier.php

Indentation +256 added lines, -256 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -39,28 +39,28 @@ 
  *     Tableau des champs et valeurs collectées
  */
 function collecter_requests($include_list, $exclude_list = [], $set = null, $tous = false) {
-	$c = $set;
-	if (!$c) {
-		$c = [];
-		foreach ($include_list as $champ) {
-			// on ne collecte que les champs reellement envoyes par defaut.
-			// le cas d'un envoi de valeur NULL peut du coup poser probleme.
-			$val = _request($champ);
-			if ($tous or $val !== null) {
-				$c[$champ] = $val;
-			}
-		}
-		// on ajoute toujours la lang en saisie possible
-		// meme si pas prevu au depart pour l'objet concerne
-		if ($l = _request('changer_lang')) {
-			$c['lang'] = $l;
-		}
-	}
-	foreach ($exclude_list as $champ) {
-		unset($c[$champ]);
-	}
-
-	return $c;
+    $c = $set;
+    if (!$c) {
+        $c = [];
+        foreach ($include_list as $champ) {
+            // on ne collecte que les champs reellement envoyes par defaut.
+            // le cas d'un envoi de valeur NULL peut du coup poser probleme.
+            $val = _request($champ);
+            if ($tous or $val !== null) {
+                $c[$champ] = $val;
+            }
+        }
+        // on ajoute toujours la lang en saisie possible
+        // meme si pas prevu au depart pour l'objet concerne
+        if ($l = _request('changer_lang')) {
+            $c['lang'] = $l;
+        }
+    }
+    foreach ($exclude_list as $champ) {
+        unset($c[$champ]);
+    }
+
+    return $c;
 }
 
 /**
@@ -97,237 +97,237 @@ 
  *     - chaîne : Texte d'un message d'erreur
  */
 function objet_modifier_champs($objet, $id_objet, $options, $c = null, $serveur = '') {
-	if (!$id_objet = intval($id_objet)) {
-		spip_log('Erreur $id_objet non defini', 'warn');
-
-		return _T('erreur_technique_enregistrement_impossible');
-	}
-
-	include_spip('inc/filtres');
-
-	$table_objet = table_objet($objet, $serveur);
-	$spip_table_objet = table_objet_sql($objet, $serveur);
-	$id_table_objet = id_table_objet($objet, $serveur);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($spip_table_objet, $serveur);
-
-	// Appels incomplets (sans $c)
-	if (!is_array($c)) {
-		spip_log('erreur appel objet_modifier_champs(' . $objet . '), manque $c');
-
-		return _T('erreur_technique_enregistrement_impossible');
-	}
-
-	// Securite : certaines variables ne sont jamais acceptees ici
-	// car elles ne relevent pas de autoriser(xxx, modifier) ;
-	// il faut passer par instituer_XX()
-	// TODO: faut-il passer ces variables interdites
-	// dans un fichier de description separe ?
-	unset($c['statut']);
-	unset($c['id_parent']);
-	unset($c['id_rubrique']);
-	unset($c['id_secteur']);
-
-	// Gerer les champs non vides
-	if (isset($options['nonvide']) and is_array($options['nonvide'])) {
-		foreach ($options['nonvide'] as $champ => $sinon) {
-			if (isset($c[$champ]) and $c[$champ] === '') {
-				$c[$champ] = $sinon;
-			}
-		}
-	}
-
-	// N'accepter que les champs qui existent
-	// TODO: ici aussi on peut valider les contenus
-	// en fonction du type
-	$champs = array_intersect_key($c, $desc['field']);
-
-	// Nettoyer les valeurs
-	$champs = array_map('corriger_caracteres', $champs);
-
-	// On récupère l'état avant toute modification
-	$row = sql_fetsel('*', $spip_table_objet, $id_table_objet . '=' . $id_objet);
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_edition',
-		[
-			'args' => [
-				'table' => $spip_table_objet, // compatibilite
-				'table_objet' => $table_objet,
-				'spip_table_objet' => $spip_table_objet,
-				'desc' => $desc,
-				'type' => $objet,
-				'id_objet' => $id_objet,
-				'data' => $options['data'] ?? null,
-				'champs' => $options['champs'] ?? [], // [doc] c'est quoi ?
-				'champs_anciens' => $row, // état du contenu avant modif
-				'serveur' => $serveur,
-				'action' => $options['action'] ?? 'modifier'
-			],
-			'data' => $champs
-		]
-	);
-
-	if (!$champs) {
-		return false;
-	}
-
-
-	// marquer le fait que l'objet est travaille par toto a telle date
-	if ($GLOBALS['meta']['articles_modif'] != 'non') {
-		include_spip('inc/drapeau_edition');
-		signale_edition($id_objet, $GLOBALS['visiteur_session'], $objet);
-	}
-
-	// Verifier si les mises a jour sont pertinentes, datees, en conflit etc
-	include_spip('inc/editer');
-	if (!isset($options['data']) or is_null($options['data'])) {
-		$options['data'] = &$_POST;
-	}
-	$conflits = controler_md5($champs, $options['data'], $objet, $id_objet, $serveur);
-	// cas hypothetique : normalement inc/editer verifie en amont le conflit edition
-	// et gere l'interface
-	// ici on ne renvoie donc qu'un messsage d'erreur, au cas ou on y arrive quand meme
-	if ($conflits) {
-		return _T('titre_conflit_edition');
-	}
-
-	if ($champs) {
-		// cas particulier de la langue : passer par instituer_langue_objet
-		if (isset($champs['lang'])) {
-			if ($changer_lang = $champs['lang']) {
-				$id_rubrique = 0;
-				if (isset($desc['field']['id_rubrique'])) {
-					$parent = ($objet == 'rubrique') ? 'id_parent' : 'id_rubrique';
-					$id_rubrique = sql_getfetsel($parent, $spip_table_objet, "$id_table_objet=" . intval($id_objet));
-				}
-				$instituer_langue_objet = charger_fonction('instituer_langue_objet', 'action');
-				$champs['lang'] = $instituer_langue_objet($objet, $id_objet, $id_rubrique, $changer_lang, $serveur);
-			}
-			// on laisse 'lang' dans $champs,
-			// ca permet de passer dans le pipeline post_edition et de journaliser
-			// et ca ne gene pas qu'on refasse un sql_updateq dessus apres l'avoir
-			// deja pris en compte
-		}
-
-		// la modif peut avoir lieu
-
-		// faut-il ajouter date_modif ?
-		if (
-			!empty($options['date_modif'])
-			and !isset($champs[$options['date_modif']])
-		) {
-			$champs[$options['date_modif']] = date('Y-m-d H:i:s');
-		}
-
-		// allez on commit la modif
-		sql_updateq($spip_table_objet, $champs, "$id_table_objet=" . intval($id_objet), [], $serveur);
-
-		// on verifie si elle est bien passee
-		$moof = sql_fetsel(
-			array_keys($champs),
-			$spip_table_objet,
-			"$id_table_objet=" . intval($id_objet),
-			[],
-			[],
-			'',
-			[],
-			$serveur
-		);
-		// si difference entre les champs, reperer les champs mal enregistres
-		if ($moof != $champs) {
-			$liste = [];
-			foreach ($moof as $k => $v) {
-				if (
-					$v !== $champs[$k]
-					// ne pas alerter si le champ est numerique est que les valeurs sont equivalentes
-					and (!is_numeric($v) or intval($v) !== intval($champs[$k]))
-					// ne pas alerter si le champ est date, qu'on a envoye une valeur vide et qu'on recupere une date nulle
-					and (strlen($champs[$k]) or !in_array($v, ['0000-00-00 00:00:00', '0000-00-00']))
-				) {
-					$liste[] = $k;
-					$conflits[$k]['post'] = $champs[$k];
-					$conflits[$k]['save'] = $v;
-
-					// cas specifique MySQL+emoji : si l'un est la
-					// conversion utf8_noplanes de l'autre alors c'est OK
-					if (defined('_MYSQL_NOPLANES') && _MYSQL_NOPLANES) {
-						include_spip('inc/charsets');
-						if ($v == utf8_noplanes($champs[$k])) {
-							array_pop($liste);
-						}
-					}
-				}
-			}
-			// si un champ n'a pas ete correctement enregistre, loger et retourner une erreur
-			// c'est un cas exceptionnel
-			if (count($liste)) {
-				spip_log(
-					"Erreur enregistrement en base $objet/$id_objet champs :" . var_export($conflits, true),
-					'modifier.' . _LOG_CRITIQUE
-				);
-
-				return _T(
-					'erreur_technique_enregistrement_champs',
-					['champs' => "<i>'" . implode("'</i>,<i>'", $liste) . "'</i>"]
-				);
-			}
-		}
-
-		// Invalider les caches
-		if (isset($options['invalideur']) and $options['invalideur']) {
-			include_spip('inc/invalideur');
-			if (is_array($options['invalideur'])) {
-				array_map('suivre_invalideur', $options['invalideur']);
-			} else {
-				suivre_invalideur($options['invalideur']);
-			}
-		}
-
-		// Notifications, gestion des revisions...
-		// en standard, appelle |nouvelle_revision ci-dessous
-		pipeline(
-			'post_edition',
-			[
-				'args' => [
-					'table' => $spip_table_objet,
-					'table_objet' => $table_objet,
-					'spip_table_objet' => $spip_table_objet,
-					'desc' => $desc,
-					'type' => $objet,
-					'id_objet' => $id_objet,
-					'champs' => $options['champs'] ?? [], // [doc] kesako ?
-					'champs_anciens' => $row, // état du contenu avant modif
-					'serveur' => $serveur,
-					'action' => $options['action'] ?? 'modifier'
-				],
-				'data' => $champs
-			]
-		);
-	}
-
-	// journaliser l'affaire
-	// message a affiner :-)
-	include_spip('inc/filtres_mini');
-	$qui = '';
-	if (!empty($GLOBALS['visiteur_session']['id_auteur'])) {
-		$qui .= ' #id_auteur:' . $GLOBALS['visiteur_session']['id_auteur'] . '#';
-	}
-	if (!empty($GLOBALS['visiteur_session']['nom'])) {
-		$qui .= ' #nom:' . $GLOBALS['visiteur_session']['nom'] . '#';
-	}
-	if ($qui == '') {
-		$qui = '#ip:' . $GLOBALS['ip'] . '#';
-	}
-	journal(_L($qui . ' a édité ' . $objet . ' ' . $id_objet . ' (' . join(
-		'+',
-		array_diff(array_keys($champs), ['date_modif'])
-	) . ')'), [
-		'faire' => 'modifier',
-		'quoi' => $objet,
-		'id' => $id_objet
-	]);
-
-	return '';
+    if (!$id_objet = intval($id_objet)) {
+        spip_log('Erreur $id_objet non defini', 'warn');
+
+        return _T('erreur_technique_enregistrement_impossible');
+    }
+
+    include_spip('inc/filtres');
+
+    $table_objet = table_objet($objet, $serveur);
+    $spip_table_objet = table_objet_sql($objet, $serveur);
+    $id_table_objet = id_table_objet($objet, $serveur);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($spip_table_objet, $serveur);
+
+    // Appels incomplets (sans $c)
+    if (!is_array($c)) {
+        spip_log('erreur appel objet_modifier_champs(' . $objet . '), manque $c');
+
+        return _T('erreur_technique_enregistrement_impossible');
+    }
+
+    // Securite : certaines variables ne sont jamais acceptees ici
+    // car elles ne relevent pas de autoriser(xxx, modifier) ;
+    // il faut passer par instituer_XX()
+    // TODO: faut-il passer ces variables interdites
+    // dans un fichier de description separe ?
+    unset($c['statut']);
+    unset($c['id_parent']);
+    unset($c['id_rubrique']);
+    unset($c['id_secteur']);
+
+    // Gerer les champs non vides
+    if (isset($options['nonvide']) and is_array($options['nonvide'])) {
+        foreach ($options['nonvide'] as $champ => $sinon) {
+            if (isset($c[$champ]) and $c[$champ] === '') {
+                $c[$champ] = $sinon;
+            }
+        }
+    }
+
+    // N'accepter que les champs qui existent
+    // TODO: ici aussi on peut valider les contenus
+    // en fonction du type
+    $champs = array_intersect_key($c, $desc['field']);
+
+    // Nettoyer les valeurs
+    $champs = array_map('corriger_caracteres', $champs);
+
+    // On récupère l'état avant toute modification
+    $row = sql_fetsel('*', $spip_table_objet, $id_table_objet . '=' . $id_objet);
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_edition',
+        [
+            'args' => [
+                'table' => $spip_table_objet, // compatibilite
+                'table_objet' => $table_objet,
+                'spip_table_objet' => $spip_table_objet,
+                'desc' => $desc,
+                'type' => $objet,
+                'id_objet' => $id_objet,
+                'data' => $options['data'] ?? null,
+                'champs' => $options['champs'] ?? [], // [doc] c'est quoi ?
+                'champs_anciens' => $row, // état du contenu avant modif
+                'serveur' => $serveur,
+                'action' => $options['action'] ?? 'modifier'
+            ],
+            'data' => $champs
+        ]
+    );
+
+    if (!$champs) {
+        return false;
+    }
+
+
+    // marquer le fait que l'objet est travaille par toto a telle date
+    if ($GLOBALS['meta']['articles_modif'] != 'non') {
+        include_spip('inc/drapeau_edition');
+        signale_edition($id_objet, $GLOBALS['visiteur_session'], $objet);
+    }
+
+    // Verifier si les mises a jour sont pertinentes, datees, en conflit etc
+    include_spip('inc/editer');
+    if (!isset($options['data']) or is_null($options['data'])) {
+        $options['data'] = &$_POST;
+    }
+    $conflits = controler_md5($champs, $options['data'], $objet, $id_objet, $serveur);
+    // cas hypothetique : normalement inc/editer verifie en amont le conflit edition
+    // et gere l'interface
+    // ici on ne renvoie donc qu'un messsage d'erreur, au cas ou on y arrive quand meme
+    if ($conflits) {
+        return _T('titre_conflit_edition');
+    }
+
+    if ($champs) {
+        // cas particulier de la langue : passer par instituer_langue_objet
+        if (isset($champs['lang'])) {
+            if ($changer_lang = $champs['lang']) {
+                $id_rubrique = 0;
+                if (isset($desc['field']['id_rubrique'])) {
+                    $parent = ($objet == 'rubrique') ? 'id_parent' : 'id_rubrique';
+                    $id_rubrique = sql_getfetsel($parent, $spip_table_objet, "$id_table_objet=" . intval($id_objet));
+                }
+                $instituer_langue_objet = charger_fonction('instituer_langue_objet', 'action');
+                $champs['lang'] = $instituer_langue_objet($objet, $id_objet, $id_rubrique, $changer_lang, $serveur);
+            }
+            // on laisse 'lang' dans $champs,
+            // ca permet de passer dans le pipeline post_edition et de journaliser
+            // et ca ne gene pas qu'on refasse un sql_updateq dessus apres l'avoir
+            // deja pris en compte
+        }
+
+        // la modif peut avoir lieu
+
+        // faut-il ajouter date_modif ?
+        if (
+            !empty($options['date_modif'])
+            and !isset($champs[$options['date_modif']])
+        ) {
+            $champs[$options['date_modif']] = date('Y-m-d H:i:s');
+        }
+
+        // allez on commit la modif
+        sql_updateq($spip_table_objet, $champs, "$id_table_objet=" . intval($id_objet), [], $serveur);
+
+        // on verifie si elle est bien passee
+        $moof = sql_fetsel(
+            array_keys($champs),
+            $spip_table_objet,
+            "$id_table_objet=" . intval($id_objet),
+            [],
+            [],
+            '',
+            [],
+            $serveur
+        );
+        // si difference entre les champs, reperer les champs mal enregistres
+        if ($moof != $champs) {
+            $liste = [];
+            foreach ($moof as $k => $v) {
+                if (
+                    $v !== $champs[$k]
+                    // ne pas alerter si le champ est numerique est que les valeurs sont equivalentes
+                    and (!is_numeric($v) or intval($v) !== intval($champs[$k]))
+                    // ne pas alerter si le champ est date, qu'on a envoye une valeur vide et qu'on recupere une date nulle
+                    and (strlen($champs[$k]) or !in_array($v, ['0000-00-00 00:00:00', '0000-00-00']))
+                ) {
+                    $liste[] = $k;
+                    $conflits[$k]['post'] = $champs[$k];
+                    $conflits[$k]['save'] = $v;
+
+                    // cas specifique MySQL+emoji : si l'un est la
+                    // conversion utf8_noplanes de l'autre alors c'est OK
+                    if (defined('_MYSQL_NOPLANES') && _MYSQL_NOPLANES) {
+                        include_spip('inc/charsets');
+                        if ($v == utf8_noplanes($champs[$k])) {
+                            array_pop($liste);
+                        }
+                    }
+                }
+            }
+            // si un champ n'a pas ete correctement enregistre, loger et retourner une erreur
+            // c'est un cas exceptionnel
+            if (count($liste)) {
+                spip_log(
+                    "Erreur enregistrement en base $objet/$id_objet champs :" . var_export($conflits, true),
+                    'modifier.' . _LOG_CRITIQUE
+                );
+
+                return _T(
+                    'erreur_technique_enregistrement_champs',
+                    ['champs' => "<i>'" . implode("'</i>,<i>'", $liste) . "'</i>"]
+                );
+            }
+        }
+
+        // Invalider les caches
+        if (isset($options['invalideur']) and $options['invalideur']) {
+            include_spip('inc/invalideur');
+            if (is_array($options['invalideur'])) {
+                array_map('suivre_invalideur', $options['invalideur']);
+            } else {
+                suivre_invalideur($options['invalideur']);
+            }
+        }
+
+        // Notifications, gestion des revisions...
+        // en standard, appelle |nouvelle_revision ci-dessous
+        pipeline(
+            'post_edition',
+            [
+                'args' => [
+                    'table' => $spip_table_objet,
+                    'table_objet' => $table_objet,
+                    'spip_table_objet' => $spip_table_objet,
+                    'desc' => $desc,
+                    'type' => $objet,
+                    'id_objet' => $id_objet,
+                    'champs' => $options['champs'] ?? [], // [doc] kesako ?
+                    'champs_anciens' => $row, // état du contenu avant modif
+                    'serveur' => $serveur,
+                    'action' => $options['action'] ?? 'modifier'
+                ],
+                'data' => $champs
+            ]
+        );
+    }
+
+    // journaliser l'affaire
+    // message a affiner :-)
+    include_spip('inc/filtres_mini');
+    $qui = '';
+    if (!empty($GLOBALS['visiteur_session']['id_auteur'])) {
+        $qui .= ' #id_auteur:' . $GLOBALS['visiteur_session']['id_auteur'] . '#';
+    }
+    if (!empty($GLOBALS['visiteur_session']['nom'])) {
+        $qui .= ' #nom:' . $GLOBALS['visiteur_session']['nom'] . '#';
+    }
+    if ($qui == '') {
+        $qui = '#ip:' . $GLOBALS['ip'] . '#';
+    }
+    journal(_L($qui . ' a édité ' . $objet . ' ' . $id_objet . ' (' . join(
+        '+',
+        array_diff(array_keys($champs), ['date_modif'])
+    ) . ')'), [
+        'faire' => 'modifier',
+        'quoi' => $objet,
+        'id' => $id_objet
+    ]);
+
+    return '';
 }