spip / SPIP

ecrire/public/styliser.php

Braces +30 added lines, -18 removed lines

@@ -11,7 +11,9 @@ 
 \***************************************************************************/
 
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // Ce fichier doit imperativement definir la fonction ci-dessous:
 
@@ -36,14 +38,16 @@ 
 
 	// s'assurer que le fond est licite
 	// car il peut etre construit a partir d'une variable d'environnement
-	if (strpos($fond,"../")!==false OR strncmp($fond,'/',1)==0)
-		$fond = "404";
+	if (strpos($fond,"../")!==false OR strncmp($fond,'/',1)==0) {
+			$fond = "404";
+	}
   
 	// Choisir entre $fond-dist.html, $fond=7.html, etc?
 	$id_rubrique = 0;
 	// Chercher le fond qui va servir de squelette
-	if ($r = quete_rubrique_fond($contexte))
-		list($id_rubrique, $lang) = $r;
+	if ($r = quete_rubrique_fond($contexte)) {
+			list($id_rubrique, $lang) = $r;
+	}
 
 	// trouver un squelette du nom demande
 	// ne rien dire si on ne trouve pas, 
@@ -65,8 +69,9 @@ 
 	);
 
 	if (test_espace_prive() OR defined('_ZPIP')) {
-		if (!$styliser_par_z)
-			$styliser_par_z = charger_fonction('styliser_par_z','public');
+		if (!$styliser_par_z) {
+					$styliser_par_z = charger_fonction('styliser_par_z','public');
+		}
 		$flux = $styliser_par_z($flux);
 	}
 
@@ -87,16 +92,18 @@ 
 			$table = table_objet(substr($flux['args']['fond'],19));
 			$table_sql = table_objet_sql($table);
 			$objets = lister_tables_objets_sql();
-			if (isset($objets[$table_sql]))
-				$flux['data'] = $echafauder($table,$table,$table_sql,"prive/objets/liste/objets",$flux['args']['ext']);
+			if (isset($objets[$table_sql])) {
+							$flux['data'] = $echafauder($table,$table,$table_sql,"prive/objets/liste/objets",$flux['args']['ext']);
+			}
 		}
 		if (strncmp($flux['args']['fond'],'prive/objets/contenu/',21)==0){
 			$type = substr($flux['args']['fond'],21);
 			$table = table_objet($type);
 			$table_sql = table_objet_sql($table);
 			$objets = lister_tables_objets_sql();
-			if (isset($objets[$table_sql]))
-				$flux['data'] = $echafauder($type,$table,$table_sql,"prive/objets/contenu/objet",$flux['args']['ext']);
+			if (isset($objets[$table_sql])) {
+							$flux['data'] = $echafauder($type,$table,$table_sql,"prive/objets/contenu/objet",$flux['args']['ext']);
+			}
 		}
 	}
 	return $flux;
@@ -129,17 +136,21 @@ 
 		array_unshift($l, 'rubrique');
 		foreach($l as $objet){
 			$id = id_table_objet($objet);
-			if (!isset($liste_objets[$id]))
-				$liste_objets[$id] = objet_type($objet,false);
+			if (!isset($liste_objets[$id])) {
+							$liste_objets[$id] = objet_type($objet,false);
+			}
 		}
 	}
 	$c = array_intersect_key($contexte,$liste_objets);
-	if (!count($c)) return false;
+	if (!count($c)) {
+	    return false;
+	}
 
 	$c = array_map('intval',$c);
 	$s = serialize($c);
-	if (isset($quete[$s]))
-		return $quete[$s];
+	if (isset($quete[$s])) {
+			return $quete[$s];
+	}
 
 	if (isset($c['id_rubrique']) AND $r = $c['id_rubrique']){
 		unset($c['id_rubrique']);
@@ -150,8 +161,9 @@ 
 		if ($id
 		  AND $row = quete_parent_lang(table_objet_sql($liste_objets[$_id]),$id)) {
 			$lang = isset($row['lang']) ? $row['lang'] : '';
-			if ($_id=='id_rubrique' OR (isset($row['id_rubrique']) AND $id=$row['id_rubrique']))
-				return $quete[$s] = array ($id, $lang);
+			if ($_id=='id_rubrique' OR (isset($row['id_rubrique']) AND $id=$row['id_rubrique'])) {
+							return $quete[$s] = array ($id, $lang);
+			}
 		}
 	}
 	return $quete[$s] = false;

ecrire/public/jointures.php

Braces +91 added lines, -52 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;
+}
 
 // deduction automatique d'une chaine de jointures 
 
@@ -48,12 +50,14 @@ 
 function trouver_champs_decomposes($champ, $desc){
 	if (!is_array($desc) // on ne se risque pas en conjectures si on ne connait pas la table
 	    OR array_key_exists($champ, $desc['field'])
-	)
-		return array($champ);
+	) {
+			return array($champ);
+	}
 	if (is_array($decompose = decompose_champ_id_objet($champ))){
 		array_pop($decompose);
-		if (count(array_intersect($decompose, array_keys($desc['field'])))==count($decompose))
-			return $decompose;
+		if (count(array_intersect($decompose, array_keys($desc['field'])))==count($decompose)) {
+					return $decompose;
+		}
 	}
 	return array($champ);
 }
