spip / SPIP

ecrire/action/supprimer_lien.php

Indentation +8 added lines, -8 removed lines

@@ -13,16 +13,16 @@
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 function action_supprimer_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_dissocier(array($objet_source=>$ids), array($objet_lie=>$idl));
+    include_spip('action/editer_liens');
+    objet_dissocier(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_supprimer_lien_dist($arg=null){
-	if (is_null($arg)){
-		$securiser_action = charger_fonction('securiser_action','inc');
+function action_supprimer_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_dissocier(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_supprimer_lien_dist($arg=null){
 	if (is_null($arg)){

ecrire/action/tester_taille.php

Indentation +51 added lines, -51 removed lines

@@ -15,11 +15,11 @@ 
 
 function action_tester_taille_error_handler($output)
 {
-	// on est ici, donc echec lors de la creation de l'image
-	if ($GLOBALS['redirect']){
-		return redirige_formulaire($GLOBALS['redirect']);
-	}
-	return $output;
+    // on est ici, donc echec lors de la creation de l'image
+    if ($GLOBALS['redirect']){
+        return redirige_formulaire($GLOBALS['redirect']);
+    }
+    return $output;
 }
 
 
@@ -27,61 +27,61 @@ 
 // http://doc.spip.org/@action_tester_taille_dist
 function action_tester_taille_dist() {
 	
-	if (!autoriser('configurer'))
-		return;
+    if (!autoriser('configurer'))
+        return;
 	
-	$taille = _request('arg');
-	$taille = explode('-',$taille);
+    $taille = _request('arg');
+    $taille = explode('-',$taille);
 
-	$GLOBALS['taille_max'] = end($taille);
-	$GLOBALS['taille_min'] = 0;
-	if (count($taille)>1)
-		$GLOBALS['taille_min'] = reset($taille);
+    $GLOBALS['taille_max'] = end($taille);
+    $GLOBALS['taille_min'] = 0;
+    if (count($taille)>1)
+        $GLOBALS['taille_min'] = reset($taille);
 
-	// si l'intervalle est assez petit, on garde la valeur min
-	if ($GLOBALS['taille_max']*$GLOBALS['taille_max']-$GLOBALS['taille_min']*$GLOBALS['taille_min']<50000){
-		ecrire_meta('max_taille_vignettes',$t=($GLOBALS['taille_min']*$GLOBALS['taille_min'])*0.9,'non');
-		echo round($t/1000000,3).' Mpx';
-		die();
-	}
+    // si l'intervalle est assez petit, on garde la valeur min
+    if ($GLOBALS['taille_max']*$GLOBALS['taille_max']-$GLOBALS['taille_min']*$GLOBALS['taille_min']<50000){
+        ecrire_meta('max_taille_vignettes',$t=($GLOBALS['taille_min']*$GLOBALS['taille_min'])*0.9,'non');
+        echo round($t/1000000,3).' Mpx';
+        die();
+    }
 
-	$taille = $GLOBALS['taille_test'] = round(($GLOBALS['taille_max']+$GLOBALS['taille_min'])/2);
+    $taille = $GLOBALS['taille_test'] = round(($GLOBALS['taille_max']+$GLOBALS['taille_min'])/2);
 
-	include_spip('inc/filtres');
-	// des inclusions representatives d'un hit prive et/ou public pour la conso memoire
-	include_spip('public/assembler');
-	include_spip('public/balises');
-	include_spip('public/boucles');
-	include_spip('public/cacher');
-	include_spip('public/compiler');
-	include_spip('public/composer');
-	include_spip('public/criteres');
-	include_spip('public/interfaces');
-	include_spip('public/parametrer');
-	include_spip('public/phraser_html');
-	include_spip('public/references');
+    include_spip('inc/filtres');
+    // des inclusions representatives d'un hit prive et/ou public pour la conso memoire
+    include_spip('public/assembler');
+    include_spip('public/balises');
+    include_spip('public/boucles');
+    include_spip('public/cacher');
+    include_spip('public/compiler');
+    include_spip('public/composer');
+    include_spip('public/criteres');
+    include_spip('public/interfaces');
+    include_spip('public/parametrer');
+    include_spip('public/phraser_html');
+    include_spip('public/references');
 
-	include_spip('inc/presentation');
-	include_spip('inc/charsets');
-	include_spip('inc/documents');
-	include_spip('inc/header');
-	propre("<doc1>"); // charger propre avec le trairement d'un modele
+    include_spip('inc/presentation');
+    include_spip('inc/charsets');
+    include_spip('inc/documents');
+    include_spip('inc/header');
+    propre("<doc1>"); // charger propre avec le trairement d'un modele
 
-	$i = _request('i')+1;
-	$image_source = chemin_image("test.png");
-	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=".$GLOBALS['taille_min']."-".$GLOBALS['taille_test']);
+    $i = _request('i')+1;
+    $image_source = chemin_image("test.png");
+    $GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=".$GLOBALS['taille_min']."-".$GLOBALS['taille_test']);
 
-	ob_start('action_tester_taille_error_handler');
-	filtrer('image_recadre',$image_source,$taille,$taille);
-	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-".$GLOBALS['taille_max']);
-	// si la valeur intermediaire a reussi, on teste la valeur maxi qui est peut etre sous estimee
-	$taille = $GLOBALS['taille_max'];
-	filtrer('image_recadre',$image_source,$taille,$taille);
-	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-".$GLOBALS['taille_max']);
-	ob_end_clean();
+    ob_start('action_tester_taille_error_handler');
+    filtrer('image_recadre',$image_source,$taille,$taille);
+    $GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-".$GLOBALS['taille_max']);
+    // si la valeur intermediaire a reussi, on teste la valeur maxi qui est peut etre sous estimee
+    $taille = $GLOBALS['taille_max'];
+    filtrer('image_recadre',$image_source,$taille,$taille);
+    $GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-".$GLOBALS['taille_max']);
+    ob_end_clean();
 
-	// on est ici, donc pas de plantage
-	echo redirige_formulaire($GLOBALS['redirect']);
+    // on est ici, donc pas de plantage
+    echo redirige_formulaire($GLOBALS['redirect']);
 }
 
 ?>

Spacing +10 added lines, -10 removed lines

@@ -16,7 +16,7 @@ 
 function action_tester_taille_error_handler($output)
 {
 	// on est ici, donc echec lors de la creation de l'image
-	if ($GLOBALS['redirect']){
+	if ($GLOBALS['redirect']) {
 		return redirige_formulaire($GLOBALS['redirect']);
 	}
 	return $output;
@@ -31,21 +31,21 @@ 
 		return;
 	
 	$taille = _request('arg');
-	$taille = explode('-',$taille);
+	$taille = explode('-', $taille);
 
 	$GLOBALS['taille_max'] = end($taille);
 	$GLOBALS['taille_min'] = 0;
-	if (count($taille)>1)
+	if (count($taille) > 1)
 		$GLOBALS['taille_min'] = reset($taille);
 
 	// si l'intervalle est assez petit, on garde la valeur min
-	if ($GLOBALS['taille_max']*$GLOBALS['taille_max']-$GLOBALS['taille_min']*$GLOBALS['taille_min']<50000){
-		ecrire_meta('max_taille_vignettes',$t=($GLOBALS['taille_min']*$GLOBALS['taille_min'])*0.9,'non');
-		echo round($t/1000000,3).' Mpx';
+	if ($GLOBALS['taille_max'] * $GLOBALS['taille_max'] - $GLOBALS['taille_min'] * $GLOBALS['taille_min'] < 50000) {
+		ecrire_meta('max_taille_vignettes', $t = ($GLOBALS['taille_min'] * $GLOBALS['taille_min']) * 0.9, 'non');
+		echo round($t / 1000000, 3).' Mpx';
 		die();
 	}
 
-	$taille = $GLOBALS['taille_test'] = round(($GLOBALS['taille_max']+$GLOBALS['taille_min'])/2);
+	$taille = $GLOBALS['taille_test'] = round(($GLOBALS['taille_max'] + $GLOBALS['taille_min']) / 2);
 
 	include_spip('inc/filtres');
 	// des inclusions representatives d'un hit prive et/ou public pour la conso memoire
@@ -67,16 +67,16 @@ 
 	include_spip('inc/header');
 	propre("<doc1>"); // charger propre avec le trairement d'un modele
 
-	$i = _request('i')+1;
+	$i = _request('i') + 1;
 	$image_source = chemin_image("test.png");
 	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=".$GLOBALS['taille_min']."-".$GLOBALS['taille_test']);
 
 	ob_start('action_tester_taille_error_handler');
-	filtrer('image_recadre',$image_source,$taille,$taille);
+	filtrer('image_recadre', $image_source, $taille, $taille);
 	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-".$GLOBALS['taille_max']);
 	// si la valeur intermediaire a reussi, on teste la valeur maxi qui est peut etre sous estimee
 	$taille = $GLOBALS['taille_max'];
-	filtrer('image_recadre',$image_source,$taille,$taille);
+	filtrer('image_recadre', $image_source, $taille, $taille);
 	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-".$GLOBALS['taille_max']);
 	ob_end_clean();
 

Braces +9 added lines, -5 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;
+}
 include_spip('inc/headers');
 
 function action_tester_taille_error_handler($output)
@@ -27,16 +29,18 @@ 
 // http://doc.spip.org/@action_tester_taille_dist
 function action_tester_taille_dist() {
 	
-	if (!autoriser('configurer'))
-		return;
+	if (!autoriser('configurer')) {
+			return;
+	}
 	
 	$taille = _request('arg');
 	$taille = explode('-',$taille);
 
 	$GLOBALS['taille_max'] = end($taille);
 	$GLOBALS['taille_min'] = 0;
-	if (count($taille)>1)
-		$GLOBALS['taille_min'] = reset($taille);
+	if (count($taille)>1) {
+			$GLOBALS['taille_min'] = reset($taille);
+	}
 
 	// si l'intervalle est assez petit, on garde la valeur min
 	if ($GLOBALS['taille_max']*$GLOBALS['taille_max']-$GLOBALS['taille_min']*$GLOBALS['taille_min']<50000){

ecrire/action/etre_webmestre.php

Indentation +18 added lines, -18 removed lines

@@ -20,28 +20,28 @@
  * @return void
  */
 function action_etre_webmestre_dist() {
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$time = $securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $time = $securiser_action();
 
-	if (time()-$time<15*60
-		AND $GLOBALS['visiteur_session']['statut']=='0minirezo'
-		AND $GLOBALS['visiteur_session']['webmestre']!=='oui') {
-		$action = _T('info_admin_etre_webmestre');
-		$admin = charger_fonction('admin', 'inc');
-		// lance la verif par ftp et l'appel
-		// a base_etre_webmestre_dist quand c'est OK
-		if ($r = $admin('etre_webmestre', $action)) {
-			echo $r;
-			exit;
-		}
-	}
+    if (time()-$time<15*60
+        AND $GLOBALS['visiteur_session']['statut']=='0minirezo'
+        AND $GLOBALS['visiteur_session']['webmestre']!=='oui') {
+        $action = _T('info_admin_etre_webmestre');
+        $admin = charger_fonction('admin', 'inc');
+        // lance la verif par ftp et l'appel
+        // a base_etre_webmestre_dist quand c'est OK
+        if ($r = $admin('etre_webmestre', $action)) {
+            echo $r;
+            exit;
+        }
+    }
 
 }
 
 function base_etre_webmestre_dist() {
-	if ($GLOBALS['visiteur_session']['statut']=='0minirezo' AND $GLOBALS['visiteur_session']['webmestre']!=='oui') {
-		include_spip('action/editer_auteur');
-		instituer_auteur($GLOBALS['visiteur_session']['id_auteur'], array('webmestre'=>'oui'), true);
-	}
+    if ($GLOBALS['visiteur_session']['statut']=='0minirezo' AND $GLOBALS['visiteur_session']['webmestre']!=='oui') {
+        include_spip('action/editer_auteur');
+        instituer_auteur($GLOBALS['visiteur_session']['id_auteur'], array('webmestre'=>'oui'), true);
+    }
 }
 ?>

Spacing +4 added lines, -4 removed lines

@@ -23,9 +23,9 @@ 
 	$securiser_action = charger_fonction('securiser_action', 'inc');
 	$time = $securiser_action();
 
-	if (time()-$time<15*60
-		AND $GLOBALS['visiteur_session']['statut']=='0minirezo'
-		AND $GLOBALS['visiteur_session']['webmestre']!=='oui') {
+	if (time() - $time < 15 * 60
+		AND $GLOBALS['visiteur_session']['statut'] == '0minirezo'
+		AND $GLOBALS['visiteur_session']['webmestre'] !== 'oui') {
 		$action = _T('info_admin_etre_webmestre');
 		$admin = charger_fonction('admin', 'inc');
 		// lance la verif par ftp et l'appel
@@ -39,7 +39,7 @@ 
 }
 
 function base_etre_webmestre_dist() {
-	if ($GLOBALS['visiteur_session']['statut']=='0minirezo' AND $GLOBALS['visiteur_session']['webmestre']!=='oui') {
+	if ($GLOBALS['visiteur_session']['statut'] == '0minirezo' AND $GLOBALS['visiteur_session']['webmestre'] !== 'oui') {
 		include_spip('action/editer_auteur');
 		instituer_auteur($GLOBALS['visiteur_session']['id_auteur'], array('webmestre'=>'oui'), true);
 	}

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;
+}
 
 include_spip('inc/actions');
 

ecrire/action/tester.php

Indentation +124 added lines, -124 removed lines

@@ -15,129 +15,129 @@
 // Tester nos capacites
 // http://doc.spip.org/@action_tester_dist
 function action_tester_dist() {
-	$arg = _request('arg');
-
-	$gd_formats = $gd_formats_read_gif = "";
-	// verifier les formats acceptes par GD
-	if ($arg == "gd1") {
-		// Si GD est installe et php >= 4.0.2
-		if (function_exists('imagetypes')) {
-
-			if (imagetypes() & IMG_GIF) {
-				$gd_formats[] = "gif";
-			} else {
-				# Attention GD sait lire le gif mais pas forcement l'ecrire
-				if (function_exists('ImageCreateFromGIF')) {
-					$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK."test.gif");
-					if ($srcImage) {
-						$gd_formats_read_gif = ",gif";
-						ImageDestroy( $srcImage );
-					}
-				}
-			}
-
-			if (imagetypes() & IMG_JPG)
-				$gd_formats[] = "jpg";
-			if (imagetypes() & IMG_PNG)
-				$gd_formats[] = "png";
-		}
-
-		else {	# ancienne methode de detection des formats, qui en plus
-				# est bugguee car elle teste les formats en lecture
-				# alors que la valeur deduite sert a identifier
-				# les formats disponibles en ecriture... (cf. inc_logos)
-
-			$gd_formats = Array();
-			if (function_exists('ImageCreateFromJPEG')) {
-				$srcImage = @ImageCreateFromJPEG(_ROOT_IMG_PACK."test.jpg");
-				if ($srcImage) {
-					$gd_formats[] = "jpg";
-					ImageDestroy( $srcImage );
-				}
-			}
-			if (function_exists('ImageCreateFromGIF')) {
-				$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK."test.gif");
-				if ($srcImage) {
-					$gd_formats[] = "gif";
-					ImageDestroy( $srcImage );
-				}
-			}
-			if (function_exists('ImageCreateFromPNG')) {
-				$srcImage = @ImageCreateFromPNG(_ROOT_IMG_PACK."test.png");
-				if ($srcImage) {
-					$gd_formats[] = "png";
-					ImageDestroy( $srcImage );
-				}
-			}
-		}
-
-		if ($gd_formats) $gd_formats = join(",", $gd_formats);
-		ecrire_meta("gd_formats_read", $gd_formats.$gd_formats_read_gif);
-		ecrire_meta("gd_formats", $gd_formats);
-	}
-
-	// verifier les formats netpbm
-	else if ($arg == "netpbm") {
-		define('_PNMSCALE_COMMAND', 'pnmscale'); // chemin a changer dans mes_options
-		if (_PNMSCALE_COMMAND == '') return;
-		$netpbm_formats= Array();
-
-		$jpegtopnm_command = str_replace("pnmscale",
-			"jpegtopnm", _PNMSCALE_COMMAND);
-		$pnmtojpeg_command = str_replace("pnmscale",
-			"pnmtojpeg", _PNMSCALE_COMMAND);
-
-		$vignette = _ROOT_IMG_PACK."test.jpg";
-		$dest = _DIR_VAR . "test-jpg.jpg";
-		$commande = "$jpegtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
-		spip_log($commande);
-		exec($commande);
-		if ($taille = @getimagesize($dest)) {
-			if ($taille[1] == 10) $netpbm_formats[] = "jpg";
-		}
-		$giftopnm_command = str_replace("pnmscale", "giftopnm", _PNMSCALE_COMMAND);
-		$pnmtojpeg_command = str_replace("pnmscale", "pnmtojpeg", _PNMSCALE_COMMAND);
-		$vignette = _ROOT_IMG_PACK."test.gif";
-		$dest = _DIR_VAR . "test-gif.jpg";
-		$commande = "$giftopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
-		spip_log($commande);
-		exec($commande);
-		if ($taille = @getimagesize($dest)) {
-			if ($taille[1] == 10) $netpbm_formats[] = "gif";
-		}
-
-		$pngtopnm_command = str_replace("pnmscale", "pngtopnm", _PNMSCALE_COMMAND);
-		$vignette = _ROOT_IMG_PACK."test.png";
-		$dest = _DIR_VAR . "test-gif.jpg";
-		$commande = "$pngtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
-		spip_log($commande);
-		exec($commande);
-		if ($taille = @getimagesize($dest)) {
-			if ($taille[1] == 10) $netpbm_formats[] = "png";
-		}
-
-
-		if ($netpbm_formats)
-			$netpbm_formats = join(",", $netpbm_formats);
-		else
-			$netpbm_formats = '';
-		ecrire_meta("netpbm_formats", $netpbm_formats);
-	}
-
-	// et maintenant envoyer la vignette de tests
-	if (in_array($arg,array("gd1","gd2","imagick","convert","netpbm"))) {
-		include_spip('inc/filtres');
-		include_spip('inc/filtres_images_mini');
-		$taille_preview = 150;
-		$image = _image_valeurs_trans(_DIR_IMG_PACK.'test_image.jpg',"reduire-$taille_preview-$taille_preview",'jpg');
-
-		$image['fichier_dest']=_DIR_VAR."test_$arg";
-		if ($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true)
-		AND ($preview['width'] * $preview['height'] > 0))
-			redirige_par_entete($preview['fichier']);
-	}
-
-	# image echec
-	redirige_par_entete(chemin_image('puce-rouge-anim.gif'));
+    $arg = _request('arg');
+
+    $gd_formats = $gd_formats_read_gif = "";
+    // verifier les formats acceptes par GD
+    if ($arg == "gd1") {
+        // Si GD est installe et php >= 4.0.2
+        if (function_exists('imagetypes')) {
+
+            if (imagetypes() & IMG_GIF) {
+                $gd_formats[] = "gif";
+            } else {
+                # Attention GD sait lire le gif mais pas forcement l'ecrire
+                if (function_exists('ImageCreateFromGIF')) {
+                    $srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK."test.gif");
+                    if ($srcImage) {
+                        $gd_formats_read_gif = ",gif";
+                        ImageDestroy( $srcImage );
+                    }
+                }
+            }
+
+            if (imagetypes() & IMG_JPG)
+                $gd_formats[] = "jpg";
+            if (imagetypes() & IMG_PNG)
+                $gd_formats[] = "png";
+        }
+
+        else {	# ancienne methode de detection des formats, qui en plus
+                # est bugguee car elle teste les formats en lecture
+                # alors que la valeur deduite sert a identifier
+                # les formats disponibles en ecriture... (cf. inc_logos)
+
+            $gd_formats = Array();
+            if (function_exists('ImageCreateFromJPEG')) {
+                $srcImage = @ImageCreateFromJPEG(_ROOT_IMG_PACK."test.jpg");
+                if ($srcImage) {
+                    $gd_formats[] = "jpg";
+                    ImageDestroy( $srcImage );
+                }
+            }
+            if (function_exists('ImageCreateFromGIF')) {
+                $srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK."test.gif");
+                if ($srcImage) {
+                    $gd_formats[] = "gif";
+                    ImageDestroy( $srcImage );
+                }
+            }
+            if (function_exists('ImageCreateFromPNG')) {
+                $srcImage = @ImageCreateFromPNG(_ROOT_IMG_PACK."test.png");
+                if ($srcImage) {
+                    $gd_formats[] = "png";
+                    ImageDestroy( $srcImage );
+                }
+            }
+        }
+
+        if ($gd_formats) $gd_formats = join(",", $gd_formats);
+        ecrire_meta("gd_formats_read", $gd_formats.$gd_formats_read_gif);
+        ecrire_meta("gd_formats", $gd_formats);
+    }
+
+    // verifier les formats netpbm
+    else if ($arg == "netpbm") {
+        define('_PNMSCALE_COMMAND', 'pnmscale'); // chemin a changer dans mes_options
+        if (_PNMSCALE_COMMAND == '') return;
+        $netpbm_formats= Array();
+
+        $jpegtopnm_command = str_replace("pnmscale",
+            "jpegtopnm", _PNMSCALE_COMMAND);
+        $pnmtojpeg_command = str_replace("pnmscale",
+            "pnmtojpeg", _PNMSCALE_COMMAND);
+
+        $vignette = _ROOT_IMG_PACK."test.jpg";
+        $dest = _DIR_VAR . "test-jpg.jpg";
+        $commande = "$jpegtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
+        spip_log($commande);
+        exec($commande);
+        if ($taille = @getimagesize($dest)) {
+            if ($taille[1] == 10) $netpbm_formats[] = "jpg";
+        }
+        $giftopnm_command = str_replace("pnmscale", "giftopnm", _PNMSCALE_COMMAND);
+        $pnmtojpeg_command = str_replace("pnmscale", "pnmtojpeg", _PNMSCALE_COMMAND);
+        $vignette = _ROOT_IMG_PACK."test.gif";
+        $dest = _DIR_VAR . "test-gif.jpg";
+        $commande = "$giftopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
+        spip_log($commande);
+        exec($commande);
+        if ($taille = @getimagesize($dest)) {
+            if ($taille[1] == 10) $netpbm_formats[] = "gif";
+        }
+
+        $pngtopnm_command = str_replace("pnmscale", "pngtopnm", _PNMSCALE_COMMAND);
+        $vignette = _ROOT_IMG_PACK."test.png";
+        $dest = _DIR_VAR . "test-gif.jpg";
+        $commande = "$pngtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
+        spip_log($commande);
+        exec($commande);
+        if ($taille = @getimagesize($dest)) {
+            if ($taille[1] == 10) $netpbm_formats[] = "png";
+        }
+
+
+        if ($netpbm_formats)
+            $netpbm_formats = join(",", $netpbm_formats);
+        else
+            $netpbm_formats = '';
+        ecrire_meta("netpbm_formats", $netpbm_formats);
+    }
+
+    // et maintenant envoyer la vignette de tests
+    if (in_array($arg,array("gd1","gd2","imagick","convert","netpbm"))) {
+        include_spip('inc/filtres');
+        include_spip('inc/filtres_images_mini');
+        $taille_preview = 150;
+        $image = _image_valeurs_trans(_DIR_IMG_PACK.'test_image.jpg',"reduire-$taille_preview-$taille_preview",'jpg');
+
+        $image['fichier_dest']=_DIR_VAR."test_$arg";
+        if ($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true)
+        AND ($preview['width'] * $preview['height'] > 0))
+            redirige_par_entete($preview['fichier']);
+    }
+
+    # image echec
+    redirige_par_entete(chemin_image('puce-rouge-anim.gif'));
 }
 ?>

Spacing +11 added lines, -11 removed lines

@@ -31,7 +31,7 @@ 
 					$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK."test.gif");
 					if ($srcImage) {
 						$gd_formats_read_gif = ",gif";
-						ImageDestroy( $srcImage );
+						ImageDestroy($srcImage);
 					}
 				}
 			}
@@ -52,21 +52,21 @@ 
 				$srcImage = @ImageCreateFromJPEG(_ROOT_IMG_PACK."test.jpg");
 				if ($srcImage) {
 					$gd_formats[] = "jpg";
-					ImageDestroy( $srcImage );
+					ImageDestroy($srcImage);
 				}
 			}
 			if (function_exists('ImageCreateFromGIF')) {
 				$srcImage = @ImageCreateFromGIF(_ROOT_IMG_PACK."test.gif");
 				if ($srcImage) {
 					$gd_formats[] = "gif";
-					ImageDestroy( $srcImage );
+					ImageDestroy($srcImage);
 				}
 			}
 			if (function_exists('ImageCreateFromPNG')) {
 				$srcImage = @ImageCreateFromPNG(_ROOT_IMG_PACK."test.png");
 				if ($srcImage) {
 					$gd_formats[] = "png";
-					ImageDestroy( $srcImage );
+					ImageDestroy($srcImage);
 				}
 			}
 		}
@@ -80,7 +80,7 @@ 
 	else if ($arg == "netpbm") {
 		define('_PNMSCALE_COMMAND', 'pnmscale'); // chemin a changer dans mes_options
 		if (_PNMSCALE_COMMAND == '') return;
-		$netpbm_formats= Array();
+		$netpbm_formats = Array();
 
 		$jpegtopnm_command = str_replace("pnmscale",
 			"jpegtopnm", _PNMSCALE_COMMAND);
@@ -88,7 +88,7 @@ 
 			"pnmtojpeg", _PNMSCALE_COMMAND);
 
 		$vignette = _ROOT_IMG_PACK."test.jpg";
-		$dest = _DIR_VAR . "test-jpg.jpg";
+		$dest = _DIR_VAR."test-jpg.jpg";
 		$commande = "$jpegtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_log($commande);
 		exec($commande);
@@ -98,7 +98,7 @@ 
 		$giftopnm_command = str_replace("pnmscale", "giftopnm", _PNMSCALE_COMMAND);
 		$pnmtojpeg_command = str_replace("pnmscale", "pnmtojpeg", _PNMSCALE_COMMAND);
 		$vignette = _ROOT_IMG_PACK."test.gif";
-		$dest = _DIR_VAR . "test-gif.jpg";
+		$dest = _DIR_VAR."test-gif.jpg";
 		$commande = "$giftopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_log($commande);
 		exec($commande);
@@ -108,7 +108,7 @@ 
 
 		$pngtopnm_command = str_replace("pnmscale", "pngtopnm", _PNMSCALE_COMMAND);
 		$vignette = _ROOT_IMG_PACK."test.png";
-		$dest = _DIR_VAR . "test-gif.jpg";
+		$dest = _DIR_VAR."test-gif.jpg";
 		$commande = "$pngtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_log($commande);
 		exec($commande);
@@ -125,13 +125,13 @@ 
 	}
 
 	// et maintenant envoyer la vignette de tests
