spip / SPIP

prive/formulaires/login.php

Indentation +249 added lines, -249 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('base/abstract_sql');
@@ -31,11 +31,11 @@ 
  *     true si espace privé, false sinon.
  **/
 function is_url_prive($cible) {
-	include_spip('inc/filtres_mini');
-	$path = parse_url(tester_url_absolue($cible) ? $cible : url_absolue($cible));
-	$path = ($path['path'] ?? '');
+    include_spip('inc/filtres_mini');
+    $path = parse_url(tester_url_absolue($cible) ? $cible : url_absolue($cible));
+    $path = ($path['path'] ?? '');
 
-	return strncmp(substr($path, -strlen(_DIR_RESTREINT_ABS)), _DIR_RESTREINT_ABS, strlen(_DIR_RESTREINT_ABS)) == 0;
+    return strncmp(substr($path, -strlen(_DIR_RESTREINT_ABS)), _DIR_RESTREINT_ABS, strlen(_DIR_RESTREINT_ABS)) == 0;
 }
 
 /**
@@ -60,111 +60,111 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_login_charger_dist($cible = '', $options = [], $deprecated = null) {
-	$erreur = _request('var_erreur');
-
-	if (!is_array($options)) {
-		$options = [
-			'login' => $options,
-			'prive' => $deprecated
-		];
-	}
-
-	$login = (empty($options['login']) ? '' : $options['login']);
-	$prive = (empty($options['prive']) ? null : $options['prive']);
-	// conserver le fonctionnement historique
-	if (!isset($options['autofocus'])) {
-		$options['autofocus'] = true;
-	}
-
-	if (!$login) {
-		$login = strval(_request('var_login'));
-	}
-	// si on est deja identifie
-	if (!$login and isset($GLOBALS['visiteur_session']['email'])) {
-		$login = $GLOBALS['visiteur_session']['email'];
-	}
-	if (!$login and isset($GLOBALS['visiteur_session']['login'])) {
-		$login = $GLOBALS['visiteur_session']['login'];
-	}
-	// ou si on a un cookie admin
-	if (!$login) {
-		if (
-			isset($_COOKIE['spip_admin'])
-			and preg_match(',^@(.*)$,', $_COOKIE['spip_admin'], $regs)
-		) {
-			$login = $regs[1];
-		}
-	}
-
-	$lang = $GLOBALS['spip_lang'];
-	include_spip('inc/auth');
-	$row = auth_informer_login($login);
-
-	// retablir la langue de l'URL si forcee (on ignore la langue de l'auteur dans ce cas)
-	if (_request('lang') === $lang and $GLOBALS['spip_lang'] !== $lang) {
-		changer_langue($lang);
-	}
-
-	// Construire l'environnement du squelette
-	// Ne pas proposer de "rester connecte quelques jours"
-	// si la duree de l'alea est inferieure a 12 h (valeur par defaut)
-
-	$valeurs = [
-		'var_login' => $login,
-		'editable' => !$row,
-		'cnx' => $row['cnx'] ?? '0',
-		'auth_http' => login_auth_http(),
-		'rester_connecte' => ((_RENOUVELLE_ALEA < 12 * 3600) ? '' : ' '),
-		'_logo' => $row['logo'] ?? '',
-		'_alea_actuel' => $row['alea_actuel'] ?? '',
-		'_alea_futur' => $row['alea_futur'] ?? '',
-		'_pipeline' => 'affiche_formulaire_login', // faire passer le formulaire dans un pipe dedie pour les methodes auth
-		'_autofocus' => ($options['autofocus'] and $options['autofocus'] !== 'non') ? ' ' : '',
-	];
-
-	if ($erreur or !isset($GLOBALS['visiteur_session']['id_auteur']) or !$GLOBALS['visiteur_session']['id_auteur']) {
-		$valeurs['editable'] = true;
-	}
-
-	if (is_null($prive) ? is_url_prive($cible) : $prive) {
-		include_spip('inc/autoriser');
-		$loge = autoriser('ecrire');
-	} else {
-		$loge = (isset($GLOBALS['visiteur_session']['auth']) and $GLOBALS['visiteur_session']['auth'] != '');
-	}
-
-	// Si on est connecte, appeler traiter()
-	// et lancer la redirection si besoin
-	if (!$valeurs['editable'] and $loge and _request('formulaire_action') !== 'login') {
-		$traiter = charger_fonction('traiter', 'formulaires/login');
-		$res = $traiter($cible, $login, $prive);
-		$valeurs = array_merge($valeurs, $res);
-
-		if (isset($res['redirect']) and $res['redirect']) {
-			include_spip('inc/headers');
-			# preparer un lien pour quand redirige_formulaire ne fonctionne pas
-			$m = redirige_formulaire($res['redirect']);
-			$valeurs['_deja_loge'] = inserer_attribut(
-				'<a>' . _T('login_par_ici') . "</a>$m",
-				'href',
-				$res['redirect']
-			);
-		}
-	}
-	// en cas d'echec de cookie, inc_auth a renvoye vers le script de
-	// pose de cookie ; s'il n'est pas la, c'est echec cookie
-	// s'il est la, c'est probablement un bookmark sur bonjour=oui,
-	// et pas un echec cookie.
-	if ($erreur == 'cookie') {
-		$valeurs['echec_cookie'] = ' ';
-	} elseif ($erreur) {
-		// une erreur d'un SSO indique dans la redirection vers ici
-		// mais il faut se proteger de toute tentative d'injection malveilante
-		include_spip('inc/filtres');
-		$valeurs['message_erreur'] = textebrut($erreur);
-	}
-
-	return $valeurs;
+    $erreur = _request('var_erreur');
+
+    if (!is_array($options)) {
+        $options = [
+            'login' => $options,
+            'prive' => $deprecated
+        ];
+    }
+
+    $login = (empty($options['login']) ? '' : $options['login']);
+    $prive = (empty($options['prive']) ? null : $options['prive']);
+    // conserver le fonctionnement historique
+    if (!isset($options['autofocus'])) {
+        $options['autofocus'] = true;
+    }
+
+    if (!$login) {
+        $login = strval(_request('var_login'));
+    }
+    // si on est deja identifie
+    if (!$login and isset($GLOBALS['visiteur_session']['email'])) {
+        $login = $GLOBALS['visiteur_session']['email'];
+    }
+    if (!$login and isset($GLOBALS['visiteur_session']['login'])) {
+        $login = $GLOBALS['visiteur_session']['login'];
+    }
+    // ou si on a un cookie admin
+    if (!$login) {
+        if (
+            isset($_COOKIE['spip_admin'])
+            and preg_match(',^@(.*)$,', $_COOKIE['spip_admin'], $regs)
+        ) {
+            $login = $regs[1];
+        }
+    }
+
+    $lang = $GLOBALS['spip_lang'];
+    include_spip('inc/auth');
+    $row = auth_informer_login($login);
+
+    // retablir la langue de l'URL si forcee (on ignore la langue de l'auteur dans ce cas)
+    if (_request('lang') === $lang and $GLOBALS['spip_lang'] !== $lang) {
+        changer_langue($lang);
+    }
+
+    // Construire l'environnement du squelette
+    // Ne pas proposer de "rester connecte quelques jours"
+    // si la duree de l'alea est inferieure a 12 h (valeur par defaut)
+
+    $valeurs = [
+        'var_login' => $login,
+        'editable' => !$row,
+        'cnx' => $row['cnx'] ?? '0',
+        'auth_http' => login_auth_http(),
+        'rester_connecte' => ((_RENOUVELLE_ALEA < 12 * 3600) ? '' : ' '),
+        '_logo' => $row['logo'] ?? '',
+        '_alea_actuel' => $row['alea_actuel'] ?? '',
+        '_alea_futur' => $row['alea_futur'] ?? '',
+        '_pipeline' => 'affiche_formulaire_login', // faire passer le formulaire dans un pipe dedie pour les methodes auth
+        '_autofocus' => ($options['autofocus'] and $options['autofocus'] !== 'non') ? ' ' : '',
+    ];
+
+    if ($erreur or !isset($GLOBALS['visiteur_session']['id_auteur']) or !$GLOBALS['visiteur_session']['id_auteur']) {
+        $valeurs['editable'] = true;
+    }
+
+    if (is_null($prive) ? is_url_prive($cible) : $prive) {
+        include_spip('inc/autoriser');
+        $loge = autoriser('ecrire');
+    } else {
+        $loge = (isset($GLOBALS['visiteur_session']['auth']) and $GLOBALS['visiteur_session']['auth'] != '');
+    }
+
+    // Si on est connecte, appeler traiter()
+    // et lancer la redirection si besoin
+    if (!$valeurs['editable'] and $loge and _request('formulaire_action') !== 'login') {
+        $traiter = charger_fonction('traiter', 'formulaires/login');
+        $res = $traiter($cible, $login, $prive);
+        $valeurs = array_merge($valeurs, $res);
+
+        if (isset($res['redirect']) and $res['redirect']) {
+            include_spip('inc/headers');
+            # preparer un lien pour quand redirige_formulaire ne fonctionne pas
+            $m = redirige_formulaire($res['redirect']);
+            $valeurs['_deja_loge'] = inserer_attribut(
+                '<a>' . _T('login_par_ici') . "</a>$m",
+                'href',
+                $res['redirect']
+            );
+        }
+    }
+    // en cas d'echec de cookie, inc_auth a renvoye vers le script de
+    // pose de cookie ; s'il n'est pas la, c'est echec cookie
+    // s'il est la, c'est probablement un bookmark sur bonjour=oui,
+    // et pas un echec cookie.
+    if ($erreur == 'cookie') {
+        $valeurs['echec_cookie'] = ' ';
+    } elseif ($erreur) {
+        // une erreur d'un SSO indique dans la redirection vers ici
+        // mais il faut se proteger de toute tentative d'injection malveilante
+        include_spip('inc/filtres');
+        $valeurs['message_erreur'] = textebrut($erreur);
+    }
+
+    return $valeurs;
 }
 
 
@@ -179,20 +179,20 @@ 
  *     - chaîne vide sinon.
  **/
 function login_auth_http() {
-	if (
-		!$GLOBALS['ignore_auth_http']
-		and _request('var_erreur') == 'cookie'
-		and (!isset($_COOKIE['spip_session']) or $_COOKIE['spip_session'] != 'test_echec_cookie')
-		and (($GLOBALS['flag_sapi_name'] and preg_match(',apache,i', @php_sapi_name()))
-			or preg_match(',^Apache.* PHP,', $_SERVER['SERVER_SOFTWARE']))
-		// Attention dans le cas 'intranet' la proposition de se loger
-		// par auth_http peut conduire a l'echec.
-		and !(isset($_SERVER['PHP_AUTH_USER']) and isset($_SERVER['PHP_AUTH_PW']))
-	) {
-		return generer_url_action('cookie', '', false, true);
-	} else {
-		return '';
-	}
+    if (
+        !$GLOBALS['ignore_auth_http']
+        and _request('var_erreur') == 'cookie'
+        and (!isset($_COOKIE['spip_session']) or $_COOKIE['spip_session'] != 'test_echec_cookie')
+        and (($GLOBALS['flag_sapi_name'] and preg_match(',apache,i', @php_sapi_name()))
+            or preg_match(',^Apache.* PHP,', $_SERVER['SERVER_SOFTWARE']))
+        // Attention dans le cas 'intranet' la proposition de se loger
+        // par auth_http peut conduire a l'echec.
+        and !(isset($_SERVER['PHP_AUTH_USER']) and isset($_SERVER['PHP_AUTH_PW']))
+    ) {
+        return generer_url_action('cookie', '', false, true);
+    } else {
+        return '';
+    }
 }
 
 