@@ -89,7 +93,9 @@ 
 		$res = calculer_chaine_jointures($boucle, $depart, $arrivee, array(), $milieu_exclus, $max);
 		$max++;
 	}
-	if (!$res) return "";
+	if (!$res) {
+	    return "";
+	}
 
 	list($nom, $desc) = $depart;
 
@@ -130,15 +136,18 @@ 
 	$cpt = &$num[$boucle->descr['nom']][$boucle->descr['gram']][$boucle->id_boucle];
 	foreach ($res as $cle => $r){
 		list($d, $a, $j) = $r;
-		if (!$id_table) $id_table = $d;
+		if (!$id_table) {
+		    $id_table = $d;
+		}
 		$n = ++$cpt;
 		if (is_array($j)){ // c'est un lien sur un champ du type id_objet,objet,'article'
 			list($j1, $j2, $obj, $type) = $j;
 			// trouver de quel cote est (id_objet,objet)
-			if ($j1=="id_$obj")
-				$obj = "$id_table.$obj";
-			else
-				$obj = "L$n.$obj";
+			if ($j1=="id_$obj") {
+							$obj = "$id_table.$obj";
+			} else {
+							$obj = "L$n.$obj";
+			}
 			// le where complementaire est envoye dans la jointure pour pouvoir etre elimine avec la jointure
 			// en cas d'optimisation
 			//$boucle->where[] = array("'='","'$obj'","sql_quote('$type')");
@@ -147,9 +156,9 @@ 
 					array("'$id_table'", "'$j2'", "'$j1'", "'$obj='.sql_quote('$type')")
 					:
 					array($id_table, $j2, $j1, "$obj=".sql_quote($type));
+		} else {
+					$boucle->join["L$n"] = $echap ? array("'$id_table'", "'$j'") : array($id_table, $j);
 		}
-		else
-			$boucle->join["L$n"] = $echap ? array("'$id_table'", "'$j'") : array($id_table, $j);
 		$boucle->from[$id_table = "L$n"] = $a[0];
 	}
 
@@ -171,13 +180,16 @@ 
 	    AND (count($boucle->from)==2)
 	        AND isset($a[1]['key']['PRIMARY KEY'])
 	            AND ($j==$a[1]['key']['PRIMARY KEY'])
-	)
-		$pk = true;
+	) {
+			$pk = true;
+	}
 
 	// la clause Group by est en conflit avec ORDER BY, a completer
 	$groups = liste_champs_jointures($nom, $desc, true);