-	if (in_array($arg,array("gd1","gd2","imagick","convert","netpbm"))) {
+	if (in_array($arg, array("gd1", "gd2", "imagick", "convert", "netpbm"))) {
 		include_spip('inc/filtres');
 		include_spip('inc/filtres_images_mini');
 		$taille_preview = 150;
-		$image = _image_valeurs_trans(_DIR_IMG_PACK.'test_image.jpg',"reduire-$taille_preview-$taille_preview",'jpg');
+		$image = _image_valeurs_trans(_DIR_IMG_PACK.'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
 
-		$image['fichier_dest']=_DIR_VAR."test_$arg";
+		$image['fichier_dest'] = _DIR_VAR."test_$arg";
 		if ($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true)
 		AND ($preview['width'] * $preview['height'] > 0))
 			redirige_par_entete($preview['fichier']);

Braces +33 added lines, -19 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;
+}
 
 // Tester nos capacites
 // http://doc.spip.org/@action_tester_dist
@@ -36,13 +38,13 @@ 
 				}
 			}
 
-			if (imagetypes() & IMG_JPG)
-				$gd_formats[] = "jpg";
-			if (imagetypes() & IMG_PNG)
-				$gd_formats[] = "png";
-		}
-
-		else {	# ancienne methode de detection des formats, qui en plus
+			if (imagetypes() & IMG_JPG) {
+							$gd_formats[] = "jpg";
+			}
+			if (imagetypes() & IMG_PNG) {
+							$gd_formats[] = "png";
+			}
+		} else {	# ancienne methode de detection des formats, qui en plus
 				# est bugguee car elle teste les formats en lecture
 				# alors que la valeur deduite sert a identifier
 				# les formats disponibles en ecriture... (cf. inc_logos)
@@ -71,7 +73,9 @@ 
 			}
 		}
 