@@ -218,65 +218,65 @@ 
  **/
 function formulaires_login_verifier_dist($cible = '', $options = [], $deprecated = null) {
 
-	$erreurs = [];
-	if (!is_array($options)) {
-		$options = [
-			'login' => $options,
-			'prive' => $deprecated
-		];
-	}
-
-	$prive = (empty($options['prive']) ? null : $options['prive']);
-
-	$session_login = _request('var_login');
-	$session_password = _request('password');
-	$session_remember = _request('session_remember');
-
-	if (!$session_login) {
-		# pas de login saisi !
-		return ['var_login' => _T('info_obligatoire')];
-	}
-
-	// appeler auth_identifier_login qui va :
-	// - renvoyer un string si echec (message d'erreur)
-	// - un array decrivant l'auteur identifie si possible
-	// - rediriger vers un SSO qui renverra in fine sur action/auth qui finira l'authentification
-	include_spip('inc/auth');
-	$auteur = auth_identifier_login($session_login, $session_password);
-	// on arrive ici si on ne s'est pas identifie avec un SSO
-	if (!is_array($auteur)) {
-		$erreurs = [];
-		if (is_string($auteur) and strlen($auteur)) {
-			$erreurs['var_login'] = $auteur;
-		}
-		include_spip('inc/cookie');
-		spip_setcookie('spip_admin', '', time() - 3600);
-		if (strlen($session_password)) {
-			$erreurs['password'] = _T('login_erreur_pass');
-		} else {
-			// sinon c'est un login en deux passe old style (ou js en panne)
-			// pas de message d'erreur
-			$erreurs['password'] = ' ';
-			$erreurs['message_erreur'] = '';
-		}
-
-		return
-			$erreurs;
-	}
-	// on a ete authentifie, construire la session
-	// en gerant la duree demandee pour son cookie
-	if ($session_remember !== null) {
-		$auteur['cookie'] = $session_remember;
-	}
-	// si la connexion est refusee on renvoi un message erreur de mot de passe
-	// car en donnant plus de detail on renseignerait un assaillant sur l'existence d'un compte
-	if (auth_loger($auteur) === false) {
-		$erreurs['message_erreur'] = _T('login_erreur_pass');
-		return $erreurs;
-	}
-
-	return (is_null($prive) ? is_url_prive($cible) : $prive)
-		? login_autoriser() : [];
+    $erreurs = [];
+    if (!is_array($options)) {
+        $options = [
+            'login' => $options,
+            'prive' => $deprecated
+        ];
+    }
+
+    $prive = (empty($options['prive']) ? null : $options['prive']);
+
+    $session_login = _request('var_login');
+    $session_password = _request('password');
+    $session_remember = _request('session_remember');
+
+    if (!$session_login) {
+        # pas de login saisi !
+        return ['var_login' => _T('info_obligatoire')];
+    }
+
+    // appeler auth_identifier_login qui va :
+    // - renvoyer un string si echec (message d'erreur)
+    // - un array decrivant l'auteur identifie si possible
+    // - rediriger vers un SSO qui renverra in fine sur action/auth qui finira l'authentification
+    include_spip('inc/auth');
+    $auteur = auth_identifier_login($session_login, $session_password);
+    // on arrive ici si on ne s'est pas identifie avec un SSO
+    if (!is_array($auteur)) {
+        $erreurs = [];
+        if (is_string($auteur) and strlen($auteur)) {
+            $erreurs['var_login'] = $auteur;
+        }
+        include_spip('inc/cookie');
+        spip_setcookie('spip_admin', '', time() - 3600);
+        if (strlen($session_password)) {
+            $erreurs['password'] = _T('login_erreur_pass');
+        } else {
+            // sinon c'est un login en deux passe old style (ou js en panne)
+            // pas de message d'erreur
+            $erreurs['password'] = ' ';
+            $erreurs['message_erreur'] = '';
+        }
+
+        return
+            $erreurs;
+    }
+    // on a ete authentifie, construire la session
+    // en gerant la duree demandee pour son cookie
+    if ($session_remember !== null) {
+        $auteur['cookie'] = $session_remember;
+    }
+    // si la connexion est refusee on renvoi un message erreur de mot de passe
+    // car en donnant plus de detail on renseignerait un assaillant sur l'existence d'un compte
+    if (auth_loger($auteur) === false) {
+        $erreurs['message_erreur'] = _T('login_erreur_pass');
+        return $erreurs;
+    }
+
+    return (is_null($prive) ? is_url_prive($cible) : $prive)
+        ? login_autoriser() : [];
 }
 
 /**
@@ -291,21 +291,21 @@ 
  *     - tableau vide sinon.
  **/
 function login_autoriser() {
-	include_spip('inc/autoriser');
-	if (!autoriser('ecrire')) {
-		$h = generer_url_action('logout', 'logout=prive&url=' . urlencode(self()));
-
-		return [
-			'message_erreur' => '<h1>'
-				. _T('avis_erreur_visiteur')
-				. '</h1><p>'
-				. _T('texte_erreur_visiteur')
-				. "</p><p class='retour'>[<a href='$h'>"
-				. _T('icone_deconnecter') . '</a>]</p>'
-		];
-	}
-
-	return [];
+    include_spip('inc/autoriser');
+    if (!autoriser('ecrire')) {
+        $h = generer_url_action('logout', 'logout=prive&url=' . urlencode(self()));
+
+        return [
+            'message_erreur' => '<h1>'
+                . _T('avis_erreur_visiteur')
+                . '</h1><p>'
+                . _T('texte_erreur_visiteur')
+                . "</p><p class='retour'>[<a href='$h'>"
+                . _T('icone_deconnecter') . '</a>]</p>'
+        ];
+    }
+
+    return [];
 }
 
 /**
@@ -326,55 +326,55 @@ 
  *     Retours du traitement
  **/
 function formulaires_login_traiter_dist($cible = '', $options = [], $deprecated = null) {
-	$res = [];
-
-	if (!is_array($options)) {
-		$options = [
-			'login' => $options,
-			'prive' => $deprecated
-		];
-	}
-
-	$login = (empty($options['login']) ? '' : $options['login']);
-	$prive = (empty($options['prive']) ? null : $options['prive']);
-
-	// Si on se connecte dans l'espace prive,
-	// ajouter "bonjour" (repere a peu pres les cookies desactives)
-	if (is_null($prive) ? is_url_prive($cible) : $prive) {
-		$cible = parametre_url($cible, 'bonjour', 'oui', '&');
-	}
-	if ($cible == '@page_auteur') {
-		$cible = generer_url_entite($GLOBALS['auteur_session']['id_auteur'], 'auteur');
-	}
-
-	if ($cible) {
-		$cible = parametre_url($cible, 'var_login', '', '&');
-
-		// transformer la cible absolue en cible relative
-		// pour pas echouer quand la meta adresse_site est foireuse
-		if (strncmp($cible, $u = url_de_base(), strlen($u)) == 0) {
-			$cible = './' . substr($cible, strlen($u));
-		} elseif (tester_url_absolue($cible) and !defined('_AUTORISER_LOGIN_ABS_REDIRECT')) {
-			// si c'est une url absolue, refuser la redirection
-			// sauf si cette securite est levee volontairement par le webmestre
-			$cible = '';
-		}
-	}
-
-	// Si on est connecte, envoyer vers la destination
-	if ($cible and ($cible != self('&')) and ($cible != self())) {
-		$res['redirect'] = $cible;
-		$res['message_ok'] = inserer_attribut(
-			'<a>' . _T('login_par_ici') . '</a>',
-			'href',
-			$cible
-		);
-	}
-
-	// avant de rediriger il faut mettre a jour les sessions sur le disque si on a charge une session
-	if (function_exists('terminer_actualiser_sessions')) {
-		terminer_actualiser_sessions();
-	}
-
-	return $res;
+    $res = [];
+
+    if (!is_array($options)) {
+        $options = [
+            'login' => $options,
+            'prive' => $deprecated
+        ];
+    }
+
+    $login = (empty($options['login']) ? '' : $options['login']);
+    $prive = (empty($options['prive']) ? null : $options['prive']);
+
+    // Si on se connecte dans l'espace prive,
+    // ajouter "bonjour" (repere a peu pres les cookies desactives)
+    if (is_null($prive) ? is_url_prive($cible) : $prive) {
+        $cible = parametre_url($cible, 'bonjour', 'oui', '&');
+    }
+    if ($cible == '@page_auteur') {
+        $cible = generer_url_entite($GLOBALS['auteur_session']['id_auteur'], 'auteur');
+    }
+
+    if ($cible) {
+        $cible = parametre_url($cible, 'var_login', '', '&');
+
+        // transformer la cible absolue en cible relative
+        // pour pas echouer quand la meta adresse_site est foireuse
+        if (strncmp($cible, $u = url_de_base(), strlen($u)) == 0) {
+            $cible = './' . substr($cible, strlen($u));
+        } elseif (tester_url_absolue($cible) and !defined('_AUTORISER_LOGIN_ABS_REDIRECT')) {
+            // si c'est une url absolue, refuser la redirection
+            // sauf si cette securite est levee volontairement par le webmestre
+            $cible = '';
+        }
+    }
+
+    // Si on est connecte, envoyer vers la destination
+    if ($cible and ($cible != self('&')) and ($cible != self())) {
+        $res['redirect'] = $cible;
+        $res['message_ok'] = inserer_attribut(
+            '<a>' . _T('login_par_ici') . '</a>',
+            'href',
+            $cible
+        );
+    }
+
+    // avant de rediriger il faut mettre a jour les sessions sur le disque si on a charge une session
+    if (function_exists('terminer_actualiser_sessions')) {
+        terminer_actualiser_sessions();
+    }
+
+    return $res;
 }

prive/formulaires/instituer_objet.php

Indentation +108 added lines, -108 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/editer');
@@ -32,17 +32,17 @@ 
  * @return array
  */
 function lister_statuts_proposes($desc, $publiable = true) {
-	if (!isset($desc['statut_textes_instituer'])) {
-		return false;
-	}
+    if (!isset($desc['statut_textes_instituer'])) {
+        return false;
+    }
 
-	$l = $desc['statut_textes_instituer'];
-	if (!$publiable) {
-		unset($l['publie']);
-		unset($l['refuse']);
-	}
+    $l = $desc['statut_textes_instituer'];
+    if (!$publiable) {
+        unset($l['publie']);
+        unset($l['refuse']);
+    }
 
-	return $l;
+    return $l;
 }
 
 /**
@@ -64,54 +64,54 @@ 
  *     Environnement du formulaire ou false si aucun affichage à faire.
  */
 function formulaires_instituer_objet_charger_dist($objet, $id_objet, $retour = '', $editable = true) {
-	$editable = ($editable ? true : false);
-
-	$table = table_objet_sql($objet);
-	$desc = lister_tables_objets_sql($table);
-
-	if (!isset($desc['statut_textes_instituer'])) {
-		return false;
-	}
-
-	if (!autoriser('modifier', $objet, $id_objet)) {
-		$editable = false;
-	}
-
-	// charger le contenu de l'objet
-	// dont son champ statut
-	$v = formulaires_editer_objet_charger($objet, $id_objet, 0, 0, '', '');
-
-	$publiable = true;
-	$statuts = lister_statuts_proposes($desc);
-	// tester si on a le droit de publier, si un statut publie existe
-	if (isset($statuts['publie'])) {
-		if (!autoriser('instituer', $objet, $id_objet, null, ['statut' => 'publie'])) {
-			if ($v['statut'] == 'publie') {
-				$editable = false;
-			} else {
-				$publiable = false;
-			}
-		}
-	}
-	$statuts = lister_statuts_proposes($desc, $editable ? $publiable : true);
-	if (count($statuts) == 1 and isset($statuts[$v['statut']])) {
-		$editable = false;
-	}
-
-	$valeurs = [
-		'editable' => $editable,
-		'statut' => $v['statut'],
-		'_objet' => $objet,
-		'_id_objet' => $id_objet,
-		'_statuts' => $statuts,
-		'_publiable' => $publiable,
-		'_label' => $desc['texte_changer_statut'] ?? 'texte_article_statut',
-		'_aide' => $desc['aide_changer_statut'] ?? '',
-		'_hidden' => "<input type='hidden' name='statut_old' value='" . $v['statut'] . "' />",
-	];
-
-	#if (!count($valeurs['statuts']))
-	return $valeurs;
+    $editable = ($editable ? true : false);
+
+    $table = table_objet_sql($objet);
+    $desc = lister_tables_objets_sql($table);
+
+    if (!isset($desc['statut_textes_instituer'])) {
+        return false;
+    }
+
+    if (!autoriser('modifier', $objet, $id_objet)) {
+        $editable = false;
+    }
+
+    // charger le contenu de l'objet
+    // dont son champ statut
+    $v = formulaires_editer_objet_charger($objet, $id_objet, 0, 0, '', '');
+
+    $publiable = true;
+    $statuts = lister_statuts_proposes($desc);
+    // tester si on a le droit de publier, si un statut publie existe
+    if (isset($statuts['publie'])) {
+        if (!autoriser('instituer', $objet, $id_objet, null, ['statut' => 'publie'])) {
+            if ($v['statut'] == 'publie') {
+                $editable = false;
+            } else {
+                $publiable = false;
+            }
+        }
+    }
+    $statuts = lister_statuts_proposes($desc, $editable ? $publiable : true);
+    if (count($statuts) == 1 and isset($statuts[$v['statut']])) {
+        $editable = false;
+    }
+
+    $valeurs = [
+        'editable' => $editable,
+        'statut' => $v['statut'],
+        '_objet' => $objet,
+        '_id_objet' => $id_objet,
+        '_statuts' => $statuts,
+        '_publiable' => $publiable,
+        '_label' => $desc['texte_changer_statut'] ?? 'texte_article_statut',
+        '_aide' => $desc['aide_changer_statut'] ?? '',
+        '_hidden' => "<input type='hidden' name='statut_old' value='" . $v['statut'] . "' />",
+    ];
+
+    #if (!count($valeurs['statuts']))
+    return $valeurs;
 }
 
 /**
@@ -131,35 +131,35 @@ 
  *     Tableau des erreurs
  */
 function formulaires_instituer_objet_verifier_dist($objet, $id_objet, $retour = '', $editable = true) {
-	$erreurs = [];
-	// charger le contenu de l'objet
-	// dont son champ statut
-	$v = formulaires_editer_objet_charger($objet, $id_objet, 0, 0, '', '');
-
-	if ($v['statut'] !== _request('statut_old')) {
-		$erreurs['statut'] = _T('instituer_erreur_statut_a_change');
-	} else {
-		$table = table_objet_sql($objet);
-		$desc = lister_tables_objets_sql($table);
-
-		$publiable = true;
-		if (
-			isset($v['id_rubrique'])
-			and !autoriser('publierdans', 'rubrique', $v['id_rubrique'])
-		) {
-			$publiable = false;
-		}
-		$l = lister_statuts_proposes($desc, $publiable);
-		$statut = _request('statut');
-		if (
-			!isset($l[$statut])
-			or !autoriser('instituer', $objet, $id_objet, '', ['statut' => $statut])
-		) {
-			$erreurs['statut'] = _T('instituer_erreur_statut_non_autorise');
-		}
-	}
-
-	return $erreurs;
+    $erreurs = [];
+    // charger le contenu de l'objet
+    // dont son champ statut
+    $v = formulaires_editer_objet_charger($objet, $id_objet, 0, 0, '', '');
+
+    if ($v['statut'] !== _request('statut_old')) {
+        $erreurs['statut'] = _T('instituer_erreur_statut_a_change');
+    } else {
+        $table = table_objet_sql($objet);
+        $desc = lister_tables_objets_sql($table);
+
+        $publiable = true;
+        if (
+            isset($v['id_rubrique'])
+            and !autoriser('publierdans', 'rubrique', $v['id_rubrique'])
+        ) {
+            $publiable = false;
+        }
+        $l = lister_statuts_proposes($desc, $publiable);
+        $statut = _request('statut');
+        if (
+            !isset($l[$statut])
+            or !autoriser('instituer', $objet, $id_objet, '', ['statut' => $statut])
+        ) {
+            $erreurs['statut'] = _T('instituer_erreur_statut_non_autorise');
+        }
+    }
+
+    return $erreurs;
 }
 
 /**
@@ -178,25 +178,25 @@ 
  */
 function formulaires_instituer_objet_traiter_dist($objet, $id_objet, $retour = '', $editable = true) {
 
-	$c = ['statut' => _request('statut')];
-	// si on a envoye une 'date_posterieure', l'enregistrer
-	// todo dans le HTML
-	if ($d = _request('date_posterieure')) {
-		$c['date'] = $d;
-	}
-
-
-	include_spip('action/editer_objet');
-	if ($err = objet_instituer($objet, $id_objet, $c)) {
-		$res = ['message_erreur' => $err];
-	} else {
-		$res = ['message_ok' => _T('info_modification_enregistree')];
-		if ($retour) {
-			$res['redirect'] = $retour;
-		}
-		set_request('statut');
-		set_request('date_posterieure');
-	}
-
-	return $res;
+    $c = ['statut' => _request('statut')];
+    // si on a envoye une 'date_posterieure', l'enregistrer
+    // todo dans le HTML
+    if ($d = _request('date_posterieure')) {
+        $c['date'] = $d;
+    }
+
+
+    include_spip('action/editer_objet');
+    if ($err = objet_instituer($objet, $id_objet, $c)) {
+        $res = ['message_erreur' => $err];
+    } else {
+        $res = ['message_ok' => _T('info_modification_enregistree')];
+        if ($retour) {
+            $res['redirect'] = $retour;
+        }
+        set_request('statut');
+        set_request('date_posterieure');
+    }
+
+    return $res;
 }

