spip / SPIP

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

Indentation +74 added lines, -74 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -33,84 +33,84 @@ 
  * @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, time() - 3600 * 24);
-				}
-			} // Ajout de cookie d'admin
-			else {
-				if ($set_cookie_admin and _DUREE_COOKIE_ADMIN) {
-					spip_setcookie('spip_admin', $set_cookie_admin, 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, time() - 3600 * 24);
+                }
+            } // Ajout de cookie d'admin
+            else {
+                if ($set_cookie_admin and _DUREE_COOKIE_ADMIN) {
+                    spip_setcookie('spip_admin', $set_cookie_admin, 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);
+    }
 }

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));
 		}

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/super_cron.php

Indentation +21 added lines, -21 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -33,26 +33,26 @@ 
  * @see action_cron() URL appelée en asynchrone pour excécuter le cron
  */
 function action_super_cron_dist() {
-	// Si fsockopen est possible, on lance le cron via un socket
-	// en asynchrone
-	if (function_exists('fsockopen')) {
-		$url = generer_url_action('cron');
-		$parts = parse_url($url);
-		$fp = fsockopen($parts['host'],
-			isset($parts['port']) ? $parts['port'] : 80,
-			$errno, $errstr, 30);
-		if ($fp) {
-			$out = "GET " . $parts['path'] . "?" . $parts['query'] . " HTTP/1.1\r\n";
-			$out .= "Host: " . $parts['host'] . "\r\n";
-			$out .= "Connection: Close\r\n\r\n";
-			fwrite($fp, $out);
-			fclose($fp);
+    // Si fsockopen est possible, on lance le cron via un socket
+    // en asynchrone
+    if (function_exists('fsockopen')) {
+        $url = generer_url_action('cron');
+        $parts = parse_url($url);
+        $fp = fsockopen($parts['host'],
+            isset($parts['port']) ? $parts['port'] : 80,
+            $errno, $errstr, 30);
+        if ($fp) {
+            $out = "GET " . $parts['path'] . "?" . $parts['query'] . " HTTP/1.1\r\n";
+            $out .= "Host: " . $parts['host'] . "\r\n";
+            $out .= "Connection: Close\r\n\r\n";
+            fwrite($fp, $out);
+            fclose($fp);
 
-			return;
-		}
-	}
-	// ici lancer le cron par un CURL asynchrone si CURL est présent
-	// TBD
+            return;
+        }
+    }
+    // ici lancer le cron par un CURL asynchrone si CURL est présent
+    // TBD
 
-	return;
+    return;
 }

Spacing +2 added lines, -2 removed lines

@@ -42,8 +42,8 @@
 			isset($parts['port']) ? $parts['port'] : 80,
 			$errno, $errstr, 30);
 		if ($fp) {
-			$out = "GET " . $parts['path'] . "?" . $parts['query'] . " HTTP/1.1\r\n";
-			$out .= "Host: " . $parts['host'] . "\r\n";
+			$out = "GET ".$parts['path']."?".$parts['query']." HTTP/1.1\r\n";
+			$out .= "Host: ".$parts['host']."\r\n";
 			$out .= "Connection: Close\r\n\r\n";
 			fwrite($fp, $out);
 			fclose($fp);

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'];

ecrire/action/menu_rubriques.php

Indentation +126 added lines, -126 removed lines

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

Spacing +4 added lines, -4 removed lines

@@ -38,14 +38,14 @@ 
 	// si pas acces a ecrire, pas acces au menu
 	// on renvoi un 401 qui fait echouer la requete ajax silencieusement
 	if (!autoriser('ecrire')) {
-		$retour = "<ul class='cols_1'><li class='toutsite'><a href='" . generer_url_ecrire('accueil') . "'>" . _T('public:lien_connecter') . "</a></li></ul>";
+		$retour = "<ul class='cols_1'><li class='toutsite'><a href='".generer_url_ecrire('accueil')."'>"._T('public:lien_connecter')."</a></li></ul>";
 		include_spip('inc/actions');
 		ajax_retour($retour);
 		exit;
 	}
 
 	if ($date = intval(_request('date'))) {
-		header("Last-Modified: " . gmdate("D, d M Y H:i:s", $date) . " GMT");
+		header("Last-Modified: ".gmdate("D, d M Y H:i:s", $date)." GMT");
 	}
 
 	$r = gen_liste_rubriques();
@@ -54,7 +54,7 @@ 
 		and !strstr($_SERVER['SERVER_SOFTWARE'], 'IIS/')
 	) {
 		include_spip('inc/headers');
-		header('Content-Type: text/html; charset=' . $GLOBALS['meta']['charset']);
+		header('Content-Type: text/html; charset='.$GLOBALS['meta']['charset']);
 		http_status(304);
 		exit;
 	} else {
@@ -75,7 +75,7 @@ 
  *     Code HTML présentant la liste des rubriques
  **/
 function menu_rubriques($complet = true) {
-	$ret = "<li class='toutsite'><a href='" . generer_url_ecrire('plan') . "'>" . _T('info_tout_site') . "</a></li>";
+	$ret = "<li class='toutsite'><a href='".generer_url_ecrire('plan')."'>"._T('info_tout_site')."</a></li>";
 
 	if (!$complet) {
 		return "<ul class='cols_1'>$ret\n</ul>\n";