-	if (!$pk) foreach ($groups as $id_prim){
+	if (!$pk) {
+	    foreach ($groups as $id_prim){
 		$id_field = $nom.'.'.$id_prim;
+	}
 		if (!in_array($id_field, $boucle->group)){
 			$boucle->group[] = $id_field;
 		}
@@ -201,9 +213,14 @@ 
  */
 function nogroupby_if($depart, $arrivee, $col){
 	$pk = $arrivee['key']['PRIMARY KEY'];
-	if (!$pk) return false;
+	if (!$pk) {
+	    return false;
+	}
 	$id_primary = $depart['key']['PRIMARY KEY'];
-	if (is_array($col)) $col = implode(', *', $col); // cas id_objet, objet
+	if (is_array($col)) {
+	    $col = implode(', *', $col);
+	}
+	// cas id_objet, objet
 	return (preg_match("/^$id_primary, *$col$/", $pk) OR
 	        preg_match("/^$col, *$id_primary$/", $pk));
 }
@@ -227,20 +244,26 @@ 
 	static $nojoin = array('idx', 'maj', 'date', 'statut');
 
 	// si cle primaire demandee, la privilegier
-	if ($primary && isset($desc['key']['PRIMARY KEY']))
-		return split_key($desc['key']['PRIMARY KEY']);
+	if ($primary && isset($desc['key']['PRIMARY KEY'])) {
+			return split_key($desc['key']['PRIMARY KEY']);
+	}
 
 	// les champs declares explicitement pour les jointures
-	if (isset($desc['join'])) return $desc['join'];
+	if (isset($desc['join'])) {
+	    return $desc['join'];
+	}
 	/*elseif (isset($GLOBALS['tables_principales'][$nom]['join'])) return $GLOBALS['tables_principales'][$nom]['join'];
 	elseif (isset($GLOBALS['tables_auxiliaires'][$nom]['join'])) return $GLOBALS['tables_auxiliaires'][$nom]['join'];*/
 
 	// si pas de cle, c'est fichu
-	if (!isset($desc['key'])) return array();
+	if (!isset($desc['key'])) {
+	    return array();
+	}
 
 	// si cle primaire
-	if (isset($desc['key']['PRIMARY KEY']))
-		return split_key($desc['key']['PRIMARY KEY']);
+	if (isset($desc['key']['PRIMARY KEY'])) {
+			return split_key($desc['key']['PRIMARY KEY']);
+	}
 
 	// ici on se rabat sur les cles secondaires, 
 	// en eliminant celles qui sont pas pertinentes (idx, maj)
@@ -248,8 +271,12 @@ 
 	// il faut declarer explicitement le champ 'join' de sa description
 
 	$join = array();
-	foreach ($desc['key'] as $v) $join = split_key($v, $join);
-	foreach ($join as $k) if (in_array($k, $nojoin)) unset($join[$k]);
+	foreach ($desc['key'] as $v) {
+	    $join = split_key($v, $join);
+	}
+	foreach ($join as $k) {
+	    if (in_array($k, $nojoin)) unset($join[$k]);
+	}
 	return $join;
 }
 
@@ -263,7 +290,9 @@ 
  * @return array
  */
 function split_key($v, $join = array()){
-	foreach (preg_split('/,\s*/', $v) as $k) $join[$k] = $k;
+	foreach (preg_split('/,\s*/', $v) as $k) {
+	    $join[$k] = $k;
+	}
 	return $join;
 }
 
@@ -287,15 +316,18 @@ 
  */
 function calculer_chaine_jointures(&$boucle, $depart, $arrivee, $vu = array(), $milieu_exclus = array(), $max_liens = 5){
 	static $trouver_table;
-	if (!$trouver_table)
-		$trouver_table = charger_fonction('trouver_table', 'base');
+	if (!$trouver_table) {
+			$trouver_table = charger_fonction('trouver_table', 'base');
+	}
 
-	if (is_string($milieu_exclus))
-		$milieu_exclus = array($milieu_exclus);
+	if (is_string($milieu_exclus)) {
+			$milieu_exclus = array($milieu_exclus);
+	}
 	// quand on a exclus id_objet comme cle de jointure, il faut aussi exclure objet
 	// faire une jointure sur objet tout seul n'a pas de sens
-	if (in_array('id_objet',$milieu_exclus) AND !in_array('objet',$milieu_exclus))
-		$milieu_exclus[] = 'objet';
+	if (in_array('id_objet',$milieu_exclus) AND !in_array('objet',$milieu_exclus)) {
+			$milieu_exclus[] = 'objet';
+	}
 
 	list($dnom, $ddesc) = $depart;
 	list($anom, $adesc) = $arrivee;
@@ -320,8 +352,9 @@ 
 
 	$v = !$keys ? false : array_intersect(array_values($keys), $akeys);
 
-	if ($v)
-		return array(array($dnom, array($adesc['table'], $adesc), array_shift($v)));
+	if ($v) {
+			return array(array($dnom, array($adesc['table'], $adesc), array_shift($v)));
+	}
 
 	// regarder si l'on a (id_objet,objet) au depart et si on peut le mapper sur un id_xx
 	if (count(array_intersect(array('id_objet', 'objet'), $keys))==2){
@@ -337,8 +370,7 @@ 
 				return array(array($dnom, array($adesc['table'], $adesc), $v));
 			}
 		}
-	}
-	else {
+	} else {
 		// regarder si l'une des cles de depart peut se decomposer en 
 		// id_objet,objet a l'arrivee
 		// si oui on la prend
@@ -353,7 +385,9 @@ 
 		}
 	}
 	// si l'on voulait une jointure direct, c'est rate !
-	if ($max_liens<=1) return array();
+	if ($max_liens<=1) {
+	    return array();
+	}
 
 	// sinon essayer de passer par une autre table
 	$new = $vu;
@@ -376,8 +410,7 @@ 
 				if (is_string($milieu)){
 					$exclure_fin[] = $milieu;
 					$test_cles[] = $milieu;
-				}
-				else{
+				} else{
 					$exclure_fin = array_merge($exclure_fin, $milieu);
 					$test_cles = array_merge($test_cles, $milieu);
 				}
@@ -406,9 +439,9 @@ 
 function trouver_cles_table($keys){
 	$res = array();
 	foreach ($keys as $v){
-		if (!strpos($v, ","))
-			$res[$v] = 1;
-		else {
+		if (!strpos($v, ",")) {
+					$res[$v] = 1;
+		} else {
 			foreach (preg_split("/\s*,\s*/", $v) as $k){
 				$res[$k] = 1;
 			}
@@ -428,15 +461,17 @@ 
  */
 function trouver_champ_exterieur($cle, $joints, &$boucle, $checkarrivee = false){
 	static $trouver_table = '';
-	if (!$trouver_table)
-		$trouver_table = charger_fonction('trouver_table', 'base');
+	if (!$trouver_table) {
+			$trouver_table = charger_fonction('trouver_table', 'base');
+	}
 
 	// support de la recherche multi champ :
 	// si en seconde etape on a decompose le champ id_xx en id_objet,objet
 	// on reentre ici soit en cherchant une table les 2 champs id_objet,objet
 	// soit une table avec les 3 champs id_xx, id_objet, objet
-	if (!is_array($cle))
-		$cle = array($cle);
+	if (!is_array($cle)) {
+			$cle = array($cle);
+	}
 
 	foreach ($joints as $k => $join){
 		if ($join && $table = $trouver_table($join, $boucle->sql_serveur)){
@@ -445,8 +480,9 @@ 
 			    AND (count(array_intersect($cle, array_keys($table['field'])))==count($cle))
 			        // si on sait ou on veut arriver, il faut que ca colle
 			        AND ($checkarrivee==false || $checkarrivee==$table['table'])
-			)
-				return array($table['table'], $table);
+			) {
+							return array($table['table'], $table);
+			}
 		}
 	}
 
@@ -475,8 +511,9 @@ 
 				$cle[] = array_shift($decompose); // id_objet
 				$cle[] = array_shift($decompose); // objet
 				$cle[] = $d;
-				if ($ext = trouver_champ_exterieur($cle, $joints, $boucle, $checkarrivee))
-					return $ext;
+				if ($ext = trouver_champ_exterieur($cle, $joints, $boucle, $checkarrivee)) {
+									return $ext;
+				}
 			}
 		}
 	}