prive/formulaires/editer_liens.php

Indentation +352 added lines, -352 removed lines

@@ -16,7 +16,7 @@ 
  * @package SPIP\Core\Formulaires
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -36,25 +36,25 @@ 
  *   ($table_source,$objet,$id_objet,$objet_lien)
  */
 function determine_source_lien_objet($a, $b, $c) {
-	$table_source = $objet_lien = $objet = $id_objet = null;
-	// auteurs, article, 23 :
-	// associer des auteurs à l'article 23, sur la table pivot spip_auteurs_liens
-	if (is_numeric($c) and !is_numeric($b)) {
-		$table_source = table_objet($a);
-		$objet_lien = objet_type($a);
-		$objet = objet_type($b);
-		$id_objet = $c;
-	}
-	// article, 23, auteurs
-	// associer des auteurs à l'article 23, sur la table pivot spip_articles_liens
-	if (is_numeric($b) and !is_numeric($c)) {
-		$table_source = table_objet($c);
-		$objet_lien = objet_type($a);
-		$objet = objet_type($a);
-		$id_objet = $b;
-	}
-
-	return [$table_source, $objet, $id_objet, $objet_lien];
+    $table_source = $objet_lien = $objet = $id_objet = null;
+    // auteurs, article, 23 :
+    // associer des auteurs à l'article 23, sur la table pivot spip_auteurs_liens
+    if (is_numeric($c) and !is_numeric($b)) {
+        $table_source = table_objet($a);
+        $objet_lien = objet_type($a);
+        $objet = objet_type($b);
+        $id_objet = $c;
+    }
+    // article, 23, auteurs
+    // associer des auteurs à l'article 23, sur la table pivot spip_articles_liens
+    if (is_numeric($b) and !is_numeric($c)) {
+        $table_source = table_objet($c);
+        $objet_lien = objet_type($a);
+        $objet = objet_type($a);
+        $id_objet = $b;
+    }
+
+    return [$table_source, $objet, $id_objet, $objet_lien];
 }
 
 /**
@@ -80,93 +80,93 @@ 
  */
 function formulaires_editer_liens_charger_dist($a, $b, $c, $options = []) {
 
-	// compat avec ancienne signature ou le 4eme argument est $editable
-	if (!is_array($options)) {
-		$options = ['editable' => $options];
-	} elseif (!isset($options['editable'])) {
-		$options['editable'] = true;
-	}
-
-	$editable = $options['editable'];
-
-	[$table_source, $objet, $id_objet, $objet_lien] = determine_source_lien_objet($a, $b, $c);
-	if (!$table_source or !$objet or !$objet_lien or !$id_objet) {
-		return false;
-	}
-
-	$objet_source = objet_type($table_source);
-	$table_sql_source = table_objet_sql($objet_source);
-
-	// verifier existence de la table xxx_liens
-	include_spip('action/editer_liens');
-	if (!objet_associable($objet_lien)) {
-		return false;
-	}
-
-	// L'éditabilité :) est définie par un test permanent (par exemple "associermots") ET le 4ème argument
-	include_spip('inc/autoriser');
-	$editable = ($editable and autoriser('associer' . $table_source, $objet, $id_objet)
-		and autoriser('modifier', $objet, $id_objet));
-
-	if (
-		!$editable and !count(objet_trouver_liens(
-			[$objet_lien => '*'],
-			[($objet_lien == $objet_source ? $objet : $objet_source) => $id_objet]
-		))
-	) {
-		return false;
-	}
-
-	// squelettes de vue et de d'association
-	// ils sont différents si des rôles sont définis.
-	$skel_vue = $table_source . '_lies';
-	$skel_ajout = $table_source . '_associer';
-
-	// description des roles
-	include_spip('inc/roles');
-	if ($roles = roles_presents($objet_source, $objet)) {
-		// on demande de nouveaux squelettes en conséquence
-		$skel_vue = $table_source . '_roles_lies';
-		$skel_ajout = $table_source . '_roles_associer';
-	}
-
-	$oups = _request('_oups') ?? '';
-	if ($oups) {
-		if (unserialize(base64_decode($oups))) {
-			// on est bon, rien a faire
-		} elseif (unserialize($oups)) {
-			// il faut encoder
-			$oups = base64_encode($oups);
-		} else {
-			$oups = '';
-		}
-	}
-	$valeurs = [
-		'id' => "$table_source-$objet-$id_objet-$objet_lien", // identifiant unique pour les id du form
-		'_vue_liee' => $skel_vue,
-		'_vue_ajout' => $skel_ajout,
-		'_objet_lien' => $objet_lien,
-		'id_lien_ajoute' => _request('id_lien_ajoute'),
-		'objet' => $objet,
-		'id_objet' => $id_objet,
-		'objet_source' => $objet_source,
-		'table_source' => $table_source,
-		'recherche' => '',
-		'visible' => 0,
-		'ajouter_lien' => '',
-		'supprimer_lien' => '',
-		'qualifier_lien' => '',
-		'ordonner_lien' => '',
-		'desordonner_liens' => '',
-		'_roles' => $roles, # description des roles
-		'_oups' => $oups,
-		'editable' => $editable,
-	];
-
-	// les options non definies dans $valeurs sont passees telles quelles au formulaire html
-	$valeurs = array_merge($options, $valeurs);
-
-	return $valeurs;
+    // compat avec ancienne signature ou le 4eme argument est $editable
+    if (!is_array($options)) {
+        $options = ['editable' => $options];
+    } elseif (!isset($options['editable'])) {
+        $options['editable'] = true;
+    }
+
+    $editable = $options['editable'];
+
+    [$table_source, $objet, $id_objet, $objet_lien] = determine_source_lien_objet($a, $b, $c);
+    if (!$table_source or !$objet or !$objet_lien or !$id_objet) {
+        return false;
+    }
+
+    $objet_source = objet_type($table_source);
+    $table_sql_source = table_objet_sql($objet_source);
+
+    // verifier existence de la table xxx_liens
+    include_spip('action/editer_liens');
+    if (!objet_associable($objet_lien)) {
+        return false;
+    }
+
+    // L'éditabilité :) est définie par un test permanent (par exemple "associermots") ET le 4ème argument
+    include_spip('inc/autoriser');
+    $editable = ($editable and autoriser('associer' . $table_source, $objet, $id_objet)
+        and autoriser('modifier', $objet, $id_objet));
+
+    if (
+        !$editable and !count(objet_trouver_liens(
+            [$objet_lien => '*'],
+            [($objet_lien == $objet_source ? $objet : $objet_source) => $id_objet]
+        ))
+    ) {
+        return false;
+    }
+
+    // squelettes de vue et de d'association
+    // ils sont différents si des rôles sont définis.
+    $skel_vue = $table_source . '_lies';
+    $skel_ajout = $table_source . '_associer';
+
+    // description des roles
+    include_spip('inc/roles');
+    if ($roles = roles_presents($objet_source, $objet)) {
+        // on demande de nouveaux squelettes en conséquence
+        $skel_vue = $table_source . '_roles_lies';
+        $skel_ajout = $table_source . '_roles_associer';
+    }
+
+    $oups = _request('_oups') ?? '';
+    if ($oups) {
+        if (unserialize(base64_decode($oups))) {
+            // on est bon, rien a faire
+        } elseif (unserialize($oups)) {
+            // il faut encoder
+            $oups = base64_encode($oups);
+        } else {
+            $oups = '';
+        }
+    }
+    $valeurs = [
+        'id' => "$table_source-$objet-$id_objet-$objet_lien", // identifiant unique pour les id du form
+        '_vue_liee' => $skel_vue,
+        '_vue_ajout' => $skel_ajout,
+        '_objet_lien' => $objet_lien,
+        'id_lien_ajoute' => _request('id_lien_ajoute'),
+        'objet' => $objet,
+        'id_objet' => $id_objet,
+        'objet_source' => $objet_source,
+        'table_source' => $table_source,
+        'recherche' => '',
+        'visible' => 0,
+        'ajouter_lien' => '',
+        'supprimer_lien' => '',
+        'qualifier_lien' => '',
+        'ordonner_lien' => '',
+        'desordonner_liens' => '',
+        '_roles' => $roles, # description des roles
+        '_oups' => $oups,
+        'editable' => $editable,
+    ];
+
+    // les options non definies dans $valeurs sont passees telles quelles au formulaire html
+    $valeurs = array_merge($options, $valeurs);
+
+    return $valeurs;
 }
 
 /**
@@ -210,165 +210,165 @@ 
  * @return array
  */
 function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = []) {
-	// compat avec ancienne signature ou le 4eme argument est $editable
-	if (!is_array($options)) {
-		$options = ['editable' => $options];
-	} elseif (!isset($options['editable'])) {
-		$options['editable'] = true;
-	}
-
-	$editable = $options['editable'];
-
-	$res = ['editable' => $editable ? true : false];
-	[$table_source, $objet, $id_objet, $objet_lien] = determine_source_lien_objet($a, $b, $c);
-	if (!$table_source or !$objet or !$objet_lien) {
-		return $res;
-	}
-
-
-	if (_request('tout_voir')) {
-		set_request('recherche', '');
-	}
-
-	include_spip('inc/autoriser');
-	if (autoriser('modifier', $objet, $id_objet)) {
-		// annuler les suppressions du coup d'avant !
-		if (
-			_request('annuler_oups')
-			and $oups = _request('_oups')
-			and $oups = base64_decode($oups)
-			and $oups = unserialize($oups)
-		) {
-			if ($oups_objets = charger_fonction("editer_liens_oups_{$table_source}_{$objet}_{$objet_lien}", 'action', true)) {
-				$oups_objets($oups);
-			} else {
-				$objet_source = objet_type($table_source);
-				include_spip('action/editer_liens');
-				foreach ($oups as $oup) {
-					if ($objet_lien == $objet_source) {
-						objet_associer([$objet_source => $oup[$objet_source]], [$objet => $oup[$objet]], $oup);
-					} else {
-						objet_associer([$objet => $oup[$objet]], [$objet_source => $oup[$objet_source]], $oup);
-					}
-				}
-			}
-			# oups ne persiste que pour la derniere action, si suppression
-			set_request('_oups');
-		}
-
-		$supprimer = _request('supprimer_lien');
-		$ajouter = _request('ajouter_lien');
-		$ordonner = _request('ordonner_lien');
-
-		if (_request('desordonner_liens')) {
-			include_spip('action/editer_liens');
-			objet_qualifier_liens([$objet_lien => '*'], [$objet => $id_objet], ['rang_lien' => 0]);
-		}
-
-		// il est possible de preciser dans une seule variable un remplacement :
-		// remplacer_lien[old][new]
-		if ($remplacer = _request('remplacer_lien')) {
-			foreach ($remplacer as $k => $v) {
-				if ($old = lien_verifier_action($k, '')) {
-					foreach (is_array($v) ? $v : [$v] as $kn => $vn) {
-						if ($new = lien_verifier_action($kn, $vn)) {
-							$supprimer[$old] = 'x';
-							$ajouter[$new] = '+';
-						}
-					}
-				}
-			}
-		}
-
-		if ($supprimer) {
-			if (
-				$supprimer_objets = charger_fonction(
-					"editer_liens_supprimer_{$table_source}_{$objet}_{$objet_lien}",
-					'action',
-					true
-				)
-			) {
-				$oups = $supprimer_objets($supprimer);
-			} else {
-				include_spip('action/editer_liens');
-				$oups = [];
-
-				foreach ($supprimer as $k => $v) {
-					if ($lien = lien_verifier_action($k, $v)) {
-						$lien = explode('-', $lien);
-						[$objet_source, $ids, $objet_lie, $idl, $role] = array_pad($lien, 5, null);
-						// appliquer une condition sur le rôle si défini ('*' pour tous les roles)
-						$cond = (!is_null($role) ? ['role' => $role] : []);
-						if ($objet_lien == $objet_source) {
-							$oups = array_merge(
-								$oups,
-								objet_trouver_liens([$objet_source => $ids], [$objet_lie => $idl], $cond)
-							);
-							objet_dissocier([$objet_source => $ids], [$objet_lie => $idl], $cond);
-						} else {
-							$oups = array_merge(
-								$oups,
-								objet_trouver_liens([$objet_lie => $idl], [$objet_source => $ids], $cond)
-							);
-							objet_dissocier([$objet_lie => $idl], [$objet_source => $ids], $cond);
-						}
-					}
-				}
-			}
-			set_request('_oups', $oups ? base64_encode(serialize($oups)) : null);
-		}
-
-		if ($ajouter) {
-			if (
-				$ajouter_objets = charger_fonction("editer_liens_ajouter_{$table_source}_{$objet}_{$objet_lien}", 'action', true)
-			) {
-				$ajout_ok = $ajouter_objets($ajouter);
-			} else {
-				$ajout_ok = false;
-				include_spip('action/editer_liens');
-				foreach ($ajouter as $k => $v) {
-					if ($lien = lien_verifier_action($k, $v)) {
-						$ajout_ok = true;
-						[$objet1, $ids, $objet2, $idl] = explode('-', $lien);
-						$qualifs = lien_retrouver_qualif($objet_lien, $lien);
-						if ($objet_lien == $objet1) {
-							lien_ajouter_liaisons($objet1, $ids, $objet2, $idl, $qualifs);
-						} else {
-							lien_ajouter_liaisons($objet2, $idl, $objet1, $ids, $qualifs);
-						}
-						set_request('id_lien_ajoute', $ids);
-					}
-				}
-			}
-			# oups ne persiste que pour la derniere action, si suppression
-			# une suppression suivie d'un ajout dans le meme hit est un remplacement
-			# non annulable !
-			if ($ajout_ok) {
-				set_request('_oups');
-			}
-		}
-
-		if ($ordonner) {
-			include_spip('action/editer_liens');
-			foreach ($ordonner as $k => $rang_lien) {
-				if ($lien = lien_verifier_action($k, '')) {
-					[$objet1, $ids, $objet2, $idl] = explode('-', $lien);
-					$qualif = ['rang_lien' => $rang_lien];
-
-					if ($objet_lien == $objet1) {
-						objet_qualifier_liens([$objet1 => $ids], [$objet2 => $idl], $qualif);
-					} else {
-						objet_qualifier_liens([$objet2 => $idl], [$objet1 => $ids], $qualif);
-					}
-					set_request('id_lien_ajoute', $ids);
-					set_request('_oups');
-				}
-			}
-		}
-	}
-
-
-	return $res;
+    // compat avec ancienne signature ou le 4eme argument est $editable
+    if (!is_array($options)) {
+        $options = ['editable' => $options];
+    } elseif (!isset($options['editable'])) {
+        $options['editable'] = true;
+    }
+
+    $editable = $options['editable'];
+
+    $res = ['editable' => $editable ? true : false];
+    [$table_source, $objet, $id_objet, $objet_lien] = determine_source_lien_objet($a, $b, $c);
+    if (!$table_source or !$objet or !$objet_lien) {
+        return $res;
+    }
+
+
+    if (_request('tout_voir')) {
+        set_request('recherche', '');
+    }
+
+    include_spip('inc/autoriser');
+    if (autoriser('modifier', $objet, $id_objet)) {
+        // annuler les suppressions du coup d'avant !
+        if (
+            _request('annuler_oups')
+            and $oups = _request('_oups')
+            and $oups = base64_decode($oups)
+            and $oups = unserialize($oups)
+        ) {
+            if ($oups_objets = charger_fonction("editer_liens_oups_{$table_source}_{$objet}_{$objet_lien}", 'action', true)) {
+                $oups_objets($oups);
+            } else {
+                $objet_source = objet_type($table_source);
+                include_spip('action/editer_liens');
+                foreach ($oups as $oup) {
+                    if ($objet_lien == $objet_source) {
+                        objet_associer([$objet_source => $oup[$objet_source]], [$objet => $oup[$objet]], $oup);
+                    } else {
+                        objet_associer([$objet => $oup[$objet]], [$objet_source => $oup[$objet_source]], $oup);
+                    }
+                }
+            }
+            # oups ne persiste que pour la derniere action, si suppression
+            set_request('_oups');
+        }
+
+        $supprimer = _request('supprimer_lien');
+        $ajouter = _request('ajouter_lien');
+        $ordonner = _request('ordonner_lien');
+
+        if (_request('desordonner_liens')) {
+            include_spip('action/editer_liens');
+            objet_qualifier_liens([$objet_lien => '*'], [$objet => $id_objet], ['rang_lien' => 0]);
+        }
+
+        // il est possible de preciser dans une seule variable un remplacement :
+        // remplacer_lien[old][new]
+        if ($remplacer = _request('remplacer_lien')) {
+            foreach ($remplacer as $k => $v) {
+                if ($old = lien_verifier_action($k, '')) {
+                    foreach (is_array($v) ? $v : [$v] as $kn => $vn) {
+                        if ($new = lien_verifier_action($kn, $vn)) {
+                            $supprimer[$old] = 'x';
+                            $ajouter[$new] = '+';
+                        }
+                    }
+                }
+            }
+        }
+
+        if ($supprimer) {
+            if (
+                $supprimer_objets = charger_fonction(
+                    "editer_liens_supprimer_{$table_source}_{$objet}_{$objet_lien}",
+                    'action',
+                    true
+                )
+            ) {
+                $oups = $supprimer_objets($supprimer);
+            } else {
+                include_spip('action/editer_liens');
+                $oups = [];
+
+                foreach ($supprimer as $k => $v) {
+                    if ($lien = lien_verifier_action($k, $v)) {
+                        $lien = explode('-', $lien);
+                        [$objet_source, $ids, $objet_lie, $idl, $role] = array_pad($lien, 5, null);
+                        // appliquer une condition sur le rôle si défini ('*' pour tous les roles)
+                        $cond = (!is_null($role) ? ['role' => $role] : []);
+                        if ($objet_lien == $objet_source) {
+                            $oups = array_merge(
+                                $oups,
+                                objet_trouver_liens([$objet_source => $ids], [$objet_lie => $idl], $cond)
+                            );
+                            objet_dissocier([$objet_source => $ids], [$objet_lie => $idl], $cond);
+                        } else {
+                            $oups = array_merge(
+                                $oups,
+                                objet_trouver_liens([$objet_lie => $idl], [$objet_source => $ids], $cond)
+                            );
+                            objet_dissocier([$objet_lie => $idl], [$objet_source => $ids], $cond);
+                        }
+                    }
+                }
+            }
+            set_request('_oups', $oups ? base64_encode(serialize($oups)) : null);
+        }
+
+        if ($ajouter) {
+            if (
+                $ajouter_objets = charger_fonction("editer_liens_ajouter_{$table_source}_{$objet}_{$objet_lien}", 'action', true)
+            ) {
+                $ajout_ok = $ajouter_objets($ajouter);
+            } else {
+                $ajout_ok = false;
+                include_spip('action/editer_liens');
+                foreach ($ajouter as $k => $v) {
+                    if ($lien = lien_verifier_action($k, $v)) {
+                        $ajout_ok = true;
+                        [$objet1, $ids, $objet2, $idl] = explode('-', $lien);
+                        $qualifs = lien_retrouver_qualif($objet_lien, $lien);
+                        if ($objet_lien == $objet1) {
+                            lien_ajouter_liaisons($objet1, $ids, $objet2, $idl, $qualifs);
+                        } else {
+                            lien_ajouter_liaisons($objet2, $idl, $objet1, $ids, $qualifs);
+                        }
+                        set_request('id_lien_ajoute', $ids);
+                    }
+                }
+            }
+            # oups ne persiste que pour la derniere action, si suppression
+            # une suppression suivie d'un ajout dans le meme hit est un remplacement
+            # non annulable !
+            if ($ajout_ok) {
+                set_request('_oups');
+            }
+        }
+
+        if ($ordonner) {
+            include_spip('action/editer_liens');
+            foreach ($ordonner as $k => $rang_lien) {
+                if ($lien = lien_verifier_action($k, '')) {
+                    [$objet1, $ids, $objet2, $idl] = explode('-', $lien);
+                    $qualif = ['rang_lien' => $rang_lien];
+
+                    if ($objet_lien == $objet1) {
+                        objet_qualifier_liens([$objet1 => $ids], [$objet2 => $idl], $qualif);
+                    } else {
+                        objet_qualifier_liens([$objet2 => $idl], [$objet1 => $ids], $qualif);
+                    }
+                    set_request('id_lien_ajoute', $ids);
+                    set_request('_oups');
+                }
+            }
+        }
+    }
+
+
+    return $res;
 }
 
 
