spip / SPIP

ecrire/action/editer_auteur.php

Indentation +317 added lines, -317 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -36,40 +36,40 @@ 
  */
 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 = (int) $arg)
-		&& ($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)) && ($logo = reset($logo))) {
-				rename($logo, str_replace($id_hack, $id_auteur, (string) $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 = (int) $arg)
+        && ($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)) && ($logo = reset($logo))) {
+                rename($logo, str_replace($id_hack, $id_auteur, (string) $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];
 }
 
 /**
@@ -86,64 +86,64 @@ 
  */
 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
-		]
-	);
-
-	// Appeler une notification
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			'auteur_inserer',
-			$id_auteur,
-			[
-				'champs' => $champs,
-			]
-		);
-		$notifications(
-			'objet_inserer',
-			$id_auteur,
-			[
-				'objet' => 'auteur',
-				'id_objet' => $id_auteur,
-				'champs' => $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
+        ]
+    );
+
+    // Appeler une notification
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            'auteur_inserer',
+            $id_auteur,
+            [
+                'champs' => $champs,
+            ]
+        );
+        $notifications(
+            'objet_inserer',
+            $id_auteur,
+            [
+                'objet' => 'auteur',
+                'id_objet' => $id_auteur,
+                'champs' => $champs,
+            ]
+        );
+    }
+
+    return $id_auteur;
 }
 
 
