spip / SPIP

ecrire/urls/page.php

Braces +22 added lines, -13 removed lines

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

ecrire/iterateur/sql.php

Braces +18 added lines, -10 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;
+}
 
 
 /**
@@ -88,10 +90,12 @@ 
 	 * @return bool
 	 */
 	public function valid() {
-		if ($this->err)
-			return false;
-		if (!$this->firstseek)
-			$this->next();
+		if ($this->err) {
+					return false;
+		}
+		if (!$this->firstseek) {
+					$this->next();
+		}
 		return is_array($this->row);
 	}
 
@@ -150,8 +154,9 @@ 
 		if ($this->valid()) {
 			$r = $this->current();
 			$this->next();
-		} else
-			$r = false;
+		} else {
+					$r = false;
+		}
 		return $r;
 	}
 
@@ -160,7 +165,9 @@ 
 	 * @return bool
 	 */
 	public function free(){
-		if (!$this->sqlresult) return true;
+		if (!$this->sqlresult) {
+		    return true;
+		}
 		$a = sql_free($this->sqlresult, $this->command['connect']);
 	  $this->sqlresult = null;
 	  return $a;
@@ -180,8 +187,9 @@ 
 					$this->valid();
 					$s = $this->current();
 					$this->total = $s['count(*)'];
-				} else
-					$this->total = sql_count($this->sqlresult, $this->command['connect']);
+				} else {
+									$this->total = sql_count($this->sqlresult, $this->command['connect']);
+				}
 			}
 		}
 		return $this->total;

ecrire/iterateur/php.php

Braces +3 added lines, -1 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;
+}
 
 //
 // creer une boucle sur un iterateur

ecrire/iterateur/pour.php

Braces +3 added lines, -1 removed lines

@@ -11,7 +11,9 @@
 \***************************************************************************/
 
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('iterateur/data');
 

ecrire/genie/maintenance.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;
+}
 
 // Diverses taches de maintenance
 // http://doc.spip.org/@genie_maintenance_dist
@@ -23,8 +25,9 @@ 
 	verifier_htaccess(_DIR_TMP);
 
 	// Verifier qu'aucune table n'est crashee
-	if (!_request('reinstall'))
-		verifier_crash_tables();
+	if (!_request('reinstall')) {
+			verifier_crash_tables();
+	}
 
 	return 1;
 }
@@ -39,8 +42,10 @@ 
 		foreach (array('tables_principales', 'tables_auxiliaires') as $com) {
 			foreach ($GLOBALS[$com] as $table => $desc) {
 				if (!sql_select('*', $table,'','','', 1)
-				AND !defined('spip_interdire_cache')) # cas "LOST CONNECTION"
+				AND !defined('spip_interdire_cache')) {
+				    # cas "LOST CONNECTION"
 					$crash[] = $table;
+				}
 			}
 		}
 		#$crash[] = 'test';

ecrire/genie/mise_a_jour.php