@@ -391,24 +391,24 @@ 
  * @return string Action demandée si trouvée, sinon ''
  */
 function lien_verifier_action($k, $v) {
-	$action = '';
-	if (preg_match(',^\w+-[\w*]+-[\w*]+-[\w*]+(-[\w*])?,', $k)) {
-		$action = $k;
-	}
-	if (preg_match(',^\w+-[\w*]+-[\w*]+-[\w*]+(-[\w*])?,', $v)) {
-		if (is_numeric($k)) {
-			$action = $v;
-		}
-		if (_request($k)) {
-			$action = $v;
-		}
-	}
-	// ajout un role null fictif (plus pratique) si pas défini
-	if ($action and count(explode('-', $action)) == 4) {
-		$action .= '-';
-	}
-
-	return $action;
+    $action = '';
+    if (preg_match(',^\w+-[\w*]+-[\w*]+-[\w*]+(-[\w*])?,', $k)) {
+        $action = $k;
+    }
+    if (preg_match(',^\w+-[\w*]+-[\w*]+-[\w*]+(-[\w*])?,', $v)) {
+        if (is_numeric($k)) {
+            $action = $v;
+        }
+        if (_request($k)) {
+            $action = $v;
+        }
+    }
+    // ajout un role null fictif (plus pratique) si pas défini
+    if ($action and count(explode('-', $action)) == 4) {
+        $action .= '-';
+    }
+
+    return $action;
 }
 
 
