spip / SPIP

ecrire/action/editer_rubrique.php

Spacing +2 added lines, -2 removed lines

@@ -142,7 +142,7 @@ 
 	if ($err = objet_modifier_champs('rubrique', $id_rubrique,
 		array(
 			'data' => $set,
-			'nonvide' => array('titre' => _T('titre_nouvelle_rubrique') . " " . _T('info_numero_abbreviation') . $id_rubrique)
+			'nonvide' => array('titre' => _T('titre_nouvelle_rubrique')." "._T('info_numero_abbreviation').$id_rubrique)
 		),
 		$c)
 	) {
@@ -240,7 +240,7 @@ 
 			)
 			) {
 				if ($s['statut'] != 'new') {
-					spip_log("deplacement de $id_rubrique vers $id_parent refuse a " . $GLOBALS['visiteur_session']['id_auteur'] . ' ' . $GLOBALS['visiteur_session']['statut']);
+					spip_log("deplacement de $id_rubrique vers $id_parent refuse a ".$GLOBALS['visiteur_session']['id_auteur'].' '.$GLOBALS['visiteur_session']['statut']);
 				}
 			} elseif (editer_rubrique_breves($id_rubrique, $id_parent, $c)) {
 				$statut_ancien = $s['statut'];

Doc Comments +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
  *     Identifiant de la rubrique parente.
  *     0 pour la racine.
  * @param array|null $set
- * @return int
+ * @return boolean|string
  *     Identifiant de la rubrique crée
  */
 function rubrique_inserer($id_parent, $set = null) {
@@ -120,7 +120,7 @@ 
  *     Identifiant de la rubrique modifiée
  * @param array|null $set
  *     Tableau qu'on peut proposer en lieu et place de _request()
- * @return bool|string
+ * @return string|false
  *     - false  : Aucune modification, aucun champ n'est à modifier
  *     - chaîne vide : Vide si tout s'est bien passé
  *     - chaîne : Texte d'un message d'erreur

Indentation +148 added lines, -148 removed lines

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

ecrire/action/purger_queue.php

Indentation +7 added lines, -7 removed lines

@@ -16,7 +16,7 @@ 
  * @package SPIP\Core\Queue
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -25,12 +25,12 @@ 
  * @return void
  */
 function action_purger_queue_dist() {
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $securiser_action();
 
-	if (autoriser('purger', 'queue')) {
-		include_spip('inc/queue');
-		queue_purger();
-	}
+    if (autoriser('purger', 'queue')) {
+        include_spip('inc/queue');
+        queue_purger();
+    }
 
 }

ecrire/action/auth.php

Indentation +22 added lines, -22 removed lines

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

ecrire/action/desinstaller_plugin.php

Indentation +25 added lines, -25 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -32,28 +32,28 @@ 
  */
 function action_desinstaller_plugin_dist() {
 
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$arg = $securiser_action();
-	list($dir_plugins, $plugin) = explode("::", $arg);
-	$dir_type = "_DIR_PLUGINS";
-	if (defined('_DIR_PLUGINS_SUPPL') and $dir_plugins == _DIR_PLUGINS_SUPPL) {
-		$dir_type = "_DIR_PLUGINS_SUPPL";
-	}
-	$installer_plugins = charger_fonction('installer', 'plugins');
-	$infos = $installer_plugins($plugin, 'uninstall', $dir_type);
-	if ($infos and !$infos['install_test'][0]) {
-		include_spip('inc/plugin');
-		ecrire_plugin_actifs(array($plugin), false, 'enleve');
-		$erreur = '';
-	} else {
-		$erreur = 'erreur_plugin_desinstalation_echouee';
-	}
-	if ($redirect = _request('redirect')) {
-		include_spip('inc/headers');
-		if ($erreur) {
-			$redirect = parametre_url($redirect, 'erreur', $erreur);
-		}
-		$redirect = str_replace('&', '&', $redirect);
-		redirige_par_entete($redirect);
-	}
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $arg = $securiser_action();
+    list($dir_plugins, $plugin) = explode("::", $arg);
+    $dir_type = "_DIR_PLUGINS";
+    if (defined('_DIR_PLUGINS_SUPPL') and $dir_plugins == _DIR_PLUGINS_SUPPL) {
+        $dir_type = "_DIR_PLUGINS_SUPPL";
+    }
+    $installer_plugins = charger_fonction('installer', 'plugins');
+    $infos = $installer_plugins($plugin, 'uninstall', $dir_type);
+    if ($infos and !$infos['install_test'][0]) {
+        include_spip('inc/plugin');
+        ecrire_plugin_actifs(array($plugin), false, 'enleve');
+        $erreur = '';
+    } else {
+        $erreur = 'erreur_plugin_desinstalation_echouee';
+    }
+    if ($redirect = _request('redirect')) {
+        include_spip('inc/headers');
+        if ($erreur) {
+            $redirect = parametre_url($redirect, 'erreur', $erreur);
+        }
+        $redirect = str_replace('&', '&', $redirect);
+        redirige_par_entete($redirect);
+    }
 }

ecrire/action/cookie.php

Spacing +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
 	if ($change_session == 'oui') {
 		$session = charger_fonction('session', 'inc');
 		$session(true);
-		spip_log("statut 204 pour " . $_SERVER['REQUEST_URI']);
+		spip_log("statut 204 pour ".$_SERVER['REQUEST_URI']);
 		http_status(204); // No Content
 		return;
 	}
@@ -77,7 +77,7 @@ 
 			ask_php_auth(_T('info_connexion_refusee'),
 				_T('login_login_pass_incorrect'),
 				_T('login_retour_site'),
-				"url=" . rawurlencode($redirect),
+				"url=".rawurlencode($redirect),
 				_T('login_nouvelle_tentative'),
 				(strpos($url, _DIR_RESTREINT_ABS) !== false));
 		}

Indentation +78 added lines, -78 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -33,88 +33,88 @@ 
  * @return void
  */
 function action_cookie_dist($set_cookie_admin = null, $change_session = null) {
-	$redirect_echec = $redirect = null;
-	$test_echec_cookie = null;
-	$url = "";
-	if (is_null($set_cookie_admin)) {
-		$set_cookie_admin = _request('cookie_admin');
-		$change_session = _request('change_session');
-		$test_echec_cookie = _request('test_echec_cookie');
+    $redirect_echec = $redirect = null;
+    $test_echec_cookie = null;
+    $url = "";
+    if (is_null($set_cookie_admin)) {
+        $set_cookie_admin = _request('cookie_admin');
+        $change_session = _request('change_session');
+        $test_echec_cookie = _request('test_echec_cookie');
 
-		// La cible de notre operation de connexion
-		$url = securiser_redirect_action(_request('url'));
-		$redirect = $url ? $url : generer_url_ecrire('accueil');
-		$redirect_echec = _request('url_echec');
-		if (!isset($redirect_echec)) {
-			if (strpos($redirect, _DIR_RESTREINT_ABS) !== false) {
-				$redirect_echec = generer_url_public('login', '', true);
-			} else {
-				$redirect_echec = $redirect;
-			}
-		}
-	}
+        // La cible de notre operation de connexion
+        $url = securiser_redirect_action(_request('url'));
+        $redirect = $url ? $url : generer_url_ecrire('accueil');
+        $redirect_echec = _request('url_echec');
+        if (!isset($redirect_echec)) {
+            if (strpos($redirect, _DIR_RESTREINT_ABS) !== false) {
+                $redirect_echec = generer_url_public('login', '', true);
+            } else {
+                $redirect_echec = $redirect;
+            }
+        }
+    }
 
 
-	// rejoue le cookie pour renouveler spip_session
-	if ($change_session == 'oui') {
-		$session = charger_fonction('session', 'inc');
-		$session(true);
-		spip_log("statut 204 pour " . $_SERVER['REQUEST_URI']);
-		http_status(204); // No Content
-		return;
-	}
+    // rejoue le cookie pour renouveler spip_session
+    if ($change_session == 'oui') {
+        $session = charger_fonction('session', 'inc');
+        $session(true);
+        spip_log("statut 204 pour " . $_SERVER['REQUEST_URI']);
+        http_status(204); // No Content
+        return;
+    }
 
-	// tentative de connexion en auth_http
-	if (_request('essai_auth_http') and !$GLOBALS['ignore_auth_http']) {
-		include_spip('inc/auth');
-		if (@$_SERVER['PHP_AUTH_USER']
-			and @$_SERVER['PHP_AUTH_PW']
-			and $auteur = lire_php_auth($_SERVER['PHP_AUTH_USER'], $_SERVER['PHP_AUTH_PW'])
-		) {
-			auth_loger($auteur);
-			redirige_par_entete(parametre_url($redirect, 't', time(), '&'));
-		} else {
-			ask_php_auth(_T('info_connexion_refusee'),
-				_T('login_login_pass_incorrect'),
-				_T('login_retour_site'),
-				"url=" . rawurlencode($redirect),
-				_T('login_nouvelle_tentative'),
-				(strpos($url, _DIR_RESTREINT_ABS) !== false));
-		}
-	} else {
+    // tentative de connexion en auth_http
+    if (_request('essai_auth_http') and !$GLOBALS['ignore_auth_http']) {
+        include_spip('inc/auth');
+        if (@$_SERVER['PHP_AUTH_USER']
+            and @$_SERVER['PHP_AUTH_PW']
+            and $auteur = lire_php_auth($_SERVER['PHP_AUTH_USER'], $_SERVER['PHP_AUTH_PW'])
+        ) {
+            auth_loger($auteur);
+            redirige_par_entete(parametre_url($redirect, 't', time(), '&'));
+        } else {
+            ask_php_auth(_T('info_connexion_refusee'),
+                _T('login_login_pass_incorrect'),
+                _T('login_retour_site'),
+                "url=" . rawurlencode($redirect),
+                _T('login_nouvelle_tentative'),
+                (strpos($url, _DIR_RESTREINT_ABS) !== false));
+        }
+    } else {
 
-		// en cas de login sur bonjour=oui, on tente de poser un cookie
-		// puis de passer au login qui diagnostiquera l'echec de cookie
-		// le cas echeant.
-		if ($test_echec_cookie == 'oui') {
-			spip_setcookie('spip_session', 'test_echec_cookie');
-			if ($redirect) {
-				$redirect = parametre_url(parametre_url($redirect_echec, 'var_echec_cookie', 'oui', '&'), 'url',
-					rawurlencode($redirect), '&');
-			}
-		} else {
+        // en cas de login sur bonjour=oui, on tente de poser un cookie
+        // puis de passer au login qui diagnostiquera l'echec de cookie
+        // le cas echeant.
+        if ($test_echec_cookie == 'oui') {
+            spip_setcookie('spip_session', 'test_echec_cookie');
+            if ($redirect) {
+                $redirect = parametre_url(parametre_url($redirect_echec, 'var_echec_cookie', 'oui', '&'), 'url',
+                    rawurlencode($redirect), '&');
+            }
+        } else {
 
-			$cook = isset($_COOKIE['spip_admin']) ? $_COOKIE['spip_admin'] : '';
-			// Suppression cookie d'admin ?
-			if ($set_cookie_admin == "non") {
-				if ($cook) {
-					spip_setcookie('spip_admin', $cook, [
-						'expires' => time() - 3600 * 24
-					]);
-				}
-			} // Ajout de cookie d'admin
-			else {
-				if ($set_cookie_admin and _DUREE_COOKIE_ADMIN) {
-					spip_setcookie('spip_admin', $set_cookie_admin, [
-						'expires' => time() + max(_DUREE_COOKIE_ADMIN, 2 * _RENOUVELLE_ALEA)
-					]);
-				}
-			}
-		}
-	}
+            $cook = isset($_COOKIE['spip_admin']) ? $_COOKIE['spip_admin'] : '';
+            // Suppression cookie d'admin ?
+            if ($set_cookie_admin == "non") {
+                if ($cook) {
+                    spip_setcookie('spip_admin', $cook, [
+                        'expires' => time() - 3600 * 24
+                    ]);
+                }
+            } // Ajout de cookie d'admin
+            else {
+                if ($set_cookie_admin and _DUREE_COOKIE_ADMIN) {
+                    spip_setcookie('spip_admin', $set_cookie_admin, [
+                        'expires' => time() + max(_DUREE_COOKIE_ADMIN, 2 * _RENOUVELLE_ALEA)
+                    ]);
+                }
+            }
+        }
+    }
 
-	// Redirection finale
-	if ($redirect) {
-		redirige_par_entete($redirect, true);
-	}
+    // Redirection finale
+    if ($redirect) {
+        redirige_par_entete($redirect, true);
+    }
 }

ecrire/action/instituer_langue_rubrique.php

Indentation +31 added lines, -31 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -25,34 +25,34 @@ 
  **/
 function action_instituer_langue_rubrique_dist() {
 
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$arg = $securiser_action();
-	$changer_lang = _request('changer_lang');
-
-	list($id_rubrique, $id_parent) = preg_split('/\W/', $arg);
-
-	if ($changer_lang
-		and $id_rubrique > 0
-		and $GLOBALS['meta']['multi_rubriques'] == 'oui'
-		and ($GLOBALS['meta']['multi_secteurs'] == 'non' or $id_parent == 0)
-	) {
-		if ($changer_lang != "herit") {
-			sql_updateq('spip_rubriques', array('lang' => $changer_lang, 'langue_choisie' => 'oui'),
-				"id_rubrique=$id_rubrique");
-		} else {
-			if ($id_parent == 0) {
-				$langue_parent = $GLOBALS['meta']['langue_site'];
-			} else {
-				$langue_parent = sql_getfetsel("lang", "spip_rubriques", "id_rubrique=$id_parent");
-			}
-			sql_updateq('spip_rubriques', array('lang' => $langue_parent, 'langue_choisie' => 'non'),
-				"id_rubrique=$id_rubrique");
-		}
-		include_spip('inc/rubriques');
-		calculer_langues_rubriques();
-
-		// invalider les caches marques de cette rubrique
-		include_spip('inc/invalideur');
-		suivre_invalideur("id='rubrique/$id_rubrique'");
-	}
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $arg = $securiser_action();
+    $changer_lang = _request('changer_lang');
+
+    list($id_rubrique, $id_parent) = preg_split('/\W/', $arg);
+
+    if ($changer_lang
+        and $id_rubrique > 0
+        and $GLOBALS['meta']['multi_rubriques'] == 'oui'
+        and ($GLOBALS['meta']['multi_secteurs'] == 'non' or $id_parent == 0)
+    ) {
+        if ($changer_lang != "herit") {
+            sql_updateq('spip_rubriques', array('lang' => $changer_lang, 'langue_choisie' => 'oui'),
+                "id_rubrique=$id_rubrique");
+        } else {
+            if ($id_parent == 0) {
+                $langue_parent = $GLOBALS['meta']['langue_site'];
+            } else {
+                $langue_parent = sql_getfetsel("lang", "spip_rubriques", "id_rubrique=$id_parent");
+            }
+            sql_updateq('spip_rubriques', array('lang' => $langue_parent, 'langue_choisie' => 'non'),
+                "id_rubrique=$id_rubrique");
+        }
+        include_spip('inc/rubriques');
+        calculer_langues_rubriques();
+
+        // invalider les caches marques de cette rubrique
+        include_spip('inc/invalideur');
+        suivre_invalideur("id='rubrique/$id_rubrique'");
+    }
 }

ecrire/action/debloquer_edition.php

Indentation +13 added lines, -13 removed lines

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

ecrire/action/confirmer_email.php

Indentation +15 added lines, -15 removed lines

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

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 	// verifier avant de rediriger pour invalider le message de confirmation
 	// si ca n'a pas marche
 	if ($redirect = _request('redirect') and !$arg == sql_getfetsel('email', 'spip_auteurs',
-			'id_auteur=' . intval($GLOBALS['visiteur_session']))
+			'id_auteur='.intval($GLOBALS['visiteur_session']))
 	) {
 		$GLOBALS['redirect'] = parametre_url($redirect, 'email_modif', '');
 	}