Braces +15 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;
+}
 
 /**
  * Verifier si une mise a jour est disponible
@@ -53,11 +55,14 @@ 
 		list(, $maj2, $min2,, $rev2) = $v;
 		$version_maj = $maj2 . '.' . $min2 . '.' . $rev2;
 		if ((spip_version_compare($version, $version_maj, '<'))
-		AND (spip_version_compare($page, $version_maj, '<')))
-			$page = $version_maj;
+		AND (spip_version_compare($page, $version_maj, '<'))) {
+					$page = $version_maj;
+		}
 	}
 
-	if (!$page) return "";
+	if (!$page) {
+	    return "";
+	}
 	return "<a class='info_maj_spip' href='"._VERSIONS_SERVEUR."$dir' title='$page'>" .
 		_T('nouvelle_version_spip',array('version'=>$page)) .
 	    '</a>';
@@ -71,14 +76,18 @@ 
 function info_maj_cache($nom, $dir, $page='')
 {
 	$re = '<archives id="a' . $GLOBALS['meta']["alea_ephemere"] . '">';
-	if (preg_match("/$re/", $page)) return $page;
+	if (preg_match("/$re/", $page)) {
+	    return $page;
+	}
 
 	$url = _VERSIONS_SERVEUR . $dir . '/' . _VERSIONS_LISTE;
 	$a = file_exists($nom) ? filemtime($nom) : '';
 	include_spip('inc/distant');
 	$res = recuperer_lapage($url, false, 'GET', _COPIE_LOCALE_MAX_SIZE, '',false, $a);
 	// Si rien de neuf (ou inaccessible), garder l'ancienne
-	if ($res) list(, $page) = $res;
+	if ($res) {
+	    list(, $page) = $res;
+	}
 	// Placer l'indicateur de fraicheur
 	$page = preg_replace('/^<archives.*?>/', $re, $page);
 	sous_repertoire(_DIR_CACHE_XML);

ecrire/genie/mail.php

Braces +11 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;
+}
 
 /**
  * Envoi du Mail des nouveautes
@@ -29,8 +31,9 @@ 
 	$jours_neuf = $GLOBALS['meta']['jours_neuf'];
 
 	$now = time();
-	if (!isset($GLOBALS['meta']['dernier_envoi_neuf']))
-		ecrire_meta('dernier_envoi_neuf',date('Y-m-d H:i:s',$now - (3600 * 24 * $jours_neuf)));
+	if (!isset($GLOBALS['meta']['dernier_envoi_neuf'])) {
+			ecrire_meta('dernier_envoi_neuf',date('Y-m-d H:i:s',$now - (3600 * 24 * $jours_neuf)));
+	}
 
 	$page = recuperer_fond('nouveautes',array('date'=>$GLOBALS['meta']['dernier_envoi_neuf'],'jours_neuf'=>$jours_neuf),array('raw'=>true));
 
@@ -38,16 +41,17 @@ 
 		// recuperer les entetes envoyes par #HTTP_HEADER
 		$headers = "";
 		if (isset($page['entetes']) AND count($page['entetes'])){
-			foreach ($page['entetes'] as $k => $v)
-				$headers .= (strlen($v)?"$k: $v":$k)."\n";
+			foreach ($page['entetes'] as $k => $v) {
+							$headers .= (strlen($v)?"$k: $v":$k)."\n";
+			}
 		}
 
 		include_spip("inc/notifications");
 		notifications_envoyer_mails($adresse_neuf,$page['texte'],"","",$headers);
 		ecrire_meta('dernier_envoi_neuf',date('Y-m-d H:i:s',$now));
+	} else {
+			spip_log("mail nouveautes : rien de neuf depuis $jours_neuf jours");
 	}
-	else
-		spip_log("mail nouveautes : rien de neuf depuis $jours_neuf jours");
 
 	return 1;
 }

ecrire/genie/optimiser.php

Braces +18 added lines, -9 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('base/abstract_sql');
 
@@ -43,17 +45,20 @@ 
 	// on n'optimise qu'une seule table a chaque fois,
 	// pour ne pas vautrer le systeme
 	// lire http://dev.mysql.com/doc/refman/5.0/fr/optimize-table.html
-	while ($row = sql_fetch($result))
-		$tables[] = array_shift($row);
+	while ($row = sql_fetch($result)) {
+			$tables[] = array_shift($row);
+	}
 
 	if ($tables) {
 		$table_op = intval($GLOBALS['meta']['optimiser_table']+1) % sizeof($tables);
 		ecrire_meta('optimiser_table', $table_op);
 		$q = $tables[$table_op];
 		spip_log("debut d'optimisation de la table $q");
-		if (sql_optimize($q))
-			spip_log("fin d'optimisation de la table $q");
-		else spip_log("Pas d'optimiseur necessaire");
+		if (sql_optimize($q)) {
+					spip_log("fin d'optimisation de la table $q");
+		} else {
+		    spip_log("Pas d'optimiseur necessaire");
+		}
 	}
 }
 
@@ -68,7 +73,9 @@ 
 function optimiser_sansref($table, $id, $sel, $and="")
 {
 	$in = array();
-	while ($row = sql_fetch($sel)) $in[$row['id']]=true;
+	while ($row = sql_fetch($sel)) {
+	    $in[$row['id']]=true;
+	}
 	sql_free($sel);
 
 	if ($in) {
@@ -143,6 +150,8 @@ 
 			'data'=>$n
 	));
 	
-	if (!$n) spip_log("Optimisation des tables: aucun lien mort");
-}
+	if (!$n) {
+	    spip_log("Optimisation des tables: aucun lien mort");
+	}
+	}
 ?>

ecrire/index.php

Braces +22 added lines, -12 removed lines

@@ -11,12 +11,16 @@ 
 \***************************************************************************/
 
 define('_ESPACE_PRIVE', true);