@@ -424,66 +424,66 @@ 
  *   Liste des qualifs pour chaque lien. Tableau vide s'il n'y en a pas.
  **/
 function lien_retrouver_qualif($objet_lien, $lien) {
-	// un role est défini dans la liaison
-	$defs = explode('-', $lien);
-	[$objet1, , $objet2, , $role] = array_pad($defs, 5, null);
-	if ($objet_lien == $objet1) {
-		$colonne_role = roles_colonne($objet1, $objet2);
-	} else {
-		$colonne_role = roles_colonne($objet2, $objet1);
-	}
-
-	// cas ou le role est defini en 5e argument de l'action sur le lien (suppression, ajout rapide sans autre attribut)
-	if ($role) {
-		return [
-			// un seul lien avec ce role
-			[$colonne_role => $role]
-		];
-	}
-
-	// retrouver les rôles postés pour cette liaison, s'il y en a.
-	$qualifier_lien = _request('qualifier_lien');
-	if (!$qualifier_lien or !is_array($qualifier_lien)) {
-		return [];
-	}
-
-	// pas avec l'action complete (incluant le role)
-	$qualif = [];
-	if (
-		(!isset($qualifier_lien[$lien]) or !$qualif = $qualifier_lien[$lien])
-		and count($defs) == 5
-	) {
-		// on tente avec l'action sans le role
-		array_pop($defs);
-		$lien = implode('-', $defs);
-		if (!isset($qualifier_lien[$lien]) or !$qualif = $qualifier_lien[$lien]) {
-			$qualif = [];
-		}
-	}
-
-	// $qualif de la forme array(role=>array(...),valeur=>array(...),....)
-	// on le reforme en array(array(role=>..,valeur=>..,..),array(role=>..,valeur=>..,..),...)
-	$qualifs = [];
-	while (is_countable($qualif) ? count($qualif) : 0) {
-		$q = [];
-		foreach ($qualif as $att => $values) {
-			if (is_array($values)) {
-				$q[$att] = array_shift($qualif[$att]);
-				if (!(is_countable($qualif[$att]) ? count($qualif[$att]) : 0)) {
-					unset($qualif[$att]);
-				}
-			} else {
-				$q[$att] = $values;
-				unset($qualif[$att]);
-			}
-		}
-		// pas de rôle vide
-		if (!$colonne_role or !isset($q[$colonne_role]) or $q[$colonne_role]) {
-			$qualifs[] = $q;
-		}
-	}
-
-	return $qualifs;
+    // un role est défini dans la liaison
+    $defs = explode('-', $lien);
+    [$objet1, , $objet2, , $role] = array_pad($defs, 5, null);
+    if ($objet_lien == $objet1) {
+        $colonne_role = roles_colonne($objet1, $objet2);
+    } else {
+        $colonne_role = roles_colonne($objet2, $objet1);
+    }
+
+    // cas ou le role est defini en 5e argument de l'action sur le lien (suppression, ajout rapide sans autre attribut)
+    if ($role) {
+        return [
+            // un seul lien avec ce role
+            [$colonne_role => $role]
+        ];
+    }
+
+    // retrouver les rôles postés pour cette liaison, s'il y en a.
+    $qualifier_lien = _request('qualifier_lien');
+    if (!$qualifier_lien or !is_array($qualifier_lien)) {
+        return [];
+    }
+
+    // pas avec l'action complete (incluant le role)
+    $qualif = [];
+    if (
+        (!isset($qualifier_lien[$lien]) or !$qualif = $qualifier_lien[$lien])
+        and count($defs) == 5
+    ) {
+        // on tente avec l'action sans le role
+        array_pop($defs);
+        $lien = implode('-', $defs);
+        if (!isset($qualifier_lien[$lien]) or !$qualif = $qualifier_lien[$lien]) {
+            $qualif = [];
+        }
+    }
+
+    // $qualif de la forme array(role=>array(...),valeur=>array(...),....)
+    // on le reforme en array(array(role=>..,valeur=>..,..),array(role=>..,valeur=>..,..),...)
+    $qualifs = [];
+    while (is_countable($qualif) ? count($qualif) : 0) {
+        $q = [];
+        foreach ($qualif as $att => $values) {
+            if (is_array($values)) {
+                $q[$att] = array_shift($qualif[$att]);
+                if (!(is_countable($qualif[$att]) ? count($qualif[$att]) : 0)) {
+                    unset($qualif[$att]);
+                }
+            } else {
+                $q[$att] = $values;
+                unset($qualif[$att]);
+            }
+        }
+        // pas de rôle vide
+        if (!$colonne_role or !isset($q[$colonne_role]) or $q[$colonne_role]) {
+            $qualifs[] = $q;
+        }
+    }
+
+    return $qualifs;
 }
 
 /**
@@ -502,12 +502,12 @@ 
  **/
 function lien_ajouter_liaisons($objet_source, $ids, $objet_lien, $idl, $qualifs) {
 
-	// retrouver la colonne de roles s'il y en a a lier
-	if (is_array($qualifs) and count($qualifs)) {
-		foreach ($qualifs as $qualif) {
-			objet_associer([$objet_source => $ids], [$objet_lien => $idl], $qualif);
-		}
-	} else {
-		objet_associer([$objet_source => $ids], [$objet_lien => $idl]);
-	}
+    // retrouver la colonne de roles s'il y en a a lier
+    if (is_array($qualifs) and count($qualifs)) {
+        foreach ($qualifs as $qualif) {
+            objet_associer([$objet_source => $ids], [$objet_lien => $idl], $qualif);
+        }
+    } else {
+        objet_associer([$objet_source => $ids], [$objet_lien => $idl]);
+    }
 }

prive/formulaires/declarer_bases.php

Indentation +195 added lines, -195 removed lines