ecrire/action/confirmer_inscription.php

Indentation +43 added lines, -43 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -28,52 +28,52 @@ 
  * @return void
  */
 function action_confirmer_inscription_dist() {
-	$jeton = _request('jeton');
-	$email = _request('email');
+    $jeton = _request('jeton');
+    $email = _request('email');
 
-	include_spip('action/inscrire_auteur');
-	if ($auteur = auteur_verifier_jeton($jeton)
-		and $auteur['email'] == $email
-		and $auteur['statut'] == 'nouveau'
-	) {
+    include_spip('action/inscrire_auteur');
+    if ($auteur = auteur_verifier_jeton($jeton)
+        and $auteur['email'] == $email
+        and $auteur['statut'] == 'nouveau'
+    ) {
 		
-		// d'abord on confirme son statut
-		$auteur = confirmer_statut_inscription($auteur);
+        // d'abord on confirme son statut
+        $auteur = confirmer_statut_inscription($auteur);
 
-		// OK c'est un nouvel inscrit qui confirme :
-		// on le loge => ca va confirmer son statut et c'est plus sympa
-		include_spip('inc/auth');
-		auth_loger($auteur);
+        // OK c'est un nouvel inscrit qui confirme :
+        // on le loge => ca va confirmer son statut et c'est plus sympa
+        include_spip('inc/auth');
+        auth_loger($auteur);
 
-		// et on efface son jeton
-		auteur_effacer_jeton($auteur['id_auteur']);
+        // et on efface son jeton
+        auteur_effacer_jeton($auteur['id_auteur']);
 
-		// si pas de redirection demandee, rediriger vers public ou prive selon le statut de l'auteur
-		// TODO: ne semble pas marcher si inscrit non visiteur, a debug
-		if (!_request('redirect')) {
-			// on passe id_auteur explicite pour forcer une lecture en base de toutes les infos
-			if (autoriser('ecrire', '', '', $auteur['id_auteur'])) {
-				// poser un cookie admin aussi
-				$cookie = charger_fonction('cookie', 'action');
-				$cookie("@" . $GLOBALS['visiteur_session']['login']);
-				$GLOBALS['redirect'] = _DIR_RESTREINT_ABS;
-			} else {
-				$GLOBALS['redirect'] = $GLOBALS['meta']['adresse_site'];
-			}
-		}
-	} else {
-		// lien perime :
-		if ($GLOBALS['visiteur_session']['id_auteur']) {
-			// on passe id_auteur explicite pour forcer une lecture en base de toutes les infos
-			if (autoriser('ecrire', '', '', $GLOBALS['visiteur_session']['id_auteur'])) {
-				$GLOBALS['redirect'] = _DIR_RESTREINT_ABS;
-			} else {
-				$GLOBALS['redirect'] = $GLOBALS['meta']['adresse_site'];
-			}
-		} else // rediriger vers la page de login si pas encore loge
-		{
-			$GLOBALS['redirect'] = parametre_url(generer_url_public('login', '', false), 'url', _request('redirect'));
-		}
-	}
+        // si pas de redirection demandee, rediriger vers public ou prive selon le statut de l'auteur
+        // TODO: ne semble pas marcher si inscrit non visiteur, a debug
+        if (!_request('redirect')) {
+            // on passe id_auteur explicite pour forcer une lecture en base de toutes les infos
+            if (autoriser('ecrire', '', '', $auteur['id_auteur'])) {
+                // poser un cookie admin aussi
+                $cookie = charger_fonction('cookie', 'action');
+                $cookie("@" . $GLOBALS['visiteur_session']['login']);
+                $GLOBALS['redirect'] = _DIR_RESTREINT_ABS;
+            } else {
+                $GLOBALS['redirect'] = $GLOBALS['meta']['adresse_site'];
+            }
+        }
+    } else {
+        // lien perime :
+        if ($GLOBALS['visiteur_session']['id_auteur']) {
+            // on passe id_auteur explicite pour forcer une lecture en base de toutes les infos
+            if (autoriser('ecrire', '', '', $GLOBALS['visiteur_session']['id_auteur'])) {
+                $GLOBALS['redirect'] = _DIR_RESTREINT_ABS;
+            } else {
+                $GLOBALS['redirect'] = $GLOBALS['meta']['adresse_site'];
+            }
+        } else // rediriger vers la page de login si pas encore loge
+        {
+            $GLOBALS['redirect'] = parametre_url(generer_url_public('login', '', false), 'url', _request('redirect'));
+        }
+    }
 
 }

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 			if (autoriser('ecrire', '', '', $auteur['id_auteur'])) {
 				// poser un cookie admin aussi
 				$cookie = charger_fonction('cookie', 'action');
-				$cookie("@" . $GLOBALS['visiteur_session']['login']);
+				$cookie("@".$GLOBALS['visiteur_session']['login']);
 				$GLOBALS['redirect'] = _DIR_RESTREINT_ABS;
 			} else {
 				$GLOBALS['redirect'] = $GLOBALS['meta']['adresse_site'];