@@ -514,7 +551,9 @@ 
 		$desc = $boucle->show;
 		$cle = calculer_jointure($boucle, array($desc['id_table'], $desc), $cle, '', $cond);
 	}
-	if ($cle) return $cle;
+	if ($cle) {
+	    return $cle;
+	}
 	spip_log("trouver_jointure_champ: $champ inconnu");
 	return '';
 }

ecrire/public/iterateur.php

Braces +48 added lines, -28 removed lines

@@ -11,7 +11,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;
+}
 
 /**
  * Fabrique d'iterateur
@@ -128,10 +130,12 @@ 
 	 * (select SQL errone, non chargement des DATA, etc)
 	 */
 	public function err() {
-		if (method_exists($this->iter, 'err'))
-			return $this->iter->err();
-		if (property_exists($this->iter, 'err'))
-			return $this->iter->err;
+		if (method_exists($this->iter, 'err')) {
+					return $this->iter->err();
+		}
+		if (property_exists($this->iter, 'err')) {
+					return $this->iter->err;
+		}
 		return false;
 	}
 
@@ -222,10 +226,16 @@ 
 			$menage = false;
 			foreach($where as $k => $v) { 
 				if (is_array($v)){
-					if ((count($v)>=2) && ($v[0]=='REGEXP') && ($v[2]=="'.*'")) $op= false;
-					elseif ((count($v)>=2) && ($v[0]=='LIKE') && ($v[2]=="'%'")) $op= false;
-					else $op = $v[0] ? $v[0] : $v;
-				} else $op = $v;
+					if ((count($v)>=2) && ($v[0]=='REGEXP') && ($v[2]=="'.*'")) {
+					    $op= false;
+					} elseif ((count($v)>=2) && ($v[0]=='LIKE') && ($v[2]=="'%'")) {
+					    $op= false;
+					} else {
+					    $op = $v[0] ? $v[0] : $v;
+					}
+				} else {
+				    $op = $v;
+				}
 				if ((!$op) OR ($op==1) OR ($op=='0=0')) {
 					unset($where[$k]);
 					$menage = true;
@@ -315,11 +325,13 @@ 
 			$op = '';
 		}
 	
-		if ($op)
-			$filtre = $a.$op.str_replace('\"', '"', $valeur);
+		if ($op) {
+					$filtre = $a.$op.str_replace('\"', '"', $valeur);
+		}
 
-		if ($not)
-			$filtre = "!($filtre)";
+		if ($not) {
+					$filtre = "!($filtre)";
+		}
 			
 		if ($filtre) {
 			$this->filtre[] = $filtre;
@@ -413,8 +425,9 @@ 
 	 * un par un tous les elements
 	 */
 	private function seek_loop($n) {
-		if ($this->pos > $n)
-			$this->rewind();
+		if ($this->pos > $n) {
+					$this->rewind();
+		}
 
 		while ($this->pos < $n AND $this->valid()) {
 			$this->next();
@@ -431,12 +444,15 @@ 
 	 */
 	public function skip($saut, $max=null){
 		// pas de saut en arriere autorise pour cette fonction
-		if (($saut=intval($saut))<=0) return $this->pos;
+		if (($saut=intval($saut))<=0) {
+		    return $this->pos;
+		}
 		$seek = $this->pos + $saut;
 		// si le saut fait depasser le maxi, on libere la resource
 		// et on sort
-		if (is_null($max))
-			$max = $this->count();
+		if (is_null($max)) {
+					$max = $this->count();
+		}
 
 		if ($seek>=$max OR $seek>=$this->count()) {
 			// sortie plus rapide que de faire next() jusqu'a la fin !
@@ -465,15 +481,18 @@ 
 			AND (
 				!$this->accept()
 				OR (isset($this->offset) AND $this->fetched++ < $this->offset)
-			))
-				$this->next();
+			)) {
+							$this->next();
+			}
 
-			if (!$this->valid())
-				return false;
+			if (!$this->valid()) {
+							return false;
+			}
 
 			if (isset($this->limit)
-			AND $this->fetched > $this->offset + $this->limit)
-				return false;
+			AND $this->fetched > $this->offset + $this->limit) {
+							return false;
+			}
 
 			$r = array();
 			foreach ($this->select as $nom) {
@@ -495,10 +514,11 @@ 
 		# clone pas proprement (directoryiterator sous php 5.2.2)
 		# on se rabat sur la version __toString()
 		if (is_object($v = $this->current())) {
-			if (method_exists($v, '__toString'))
-				$v = $v->__toString();
-			else
-				$v = (array) $v;
+			if (method_exists($v, '__toString')) {
+							$v = $v->__toString();
+			} else {
+							$v = (array) $v;
+			}
 		}
 		return $v;
 	}

ecrire/action/editer_objet.php

Braces +79 added lines, -57 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;
+}
 
 /**
  * Point d'entree d'edition d'un objet
@@ -39,8 +41,9 @@ 
 		$id = objet_inserer($objet, $id_parent);
 	}
 
-	if (!($id = intval($id))>0)
-		return array($id,_L('echec enregistrement en base'));
+	if (!($id = intval($id))>0) {
+			return array($id,_L('echec enregistrement en base'));
+	}
 
 	// Enregistre l'envoi dans la BD
 	$err = objet_modifier($objet, $id, $set);
@@ -59,8 +62,9 @@ 
  */
 function objet_modifier($objet, $id, $set=null) {
 	if (include_spip('action/editer_'.$objet)
-	  AND function_exists($modifier = $objet."_modifier"))
-		return $modifier($id,$set);
+	  AND function_exists($modifier = $objet."_modifier")) {
+			return $modifier($id,$set);
+	}
 
 	$table_sql = table_objet_sql($objet);
 	$trouver_table = charger_fonction('trouver_table','base');