@@ -11,223 +11,223 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/install');
 
 function formulaires_declarer_bases_charger_dist() {
-	[$adresse_db, $login_db, $pass_db, $sel, $server_db] = analyse_fichier_connection(_FILE_CONNECT);
-
-	$deja = bases_referencees(_FILE_CONNECT);
-	// proposer un nom de connect si pas encore saisi
-	$nom_connect = '';
-	if (defined('_DECLARER_CHOIX_DB')) {
-		$nom_connect = _DECLARER_CHOIX_DB;
-		$n = '';
-		while (in_array($nom_connect . $n, $deja)) {
-			$n = ($n ? $n + 1 : 1);
-		}
-		$nom_connect = $nom_connect . $n;
-	}
-
-	$valeurs = [
-		'_etapes' => 3,
-		'_bases_deja' => $deja,
-		'_bases_prop' => defined('_DECLARER_SERVEUR_DB') ? liste_bases(_DECLARER_SERVEUR_DB) : '',
-		'_tables' => (defined('_DECLARER_SERVEUR_DB') and defined('_DECLARER_CHOIX_DB')) ?
-			$tables = sql_alltable('%', _DECLARER_SERVEUR_DB)
-			:
-			[],
-		'main_db' => '',
-		'_serveurs' => liste_serveurs(),
-		'sql_serveur_db' => 'sqlite3', // valeur par defaut
-		'adresse_db' => $adresse_db,
-		'login_db' => '',
-		'pass_db' => '',
-		'choix_db' => '',
-		'table_new' => '',
-		'nom_connect' => $nom_connect,
-	];
-
-	return $valeurs;
+    [$adresse_db, $login_db, $pass_db, $sel, $server_db] = analyse_fichier_connection(_FILE_CONNECT);
+
+    $deja = bases_referencees(_FILE_CONNECT);
+    // proposer un nom de connect si pas encore saisi
+    $nom_connect = '';
+    if (defined('_DECLARER_CHOIX_DB')) {
+        $nom_connect = _DECLARER_CHOIX_DB;
+        $n = '';
+        while (in_array($nom_connect . $n, $deja)) {
+            $n = ($n ? $n + 1 : 1);
+        }
+        $nom_connect = $nom_connect . $n;
+    }
+
+    $valeurs = [
+        '_etapes' => 3,
+        '_bases_deja' => $deja,
+        '_bases_prop' => defined('_DECLARER_SERVEUR_DB') ? liste_bases(_DECLARER_SERVEUR_DB) : '',
+        '_tables' => (defined('_DECLARER_SERVEUR_DB') and defined('_DECLARER_CHOIX_DB')) ?
+            $tables = sql_alltable('%', _DECLARER_SERVEUR_DB)
+            :
+            [],
+        'main_db' => '',
+        '_serveurs' => liste_serveurs(),
+        'sql_serveur_db' => 'sqlite3', // valeur par defaut
+        'adresse_db' => $adresse_db,
+        'login_db' => '',
+        'pass_db' => '',
+        'choix_db' => '',
+        'table_new' => '',
+        'nom_connect' => $nom_connect,
+    ];
+
+    return $valeurs;
 }
 
 function liste_serveurs() {
-	$options = [];
-	$dir = _DIR_RESTREINT . 'req/';
-	$d = opendir($dir);
-	if (!$d) {
-		return [];
-	}
-	while ($f = readdir($d)) {
-		if (
-			(preg_match('/^(.*)[.]php$/', $f, $s))
-			and is_readable($f = $dir . $f)
-		) {
-			require_once($f);
-			$s = $s[1];
-			$v = 'spip_versions_' . $s;
-			if (function_exists($v) and $v()) {
-				$options[$s] = "install_select_type_$s";
-			} else {
-				spip_log("$s: portage indisponible");
-			}
-		}
-	}
-	ksort($options);
-
-	return $options;
+    $options = [];
+    $dir = _DIR_RESTREINT . 'req/';
+    $d = opendir($dir);
+    if (!$d) {
+        return [];
+    }
+    while ($f = readdir($d)) {
+        if (
+            (preg_match('/^(.*)[.]php$/', $f, $s))
+            and is_readable($f = $dir . $f)
+        ) {
+            require_once($f);
+            $s = $s[1];
+            $v = 'spip_versions_' . $s;
+            if (function_exists($v) and $v()) {
+                $options[$s] = "install_select_type_$s";
+            } else {
+                spip_log("$s: portage indisponible");
+            }
+        }
+    }
+    ksort($options);
+
+    return $options;
 }
 
 function liste_bases($server_db) {
-	if (
-		is_null($server_db)
-		or !$result = sql_listdbs($server_db)
-	) {
-		return '';
-	}
-
-	$noms = [];
-
-	// si sqlite : result est deja un tableau
-	if (is_array($result)) {
-		$noms = $result;
-	} else {
-		while ($row = sql_fetch($result, $server_db)) {
-			$noms[] = reset($row);
-		}
-	}
-
-	return $noms;
+    if (
+        is_null($server_db)
+        or !$result = sql_listdbs($server_db)
+    ) {
+        return '';
+    }
+
+    $noms = [];
+
+    // si sqlite : result est deja un tableau
+    if (is_array($result)) {
+        $noms = $result;
+    } else {
+        while ($row = sql_fetch($result, $server_db)) {
+            $noms[] = reset($row);
+        }
+    }
+
+    return $noms;
 }
 
 function formulaires_declarer_bases_verifier_1_dist() {
-	$erreurs = [];
-	[$def_adresse_db, $def_login_db, $def_pass_db, $sel_db, $def_serveur_db] = analyse_fichier_connection(_FILE_CONNECT);
-
-	if (!$adresse_db = _request('adresse_db')) {
-		if (defined('_INSTALL_HOST_DB')) {
-			$adresse_db = _INSTALL_HOST_DB;
-		} else {
-			$adresse_db = $def_adresse_db;
-		}
-	}
-	if (!$serveur_db = _request('sql_serveur_db')) {
-		if (defined('_INSTALL_SERVER_DB')) {
-			$serveur_db = _INSTALL_SERVER_DB;
-		} else {
-			$serveur_db = $def_serveur_db;
-		}
-	}
-
-	$login_db = $pass_db = '';
-	if (!preg_match(',^sqlite,i', $serveur_db)) {
-		if (!$login_db = _request('login_db')) {
-			if (defined('_INSTALL_USER_DB')) {
-				$login_db = _INSTALL_USER_DB;
-			} else {
-				$login_db = $def_login_db;
-			}
-		}
-		if (!$pass_db = _request('pass_db')) {
-			if (defined('_INSTALL_PASS_DB')) {
-				$pass_db = _INSTALL_PASS_DB;
-			} else {
-				$pass_db = $def_pass_db;
-			}
-		}
-	}
-
-	$link = spip_connect_db($adresse_db, '', $login_db, $pass_db, '@test@', $serveur_db);
-	if ($link) {
-		$GLOBALS['connexions'][$serveur_db][$GLOBALS['spip_sql_version']] = $GLOBALS['spip_' . $serveur_db . '_functions_' . $GLOBALS['spip_sql_version']];
-		$GLOBALS['connexions'][$serveur_db] = $link;
-		define('_DECLARER_SERVEUR_DB', $serveur_db);
-		define('_DECLARER_ADRESSE_DB', $adresse_db);
-		define('_DECLARER_LOGIN_DB', $login_db);
-		define('_DECLARER_PASS_DB', $pass_db);
-		// si on est sur le meme serveur que connect.php
-		// indiquer quelle est la db utilisee pour l'exclure des choix possibles
-		if ($serveur_db == $def_serveur_db and $adresse_db == $def_adresse_db) {
-			set_request('main_db', $sel_db);
-		} else {
-			set_request('main_db', '');
-		}
-	} else {
-		$erreurs['message_erreur'] = _T('avis_connexion_echec_1');
-	}
-
-	return $erreurs;
+    $erreurs = [];
+    [$def_adresse_db, $def_login_db, $def_pass_db, $sel_db, $def_serveur_db] = analyse_fichier_connection(_FILE_CONNECT);
+
+    if (!$adresse_db = _request('adresse_db')) {
+        if (defined('_INSTALL_HOST_DB')) {
+            $adresse_db = _INSTALL_HOST_DB;
+        } else {
+            $adresse_db = $def_adresse_db;
+        }
+    }
+    if (!$serveur_db = _request('sql_serveur_db')) {
+        if (defined('_INSTALL_SERVER_DB')) {
+            $serveur_db = _INSTALL_SERVER_DB;
+        } else {
+            $serveur_db = $def_serveur_db;
+        }
+    }
+
+    $login_db = $pass_db = '';
+    if (!preg_match(',^sqlite,i', $serveur_db)) {
+        if (!$login_db = _request('login_db')) {
+            if (defined('_INSTALL_USER_DB')) {
+                $login_db = _INSTALL_USER_DB;
+            } else {
+                $login_db = $def_login_db;
+            }
+        }
+        if (!$pass_db = _request('pass_db')) {
+            if (defined('_INSTALL_PASS_DB')) {
+                $pass_db = _INSTALL_PASS_DB;
+            } else {
+                $pass_db = $def_pass_db;
+            }
+        }
+    }
+
+    $link = spip_connect_db($adresse_db, '', $login_db, $pass_db, '@test@', $serveur_db);
+    if ($link) {
+        $GLOBALS['connexions'][$serveur_db][$GLOBALS['spip_sql_version']] = $GLOBALS['spip_' . $serveur_db . '_functions_' . $GLOBALS['spip_sql_version']];
+        $GLOBALS['connexions'][$serveur_db] = $link;
+        define('_DECLARER_SERVEUR_DB', $serveur_db);
+        define('_DECLARER_ADRESSE_DB', $adresse_db);
+        define('_DECLARER_LOGIN_DB', $login_db);
+        define('_DECLARER_PASS_DB', $pass_db);
+        // si on est sur le meme serveur que connect.php
+        // indiquer quelle est la db utilisee pour l'exclure des choix possibles
+        if ($serveur_db == $def_serveur_db and $adresse_db == $def_adresse_db) {
+            set_request('main_db', $sel_db);
+        } else {
+            set_request('main_db', '');
+        }
+    } else {
+        $erreurs['message_erreur'] = _T('avis_connexion_echec_1');
+    }
+
+    return $erreurs;
 }
 
 function formulaires_declarer_bases_verifier_2_dist() {
-	$erreurs = [];
-	$choix_db = _request('choix_db');
-	if ($choix_db == '-1') {
-		$choix_db = _request('table_new');
-	}
-	if (!$choix_db) {
-		$erreurs['choix_db'] = _T('info_obligatoire');
-	} else {
-		define('_ECRIRE_INSTALL', 1); // hackons sqlite
-		if (!sql_selectdb($choix_db, _DECLARER_SERVEUR_DB)) {
-			$erreurs['choix_db'] = _T('avis_base_inaccessible', ['base' => $choix_db]);
-		} else {
-			define('_DECLARER_CHOIX_DB', $choix_db);
-		}
-	}
-
-	return $erreurs;
+    $erreurs = [];
+    $choix_db = _request('choix_db');
+    if ($choix_db == '-1') {
+        $choix_db = _request('table_new');
+    }
+    if (!$choix_db) {
+        $erreurs['choix_db'] = _T('info_obligatoire');
+    } else {
+        define('_ECRIRE_INSTALL', 1); // hackons sqlite
+        if (!sql_selectdb($choix_db, _DECLARER_SERVEUR_DB)) {
+            $erreurs['choix_db'] = _T('avis_base_inaccessible', ['base' => $choix_db]);
+        } else {
+            define('_DECLARER_CHOIX_DB', $choix_db);
+        }
+    }
+
+    return $erreurs;
 }
 
 function formulaires_declarer_bases_verifier_3_dist() {
-	$erreurs = [];
-	$nom_connect = _request('nom_connect');
-	if (!$nom_connect) {
-		$erreurs['nom_connect'] = _T('info_obligatoire');
-	} else {
-		// securite : le nom doit etre un mot sans caracteres speciaux
-		$f = preg_replace(',[^\w],', '', $nom_connect);
-		if ($f !== $nom_connect) {
-			$erreurs['nom_connect'] = _T('erreur_nom_connect_incorrect');
-		} elseif (file_exists(_DIR_CONNECT . $nom_connect . '.php')) {
-			$erreurs['nom_connect'] = _T('erreur_connect_deja_existant');
-		} else {
-			define('_DECLARER_NOM_CONNECT', $nom_connect);
-		}
-	}
-
-	return $erreurs;
+    $erreurs = [];
+    $nom_connect = _request('nom_connect');
+    if (!$nom_connect) {
+        $erreurs['nom_connect'] = _T('info_obligatoire');
+    } else {
+        // securite : le nom doit etre un mot sans caracteres speciaux
+        $f = preg_replace(',[^\w],', '', $nom_connect);
+        if ($f !== $nom_connect) {
+            $erreurs['nom_connect'] = _T('erreur_nom_connect_incorrect');
+        } elseif (file_exists(_DIR_CONNECT . $nom_connect . '.php')) {
+            $erreurs['nom_connect'] = _T('erreur_connect_deja_existant');
+        } else {
+            define('_DECLARER_NOM_CONNECT', $nom_connect);
+        }
+    }
+
+    return $erreurs;
 }
 
 function formulaires_declarer_bases_traiter_dist() {
 
-	$adresse_db = _DECLARER_ADRESSE_DB;
-	if (preg_match(',(.*):(.*),', $adresse_db, $r)) {
-		[, $adresse_db, $port] = $r;
-	} else {
-		$port = '';
-	}
-
-	$server_db = addcslashes(_DECLARER_SERVEUR_DB, "'\\");
-
-	$conn = install_mode_appel($server_db)
-		. install_connexion(
-			$adresse_db,
-			$port,
-			_DECLARER_LOGIN_DB,
-			_DECLARER_PASS_DB,
-			_DECLARER_CHOIX_DB,
-			_DECLARER_SERVEUR_DB,
-			'',
-			'',
-			''
-		);
-
-	install_fichier_connexion(_DIR_CONNECT . _DECLARER_NOM_CONNECT . '.php', $conn);
-
-	return [
-		'message_ok' => _T('install_connect_ok', ['connect' => '<strong>' . _DECLARER_NOM_CONNECT . '</strong>'])
-	];
+    $adresse_db = _DECLARER_ADRESSE_DB;
+    if (preg_match(',(.*):(.*),', $adresse_db, $r)) {
+        [, $adresse_db, $port] = $r;
+    } else {
+        $port = '';
+    }
+
+    $server_db = addcslashes(_DECLARER_SERVEUR_DB, "'\\");
+
+    $conn = install_mode_appel($server_db)
+        . install_connexion(
+            $adresse_db,
+            $port,
+            _DECLARER_LOGIN_DB,
+            _DECLARER_PASS_DB,
+            _DECLARER_CHOIX_DB,
+            _DECLARER_SERVEUR_DB,
+            '',
+            '',
+            ''
+        );
+
+    install_fichier_connexion(_DIR_CONNECT . _DECLARER_NOM_CONNECT . '.php', $conn);
+
+    return [
+        'message_ok' => _T('install_connect_ok', ['connect' => '<strong>' . _DECLARER_NOM_CONNECT . '</strong>'])
+    ];
 }

prive/formulaires/recherche_ecrire.php

Indentation +15 added lines, -15 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -33,19 +33,19 @@ 
  * @return array Environnement du formulaire
  **/
 function formulaires_recherche_ecrire_charger_dist($action = '', $class = '') {
-	if ($GLOBALS['spip_lang'] != $GLOBALS['meta']['langue_site']) {
-		$lang = $GLOBALS['spip_lang'];
-	} else {
-		$lang = '';
-	}
+    if ($GLOBALS['spip_lang'] != $GLOBALS['meta']['langue_site']) {
+        $lang = $GLOBALS['spip_lang'];
+    } else {
+        $lang = '';
+    }
 
-	return
-		[
-			'action' => ($action ?: generer_url_ecrire('recherche')),
-			# action specifique, ne passe pas par Verifier, ni Traiter
-			'recherche' => _request('recherche'),
-			'lang' => $lang,
-			'class' => $class,
-			'_id_champ' => 'rechercher_' . substr(md5($action . $class), 0, 4),
-		];
+    return
+        [
+            'action' => ($action ?: generer_url_ecrire('recherche')),
+            # action specifique, ne passe pas par Verifier, ni Traiter
+            'recherche' => _request('recherche'),
+            'lang' => $lang,
+            'class' => $class,
+            '_id_champ' => 'rechercher_' . substr(md5($action . $class), 0, 4),
+        ];
 }

prive/formulaires/configurer_reducteur.php

Indentation +91 added lines, -91 removed lines

@@ -18,7 +18,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -28,24 +28,24 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_configurer_reducteur_charger_dist() {
-	$valeurs = [];
-	foreach (
-		[
-			'image_process',
-			'formats_graphiques',
-			'creer_preview',
-			'taille_preview',
-		] as $m
-	) {
-		$valeurs[$m] = $GLOBALS['meta'][$m] ?? null;
-	}
+    $valeurs = [];
+    foreach (
+        [
+            'image_process',
+            'formats_graphiques',
+            'creer_preview',
+            'taille_preview',
+        ] as $m
+    ) {
+        $valeurs[$m] = $GLOBALS['meta'][$m] ?? null;
+    }
 
-	$valeurs['taille_preview'] = intval($valeurs['taille_preview']);
-	if ($valeurs['taille_preview'] < 10) {
-		$valeurs['taille_preview'] = 120;
-	}
+    $valeurs['taille_preview'] = intval($valeurs['taille_preview']);
+    if ($valeurs['taille_preview'] < 10) {
+        $valeurs['taille_preview'] = 120;
+    }
 
-	return $valeurs;
+    return $valeurs;
 }
 
 
@@ -56,53 +56,53 @@ 
  *     Retours des traitements
  **/
 function formulaires_configurer_reducteur_traiter_dist() {
-	$res = ['editable' => true];
+    $res = ['editable' => true];
 
-	if (is_array($image_process = _request('image_process_'))) {
-		$image_process = array_keys($image_process);
-		$image_process = reset($image_process);
+    if (is_array($image_process = _request('image_process_'))) {
+        $image_process = array_keys($image_process);
+        $image_process = reset($image_process);
 
-		// application du choix de vignette
-		if ($image_process) {
-			// mettre a jour les formats graphiques lisibles
-			switch ($image_process) {
-				case 'gd1':
-				case 'gd2':
-					$formats_graphiques = $GLOBALS['meta']['gd_formats_read'];
-					break;
-				case 'netpbm':
-					$formats_graphiques = $GLOBALS['meta']['netpbm_formats'];
-					break;
-				case 'convert':
-				case 'imagick':
-					$formats_graphiques = 'gif,jpg,png,webp';
-					break;
-				default: #debug
-					$formats_graphiques = '';
-					$image_process = 'non';
-					break;
-			}
-			ecrire_meta('formats_graphiques', $formats_graphiques, 'non');
-			ecrire_meta('image_process', $image_process, 'non');
-		}
-	}
+        // application du choix de vignette
+        if ($image_process) {
+            // mettre a jour les formats graphiques lisibles
+            switch ($image_process) {
+                case 'gd1':
+                case 'gd2':
+                    $formats_graphiques = $GLOBALS['meta']['gd_formats_read'];
+                    break;
+                case 'netpbm':
+                    $formats_graphiques = $GLOBALS['meta']['netpbm_formats'];
+                    break;
+                case 'convert':
+                case 'imagick':
+                    $formats_graphiques = 'gif,jpg,png,webp';
+                    break;
+                default: #debug
+                    $formats_graphiques = '';
+                    $image_process = 'non';
+                    break;
+            }
+            ecrire_meta('formats_graphiques', $formats_graphiques, 'non');
+            ecrire_meta('image_process', $image_process, 'non');
+        }
+    }
 
-	foreach (
-		[
-			'creer_preview'
-		] as $m
-	) {
-		if (!is_null($v = _request($m))) {
-			ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
-		}
-	}
-	if (!is_null($v = _request('taille_preview'))) {
-		ecrire_meta('taille_preview', intval($v));
-	}
+    foreach (
+        [
+            'creer_preview'
+        ] as $m
+    ) {
+        if (!is_null($v = _request($m))) {
+            ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
+        }
+    }
+    if (!is_null($v = _request('taille_preview'))) {
+        ecrire_meta('taille_preview', intval($v));
+    }
 
-	$res['message_ok'] = _T('config_info_enregistree');
+    $res['message_ok'] = _T('config_info_enregistree');
 
-	return $res;
+    return $res;
 }
 
 /**
@@ -115,37 +115,37 @@ 
  *     URL d'action pour tester la librairie graphique en créant une vignette
  **/
 function url_vignette_choix($process) {
-	switch ($process) {
-		case 'gd2':
-			if (!function_exists('ImageCreateTrueColor')) {
-				return '';
-			}
-			break;
-		case 'gd1':
-			if (
-				!function_exists('ImageGif')
-				and !function_exists('ImageJpeg')
-				and !function_exists('ImagePng')
-			) {
-				return '';
-			}
-			break;
-		case 'netpbm':
-			if (defined('_PNMSCALE_COMMAND') and _PNMSCALE_COMMAND == '') {
-				return '';
-			}
-			break;
-		case 'imagick':
-			if (!method_exists(\Imagick::class, 'readImage')) {
-				return '';
-			}
-			break;
-		case 'convert':
-			if (defined('_CONVERT_COMMAND') and _CONVERT_COMMAND == '') {
-				return '';
-			}
-			break;
-	}
+    switch ($process) {
+        case 'gd2':
+            if (!function_exists('ImageCreateTrueColor')) {
+                return '';
+            }
+            break;
+        case 'gd1':
+            if (
+                !function_exists('ImageGif')
+                and !function_exists('ImageJpeg')
+                and !function_exists('ImagePng')
+            ) {
+                return '';
+            }
+            break;
+        case 'netpbm':
+            if (defined('_PNMSCALE_COMMAND') and _PNMSCALE_COMMAND == '') {
+                return '';
+            }
+            break;
+        case 'imagick':
+            if (!method_exists(\Imagick::class, 'readImage')) {
+                return '';
+            }
+            break;
+        case 'convert':
+            if (defined('_CONVERT_COMMAND') and _CONVERT_COMMAND == '') {
+                return '';
+            }
+            break;
+    }
 
-	return generer_url_action('tester', "arg=$process&time=" . time());
+    return generer_url_action('tester', "arg=$process&time=" . time());
 }

