spip / SPIP

ecrire/action/confirmer_email.php

Braces +6 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;
+}
 
 
 // Cette action permet de confirmer un changement d'email
@@ -27,8 +29,9 @@ 
 	}
 	// verifier avant de rediriger pour invalider le message de confirmation
 	// si ca n'a pas marche
-	if ($redirect = _request('redirect') AND !$arg == sql_getfetsel('email', 'spip_auteurs', 'id_auteur='.intval($GLOBALS['visiteur_session'])))
-		$GLOBALS['redirect'] = parametre_url($redirect, 'email_modif','');
+	if ($redirect = _request('redirect') AND !$arg == sql_getfetsel('email', 'spip_auteurs', 'id_auteur='.intval($GLOBALS['visiteur_session']))) {
+			$GLOBALS['redirect'] = parametre_url($redirect, 'email_modif','');
+	}
 
 }
 

ecrire/action/super_cron.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;
+}
 
 /**
  * Url pour lancer le cron de manière asynchrone si le serveur

ecrire/action/auth.php

Braces +4 added lines, -3 removed lines

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

ecrire/action/purger.php

Braces +3 added lines, -1 removed lines

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

ecrire/action/iconifier.php

Braces +40 added lines, -33 removed lines

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

ecrire/action/purger_queue.php

Braces +3 added lines, -1 removed lines

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

ecrire/action/calculer_taille_cache.php

Braces +14 added lines, -11 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;
+}
 
 
 
@@ -34,8 +36,7 @@ 
 			'taille' => "<b>".(taille_en_octets($taille) > 0 ? taille_en_octets($taille) : "0 octet")."</b>"
 			)
 		);
-	}
-	else {
+	} else {
 		include_spip('inc/invalideur');
 		$taille =
 			calculer_taille_dossier(_DIR_CACHE_XML)
@@ -46,11 +47,9 @@ 
 		$taille += intval(taille_du_cache());
 		if ($taille<=150000){
 			$res = _T('taille_cache_vide');
-		}
-		elseif ($taille<=1024*1024){
+		} elseif ($taille<=1024*1024){
 			$res = _T('taille_cache_moins_de',array('octets'=>taille_en_octets(1024*1024)));
-		}
-		else {
+		} else {
 			$res = _T('taille_cache_octets',array('octets'=>taille_en_octets($taille)));
 		}
 		$res = "<b>$res</b>";
@@ -71,16 +70,20 @@ 
  */
 function calculer_taille_dossier ($dir) {
 	$handle = @opendir($dir);
-	if (!$handle) return 0;
+	if (!$handle) {
+	    return 0;
+	}
 	$taille = 0;
 	while (($fichier = @readdir($handle)) !== false) {
 		// Eviter ".", "..", ".htaccess", etc.
-		if ($fichier[0] == '.') continue;
+		if ($fichier[0] == '.') {
+		    continue;
+		}
 		if (is_file($d = "$dir/$fichier")) {
 			$taille += filesize($d);
+		} else if (is_dir($d)) {
+					$taille += calculer_taille_dossier($d);
 		}
-		else if (is_dir($d))
-			$taille += calculer_taille_dossier($d);
 	}
 	closedir($handle);
 	return $taille;

ecrire/action/annuler_job.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;
+}
 
 /**
  * Annuler un travail

ecrire/action/editer_liens.php

Braces +63 added lines, -41 removed lines

@@ -16,7 +16,9 @@ 
  * @package SPIP\Liens\API
  */
  
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 
 
@@ -39,8 +41,9 @@ 
 	if ($primary = id_table_objet($objet)
 	  AND $trouver_table($l = $table_sql."_liens")
 		AND !preg_match(',[^\w],',$primary)
-		AND !preg_match(',[^\w],',$l))
-		return array($primary,$l);
+		AND !preg_match(',[^\w],',$l)) {
+			return array($primary,$l);
+	}
 
 	spip_log("Objet $objet non associable : ne dispose pas d'une cle primaire $primary OU d'une table liens $l");
 	return false;