@@ -72,10 +76,11 @@ 
 	include_spip('inc/modifier');
 
 	$champ_date = '';
-	if (isset($desc['date']) AND $desc['date'])
-		$champ_date = $desc['date'];
-	elseif (isset($desc['field']['date']))
-		$champ_date = 'date';
+	if (isset($desc['date']) AND $desc['date']) {
+			$champ_date = $desc['date'];
+	} elseif (isset($desc['field']['date'])) {
+			$champ_date = 'date';
+	}
 
 	$white = array_keys($desc['field']);
 	// on ne traite pas la cle primaire par defaut, notamment car
@@ -98,8 +103,7 @@ 
 	if (objet_test_si_publie($objet,$id)){
 		$invalideur = "id='$objet/$id'";
 		$indexation = true;
-	}
-	else {
+	} else {
 		$invalideur = "";
 		$indexation = false;
 	}
@@ -113,8 +117,9 @@ 
 			 // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
 			'date_modif' => (isset($desc['field']['date_modif'])?'date_modif':'')
 		),
-		$c))
-		return $err;
+		$c)) {
+			return $err;
+	}
 
 	// Modification de statut, changement de rubrique ?
 	// FIXME: Ici lorsqu'un $set est passé, la fonction collecter_requests() retourne tout
@@ -134,14 +139,16 @@ 
  */
 function objet_inserer($objet, $id_parent=null, $set=null) {
 	if (include_spip('action/editer_'.$objet)
-	  AND function_exists($inserer = $objet."_inserer"))
-		return $inserer($id_parent);
+	  AND function_exists($inserer = $objet."_inserer")) {
+			return $inserer($id_parent);
+	}
 
 	$table_sql = table_objet_sql($objet);
 	$trouver_table = charger_fonction('trouver_table','base');
 	$desc = $trouver_table($table_sql);
-	if (!$desc OR !isset($desc['field']))
-		return 0;
+	if (!$desc OR !isset($desc['field'])) {
+			return 0;
+	}
 
 	$lang_rub = "";
 	$champs = array();
@@ -151,13 +158,14 @@ 
 		if (!$id_rubrique = intval($id_parent)) {
 			$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=".intval($id_rubrique));
 		}
-		else
-			$row = sql_fetsel("lang, id_secteur", "spip_rubriques", "id_rubrique=".intval($id_rubrique));
 
 		$champs['id_rubrique'] = $id_rubrique;
-		if (isset($desc['field']['id_secteur']))
-			$champs['id_secteur'] = $row['id_secteur'];
+		if (isset($desc['field']['id_secteur'])) {
+					$champs['id_secteur'] = $row['id_secteur'];
+		}
 		$lang_rub = $row['lang'];
 	}
 
@@ -170,11 +178,11 @@ 
 		if (in_array($GLOBALS['spip_lang'],
 		explode(',', $GLOBALS['meta']['langues_multilingue']))) {
 			$champs['lang'] = $GLOBALS['spip_lang'];
-			if (isset($desc['field']['langue_choisie']))
-				$champs['langue_choisie'] = 'oui';
+			if (isset($desc['field']['langue_choisie'])) {
+							$champs['langue_choisie'] = 'oui';
+			}
 		}
-	}
-	elseif (isset($desc['field']['lang']) AND isset($desc['field']['langue_choisie'])) {
+	} elseif (isset($desc['field']['lang']) AND isset($desc['field']['langue_choisie'])) {
 		$champs['lang'] = ($lang_rub ? $lang_rub : $GLOBALS['meta']['langue_site']);
 		$champs['langue_choisie'] = 'non';
 	}