prive/formulaires/configurer_preferences.php

Indentation +47 added lines, -47 removed lines

@@ -21,7 +21,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -31,28 +31,28 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_configurer_preferences_charger_dist() {
-	// travailler sur des meta fraiches
-	include_spip('inc/meta');
-	lire_metas();
+    // travailler sur des meta fraiches
+    include_spip('inc/meta');
+    lire_metas();
 
-	$valeurs = [];
-	$valeurs['display_navigation'] = $GLOBALS['visiteur_session']['prefs']['display_navigation'] ?? 'navigation_avec_icones';
-	$valeurs['display_outils'] = $GLOBALS['visiteur_session']['prefs']['display_outils'] ?? 'oui';
-	$valeurs['display'] = (isset($GLOBALS['visiteur_session']['prefs']['display']) and $GLOBALS['visiteur_session']['prefs']['display'] > 0) ? $GLOBALS['visiteur_session']['prefs']['display'] : 2;
-	$valeurs['couleur'] = (isset($GLOBALS['visiteur_session']['prefs']['couleur']) and $GLOBALS['visiteur_session']['prefs']['couleur'] > 0) ? $GLOBALS['visiteur_session']['prefs']['couleur'] : 1;
+    $valeurs = [];
+    $valeurs['display_navigation'] = $GLOBALS['visiteur_session']['prefs']['display_navigation'] ?? 'navigation_avec_icones';
+    $valeurs['display_outils'] = $GLOBALS['visiteur_session']['prefs']['display_outils'] ?? 'oui';
+    $valeurs['display'] = (isset($GLOBALS['visiteur_session']['prefs']['display']) and $GLOBALS['visiteur_session']['prefs']['display'] > 0) ? $GLOBALS['visiteur_session']['prefs']['display'] : 2;
+    $valeurs['couleur'] = (isset($GLOBALS['visiteur_session']['prefs']['couleur']) and $GLOBALS['visiteur_session']['prefs']['couleur'] > 0) ? $GLOBALS['visiteur_session']['prefs']['couleur'] : 1;
 
-	$couleurs = charger_fonction('couleurs', 'inc');
-	$les_couleurs = $couleurs();
-	foreach ($les_couleurs as $k => $c) {
-		$valeurs['_couleurs_url'][$k] = generer_url_public('style_prive.css', 'ltr='
-			. $GLOBALS['spip_lang_left'] . '&'
-			. $couleurs($k));
-		$valeurs['couleurs'][$k] = $c;
-	}
+    $couleurs = charger_fonction('couleurs', 'inc');
+    $les_couleurs = $couleurs();
+    foreach ($les_couleurs as $k => $c) {
+        $valeurs['_couleurs_url'][$k] = generer_url_public('style_prive.css', 'ltr='
+            . $GLOBALS['spip_lang_left'] . '&'
+            . $couleurs($k));
+        $valeurs['couleurs'][$k] = $c;
+    }
 
-	$valeurs['imessage'] = $GLOBALS['visiteur_session']['imessage'];
+    $valeurs['imessage'] = $GLOBALS['visiteur_session']['imessage'];
 
-	return $valeurs;
+    return $valeurs;
 }
 
 /**
@@ -63,36 +63,36 @@ 
  **/
 function formulaires_configurer_preferences_traiter_dist() {
 
-	if ($couleur = _request('couleur')) {
-		$couleurs = charger_fonction('couleurs', 'inc');
-		$les_couleurs = $couleurs([], true);
-		if (isset($les_couleurs[$couleur])) {
-			$GLOBALS['visiteur_session']['prefs']['couleur'] = $couleur;
-		}
-	}
-	if ($display = intval(_request('display'))) {
-		$GLOBALS['visiteur_session']['prefs']['display'] = $display;
-	}
-	if (
-		$display_navigation = _request('display_navigation')
-		and in_array($display_navigation, ['navigation_sans_icone', 'navigation_avec_icones'])
-	) {
-		$GLOBALS['visiteur_session']['prefs']['display_navigation'] = $display_navigation;
-	}
-	if (!is_null($display_outils = _request('display_outils'))) {
-		$GLOBALS['visiteur_session']['prefs']['display_outils'] = ($display_outils ? 'oui' : '');
-	}
+    if ($couleur = _request('couleur')) {
+        $couleurs = charger_fonction('couleurs', 'inc');
+        $les_couleurs = $couleurs([], true);
+        if (isset($les_couleurs[$couleur])) {
+            $GLOBALS['visiteur_session']['prefs']['couleur'] = $couleur;
+        }
+    }
+    if ($display = intval(_request('display'))) {
+        $GLOBALS['visiteur_session']['prefs']['display'] = $display;
+    }
+    if (
+        $display_navigation = _request('display_navigation')
+        and in_array($display_navigation, ['navigation_sans_icone', 'navigation_avec_icones'])
+    ) {
+        $GLOBALS['visiteur_session']['prefs']['display_navigation'] = $display_navigation;
+    }
+    if (!is_null($display_outils = _request('display_outils'))) {
+        $GLOBALS['visiteur_session']['prefs']['display_outils'] = ($display_outils ? 'oui' : '');
+    }
 
-	if (intval($GLOBALS['visiteur_session']['id_auteur'])) {
-		include_spip('action/editer_auteur');
-		$c = ['prefs' => serialize($GLOBALS['visiteur_session']['prefs'])];
+    if (intval($GLOBALS['visiteur_session']['id_auteur'])) {
+        include_spip('action/editer_auteur');
+        $c = ['prefs' => serialize($GLOBALS['visiteur_session']['prefs'])];
 
-		if ($imessage = _request('imessage') and in_array($imessage, ['oui', 'non'])) {
-			$c['imessage'] = $imessage;
-		}
+        if ($imessage = _request('imessage') and in_array($imessage, ['oui', 'non'])) {
+            $c['imessage'] = $imessage;
+        }
 
-		auteur_modifier($GLOBALS['visiteur_session']['id_auteur'], $c);
-	}
+        auteur_modifier($GLOBALS['visiteur_session']['id_auteur'], $c);
+    }
 
-	return ['message_ok' => _T('config_info_enregistree'), 'editable' => true];
+    return ['message_ok' => _T('config_info_enregistree'), 'editable' => true];
 }

prive/formulaires/configurer_identite.php

Indentation +34 added lines, -34 removed lines

@@ -11,56 +11,56 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function formulaires_configurer_identite_charger_dist() {
-	// travailler sur des meta fraiches
-	include_spip('inc/meta');
-	lire_metas();
+    // travailler sur des meta fraiches
+    include_spip('inc/meta');
+    lire_metas();
 
-	$valeurs = [];
-	foreach (['nom_site', 'adresse_site', 'slogan_site', 'descriptif_site', 'email_webmaster'] as $k) {
-		$valeurs[$k] = $GLOBALS['meta'][$k] ?? '';
-	}
+    $valeurs = [];
+    foreach (['nom_site', 'adresse_site', 'slogan_site', 'descriptif_site', 'email_webmaster'] as $k) {
+        $valeurs[$k] = $GLOBALS['meta'][$k] ?? '';
+    }
 
-	return $valeurs;
+    return $valeurs;
 }
 
 function formulaires_configurer_identite_verifier_dist() {
-	$erreurs = [];
+    $erreurs = [];
 
-	// adresse_site est obligatoire mais rempli automatiquement si absent !
-	foreach (['nom_site'/*,'adresse_site'*/] as $obli) {
-		if (!_request($obli)) {
-			$erreurs[$obli] = _T('info_obligatoire');
-		}
-	}
+    // adresse_site est obligatoire mais rempli automatiquement si absent !
+    foreach (['nom_site'/*,'adresse_site'*/] as $obli) {
+        if (!_request($obli)) {
+            $erreurs[$obli] = _T('info_obligatoire');
+        }
+    }
 
-	if ($email = _request('email_webmaster') and !email_valide($email)) {
-		$erreurs['email_webmaster'] = _T('info_email_invalide');
-	}
+    if ($email = _request('email_webmaster') and !email_valide($email)) {
+        $erreurs['email_webmaster'] = _T('info_email_invalide');
+    }
 
-	return $erreurs;
+    return $erreurs;
 }
 
 function formulaires_configurer_identite_traiter_dist() {
-	include_spip('inc/config');
-	$adresse_site = $GLOBALS['meta']['adresse_site'] ?? '';
-	if (_request('adresse_site') != $adresse_site) {
-		refuser_traiter_formulaire_ajax();
-	}
+    include_spip('inc/config');
+    $adresse_site = $GLOBALS['meta']['adresse_site'] ?? '';
+    if (_request('adresse_site') != $adresse_site) {
+        refuser_traiter_formulaire_ajax();
+    }
 
-	set_request('adresse_site', appliquer_adresse_site(_request('adresse_site')));
+    set_request('adresse_site', appliquer_adresse_site(_request('adresse_site')));
 
-	include_spip('inc/meta');
-	foreach (['nom_site', 'slogan_site', 'descriptif_site', 'email_webmaster'] as $k) {
-		ecrire_meta($k, _request($k));
-	}
+    include_spip('inc/meta');
+    foreach (['nom_site', 'slogan_site', 'descriptif_site', 'email_webmaster'] as $k) {
+        ecrire_meta($k, _request($k));
+    }
 
-	include_spip('inc/texte_mini');
-	$reload = texte_script(couper(_request('nom_site'), 35));
-	$reload = "<script type='text/javascript'>if (window.jQuery) jQuery('#bando_identite .nom_site_spip .nom').html('$reload');</script>";
+    include_spip('inc/texte_mini');
+    $reload = texte_script(couper(_request('nom_site'), 35));
+    $reload = "<script type='text/javascript'>if (window.jQuery) jQuery('#bando_identite .nom_site_spip .nom').html('$reload');</script>";
 
-	return ['message_ok' => _T('config_info_enregistree') . $reload, 'editable' => true];
+    return ['message_ok' => _T('config_info_enregistree') . $reload, 'editable' => true];
 }

prive/formulaires/editer_logo.php

Indentation +160 added lines, -160 removed lines

