spip / SPIP

ecrire/action/super_cron.php

Indentation +20 added lines, -20 removed lines

@@ -25,26 +25,26 @@
  * "* * * * * curl  http://www.mondomaine.tld/spip.php?action=super_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);
-			return;
-		}
-	}
-	// ici lancer le cron par un CURL asynchrone si CURL est présent
-	// TBD
+    // 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;
+    return;
 }
 ?>
\ No newline at end of file

Spacing +6 added lines, -6 removed lines

@@ -24,19 +24,19 @@
  * Exemple de tache cron Unix pour un appel toutes les minutes :
  * "* * * * * curl  http://www.mondomaine.tld/spip.php?action=super_cron"
  */
-function action_super_cron_dist(){
+function action_super_cron_dist() {
 	// Si fsockopen est possible, on lance le cron via un socket
 	// en asynchrone
-	if(function_exists('fsockopen')){
+	if (function_exists('fsockopen')) {
 		$url = generer_url_action('cron');
-		$parts=parse_url($url);
+		$parts = parse_url($url);
 		$fp = fsockopen($parts['host'],
-	        isset($parts['port'])?$parts['port']:80,
+	        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";
+    		$out .= "Host: ".$parts['host']."\r\n";
+    		$out .= "Connection: Close\r\n\r\n";
 			fwrite($fp, $out);
 			fclose($fp);
 			return;

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * Url pour lancer le cron de manière asynchrone si le serveur

ecrire/action/auth.php

Indentation +22 added lines, -22 removed lines

@@ -17,28 +17,28 @@
  */
 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);
+    }
 }
 
 

Spacing +2 added lines, -2 removed lines

@@ -29,9 +29,9 @@
 		include_spip('inc/auth');
 		$res = auth_terminer_identifier_login($auth_methode, $login);
 
-		if (is_string($res)){ // Erreur
+		if (is_string($res)) { // Erreur
 			$redirect = _request('redirect');
-			$redirect = parametre_url($redirect,'var_erreur',$res,'&');
+			$redirect = parametre_url($redirect, 'var_erreur', $res, '&');
 			include_spip('inc/headers');
 			redirige_par_entete($redirect);
 		}

Braces +4 added lines, -3 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * Retour d'authentification pour les SSO
@@ -22,8 +24,7 @@ 
 
 	if (!preg_match(",^(\w+)[/](.+)$,", $arg, $r)) {
 		spip_log("action_auth_dist $arg pas compris");
-	}
-	else {
+	} else {
 		$auth_methode = $r[1];
 		$login = $r[2];
 		include_spip('inc/auth');

ecrire/action/purger.php

Indentation +43 added lines, -43 removed lines

@@ -15,56 +15,56 @@
 // http://doc.spip.org/@action_purger_dist
 function action_purger_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();
+    }
 
-	include_spip('inc/invalideur');
+    include_spip('inc/invalideur');
 
-	spip_log("purger $arg");
+    spip_log("purger $arg");
 
-	switch ($arg) {
-		case 'inhibe_cache':
-			// inhiber le cache pendant 24h
-			ecrire_meta('cache_inhib',$_SERVER['REQUEST_TIME']+24*3600);
-			break;
-		case 'reactive_cache':
-			effacer_meta('cache_inhib');
-			break;
+    switch ($arg) {
+        case 'inhibe_cache':
+            // inhiber le cache pendant 24h
+            ecrire_meta('cache_inhib',$_SERVER['REQUEST_TIME']+24*3600);
+            break;
+        case 'reactive_cache':
+            effacer_meta('cache_inhib');
+            break;
 
-		case 'cache':
-			supprime_invalideurs();
-			@spip_unlink(_CACHE_RUBRIQUES);
-			@spip_unlink(_CACHE_CHEMIN);
-			@spip_unlink(_DIR_TMP."plugin_xml_cache.gz");
-			// on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
-			// _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
-			// pour eviter des problemes de concurence
-			// cf http://core.spip.org/issues/2989
-			//@spip_unlink(_CACHE_PIPELINES);
-			//@spip_unlink(_CACHE_PLUGINS_PATH);
-			//@spip_unlink(_CACHE_PLUGINS_FCT);
-			@spip_unlink(_CACHE_PLUGINS_OPT);
-			purger_repertoire(_DIR_CACHE,array('subdir'=>true));
-			purger_repertoire(_DIR_AIDE);
-			purger_repertoire(_DIR_VAR.'cache-css');
-			purger_repertoire(_DIR_VAR.'cache-js');
-			break;
+        case 'cache':
+            supprime_invalideurs();
+            @spip_unlink(_CACHE_RUBRIQUES);
+            @spip_unlink(_CACHE_CHEMIN);
+            @spip_unlink(_DIR_TMP."plugin_xml_cache.gz");
+            // on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
+            // _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
+            // pour eviter des problemes de concurence
+            // cf http://core.spip.org/issues/2989
+            //@spip_unlink(_CACHE_PIPELINES);
+            //@spip_unlink(_CACHE_PLUGINS_PATH);
+            //@spip_unlink(_CACHE_PLUGINS_FCT);
+            @spip_unlink(_CACHE_PLUGINS_OPT);
+            purger_repertoire(_DIR_CACHE,array('subdir'=>true));
+            purger_repertoire(_DIR_AIDE);
+            purger_repertoire(_DIR_VAR.'cache-css');
+            purger_repertoire(_DIR_VAR.'cache-js');
+            break;
 
-		case 'squelettes':
-			purger_repertoire(_DIR_SKELS);
-			break;
+        case 'squelettes':
+            purger_repertoire(_DIR_SKELS);
+            break;
 
-		case 'vignettes':
-			purger_repertoire(_DIR_VAR,array('subdir'=>true));
-			supprime_invalideurs();
-			purger_repertoire(_DIR_CACHE);
-			break;
-	}
+        case 'vignettes':
+            purger_repertoire(_DIR_VAR,array('subdir'=>true));
+            supprime_invalideurs();
+            purger_repertoire(_DIR_CACHE);
+            break;
+    }
 
-	// le faire savoir aux plugins
-	pipeline('trig_purger',$arg);
+    // le faire savoir aux plugins
+    pipeline('trig_purger',$arg);
 }
 
 ?>

Spacing +5 added lines, -5 removed lines

@@ -13,7 +13,7 @@ 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 // http://doc.spip.org/@action_purger_dist
-function action_purger_dist($arg=null)
+function action_purger_dist($arg = null)
 {
 	if (is_null($arg)) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
@@ -27,7 +27,7 @@ 
 	switch ($arg) {
 		case 'inhibe_cache':
 			// inhiber le cache pendant 24h
-			ecrire_meta('cache_inhib',$_SERVER['REQUEST_TIME']+24*3600);
+			ecrire_meta('cache_inhib', $_SERVER['REQUEST_TIME'] + 24 * 3600);
 			break;
 		case 'reactive_cache':
 			effacer_meta('cache_inhib');
@@ -46,7 +46,7 @@ 
 			//@spip_unlink(_CACHE_PLUGINS_PATH);
 			//@spip_unlink(_CACHE_PLUGINS_FCT);
 			@spip_unlink(_CACHE_PLUGINS_OPT);
-			purger_repertoire(_DIR_CACHE,array('subdir'=>true));
+			purger_repertoire(_DIR_CACHE, array('subdir'=>true));
 			purger_repertoire(_DIR_AIDE);
 			purger_repertoire(_DIR_VAR.'cache-css');
 			purger_repertoire(_DIR_VAR.'cache-js');
@@ -57,14 +57,14 @@ 
 			break;
 
 		case 'vignettes':
-			purger_repertoire(_DIR_VAR,array('subdir'=>true));
+			purger_repertoire(_DIR_VAR, array('subdir'=>true));
 			supprime_invalideurs();
 			purger_repertoire(_DIR_CACHE);
 			break;
 	}
 
 	// le faire savoir aux plugins
-	pipeline('trig_purger',$arg);
+	pipeline('trig_purger', $arg);
 }
 
 ?>

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // http://doc.spip.org/@action_purger_dist
 function action_purger_dist($arg=null)

ecrire/action/iconifier.php

Indentation +85 added lines, -85 removed lines

@@ -18,30 +18,30 @@ 
  */
 function action_iconifier_dist()
 {
-	include_spip('inc/actions');
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$arg = $securiser_action();
-	$iframe_redirect = _request('iframe_redirect');
+    include_spip('inc/actions');
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $arg = $securiser_action();
+    $iframe_redirect = _request('iframe_redirect');
 
-	$arg = rawurldecode($arg);
+    $arg = rawurldecode($arg);
 
-	if (!preg_match(',^-?\d*(\D)(.*)$,',$arg, $r))
-		spip_log("action iconifier: $arg pas compris");
-	elseif ($r[1] == '+')
-		action_spip_image_ajouter_dist($r[2], _request('sousaction2'), _request('source'));
-	else	action_spip_image_effacer_dist($r[2]);
+    if (!preg_match(',^-?\d*(\D)(.*)$,',$arg, $r))
+        spip_log("action iconifier: $arg pas compris");
+    elseif ($r[1] == '+')
+        action_spip_image_ajouter_dist($r[2], _request('sousaction2'), _request('source'));
+    else	action_spip_image_effacer_dist($r[2]);
 	
-	if(_request("iframe") == 'iframe') {
-		$redirect = urldecode($iframe_redirect)."&iframe=iframe";
-		redirige_par_entete(urldecode($redirect));
-	}
+    if(_request("iframe") == 'iframe') {
+        $redirect = urldecode($iframe_redirect)."&iframe=iframe";
+        redirige_par_entete(urldecode($redirect));
+    }
 }
 
 // http://doc.spip.org/@action_spip_image_effacer_dist
 function action_spip_image_effacer_dist($arg) {
 
-	if (!strstr($arg, ".."))
-		spip_unlink(_DIR_LOGOS . $arg);
+    if (!strstr($arg, ".."))
+        spip_unlink(_DIR_LOGOS . $arg);
 }
 
 //
@@ -52,80 +52,80 @@ 
 // $dest = arton12.xxx
 // http://doc.spip.org/@action_spip_image_ajouter_dist
 function action_spip_image_ajouter_dist($arg,$sousaction2,$source,$return=false) {
-	global $formats_logos;
+    global $formats_logos;
 
-	include_spip('inc/documents');
-	if (!$sousaction2) {
-		if (!$_FILES) $_FILES = $GLOBALS['HTTP_POST_FILES'];
-		$source = (is_array($_FILES) ? array_pop($_FILES) : "");
-	}
-	$erreur = "";
-	if (!$source)
-		spip_log("spip_image_ajouter : source inconnue");
-	else {
-		$f =_DIR_LOGOS . $arg . '.tmp';
+    include_spip('inc/documents');
+    if (!$sousaction2) {
+        if (!$_FILES) $_FILES = $GLOBALS['HTTP_POST_FILES'];
+        $source = (is_array($_FILES) ? array_pop($_FILES) : "");
+    }
+    $erreur = "";
+    if (!$source)
+        spip_log("spip_image_ajouter : source inconnue");
+    else {
+        $f =_DIR_LOGOS . $arg . '.tmp';
 
-		if (!is_array($source)) 
-		// fichier dans upload/
-	  		$source = @copy(determine_upload() . $source, $f);
-		else {
-		// Intercepter une erreur a l'envoi
-			if ($erreur = check_upload_error($source['error'],"",$return))
-				$source ="";
-			else
-		// analyse le type de l'image (on ne fait pas confiance au nom de
-		// fichier envoye par le browser : pour les Macs c'est plus sur)
+        if (!is_array($source)) 
+        // fichier dans upload/
+                $source = @copy(determine_upload() . $source, $f);
+        else {
+        // Intercepter une erreur a l'envoi
+            if ($erreur = check_upload_error($source['error'],"",$return))
+                $source ="";
+            else
+        // analyse le type de l'image (on ne fait pas confiance au nom de
+        // fichier envoye par le browser : pour les Macs c'est plus sur)
 
-				$source = deplacer_fichier_upload($source['tmp_name'], $f);
-		}
-		if (!$source)
-			spip_log("pb de copie pour $f");
-	}
-	if ($source AND $f) {
-		$size = @getimagesize($f);
-		$type = !$size ? '': ($size[2] > 3 ? '' : $formats_logos[$size[2]-1]);
-		if ($type) {
-			$poids = filesize($f);
+                $source = deplacer_fichier_upload($source['tmp_name'], $f);
+        }
+        if (!$source)
+            spip_log("pb de copie pour $f");
+    }
+    if ($source AND $f) {
+        $size = @getimagesize($f);
+        $type = !$size ? '': ($size[2] > 3 ? '' : $formats_logos[$size[2]-1]);
+        if ($type) {
+            $poids = filesize($f);
 
-			if (_LOGO_MAX_SIZE > 0
-			AND $poids > _LOGO_MAX_SIZE*1024) {
-				spip_unlink ($f);
-				$erreur = _T('info_logo_max_poids',
-									array('maxi' => taille_en_octets(_LOGO_MAX_SIZE*1024),
-									'actuel' => taille_en_octets($poids)));
-			}
+            if (_LOGO_MAX_SIZE > 0
+            AND $poids > _LOGO_MAX_SIZE*1024) {
+                spip_unlink ($f);
+                $erreur = _T('info_logo_max_poids',
+                                    array('maxi' => taille_en_octets(_LOGO_MAX_SIZE*1024),
+                                    'actuel' => taille_en_octets($poids)));
+            }
 
-			elseif (_LOGO_MAX_WIDTH * _LOGO_MAX_HEIGHT
-			AND ($size[0] > _LOGO_MAX_WIDTH
-			OR $size[1] > _LOGO_MAX_HEIGHT)) {
-				spip_unlink ($f);
-				$erreur = _T('info_logo_max_poids',
-									array(
-									'maxi' =>
-										_T('info_largeur_vignette',
-											array('largeur_vignette' => _LOGO_MAX_WIDTH,
-											'hauteur_vignette' => _LOGO_MAX_HEIGHT)),
-									'actuel' =>
-										_T('info_largeur_vignette',
-											array('largeur_vignette' => $size[0],
-											'hauteur_vignette' => $size[1]))
-									));
-			}
-			else
-				@rename ($f, _DIR_LOGOS . $arg . ".$type");
-		}
-		else {
-			spip_unlink ($f);
-			$erreur = _T('info_logo_format_interdit',
-									array('formats' => join(', ', $formats_logos)));
-		}
+            elseif (_LOGO_MAX_WIDTH * _LOGO_MAX_HEIGHT
+            AND ($size[0] > _LOGO_MAX_WIDTH
+            OR $size[1] > _LOGO_MAX_HEIGHT)) {
+                spip_unlink ($f);
+                $erreur = _T('info_logo_max_poids',
+                                    array(
+                                    'maxi' =>
+                                        _T('info_largeur_vignette',
+                                            array('largeur_vignette' => _LOGO_MAX_WIDTH,
+                                            'hauteur_vignette' => _LOGO_MAX_HEIGHT)),
+                                    'actuel' =>
+                                        _T('info_largeur_vignette',
+                                            array('largeur_vignette' => $size[0],
+                                            'hauteur_vignette' => $size[1]))
+                                    ));
+            }
+            else
+                @rename ($f, _DIR_LOGOS . $arg . ".$type");
+        }
+        else {
+            spip_unlink ($f);
+            $erreur = _T('info_logo_format_interdit',
+                                    array('formats' => join(', ', $formats_logos)));
+        }
 	
-	}
-	if ($erreur){
-		if ($return)
-			return $erreur;
-		else
-			check_upload_error(6,$erreur);
-	}
+    }
+    if ($erreur){
+        if ($return)
+            return $erreur;
+        else
+            check_upload_error(6,$erreur);
+    }
 }
 ?>

Spacing +17 added lines, -17 removed lines

@@ -25,13 +25,13 @@ 
 
 	$arg = rawurldecode($arg);
 
-	if (!preg_match(',^-?\d*(\D)(.*)$,',$arg, $r))
+	if (!preg_match(',^-?\d*(\D)(.*)$,', $arg, $r))
 		spip_log("action iconifier: $arg pas compris");
 	elseif ($r[1] == '+')
 		action_spip_image_ajouter_dist($r[2], _request('sousaction2'), _request('source'));
 	else	action_spip_image_effacer_dist($r[2]);
 	
-	if(_request("iframe") == 'iframe') {
+	if (_request("iframe") == 'iframe') {
 		$redirect = urldecode($iframe_redirect)."&iframe=iframe";
 		redirige_par_entete(urldecode($redirect));
 	}
@@ -41,7 +41,7 @@ 
 function action_spip_image_effacer_dist($arg) {
 
 	if (!strstr($arg, ".."))
-		spip_unlink(_DIR_LOGOS . $arg);
+		spip_unlink(_DIR_LOGOS.$arg);
 }
 
 //
@@ -51,7 +51,7 @@ 
 // $source = $_FILES[0]
 // $dest = arton12.xxx
 // http://doc.spip.org/@action_spip_image_ajouter_dist
-function action_spip_image_ajouter_dist($arg,$sousaction2,$source,$return=false) {
+function action_spip_image_ajouter_dist($arg, $sousaction2, $source, $return = false) {
 	global $formats_logos;
 
 	include_spip('inc/documents');
@@ -63,15 +63,15 @@ 
 	if (!$source)
 		spip_log("spip_image_ajouter : source inconnue");
 	else {
-		$f =_DIR_LOGOS . $arg . '.tmp';
+		$f = _DIR_LOGOS.$arg.'.tmp';
 
 		if (!is_array($source)) 
 		// fichier dans upload/
-	  		$source = @copy(determine_upload() . $source, $f);
+	  		$source = @copy(determine_upload().$source, $f);
 		else {
 		// Intercepter une erreur a l'envoi
-			if ($erreur = check_upload_error($source['error'],"",$return))
-				$source ="";
+			if ($erreur = check_upload_error($source['error'], "", $return))
+				$source = "";
 			else
 		// analyse le type de l'image (on ne fait pas confiance au nom de
 		// fichier envoye par le browser : pour les Macs c'est plus sur)
@@ -83,22 +83,22 @@ 
 	}
 	if ($source AND $f) {
 		$size = @getimagesize($f);
-		$type = !$size ? '': ($size[2] > 3 ? '' : $formats_logos[$size[2]-1]);
+		$type = !$size ? '' : ($size[2] > 3 ? '' : $formats_logos[$size[2] - 1]);
 		if ($type) {
 			$poids = filesize($f);
 
 			if (_LOGO_MAX_SIZE > 0
-			AND $poids > _LOGO_MAX_SIZE*1024) {
-				spip_unlink ($f);
+			AND $poids > _LOGO_MAX_SIZE * 1024) {
+				spip_unlink($f);
 				$erreur = _T('info_logo_max_poids',
-									array('maxi' => taille_en_octets(_LOGO_MAX_SIZE*1024),
+									array('maxi' => taille_en_octets(_LOGO_MAX_SIZE * 1024),
 									'actuel' => taille_en_octets($poids)));
 			}
 
 			elseif (_LOGO_MAX_WIDTH * _LOGO_MAX_HEIGHT
 			AND ($size[0] > _LOGO_MAX_WIDTH
 			OR $size[1] > _LOGO_MAX_HEIGHT)) {
-				spip_unlink ($f);
+				spip_unlink($f);
 				$erreur = _T('info_logo_max_poids',
 									array(
 									'maxi' =>
@@ -112,20 +112,20 @@ 
 									));
 			}
 			else
-				@rename ($f, _DIR_LOGOS . $arg . ".$type");
+				@rename($f, _DIR_LOGOS.$arg.".$type");
 		}
 		else {
-			spip_unlink ($f);
+			spip_unlink($f);
 			$erreur = _T('info_logo_format_interdit',
 									array('formats' => join(', ', $formats_logos)));
 		}
 	
 	}
-	if ($erreur){
+	if ($erreur) {
 		if ($return)
 			return $erreur;
 		else
-			check_upload_error(6,$erreur);
+			check_upload_error(6, $erreur);
 	}
 }
 ?>

Braces +40 added lines, -33 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * L'entree par l'action ne sert plus qu'a une retro compat eventuelle
@@ -25,11 +27,13 @@ 
 
 	$arg = rawurldecode($arg);
 
-	if (!preg_match(',^-?\d*(\D)(.*)$,',$arg, $r))
-		spip_log("action iconifier: $arg pas compris");
-	elseif ($r[1] == '+')
-		action_spip_image_ajouter_dist($r[2], _request('sousaction2'), _request('source'));
-	else	action_spip_image_effacer_dist($r[2]);
+	if (!preg_match(',^-?\d*(\D)(.*)$,',$arg, $r)) {
+			spip_log("action iconifier: $arg pas compris");
+	} elseif ($r[1] == '+') {
+			action_spip_image_ajouter_dist($r[2], _request('sousaction2'), _request('source'));
+	} else {
+	    action_spip_image_effacer_dist($r[2]);
+	}
 	
 	if(_request("iframe") == 'iframe') {
 		$redirect = urldecode($iframe_redirect)."&iframe=iframe";
@@ -40,9 +44,10 @@ 
 // http://doc.spip.org/@action_spip_image_effacer_dist
 function action_spip_image_effacer_dist($arg) {
 
-	if (!strstr($arg, ".."))
-		spip_unlink(_DIR_LOGOS . $arg);
-}
+	if (!strstr($arg, "..")) {
+			spip_unlink(_DIR_LOGOS . $arg);
+	}
+	}
 
 //
 // Ajouter un logo
@@ -56,30 +61,34 @@ 
 
 	include_spip('inc/documents');
 	if (!$sousaction2) {
-		if (!$_FILES) $_FILES = $GLOBALS['HTTP_POST_FILES'];
+		if (!$_FILES) {
+		    $_FILES = $GLOBALS['HTTP_POST_FILES'];
+		}
 		$source = (is_array($_FILES) ? array_pop($_FILES) : "");
 	}
 	$erreur = "";
-	if (!$source)
-		spip_log("spip_image_ajouter : source inconnue");
-	else {
+	if (!$source) {
+			spip_log("spip_image_ajouter : source inconnue");
+	} else {
 		$f =_DIR_LOGOS . $arg . '.tmp';
 
-		if (!is_array($source)) 
-		// fichier dans upload/
+		if (!is_array($source)) {
+				// fichier dans upload/
 	  		$source = @copy(determine_upload() . $source, $f);
-		else {
+		} else {
 		// Intercepter une erreur a l'envoi
-			if ($erreur = check_upload_error($source['error'],"",$return))
-				$source ="";
-			else
-		// analyse le type de l'image (on ne fait pas confiance au nom de
+			if ($erreur = check_upload_error($source['error'],"",$return)) {
+							$source ="";
+			} else {
+					// analyse le type de l'image (on ne fait pas confiance au nom de
 		// fichier envoye par le browser : pour les Macs c'est plus sur)
 
 				$source = deplacer_fichier_upload($source['tmp_name'], $f);
+			}
+		}
+		if (!$source) {
+					spip_log("pb de copie pour $f");
 		}
-		if (!$source)
-			spip_log("pb de copie pour $f");
 	}
 	if ($source AND $f) {
 		$size = @getimagesize($f);
@@ -93,9 +102,7 @@ 
 				$erreur = _T('info_logo_max_poids',
 									array('maxi' => taille_en_octets(_LOGO_MAX_SIZE*1024),
 									'actuel' => taille_en_octets($poids)));
-			}
-
-			elseif (_LOGO_MAX_WIDTH * _LOGO_MAX_HEIGHT
+			} elseif (_LOGO_MAX_WIDTH * _LOGO_MAX_HEIGHT
 			AND ($size[0] > _LOGO_MAX_WIDTH
 			OR $size[1] > _LOGO_MAX_HEIGHT)) {
 				spip_unlink ($f);
@@ -110,11 +117,10 @@ 
 											array('largeur_vignette' => $size[0],
 											'hauteur_vignette' => $size[1]))
 									));
+			} else {
+							@rename ($f, _DIR_LOGOS . $arg . ".$type");
 			}
-			else
-				@rename ($f, _DIR_LOGOS . $arg . ".$type");
-		}
-		else {
+		} else {
 			spip_unlink ($f);
 			$erreur = _T('info_logo_format_interdit',
 									array('formats' => join(', ', $formats_logos)));
@@ -122,10 +128,11 @@ 
 	
 	}
 	if ($erreur){
-		if ($return)
-			return $erreur;
-		else
-			check_upload_error(6,$erreur);
+		if ($return) {
+					return $erreur;
+		} else {
+					check_upload_error(6,$erreur);
+		}
 	}
 }
 ?>

ecrire/action/purger_queue.php

Indentation +6 added lines, -6 removed lines

@@ -17,13 +17,13 @@
  * @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();
+    }
 
 }
 

Spacing +3 added lines, -3 removed lines

@@ -16,11 +16,11 @@
  * Purger la liste des travaux en attente
  * @return void
  */
-function action_purger_queue_dist(){
-	$securiser_action = charger_fonction('securiser_action','inc');
+function action_purger_queue_dist() {
+	$securiser_action = charger_fonction('securiser_action', 'inc');
 	$securiser_action();
 
-	if (autoriser('purger','queue')){
+	if (autoriser('purger', 'queue')) {
 		include_spip('inc/queue');
 		queue_purger();
 	}

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * Purger la liste des travaux en attente

ecrire/action/annuler_job.php

Indentation +7 added lines, -7 removed lines

@@ -17,14 +17,14 @@
  * @return void
  */
 function action_annuler_job_dist(){
-	$securiser_action = charger_fonction('securiser_action','inc');
-	$id_job = $securiser_action();
+    $securiser_action = charger_fonction('securiser_action','inc');
+    $id_job = $securiser_action();
 
-	if ($id_job = intval($id_job)
-		AND autoriser('annuler','job',$id_job)
-	){
-		job_queue_remove($id_job);
-	}
+    if ($id_job = intval($id_job)
+        AND autoriser('annuler','job',$id_job)
+    ){
+        job_queue_remove($id_job);
+    }
 }
 
 ?>
\ No newline at end of file

Spacing +4 added lines, -4 removed lines

@@ -16,13 +16,13 @@
  * Annuler un travail
  * @return void
  */
-function action_annuler_job_dist(){
-	$securiser_action = charger_fonction('securiser_action','inc');
+function action_annuler_job_dist() {
+	$securiser_action = charger_fonction('securiser_action', 'inc');
 	$id_job = $securiser_action();
 
 	if ($id_job = intval($id_job)
-		AND autoriser('annuler','job',$id_job)
-	){
+		AND autoriser('annuler', 'job', $id_job)
+	) {
 		job_queue_remove($id_job);
 	}
 }

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * Annuler un travail

ecrire/action/activer_plugins.php

Indentation +35 added lines, -35 removed lines

@@ -15,53 +15,53 @@
 // mise a jour des donnees si envoi via formulaire
 // http://doc.spip.org/@enregistre_modif_plugin
 function enregistre_modif_plugin(){
-	include_spip('inc/plugin');
-  // recuperer les plugins dans l'ordre des $_POST
-	$test = array();
-	foreach(liste_plugin_files() as $file){
-	  $test['s'.substr(md5(_DIR_PLUGINS.$file),0,16)] = $file;
-	}
-	if (defined('_DIR_PLUGINS_SUPPL'))
-		foreach(liste_plugin_files(_DIR_PLUGINS_SUPPL) as $file){
-		  $test['s'.substr(md5(_DIR_PLUGINS_SUPPL.$file),0,16)] = $file;
-		}
+    include_spip('inc/plugin');
+    // recuperer les plugins dans l'ordre des $_POST
+    $test = array();
+    foreach(liste_plugin_files() as $file){
+        $test['s'.substr(md5(_DIR_PLUGINS.$file),0,16)] = $file;
+    }
+    if (defined('_DIR_PLUGINS_SUPPL'))
+        foreach(liste_plugin_files(_DIR_PLUGINS_SUPPL) as $file){
+            $test['s'.substr(md5(_DIR_PLUGINS_SUPPL.$file),0,16)] = $file;
+        }
 
-	$plugin=array();
+    $plugin=array();
 
-	foreach($_POST as $choix=>$val){
-		if (isset($test[$choix])&&$val=='O')
-			$plugin[]=$test[$choix];
-	}
+    foreach($_POST as $choix=>$val){
+        if (isset($test[$choix])&&$val=='O')
+            $plugin[]=$test[$choix];
+    }
 
-	spip_log("Changement des plugins actifs par l'auteur " . $GLOBALS['visiteur_session']['id_auteur'] . ": " . join(',', $plugin));
-	ecrire_plugin_actifs($plugin);
+    spip_log("Changement des plugins actifs par l'auteur " . $GLOBALS['visiteur_session']['id_auteur'] . ": " . join(',', $plugin));
+    ecrire_plugin_actifs($plugin);
 
-	// Chaque fois que l'on valide des plugins, on memorise la liste de ces plugins comme etant "interessants", avec un score initial, qui sera decremente a chaque tour : ainsi un plugin active pourra reter visible a l'ecran, jusqu'a ce qu'il tombe dans l'oubli.
-	$plugins_interessants = @unserialize($GLOBALS['meta']['plugins_interessants']);
-	if (!is_array($plugins_interessants))
-		$plugins_interessants = array();
+    // Chaque fois que l'on valide des plugins, on memorise la liste de ces plugins comme etant "interessants", avec un score initial, qui sera decremente a chaque tour : ainsi un plugin active pourra reter visible a l'ecran, jusqu'a ce qu'il tombe dans l'oubli.
+    $plugins_interessants = @unserialize($GLOBALS['meta']['plugins_interessants']);
+    if (!is_array($plugins_interessants))
+        $plugins_interessants = array();
 
-	$plugins_interessants2 = array();
+    $plugins_interessants2 = array();
 
-	foreach($plugins_interessants as $plug => $score)
-		if ($score > 1)
-			$plugins_interessants2[$plug] = $score-1;
-	foreach ($plugin as $plug)
-		$plugins_interessants2[$plug] = 10; // score initial
-	ecrire_meta('plugins_interessants', serialize($plugins_interessants2));
+    foreach($plugins_interessants as $plug => $score)
+        if ($score > 1)
+            $plugins_interessants2[$plug] = $score-1;
+    foreach ($plugin as $plug)
+        $plugins_interessants2[$plug] = 10; // score initial
+    ecrire_meta('plugins_interessants', serialize($plugins_interessants2));
 }
 
 // http://doc.spip.org/@action_activer_plugins_dist
 function action_activer_plugins_dist() {
 
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $securiser_action();
 
-	if (!autoriser('configurer', '_plugins'))
-		die('erreur');
-	// forcer la maj des meta pour les cas de modif de numero de version base via phpmyadmin
-	lire_metas();
-	enregistre_modif_plugin();
+    if (!autoriser('configurer', '_plugins'))
+        die('erreur');
+    // forcer la maj des meta pour les cas de modif de numero de version base via phpmyadmin
+    lire_metas();
+    enregistre_modif_plugin();
 }
 
 ?>

Spacing +12 added lines, -12 removed lines

@@ -14,26 +14,26 @@ 
 
 // mise a jour des donnees si envoi via formulaire
 // http://doc.spip.org/@enregistre_modif_plugin
-function enregistre_modif_plugin(){
+function enregistre_modif_plugin() {
 	include_spip('inc/plugin');
   // recuperer les plugins dans l'ordre des $_POST
 	$test = array();
-	foreach(liste_plugin_files() as $file){
-	  $test['s'.substr(md5(_DIR_PLUGINS.$file),0,16)] = $file;
+	foreach (liste_plugin_files() as $file) {
+	  $test['s'.substr(md5(_DIR_PLUGINS.$file), 0, 16)] = $file;
 	}
 	if (defined('_DIR_PLUGINS_SUPPL'))
-		foreach(liste_plugin_files(_DIR_PLUGINS_SUPPL) as $file){
-		  $test['s'.substr(md5(_DIR_PLUGINS_SUPPL.$file),0,16)] = $file;
+		foreach (liste_plugin_files(_DIR_PLUGINS_SUPPL) as $file) {
+		  $test['s'.substr(md5(_DIR_PLUGINS_SUPPL.$file), 0, 16)] = $file;
 		}
 
-	$plugin=array();
+	$plugin = array();
 
-	foreach($_POST as $choix=>$val){
-		if (isset($test[$choix])&&$val=='O')
-			$plugin[]=$test[$choix];
+	foreach ($_POST as $choix=>$val) {
+		if (isset($test[$choix]) && $val == 'O')
+			$plugin[] = $test[$choix];
 	}
 
-	spip_log("Changement des plugins actifs par l'auteur " . $GLOBALS['visiteur_session']['id_auteur'] . ": " . join(',', $plugin));
+	spip_log("Changement des plugins actifs par l'auteur ".$GLOBALS['visiteur_session']['id_auteur'].": ".join(',', $plugin));
 	ecrire_plugin_actifs($plugin);
 
 	// Chaque fois que l'on valide des plugins, on memorise la liste de ces plugins comme etant "interessants", avec un score initial, qui sera decremente a chaque tour : ainsi un plugin active pourra reter visible a l'ecran, jusqu'a ce qu'il tombe dans l'oubli.
@@ -43,9 +43,9 @@ 
 
 	$plugins_interessants2 = array();
 
-	foreach($plugins_interessants as $plug => $score)
+	foreach ($plugins_interessants as $plug => $score)
 		if ($score > 1)
-			$plugins_interessants2[$plug] = $score-1;
+			$plugins_interessants2[$plug] = $score - 1;
 	foreach ($plugin as $plug)
 		$plugins_interessants2[$plug] = 10; // score initial
 	ecrire_meta('plugins_interessants', serialize($plugins_interessants2));

Braces +22 added lines, -13 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // mise a jour des donnees si envoi via formulaire
 // http://doc.spip.org/@enregistre_modif_plugin
@@ -21,16 +23,18 @@ 
 	foreach(liste_plugin_files() as $file){
 	  $test['s'.substr(md5(_DIR_PLUGINS.$file),0,16)] = $file;
 	}
-	if (defined('_DIR_PLUGINS_SUPPL'))
-		foreach(liste_plugin_files(_DIR_PLUGINS_SUPPL) as $file){
+	if (defined('_DIR_PLUGINS_SUPPL')) {
+			foreach(liste_plugin_files(_DIR_PLUGINS_SUPPL) as $file){
 		  $test['s'.substr(md5(_DIR_PLUGINS_SUPPL.$file),0,16)] = $file;
+	}
 		}
 
 	$plugin=array();
 
 	foreach($_POST as $choix=>$val){
-		if (isset($test[$choix])&&$val=='O')
-			$plugin[]=$test[$choix];
+		if (isset($test[$choix])&&$val=='O') {
+					$plugin[]=$test[$choix];
+		}
 	}
 
 	spip_log("Changement des plugins actifs par l'auteur " . $GLOBALS['visiteur_session']['id_auteur'] . ": " . join(',', $plugin));
@@ -38,16 +42,20 @@ 
 
 	// Chaque fois que l'on valide des plugins, on memorise la liste de ces plugins comme etant "interessants", avec un score initial, qui sera decremente a chaque tour : ainsi un plugin active pourra reter visible a l'ecran, jusqu'a ce qu'il tombe dans l'oubli.
 	$plugins_interessants = @unserialize($GLOBALS['meta']['plugins_interessants']);
-	if (!is_array($plugins_interessants))
-		$plugins_interessants = array();
+	if (!is_array($plugins_interessants)) {
+			$plugins_interessants = array();
+	}
 
 	$plugins_interessants2 = array();
 
-	foreach($plugins_interessants as $plug => $score)
-		if ($score > 1)
+	foreach($plugins_interessants as $plug => $score) {
+			if ($score > 1)
 			$plugins_interessants2[$plug] = $score-1;
-	foreach ($plugin as $plug)
-		$plugins_interessants2[$plug] = 10; // score initial
+	}
+	foreach ($plugin as $plug) {
+			$plugins_interessants2[$plug] = 10;
+	}
+	// score initial
 	ecrire_meta('plugins_interessants', serialize($plugins_interessants2));
 }
 
@@ -57,8 +65,9 @@ 
 	$securiser_action = charger_fonction('securiser_action', 'inc');
 	$securiser_action();
 
-	if (!autoriser('configurer', '_plugins'))
-		die('erreur');
+	if (!autoriser('configurer', '_plugins')) {
+			die('erreur');
+	}
 	// forcer la maj des meta pour les cas de modif de numero de version base via phpmyadmin
 	lire_metas();
 	enregistre_modif_plugin();

ecrire/action/ajouter_lien.php

Indentation +8 added lines, -8 removed lines

@@ -13,16 +13,16 @@
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 function action_ajouter_lien_dist($arg=null){
-	if (is_null($arg)){
-		$securiser_action = charger_fonction('securiser_action','inc');
-		$arg = $securiser_action();
-	}
+    if (is_null($arg)){
+        $securiser_action = charger_fonction('securiser_action','inc');
+        $arg = $securiser_action();
+    }
 
-	$arg = explode("-",$arg);
-	list($objet_source,$ids,$objet_lie,$idl) = $arg;
+    $arg = explode("-",$arg);
+    list($objet_source,$ids,$objet_lie,$idl) = $arg;
 
-	include_spip('action/editer_liens');
-	objet_associer(array($objet_source=>$ids), array($objet_lie=>$idl));
+    include_spip('action/editer_liens');
+    objet_associer(array($objet_source=>$ids), array($objet_lie=>$idl));
 }
 
 

Spacing +5 added lines, -5 removed lines

@@ -12,14 +12,14 @@
 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
-function action_ajouter_lien_dist($arg=null){
-	if (is_null($arg)){
-		$securiser_action = charger_fonction('securiser_action','inc');
+function action_ajouter_lien_dist($arg = null) {
+	if (is_null($arg)) {
+		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 	}
 
-	$arg = explode("-",$arg);
-	list($objet_source,$ids,$objet_lie,$idl) = $arg;
+	$arg = explode("-", $arg);
+	list($objet_source, $ids, $objet_lie, $idl) = $arg;
 
 	include_spip('action/editer_liens');
 	objet_associer(array($objet_source=>$ids), array($objet_lie=>$idl));

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 function action_ajouter_lien_dist($arg=null){
 	if (is_null($arg)){

ecrire/urls/page.php

Indentation +45 added lines, -45 removed lines

@@ -28,16 +28,16 @@ 
 // http://doc.spip.org/@_generer_url_page
 function _generer_url_page($type,$id, $args='', $ancre='') {
 
-	if ($generer_url_externe = charger_fonction("generer_url_$type",'urls',true)) {
-		$url = $generer_url_externe($id, $args, $ancre);
-		if (NULL != $url) return $url;
-	}
+    if ($generer_url_externe = charger_fonction("generer_url_$type",'urls',true)) {
+        $url = $generer_url_externe($id, $args, $ancre);
+        if (NULL != $url) return $url;
+    }
 
-	$url = _debut_urls_page . $type . _separateur_urls_page
-	  . $id . _terminaison_urls_page;
+    $url = _debut_urls_page . $type . _separateur_urls_page
+        . $id . _terminaison_urls_page;
 
-	if ($args) $args = strpos($url,'?') ? "&$args" : "?$args";
-	return _DIR_RACINE . $url . $args . ($ancre ? "#$ancre" : '');
+    if ($args) $args = strpos($url,'?') ? "&$args" : "?$args";
+    return _DIR_RACINE . $url . $args . ($ancre ? "#$ancre" : '');
 }
 
 // retrouve le fond et les parametres d'une URL abregee
@@ -45,51 +45,51 @@ 
 // http://doc.spip.org/@urls_page_dist
 function urls_page_dist($i, &$entite, $args='', $ancre='')
 {
-	if (is_numeric($i))
-		return _generer_url_page($entite, $i, $args, $ancre);
+    if (is_numeric($i))
+        return _generer_url_page($entite, $i, $args, $ancre);
 
-	// traiter les injections du type domaine.org/spip.php/cestnimportequoi/ou/encore/plus/rubrique23
-	if ($GLOBALS['profondeur_url']>0 AND $entite=='sommaire'){
-		return array(array(),'404');
-	}
+    // traiter les injections du type domaine.org/spip.php/cestnimportequoi/ou/encore/plus/rubrique23
+    if ($GLOBALS['profondeur_url']>0 AND $entite=='sommaire'){
+        return array(array(),'404');
+    }
 
-	// voir s'il faut recuperer le id_* implicite et les &debut_xx;
-	if (is_array($args))
-		$contexte = $args;
-	else
-		parse_str($args,$contexte);
-	include_spip('inc/urls');
-	$r = nettoyer_url_page($i, $contexte);
-	if ($r) {
-		array_pop($r); // nettoyer_url_page renvoie un argument de plus inutile ici
-		return $r;
-	}
+    // voir s'il faut recuperer le id_* implicite et les &debut_xx;
+    if (is_array($args))
+        $contexte = $args;
+    else
+        parse_str($args,$contexte);
+    include_spip('inc/urls');
+    $r = nettoyer_url_page($i, $contexte);
+    if ($r) {
+        array_pop($r); // nettoyer_url_page renvoie un argument de plus inutile ici
+        return $r;
+    }
 
-	/*
+    /*
 	 * Le bloc qui suit sert a faciliter les transitions depuis
 	 * le mode 'urls-propres' vers les modes 'urls-standard' et 'url-html'
 	 * Il est inutile de le recopier si vous personnalisez vos URLs
 	 * et votre .htaccess
 	 */
-	// Si on est revenu en mode html, mais c'est une ancienne url_propre
-	// on ne redirige pas, on assume le nouveau contexte (si possible)
-	$url = $i;
-	$url_propre = isset($url)
-		? $url
-		: (isset($_SERVER['REDIRECT_url_propre'])
-			? $_SERVER['REDIRECT_url_propre']
-			: (isset($_ENV['url_propre'])
-				? $_ENV['url_propre']
-				: ''
-				));
-	if ($url_propre) {
-		if ($GLOBALS['profondeur_url']<=0)
-			$urls_anciennes = charger_fonction('propres','urls',true);
-		else
-			$urls_anciennes = charger_fonction('arbo','urls',true);
-		return $urls_anciennes?$urls_anciennes($url_propre, $entite, $contexte):'';
-	}
-	/* Fin du bloc compatibilite url-propres */
+    // Si on est revenu en mode html, mais c'est une ancienne url_propre
+    // on ne redirige pas, on assume le nouveau contexte (si possible)
+    $url = $i;
+    $url_propre = isset($url)
+        ? $url
+        : (isset($_SERVER['REDIRECT_url_propre'])
+            ? $_SERVER['REDIRECT_url_propre']
+            : (isset($_ENV['url_propre'])
+                ? $_ENV['url_propre']
+                : ''
+                ));
+    if ($url_propre) {
+        if ($GLOBALS['profondeur_url']<=0)
+            $urls_anciennes = charger_fonction('propres','urls',true);
+        else
+            $urls_anciennes = charger_fonction('arbo','urls',true);
+        return $urls_anciennes?$urls_anciennes($url_propre, $entite, $contexte):'';
+    }
+    /* Fin du bloc compatibilite url-propres */
 }
 
 ?>

Spacing +17 added lines, -17 removed lines

@@ -16,48 +16,48 @@ 
 
 ####### modifications possibles dans ecrire/mes_options
 # on peut indiquer '.html' pour faire joli
-define ('_terminaison_urls_page', '');
+define('_terminaison_urls_page', '');
 # ci-dessous, ce qu'on veut ou presque (de preference pas de '/')
 # attention toutefois seuls '' et '=' figurent dans les modes de compatibilite
-define ('_separateur_urls_page', '');
+define('_separateur_urls_page', '');
 # on peut indiquer '' si on a installe le .htaccess
-define ('_debut_urls_page', get_spip_script('./').'?');
+define('_debut_urls_page', get_spip_script('./').'?');
 #######
 
 
 // http://doc.spip.org/@_generer_url_page
-function _generer_url_page($type,$id, $args='', $ancre='') {
+function _generer_url_page($type, $id, $args = '', $ancre = '') {
 
-	if ($generer_url_externe = charger_fonction("generer_url_$type",'urls',true)) {
+	if ($generer_url_externe = charger_fonction("generer_url_$type", 'urls', true)) {
 		$url = $generer_url_externe($id, $args, $ancre);
 		if (NULL != $url) return $url;
 	}
 
-	$url = _debut_urls_page . $type . _separateur_urls_page
-	  . $id . _terminaison_urls_page;
+	$url = _debut_urls_page.$type._separateur_urls_page
+	  . $id._terminaison_urls_page;
 
-	if ($args) $args = strpos($url,'?') ? "&$args" : "?$args";
-	return _DIR_RACINE . $url . $args . ($ancre ? "#$ancre" : '');
+	if ($args) $args = strpos($url, '?') ? "&$args" : "?$args";
+	return _DIR_RACINE.$url.$args.($ancre ? "#$ancre" : '');
 }
 
 // retrouve le fond et les parametres d'une URL abregee
 // le contexte deja existant est fourni dans args sous forme de tableau ou query string
 // http://doc.spip.org/@urls_page_dist
-function urls_page_dist($i, &$entite, $args='', $ancre='')
+function urls_page_dist($i, &$entite, $args = '', $ancre = '')
 {
 	if (is_numeric($i))
 		return _generer_url_page($entite, $i, $args, $ancre);
 
 	// traiter les injections du type domaine.org/spip.php/cestnimportequoi/ou/encore/plus/rubrique23
-	if ($GLOBALS['profondeur_url']>0 AND $entite=='sommaire'){
-		return array(array(),'404');
+	if ($GLOBALS['profondeur_url'] > 0 AND $entite == 'sommaire') {
+		return array(array(), '404');
 	}
 
 	// voir s'il faut recuperer le id_* implicite et les &debut_xx;
 	if (is_array($args))
 		$contexte = $args;
 	else
-		parse_str($args,$contexte);
+		parse_str($args, $contexte);
 	include_spip('inc/urls');
 	$r = nettoyer_url_page($i, $contexte);
 	if ($r) {
@@ -83,11 +83,11 @@ 
 				: ''
 				));
 	if ($url_propre) {
-		if ($GLOBALS['profondeur_url']<=0)
-			$urls_anciennes = charger_fonction('propres','urls',true);
+		if ($GLOBALS['profondeur_url'] <= 0)
+			$urls_anciennes = charger_fonction('propres', 'urls', true);
 		else
-			$urls_anciennes = charger_fonction('arbo','urls',true);
-		return $urls_anciennes?$urls_anciennes($url_propre, $entite, $contexte):'';
+			$urls_anciennes = charger_fonction('arbo', 'urls', true);
+		return $urls_anciennes ? $urls_anciennes($url_propre, $entite, $contexte) : '';
 	}
 	/* Fin du bloc compatibilite url-propres */
 }

Braces +22 added lines, -13 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 define('URLS_PAGE_EXEMPLE', 'spip.php?article12');
 
@@ -30,13 +32,17 @@ 
 
 	if ($generer_url_externe = charger_fonction("generer_url_$type",'urls',true)) {
 		$url = $generer_url_externe($id, $args, $ancre);
-		if (NULL != $url) return $url;
+		if (NULL != $url) {
+		    return $url;
+		}
 	}
 
 	$url = _debut_urls_page . $type . _separateur_urls_page
 	  . $id . _terminaison_urls_page;
 
-	if ($args) $args = strpos($url,'?') ? "&$args" : "?$args";
+	if ($args) {
+	    $args = strpos($url,'?') ? "&$args" : "?$args";
+	}
 	return _DIR_RACINE . $url . $args . ($ancre ? "#$ancre" : '');
 }
 
@@ -45,8 +51,9 @@ 
 // http://doc.spip.org/@urls_page_dist
 function urls_page_dist($i, &$entite, $args='', $ancre='')
 {
-	if (is_numeric($i))
-		return _generer_url_page($entite, $i, $args, $ancre);
+	if (is_numeric($i)) {
+			return _generer_url_page($entite, $i, $args, $ancre);
+	}
 
 	// traiter les injections du type domaine.org/spip.php/cestnimportequoi/ou/encore/plus/rubrique23
 	if ($GLOBALS['profondeur_url']>0 AND $entite=='sommaire'){
@@ -54,10 +61,11 @@ 
 	}
 
 	// voir s'il faut recuperer le id_* implicite et les &debut_xx;
-	if (is_array($args))
-		$contexte = $args;
-	else
-		parse_str($args,$contexte);
+	if (is_array($args)) {
+			$contexte = $args;
+	} else {
+			parse_str($args,$contexte);
+	}
 	include_spip('inc/urls');
 	$r = nettoyer_url_page($i, $contexte);
 	if ($r) {
@@ -83,10 +91,11 @@ 
 				: ''
 				));
 	if ($url_propre) {
-		if ($GLOBALS['profondeur_url']<=0)
-			$urls_anciennes = charger_fonction('propres','urls',true);
-		else
-			$urls_anciennes = charger_fonction('arbo','urls',true);
+		if ($GLOBALS['profondeur_url']<=0) {
+					$urls_anciennes = charger_fonction('propres','urls',true);
+		} else {
+					$urls_anciennes = charger_fonction('arbo','urls',true);
+		}
 		return $urls_anciennes?$urls_anciennes($url_propre, $entite, $contexte):'';
 	}
 	/* Fin du bloc compatibilite url-propres */