@@ -183,17 +191,19 @@ 
 		if (isset($desc['statut_textes_instituer'])){
 			$cles_statut = array_keys($desc['statut_textes_instituer']); 
 			$champs['statut'] = reset($cles_statut);
+		} else {
+					$champs['statut'] = 'prepa';
 		}
-		else
-			$champs['statut'] = 'prepa';
 	}
 
 
-	if ((isset($desc['date']) AND $d=$desc['date']) OR isset($desc['field'][$d='date']))
-		$champs[$d] = date('Y-m-d H:i:s');
+	if ((isset($desc['date']) AND $d=$desc['date']) OR isset($desc['field'][$d='date'])) {
+			$champs[$d] = date('Y-m-d H:i:s');
+	}
 
-	if ($set)
-		$champs = array_merge($champs, $set);
+	if ($set) {
+			$champs = array_merge($champs, $set);
+	}
 
 	// Envoyer aux plugins
 	$champs = pipeline('pre_insertion',
@@ -251,14 +261,16 @@ 
  */
 function objet_instituer($objet, $id, $c, $calcul_rub=true) {
 	if (include_spip('action/editer_'.$objet)
-	  AND function_exists($instituer = $objet."_instituer"))
-		return $instituer($id,$c,$calcul_rub);
+	  AND function_exists($instituer = $objet."_instituer")) {
+			return $instituer($id,$c,$calcul_rub);
+	}
 
 	$table_sql = table_objet_sql($objet);
 	$trouver_table = charger_fonction('trouver_table','base');
 	$desc = $trouver_table($table_sql);
-	if (!$desc OR !isset($desc['field']))
-		return _L("Impossible d'instituer $objet : non connu en base");
+	if (!$desc OR !isset($desc['field'])) {
+			return _L("Impossible d'instituer $objet : non connu en base");
+	}
 
 	include_spip('inc/autoriser');
 	include_spip('inc/rubriques');
@@ -268,10 +280,11 @@ 
 	$sel[] = (isset($desc['field']['statut'])?"statut":"'' as statut");
 
 	$champ_date = '';
-	if (isset($desc['date']) AND $desc['date'])
-		$champ_date = $desc['date'];
-	elseif (isset($desc['field']['date']))
-		$champ_date = 'date';
+	if (isset($desc['date']) AND $desc['date']) {
+			$champ_date = $desc['date'];
+	} elseif (isset($desc['field']['date'])) {
+			$champ_date = 'date';
+	}
 
 	$sel[] = ($champ_date ? "$champ_date as date" : "'' as date");
 	$sel[] = (isset($desc['field']['id_rubrique'])?'id_rubrique':"0 as id_rubrique");
@@ -292,12 +305,13 @@ 
 				autoriser('publierdans', 'rubrique', $id_rubrique)
 			:
 				autoriser('instituer', $objet, $id, null, array('statut'=>$s))
-			)
-			$statut = $champs['statut'] = $s;
-		else if ($s!='publie' AND autoriser('modifier', $objet, $id))
-			$statut = $champs['statut'] = $s;
-		else
-			spip_log("editer_objet $id refus " . join(' ', $c));
+			) {
+					$statut = $champs['statut'] = $s;
+		} else if ($s!='publie' AND autoriser('modifier', $objet, $id)) {
+					$statut = $champs['statut'] = $s;
+		} else {
+					spip_log("editer_objet $id refus " . join(' ', $c));
+		}
 
 		// En cas de publication, fixer la date a "maintenant"
 		// sauf si $c commande autre chose
@@ -308,10 +322,11 @@ 
 			 OR ($champs['statut'] == 'prop' AND !in_array($statut_ancien, array('publie', 'prop')))
 			 OR $d
 			) {
-				if ($d OR strtotime($d=$date)>time())
-					$champs[$champ_date] = $date = $d;
-				else
-					$champs[$champ_date] = $date = date('Y-m-d H:i:s');
+				if ($d OR strtotime($d=$date)>time()) {
+									$champs[$champ_date] = $date = $d;
+				} else {
+									$champs[$champ_date] = $date = date('Y-m-d H:i:s');
+				}
 			}
 		}
 	}
@@ -328,8 +343,9 @@ 
 		// et que le demandeur n'est pas admin de la rubrique
 		// repasser l'objet en statut 'propose'.
 		if ($statut == 'publie'
-		AND !autoriser('publierdans', 'rubrique', $id_rubrique))
-			$champs['statut'] = 'prop';
+		AND !autoriser('publierdans', 'rubrique', $id_rubrique)) {
+					$champs['statut'] = 'prop';
+		}
 	}
 
 
@@ -348,7 +364,9 @@ 
 		)
 	);
 