-		if ($gd_formats) $gd_formats = join(",", $gd_formats);
+		if ($gd_formats) {
+		    $gd_formats = join(",", $gd_formats);
+		}
 		ecrire_meta("gd_formats_read", $gd_formats.$gd_formats_read_gif);
 		ecrire_meta("gd_formats", $gd_formats);
 	}
@@ -79,7 +83,9 @@ 
 	// verifier les formats netpbm
 	else if ($arg == "netpbm") {
 		define('_PNMSCALE_COMMAND', 'pnmscale'); // chemin a changer dans mes_options
-		if (_PNMSCALE_COMMAND == '') return;
+		if (_PNMSCALE_COMMAND == '') {
+		    return;
+		}
 		$netpbm_formats= Array();
 
 		$jpegtopnm_command = str_replace("pnmscale",
@@ -93,7 +99,9 @@ 
 		spip_log($commande);
 		exec($commande);
 		if ($taille = @getimagesize($dest)) {
-			if ($taille[1] == 10) $netpbm_formats[] = "jpg";
+			if ($taille[1] == 10) {
+			    $netpbm_formats[] = "jpg";
+			}
 		}
 		$giftopnm_command = str_replace("pnmscale", "giftopnm", _PNMSCALE_COMMAND);
 		$pnmtojpeg_command = str_replace("pnmscale", "pnmtojpeg", _PNMSCALE_COMMAND);
@@ -103,7 +111,9 @@ 
 		spip_log($commande);
 		exec($commande);
 		if ($taille = @getimagesize($dest)) {
-			if ($taille[1] == 10) $netpbm_formats[] = "gif";
+			if ($taille[1] == 10) {
+			    $netpbm_formats[] = "gif";
+			}
 		}
 
 		$pngtopnm_command = str_replace("pnmscale", "pngtopnm", _PNMSCALE_COMMAND);
@@ -113,14 +123,17 @@ 
 		spip_log($commande);
 		exec($commande);
 		if ($taille = @getimagesize($dest)) {
-			if ($taille[1] == 10) $netpbm_formats[] = "png";
+			if ($taille[1] == 10) {
+			    $netpbm_formats[] = "png";
+			}
 		}
 
 
-		if ($netpbm_formats)
-			$netpbm_formats = join(",", $netpbm_formats);
-		else
-			$netpbm_formats = '';
+		if ($netpbm_formats) {
+					$netpbm_formats = join(",", $netpbm_formats);
+		} else {
+					$netpbm_formats = '';
+		}
 		ecrire_meta("netpbm_formats", $netpbm_formats);
 	}
 
@@ -133,8 +146,9 @@ 
 
 		$image['fichier_dest']=_DIR_VAR."test_$arg";
 		if ($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true)
-		AND ($preview['width'] * $preview['height'] > 0))
-			redirige_par_entete($preview['fichier']);
+		AND ($preview['width'] * $preview['height'] > 0)) {
+					redirige_par_entete($preview['fichier']);
+		}
 	}
 
 	# image echec

ecrire/action/supprimer_rubrique.php

Indentation +31 added lines, -31 removed lines

@@ -24,42 +24,42 @@
  */
 function action_supprimer_rubrique_dist($id_rubrique=null) {
 
-	if (is_null($id_rubrique)){
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$id_rubrique = $securiser_action();
-	}
+    if (is_null($id_rubrique)){
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $id_rubrique = $securiser_action();
+    }
 
-	if (intval($id_rubrique)){
+    if (intval($id_rubrique)){
 
-		sql_delete("spip_rubriques", "id_rubrique=".intval($id_rubrique));
-		// Les admin restreints qui n'administraient que cette rubrique
-		// deviennent redacteurs
-		// (il y a sans doute moyen de faire ca avec un having)
+        sql_delete("spip_rubriques", "id_rubrique=".intval($id_rubrique));
+        // Les admin restreints qui n'administraient que cette rubrique
+        // deviennent redacteurs
+        // (il y a sans doute moyen de faire ca avec un having)
 
-		$q = sql_select("id_auteur", "spip_auteurs_liens", "objet='rubrique' AND id_objet=".intval($id_rubrique));
-		while ($r = sql_fetch($q)) {
-			$id_auteur = $r['id_auteur'];
-			// degrader avant de supprimer la restriction d'admin
-			// section critique sur les droits
-			$n = sql_countsel("spip_auteurs_liens", "objet='rubrique' AND id_objet!=".intval($id_rubrique)." AND id_auteur=".intval($id_auteur));
-			if (!$n) {
-				include_spip('action/editer_auteur');
-				auteurs_set($id_auteur,array("statut" => '1comite'));
-			}
-			sql_delete("spip_auteurs_liens", "objet='rubrique' AND id_objet=".intval($id_rubrique)." AND id_auteur=".intval($id_auteur));
-		}
-		// menu_rubriques devra recalculer
-		effacer_meta("date_calcul_rubriques");
+        $q = sql_select("id_auteur", "spip_auteurs_liens", "objet='rubrique' AND id_objet=".intval($id_rubrique));
+        while ($r = sql_fetch($q)) {
+            $id_auteur = $r['id_auteur'];
+            // degrader avant de supprimer la restriction d'admin
+            // section critique sur les droits
+            $n = sql_countsel("spip_auteurs_liens", "objet='rubrique' AND id_objet!=".intval($id_rubrique)." AND id_auteur=".intval($id_auteur));
+            if (!$n) {
+                include_spip('action/editer_auteur');
+                auteurs_set($id_auteur,array("statut" => '1comite'));
+            }
+            sql_delete("spip_auteurs_liens", "objet='rubrique' AND id_objet=".intval($id_rubrique)." AND id_auteur=".intval($id_auteur));
+        }
+        // menu_rubriques devra recalculer
+        effacer_meta("date_calcul_rubriques");
 
-		// Une rubrique supprimable n'avait pas le statut "publie"
-		// donc rien de neuf pour la rubrique parente
-		include_spip('inc/rubriques');
-		calculer_langues_rubriques();
+        // Une rubrique supprimable n'avait pas le statut "publie"
+        // donc rien de neuf pour la rubrique parente
+        include_spip('inc/rubriques');
+        calculer_langues_rubriques();
 
-		// invalider les caches marques de cette rubrique
-		include_spip('inc/invalideur');
-		suivre_invalideur("id='rubrique/$id_rubrique'");
-	}
+        // invalider les caches marques de cette rubrique
+        include_spip('inc/invalideur');
+        suivre_invalideur("id='rubrique/$id_rubrique'");
+    }
 }
 
 ?>