-if (!defined('_ECRIRE_INC_VERSION')) include 'inc_version.php';
+if (!defined('_ECRIRE_INC_VERSION')) {
+    include 'inc_version.php';
+}
 
 // Verification anti magic_quotes_sybase, pour qui addslashes("'") = "''"
 // On prefere la faire ici plutot que dans inc_version, c'est moins souvent et
 // si le reglage est modifie sur un site en prod, ca fait moins mal
-if (addslashes("'") !== "\\'") die('SPIP incompatible magic_quotes_sybase');
+if (addslashes("'") !== "\\'") {
+    die('SPIP incompatible magic_quotes_sybase');
+}
 
 include_spip('inc/cookie');
 
@@ -31,7 +35,9 @@ 
 // alors il faut blinder les variables d'URL
 //
 if (autoriser_sans_cookie($exec)) {
-	if (!isset($reinstall)) $reinstall = 'non';
+	if (!isset($reinstall)) {
+	    $reinstall = 'non';
+	}
 	set_request('transformer_xml');
 	$var_auth = true;
 } else {
@@ -62,8 +68,10 @@ 
 		traiter_appels_inclusions_ajax()
 	 OR 
 	 	// cas des formulaires charger/verifier/traiter
-	  traiter_formulaires_dynamiques())
-	  exit; // le hit est fini !
+	  traiter_formulaires_dynamiques()) {
+		  exit;
+	}
+	// le hit est fini !
 }
 // securiser les redirect du back-office
 if (_request('redirect')) {
@@ -82,8 +90,9 @@ 
 if (!$reinstall=='oui'
 AND !_AJAX
 AND isset($GLOBALS['meta']['version_installee'])
-AND ($GLOBALS['spip_version_base'] != (str_replace(',','.',$GLOBALS['meta']['version_installee']))))
+AND ($GLOBALS['spip_version_base'] != (str_replace(',','.',$GLOBALS['meta']['version_installee'])))) {
 	$exec = 'demande_mise_a_jour';
+}
 
 // Quand une action d'administration est en cours (meta "admin"),
 // refuser les connexions non-admin ou Ajax pour laisser la base intacte.
@@ -92,8 +101,9 @@ 
 // sinon c'est qu'elle a ete interrompue et il faut la reprendre
 
 elseif (isset($GLOBALS['meta']["admin"])) {
-	if (preg_match('/^(.*)_(\d+)_/', $GLOBALS['meta']["admin"], $l))
-		list(,$var_f,$n) = $l;
+	if (preg_match('/^(.*)_(\d+)_/', $GLOBALS['meta']["admin"], $l)) {
+			list(,$var_f,$n) = $l;
+	}
 	if (_AJAX 
 		OR !(
 			isset($_COOKIE['spip_admin'])
@@ -108,8 +118,9 @@ 
 	}
 	if ($n) {
 		list(,$var_f,$n) = $l;
-		if (tester_url_ecrire("base_$var_f"))
-			$var_f = "base_$var_f";
+		if (tester_url_ecrire("base_$var_f")) {
+					$var_f = "base_$var_f";
+		}
 		if ($var_f != $exec) {
 			spip_log("Le script $var_f lance par auteur$n se substitue a l'exec $exec");
 			$exec = $var_f;
@@ -150,8 +161,7 @@ 
 if ($var_f = tester_url_ecrire($exec)) {
 	$var_f = charger_fonction ($var_f);
 	$var_f(); // at last
-}
-else {
+} else {
 	// Rien de connu: rerouter vers exec=404 au lieu d'echouer
 	// ce qui permet de laisser la main a un plugin
 	$var_f = charger_fonction('404');