@@ -71,8 +74,9 @@ 
 function objet_associer($objets_source, $objets_lies, $qualif = null){
 	$modifs = objet_traiter_liaisons('lien_insert', $objets_source, $objets_lies);
 
-	if ($qualif)
-		objet_qualifier_liens($objets_source, $objets_lies, $qualif);
+	if ($qualif) {
+			objet_qualifier_liens($objets_source, $objets_lies, $qualif);
+	}
 
 	return $modifs; // pas d'erreur
 }
@@ -218,8 +222,7 @@ 
 					$n++;
 					if ($infos['type']==$objet){
 						objet_associer(array($objet=>$id_cible),array($lien['objet']=>$lien[$lien['objet']]),$lien);
-					}
-					else {
+					} else {
 						objet_associer(array($infos['type']=>$lien[$infos['type']]),array($objet=>$id_cible),$lien);
 					}
 				}
@@ -259,15 +262,17 @@ 
  */
 function objet_traiter_liaisons($operation,$objets_source,$objets_lies, $set = null){
 	// accepter une syntaxe minimale pour supprimer tous les liens
-	if ($objets_lies=='*') $objets_lies = array('*'=>'*');
+	if ($objets_lies=='*') {
+	    $objets_lies = array('*'=>'*');
+	}
 	$modifs = 0; // compter le nombre de modifications
 	$echec = null;
 	foreach($objets_source as $objet=>$ids){
 		if ($a = objet_associable($objet)) {
 			list($primary,$l) = $a;
-			if (!is_array($ids))
-				$ids = array($ids);
-			elseif(reset($ids)=="NOT"){
+			if (!is_array($ids)) {
+							$ids = array($ids);
+			} elseif(reset($ids)=="NOT"){
 				// si on demande un array('NOT',...) => recuperer la liste d'ids correspondants
 				$where = lien_where($primary,$ids,'*','*');
 				$ids = sql_allfetsel($primary,$l,$where);
@@ -278,13 +283,13 @@ 
 				if ($res===false) {
 					spip_log("objet_traiter_liaisons [Echec] : $operation sur $objet/$primary/$l/$id",_LOG_ERREUR);
 					$echec = true;
+				} else {
+									$modifs=($modifs?(is_array($res)?array_merge($modifs,$res):$modifs+$res):$res);
 				}
-				else
-					$modifs=($modifs?(is_array($res)?array_merge($modifs,$res):$modifs+$res):$res);
 			}
+		} else {
+					$echec = true;
 		}
-		else
-			$echec = true;
 	}
 
 	return ($echec?false:$modifs); // pas d'erreur
@@ -313,7 +318,9 @@ 
 	$ins = 0;
 	$echec = null;
 	foreach($objets as $objet => $id_objets){
-		if (!is_array($id_objets)) $id_objets = array($id_objets);
+		if (!is_array($id_objets)) {
+		    $id_objets = array($id_objets);
+		}
 		foreach($id_objets as $id_objet) {
 			$objet = ($objet=='*')?$objet:objet_type($objet); # securite
 			// Envoyer aux plugins
@@ -356,9 +363,9 @@ 
 								'data' => $id_objet
 							)
 						);
+					} else {
+											$echec = true;
 					}
-					else
-						$echec = true;
 			}
 		}
 	}
@@ -378,8 +385,10 @@ 
 function lien_where($primary, $id_source, $objet, $id_objet){
 	if ((!is_array($id_source) AND !strlen($id_source))
 	  OR !strlen($objet)
-	  OR (!is_array($id_objet) AND !strlen($id_objet)))
-		return array("0=1"); // securite
+	  OR (!is_array($id_objet) AND !strlen($id_objet))) {
+			return array("0=1");
+	}
+	// securite
 
 	$not="";
 	if (is_array($id_source) AND reset($id_source)=="NOT"){
@@ -387,10 +396,12 @@ 
 		$id_source = reset($id_source);
 	}
 	$where = array();
-	if ($id_source!=='*')
-		$where[] = (is_array($id_source)?sql_in(addslashes($primary),array_map('intval',$id_source),$not):addslashes($primary) . ($not?"<>":"=") . intval($id_source));
-	elseif ($not)
-		$where[] = "0=1"; // idiot mais quand meme
+	if ($id_source!=='*') {
+			$where[] = (is_array($id_source)?sql_in(addslashes($primary),array_map('intval',$id_source),$not):addslashes($primary) . ($not?"<>":"=") . intval($id_source));
+	} elseif ($not) {
+			$where[] = "0=1";
+	}
+	// idiot mais quand meme
 
 	$not="";
 	if (is_array($id_objet) AND reset($id_objet)=="NOT"){
@@ -398,12 +409,15 @@ 
 		$id_objet = reset($id_objet);
 	}
 
