spip / SPIP

ecrire/action/tester.php

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

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

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

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

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

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');
 
@@ -30,15 +32,17 @@ 
 	$lang = action_converser_changer_langue($update_session);
 	$redirect = rawurldecode(_request('redirect'));
 
-	if (!$redirect) $redirect = _DIR_RESTREINT_ABS;
+	if (!$redirect) {
+	    $redirect = _DIR_RESTREINT_ABS;
+	}
 	$redirect = parametre_url($redirect,'lang',$lang,'&');
 	redirige_par_entete($redirect, true);
 }
 
 function action_converser_changer_langue($update_session){
-	if ($lang = _request('var_lang'))
-		action_converser_post($lang);
-	elseif ($lang = _request('var_lang_ecrire')) {
+	if ($lang = _request('var_lang')) {
+			action_converser_post($lang);
+	} elseif ($lang = _request('var_lang_ecrire')) {
 		if ($update_session) {
 			sql_updateq("spip_auteurs", array("lang" => $lang), "id_auteur = " . $GLOBALS['visiteur_session']['id_auteur']);
 			$GLOBALS['visiteur_session']['lang'] = $lang;
@@ -63,8 +67,9 @@ 
 		include_spip('inc/lang');
 		if (changer_langue($lang)) {
 			spip_setcookie('spip_lang', $_COOKIE['spip_lang'] = $lang, time() + 365 * 24 * 3600);
-			if ($ecrire)
-				spip_setcookie('spip_lang_ecrire', $_COOKIE['spip_lang_ecrire'] = $lang, time() + 365 * 24 * 3600);
+			if ($ecrire) {
+							spip_setcookie('spip_lang_ecrire', $_COOKIE['spip_lang_ecrire'] = $lang, time() + 365 * 24 * 3600);
+			}
 		}
 	}
 }

ecrire/action/logout.php

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

ecrire/action/referencer_traduction.php

Braces +9 added lines, -4 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;
+}
 
 /**
  * Definir le lien de traduction vers un objet de reference
@@ -26,7 +28,9 @@ 
 function action_referencer_traduction_dist($objet, $id_objet, $id_trad) {
 
 	// ne rien faire si id_trad est ambigu
-	if (!is_numeric($id_trad)) return false;
+	if (!is_numeric($id_trad)) {
+	    return false;
+	}
 
 	$table_objet_sql = table_objet_sql($objet);
 	$id_table_objet = id_table_objet($objet);
@@ -65,8 +69,9 @@ 
 
 		// Verifier si l'ancien groupe ne comporte plus qu'un seul objet. Alors mettre a zero.
 		$cpt = sql_countsel($table_objet_sql, "id_trad=".intval($old_id_trad));
-		if ($cpt == 1)
-			sql_updateq($table_objet_sql, array("id_trad" => 0), "id_trad=".intval($old_id_trad));
+		if ($cpt == 1) {
+					sql_updateq($table_objet_sql, array("id_trad" => 0), "id_trad=".intval($old_id_trad));
+		}
 	}
 
 	return true;

ecrire/action/desinstaller_plugin.php

Braces +12 added lines, -6 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_desinstaller_plugin_dist
 function action_desinstaller_plugin_dist() {
@@ -19,19 +21,23 @@ 
 	$arg = $securiser_action();
 	list($dir_plugins,$plugin) = explode("::",$arg);
 	$dir_type = "_DIR_PLUGINS";
-	if (defined('_DIR_PLUGINS_SUPPL') AND $dir_plugins==_DIR_PLUGINS_SUPPL)
-		$dir_type = "_DIR_PLUGINS_SUPPL";
+	if (defined('_DIR_PLUGINS_SUPPL') AND $dir_plugins==_DIR_PLUGINS_SUPPL) {
+			$dir_type = "_DIR_PLUGINS_SUPPL";
+	}
 	$installer_plugins = charger_fonction('installer', 'plugins');
 	$infos = $installer_plugins($plugin, 'uninstall',$dir_type);
 	if ($infos AND !$infos['install_test'][0]) {
 		include_spip('inc/plugin');
 		ecrire_plugin_actifs(array($plugin),false,'enleve');
 		$erreur = '';
-	} else 	$erreur = 'erreur_plugin_desinstalation_echouee';
+	} else {
+	    $erreur = 'erreur_plugin_desinstalation_echouee';
+	}
 	if ($redirect = _request('redirect')){
 		include_spip('inc/headers');
-		if ($erreur)
-			$redirect = parametre_url($redirect, 'erreur',$erreur);
+		if ($erreur) {
+					$redirect = parametre_url($redirect, 'erreur',$erreur);
+		}
 		$redirect = str_replace('&','&',$redirect);
 		redirige_par_entete($redirect);
 	}