@@ -168,70 +168,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']) && !isset($c['login'])) {
-			$c['login'] = $c['new_login'];
-		}
-		if (isset($c['new_pass']) && !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']) && !isset($c['login'])) {
+            $c['login'] = $c['new_login'];
+        }
+        if (isset($c['new_pass']) && !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;
 }
 
 /**
@@ -252,9 +252,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);
 }
 
 /**
@@ -273,9 +273,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);
 }
 
 /**
@@ -296,9 +296,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);
 }
 
 
@@ -317,158 +317,158 @@ 
  * @return bool|string
  */
 function auteur_instituer($id_auteur, $c, $force_webmestre = false) {
-	if (!$id_auteur = (int) $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']) && strlen((string) $c['login'])) {
-		$champs['login'] = $c['login'];
-	}
-	if (isset($c['pass']) && strlen((string) $c['pass'])) {
-		$champs['pass'] = $c['pass'];
-	}
-
-	$statut = $statut_ancien = sql_getfetsel('statut', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
-
-	if (
-		isset($c['statut']) && 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']) && $c['id_parent']) {
-		if (is_array($c['restreintes'])) {
-			$c['restreintes'][] = $c['id_parent'];
-		} else {
-			$c['restreintes'] = [$c['id_parent']];
-		}
-	}
-
-	if (
-		isset($c['webmestre'])
-		&& ($force_webmestre || 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']) && $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']) && is_array($c['restreintes'])
-		&& 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']) || isset($champs['pass'])) {
-		$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
-		include_spip('inc/auth');
-		if (
-			isset($champs['login'])
-			&& strlen((string) $champs['login'])
-			&& !auth_modifier_login($auth_methode, $champs['login'], $id_auteur)
-		) {
-			$erreurs[] = 'ecrire:impossible_modifier_login_auteur';
-		}
-		if (isset($champs['pass']) && strlen((string) $champs['pass'])) {
-			$champs['login'] = sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . (int) $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 || 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(
-			'auteur_instituer',
-			$id_auteur,
-			[
-				'statut' => $statut,
-				'statut_ancien' => $statut_ancien,
-				'champs' => $champs,
-			]
-		);
-		$notifications(
-			'objet_instituer',
-			$id_auteur,
-			[
-				'objet' => 'auteur',
-				'id_objet' => $id_auteur,
-				'statut' => $statut,
-				'statut_ancien' => $statut_ancien,
-				'champs' => $champs,
-			]
-		);
-
-		// Rétro-compat
-		$notifications(
-			'instituerauteur',
-			$id_auteur,
-			['statut' => $statut, 'statut_ancien' => $statut_ancien]
-		);
-	}
-
-	return implode(' ', array_map('_T', $erreurs));
+    if (!$id_auteur = (int) $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']) && strlen((string) $c['login'])) {
+        $champs['login'] = $c['login'];
+    }
+    if (isset($c['pass']) && strlen((string) $c['pass'])) {
+        $champs['pass'] = $c['pass'];
+    }
+
+    $statut = $statut_ancien = sql_getfetsel('statut', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+
+    if (
+        isset($c['statut']) && 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']) && $c['id_parent']) {
+        if (is_array($c['restreintes'])) {
+            $c['restreintes'][] = $c['id_parent'];
+        } else {
+            $c['restreintes'] = [$c['id_parent']];
+        }
+    }
+
+    if (
+        isset($c['webmestre'])
+        && ($force_webmestre || 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']) && $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']) && is_array($c['restreintes'])
+        && 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']) || isset($champs['pass'])) {
+        $auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+        include_spip('inc/auth');
+        if (
+            isset($champs['login'])
+            && strlen((string) $champs['login'])
+            && !auth_modifier_login($auth_methode, $champs['login'], $id_auteur)
+        ) {
+            $erreurs[] = 'ecrire:impossible_modifier_login_auteur';
+        }
+        if (isset($champs['pass']) && strlen((string) $champs['pass'])) {
+            $champs['login'] = sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . (int) $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 || 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(
+            'auteur_instituer',
+            $id_auteur,
+            [
+                'statut' => $statut,
+                'statut_ancien' => $statut_ancien,
+                'champs' => $champs,
+            ]
+        );
+        $notifications(
+            'objet_instituer',
+            $id_auteur,
+            [
+                'objet' => 'auteur',
+                'id_objet' => $id_auteur,
+                'statut' => $statut,
+                'statut_ancien' => $statut_ancien,
+                'champs' => $champs,
+            ]
+        );
+
+        // Rétro-compat
+        $notifications(
+            'instituerauteur',
+            $id_auteur,
+            ['statut' => $statut, 'statut_ancien' => $statut_ancien]
+        );
+    }
+
+    return implode(' ', array_map('_T', $erreurs));
 }

Spacing +5 added lines, -5 removed lines

@@ -91,7 +91,7 @@ 
 	$champs['source'] = $source ?: 'spip';
 
 	$champs['login'] = '';
-	$champs['statut'] = '5poubelle';  // inutilisable tant qu'il n'a pas ete renseigne et institue
+	$champs['statut'] = '5poubelle'; // inutilisable tant qu'il n'a pas ete renseigne et institue
 	$champs['webmestre'] = 'non';
 	if (empty($champs['imessage'])) {
 		$champs['imessage'] = 'oui';
@@ -331,7 +331,7 @@ 
 		$champs['pass'] = $c['pass'];
 	}
 
-	$statut = $statut_ancien = sql_getfetsel('statut', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+	$statut = $statut_ancien = sql_getfetsel('statut', 'spip_auteurs', 'id_auteur='.(int) $id_auteur);
 
 	if (
 		isset($c['statut']) && autoriser('modifier', 'auteur', $id_auteur, null, ['statut' => $c['statut']])
@@ -390,7 +390,7 @@ 
 	// commencer par traiter les cas particuliers des logins et pass
 	// avant les autres ecritures en base
 	if (isset($champs['login']) || isset($champs['pass'])) {
-		$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+		$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur='.(int) $id_auteur);
 		include_spip('inc/auth');
 		if (
 			isset($champs['login'])
@@ -400,7 +400,7 @@ 
 			$erreurs[] = 'ecrire:impossible_modifier_login_auteur';
 		}
 		if (isset($champs['pass']) && strlen((string) $champs['pass'])) {
-			$champs['login'] = sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+			$champs['login'] = sql_getfetsel('login', 'spip_auteurs', 'id_auteur='.(int) $id_auteur);
 			if (!auth_modifier_pass($auth_methode, $champs['login'], $champs['pass'], $id_auteur)) {
 				$erreurs[] = 'ecrire:impossible_modifier_pass_auteur';
 			}
@@ -413,7 +413,7 @@ 
 	if (!(is_countable($champs) ? count($champs) : 0)) {
 		return implode(' ', array_map('_T', $erreurs));
 	}
-	sql_updateq('spip_auteurs', $champs, 'id_auteur=' . $id_auteur);
+	sql_updateq('spip_auteurs', $champs, 'id_auteur='.$id_auteur);
 
 	// .. mettre a jour les fichiers .htpasswd et .htpasswd-admin
 	if ($flag_ecrire_acces || isset($champs['statut'])) {

ecrire/action/ajouter_lien.php

Indentation +9 added lines, -9 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -33,14 +33,14 @@ 
  * @return void
  */
 function action_ajouter_lien_dist($arg = null) {
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
 
-	$arg = explode('-', (string) $arg);
-	[$objet_source, $ids, $objet_lie, $idl] = $arg;
+    $arg = explode('-', (string) $arg);
+    [$objet_source, $ids, $objet_lie, $idl] = $arg;
 
-	include_spip('action/editer_liens');
-	objet_associer([$objet_source => $ids], [$objet_lie => $idl]);
+    include_spip('action/editer_liens');
+    objet_associer([$objet_source => $ids], [$objet_lie => $idl]);
 }

ecrire/action/redirect.php

Indentation +69 added lines, -69 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -35,53 +35,53 @@ 
  *   ```
  **/
 function action_redirect_dist() {
-	$type = _request('type');
-	$id = (int) _request('id');
-	$page = false;
+    $type = _request('type');
+    $id = (int) _request('id');
+    $page = false;
 
-	// verifier le type ou page transmis
-	if (!preg_match('/^\w+$/', (string) $type)) {
-		$page = _request('page');
-		if (!preg_match('/^\w+$/', (string) $page)) {
-			return;
-		}
-	}
+    // verifier le type ou page transmis
+    if (!preg_match('/^\w+$/', (string) $type)) {
+        $page = _request('page');
+        if (!preg_match('/^\w+$/', (string) $page)) {
+            return;
+        }
+    }
 
-	$var_mode = _request('var_mode');
-	// forcer la mise a jour de l'url de cet objet !
-	if ($var_mode && !defined('_VAR_URLS')) {
-		define('_VAR_URLS', true);
-	}
+    $var_mode = _request('var_mode');
+    // forcer la mise a jour de l'url de cet objet !
+    if ($var_mode && !defined('_VAR_URLS')) {
+        define('_VAR_URLS', true);
+    }
 
-	$url = $page
-		? generer_url_public($page, '', true)
-		: calculer_url_redirect_entite($type, $id, $var_mode);
+    $url = $page
+        ? generer_url_public($page, '', true)
+        : calculer_url_redirect_entite($type, $id, $var_mode);
 
-	$status = '302';
-	if ($url) {
-		if ($var_mode) {
-			$url = parametre_url($url, 'var_mode', $var_mode, '&');
-		}
+    $status = '302';
+    if ($url) {
+        if ($var_mode) {
+            $url = parametre_url($url, 'var_mode', $var_mode, '&');
+        }
 
-		if (
-			$var_mode == 'preview'
-			&& defined('_PREVIEW_TOKEN')
-			&& _PREVIEW_TOKEN
-			&& autoriser('previsualiser')
-		) {
-			include_spip('inc/securiser_action');
-			$token = calculer_token_previsu($url);
-			$url = parametre_url($url, 'var_previewtoken', $token);
-		}
+        if (
+            $var_mode == 'preview'
+            && defined('_PREVIEW_TOKEN')
+            && _PREVIEW_TOKEN
+            && autoriser('previsualiser')
+        ) {
+            include_spip('inc/securiser_action');
+            $token = calculer_token_previsu($url);
+            $url = parametre_url($url, 'var_previewtoken', $token);
+        }
 
-		if (_request('status') && _request('status') == '301') {
-			$status = '301';
-		}
-	} else {
-		$url = generer_url_public('404', '', true);
-	}
+        if (_request('status') && _request('status') == '301') {
+            $status = '301';
+        }
+    } else {
+        $url = generer_url_public('404', '', true);
+    }
 
-	redirige_par_entete(str_replace('&', '&', (string) $url), '', $status);
+    redirige_par_entete(str_replace('&', '&', (string) $url), '', $status);
 }
 
 /**
@@ -96,35 +96,35 @@ 
  * @return string|null
  */
 function calculer_url_redirect_entite($type, $id, $var_mode) {
-	$desc = null;
-	$publie = null;
-	$url = null;
-	// invalider le cache à chaque modif en bdd
-	$date = 0;
-	if (isset($GLOBALS['meta']['derniere_modif'])) {
-		$date = $GLOBALS['meta']['derniere_modif'];
-	}
-	$key = "url-$date-$type-$id";
+    $desc = null;
+    $publie = null;
+    $url = null;
+    // invalider le cache à chaque modif en bdd
+    $date = 0;
+    if (isset($GLOBALS['meta']['derniere_modif'])) {
+        $date = $GLOBALS['meta']['derniere_modif'];
+    }
+    $key = "url-$date-$type-$id";
 
-	// Obtenir l’url et si elle est publié du cache memoization
-	if (function_exists('cache_get') && ($desc = cache_get($key))) {
-		[$url, $publie] = $desc;
-	}
-	// Si on ne l’a pas trouvé, ou si var mode, on calcule l’url et son état publie
-	if (empty($desc) || $var_mode) {
-		$publie = objet_test_si_publie($type, $id);
-		$url = generer_objet_url_absolue($id, $type, '', '', true);
-		if (function_exists('cache_set')) {
-			cache_set($key, [$url, $publie], 3600);
-		}
-	}
+    // Obtenir l’url et si elle est publié du cache memoization
+    if (function_exists('cache_get') && ($desc = cache_get($key))) {
+        [$url, $publie] = $desc;
+    }
+    // Si on ne l’a pas trouvé, ou si var mode, on calcule l’url et son état publie
+    if (empty($desc) || $var_mode) {
+        $publie = objet_test_si_publie($type, $id);
+        $url = generer_objet_url_absolue($id, $type, '', '', true);
+        if (function_exists('cache_set')) {
+            cache_set($key, [$url, $publie], 3600);
+        }
+    }
 
-	// On valide l’url si elle est publiee ; sinon si preview on teste l’autorisation
-	if ($publie) {
-		return $url;
-	} elseif (defined('_VAR_PREVIEW') && _VAR_PREVIEW && autoriser('voir', $type, $id)) {
-		return $url;
-	}
+    // On valide l’url si elle est publiee ; sinon si preview on teste l’autorisation
+    if ($publie) {
+        return $url;
+    } elseif (defined('_VAR_PREVIEW') && _VAR_PREVIEW && autoriser('voir', $type, $id)) {
+        return $url;
+    }
 
-	return;
+    return;
 }

ecrire/action/editer_objet.php

Spacing +18 added lines, -18 removed lines

@@ -75,11 +75,11 @@ 
  */
 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);
+		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) && function_exists($modifier = $objet . '_modifier')
+		include_spip('action/editer_'.$objet) && function_exists($modifier = $objet.'_modifier')
 	) {
 		return $modifier($id, $set);
 	}
@@ -88,7 +88,7 @@ 
 	$trouver_table = charger_fonction('trouver_table', 'base');
 	$desc = $trouver_table($table_sql);
 	if (!$desc || !isset($desc['field'])) {
-		spip_log("Objet $objet inconnu dans objet_modifier", 'editer' . _LOG_ERREUR);
+		spip_log("Objet $objet inconnu dans objet_modifier", 'editer'._LOG_ERREUR);
 
 		return _L("Erreur objet $objet inconnu");
 	}
@@ -167,12 +167,12 @@ 
 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);
+		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)
-		&& function_exists($inserer = $objet . '_inserer')
+		include_spip('action/editer_'.$objet)
+		&& function_exists($inserer = $objet.'_inserer')
 	) {
 		return $inserer($id_parent, $set);
 	}
@@ -193,7 +193,7 @@ 
 			$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=' . (int) $id_rubrique);
+			$row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique='.(int) $id_rubrique);
 		}
 
 		$champs['id_rubrique'] = $id_rubrique;
@@ -330,11 +330,11 @@ 
  */
 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);
+		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) && function_exists($instituer = $objet . '_instituer')
+		include_spip('action/editer_'.$objet) && function_exists($instituer = $objet.'_instituer')
 	) {
 		return $instituer($id, $c, $calcul_rub);
 	}
@@ -363,7 +363,7 @@ 
 	$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) . '=' . (int) $id);
+	$row = sql_fetsel($sel, $table_sql, id_table_objet($objet).'='.(int) $id);
 
 	$id_rubrique = $row['id_rubrique'];
 	$statut_ancien = $statut = $row['statut'];
@@ -386,7 +386,7 @@ 
 			if ($s != 'publie' && 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);
+				spip_log("editer_objet $objet #$id refus ".json_encode($c, JSON_THROW_ON_ERROR), 'editer'._LOG_INFO_IMPORTANTE);
 			}
 		}
 
@@ -414,7 +414,7 @@ 
 		&& isset($c['id_parent'])
 		&& ($id_parent = $c['id_parent'])
 		&& $id_parent != $id_rubrique
-		&& sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . (int) $id_parent)
+		&& sql_fetsel('1', 'spip_rubriques', 'id_rubrique='.(int) $id_parent)
 	) {
 		$champs['id_rubrique'] = $id_parent;
 
@@ -541,7 +541,7 @@ 
 	// 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']));
+		$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'])) {
@@ -554,7 +554,7 @@ 
 			&& sql_fetsel(
 				'1',
 				$table_sql,
-				id_table_objet($objet) . '=' . (int) $id . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
+				id_table_objet($objet).'='.(int) $id." AND langue_choisie<>'oui' AND lang<>".sql_quote($langue)
 			)
 		) {
 			$champs['lang'] = $langue;
@@ -564,7 +564,7 @@ 
 	if (!$champs) {
 		return;
 	}
-	sql_updateq($table_sql, $champs, id_table_objet($objet) . '=' . (int) $id);
+	sql_updateq($table_sql, $champs, id_table_objet($objet).'='.(int) $id);
 
 	// Changer le statut des rubriques concernees
 	if ($cond) {
@@ -608,7 +608,7 @@ 
  */
 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);
+		spip_log("objet_lire: appel avec type $objet invalide au lieu de $t", 'editer'._LOG_INFO_IMPORTANTE);
 		$objet = $t;
 	}
 
@@ -630,7 +630,7 @@ 
 	) {
 		// 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)
+			include_spip('action/editer_'.$objet)
 			&& function_exists($lire = "{$objet}_lire_champs")
 		) {
 			$valeurs = $lire($objet, $valeur_id, $champ_id);
@@ -640,7 +640,7 @@ 
 
 			// La condition est appliquée sur le champ désigné par l'utilisateur.
 			$where = [
-				$champ_id . '=' . sql_quote($valeur_id)
+				$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.

Indentation +551 added lines, -551 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -32,36 +32,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) || is_null($objet)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-		[$objet, $id] = array_pad(explode('/', (string) $arg, 2), 2, null);
-	}
-
-	// appel incorrect ou depuis une url erronnée interdit
-	if (is_null($id) || 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 = (int) $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 = (int) $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) || is_null($objet)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+        [$objet, $id] = array_pad(explode('/', (string) $arg, 2), 2, null);
+    }
+
+    // appel incorrect ou depuis une url erronnée interdit
+    if (is_null($id) || 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 = (int) $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 = (int) $id) > 0) {
+        return [$id, _L('echec enregistrement en base')];
+    }
+
+    // Enregistre l'envoi dans la BD
+    $err = objet_modifier($objet, $id, $set);
+
+    return [$id, $err];
 }
 
 /**
@@ -90,83 +90,83 @@ 
  *   - En cas d'erreur : (string) message d'erreur
  */
 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) && 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 || !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']) && $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']) && 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);
-
-	return objet_instituer($objet, $id, $c);
+    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) && 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 || !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']) && $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']) && 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);
+
+    return objet_instituer($objet, $id, $c);
 }
 
 /**
@@ -197,154 +197,154 @@ 
  *     - (int) 0 si le type d'objet n'existe pas ou que la table est mal déclarée
  */
 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)
-		&& 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 || !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 = (int) $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=' . (int) $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']) && !empty($GLOBALS['meta']['multi_objets']) && in_array(
-			$table_sql,
-			explode(',', (string) $GLOBALS['meta']['multi_objets'])
-		)
-	) {
-		lang_select($GLOBALS['visiteur_session']['lang']);
-		if (
-			in_array(
-				$GLOBALS['spip_lang'],
-				explode(',', (string) $GLOBALS['meta']['langues_multilingue'])
-			)
-		) {
-			$champs['lang'] = $GLOBALS['spip_lang'];
-			if (isset($desc['field']['langue_choisie'])) {
-				$champs['langue_choisie'] = 'oui';
-			}
-		}
-	} elseif (isset($desc['field']['lang']) && 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']) && ($d = $desc['date']) || 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 && !isset($desc['field']['id_auteur'])) {
-			$id_auteur = ((is_null(_request('id_auteur')) && 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
-			]
-		);
-	}
-
-	// Appeler une notification
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			"{$objet}_inserer",
-			$id,
-			[
-				'id_parent' => $id_parent,
-				'champs' => $champs,
-			]
-		);
-		$notifications(
-			'objet_inserer',
-			$id,
-			[
-				'objet' => $objet,
-				'id_objet' => $id,
-				'id_parent' => $id_parent,
-				'champs' => $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)
+        && 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 || !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 = (int) $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=' . (int) $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']) && !empty($GLOBALS['meta']['multi_objets']) && in_array(
+            $table_sql,
+            explode(',', (string) $GLOBALS['meta']['multi_objets'])
+        )
+    ) {
+        lang_select($GLOBALS['visiteur_session']['lang']);
+        if (
+            in_array(
+                $GLOBALS['spip_lang'],
+                explode(',', (string) $GLOBALS['meta']['langues_multilingue'])
+            )
+        ) {
+            $champs['lang'] = $GLOBALS['spip_lang'];
+            if (isset($desc['field']['langue_choisie'])) {
+                $champs['langue_choisie'] = 'oui';
+            }
+        }
+    } elseif (isset($desc['field']['lang']) && 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']) && ($d = $desc['date']) || 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 && !isset($desc['field']['id_auteur'])) {
+            $id_auteur = ((is_null(_request('id_auteur')) && 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
+            ]
+        );
+    }
+
+    // Appeler une notification
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            "{$objet}_inserer",
+            $id,
+            [
+                'id_parent' => $id_parent,
+                'champs' => $champs,
+            ]
+        );
+        $notifications(
+            'objet_inserer',
+            $id,
+            [
+                'objet' => $objet,
+                'id_objet' => $id,
+                'id_parent' => $id_parent,
+                'champs' => $champs,
+            ]
+        );
+    }
+
+    return $id;
 }
 
 
@@ -361,135 +361,135 @@ 
  * @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) && 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 || !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']) && $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) . '=' . (int) $id);
-
-	$id_rubrique = $row['id_rubrique'];
-	$statut_ancien = $statut = $row['statut'];
-	$date_ancienne = $date = $row['date'];
-	$champs = [];
-
-	$d = ($date && isset($c[$champ_date])) ? $c[$champ_date] : null;
-	$s = (isset($desc['field']['statut']) && isset($c['statut'])) ? $c['statut'] : $statut;
-
-	// cf autorisations dans inc/instituer_objet
-	if ($s != $statut || $d && $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' && 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
-			&& (
-				$champs['statut'] == 'publie'
-				|| $champs['statut'] == 'prop' && !in_array($statut_ancien, ['publie', 'prop'])
-				|| $d
-			)
-		) {
-			$date = ($d || strtotime((string) ($d = $date)) > time()) ? $d : date('Y-m-d H:i:s');
-			$champs[$champ_date] = $date;
-		}
-	}
-
-	// Verifier que la rubrique demandee existe et est differente
-	// de la rubrique actuelle
-	if (
-		$id_rubrique
-		&& isset($c['id_parent'])
-		&& ($id_parent = $c['id_parent'])
-		&& $id_parent != $id_rubrique
-		&& sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . (int) $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'
-			&& !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) && 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 || !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']) && $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) . '=' . (int) $id);
+
+    $id_rubrique = $row['id_rubrique'];
+    $statut_ancien = $statut = $row['statut'];
+    $date_ancienne = $date = $row['date'];
+    $champs = [];
+
+    $d = ($date && isset($c[$champ_date])) ? $c[$champ_date] : null;
+    $s = (isset($desc['field']['statut']) && isset($c['statut'])) ? $c['statut'] : $statut;
+
+    // cf autorisations dans inc/instituer_objet
+    if ($s != $statut || $d && $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' && 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
+            && (
+                $champs['statut'] == 'publie'
+                || $champs['statut'] == 'prop' && !in_array($statut_ancien, ['publie', 'prop'])
+                || $d
+            )
+        ) {
+            $date = ($d || strtotime((string) ($d = $date)) > time()) ? $d : date('Y-m-d H:i:s');
+            $champs[$champ_date] = $date;
+        }
+    }
+
+    // Verifier que la rubrique demandee existe et est differente
+    // de la rubrique actuelle
+    if (
+        $id_rubrique
+        && isset($c['id_parent'])
+        && ($id_parent = $c['id_parent'])
+        && $id_parent != $id_rubrique
+        && sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . (int) $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'
+            && !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'];
@@ -498,60 +498,60 @@ 
 		}
 	}*/
 
-	// 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(
-			"{$objet}_instituer",
-			$id,
-			[
-				'statut' => $statut,
-				'statut_ancien' => $statut_ancien,
-				'date' => $date,
-				'date_ancienne' => $date_ancienne,
-				'id_parent_ancien' => $id_rubrique,
-				'champs' => $champs,
-			]
-		);
-		$notifications(
-			'objet_instituer',
-			$id,
-			[
-				'objet' => $objet,
-				'id_objet' => $id,
-				'statut' => $statut,
-				'statut_ancien' => $statut_ancien,
-				'date' => $date,
-				'date_ancienne' => $date_ancienne,
-				'id_parent_ancien' => $id_rubrique,
-				'champs' => $champs,
-			]
-		);
-
-		// Rétro-compat
-		$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(
+            "{$objet}_instituer",
+            $id,
+            [
+                'statut' => $statut,
+                'statut_ancien' => $statut_ancien,
+                'date' => $date,
+                'date_ancienne' => $date_ancienne,
+                'id_parent_ancien' => $id_rubrique,
+                'champs' => $champs,
+            ]
+        );
+        $notifications(
+            'objet_instituer',
+            $id,
+            [
+                'objet' => $objet,
+                'id_objet' => $id,
+                'statut' => $statut,
+                'statut_ancien' => $statut_ancien,
+                'date' => $date,
+                'date_ancienne' => $date_ancienne,
+                'id_parent_ancien' => $id_rubrique,
+                'champs' => $champs,
+            ]
+        );
+
+        // Rétro-compat
+        $notifications(
+            "instituer$objet",
+            $id,
+            ['statut' => $statut, 'statut_ancien' => $statut_ancien, 'date' => $date, 'date_ancienne' => $date_ancienne]
+        );
+    }
+
+    return ''; // pas d'erreur
 }
 
 /**
@@ -566,51 +566,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'])
-			&& isset($desc['field']['langue_choisie'])
-			&& sql_fetsel(
-				'1',
-				$table_sql,
-				id_table_objet($objet) . '=' . (int) $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) . '=' . (int) $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'])
+            && isset($desc['field']['langue_choisie'])
+            && sql_fetsel(
+                '1',
+                $table_sql,
+                id_table_objet($objet) . '=' . (int) $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) . '=' . (int) $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);
+    }
 }
 
 
@@ -639,71 +639,71 @@ 
  *     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']) ? $primary : $options['champ_id']);
-
-	// 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])
-		|| isset($options['force']) && $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)
-			&& 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 && 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 && !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.
-		$retour = is_array($champs)
-			? array_intersect_key($retour, array_flip($champs))
-			: $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']) ? $primary : $options['champ_id']);
+
+    // 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])
+        || isset($options['force']) && $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)
+            && 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 && 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 && !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.
+        $retour = is_array($champs)
+            ? array_intersect_key($retour, array_flip($champs))
+            : $retour[$champs] ?? false;
+    }
+
+    return $retour;
 }

ecrire/action/confirmer_email.php

Indentation +19 added lines, -19 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -27,23 +27,23 @@ 
  * @return void
  */
 function action_confirmer_email_dist() {
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$arg = $securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $arg = $securiser_action();
 
-	include_spip('inc/filtres');
-	if ($GLOBALS['visiteur_session']['id_auteur'] && email_valide($arg)) {
-		include_spip('action/editer_auteur');
-		auteur_modifier($GLOBALS['visiteur_session']['id_auteur'], ['email' => $arg]);
-	}
-	// verifier avant de rediriger pour invalider le message de confirmation
-	// si ca n'a pas marche
-	if (
-		($redirect = _request('redirect')) && !$arg == sql_getfetsel(
-			'email',
-			'spip_auteurs',
-			'id_auteur=' . (int) $GLOBALS['visiteur_session']
-		)
-	) {
-		$GLOBALS['redirect'] = parametre_url($redirect, 'email_modif', '');
-	}
+    include_spip('inc/filtres');
+    if ($GLOBALS['visiteur_session']['id_auteur'] && email_valide($arg)) {
+        include_spip('action/editer_auteur');
+        auteur_modifier($GLOBALS['visiteur_session']['id_auteur'], ['email' => $arg]);
+    }
+    // verifier avant de rediriger pour invalider le message de confirmation
+    // si ca n'a pas marche
+    if (
+        ($redirect = _request('redirect')) && !$arg == sql_getfetsel(
+            'email',
+            'spip_auteurs',
+            'id_auteur=' . (int) $GLOBALS['visiteur_session']
+        )
+    ) {
+        $GLOBALS['redirect'] = parametre_url($redirect, 'email_modif', '');
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 		($redirect = _request('redirect')) && !$arg == sql_getfetsel(
 			'email',
 			'spip_auteurs',
-			'id_auteur=' . (int) $GLOBALS['visiteur_session']
+			'id_auteur='.(int) $GLOBALS['visiteur_session']
 		)
 	) {
 		$GLOBALS['redirect'] = parametre_url($redirect, 'email_modif', '');

ecrire/action/debloquer_edition.php

Indentation +13 added lines, -13 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -30,17 +30,17 @@ 
  */
 function action_debloquer_edition_dist() {
 
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$arg = $securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $arg = $securiser_action();
 
-	if ($arg) {
-		include_spip('inc/drapeau_edition');
-		if ($arg == 'tous') {
-			debloquer_tous($GLOBALS['visiteur_session']['id_auteur']);
-		} else {
-			$arg = explode('-', (string) $arg);
-			[$objet, $id_objet] = $arg;
-			debloquer_edition($GLOBALS['visiteur_session']['id_auteur'], $id_objet, $objet);
-		}
-	}
+    if ($arg) {
+        include_spip('inc/drapeau_edition');
+        if ($arg == 'tous') {
+            debloquer_tous($GLOBALS['visiteur_session']['id_auteur']);
+        } else {
+            $arg = explode('-', (string) $arg);
+            [$objet, $id_objet] = $arg;
+            debloquer_edition($GLOBALS['visiteur_session']['id_auteur'], $id_objet, $objet);
+        }
+    }
 }

ecrire/action/calculer_taille_cache.php

Indentation +57 added lines, -57 removed lines

@@ -15,7 +15,7 @@ 
  * @package SPIP\Core\Cache
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -30,41 +30,41 @@ 
  *     de l'action sécurisée.
  */
 function action_calculer_taille_cache_dist($arg = null) {
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
-	include_spip('inc/filtres');
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
+    include_spip('inc/filtres');
 
-	if ($arg == 'images') {
-		$taille = calculer_taille_dossier(_DIR_VAR);
-		$res = _T(
-			'ecrire:taille_cache_image',
-			[
-				'dir' => joli_repertoire(_DIR_VAR),
-				'taille' => '<b>' . (taille_en_octets($taille) > 0 ? taille_en_octets($taille) : '0 octet') . '</b>'
-			]
-		);
-	} else {
-		include_spip('inc/invalideur');
-		$taille =
-			calculer_taille_dossier(_DIR_CACHE_XML)
-			+ calculer_taille_dossier(_DIR_CACHE . 'skel/')
-			+ calculer_taille_dossier(_DIR_CACHE . 'wheels/')
-			+ calculer_taille_dossier(_DIR_CACHE . 'contextes/');
-		$taille += (int) taille_du_cache();
-		if ($taille <= 150000) {
-			$res = _T('taille_cache_vide');
-		} elseif ($taille <= 1024 * 1024) {
-			$res = _T('taille_cache_moins_de', ['octets' => taille_en_octets(1024 * 1024)]);
-		} else {
-			$res = _T('taille_cache_octets', ['octets' => taille_en_octets($taille)]);
-		}
-		$res = "<b>$res</b>";
-	}
+    if ($arg == 'images') {
+        $taille = calculer_taille_dossier(_DIR_VAR);
+        $res = _T(
+            'ecrire:taille_cache_image',
+            [
+                'dir' => joli_repertoire(_DIR_VAR),
+                'taille' => '<b>' . (taille_en_octets($taille) > 0 ? taille_en_octets($taille) : '0 octet') . '</b>'
+            ]
+        );
+    } else {
+        include_spip('inc/invalideur');
+        $taille =
+            calculer_taille_dossier(_DIR_CACHE_XML)
+            + calculer_taille_dossier(_DIR_CACHE . 'skel/')
+            + calculer_taille_dossier(_DIR_CACHE . 'wheels/')
+            + calculer_taille_dossier(_DIR_CACHE . 'contextes/');
+        $taille += (int) taille_du_cache();
+        if ($taille <= 150000) {
+            $res = _T('taille_cache_vide');
+        } elseif ($taille <= 1024 * 1024) {
+            $res = _T('taille_cache_moins_de', ['octets' => taille_en_octets(1024 * 1024)]);
+        } else {
+            $res = _T('taille_cache_octets', ['octets' => taille_en_octets($taille)]);
+        }
+        $res = "<b>$res</b>";
+    }
 
-	$res = "<p>$res</p>";
-	ajax_retour($res);
+    $res = "<p>$res</p>";
+    ajax_retour($res);
 }
 
 
@@ -75,28 +75,28 @@ 
  * @return int Taille en octets
  */
 function calculer_taille_dossier($dir) {
-	if (!is_dir($dir) || !is_readable($dir)) {
-		return 0;
-	}
-	$handle = opendir($dir);
-	if (!$handle) {
-		return 0;
-	}
-	$taille = 0;
-	while (($fichier = @readdir($handle)) !== false) {
-		// Eviter ".", "..", ".htaccess", etc.
-		if ($fichier[0] == '.') {
-			continue;
-		}
-		if (is_file($d = "$dir/$fichier")) {
-			$taille += filesize($d);
-		} else {
-			if (is_dir($d)) {
-				$taille += calculer_taille_dossier($d);
-			}
-		}
-	}
-	closedir($handle);
+    if (!is_dir($dir) || !is_readable($dir)) {
+        return 0;
+    }
+    $handle = opendir($dir);
+    if (!$handle) {
+        return 0;
+    }
+    $taille = 0;
+    while (($fichier = @readdir($handle)) !== false) {
+        // Eviter ".", "..", ".htaccess", etc.
+        if ($fichier[0] == '.') {
+            continue;
+        }
+        if (is_file($d = "$dir/$fichier")) {
+            $taille += filesize($d);
+        } else {
+            if (is_dir($d)) {
+                $taille += calculer_taille_dossier($d);
+            }
+        }
+    }
+    closedir($handle);
 
-	return $taille;
+    return $taille;
 }

Spacing +4 added lines, -4 removed lines

@@ -42,16 +42,16 @@
 			'ecrire:taille_cache_image',
 			[
 				'dir' => joli_repertoire(_DIR_VAR),
-				'taille' => '<b>' . (taille_en_octets($taille) > 0 ? taille_en_octets($taille) : '0 octet') . '</b>'
+				'taille' => '<b>'.(taille_en_octets($taille) > 0 ? taille_en_octets($taille) : '0 octet').'</b>'
 			]
 		);
 	} else {
 		include_spip('inc/invalideur');
 		$taille =
 			calculer_taille_dossier(_DIR_CACHE_XML)
-			+ calculer_taille_dossier(_DIR_CACHE . 'skel/')
-			+ calculer_taille_dossier(_DIR_CACHE . 'wheels/')
-			+ calculer_taille_dossier(_DIR_CACHE . 'contextes/');
+			+ calculer_taille_dossier(_DIR_CACHE.'skel/')
+			+ calculer_taille_dossier(_DIR_CACHE.'wheels/')
+			+ calculer_taille_dossier(_DIR_CACHE.'contextes/');
 		$taille += (int) taille_du_cache();
 		if ($taille <= 150000) {
 			$res = _T('taille_cache_vide');

ecrire/action/menu_rubriques.php

Indentation +155 added lines, -155 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/autoriser');
@@ -34,42 +34,42 @@ 
  **/
 function action_menu_rubriques_dist() {
 
-	// si pas acces a ecrire, pas acces au menu
-	// on renvoi un 401 qui fait echouer la requete ajax silencieusement
-	if (!autoriser('ecrire')) {
-		$retour =
-		'<ul class="deroulant__sous-menu" data-profondeur="1">' .
-			'<li class="deroulant__item deroulant__item_plan plan_site" data-profondeur="1">' .
-				'<a class="deroulant__lien" href="' . generer_url_ecrire('accueil') . '" data-profondeur="1">' .
-					'<span class="libelle">' . _T('public:lien_connecter') . '</span>' .
-				'</a>' .
-			'</li>' .
-		'</ul>';
-		include_spip('inc/actions');
-		ajax_retour($retour);
-		exit;
-	}
-
-	if ($date = (int) _request('date')) {
-		header('Last-Modified: ' . gmdate('D, d M Y H:i:s', $date) . ' GMT');
-	}
-
-	$r = gen_liste_rubriques();
-	if (
-		!$r
-		&& isset($_SERVER['HTTP_IF_MODIFIED_SINCE'])
-		&& !strstr((string) $_SERVER['SERVER_SOFTWARE'], 'IIS/')
-	) {
-		include_spip('inc/headers');
-		header('Content-Type: text/html; charset=' . $GLOBALS['meta']['charset']);
-		http_response_code(304);
-		exit;
-	} else {
-		include_spip('inc/actions');
-		$ret = menu_rubriques();
-		ajax_retour($ret);
-		exit;
-	}
+    // si pas acces a ecrire, pas acces au menu
+    // on renvoi un 401 qui fait echouer la requete ajax silencieusement
+    if (!autoriser('ecrire')) {
+        $retour =
+        '<ul class="deroulant__sous-menu" data-profondeur="1">' .
+            '<li class="deroulant__item deroulant__item_plan plan_site" data-profondeur="1">' .
+                '<a class="deroulant__lien" href="' . generer_url_ecrire('accueil') . '" data-profondeur="1">' .
+                    '<span class="libelle">' . _T('public:lien_connecter') . '</span>' .
+                '</a>' .
+            '</li>' .
+        '</ul>';
+        include_spip('inc/actions');
+        ajax_retour($retour);
+        exit;
+    }
+
+    if ($date = (int) _request('date')) {
+        header('Last-Modified: ' . gmdate('D, d M Y H:i:s', $date) . ' GMT');
+    }
+
+    $r = gen_liste_rubriques();
+    if (
+        !$r
+        && isset($_SERVER['HTTP_IF_MODIFIED_SINCE'])
+        && !strstr((string) $_SERVER['SERVER_SOFTWARE'], 'IIS/')
+    ) {
+        include_spip('inc/headers');
+        header('Content-Type: text/html; charset=' . $GLOBALS['meta']['charset']);
+        http_response_code(304);
+        exit;
+    } else {
+        include_spip('inc/actions');
+        $ret = menu_rubriques();
+        ajax_retour($ret);
+        exit;
+    }
 }
 
 /**
@@ -83,44 +83,44 @@ 
  *     Code HTML présentant la liste des rubriques
  **/
 function menu_rubriques($complet = true) {
-	$ret = '<li class="deroulant__item deroulant__item_tout toutsite" data-profondeur="1">'
-		. '<a class="deroulant__lien" href="' . generer_url_ecrire('plan') . '" data-profondeur="1">'
-		. '<span class="libelle">' . _T('info_tout_site') . '</span>'
-		. '</a>'
-		. '</li>';
-
-	if (!$complet) {
-		return "<ul class=\"deroulant__sous-menu\" data-profondeur=\"1\">$ret\n</ul>\n";
-	}
-
-	if (!isset($GLOBALS['db_art_cache'])) {
-		gen_liste_rubriques();
-	}
-	$arr_low = extraire_article(0, $GLOBALS['db_art_cache']);
-
-	$total_lignes = $i = count($arr_low);
-
-	if ($i > 0) {
-		$nb_col = min(8, ceil($total_lignes / 30));
-		if ($nb_col <= 1) {
-			$nb_col = ceil($total_lignes / 10);
-		}
-		foreach ($arr_low as $id_rubrique => $titre_rubrique) {
-			if (autoriser('voir', 'rubrique', $id_rubrique)) {
-				$ret .= bandeau_rubrique($id_rubrique, $titre_rubrique, $i);
-				$i++;
-			}
-		}
-
-		$class_cols = ($nb_col > 1 ? "cols-$nb_col" : '');
-		$ret = "<ul class=\"deroulant__sous-menu $class_cols\" data-profondeur=\"1\">"
-			. $ret
-			. "\n</ul>\n";
-	} else {
-		$ret = "<ul class=\"deroulant__sous-menu\" data-profondeur=\"1\">$ret\n</ul>\n";
-	}
-
-	return $ret;
+    $ret = '<li class="deroulant__item deroulant__item_tout toutsite" data-profondeur="1">'
+        . '<a class="deroulant__lien" href="' . generer_url_ecrire('plan') . '" data-profondeur="1">'
+        . '<span class="libelle">' . _T('info_tout_site') . '</span>'
+        . '</a>'
+        . '</li>';
+
+    if (!$complet) {
+        return "<ul class=\"deroulant__sous-menu\" data-profondeur=\"1\">$ret\n</ul>\n";
+    }
+
+    if (!isset($GLOBALS['db_art_cache'])) {
+        gen_liste_rubriques();
+    }
+    $arr_low = extraire_article(0, $GLOBALS['db_art_cache']);
+
+    $total_lignes = $i = count($arr_low);
+
+    if ($i > 0) {
+        $nb_col = min(8, ceil($total_lignes / 30));
+        if ($nb_col <= 1) {
+            $nb_col = ceil($total_lignes / 10);
+        }
+        foreach ($arr_low as $id_rubrique => $titre_rubrique) {
+            if (autoriser('voir', 'rubrique', $id_rubrique)) {
+                $ret .= bandeau_rubrique($id_rubrique, $titre_rubrique, $i);
+                $i++;
+            }
+        }
+
+        $class_cols = ($nb_col > 1 ? "cols-$nb_col" : '');
+        $ret = "<ul class=\"deroulant__sous-menu $class_cols\" data-profondeur=\"1\">"
+            . $ret
+            . "\n</ul>\n";
+    } else {
+        $ret = "<ul class=\"deroulant__sous-menu\" data-profondeur=\"1\">$ret\n</ul>\n";
+    }
+
+    return $ret;
 }
 
 /**
@@ -141,47 +141,47 @@ 
  *     Code HTML présentant la liste des rubriques
  **/
 function bandeau_rubrique($id_rubrique, $titre_rubrique, $zdecal, $profondeur = 1) {
-	static $zmax = 6;
-	$profondeur_next = $profondeur + 1;
-
-	$nav = '<a class="deroulant__lien" href="' . generer_objet_url($id_rubrique, 'rubrique', '', '', false) . "\" data-profondeur=\"$profondeur\">"
-		. '<span class="libelle">' . supprimer_tags(preg_replace(',[\x00-\x1f]+,', ' ', $titre_rubrique)) . '</span>'
-		. "</a>\n";
-
-	// Limiter volontairement le nombre de sous-menus
-	if (!(--$zmax)) {
-		$zmax++;
-
-		return "\n<li class=\"deroulant__item\" data-profondeur=\"$profondeur\">$nav</li>";
-	}
-
-	$arr_rub = extraire_article($id_rubrique, $GLOBALS['db_art_cache']);
-	$i = count($arr_rub);
-	if (!$i) {
-		$zmax++;
-
-		return "\n<li class=\"deroulant__item\" data-profondeur=\"$profondeur\">$nav</li>";
-	}
-
-
-	$nb_col = 1;
-	if ($nb_rub = count($arr_rub)) {
-		$nb_col = min(10, max(1, ceil($nb_rub / 10)));
-	}
-	$class_cols = ($nb_col > 1 ? "cols-$nb_col" : '');
-	$ret = "<li class=\"deroulant__item deroulant__item_parent\" data-profondeur=\"$profondeur\">"
-	 . $nav
-	 . "<ul class=\"deroulant__sous-menu $class_cols\" data-profondeur=\"$profondeur_next\">";
-	foreach ($arr_rub as $id_rub => $titre_rub) {
-		if (autoriser('voir', 'rubrique', $id_rub)) {
-			$titre = supprimer_numero(typo($titre_rub));
-			$ret .= bandeau_rubrique($id_rub, $titre, $zdecal + $i, $profondeur_next);
-			$i++;
-		}
-	}
-	$zmax++;
-
-	return $ret . "</ul></li>\n";
+    static $zmax = 6;
+    $profondeur_next = $profondeur + 1;
+
+    $nav = '<a class="deroulant__lien" href="' . generer_objet_url($id_rubrique, 'rubrique', '', '', false) . "\" data-profondeur=\"$profondeur\">"
+        . '<span class="libelle">' . supprimer_tags(preg_replace(',[\x00-\x1f]+,', ' ', $titre_rubrique)) . '</span>'
+        . "</a>\n";
+
+    // Limiter volontairement le nombre de sous-menus
+    if (!(--$zmax)) {
+        $zmax++;
+
+        return "\n<li class=\"deroulant__item\" data-profondeur=\"$profondeur\">$nav</li>";
+    }
+
+    $arr_rub = extraire_article($id_rubrique, $GLOBALS['db_art_cache']);
+    $i = count($arr_rub);
+    if (!$i) {
+        $zmax++;
+
+        return "\n<li class=\"deroulant__item\" data-profondeur=\"$profondeur\">$nav</li>";
+    }
+
+
+    $nb_col = 1;
+    if ($nb_rub = count($arr_rub)) {
+        $nb_col = min(10, max(1, ceil($nb_rub / 10)));
+    }
+    $class_cols = ($nb_col > 1 ? "cols-$nb_col" : '');
+    $ret = "<li class=\"deroulant__item deroulant__item_parent\" data-profondeur=\"$profondeur\">"
+        . $nav
+        . "<ul class=\"deroulant__sous-menu $class_cols\" data-profondeur=\"$profondeur_next\">";
+    foreach ($arr_rub as $id_rub => $titre_rub) {
+        if (autoriser('voir', 'rubrique', $id_rub)) {
+            $titre = supprimer_numero(typo($titre_rub));
+            $ret .= bandeau_rubrique($id_rub, $titre, $zdecal + $i, $profondeur_next);
+            $i++;
+        }
+    }
+    $zmax++;
+
+    return $ret . "</ul></li>\n";
 }
 
 
@@ -199,7 +199,7 @@ 
  *     Liste des rubriques enfants de la rubrique (et leur titre)
  **/
 function extraire_article($id_p, $t) {
-	return array_key_exists($id_p, $t) ? $t[$id_p] : [];
+    return array_key_exists($id_p, $t) ? $t[$id_p] : [];
 }
 
 /**
@@ -215,42 +215,42 @@ 
  **/
 function gen_liste_rubriques() {
 
-	$cache = null;
-	include_spip('inc/config');
-	// ici, un petit fichier cache ne fait pas de mal
-	$last = lire_config('date_calcul_rubriques', 0);
-	if (lire_fichier(_CACHE_RUBRIQUES, $cache)) {
-		[$date, $GLOBALS['db_art_cache']] = @unserialize($cache);
-		if ($date == $last) {
-			return false;
-		} // c'etait en cache :-)
-	}
-	// se restreindre aux rubriques utilisees recemment +secteurs
-
-	$where = sql_in_select(
-		'id_rubrique',
-		'id_rubrique',
-		'spip_rubriques',
-		'',
-		'',
-		'id_parent=0 DESC, date DESC',
-		_CACHE_RUBRIQUES_MAX
-	);
-
-	// puis refaire la requete pour avoir l'ordre alphabetique
-
-	$res = sql_select('id_rubrique, titre, id_parent', 'spip_rubriques', $where, '', 'id_parent, 0+titre, titre');
-
-	// il ne faut pas filtrer le autoriser voir ici
-	// car on met le resultat en cache, commun a tout le monde
-	$GLOBALS['db_art_cache'] = [];
-	while ($r = sql_fetch($res)) {
-		$t = sinon($r['titre'], _T('ecrire:info_sans_titre'));
-		$GLOBALS['db_art_cache'][$r['id_parent']][$r['id_rubrique']] = supprimer_numero(typo($t));
-	}
-
-	$t = [$last ?: time(), $GLOBALS['db_art_cache']];
-	ecrire_fichier(_CACHE_RUBRIQUES, serialize($t));
-
-	return true;
+    $cache = null;
+    include_spip('inc/config');
+    // ici, un petit fichier cache ne fait pas de mal
+    $last = lire_config('date_calcul_rubriques', 0);
+    if (lire_fichier(_CACHE_RUBRIQUES, $cache)) {
+        [$date, $GLOBALS['db_art_cache']] = @unserialize($cache);
+        if ($date == $last) {
+            return false;
+        } // c'etait en cache :-)
+    }
+    // se restreindre aux rubriques utilisees recemment +secteurs
+
+    $where = sql_in_select(
+        'id_rubrique',
+        'id_rubrique',
+        'spip_rubriques',
+        '',
+        '',
+        'id_parent=0 DESC, date DESC',
+        _CACHE_RUBRIQUES_MAX
+    );
+
+    // puis refaire la requete pour avoir l'ordre alphabetique
+
+    $res = sql_select('id_rubrique, titre, id_parent', 'spip_rubriques', $where, '', 'id_parent, 0+titre, titre');
+
+    // il ne faut pas filtrer le autoriser voir ici
+    // car on met le resultat en cache, commun a tout le monde
+    $GLOBALS['db_art_cache'] = [];
+    while ($r = sql_fetch($res)) {
+        $t = sinon($r['titre'], _T('ecrire:info_sans_titre'));
+        $GLOBALS['db_art_cache'][$r['id_parent']][$r['id_rubrique']] = supprimer_numero(typo($t));
+    }
+
+    $t = [$last ?: time(), $GLOBALS['db_art_cache']];
+    ecrire_fichier(_CACHE_RUBRIQUES, serialize($t));
+
+    return true;
 }

Spacing +13 added lines, -13 removed lines

@@ -38,12 +38,12 @@ 
 	// on renvoi un 401 qui fait echouer la requete ajax silencieusement
 	if (!autoriser('ecrire')) {
 		$retour =
-		'<ul class="deroulant__sous-menu" data-profondeur="1">' .
-			'<li class="deroulant__item deroulant__item_plan plan_site" data-profondeur="1">' .
-				'<a class="deroulant__lien" href="' . generer_url_ecrire('accueil') . '" data-profondeur="1">' .
-					'<span class="libelle">' . _T('public:lien_connecter') . '</span>' .
-				'</a>' .
-			'</li>' .
+		'<ul class="deroulant__sous-menu" data-profondeur="1">'.
+			'<li class="deroulant__item deroulant__item_plan plan_site" data-profondeur="1">'.
+				'<a class="deroulant__lien" href="'.generer_url_ecrire('accueil').'" data-profondeur="1">'.
+					'<span class="libelle">'._T('public:lien_connecter').'</span>'.
+				'</a>'.
+			'</li>'.
 		'</ul>';
 		include_spip('inc/actions');
 		ajax_retour($retour);
@@ -51,7 +51,7 @@ 
 	}
 
 	if ($date = (int) _request('date')) {
-		header('Last-Modified: ' . gmdate('D, d M Y H:i:s', $date) . ' GMT');
+		header('Last-Modified: '.gmdate('D, d M Y H:i:s', $date).' GMT');
 	}
 
 	$r = gen_liste_rubriques();
@@ -61,7 +61,7 @@ 
 		&& !strstr((string) $_SERVER['SERVER_SOFTWARE'], 'IIS/')
 	) {
 		include_spip('inc/headers');
-		header('Content-Type: text/html; charset=' . $GLOBALS['meta']['charset']);
+		header('Content-Type: text/html; charset='.$GLOBALS['meta']['charset']);
 		http_response_code(304);
 		exit;
 	} else {
@@ -84,8 +84,8 @@ 
  **/
 function menu_rubriques($complet = true) {
 	$ret = '<li class="deroulant__item deroulant__item_tout toutsite" data-profondeur="1">'
-		. '<a class="deroulant__lien" href="' . generer_url_ecrire('plan') . '" data-profondeur="1">'
-		. '<span class="libelle">' . _T('info_tout_site') . '</span>'
+		. '<a class="deroulant__lien" href="'.generer_url_ecrire('plan').'" data-profondeur="1">'
+		. '<span class="libelle">'._T('info_tout_site').'</span>'
 		. '</a>'
 		. '</li>';
 
@@ -144,8 +144,8 @@ 
 	static $zmax = 6;
 	$profondeur_next = $profondeur + 1;
 
-	$nav = '<a class="deroulant__lien" href="' . generer_objet_url($id_rubrique, 'rubrique', '', '', false) . "\" data-profondeur=\"$profondeur\">"
-		. '<span class="libelle">' . supprimer_tags(preg_replace(',[\x00-\x1f]+,', ' ', $titre_rubrique)) . '</span>'
+	$nav = '<a class="deroulant__lien" href="'.generer_objet_url($id_rubrique, 'rubrique', '', '', false)."\" data-profondeur=\"$profondeur\">"
+		. '<span class="libelle">'.supprimer_tags(preg_replace(',[\x00-\x1f]+,', ' ', $titre_rubrique)).'</span>'
 		. "</a>\n";
 
 	// Limiter volontairement le nombre de sous-menus
@@ -181,7 +181,7 @@ 
 	}
 	$zmax++;
 
-	return $ret . "</ul></li>\n";
+	return $ret."</ul></li>\n";
 }
 
 

ecrire/action/auth.php

Indentation +22 added lines, -22 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -24,25 +24,25 @@ 
  */
 function action_auth_dist() {
 
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$arg = $securiser_action();
-
-	if (!preg_match(',^(\w+)[/](.+)$,', (string) $arg, $r)) {
-		spip_log("action_auth_dist $arg pas compris");
-	} else {
-		$auth_methode = $r[1];
-		$login = $r[2];
-		include_spip('inc/auth');
-		$res = auth_terminer_identifier_login($auth_methode, $login);
-
-		if (is_string($res)) { // Erreur
-			$redirect = _request('redirect');
-			$redirect = parametre_url($redirect, 'var_erreur', $res, '&');
-			include_spip('inc/headers');
-			redirige_par_entete($redirect);
-		}
-
-		// sinon on loge l'auteur identifie, et on finit (redirection automatique)
-		auth_loger($res);
-	}
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $arg = $securiser_action();
+
+    if (!preg_match(',^(\w+)[/](.+)$,', (string) $arg, $r)) {
+        spip_log("action_auth_dist $arg pas compris");
+    } else {
+        $auth_methode = $r[1];
+        $login = $r[2];
+        include_spip('inc/auth');
+        $res = auth_terminer_identifier_login($auth_methode, $login);
+
+        if (is_string($res)) { // Erreur
+            $redirect = _request('redirect');
+            $redirect = parametre_url($redirect, 'var_erreur', $res, '&');
+            include_spip('inc/headers');
+            redirige_par_entete($redirect);
+        }
+
+        // sinon on loge l'auteur identifie, et on finit (redirection automatique)
+        auth_loger($res);
+    }
 }