-	if (!count($champs)) return '';
+	if (!count($champs)) {
+	    return '';
+	}
 
 	// Envoyer les modifs.
 	objet_editer_heritage($objet, $id, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
@@ -414,16 +432,20 @@ 
 		$row_rub = sql_fetsel("id_secteur, lang", "spip_rubriques", "id_rubrique=".sql_quote($champs['id_rubrique']));
 		$langue = $row_rub['lang'];
 
-		if (isset($desc['field']['id_secteur']))
-			$champs['id_secteur'] = $row_rub['id_secteur'];
+		if (isset($desc['field']['id_secteur'])) {
+					$champs['id_secteur'] = $row_rub['id_secteur'];
+		}
 
-		if (isset($desc['field']['lang']) AND isset($desc['field']['langue_choisie']))
-			if (sql_fetsel('1', $table_sql, id_table_objet($objet)."=".intval($id)." AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue))) {
+		if (isset($desc['field']['lang']) AND isset($desc['field']['langue_choisie'])) {
+					if (sql_fetsel('1', $table_sql, id_table_objet($objet)."=".intval($id)." AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue))) {
 				$champs['lang'] = $langue;
+		}
 			}
 	}
 
-	if (!$champs) return;
+	if (!$champs) {
+	    return;
+	}
 	sql_updateq($table_sql, $champs, id_table_objet($objet).'='.intval($id));
 
 	// Changer le statut des rubriques concernees

ecrire/action/inscrire_auteur.php

Braces +66 added lines, -43 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;
+}
 
 
 /**
@@ -30,23 +32,28 @@ 
  * @return array|string
  */
 function action_inscrire_auteur_dist($statut, $mail_complet, $nom, $options = array()){
-	if (!is_array($options))
-		$options = array('id'=>$options);
+	if (!is_array($options)) {
+			$options = array('id'=>$options);
+	}
 
-	if (function_exists('test_inscription'))
-		$f = 'test_inscription';
-	else 	$f = 'test_inscription_dist';
+	if (function_exists('test_inscription')) {
+			$f = 'test_inscription';
+	} else {
+	    $f = 'test_inscription_dist';
+	}
 	$desc = $f($statut, $mail_complet, $nom, $options);
 
 	// erreur ?
-	if (!is_array($desc))
-		return _T($desc);
+	if (!is_array($desc)) {
+			return _T($desc);
+	}
 
 	include_spip('base/abstract_sql');
 	$res = sql_select("statut, id_auteur, login, email", "spip_auteurs", "email=" . sql_quote($desc['email']));
 	// erreur ?
-	if (!$res)
-		return _T('titre_probleme_technique');
+	if (!$res) {
+			return _T('titre_probleme_technique');
+	}
 
 	$row = sql_fetch($res);
 	sql_free($res);
@@ -54,17 +61,18 @@ 
 		if (isset($options['force_nouveau']) AND $options['force_nouveau']==true){
 			$desc['id_auteur'] = $row['id_auteur'];
 			$desc = inscription_nouveau($desc);
+		} else {
+					$desc = $row;
 		}
-		else
-			$desc = $row;
-	}
-	else
-		// s'il n'existe pas deja, creer les identifiants
+	} else {
+			// s'il n'existe pas deja, creer les identifiants
 		$desc = inscription_nouveau($desc);
+	}
 
 	// erreur ?
-	if (!is_array($desc))
-		return $desc;
+	if (!is_array($desc)) {
+			return $desc;
+	}
 
 
 	// generer le mot de passe (ou le refaire si compte inutilise)
@@ -108,16 +116,20 @@ 
  */
 function test_inscription_dist($statut, $mail, $nom, $options) {
 	include_spip('inc/filtres');
-	if (!$r = email_valide($mail)) return 'info_email_invalide';
+	if (!$r = email_valide($mail)) {
+	    return 'info_email_invalide';
+	}
 	$nom = trim(corriger_caracteres($nom));
 	$res = array('email' => $r, 'nom' => $nom, 'prefs' => $statut);
 	if (isset($options['login'])) {
 		$login = trim(corriger_caracteres($options['login']));
-		if((strlen ($login) >= _LOGIN_TROP_COURT) AND (strlen($nom) <= 64))
-			$res['login'] = $login;
+		if((strlen ($login) >= _LOGIN_TROP_COURT) AND (strlen($nom) <= 64)) {
+					$res['login'] = $login;
+		}
+	}
+	if(!isset($res['login']) AND ((strlen ($nom) < _LOGIN_TROP_COURT) OR (strlen($nom) > 64))) {
+			return 'ecrire:info_login_trop_court';
 	}
-	if(!isset($res['login']) AND ((strlen ($nom) < _LOGIN_TROP_COURT) OR (strlen($nom) > 64)))
-		return 'ecrire:info_login_trop_court';
 	return $res;
 }
 
@@ -134,17 +146,21 @@ 
  */
 function inscription_nouveau($desc)
 {
-	if (!isset($desc['login']) OR !strlen($desc['login']))
-		$desc['login'] = test_login($desc['nom'], $desc['email']);
+	if (!isset($desc['login']) OR !strlen($desc['login'])) {
+			$desc['login'] = test_login($desc['nom'], $desc['email']);
+	}
 
 	$desc['statut'] = 'nouveau';
 	include_spip('action/editer_auteur');
-	if (isset($desc['id_auteur']))
-		$id_auteur = $desc['id_auteur'];
-	else
-		$id_auteur = auteur_inserer();
+	if (isset($desc['id_auteur'])) {
+			$id_auteur = $desc['id_auteur'];
+	} else {
+			$id_auteur = auteur_inserer();
+	}
 
-	if (!$id_auteur) return _T('titre_probleme_technique');
+	if (!$id_auteur) {
+	    return _T('titre_probleme_technique');
+	}
 
 	include_spip('inc/autoriser');
 	// lever l'autorisation pour pouvoir modifier le statut
@@ -175,14 +191,16 @@ 
 		$mail = strtolower(translitteration(preg_replace('/@.*/', '', $mail)));
 		$login_base = preg_replace("/[^\w\d]/", "_", $mail);
 	}
-	if (strlen($login_base) < 3)
-		$login_base = 'user';
+	if (strlen($login_base) < 3) {
+			$login_base = 'user';
+	}
 
 	$login = $login_base;
 
 	for ($i = 1; ; $i++) {
-		if (!sql_countsel('spip_auteurs', "login='$login'"))
-			return $login;
+		if (!sql_countsel('spip_auteurs', "login='$login'")) {
+					return $login;
+		}
 		$login = $login_base.$i;
 	}
 	return $login;
@@ -249,12 +267,13 @@ 
  */
 function tester_statut_inscription($statut_tmp, $id){
 	include_spip('inc/autoriser');
-	if ($statut_tmp)
-		return autoriser('inscrireauteur', $statut_tmp, $id) ? $statut_tmp : '';
-	elseif (
+	if ($statut_tmp) {
+			return autoriser('inscrireauteur', $statut_tmp, $id) ? $statut_tmp : '';
+	} elseif (
 		   autoriser('inscrireauteur', $statut_tmp = "1comite", $id)
-	  OR autoriser('inscrireauteur', $statut_tmp = "6forum", $id))
-		return $statut_tmp;
+	  OR autoriser('inscrireauteur', $statut_tmp = "6forum", $id)) {
+			return $statut_tmp;
+	}
 
 	return '';
 }
@@ -273,11 +292,14 @@ 
  */
 function confirmer_statut_inscription($auteur){
 	// securite
-	if ($auteur['statut'] != 'nouveau') return $auteur;
+	if ($auteur['statut'] != 'nouveau') {
+	    return $auteur;
+	}
 
 	include_spip('inc/autoriser');
-	if (!autoriser('inscrireauteur', $auteur['prefs']))
-		return $auteur;
+	if (!autoriser('inscrireauteur', $auteur['prefs'])) {
+			return $auteur;
+	}
 	$s = $auteur['prefs'];
 
 	include_spip('inc/autoriser');
@@ -320,8 +342,9 @@ 
  */
 function auteur_verifier_jeton($jeton){
 	// refuser un jeton corrompu
-	if (preg_match(',[^0-9a-f.],i',$jeton))
-		return false;
+	if (preg_match(',[^0-9a-f.],i',$jeton)) {
+			return false;
+	}
 
 	$desc = sql_fetsel('*','spip_auteurs',"cookie_oubli=".sql_quote($jeton, $serveur, 'string'));
 	return $desc;

ecrire/action/redirect.php

Braces +18 added lines, -10 removed lines

@@ -10,28 +10,36 @@
  *  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;
+}
 
 // Un script utile pour recalculer une URL symbolique des son changement
 
 function action_redirect_dist()
 {
 	$type = _request('type');
-	if (!preg_match('/^\w+$/', $type)) return;
+	if (!preg_match('/^\w+$/', $type)) {
+	    return;
+	}
 	if ($m = _request('var_mode')) {
 		// forcer la mise a jour de l'url de cet objet !
-		if (!defined('_VAR_URLS')) define('_VAR_URLS',true);
+		if (!defined('_VAR_URLS')) {
+		    define('_VAR_URLS',true);
+		}
 		$m = 'var_mode='.urlencode($m);
 	}
 	$h = generer_url_entite_absolue(intval(_request('id')), $type, $m, '', true);
 	$status = '302';
-	if (_request('status') AND _request('status')=='301')
-		$status = '301';
+	if (_request('status') AND _request('status')=='301') {
+			$status = '301';
+	}
 
-	if ($h)
-		redirige_par_entete(str_replace('&', '&', $h),'',$status);
-	else
-		redirige_par_entete('/','',$status);
-}
+	if ($h) {
+			redirige_par_entete(str_replace('&', '&', $h),'',$status);
+	} else {
+			redirige_par_entete('/','',$status);
+	}
+	}
 
 ?>

ecrire/action/debloquer_edition.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;
+}
 
 // http://doc.spip.org/@action_instituer_collaboration_dist
 function action_debloquer_edition_dist() {
@@ -22,8 +24,7 @@ 
 		include_spip('inc/drapeau_edition');
 		if ($arg == 'tous') {
 			debloquer_tous($GLOBALS['visiteur_session']['id_auteur']);
-		}
-		else {
+		} else {
 			$arg = explode("-",$arg);
 			list($objet,$id_objet) = $arg;
 			debloquer_edition($GLOBALS['visiteur_session']['id_auteur'], $id_objet, $objet);

ecrire/action/supprimer_lien.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;
+}
 
 function action_supprimer_lien_dist($arg=null){
 	if (is_null($arg)){

ecrire/action/tester_taille.php

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){