@@ -30,7 +30,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 global $logo_libelles;
@@ -49,91 +49,91 @@ 
  * @return array               Variables d'environnement pour le fond
  */
 function formulaires_editer_logo_charger_dist($objet, $id_objet, $retour = '', $options = []) {
-	// pas dans une boucle ? formulaire pour le logo du site
-	// dans ce cas, il faut chercher un 'siteon0.ext'
-	if (!$objet) {
-		$objet = 'site';
-	}
+    // pas dans une boucle ? formulaire pour le logo du site
+    // dans ce cas, il faut chercher un 'siteon0.ext'
+    if (!$objet) {
+        $objet = 'site';
+    }
 
-	$objet = objet_type($objet);
-	$_id_objet = id_table_objet($objet);
+    $objet = objet_type($objet);
+    $_id_objet = id_table_objet($objet);
 
-	if (!is_array($options)) {
-		$options = unserialize($options);
-	}
-	$options = spip_sanitize_from_request($options, '*');
+    if (!is_array($options)) {
+        $options = unserialize($options);
+    }
+    $options = spip_sanitize_from_request($options, '*');
 
-	if (!isset($options['titre'])) {
-		$balise_img = chercher_filtre('balise_img');
-		$img = $balise_img(chemin_image('image-24.png'), '', 'cadre-icone');
-		$libelles = pipeline('libeller_logo', $GLOBALS['logo_libelles']);
-		$libelle = (($id_objet or $objet != 'rubrique') ? $objet : 'racine');
-		if (isset($libelles[$libelle])) {
-			$libelle = $libelles[$libelle];
-		} elseif ($libelle = objet_info($objet, 'texte_logo_objet')) {
-			$libelle = _T($libelle);
-		} else {
-			$libelle = _L('Logo');
-		}
-		switch ($objet) {
-			case 'article':
-				$libelle .= ' ' . aider('logoart');
-				break;
-			case 'breve':
-				$libelle .= ' ' . aider('breveslogo');
-				break;
-			case 'rubrique':
-				$libelle .= ' ' . aider('rublogo');
-				break;
-			default:
-				break;
-		}
+    if (!isset($options['titre'])) {
+        $balise_img = chercher_filtre('balise_img');
+        $img = $balise_img(chemin_image('image-24.png'), '', 'cadre-icone');
+        $libelles = pipeline('libeller_logo', $GLOBALS['logo_libelles']);
+        $libelle = (($id_objet or $objet != 'rubrique') ? $objet : 'racine');
+        if (isset($libelles[$libelle])) {
+            $libelle = $libelles[$libelle];
+        } elseif ($libelle = objet_info($objet, 'texte_logo_objet')) {
+            $libelle = _T($libelle);
+        } else {
+            $libelle = _L('Logo');
+        }
+        switch ($objet) {
+            case 'article':
+                $libelle .= ' ' . aider('logoart');
+                break;
+            case 'breve':
+                $libelle .= ' ' . aider('breveslogo');
+                break;
+            case 'rubrique':
+                $libelle .= ' ' . aider('rublogo');
+                break;
+            default:
+                break;
+        }
 
-		$options['titre'] = $img . $libelle;
-	}
-	if (!isset($options['editable'])) {
-		include_spip('inc/autoriser');
-		$options['editable'] = autoriser('iconifier', $objet, $id_objet);
-	}
+        $options['titre'] = $img . $libelle;
+    }
+    if (!isset($options['editable'])) {
+        include_spip('inc/autoriser');
+        $options['editable'] = autoriser('iconifier', $objet, $id_objet);
+    }
 
-	$res = [
-		'editable' => ($GLOBALS['meta']['activer_logos'] == 'oui' ? ' ' : '') && (!isset($options['editable']) or $options['editable']),
-		'logo_survol' => ($GLOBALS['meta']['activer_logos_survol'] == 'oui' ? ' ' : ''),
-		'objet' => $objet,
-		'id_objet' => $id_objet,
-		'_options' => $options,
-		'_show_upload_off' => '',
-	];
+    $res = [
+        'editable' => ($GLOBALS['meta']['activer_logos'] == 'oui' ? ' ' : '') && (!isset($options['editable']) or $options['editable']),
+        'logo_survol' => ($GLOBALS['meta']['activer_logos_survol'] == 'oui' ? ' ' : ''),
+        'objet' => $objet,
+        'id_objet' => $id_objet,
+        '_options' => $options,
+        '_show_upload_off' => '',
+    ];
 
-	// rechercher le logo de l'objet
-	// la fonction prend un parametre '_id_objet' etrange :
-	// le nom de la cle primaire (et non le nom de la table)
-	// ou directement le nom du raccourcis a chercher
-	$chercher_logo = charger_fonction('chercher_logo', 'inc');
-	$etats = $res['logo_survol'] ? ['on', 'off'] : ['on'];
-	foreach ($etats as $etat) {
-		$logo = $chercher_logo($id_objet, $_id_objet, $etat);
-		if ($logo) {
-			$res['logo_' . $etat] = $logo[0];
-		}
-	}
-	// pas de logo_on -> pas de formulaire pour le survol
-	if (!isset($res['logo_on'])) {
-		$res['logo_survol'] = '';
-	} elseif (!isset($res['logo_off']) and _request('logo_up')) {
-		$res['_show_upload_off'] = ' ';
-	}
+    // rechercher le logo de l'objet
+    // la fonction prend un parametre '_id_objet' etrange :
+    // le nom de la cle primaire (et non le nom de la table)
+    // ou directement le nom du raccourcis a chercher
+    $chercher_logo = charger_fonction('chercher_logo', 'inc');
+    $etats = $res['logo_survol'] ? ['on', 'off'] : ['on'];
+    foreach ($etats as $etat) {
+        $logo = $chercher_logo($id_objet, $_id_objet, $etat);
+        if ($logo) {
+            $res['logo_' . $etat] = $logo[0];
+        }
+    }
+    // pas de logo_on -> pas de formulaire pour le survol
+    if (!isset($res['logo_on'])) {
+        $res['logo_survol'] = '';
+    } elseif (!isset($res['logo_off']) and _request('logo_up')) {
+        $res['_show_upload_off'] = ' ';
+    }
 
-	// si le logo n'est pas editable et qu'il n'y en a pas, on affiche pas du tout le formulaire
-	if (
-		!$res['editable']
-		and !isset($res['logo_off'])
-		and !isset($res['logo_on'])
-	) {
-		return false;
-	}
+    // si le logo n'est pas editable et qu'il n'y en a pas, on affiche pas du tout le formulaire
+    if (
+        !$res['editable']
+        and !isset($res['logo_off'])
+        and !isset($res['logo_on'])
+    ) {
+        return false;
+    }
 
-	return $res;
+    return $res;
 }
 
 /**
@@ -147,7 +147,7 @@ 
  * @return string              Hash du formulaire
  */
 function formulaires_editer_logo_identifier_dist($objet, $id_objet, $retour = '', $options = []) {
-	return serialize([$objet, $id_objet]);
+    return serialize([$objet, $id_objet]);
 }
 
 /**
@@ -163,31 +163,31 @@ 
  * @return array               Erreurs du formulaire
  */
 function formulaires_editer_logo_verifier_dist($objet, $id_objet, $retour = '', $options = []) {
-	$erreurs = [];
-	// verifier les extensions
-	$sources = formulaire_editer_logo_get_sources();
-	include_spip('action/editer_logo');
-	$extensions_possibles = $GLOBALS['formats_logos'];
-	if (in_array('jpg', $extensions_possibles)) {
-		$extensions_possibles[] = 'jpeg';
-	}
-	foreach ($sources as $etat => $file) {
-		// seulement si une reception correcte a eu lieu
-		if ($file and $file['error'] == 0) {
-			if (
-				!in_array(
-					strtolower(pathinfo($file['name'], PATHINFO_EXTENSION)),
-					$extensions_possibles
-				)
-			) {
-				$erreurs['logo_' . $etat] = _L('Extension non reconnue');
-			}
-		} elseif ($file and $file['error'] != 0 and isset($file['msg'])) {
-			$erreurs['message_erreur'] = $file['msg'];
-		}
-	}
+    $erreurs = [];
+    // verifier les extensions
+    $sources = formulaire_editer_logo_get_sources();
+    include_spip('action/editer_logo');
+    $extensions_possibles = $GLOBALS['formats_logos'];
+    if (in_array('jpg', $extensions_possibles)) {
+        $extensions_possibles[] = 'jpeg';
+    }
+    foreach ($sources as $etat => $file) {
+        // seulement si une reception correcte a eu lieu
+        if ($file and $file['error'] == 0) {
+            if (
+                !in_array(
+                    strtolower(pathinfo($file['name'], PATHINFO_EXTENSION)),
+                    $extensions_possibles
+                )
+            ) {
+                $erreurs['logo_' . $etat] = _L('Extension non reconnue');
+            }
+        } elseif ($file and $file['error'] != 0 and isset($file['msg'])) {
+            $erreurs['message_erreur'] = $file['msg'];
+        }
+    }
 
-	return $erreurs;
+    return $erreurs;
 }
 
 /**
@@ -203,47 +203,47 @@ 
  * @return array               Retour des traitements
  */
 function formulaires_editer_logo_traiter_dist($objet, $id_objet, $retour = '', $options = []) {
-	$res = ['editable' => ' '];
+    $res = ['editable' => ' '];
 
-	// pas dans une boucle ? formulaire pour le logo du site
-	// dans ce cas, il faut chercher un 'siteon0.ext'
-	if (!$objet) {
-		$objet = 'site';
-	}
+    // pas dans une boucle ? formulaire pour le logo du site
+    // dans ce cas, il faut chercher un 'siteon0.ext'
+    if (!$objet) {
+        $objet = 'site';
+    }
 
-	include_spip('action/editer_logo');
+    include_spip('action/editer_logo');
 
-	// effectuer la suppression si demandee d'un logo
-	$on = _request('supprimer_logo_on');
-	if ($on or _request('supprimer_logo_off')) {
-		logo_supprimer($objet, $id_objet, $on ? 'on' : 'off');
-		$res['message_ok'] = ''; // pas besoin de message : la validation est visuelle
-		set_request('logo_up', ' ');
-	} // sinon supprimer ancien logo puis copier le nouveau
-	else {
-		$sources = formulaire_editer_logo_get_sources();
-		foreach ($sources as $etat => $file) {
-			if ($file and $file['error'] == 0) {
-				if ($err = logo_modifier($objet, $id_objet, $etat, $file)) {
-					$res['message_erreur'] = $err;
-				} else {
-					$res['message_ok'] = '';
-				} // pas besoin de message : la validation est visuelle
-				set_request('logo_up', ' ');
-			}
-		}
-	}
+    // effectuer la suppression si demandee d'un logo
+    $on = _request('supprimer_logo_on');
+    if ($on or _request('supprimer_logo_off')) {
+        logo_supprimer($objet, $id_objet, $on ? 'on' : 'off');
+        $res['message_ok'] = ''; // pas besoin de message : la validation est visuelle
+        set_request('logo_up', ' ');
+    } // sinon supprimer ancien logo puis copier le nouveau
+    else {
+        $sources = formulaire_editer_logo_get_sources();
+        foreach ($sources as $etat => $file) {
+            if ($file and $file['error'] == 0) {
+                if ($err = logo_modifier($objet, $id_objet, $etat, $file)) {
+                    $res['message_erreur'] = $err;
+                } else {
+                    $res['message_ok'] = '';
+                } // pas besoin de message : la validation est visuelle
+                set_request('logo_up', ' ');
+            }
+        }
+    }
 
-	// Invalider les caches de l'objet
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='$objet/$id_objet'");
+    // Invalider les caches de l'objet
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='$objet/$id_objet'");
 
 
-	if ($retour) {
-		$res['redirect'] = $retour;
-	}
+    if ($retour) {
+        $res['redirect'] = $retour;
+    }
 
-	return $res;
+    return $res;
 }
 
 
@@ -255,29 +255,29 @@ 
  *     Sources des fichiers dans les clés `on` ou `off`
  */
 function formulaire_editer_logo_get_sources() {
-	if (!$_FILES) {
-		$_FILES = $GLOBALS['HTTP_POST_FILES'] ?? [];
-	}
-	if (!is_array($_FILES)) {
-		return [];
-	}
+    if (!$_FILES) {
+        $_FILES = $GLOBALS['HTTP_POST_FILES'] ?? [];
+    }
+    if (!is_array($_FILES)) {
+        return [];
+    }
 
-	include_spip('inc/documents');
-	$sources = [];
-	foreach (['on', 'off'] as $etat) {
-		$logo = 'logo_' . $etat;
-		if (isset($_FILES[$logo])) {
-			if ($_FILES[$logo]['error'] == 0) {
-				$sources[$etat] = $_FILES[$logo];
-			} elseif ($_FILES[$logo]['error'] != 0) {
-				$msg = check_upload_error($_FILES[$logo]['error'], false, true);
-				if ($msg and is_string($msg)) {
-					$sources[$etat] = $_FILES[$logo];
-					$sources[$etat]['msg'] = $msg;
-				}
-			}
-		}
-	}
+    include_spip('inc/documents');
+    $sources = [];
+    foreach (['on', 'off'] as $etat) {
+        $logo = 'logo_' . $etat;
+        if (isset($_FILES[$logo])) {
+            if ($_FILES[$logo]['error'] == 0) {
+                $sources[$etat] = $_FILES[$logo];
+            } elseif ($_FILES[$logo]['error'] != 0) {
+                $msg = check_upload_error($_FILES[$logo]['error'], false, true);
+                if ($msg and is_string($msg)) {
+                    $sources[$etat] = $_FILES[$logo];
+                    $sources[$etat]['msg'] = $msg;
+                }
+            }
+        }
+    }
 
-	return $sources;
+    return $sources;
 }