Spacing +5 added lines, -5 removed lines

@@ -12,7 +12,7 @@ 
 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
-include_spip('inc/charsets');	# pour le nom de fichier
+include_spip('inc/charsets'); # pour le nom de fichier
 
 /**
  * Effacer une rubrique
@@ -22,14 +22,14 @@ 
  * @param null $id_rubrique
  * @return void
  */
-function action_supprimer_rubrique_dist($id_rubrique=null) {
+function action_supprimer_rubrique_dist($id_rubrique = null) {
 
-	if (is_null($id_rubrique)){
+	if (is_null($id_rubrique)) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$id_rubrique = $securiser_action();
 	}
 
-	if (intval($id_rubrique)){
+	if (intval($id_rubrique)) {
 
 		sql_delete("spip_rubriques", "id_rubrique=".intval($id_rubrique));
 		// Les admin restreints qui n'administraient que cette rubrique
@@ -44,7 +44,7 @@ 
 			$n = sql_countsel("spip_auteurs_liens", "objet='rubrique' AND id_objet!=".intval($id_rubrique)." AND id_auteur=".intval($id_auteur));
 			if (!$n) {
 				include_spip('action/editer_auteur');
-				auteurs_set($id_auteur,array("statut" => '1comite'));
+				auteurs_set($id_auteur, array("statut" => '1comite'));
 			}
 			sql_delete("spip_auteurs_liens", "objet='rubrique' AND id_objet=".intval($id_rubrique)." AND id_auteur=".intval($id_auteur));
 		}

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;
+}
 
 include_spip('inc/charsets');	# pour le nom de fichier
 

ecrire/action/session.php

Indentation +11 added lines, -11 removed lines

@@ -18,18 +18,18 @@
 // http://doc.spip.org/@action_session_dist
 function action_session_dist()
 {
-	if ($var = _request('var')
-	AND preg_match(',^[a-z_0-9-]+$,i', $var)
-	) {
-		if ($_SERVER['REQUEST_METHOD'] == 'POST'){
-			include_spip('inc/session');
-			session_set('session_'.$var, $val=_request('val'));
-			#spip_log("autosave:$var:$val",'autosave');
-		}
-	}
+    if ($var = _request('var')
+    AND preg_match(',^[a-z_0-9-]+$,i', $var)
+    ) {
+        if ($_SERVER['REQUEST_METHOD'] == 'POST'){
+            include_spip('inc/session');
+            session_set('session_'.$var, $val=_request('val'));
+            #spip_log("autosave:$var:$val",'autosave');
+        }
+    }
 
-	# TODO: mode lecture de session ; n'afficher que ce qu'il faut
-	#echo json_encode($GLOBALS['visiteur_session']);
+    # TODO: mode lecture de session ; n'afficher que ce qu'il faut
+    #echo json_encode($GLOBALS['visiteur_session']);
 }
 
 

Spacing +2 added lines, -2 removed lines

@@ -21,9 +21,9 @@
 	if ($var = _request('var')
 	AND preg_match(',^[a-z_0-9-]+$,i', $var)
 	) {
-		if ($_SERVER['REQUEST_METHOD'] == 'POST'){
+		if ($_SERVER['REQUEST_METHOD'] == 'POST') {
 			include_spip('inc/session');
-			session_set('session_'.$var, $val=_request('val'));
+			session_set('session_'.$var, $val = _request('val'));
 			#spip_log("autosave:$var:$val",'autosave');
 		}
 	}

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;
+}
 
 // Pour poser une variable de session
 // poster sur cette action en indiquant var/val

ecrire/action/editer_article.php

Indentation +271 added lines, -271 removed lines

@@ -14,32 +14,32 @@ 
 
 // http://doc.spip.org/@action_editer_article_dist
 function action_editer_article_dist($arg=null) {
-	include_spip('inc/autoriser');
-	$err="";
-	if (is_null($arg)){
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
-
-	// si id_article n'est pas un nombre, c'est une creation 
-	// mais on verifie qu'on a toutes les donnees qu'il faut.
-	if (!$id_article = intval($arg)) {
-		$id_parent = _request('id_parent');
-		if (!$id_parent)
-			$err = _L("creation interdite d'un article sans rubrique");
-		elseif(!autoriser('creerarticledans','rubrique',$id_parent))
-			$err = _T("info_creerdansrubrique_non_autorise");
-		else
-			$id_article = article_inserer($id_parent);
-	}
-
-	// Enregistre l'envoi dans la BD
-	if ($id_article > 0) $err = article_modifier($id_article);
-
-	if ($err)
-		spip_log("echec editeur article: $err",_LOG_ERREUR);
-
-	return array($id_article,$err);
+    include_spip('inc/autoriser');
+    $err="";
+    if (is_null($arg)){
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
+
+    // si id_article n'est pas un nombre, c'est une creation 
+    // mais on verifie qu'on a toutes les donnees qu'il faut.
+    if (!$id_article = intval($arg)) {
+        $id_parent = _request('id_parent');
+        if (!$id_parent)
+            $err = _L("creation interdite d'un article sans rubrique");
+        elseif(!autoriser('creerarticledans','rubrique',$id_parent))
+            $err = _T("info_creerdansrubrique_non_autorise");
+        else
+            $id_article = article_inserer($id_parent);
+    }
+
+    // Enregistre l'envoi dans la BD
+    if ($id_article > 0) $err = article_modifier($id_article);
+
+    if ($err)
+        spip_log("echec editeur article: $err",_LOG_ERREUR);
+
+    return array($id_article,$err);
 }
 
 /**
@@ -53,44 +53,44 @@ 
  */
 function article_modifier($id_article, $set=null) {
 
-	// unifier $texte en cas de texte trop long
-	trop_longs_articles();
-
-	include_spip('inc/modifier');
-	include_spip('inc/filtres');
-	$c = collecter_requests(
-		// white list
-		objet_info('article','champs_editables'),
-		// black list
-		array('date','statut','id_parent'),
-		// donnees eventuellement fournies
-		$set
-	);
-
-	// Si l'article est publie, invalider les caches et demander sa reindexation
-	$t = sql_getfetsel("statut", "spip_articles", "id_article=".intval($id_article));
-	$invalideur = $indexation = false;
-	if ($t == 'publie') {
-		$invalideur = "id='article/$id_article'";
-		$indexation = true;
-	}
-
-	if ($err = objet_modifier_champs('article', $id_article,
-		array(
-			'data' => $set,
-			'nonvide' => array('titre' => _T('info_nouvel_article')." "._T('info_numero_abbreviation').$id_article),
-			'invalideur' => $invalideur,
-			'indexation' => $indexation,
-			'date_modif' => 'date_modif' // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
-		),
-		$c))
-		return $err;
-
-	// Modification de statut, changement de rubrique ?
-	$c = collecter_requests(array('date', 'statut', 'id_parent'),array(),$set);
-	$err = article_instituer($id_article, $c);
-
-	return $err;
+    // unifier $texte en cas de texte trop long
+    trop_longs_articles();
+
+    include_spip('inc/modifier');
+    include_spip('inc/filtres');
+    $c = collecter_requests(
+        // white list
+        objet_info('article','champs_editables'),
+        // black list
+        array('date','statut','id_parent'),
+        // donnees eventuellement fournies
+        $set
+    );
+
+    // Si l'article est publie, invalider les caches et demander sa reindexation
+    $t = sql_getfetsel("statut", "spip_articles", "id_article=".intval($id_article));
+    $invalideur = $indexation = false;
+    if ($t == 'publie') {
+        $invalideur = "id='article/$id_article'";
+        $indexation = true;
+    }
+
+    if ($err = objet_modifier_champs('article', $id_article,
+        array(
+            'data' => $set,
+            'nonvide' => array('titre' => _T('info_nouvel_article')." "._T('info_numero_abbreviation').$id_article),
+            'invalideur' => $invalideur,
+            'indexation' => $indexation,
+            'date_modif' => 'date_modif' // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
+        ),
+        $c))
+        return $err;
+
+    // Modification de statut, changement de rubrique ?
+    $c = collecter_requests(array('date', 'statut', 'id_parent'),array(),$set);
+    $err = article_instituer($id_article, $c);
+
+    return $err;
 }
 
 /**
@@ -103,81 +103,81 @@ 
 function article_inserer($id_rubrique) {
 
 
-	// Si id_rubrique vaut 0 ou n'est pas definie, creer l'article
-	// dans la premiere rubrique racine
-	if (!$id_rubrique = intval($id_rubrique)) {
-		$row = sql_fetsel("id_rubrique, id_secteur, lang", "spip_rubriques", "id_parent=0",'', '0+titre,titre', "1");
-		$id_rubrique = $row['id_rubrique'];
-	} else $row = sql_fetsel("lang, id_secteur", "spip_rubriques", "id_rubrique=$id_rubrique");
+    // Si id_rubrique vaut 0 ou n'est pas definie, creer l'article
+    // dans la premiere rubrique racine
+    if (!$id_rubrique = intval($id_rubrique)) {
+        $row = sql_fetsel("id_rubrique, id_secteur, lang", "spip_rubriques", "id_parent=0",'', '0+titre,titre', "1");
+        $id_rubrique = $row['id_rubrique'];
+    } else $row = sql_fetsel("lang, id_secteur", "spip_rubriques", "id_rubrique=$id_rubrique");
 
-	// eviter $id_secteur = NULL (erreur sqlite) si la requete precedente echoue 
-	// cas de id_rubrique = -1 par exemple avec plugin "pages"
-	$id_secteur = isset($row['id_secteur']) ? $row['id_secteur'] : 0;
+    // eviter $id_secteur = NULL (erreur sqlite) si la requete precedente echoue 
+    // cas de id_rubrique = -1 par exemple avec plugin "pages"
+    $id_secteur = isset($row['id_secteur']) ? $row['id_secteur'] : 0;
 	
-	$lang_rub = $row['lang'];
-
-	$lang = "";
-	$choisie = 'non';
-	// La langue a la creation : si les liens de traduction sont autorises
-	// dans les rubriques, on essaie avec la langue de l'auteur,
-	// ou a defaut celle de la rubrique
-	// Sinon c'est la langue de la rubrique qui est choisie + heritee
-	if (in_array('spip_articles',explode(',',$GLOBALS['meta']['multi_objets']))) {
-		lang_select($GLOBALS['visiteur_session']['lang']);
-		if (in_array($GLOBALS['spip_lang'],
-		explode(',', $GLOBALS['meta']['langues_multilingue']))) {
-			$lang = $GLOBALS['spip_lang'];
-			$choisie = 'oui';
-		}
-	}
-
-	if (!$lang) {
-		$choisie = 'non';
-		$lang = $lang_rub ? $lang_rub : $GLOBALS['meta']['langue_site'];
-	}
-
-	$champs = array(
-		'id_rubrique' => $id_rubrique,
-		'id_secteur' =>  $id_secteur,
-		'statut' =>  'prepa',
-		'date' => date('Y-m-d H:i:s'),
-		'lang' => $lang,
-		'langue_choisie' =>$choisie);
-
-	// Envoyer aux plugins
-	$champs = pipeline('pre_insertion',
-		array(
-			'args' => array(
-				'table' => 'spip_articles',
-			),
-			'data' => $champs
-		)
-	);
-
-	$id_article = sql_insertq("spip_articles", $champs);
-
-	pipeline('post_insertion',
-		array(
-			'args' => array(
-				'table' => 'spip_articles',
-				'id_objet' => $id_article
-			),
-			'data' => $champs
-		)
-	);
-
-	// controler si le serveur n'a pas renvoye une erreur
-	if ($id_article > 0){
-		$id_auteur = ((is_null(_request('id_auteur')) AND isset($GLOBALS['visiteur_session']['id_auteur']))?
-			$GLOBALS['visiteur_session']['id_auteur']
-			:_request('id_auteur'));
-		if ($id_auteur){
-			include_spip('action/editer_auteur');
-			auteur_associer($id_auteur, array('article'=>$id_article));
-		}
-	}
-
-	return $id_article;
+    $lang_rub = $row['lang'];
+
+    $lang = "";
+    $choisie = 'non';
+    // La langue a la creation : si les liens de traduction sont autorises
+    // dans les rubriques, on essaie avec la langue de l'auteur,
+    // ou a defaut celle de la rubrique
+    // Sinon c'est la langue de la rubrique qui est choisie + heritee
+    if (in_array('spip_articles',explode(',',$GLOBALS['meta']['multi_objets']))) {
+        lang_select($GLOBALS['visiteur_session']['lang']);
+        if (in_array($GLOBALS['spip_lang'],
+        explode(',', $GLOBALS['meta']['langues_multilingue']))) {
+            $lang = $GLOBALS['spip_lang'];
+            $choisie = 'oui';
+        }
+    }
+
+    if (!$lang) {
+        $choisie = 'non';
+        $lang = $lang_rub ? $lang_rub : $GLOBALS['meta']['langue_site'];
+    }
+
+    $champs = array(
+        'id_rubrique' => $id_rubrique,
+        'id_secteur' =>  $id_secteur,
+        'statut' =>  'prepa',
+        'date' => date('Y-m-d H:i:s'),
+        'lang' => $lang,
+        'langue_choisie' =>$choisie);
+
+    // Envoyer aux plugins
+    $champs = pipeline('pre_insertion',
+        array(
+            'args' => array(
+                'table' => 'spip_articles',
+            ),
+            'data' => $champs
+        )
+    );
+
+    $id_article = sql_insertq("spip_articles", $champs);
+
+    pipeline('post_insertion',
+        array(
+            'args' => array(
+                'table' => 'spip_articles',
+                'id_objet' => $id_article
+            ),
+            'data' => $champs
+        )
+    );
+
+    // controler si le serveur n'a pas renvoye une erreur
+    if ($id_article > 0){
+        $id_auteur = ((is_null(_request('id_auteur')) AND isset($GLOBALS['visiteur_session']['id_auteur']))?
+            $GLOBALS['visiteur_session']['id_auteur']
+            :_request('id_auteur'));
+        if ($id_auteur){
+            include_spip('action/editer_auteur');
+            auteur_associer($id_auteur, array('article'=>$id_article));
+        }
+    }
+
+    return $id_article;
 }
 
 
@@ -188,111 +188,111 @@ 
 // http://doc.spip.org/@instituer_article
 function article_instituer($id_article, $c, $calcul_rub=true) {
 
-	include_spip('inc/autoriser');
-	include_spip('inc/rubriques');
-	include_spip('inc/modifier');
-
-	$row = sql_fetsel("statut, date, id_rubrique", "spip_articles", "id_article=$id_article");
-	$id_rubrique = $row['id_rubrique'];
-	$statut_ancien = $statut = $row['statut'];
-	$date_ancienne = $date = $row['date'];
-	$champs = array();
-
-	$d = isset($c['date'])?$c['date']:null;
-	$s = isset($c['statut'])?$c['statut']:$statut;
-
-	// cf autorisations dans inc/instituer_article
-	if ($s != $statut OR ($d AND $d != $date)) {
-		if (autoriser('publierdans', 'rubrique', $id_rubrique))
-			$statut = $champs['statut'] = $s;
-		else if (autoriser('modifier', 'article', $id_article) AND $s != 'publie')
-			$statut = $champs['statut'] = $s;
-		else
-			spip_log("editer_article $id_article refus " . join(' ', $c));
-
-		// En cas de publication, fixer la date a "maintenant"
-		// sauf si $c commande autre chose
-		// ou si l'article est deja date dans le futur
-		// En cas de proposition d'un article (mais pas depublication), idem
-		if ($champs['statut'] == 'publie'
-		 OR ($champs['statut'] == 'prop' AND ($d OR !in_array($statut_ancien, array('publie', 'prop'))))
-		) {
-			if ($d OR strtotime($d=$date)>time())
-				$champs['date'] = $date = $d;
-			else
-				$champs['date'] = $date = date('Y-m-d H:i:s');
-		}
-	}
-
-	// Verifier que la rubrique demandee existe et est differente
-	// de la rubrique actuelle
-	if ($id_parent = $c['id_parent']
-	AND $id_parent != $id_rubrique
-	AND (sql_fetsel('1', "spip_rubriques", "id_rubrique=$id_parent"))) {
-		$champs['id_rubrique'] = $id_parent;
-
-		// si l'article etait publie
-		// et que le demandeur n'est pas admin de la rubrique de destination
-		// repasser l'article en statut 'propose'.
-		if ($statut == 'publie'
-		AND !autoriser('publierdans', 'rubrique', $id_parent))
-			$champs['statut'] = 'prop';
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline('pre_edition',
-		array(
-			'args' => array(
-				'table' => 'spip_articles',
-				'id_objet' => $id_article,
-				'action'=>'instituer',
-				'statut_ancien' => $statut_ancien,
-				'date_ancienne' => $date_ancienne,
-			),
-			'data' => $champs
-		)
-	);
-
-	if (!count($champs)) return '';
-
-	// Envoyer les modifs.
-
-	editer_article_heritage($id_article, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
-
-	// Invalider les caches
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='article/$id_article'");
-
-	if ($date) {
-		$t = strtotime($date);
-		$p = @$GLOBALS['meta']['date_prochain_postdate'];
-		if ($t > time() AND (!$p OR ($t < $p))) {
-			ecrire_meta('date_prochain_postdate', $t);
-		}
-	}
-
-	// Pipeline
-	pipeline('post_edition',
-		array(
-			'args' => array(
-				'table' => 'spip_articles',
-				'id_objet' => $id_article,
-				'action'=>'instituer',
-				'statut_ancien' => $statut_ancien,
-				'date_ancienne' => $date_ancienne,
-			),
-			'data' => $champs
-		)
-	);
-
-	// Notifications
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications('instituerarticle', $id_article,
-			array('statut' => $statut, 'statut_ancien' => $statut_ancien, 'date'=>$date, 'date_ancienne' => $date_ancienne)
-		);
-	}
-
-	return ''; // pas d'erreur
+    include_spip('inc/autoriser');
+    include_spip('inc/rubriques');
+    include_spip('inc/modifier');
+
+    $row = sql_fetsel("statut, date, id_rubrique", "spip_articles", "id_article=$id_article");
+    $id_rubrique = $row['id_rubrique'];
+    $statut_ancien = $statut = $row['statut'];
+    $date_ancienne = $date = $row['date'];
+    $champs = array();
+
+    $d = isset($c['date'])?$c['date']:null;
+    $s = isset($c['statut'])?$c['statut']:$statut;
+
+    // cf autorisations dans inc/instituer_article
+    if ($s != $statut OR ($d AND $d != $date)) {
+        if (autoriser('publierdans', 'rubrique', $id_rubrique))
+            $statut = $champs['statut'] = $s;
+        else if (autoriser('modifier', 'article', $id_article) AND $s != 'publie')
+            $statut = $champs['statut'] = $s;
+        else
+            spip_log("editer_article $id_article refus " . join(' ', $c));
+
+        // En cas de publication, fixer la date a "maintenant"
+        // sauf si $c commande autre chose
+        // ou si l'article est deja date dans le futur
+        // En cas de proposition d'un article (mais pas depublication), idem
+        if ($champs['statut'] == 'publie'
+         OR ($champs['statut'] == 'prop' AND ($d OR !in_array($statut_ancien, array('publie', 'prop'))))
+        ) {
+            if ($d OR strtotime($d=$date)>time())
+                $champs['date'] = $date = $d;
+            else
+                $champs['date'] = $date = date('Y-m-d H:i:s');
+        }
+    }
+
+    // Verifier que la rubrique demandee existe et est differente
+    // de la rubrique actuelle
+    if ($id_parent = $c['id_parent']
+    AND $id_parent != $id_rubrique
+    AND (sql_fetsel('1', "spip_rubriques", "id_rubrique=$id_parent"))) {
+        $champs['id_rubrique'] = $id_parent;
+
+        // si l'article etait publie
+        // et que le demandeur n'est pas admin de la rubrique de destination
+        // repasser l'article en statut 'propose'.
+        if ($statut == 'publie'
+        AND !autoriser('publierdans', 'rubrique', $id_parent))
+            $champs['statut'] = 'prop';
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline('pre_edition',
+        array(
+            'args' => array(
+                'table' => 'spip_articles',
+                'id_objet' => $id_article,
+                'action'=>'instituer',
+                'statut_ancien' => $statut_ancien,
+                'date_ancienne' => $date_ancienne,
+            ),
+            'data' => $champs
+        )
+    );
+
+    if (!count($champs)) return '';
+
+    // Envoyer les modifs.
+
+    editer_article_heritage($id_article, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
+
+    // Invalider les caches
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='article/$id_article'");
+
+    if ($date) {
+        $t = strtotime($date);
+        $p = @$GLOBALS['meta']['date_prochain_postdate'];
+        if ($t > time() AND (!$p OR ($t < $p))) {
+            ecrire_meta('date_prochain_postdate', $t);
+        }
+    }
+
+    // Pipeline
+    pipeline('post_edition',
+        array(
+            'args' => array(
+                'table' => 'spip_articles',
+                'id_objet' => $id_article,
+                'action'=>'instituer',
+                'statut_ancien' => $statut_ancien,
+                'date_ancienne' => $date_ancienne,
+            ),
+            'data' => $champs
+        )
+    );
+
+    // Notifications
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications('instituerarticle', $id_article,
+            array('statut' => $statut, 'statut_ancien' => $statut_ancien, 'date'=>$date, 'date_ancienne' => $date_ancienne)
+        );
+    }
+
+    return ''; // pas d'erreur
 }
 
 // fabrique la requete de modification de l'article, avec champs herites
@@ -300,30 +300,30 @@ 
 // http://doc.spip.org/@editer_article_heritage
 function editer_article_heritage($id_article, $id_rubrique, $statut, $champs, $cond=true) {
 
-	// Si on deplace l'article
-	//  changer aussi son secteur et sa langue (si heritee)
-	if (isset($champs['id_rubrique'])) {
+    // Si on deplace l'article
+    //  changer aussi son secteur et sa langue (si heritee)
+    if (isset($champs['id_rubrique'])) {
 
-		$row_rub = sql_fetsel("id_secteur, lang", "spip_rubriques", "id_rubrique=".sql_quote($champs['id_rubrique']));
+        $row_rub = sql_fetsel("id_secteur, lang", "spip_rubriques", "id_rubrique=".sql_quote($champs['id_rubrique']));
 
-		$langue = $row_rub['lang'];
-		$champs['id_secteur'] = $row_rub['id_secteur'];
-		if (sql_fetsel('1', 'spip_articles', "id_article=$id_article AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue))) {
-			$champs['lang'] = $langue;
-		}
-	}
+        $langue = $row_rub['lang'];
+        $champs['id_secteur'] = $row_rub['id_secteur'];
+        if (sql_fetsel('1', 'spip_articles', "id_article=$id_article AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue))) {
+            $champs['lang'] = $langue;
+        }
+    }
 
-	if (!$champs) return;
+    if (!$champs) return;
 
-	sql_updateq('spip_articles', $champs, "id_article=$id_article");
+    sql_updateq('spip_articles', $champs, "id_article=$id_article");
 
-	// Changer le statut des rubriques concernees 
+    // Changer le statut des rubriques concernees 
 
-	if ($cond) {
-		include_spip('inc/rubriques');
-		$postdate = ($GLOBALS['meta']["post_dates"] == "non" AND isset($champs['date']) AND (strtotime($champs['date']) < time()))?$champs['date']:false;
-		calculer_rubriques_if($id_rubrique, $champs, $statut, $postdate);
-	}
+    if ($cond) {
+        include_spip('inc/rubriques');
+        $postdate = ($GLOBALS['meta']["post_dates"] == "non" AND isset($champs['date']) AND (strtotime($champs['date']) < time()))?$champs['date']:false;
+        calculer_rubriques_if($id_rubrique, $champs, $statut, $postdate);
+    }
 }
 
 //
@@ -332,29 +332,29 @@ 
 
 // http://doc.spip.org/@trop_longs_articles
 function trop_longs_articles() {
-	if (is_array($plus = _request('texte_plus'))) {
-		foreach ($plus as $n=>$t) {
-			$plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,","", $t);
-		}
-		set_request('texte', join('',$plus) . _request('texte'));
-	}
+    if (is_array($plus = _request('texte_plus'))) {
+        foreach ($plus as $n=>$t) {
+            $plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,","", $t);
+        }
+        set_request('texte', join('',$plus) . _request('texte'));
+    }
 }
 
 
 // obsoletes
 function revisions_articles ($id_article, $c=false) {
-	return article_modifier($id_article,$c);
+    return article_modifier($id_article,$c);
 }
 function revision_article ($id_article, $c=false) {
-	return article_modifier($id_article,$c);
+    return article_modifier($id_article,$c);
 }
 function articles_set($id_article, $set=null) {
-	return article_modifier($id_article,$set);
+    return article_modifier($id_article,$set);
 }
 function insert_article($id_rubrique) {
-	return article_inserer($id_rubrique);
+    return article_inserer($id_rubrique);
 }
 function instituer_article($id_article, $c, $calcul_rub=true) {
-	return article_instituer($id_article,$c,$calcul_rub);
+    return article_instituer($id_article,$c,$calcul_rub);
 }
 ?>

Spacing +33 added lines, -33 removed lines

@@ -13,10 +13,10 @@ 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 // http://doc.spip.org/@action_editer_article_dist
-function action_editer_article_dist($arg=null) {
+function action_editer_article_dist($arg = null) {
 	include_spip('inc/autoriser');
-	$err="";
-	if (is_null($arg)){
+	$err = "";
+	if (is_null($arg)) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 	}
@@ -27,7 +27,7 @@ 
 		$id_parent = _request('id_parent');
 		if (!$id_parent)
 			$err = _L("creation interdite d'un article sans rubrique");
-		elseif(!autoriser('creerarticledans','rubrique',$id_parent))
+		elseif (!autoriser('creerarticledans', 'rubrique', $id_parent))
 			$err = _T("info_creerdansrubrique_non_autorise");
 		else
 			$id_article = article_inserer($id_parent);
@@ -37,9 +37,9 @@ 
 	if ($id_article > 0) $err = article_modifier($id_article);
 
 	if ($err)
-		spip_log("echec editeur article: $err",_LOG_ERREUR);
+		spip_log("echec editeur article: $err", _LOG_ERREUR);
 
-	return array($id_article,$err);
+	return array($id_article, $err);
 }
 
 /**
@@ -51,7 +51,7 @@ 
  * @param null $set
  * @return string
  */
-function article_modifier($id_article, $set=null) {
+function article_modifier($id_article, $set = null) {
 
 	// unifier $texte en cas de texte trop long
 	trop_longs_articles();
@@ -60,9 +60,9 @@ 
 	include_spip('inc/filtres');
 	$c = collecter_requests(
 		// white list
-		objet_info('article','champs_editables'),
+		objet_info('article', 'champs_editables'),
 		// black list
-		array('date','statut','id_parent'),
+		array('date', 'statut', 'id_parent'),
 		// donnees eventuellement fournies
 		$set
 	);
@@ -87,7 +87,7 @@ 
 		return $err;
 
 	// Modification de statut, changement de rubrique ?
-	$c = collecter_requests(array('date', 'statut', 'id_parent'),array(),$set);
+	$c = collecter_requests(array('date', 'statut', 'id_parent'), array(), $set);
 	$err = article_instituer($id_article, $c);
 
 	return $err;
@@ -106,7 +106,7 @@ 
 	// Si id_rubrique vaut 0 ou n'est pas definie, creer l'article
 	// dans la premiere rubrique racine
 	if (!$id_rubrique = intval($id_rubrique)) {
-		$row = sql_fetsel("id_rubrique, id_secteur, lang", "spip_rubriques", "id_parent=0",'', '0+titre,titre', "1");
+		$row = sql_fetsel("id_rubrique, id_secteur, lang", "spip_rubriques", "id_parent=0", '', '0+titre,titre', "1");
 		$id_rubrique = $row['id_rubrique'];
 	} else $row = sql_fetsel("lang, id_secteur", "spip_rubriques", "id_rubrique=$id_rubrique");
 
@@ -122,7 +122,7 @@ 
 	// dans les rubriques, on essaie avec la langue de l'auteur,
 	// ou a defaut celle de la rubrique
 	// Sinon c'est la langue de la rubrique qui est choisie + heritee
-	if (in_array('spip_articles',explode(',',$GLOBALS['meta']['multi_objets']))) {
+	if (in_array('spip_articles', explode(',', $GLOBALS['meta']['multi_objets']))) {
 		lang_select($GLOBALS['visiteur_session']['lang']);
 		if (in_array($GLOBALS['spip_lang'],
 		explode(',', $GLOBALS['meta']['langues_multilingue']))) {
@@ -167,11 +167,11 @@ 
 	);
 
 	// controler si le serveur n'a pas renvoye une erreur
-	if ($id_article > 0){
-		$id_auteur = ((is_null(_request('id_auteur')) AND isset($GLOBALS['visiteur_session']['id_auteur']))?
+	if ($id_article > 0) {
+		$id_auteur = ((is_null(_request('id_auteur')) AND isset($GLOBALS['visiteur_session']['id_auteur'])) ?
 			$GLOBALS['visiteur_session']['id_auteur']
 			:_request('id_auteur'));
-		if ($id_auteur){
+		if ($id_auteur) {
 			include_spip('action/editer_auteur');
 			auteur_associer($id_auteur, array('article'=>$id_article));
 		}
@@ -186,7 +186,7 @@ 
 // statut et rubrique sont lies, car un admin restreint peut deplacer
 // un article publie vers une rubrique qu'il n'administre pas
 // http://doc.spip.org/@instituer_article
-function article_instituer($id_article, $c, $calcul_rub=true) {
+function article_instituer($id_article, $c, $calcul_rub = true) {
 
 	include_spip('inc/autoriser');
 	include_spip('inc/rubriques');
@@ -198,8 +198,8 @@ 
 	$date_ancienne = $date = $row['date'];
 	$champs = array();
 
-	$d = isset($c['date'])?$c['date']:null;
-	$s = isset($c['statut'])?$c['statut']:$statut;
+	$d = isset($c['date']) ? $c['date'] : null;
+	$s = isset($c['statut']) ? $c['statut'] : $statut;
 
 	// cf autorisations dans inc/instituer_article
 	if ($s != $statut OR ($d AND $d != $date)) {
@@ -208,7 +208,7 @@ 
 		else if (autoriser('modifier', 'article', $id_article) AND $s != 'publie')
 			$statut = $champs['statut'] = $s;
 		else
-			spip_log("editer_article $id_article refus " . join(' ', $c));
+			spip_log("editer_article $id_article refus ".join(' ', $c));
 
 		// En cas de publication, fixer la date a "maintenant"
 		// sauf si $c commande autre chose
@@ -217,7 +217,7 @@ 
 		if ($champs['statut'] == 'publie'
 		 OR ($champs['statut'] == 'prop' AND ($d OR !in_array($statut_ancien, array('publie', 'prop'))))
 		) {
-			if ($d OR strtotime($d=$date)>time())
+			if ($d OR strtotime($d = $date) > time())
 				$champs['date'] = $date = $d;
 			else
 				$champs['date'] = $date = date('Y-m-d H:i:s');
@@ -298,7 +298,7 @@ 
 // fabrique la requete de modification de l'article, avec champs herites
 
 // http://doc.spip.org/@editer_article_heritage
-function editer_article_heritage($id_article, $id_rubrique, $statut, $champs, $cond=true) {
+function editer_article_heritage($id_article, $id_rubrique, $statut, $champs, $cond = true) {
 
 	// Si on deplace l'article
 	//  changer aussi son secteur et sa langue (si heritee)
@@ -308,7 +308,7 @@ 
 
 		$langue = $row_rub['lang'];
 		$champs['id_secteur'] = $row_rub['id_secteur'];
-		if (sql_fetsel('1', 'spip_articles', "id_article=$id_article AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue))) {
+		if (sql_fetsel('1', 'spip_articles', "id_article=$id_article AND langue_choisie<>'oui' AND lang<>".sql_quote($langue))) {
 			$champs['lang'] = $langue;
 		}
 	}
@@ -321,7 +321,7 @@ 
 
 	if ($cond) {
 		include_spip('inc/rubriques');
-		$postdate = ($GLOBALS['meta']["post_dates"] == "non" AND isset($champs['date']) AND (strtotime($champs['date']) < time()))?$champs['date']:false;
+		$postdate = ($GLOBALS['meta']["post_dates"] == "non" AND isset($champs['date']) AND (strtotime($champs['date']) < time())) ? $champs['date'] : false;
 		calculer_rubriques_if($id_rubrique, $champs, $statut, $postdate);
 	}
 }
@@ -334,27 +334,27 @@ 
 function trop_longs_articles() {
 	if (is_array($plus = _request('texte_plus'))) {
 		foreach ($plus as $n=>$t) {
-			$plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,","", $t);
+			$plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,", "", $t);
 		}
-		set_request('texte', join('',$plus) . _request('texte'));
+		set_request('texte', join('', $plus)._request('texte'));
 	}
 }
 
 
 // obsoletes
-function revisions_articles ($id_article, $c=false) {
-	return article_modifier($id_article,$c);
+function revisions_articles($id_article, $c = false) {
+	return article_modifier($id_article, $c);
 }
-function revision_article ($id_article, $c=false) {
-	return article_modifier($id_article,$c);
+function revision_article($id_article, $c = false) {
+	return article_modifier($id_article, $c);
 }
-function articles_set($id_article, $set=null) {
-	return article_modifier($id_article,$set);
+function articles_set($id_article, $set = null) {
+	return article_modifier($id_article, $set);
 }
 function insert_article($id_rubrique) {
 	return article_inserer($id_rubrique);
 }
-function instituer_article($id_article, $c, $calcul_rub=true) {
-	return article_instituer($id_article,$c,$calcul_rub);
+function instituer_article($id_article, $c, $calcul_rub = true) {
+	return article_instituer($id_article, $c, $calcul_rub);
 }
 ?>

Braces +43 added lines, -27 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_editer_article_dist
 function action_editer_article_dist($arg=null) {
@@ -25,19 +27,23 @@ 
 	// mais on verifie qu'on a toutes les donnees qu'il faut.
 	if (!$id_article = intval($arg)) {
 		$id_parent = _request('id_parent');
-		if (!$id_parent)
-			$err = _L("creation interdite d'un article sans rubrique");
-		elseif(!autoriser('creerarticledans','rubrique',$id_parent))
-			$err = _T("info_creerdansrubrique_non_autorise");
-		else
-			$id_article = article_inserer($id_parent);
+		if (!$id_parent) {
+					$err = _L("creation interdite d'un article sans rubrique");
+		} elseif(!autoriser('creerarticledans','rubrique',$id_parent)) {
+					$err = _T("info_creerdansrubrique_non_autorise");
+		} else {
+					$id_article = article_inserer($id_parent);
+		}
 	}
 
 	// Enregistre l'envoi dans la BD
-	if ($id_article > 0) $err = article_modifier($id_article);
+	if ($id_article > 0) {
+	    $err = article_modifier($id_article);
+	}
 
-	if ($err)
-		spip_log("echec editeur article: $err",_LOG_ERREUR);
+	if ($err) {
+			spip_log("echec editeur article: $err",_LOG_ERREUR);
+	}
 
 	return array($id_article,$err);
 }
@@ -83,8 +89,9 @@ 
 			'indexation' => $indexation,
 			'date_modif' => 'date_modif' // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
 		),
-		$c))
-		return $err;
+		$c)) {
+			return $err;
+	}
 
 	// Modification de statut, changement de rubrique ?
 	$c = collecter_requests(array('date', 'statut', 'id_parent'),array(),$set);
@@ -108,7 +115,9 @@ 
 	if (!$id_rubrique = intval($id_rubrique)) {
 		$row = sql_fetsel("id_rubrique, id_secteur, lang", "spip_rubriques", "id_parent=0",'', '0+titre,titre', "1");
 		$id_rubrique = $row['id_rubrique'];
-	} else $row = sql_fetsel("lang, id_secteur", "spip_rubriques", "id_rubrique=$id_rubrique");
+	} else {
+	    $row = sql_fetsel("lang, id_secteur", "spip_rubriques", "id_rubrique=$id_rubrique");
+	}
 
 	// eviter $id_secteur = NULL (erreur sqlite) si la requete precedente echoue 
 	// cas de id_rubrique = -1 par exemple avec plugin "pages"
@@ -203,12 +212,13 @@ 
 
 	// cf autorisations dans inc/instituer_article
 	if ($s != $statut OR ($d AND $d != $date)) {
-		if (autoriser('publierdans', 'rubrique', $id_rubrique))
-			$statut = $champs['statut'] = $s;
-		else if (autoriser('modifier', 'article', $id_article) AND $s != 'publie')
-			$statut = $champs['statut'] = $s;
-		else
-			spip_log("editer_article $id_article refus " . join(' ', $c));
+		if (autoriser('publierdans', 'rubrique', $id_rubrique)) {
+					$statut = $champs['statut'] = $s;
+		} else if (autoriser('modifier', 'article', $id_article) AND $s != 'publie') {
+					$statut = $champs['statut'] = $s;
+		} else {
+					spip_log("editer_article $id_article refus " . join(' ', $c));
+		}
 
 		// En cas de publication, fixer la date a "maintenant"
 		// sauf si $c commande autre chose
@@ -217,10 +227,11 @@ 
 		if ($champs['statut'] == 'publie'
 		 OR ($champs['statut'] == 'prop' AND ($d OR !in_array($statut_ancien, array('publie', 'prop'))))
 		) {
-			if ($d OR strtotime($d=$date)>time())
-				$champs['date'] = $date = $d;
-			else
-				$champs['date'] = $date = date('Y-m-d H:i:s');
+			if ($d OR strtotime($d=$date)>time()) {
+							$champs['date'] = $date = $d;
+			} else {
+							$champs['date'] = $date = date('Y-m-d H:i:s');
+			}
 		}
 	}
 
@@ -235,8 +246,9 @@ 
 		// et que le demandeur n'est pas admin de la rubrique de destination
 		// repasser l'article en statut 'propose'.
 		if ($statut == 'publie'
-		AND !autoriser('publierdans', 'rubrique', $id_parent))
-			$champs['statut'] = 'prop';
+		AND !autoriser('publierdans', 'rubrique', $id_parent)) {
+					$champs['statut'] = 'prop';
+		}
 	}
 
 	// Envoyer aux plugins
@@ -253,7 +265,9 @@ 
 		)
 	);
 
-	if (!count($champs)) return '';
+	if (!count($champs)) {
+	    return '';
+	}
 
 	// Envoyer les modifs.
 
@@ -313,7 +327,9 @@ 
 		}
 	}
 
-	if (!$champs) return;
+	if (!$champs) {
+	    return;
+	}
 
 	sql_updateq('spip_articles', $champs, "id_article=$id_article");
 

ecrire/action/instituer_objet.php

Indentation +12 added lines, -12 removed lines

@@ -19,21 +19,21 @@
  */
 function action_instituer_objet_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();
+    }
 
-	list($objet, $id_objet, $statut) = preg_split('/\W/', $arg);
-	if (!$statut) $statut = _request('statut_nouv'); // cas POST
-	if (!$statut) return; // impossible mais sait-on jamais
+    list($objet, $id_objet, $statut) = preg_split('/\W/', $arg);
+    if (!$statut) $statut = _request('statut_nouv'); // cas POST
+    if (!$statut) return; // impossible mais sait-on jamais
 
-	if ($id_objet = intval($id_objet)
-		AND autoriser('instituer',$objet,$id_objet,'',array('statut'=>$statut))){
+    if ($id_objet = intval($id_objet)
+        AND autoriser('instituer',$objet,$id_objet,'',array('statut'=>$statut))){
 
-		include_spip('action/editer_objet');
-		objet_modifier($objet,$id_objet,array('statut' => $statut));
-	}
+        include_spip('action/editer_objet');
+        objet_modifier($objet,$id_objet,array('statut' => $statut));
+    }
 
 }
 

Spacing +4 added lines, -4 removed lines

@@ -17,9 +17,9 @@ 
  * @param null|string $arg
  * @return 
  */
-function action_instituer_objet_dist($arg=null) {
+function action_instituer_objet_dist($arg = null) {
 
-	if (is_null($arg)){
+	if (is_null($arg)) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 	}
@@ -29,10 +29,10 @@ 
 	if (!$statut) return; // impossible mais sait-on jamais
 
 	if ($id_objet = intval($id_objet)
-		AND autoriser('instituer',$objet,$id_objet,'',array('statut'=>$statut))){
+		AND autoriser('instituer', $objet, $id_objet, '', array('statut'=>$statut))) {
 
 		include_spip('action/editer_objet');
-		objet_modifier($objet,$id_objet,array('statut' => $statut));
+		objet_modifier($objet, $id_objet, array('statut' => $statut));
 	}
 
 }

Braces +11 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;
+}
 
 /**
  * Instituer un objet avec les puces rapides
@@ -25,8 +27,14 @@ 
 	}
 
 	list($objet, $id_objet, $statut) = preg_split('/\W/', $arg);
-	if (!$statut) $statut = _request('statut_nouv'); // cas POST
-	if (!$statut) return; // impossible mais sait-on jamais
+	if (!$statut) {
+	    $statut = _request('statut_nouv');
+	}
+	// cas POST
+	if (!$statut) {
+	    return;
+	}
+	// impossible mais sait-on jamais
 
 	if ($id_objet = intval($id_objet)
 		AND autoriser('instituer',$objet,$id_objet,'',array('statut'=>$statut))){

ecrire/action/logout.php

Indentation +74 added lines, -74 removed lines

@@ -25,63 +25,63 @@ 
  */
 function action_logout_dist()
 {
-	$logout =_request('logout');
-	$url = securiser_redirect_action(_request('url'));
-	// cas particulier, logout dans l'espace public
-	if ($logout == 'public' AND !$url)
-		$url = url_de_base();
+    $logout =_request('logout');
+    $url = securiser_redirect_action(_request('url'));
+    // cas particulier, logout dans l'espace public
+    if ($logout == 'public' AND !$url)
+        $url = url_de_base();
 
-	// seul le loge peut se deloger (mais id_auteur peut valoir 0 apres une restauration avortee)
-	if (isset($GLOBALS['visiteur_session']['id_auteur']) 
-		AND is_numeric($GLOBALS['visiteur_session']['id_auteur'])
-		// des sessions anonymes avec id_auteur=0 existent, mais elle n'ont pas de statut : double check
-	  AND isset($GLOBALS['visiteur_session']['statut'])) {
+    // seul le loge peut se deloger (mais id_auteur peut valoir 0 apres une restauration avortee)
+    if (isset($GLOBALS['visiteur_session']['id_auteur']) 
+        AND is_numeric($GLOBALS['visiteur_session']['id_auteur'])
+        // des sessions anonymes avec id_auteur=0 existent, mais elle n'ont pas de statut : double check
+      AND isset($GLOBALS['visiteur_session']['statut'])) {
 
-		// il faut un jeton pour fermer la session (eviter les CSRF)
-		if (!$jeton = _request('jeton')
-		 OR !verifier_jeton_logout($jeton,$GLOBALS['visiteur_session'])){
-			$jeton = generer_jeton_logout($GLOBALS['visiteur_session']);
-			$action = generer_url_action("logout","jeton=$jeton");
-			$action = parametre_url($action,'logout',_request('logout'));
-			$action = parametre_url($action,'url',_request('url'));
-			include_spip("inc/minipres");
-			include_spip("inc/filtres");
-			$texte = bouton_action(_T('spip:icone_deconnecter'),$action);
-			$texte = "<div class='boutons'>$texte</div>";
-			$texte .= '<script type="text/javascript">document.write("<style>body{visibility:hidden;}</style>");window.document.forms[0].submit();</script>';
-			$res = minipres(_T('spip:icone_deconnecter'),$texte,'',true);
-			echo $res;
-			return;
-		}
+        // il faut un jeton pour fermer la session (eviter les CSRF)
+        if (!$jeton = _request('jeton')
+         OR !verifier_jeton_logout($jeton,$GLOBALS['visiteur_session'])){
+            $jeton = generer_jeton_logout($GLOBALS['visiteur_session']);
+            $action = generer_url_action("logout","jeton=$jeton");
+            $action = parametre_url($action,'logout',_request('logout'));
+            $action = parametre_url($action,'url',_request('url'));
+            include_spip("inc/minipres");
+            include_spip("inc/filtres");
+            $texte = bouton_action(_T('spip:icone_deconnecter'),$action);
+            $texte = "<div class='boutons'>$texte</div>";
+            $texte .= '<script type="text/javascript">document.write("<style>body{visibility:hidden;}</style>");window.document.forms[0].submit();</script>';
+            $res = minipres(_T('spip:icone_deconnecter'),$texte,'',true);
+            echo $res;
+            return;
+        }
 
-		include_spip('inc/auth');
-		auth_trace($GLOBALS['visiteur_session'], '0000-00-00 00:00:00');
-	// le logout explicite vaut destruction de toutes les sessions
-		if (isset($_COOKIE['spip_session'])) {
-			$session = charger_fonction('session', 'inc');
-			$session($GLOBALS['visiteur_session']['id_auteur']);
-			spip_setcookie('spip_session', $_COOKIE['spip_session'], time()-3600);
-		}
-		// si authentification http, et que la personne est loge,
-		// pour se deconnecter, il faut proposer un nouveau formulaire de connexion http
-		if (isset($_SERVER['PHP_AUTH_USER'])
-			AND !$GLOBALS['ignore_auth_http']
-			AND $GLOBALS['auth_can_disconnect']) {
-			  ask_php_auth(_T('login_deconnexion_ok'),
-				       _T('login_verifiez_navigateur'),
-				       _T('login_retour_public'),
-				       	"redirect=". _DIR_RESTREINT_ABS, 
-				       _T('login_test_navigateur'),
-				       true);
+        include_spip('inc/auth');
+        auth_trace($GLOBALS['visiteur_session'], '0000-00-00 00:00:00');
+    // le logout explicite vaut destruction de toutes les sessions
+        if (isset($_COOKIE['spip_session'])) {
+            $session = charger_fonction('session', 'inc');
+            $session($GLOBALS['visiteur_session']['id_auteur']);
+            spip_setcookie('spip_session', $_COOKIE['spip_session'], time()-3600);
+        }
+        // si authentification http, et que la personne est loge,
+        // pour se deconnecter, il faut proposer un nouveau formulaire de connexion http
+        if (isset($_SERVER['PHP_AUTH_USER'])
+            AND !$GLOBALS['ignore_auth_http']
+            AND $GLOBALS['auth_can_disconnect']) {
+                ask_php_auth(_T('login_deconnexion_ok'),
+                        _T('login_verifiez_navigateur'),
+                        _T('login_retour_public'),
+                            "redirect=". _DIR_RESTREINT_ABS, 
+                        _T('login_test_navigateur'),
+                        true);
 			
-		}
-	}
+        }
+    }
 
-	// Rediriger en contrant le cache navigateur (Safari3)
-	include_spip('inc/headers');
-	redirige_par_entete($url
-		? parametre_url($url, 'var_hasard', uniqid(rand()), '&')
-		: generer_url_public('login'));
+    // Rediriger en contrant le cache navigateur (Safari3)
+    include_spip('inc/headers');
+    redirige_par_entete($url
+        ? parametre_url($url, 'var_hasard', uniqid(rand()), '&')
+        : generer_url_public('login'));
 }
 
 /**
@@ -91,20 +91,20 @@ 
  * @return string
  */
 function generer_jeton_logout($session,$alea=null){
-	if (is_null($alea)){
-		if (!isset($GLOBALS['meta']['alea_ephemere'])){
-			include_spip('base/abstract_sql');
-			$GLOBALS['meta']['alea_ephemere'] = sql_getfetsel('valeur', 'spip_meta', "nom='alea_ephemere'");
-		}
-		$alea = $GLOBALS['meta']['alea_ephemere'];
-	}
+    if (is_null($alea)){
+        if (!isset($GLOBALS['meta']['alea_ephemere'])){
+            include_spip('base/abstract_sql');
+            $GLOBALS['meta']['alea_ephemere'] = sql_getfetsel('valeur', 'spip_meta', "nom='alea_ephemere'");
+        }
+        $alea = $GLOBALS['meta']['alea_ephemere'];
+    }
 
-	$jeton = md5($session['date_session']
-	  .$session['id_auteur']
-	  .$session['statut']
-	  .$alea
-	);
-	return $jeton;
+    $jeton = md5($session['date_session']
+        .$session['id_auteur']
+        .$session['statut']
+        .$alea
+    );
+    return $jeton;
 }
 
 /**
@@ -116,15 +116,15 @@ 
  * @return bool
  */
 function verifier_jeton_logout($jeton,$session){
-	if (generer_jeton_logout($session)===$jeton)
-		return true;
-	if (!isset($GLOBALS['meta']['alea_ephemere_ancien'])){
-		include_spip('base/abstract_sql');
-		$GLOBALS['meta']['alea_ephemere_ancien'] = sql_getfetsel('valeur', 'spip_meta', "nom='alea_ephemere_ancien'");
-	}
-	if (generer_jeton_logout($session,$GLOBALS['meta']['alea_ephemere_ancien'])===$jeton)
-		return true;
-	return false;
+    if (generer_jeton_logout($session)===$jeton)
+        return true;
+    if (!isset($GLOBALS['meta']['alea_ephemere_ancien'])){
+        include_spip('base/abstract_sql');
+        $GLOBALS['meta']['alea_ephemere_ancien'] = sql_getfetsel('valeur', 'spip_meta', "nom='alea_ephemere_ancien'");
+    }
+    if (generer_jeton_logout($session,$GLOBALS['meta']['alea_ephemere_ancien'])===$jeton)
+        return true;
+    return false;
 }
 
 ?>
\ No newline at end of file

Spacing +16 added lines, -16 removed lines

@@ -25,7 +25,7 @@ 
  */
 function action_logout_dist()
 {
-	$logout =_request('logout');
+	$logout = _request('logout');
 	$url = securiser_redirect_action(_request('url'));
 	// cas particulier, logout dans l'espace public
 	if ($logout == 'public' AND !$url)
@@ -39,17 +39,17 @@ 
 
 		// il faut un jeton pour fermer la session (eviter les CSRF)
 		if (!$jeton = _request('jeton')
-		 OR !verifier_jeton_logout($jeton,$GLOBALS['visiteur_session'])){
+		 OR !verifier_jeton_logout($jeton, $GLOBALS['visiteur_session'])) {
 			$jeton = generer_jeton_logout($GLOBALS['visiteur_session']);
-			$action = generer_url_action("logout","jeton=$jeton");
-			$action = parametre_url($action,'logout',_request('logout'));
-			$action = parametre_url($action,'url',_request('url'));
+			$action = generer_url_action("logout", "jeton=$jeton");
+			$action = parametre_url($action, 'logout', _request('logout'));
+			$action = parametre_url($action, 'url', _request('url'));
 			include_spip("inc/minipres");
 			include_spip("inc/filtres");
-			$texte = bouton_action(_T('spip:icone_deconnecter'),$action);
+			$texte = bouton_action(_T('spip:icone_deconnecter'), $action);
 			$texte = "<div class='boutons'>$texte</div>";
 			$texte .= '<script type="text/javascript">document.write("<style>body{visibility:hidden;}</style>");window.document.forms[0].submit();</script>';
-			$res = minipres(_T('spip:icone_deconnecter'),$texte,'',true);
+			$res = minipres(_T('spip:icone_deconnecter'), $texte, '', true);
 			echo $res;
 			return;
 		}
@@ -60,7 +60,7 @@ 
 		if (isset($_COOKIE['spip_session'])) {
 			$session = charger_fonction('session', 'inc');
 			$session($GLOBALS['visiteur_session']['id_auteur']);
-			spip_setcookie('spip_session', $_COOKIE['spip_session'], time()-3600);
+			spip_setcookie('spip_session', $_COOKIE['spip_session'], time() - 3600);
 		}
 		// si authentification http, et que la personne est loge,
 		// pour se deconnecter, il faut proposer un nouveau formulaire de connexion http
@@ -70,7 +70,7 @@ 
 			  ask_php_auth(_T('login_deconnexion_ok'),
 				       _T('login_verifiez_navigateur'),
 				       _T('login_retour_public'),
-				       	"redirect=". _DIR_RESTREINT_ABS, 
+				       	"redirect="._DIR_RESTREINT_ABS, 
 				       _T('login_test_navigateur'),
 				       true);
 			
@@ -90,9 +90,9 @@ 
  * @param null|string $alea
  * @return string
  */
-function generer_jeton_logout($session,$alea=null){
-	if (is_null($alea)){
-		if (!isset($GLOBALS['meta']['alea_ephemere'])){
+function generer_jeton_logout($session, $alea = null) {
+	if (is_null($alea)) {
+		if (!isset($GLOBALS['meta']['alea_ephemere'])) {
 			include_spip('base/abstract_sql');
 			$GLOBALS['meta']['alea_ephemere'] = sql_getfetsel('valeur', 'spip_meta', "nom='alea_ephemere'");
 		}
@@ -115,14 +115,14 @@ 
  * @param array $session
  * @return bool
  */
-function verifier_jeton_logout($jeton,$session){
-	if (generer_jeton_logout($session)===$jeton)
+function verifier_jeton_logout($jeton, $session) {
+	if (generer_jeton_logout($session) === $jeton)
 		return true;
-	if (!isset($GLOBALS['meta']['alea_ephemere_ancien'])){
+	if (!isset($GLOBALS['meta']['alea_ephemere_ancien'])) {
 		include_spip('base/abstract_sql');
 		$GLOBALS['meta']['alea_ephemere_ancien'] = sql_getfetsel('valeur', 'spip_meta', "nom='alea_ephemere_ancien'");
 	}
-	if (generer_jeton_logout($session,$GLOBALS['meta']['alea_ephemere_ancien'])===$jeton)
+	if (generer_jeton_logout($session, $GLOBALS['meta']['alea_ephemere_ancien']) === $jeton)
 		return true;
 	return false;
 }

Braces +12 added lines, -7 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;
+}
 
 include_spip('inc/cookie');
 
@@ -28,8 +30,9 @@ 
 	$logout =_request('logout');
 	$url = securiser_redirect_action(_request('url'));
 	// cas particulier, logout dans l'espace public
-	if ($logout == 'public' AND !$url)
-		$url = url_de_base();
+	if ($logout == 'public' AND !$url) {
+			$url = url_de_base();
+	}
 
 	// seul le loge peut se deloger (mais id_auteur peut valoir 0 apres une restauration avortee)
 	if (isset($GLOBALS['visiteur_session']['id_auteur']) 
@@ -116,14 +119,16 @@ 
  * @return bool
  */
 function verifier_jeton_logout($jeton,$session){
-	if (generer_jeton_logout($session)===$jeton)
-		return true;
+	if (generer_jeton_logout($session)===$jeton) {
+			return true;
+	}
 	if (!isset($GLOBALS['meta']['alea_ephemere_ancien'])){
 		include_spip('base/abstract_sql');
 		$GLOBALS['meta']['alea_ephemere_ancien'] = sql_getfetsel('valeur', 'spip_meta', "nom='alea_ephemere_ancien'");
 	}
-	if (generer_jeton_logout($session,$GLOBALS['meta']['alea_ephemere_ancien'])===$jeton)
-		return true;
+	if (generer_jeton_logout($session,$GLOBALS['meta']['alea_ephemere_ancien'])===$jeton) {
+			return true;
+	}
 	return false;
 }