-	if ($objet!=='*')
-		$where[] = "objet=".sql_quote($objet);
-	if ($id_objet!=='*')
-		$where[] = (is_array($id_objet)?sql_in('id_objet',array_map('intval',$id_objet),$not):"id_objet" . ($not?"<>":"=") . intval($id_objet));
-	elseif ($not)
-		$where[] = "0=1"; // idiot mais quand meme
+	if ($objet!=='*') {
+			$where[] = "objet=".sql_quote($objet);
+	}
+	if ($id_objet!=='*') {
+			$where[] = (is_array($id_objet)?sql_in('id_objet',array_map('intval',$id_objet),$not):"id_objet" . ($not?"<>":"=") . intval($id_objet));
+	} elseif ($not) {
+			$where[] = "0=1";
+	}
+	// idiot mais quand meme
 
 	return $where;
 }
@@ -431,7 +445,9 @@ 
 	$echec = false;
 	foreach($objets as $objet => $id_objets){
 		$objet = ($objet=='*')?$objet:objet_type($objet); # securite
-		if (!is_array($id_objets) OR reset($id_objets)=="NOT") $id_objets = array($id_objets);
+		if (!is_array($id_objets) OR reset($id_objets)=="NOT") {
+		    $id_objets = array($id_objets);
+		}
 		foreach($id_objets as $id_objet) {
 			// id_objet peut valoir '*'
 			$where = lien_where($primary, $id, $objet, $id_objet);
@@ -460,9 +476,9 @@ 
 						$dels+=$e;
 						lien_propage_date_modif($l['objet'],$id_o);
 						lien_propage_date_modif($objet_source,$l[$primary]);
+					} else {
+											$echec = true;
 					}
-					else
-						$echec = true;
 					$retire[] = array('source'=>array($objet_source=>$l[$primary]),'lien'=>array($l['objet']=>$id_o),'type'=>$l['objet'],'id'=>$id_o);
 					// Envoyer aux plugins
 					pipeline('post_edition_lien',
@@ -512,7 +528,9 @@ 
 	$dels = 0;
 	foreach($objets as $objet => $id_objets){
 		$objet = ($objet=='*')?$objet:objet_type($objet); # securite
-		if (!is_array($id_objets) OR reset($id_objets)=="NOT") $id_objets = array($id_objets);
+		if (!is_array($id_objets) OR reset($id_objets)=="NOT") {
+		    $id_objets = array($id_objets);
+		}
 		foreach($id_objets as $id_objet) {
 			$where = lien_where($primary, $id, $objet, $id_objet);
 			# les liens vers un objet inexistant
@@ -579,8 +597,9 @@ 
 function lien_set($objet_source,$primary,$table_lien,$id,$objets,$qualif){
 	$echec = null;
 	$ok = 0;
-	if (!$qualif)
-		return false;
+	if (!$qualif) {
+			return false;
+	}
 	// nettoyer qualif qui peut venir directement d'un objet_trouver_lien :
 	unset($qualif[$primary]);
 	unset($qualif[$objet_source]);
@@ -591,14 +610,17 @@ 
 	unset($qualif['id_objet']);
 	foreach($objets as $objet => $id_objets){
 		$objet = ($objet=='*')?$objet:objet_type($objet); # securite
-		if (!is_array($id_objets) OR reset($id_objets)=="NOT") $id_objets = array($id_objets);
+		if (!is_array($id_objets) OR reset($id_objets)=="NOT") {
+		    $id_objets = array($id_objets);
+		}
 		foreach($id_objets as $id_objet) {
 			$where = lien_where($primary, $id, $objet, $id_objet);
 			$e = sql_updateq($table_lien,$qualif,$where);
-			if ($e===false)
-				$echec = true;
-		  else
-			  $ok++;
+			if ($e===false) {
+							$echec = true;
+			} else {
+		  			  $ok++;
+		  }
 		}
 	}
 	return ($echec?false:$ok);