spip / SPIP

ecrire/xml/valider.php

Braces +59 added lines, -36 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;
+}
 
 // Validateur XML en deux passes, fonde sur SAX pour la premiere
 // Faudrait faire deux classes car pour la premiere passe
@@ -69,16 +71,18 @@ 
 	    }
 	}
 	// Init de la suite des balises a memoriser si regle difficile
-	if ($this->dtc->regles[$name][0]=='/')
-	    $this->fratrie[$depth]='';
+	if ($this->dtc->regles[$name][0]=='/') {
+		    $this->fratrie[$depth]='';
+	}
 	if (isset($this->dtc->attributs[$name])) {
 		  foreach ($this->dtc->attributs[$name] as $n => $v)
-		    { if (($v[1] == '#REQUIRED') AND (!isset($attrs[$n])))
-			coordonnees_erreur($this, " <b>$n</b>"
+		    { if (($v[1] == '#REQUIRED') AND (!isset($attrs[$n]))) {
+		    			coordonnees_erreur($this, " <b>$n</b>"
 			  . '&nbsp;:&nbsp;'
 			  . _T('zxml_obligatoire_attribut')
 			  . " <b>$name</b>");
 		    }
+		    }
 	}
 }
 
@@ -86,18 +90,20 @@ 
 function validerAttribut($phraseur, $name, $val, $bal)
 {
 	// Si la balise est inconnue, eviter d'insister
-	if (!isset($this->dtc->attributs[$bal]))
-		return ;
+	if (!isset($this->dtc->attributs[$bal])) {
+			return ;
+	}
 		
 	$a = $this->dtc->attributs[$bal];
 	if (!isset($a[$name])) {
 		$bons = join(', ',array_keys($a));
-		if ($bons)
-		  $bons = " title=' " .
+		if ($bons) {
+				  $bons = " title=' " .
 		    _T('zxml_connus_attributs') .
 		    '&nbsp;: ' .
 		    $bons .
 		    "'";
+		}
 		$bons .= " style='font-weight: bold'";
 		coordonnees_erreur($this, " <b>$name</b> "
 		. _T('zxml_inconnu_attribut').' '._T('zxml_de')
@@ -106,11 +112,12 @@ 
 		. ")");
 	} else{
 		$type =  $a[$name][0];
-		if (!preg_match('/^\w+$/', $type))
-			$this->valider_motif($phraseur, $name, $val, $bal, $type);
-		else if (method_exists($this, $f = 'validerAttribut_' . $type))
-			$this->$f($phraseur, $name, $val, $bal);
-#		else spip_log("$type type d'attribut inconnu");
+		if (!preg_match('/^\w+$/', $type)) {
+					$this->valider_motif($phraseur, $name, $val, $bal, $type);
+		} else if (method_exists($this, $f = 'validerAttribut_' . $type)) {
+					$this->$f($phraseur, $name, $val, $bal);
+		}
+		#		else spip_log("$type type d'attribut inconnu");
 	}
 }
 
@@ -172,9 +179,10 @@ 
 // http://doc.spip.org/@valider_idref
 function valider_idref($nom, $ligne, $col)
 {
-	if (!isset($this->ids[$nom]))
-		$this->err[]= array(" <p><b>$nom</b> " . _T('zxml_inconnu_id'), $ligne, $col);
-}
+	if (!isset($this->ids[$nom])) {
+			$this->err[]= array(" <p><b>$nom</b> " . _T('zxml_inconnu_id'), $ligne, $col);
+	}
+	}
 
 // http://doc.spip.org/@valider_passe2
 function valider_passe2()
@@ -186,8 +194,9 @@ 
 		}
 		foreach ($this->idrefss as $idref) {
 			list($noms, $ligne, $col) = $idref;
-			foreach(preg_split('/\s+/', $noms) as $nom)
-				$this->valider_idref($nom, $ligne, $col);
+			foreach(preg_split('/\s+/', $noms) as $nom) {
+							$this->valider_idref($nom, $ligne, $col);
+			}
 		}
 	}
 }
@@ -195,10 +204,13 @@ 
 // http://doc.spip.org/@debutElement
 function debutElement($phraseur, $name, $attrs)
 { 
-	if ($this->dtc->elements)
-		$this->validerElement($phraseur, $name, $attrs);
+	if ($this->dtc->elements) {
+			$this->validerElement($phraseur, $name, $attrs);
+	}
 
-	if ($f = $this->process['debut']) $f($this, $name, $attrs);
+	if ($f = $this->process['debut']) {
+	    $f($this, $name, $attrs);
+	}
 	$depth = $this->depth;
 	$this->debuts[$depth] =  strlen($this->res);
 	foreach ($attrs as $k => $v) {
@@ -220,9 +232,10 @@ 
 	$vide = ($regle  == 'EMPTY');
 	// controler que les balises devant etre vides le sont 
 	if ($vide) {
-		if ($n <> ($k + $c))
-			coordonnees_erreur($this, " <p><b>$name</b> "
+		if ($n <> ($k + $c)) {
+					coordonnees_erreur($this, " <p><b>$name</b> "
 					   . _T('zxml_nonvide_balise'));
+		}
 	// pour les regles PCDATA ou iteration de disjonction, tout est fait
 	} elseif ($regle AND ($regle != '*')) {
 		if ($regle == '+') {
@@ -244,8 +257,10 @@ 
 		}
 
 	}
-	if ($f = $this->process['fin']) $f($this, $name, $vide);
-}
+	if ($f = $this->process['fin']) {
+	    $f($this, $name, $vide);
+	}
+	}
 
 // http://doc.spip.org/@textElement
 function textElement($phraseur, $data)
@@ -260,13 +275,17 @@ 
 			);
 		}
 	}
-	if ($f = $this->process['text']) $f($this, $data);
-}
+	if ($f = $this->process['text']) {
+	    $f($this, $data);
+	}
+	}
 
 function piElement($phraseur, $target, $data)
 {
-	if ($f = $this->process['pi']) $f($this, $target, $data);
-}
+	if ($f = $this->process['pi']) {
+	    $f($this, $target, $data);
+	}
+	}
 
 // Denonciation des entitees XML inconnues
 // Pour contourner le bug de conception de SAX qui ne signale pas si elles
@@ -279,14 +298,16 @@ 
 function defaultElement($phraseur, $data)
 {
 	if (!preg_match('/^<!--/', $data)
-	AND (preg_match_all('/&([^;]*)?/', $data, $r, PREG_SET_ORDER)))
-		foreach ($r as $m) {
+	AND (preg_match_all('/&([^;]*)?/', $data, $r, PREG_SET_ORDER))) {
+			foreach ($r as $m) {
 			list($t,$e) = $m;
-			if (!isset($this->dtc->entites[$e]))
-				coordonnees_erreur($this, " <b>$e</b> "
+	}
+			if (!isset($this->dtc->entites[$e])) {
+							coordonnees_erreur($this, " <b>$e</b> "
 				  . _T('zxml_inconnu_entite')
 				  . ' '
 				  );
+			}
 		}
 	if (isset($this->process['default']) AND ($f = $this->process['default'])) {
 		$f($this, $data);
@@ -307,8 +328,10 @@ 
 
 // Init
 function ValidateurXML($process=array()) {
-	if (is_array($process)) $this->process = $process;
-}
+	if (is_array($process)) {
+	    $this->process = $process;
+	}
+	}
 
  var $ids = array();
  var $idrefs = array();

ecrire/xml/interfaces.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;
+}
 
 define('_REGEXP_DOCTYPE',
 	'/^((?:<\001?[?][^>]*>\s*)*(?:<!--.*?-->\s*)*)*<!DOCTYPE\s+(\w+)\s+(\w+)\s*([^>]*)>\s*/s');

ecrire/base/upgrade.php

Braces +59 added lines, -36 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;
+}
 
 /**
  * Programme de mise a jour des tables SQL lors d'un chgt de version.
@@ -30,7 +32,10 @@ 
  */
 function base_upgrade_dist($titre='', $reprise='')
 {
-	if (!$titre) return; // anti-testeur automatique
+	if (!$titre) {
+	    return;
+	}
+	// anti-testeur automatique
 	if ($GLOBALS['spip_version_base']!=$GLOBALS['meta']['version_installee']) {
 		if (!is_numeric(_request('reinstall'))) {
 			include_spip('base/create');
@@ -93,7 +98,9 @@ 
 			    'impt' => 'non'));
 		return false;
 	}
-	if (!upgrade_test()) return true;
+	if (!upgrade_test()) {
+	    return true;
+	}
 	
 	$cible = ($version_cible ? $version_cible : $spip_version_base);
 
@@ -105,27 +112,31 @@ 
 			if ($f) {
 				spip_log( "$f repercute les modifications de la version " . ($n/10),"maj."._LOG_INFO_IMPORTANTE);
 				$f($version_installee, $spip_version_base);
-			} else spip_log( "pas de fonction pour la maj $n $nom","maj."._LOG_INFO_IMPORTANTE);
+			} else {
+			    spip_log( "pas de fonction pour la maj $n $nom","maj."._LOG_INFO_IMPORTANTE);
+			}
 			$n++;
 		}
 		include_spip('maj/v019_pre193');
 		v019_pre193($version_installee, $version_cible);
 	}
 	if ($version_installee < 2000) {
-		if ($version_installee < 2)
-			$version_installee = $version_installee*1000;
+		if ($version_installee < 2) {
+					$version_installee = $version_installee*1000;
+		}
 		include_spip('maj/v019');
 	}
-	if ($cible < 2)
-		$cible = $cible*1000;
+	if ($cible < 2) {
+			$cible = $cible*1000;
+	}
 
 	include_spip('maj/svn10000');
 	ksort($GLOBALS['maj']);
 	$res = maj_while($version_installee, $cible, $GLOBALS['maj'], 'version_installee','meta', $redirect, true);
 	if ($res) {
-		if (!is_array($res))
-			spip_log("Pb d'acces SQL a la mise a jour","maj."._LOG_INFO_ERREUR);
-		else {
+		if (!is_array($res)) {
+					spip_log("Pb d'acces SQL a la mise a jour","maj."._LOG_INFO_ERREUR);
+		} else {
 			echo _T('avis_operation_echec') . ' ' . join(' ', $res);
 			echo install_fin_html();
 		}
@@ -166,8 +177,9 @@ 
  */
 function maj_plugin($nom_meta_base_version, $version_cible, $maj, $table_meta='meta'){
 
-	if ($table_meta!=='meta')
-		lire_metas($table_meta);
+	if ($table_meta!=='meta') {
+			lire_metas($table_meta);
+	}
 	if ( (!isset($GLOBALS[$table_meta][$nom_meta_base_version]) )
 			|| (!spip_version_compare($current_version = $GLOBALS[$table_meta][$nom_meta_base_version],$version_cible,'='))){
 
@@ -201,9 +213,9 @@ 
 		
 		$res = maj_while($current_version, $version_cible, $maj, $nom_meta_base_version, $table_meta, $redirect);
 		if ($res) {
-			if (!is_array($res))
-				spip_log("Pb d'acces SQL a la mise a jour","maj."._LOG_INFO_ERREUR);
-			else {
+			if (!is_array($res)) {
+							spip_log("Pb d'acces SQL a la mise a jour","maj."._LOG_INFO_ERREUR);
+			} else {
 				echo "<p>"._T('avis_operation_echec') . ' ' . join(' ', $res)."</p>";
 			}
 		}
@@ -245,7 +257,9 @@ 
  */
 function maj_debut_page($installee,$meta,$table){
 	static $done = false;
-	if ($done) return;
+	if ($done) {
+	    return;
+	}
 	include_spip('inc/minipres');
 	@ini_set("zlib.output_compression","0"); // pour permettre l'affichage au fur et a mesure
 	$timeout = _UPGRADE_TIME_OUT*2;
@@ -315,17 +329,22 @@ 
 		if ($v=='init' OR
 			(spip_version_compare($v,$installee,'>')
 			AND spip_version_compare($v,$cible,'<='))) {
-			if ($debut_page)
-				maj_debut_page($v,$meta,$table);
+			if ($debut_page) {
+							maj_debut_page($v,$meta,$table);
+			}
 			echo "MAJ $v";
 			$etape = serie_alter($v, $maj[$v], $meta, $table, $redirect);
 			$trouver_table(''); // vider le cache des descriptions de table
 			# echec sur une etape en cours ?
 			# on sort
-			if ($etape) return array($v, $etape);
+			if ($etape) {
+			    return array($v, $etape);
+			}
 			$n = time() - $time;
 			spip_log( "$table $meta: $v en $n secondes",'maj.'._LOG_INFO_IMPORTANTE);
-			if ($meta) ecrire_meta($meta, $installee=$v,'oui', $table);
+			if ($meta) {
+			    ecrire_meta($meta, $installee=$v,'oui', $table);
+			}
 			echo "<br />";
 		}
 		if (time() >= _TIME_OUT) {
@@ -335,7 +354,9 @@ 
 	$trouver_table(''); // vider le cache des descriptions de table
 	// indispensable pour les chgt de versions qui n'ecrivent pas en base
 	// tant pis pour la redondance eventuelle avec ci-dessus
-	if ($meta) ecrire_meta($meta, $cible,'oui',$table);
+	if ($meta) {
+	    ecrire_meta($meta, $cible,'oui',$table);
+	}
 	spip_log( "MAJ terminee. $meta: $installee",'maj.'._LOG_INFO_IMPORTANTE);
 	return array();
 }
@@ -373,8 +394,9 @@ 
 				// mais pour les fonctions complexes,
 				// il faut les rejouer jusqu'a achevement.
 				// C'est a elle d'assurer qu'elles progressent a chaque rappel
-				if (strncmp($f,"sql_",4)==0)
-					ecrire_meta($meta2, $i+1, 'non', $table);
+				if (strncmp($f,"sql_",4)==0) {
+									ecrire_meta($meta2, $i+1, 'non', $table);
+				}
 				echo " <span title='$i'>.</span>";
 				call_user_func_array($f, $r);
 				// si temps imparti depasse, on relance sans ecrire en meta
@@ -384,12 +406,12 @@ 
 				}
 				ecrire_meta($meta2, $i+1, 'non', $table);
 				spip_log( "$meta2: ok", 'maj.'._LOG_INFO_IMPORTANTE);
-			}
-			else {
-				if (!is_array($r))
-					spip_log("maj $i format incorrect","maj."._LOG_ERREUR);
-				else
-					spip_log("maj $i fonction $f non definie","maj."._LOG_ERREUR);
+			} else {
+				if (!is_array($r)) {
+									spip_log("maj $i format incorrect","maj."._LOG_ERREUR);
+				} else {
+									spip_log("maj $i fonction $f non definie","maj."._LOG_ERREUR);
+				}
 				// en cas d'erreur serieuse, on s'arrete
 				// mais on permet de passer par dessus en rechargeant la page.
 				return $i+1;
@@ -408,9 +430,10 @@ 
 // http://doc.spip.org/@upgrade_types_documents
 function upgrade_types_documents() {
 	if (include_spip('base/medias')
-	AND function_exists('creer_base_types_doc'))
-		creer_base_types_doc();
-}
+	AND function_exists('creer_base_types_doc')) {
+			creer_base_types_doc();
+	}
+	}
 
 // http://doc.spip.org/@upgrade_test
 function upgrade_test() {
@@ -431,9 +454,9 @@ 
 // http://doc.spip.org/@maj_version
 function maj_version ($version, $test = true) {
 	if ($test) {
-		if ($version>=1.922)
-			ecrire_meta('version_installee', $version, 'oui');
-		else {
+		if ($version>=1.922) {
+					ecrire_meta('version_installee', $version, 'oui');
+		} else {
 			// on le fait manuellement, car ecrire_meta utilise le champs impt qui est absent sur les vieilles versions
 			$GLOBALS['meta']['version_installee'] = $version;
 			sql_updateq('spip_meta',  array('valeur' => $version), "nom=" . sql_quote('version_installee') );

ecrire/base/trouver_table.php

Braces +24 added lines, -19 removed lines

@@ -15,7 +15,9 @@ 
  *
  * @package SPIP\SQL\Tables
 **/
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 include_spip('base/objets');
 
 /**
@@ -66,8 +68,9 @@ 
 	global $tables_principales, $tables_auxiliaires, $table_des_tables;
 
 	if (!spip_connect($serveur)
-	OR !preg_match('/^[a-zA-Z0-9._-]*/',$nom))
-		return null;
+	OR !preg_match('/^[a-zA-Z0-9._-]*/',$nom)) {
+			return null;
+	}
 
 	$connexion = &$GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
 	$objets_sql = lister_tables_objets_sql("::md5");
@@ -93,10 +96,11 @@ 
 	}
 
 	$nom_sql = $nom;
-	if (preg_match('/\.(.*)$/', $nom, $s))
-		$nom_sql = $s[1];
-	else
-		$nom_sql = $nom;
+	if (preg_match('/\.(.*)$/', $nom, $s)) {
+			$nom_sql = $s[1];
+	} else {
+			$nom_sql = $nom;
+	}
 
 	$fdesc = $desc = '';
 	$connexion = &$GLOBALS['connexions'][$serveur ? $serveur : 0];
@@ -117,21 +121,22 @@ 
 	  AND defined('_VAR_MODE') AND _VAR_MODE!=='recalcul'
 	  AND (!isset($connexion['tables']) OR !$connexion['tables'])) {
 		if (lire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],$desc_cache)
-		  AND $desc_cache=unserialize($desc_cache))
-		  $connexion['tables'] = $desc_cache;
+		  AND $desc_cache=unserialize($desc_cache)) {
+				  $connexion['tables'] = $desc_cache;
+		}
 	}
 	if ($table_spip AND !isset($connexion['tables'][$nom_sql])) {
 
-		if (isset($tables_principales[$nom_sql]))
-			$fdesc = $tables_principales[$nom_sql];
+		if (isset($tables_principales[$nom_sql])) {
+					$fdesc = $tables_principales[$nom_sql];
+		}
 		// meme si pas d'abreviation declaree, trouver la table spip_$nom
 		// si c'est une table principale,
 		// puisqu'on le fait aussi pour les tables auxiliaires
 		elseif ($nom_sql==$nom AND isset($tables_principales['spip_' .$nom])){
 			$nom_sql = 'spip_' . $nom;
 			$fdesc = &$tables_principales[$nom_sql];
-		}
-		elseif (isset($tables_auxiliaires[$n=$nom])
+		} elseif (isset($tables_auxiliaires[$n=$nom])
 		  OR isset($tables_auxiliaires[$n='spip_'.$nom])) {
 			$nom_sql = $n;
 			$fdesc = &$tables_auxiliaires[$n];
@@ -149,8 +154,7 @@ 
 			// on ne sait pas lire la structure de la table :
 			// on retombe sur la description donnee dans les fichiers spip
 			$desc = $fdesc;
-		}
-		else {
+		} else {
 			$desc['exist'] = true;
 		}
 
@@ -167,11 +171,12 @@ 
 		$res = &$connexion['tables'][$nom_sql];
 		// une nouvelle table a ete decrite
 		// mettons donc a jour le cache des descriptions de ce serveur
-		if (is_writeable(_DIR_CACHE))
-			ecrire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],serialize($connexion['tables']),true);
+		if (is_writeable(_DIR_CACHE)) {
+					ecrire_fichier($nom_cache_desc_sql[$serveur][$objets_sql],serialize($connexion['tables']),true);
+		}
+	} else {
+			$res = &$connexion['tables'][$nom_sql];
 	}
-	else
-		$res = &$connexion['tables'][$nom_sql];
 
 	// toujours retourner $nom dans id_table
 	$res['id_table']=$nom;

ecrire/base/connect_sql.php

Braces +68 added lines, -35 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;
+}
 require_once _ROOT_RESTREINT . 'base/objets.php';
 
 //
@@ -30,8 +32,12 @@ 
 
 	$serveur = !is_string($serveur) ? '' : strtolower($serveur);
 	$index = $serveur ? $serveur : 0;
-	if (!$version) $version = $spip_sql_version;
-	if (isset($connexions[$index][$version])) return $connexions[$index];
+	if (!$version) {
+	    $version = $spip_sql_version;
+	}
+	if (isset($connexions[$index][$version])) {
+	    return $connexions[$index];
+	}
 
 	include_spip('base/abstract_sql');
 	$install = (_request('exec') == 'install');
@@ -60,11 +66,14 @@ 
 		}
 		if (!isset($GLOBALS['db_ok'])) {
 		  // fera mieux la prochaine fois
-			if ($install) return false;
-			if ($f AND is_readable($f))
-				spip_log("spip_connect: fichier de connexion '$f' OK.", _LOG_INFO_IMPORTANTE);
-			else
-				spip_log("spip_connect: fichier de connexion '$f' non trouve", _LOG_INFO_IMPORTANTE);
+			if ($install) {
+			    return false;
+			}
+			if ($f AND is_readable($f)) {
+							spip_log("spip_connect: fichier de connexion '$f' OK.", _LOG_INFO_IMPORTANTE);
+			} else {
+							spip_log("spip_connect: fichier de connexion '$f' non trouve", _LOG_INFO_IMPORTANTE);
+			}
 			spip_log("spip_connect: echec connexion ou serveur $index mal defini dans '$f'.", _LOG_HS);
 			// ne plus reessayer si ce n'est pas l'install
 			return $connexions[$index]=false;
@@ -72,7 +81,9 @@ 
 		$connexions[$index] = $GLOBALS['db_ok'];
 	}
 	// si la connexion a deja ete tentee mais a echoue, le dire!
-	if (!$connexions[$index]) return false;
+	if (!$connexions[$index]) {
+	    return false;
+	}
 
 	// la connexion a reussi ou etait deja faite.
 	// chargement de la version du jeu de fonctions
@@ -87,7 +98,9 @@ 
 		}
 	}
 	$connexions[$index][$version] = $GLOBALS[$jeu];
-	if ($old) return $connexions[$index];
+	if ($old) {
+	    return $connexions[$index];
+	}
 
 	$connexions[$index]['spip_connect_version'] = isset($GLOBALS['spip_connect_version']) ? $GLOBALS['spip_connect_version'] : 0;
 
@@ -108,14 +121,17 @@ 
 		// C'est le cas d'un dump sqlite par exemple 
 		if ($connexions[$index]['spip_connect_version']
 		AND sql_showtable('spip_meta', true, $serveur)
-		AND $r = sql_getfetsel('valeur', 'spip_meta', "nom='charset_sql_connexion'",'','','','',$serveur))
-			$charset = $r;
-		else $charset = -1;
+		AND $r = sql_getfetsel('valeur', 'spip_meta', "nom='charset_sql_connexion'",'','','','',$serveur)) {
+					$charset = $r;
+		} else {
+		    $charset = -1;
+		}
 	}
 	if ($charset != -1) {
 		$f = $GLOBALS[$jeu]['set_charset'];
-		if (function_exists($f))
-			$f($charset, $serveur);
+		if (function_exists($f)) {
+					$f($charset, $serveur);
+		}
 	}
 	return $connexions[$index];
 }
@@ -138,10 +154,15 @@ 
 // http://doc.spip.org/@spip_connect_sql
 function spip_connect_sql($version, $ins='', $serveur='', $cont=false) {
 	$desc = spip_connect($serveur, $version);
-	if (function_exists($f = @$desc[$version][$ins])) return $f;
-	if ($cont) return $desc;
-	if ($ins)
-		spip_log("Le serveur '$serveur' version $version n'a pas '$ins'", _LOG_ERREUR);
+	if (function_exists($f = @$desc[$version][$ins])) {
+	    return $f;
+	}
+	if ($cont) {
+	    return $desc;
+	}
+	if ($ins) {
+			spip_log("Le serveur '$serveur' version $version n'a pas '$ins'", _LOG_ERREUR);
+	}
 	include_spip('inc/minipres');
 	echo minipres(_T('info_travaux_titre'), _T('titre_probleme_technique'));
 	exit;
@@ -171,18 +192,20 @@ 
 
 	// temps avant nouvelle tentative de connexion
 	// suite a une connection echouee
-	if (!defined('_CONNECT_RETRY_DELAY'))
-		define('_CONNECT_RETRY_DELAY',30);
+	if (!defined('_CONNECT_RETRY_DELAY')) {
+			define('_CONNECT_RETRY_DELAY',30);
+	}
 
 	$f = "";
 	// un fichier de identifiant par combinaison (type,host,port,db)
 	// pour ne pas declarer tout indisponible d'un coup
 	// si en cours d'installation ou si db=@test@ on ne pose rien
 	// car c'est un test de connexion
-	if (!defined('_ECRIRE_INSTALL') AND !$db=="@test@")
-		$f = _DIR_TMP . $type . '.' . substr(md5($host.$port.$db),0,8) . '.out';
-	elseif ($db=='@test@')
-		$db = '';
+	if (!defined('_ECRIRE_INSTALL') AND !$db=="@test@") {
+			$f = _DIR_TMP . $type . '.' . substr(md5($host.$port.$db),0,8) . '.out';
+	} elseif ($db=='@test@') {
+			$db = '';
+	}
 
 	if ($f
 		AND @file_exists($f)
@@ -191,9 +214,10 @@ 
 		return;
 	}
 
-	if (!$prefixe)
-		$prefixe = isset($GLOBALS['table_prefix'])
+	if (!$prefixe) {
+			$prefixe = isset($GLOBALS['table_prefix'])
 		? $GLOBALS['table_prefix'] : $db;
+	}
 	$h = charger_fonction($type, 'req', true);
 	if (!$h) {
 		spip_log( "les requetes $type ne sont pas fournies", _LOG_HS);
@@ -238,12 +262,17 @@ 
 		redirige_url_ecrire('upgrade', 'reinstall=oui');
 	}
 
-	if (!($f = $connexion['select'])) return false;
+	if (!($f = $connexion['select'])) {
+	    return false;
+	}
 	// en cas d'erreur select retourne la requette (is_string=true donc)
 	if (!$r = $f('valeur','spip_meta', "nom='charset_sql_connexion'")
-	  OR is_string($r))
-		return false;
-	if (!($f = $connexion['fetch'])) return false;
+	  OR is_string($r)) {
+			return false;
+	}
+	if (!($f = $connexion['fetch'])) {
+	    return false;
+	}
 	$r = $f($r);
 	return ($r['valeur'] ? $r['valeur'] : -1);
 }
@@ -273,8 +302,12 @@ 
 	$ydesc = $trouver_table(table_objet($y));
 	$ix = @$xdesc['key']["PRIMARY KEY"];
 	$iy = @$ydesc['key']["PRIMARY KEY"];
-	if ($table = $ydesc['tables_jointures'][$ix]) return $table;
-	if ($table = $xdesc['tables_jointures'][$iy]) return $table;
+	if ($table = $ydesc['tables_jointures'][$ix]) {
+	    return $table;
+	}
+	if ($table = $xdesc['tables_jointures'][$iy]) {
+	    return $table;
+	}
 	return '';
 }
 
@@ -306,9 +339,9 @@ 
 				break;
 		}
 		$query = str_replace($textes,$replace,$query);
+	} else {
+			$textes = array();
 	}
-	else
-		$textes = array();
 
 	return array($query, $textes);
 }

ecrire/base/abstract_sql.php

Braces +289 added lines, -107 removed lines

@@ -24,7 +24,9 @@ 
  * @version 1
  */
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /** Version de l'API SQL */
 define('sql_ABSTRACT_VERSION', 1);
@@ -56,7 +58,9 @@ 
 	static $sql_serveur = array();
 	if (!isset($sql_serveur[$serveur][$ins_sql])){
 		$f = spip_connect_sql(sql_ABSTRACT_VERSION, $ins_sql, $serveur, $continue);
-		if (!is_string($f) OR !$f) return $f;
+		if (!is_string($f) OR !$f) {
+		    return $f;
+		}
 		$sql_serveur[$serveur][$ins_sql] = $f;
 	}
 	return $sql_serveur[$serveur][$ins_sql];
@@ -86,8 +90,9 @@ 
 	$desc = $desc[sql_ABSTRACT_VERSION];
 	$c = $desc['charsets'][$charset];
 	if ($c) {
-		if (function_exists($f=@$desc['get_charset'])) 
-			if ($f($c, $serveur, $option!==false)) return $c;
+		if (function_exists($f=@$desc['get_charset'])) {
+					if ($f($c, $serveur, $option!==false)) return $c;
+		}
 	}
 	spip_log( "SPIP ne connait pas les Charsets disponibles sur le serveur $serveur. Le serveur choisira seul.", _LOG_AVERTISSEMENT);
 	return false;
@@ -117,7 +122,9 @@ 
 **/
 function sql_set_charset($charset,$serveur='', $option=true){
 	$f = sql_serveur('set_charset', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	return $f($charset, $serveur, $option!==false);
 }
 
@@ -171,14 +178,18 @@ 
 	$groupby = array(), $orderby = array(), $limit = '', $having = array(),
 	$serveur='', $option=true) {
 	$f = sql_serveur('select', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 
 	$debug = (defined('_VAR_MODE') AND _VAR_MODE == 'debug');
 	if (($option !== false) AND !$debug) {
 		$res = $f($select, $from, $where, $groupby, $orderby, $limit, $having, $serveur, is_array($option) ? true : $option);
 	} else {
 		$query = $f($select, $from, $where, $groupby, $orderby, $limit, $having, $serveur, false);
-		if (!$option) return $query;
+		if (!$option) {
+		    return $query;
+		}
 		// le debug, c'est pour ce qui a ete produit par le compilateur
 		if (isset($GLOBALS['debug']['aucasou'])) {
 			list($table, $id,) = $GLOBALS['debug']['aucasou'];
@@ -189,7 +200,9 @@ 
 	}
 
 	// en cas d'erreur
-	if (!is_string($res)) return $res;
+	if (!is_string($res)) {
+	    return $res;
+	}
 	// denoncer l'erreur SQL dans sa version brute
 	spip_sql_erreur($serveur);
 	// idem dans sa version squelette (prefixe des tables non substitue)
@@ -270,9 +283,13 @@ 
 		      $groupby = array(), $having = array(),
 	$serveur='', $option=true) {
 	$f = sql_serveur('countsel', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($from, $where, $groupby, $having, $serveur, $option!==false);
-	if ($r===false) spip_sql_erreur($serveur);
+	if ($r===false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -301,9 +318,13 @@ 
 **/
 function sql_alter($q, $serveur='', $option=true) {
 	$f = sql_serveur('alter', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($q, $serveur, $option!==false);
-	if ($r===false) spip_sql_erreur($serveur);
+	if ($r===false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -328,7 +349,9 @@ 
  */
 function sql_fetch($res, $serveur='', $option=true) {
 	$f = sql_serveur('fetch', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	return $f($res, NULL, $serveur, $option!==false);
 }
 
@@ -357,11 +380,16 @@ 
  */
 function sql_fetch_all($res, $serveur='', $option=true){
 	$rows = array();
-	if (!$res) return $rows;
+	if (!$res) {
+	    return $rows;
+	}
 	$f = sql_serveur('fetch', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return array();
-	while ($r = $f($res, NULL, $serveur, $option!==false))
-		$rows[] = $r;
+	if (!is_string($f) OR !$f) {
+	    return array();
+	}
+	while ($r = $f($res, NULL, $serveur, $option!==false)) {
+			$rows[] = $r;
+	}
 	sql_free($res, $serveur);
 	return $rows;
 }
@@ -392,9 +420,13 @@ 
 **/
 function sql_seek($res, $row_number, $serveur='', $option=true) {
 	$f = sql_serveur('seek', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($res, $row_number, $serveur, $option!==false);
-	if ($r===false) spip_sql_erreur($serveur);
+	if ($r===false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -421,9 +453,13 @@ 
 **/
 function sql_listdbs($serveur='', $option=true) {
 	$f = sql_serveur('listdbs', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($serveur);
-	if ($r===false) spip_sql_erreur($serveur);
+	if ($r===false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -448,9 +484,13 @@ 
 function sql_selectdb($nom, $serveur='', $option=true)
 {
 	$f = sql_serveur('selectdb', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($nom, $serveur, $option!==false);
-	if ($r===false) spip_sql_erreur($serveur);
+	if ($r===false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -458,9 +498,13 @@ 
 function sql_count($res, $serveur='', $option=true)
 {
 	$f = sql_serveur('count', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($res, $serveur, $option!==false);
-	if ($r===false) spip_sql_erreur($serveur);
+	if ($r===false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -468,7 +512,9 @@ 
 function sql_free($res, $serveur='', $option=true)
 {
 	$f = sql_serveur('free', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	return $f($res);
 }
 
@@ -480,9 +526,13 @@ 
 function sql_insert($table, $noms, $valeurs, $desc=array(), $serveur='', $option=true)
 {
 	$f = sql_serveur('insert', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $noms, $valeurs, $desc, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -490,9 +540,13 @@ 
 function sql_insertq($table, $couples=array(), $desc=array(), $serveur='', $option=true)
 {
 	$f = sql_serveur('insertq', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $couples, $desc, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -500,9 +554,13 @@ 
 function sql_insertq_multi($table, $couples=array(), $desc=array(), $serveur='', $option=true)
 {
 	$f = sql_serveur('insertq_multi', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $couples, $desc, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -510,9 +568,13 @@ 
 function sql_update($table, $exp, $where='', $desc=array(), $serveur='', $option=true)
 {
 	$f = sql_serveur('update', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $exp, $where, $desc, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -523,9 +585,13 @@ 
 function sql_updateq($table, $exp, $where='', $desc=array(), $serveur='', $option=true)
 {
 	$f = sql_serveur('updateq', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $exp, $where, $desc, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -533,9 +599,13 @@ 
 function sql_delete($table, $where='', $serveur='', $option=true)
 {
 	$f = sql_serveur('delete', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $where, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -543,9 +613,13 @@ 
 function sql_replace($table, $couples, $desc=array(), $serveur='', $option=true)
 {
 	$f = sql_serveur('replace', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $couples, $desc, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -554,9 +628,13 @@ 
 function sql_replace_multi($table, $tab_couples, $desc=array(), $serveur='', $option=true)
 {
 	$f = sql_serveur('replace_multi', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $tab_couples, $desc, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -564,9 +642,13 @@ 
 function sql_drop_table($table, $exist='', $serveur='', $option=true)
 {
 	$f = sql_serveur('drop_table', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $exist, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -575,9 +657,13 @@ 
 function sql_drop_view($table, $exist='', $serveur='', $option=true)
 {
 	$f = sql_serveur('drop_view', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $exist, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -602,7 +688,9 @@ 
 function sql_showbase($spip=NULL, $serveur='', $option=true)
 {
 	$f = sql_serveur('showbase', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 
 	// la globale n'est remplie qu'apres l'appel de sql_serveur.
 	if ($spip == NULL){
@@ -636,7 +724,10 @@ 
 {
 	$q = sql_showbase($spip, $serveur, $option);
 	$r = array();
-	if ($q) while ($t = sql_fetch($q, $serveur)) { $r[] = array_shift($t);}
+	if ($q) {
+	    while ($t = sql_fetch($q, $serveur)) { $r[] = array_shift($t);
+	}
+	}
 	return $r;
 }
 
@@ -644,39 +735,54 @@ 
 function sql_showtable($table, $table_spip = false, $serveur='', $option=true)
 {
 	$f = sql_serveur('showtable', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 
 	// la globale n'est remplie qu'apres l'appel de sql_serveur.
 	if ($table_spip){
 		$connexion = $GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
 		$prefixe = $connexion['prefixe'];
 		$vraie_table = preg_replace('/^spip/', $prefixe, $table);
-	} else $vraie_table = $table;
+	} else {
+	    $vraie_table = $table;
+	}
 
 	$f = $f($vraie_table, $serveur, $option!==false);
-	if (!$f) return array();
-	if (isset($GLOBALS['tables_principales'][$table]['join']))
-		$f['join'] = $GLOBALS['tables_principales'][$table]['join'];
-	elseif (isset($GLOBALS['tables_auxiliaires'][$table]['join']))
-		$f['join'] = $GLOBALS['tables_auxiliaires'][$table]['join'];
+	if (!$f) {
+	    return array();
+	}
+	if (isset($GLOBALS['tables_principales'][$table]['join'])) {
+			$f['join'] = $GLOBALS['tables_principales'][$table]['join'];
+	} elseif (isset($GLOBALS['tables_auxiliaires'][$table]['join'])) {
+			$f['join'] = $GLOBALS['tables_auxiliaires'][$table]['join'];
+	}
 	return $f;
 }
 
 // http://doc.spip.org/@sql_create
 function sql_create($nom, $champs, $cles=array(), $autoinc=false, $temporary=false, $serveur='', $option=true) {
 	$f = sql_serveur('create', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($nom, $champs, $cles, $autoinc, $temporary, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
 function sql_create_base($nom, $serveur='', $option=true)
 {
 	$f = sql_serveur('create_base', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($nom, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -687,9 +793,13 @@ 
 // http://doc.spip.org/@sql_create_view
 function sql_create_view($nom, $select_query, $serveur='', $option=true) {
 	$f = sql_serveur('create_view', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($nom, $select_query, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -697,7 +807,9 @@ 
 function sql_multi($sel, $lang, $serveur='', $option=true)
 {
 	$f = sql_serveur('multi', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	return $f($sel, $lang);
 }
 
@@ -714,7 +826,9 @@ 
  */
 function sql_error($serveur='') {
 	$f = sql_serveur('error', $serveur, 'continue');
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	return $f('query inconnue', $serveur);
 }
 
@@ -730,34 +844,48 @@ 
  */
 function sql_errno($serveur='') {
 	$f = sql_serveur('errno', $serveur, 'continue');
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	return $f($serveur);
 }
 
 // http://doc.spip.org/@sql_explain
 function sql_explain($q, $serveur='', $option=true) {
 	$f = sql_serveur('explain', $serveur,  'continue');
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($q, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
 // http://doc.spip.org/@sql_optimize
 function sql_optimize($table, $serveur='', $option=true) {
 	$f = sql_serveur('optimize', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
 // http://doc.spip.org/@sql_repair
 function sql_repair($table, $serveur='', $option=true) {
 	$f = sql_serveur('repair', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($table, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -767,9 +895,13 @@ 
 // http://doc.spip.org/@sql_query
 function sql_query($ins, $serveur='', $option=true) {
 	$f = sql_serveur('query', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($ins, $serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 }
 
@@ -822,8 +954,12 @@ 
 	$groupby = array(), $orderby = array(), $limit = '',
 	$having = array(), $serveur='', $option=true) {
 	$q = sql_select($select, $from, $where,	$groupby, $orderby, $limit, $having, $serveur, $option);
-	if ($option===false) return $q;
-	if (!$q) return array();
+	if ($option===false) {
+	    return $q;
+	}
+	if (!$q) {
+	    return array();
+	}
 	$r = sql_fetch($q, $serveur, $option);
 	sql_free($q, $serveur, $option);
 	return $r;
@@ -886,7 +1022,9 @@ 
 	$groupby = array(), $orderby = array(), $limit = '',
 	$having = array(), $serveur='', $option=true) {
 	$q = sql_select($select, $from, $where,	$groupby, $orderby, $limit, $having, $serveur, $option);
-	if ($option===false) return $q;
+	if ($option===false) {
+	    return $q;
+	}
 	return sql_fetch_all($q, $serveur, $option);
 }
 
@@ -933,12 +1071,18 @@ 
 **/
 function sql_getfetsel($select, $from = array(), $where = array(), $groupby = array(), 
 	$orderby = array(), $limit = '', $having = array(), $serveur='', $option=true) {
-	if (preg_match('/\s+as\s+(\w+)$/i', $select, $c)) $id = $c[1];
-	elseif (!preg_match('/\W/', $select)) $id = $select;
-	else {$id = 'n'; $select .= ' AS n';}
+	if (preg_match('/\s+as\s+(\w+)$/i', $select, $c)) {
+	    $id = $c[1];
+	} elseif (!preg_match('/\W/', $select)) {
+	    $id = $select;
+	} else {$id = 'n'; $select .= ' AS n';}
 	$r = sql_fetsel($select, $from, $where,	$groupby, $orderby, $limit, $having, $serveur, $option);
-	if ($option===false) return $r;
-	if (!$r) return NULL;
+	if ($option===false) {
+	    return $r;
+	}
+	if (!$r) {
+	    return NULL;
+	}
 	return $r[$id]; 
 }
 
@@ -995,9 +1139,13 @@ 
 **/
 function sql_preferer_transaction($serveur='', $option=true) {
 	$f = sql_serveur('preferer_transaction', $serveur,  'continue');
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 };
 
@@ -1021,9 +1169,13 @@ 
 **/
 function sql_demarrer_transaction($serveur='', $option=true) {
 	$f = sql_serveur('demarrer_transaction', $serveur,  'continue');
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 };
 
@@ -1047,9 +1199,13 @@ 
 **/
 function sql_terminer_transaction($serveur='', $option=true) {
 	$f = sql_serveur('terminer_transaction', $serveur,  'continue');
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	$r = $f($serveur, $option!==false);
-	if ($r === false) spip_sql_erreur($serveur);
+	if ($r === false) {
+	    spip_sql_erreur($serveur);
+	}
 	return $r;
 };
 
@@ -1076,7 +1232,9 @@ 
 function sql_hex($val, $serveur='', $option=true)
 {
 	$f = sql_serveur('hex', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	return $f($val);
 }
 
@@ -1103,14 +1261,18 @@ 
 function sql_quote($val, $serveur='', $type='')
 {
 	$f = sql_serveur('quote', $serveur, true);
-	if (!is_string($f) OR !$f) $f = '_q';
+	if (!is_string($f) OR !$f) {
+	    $f = '_q';
+	}
 	return $f($val, $type);
 }
 
 function sql_date_proche($champ, $interval, $unite, $serveur='', $option=true)
 {
 	$f = sql_serveur('date_proche', $serveur, true);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	return $f($champ, $interval, $unite);
 }
 
@@ -1145,13 +1307,21 @@ 
 function sql_in($val, $valeurs, $not='', $serveur='', $option=true) {
 	if (is_array($valeurs)) {
 		$f = sql_serveur('quote', $serveur, true);
-		if (!is_string($f) OR !$f) return false;
+		if (!is_string($f) OR !$f) {
+		    return false;
+		}
 		$valeurs = join(',', array_map($f, array_unique($valeurs)));
-	} elseif (isset($valeurs[0]) AND $valeurs[0]===',') $valeurs = substr($valeurs,1);
-	if (!strlen(trim($valeurs))) return ($not ? "0=0" : '0=1');
+	} elseif (isset($valeurs[0]) AND $valeurs[0]===',') {
+	    $valeurs = substr($valeurs,1);
+	}
+	if (!strlen(trim($valeurs))) {
+	    return ($not ? "0=0" : '0=1');
+	}
 
 	$f = sql_serveur('in', $serveur,  $option==='continue' OR $option===false);
-	if (!is_string($f) OR !$f) return false;
+	if (!is_string($f) OR !$f) {
+	    return false;
+	}
 	return $f($val, $valeurs, $not, $serveur, $option!==false);
 }
 
@@ -1198,18 +1368,21 @@ 
  */
 function sql_skip($res, $pos, $saut, $count, $serveur='', $option=true){
 	// pas de saut en arriere qu'on ne sait pas faire sans sql_seek
-	if (($saut=intval($saut))<=0) return $pos;
+	if (($saut=intval($saut))<=0) {
+	    return $pos;
+	}
 
 	$seek = $pos + $saut;
 	// si le saut fait depasser le maxi, on libere la resource
 	// et on sort
 	if ($seek>=$count) {sql_free($res, $serveur, $option); return $count;}
 
-	if (sql_seek($res, $seek))
-		$pos = $seek;
-	else
-		while ($pos<$seek AND sql_fetch($res, $serveur, $option))
+	if (sql_seek($res, $seek)) {
+			$pos = $seek;
+	} else {
+			while ($pos<$seek AND sql_fetch($res, $serveur, $option))
 			$pos++;
+	}
 	return $pos;
 }
 
@@ -1248,8 +1421,12 @@ 
 	$annee = sprintf("%04s",$annee);
 	$mois = sprintf("%02s",$mois);
 
-	if ($annee == "0000") $mois = 0;
-	if ($mois == "00") $jour = 0;
+	if ($annee == "0000") {
+	    $mois = 0;
+	}
+	if ($mois == "00") {
+	    $jour = 0;
+	}
 
 	return sprintf("%04u",$annee) . '-' . sprintf("%02u",$mois) . '-'
 		. sprintf("%02u",$jour) . ' ' . sprintf("%02u",$h) . ':'
@@ -1285,18 +1462,23 @@ 
 	 car sinon on va se comporter differement selon que la table est declaree
 	 ou non
 	 */
-	if (!$trouver_table) $trouver_table = charger_fonction('trouver_table', 'base');
-	if ($desc = $trouver_table($nom, $serveur))
-		return $desc;
+	if (!$trouver_table) {
+	    $trouver_table = charger_fonction('trouver_table', 'base');
+	}
+	if ($desc = $trouver_table($nom, $serveur)) {
+			return $desc;
+	}
 
 	// sauf a l'installation :
 	include_spip('base/serial');
-	if (isset($tables_principales[$nom]))
-		return $tables_principales[$nom];
+	if (isset($tables_principales[$nom])) {
+			return $tables_principales[$nom];
+	}
 
 	include_spip('base/auxiliaires');
-	if (isset($tables_auxiliaires[$nom]))
-		return $tables_auxiliaires[$nom];
+	if (isset($tables_auxiliaires[$nom])) {
+			return $tables_auxiliaires[$nom];
+	}
 
 	return false;
 }

ecrire/base/create.php

Braces +28 added lines, -20 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/acces');
 include_spip('base/objets');
@@ -26,11 +28,11 @@ 
  * @return bool
  */
 function base_determine_autoinc($table,$desc=array()){
-	if ($t=lister_tables_principales() AND isset($t[$table]))
-		$autoinc = true;
-	elseif ($t=lister_tables_auxiliaires() AND isset($t[$table]))
-		$autoinc = false;
-	else {
+	if ($t=lister_tables_principales() AND isset($t[$table])) {
+			$autoinc = true;
+	} elseif ($t=lister_tables_auxiliaires() AND isset($t[$table])) {
+			$autoinc = false;
+	} else {
 		// essayer de faire au mieux !
 		$autoinc = (isset($desc['key']['PRIMARY KEY'])
 						AND strpos($desc['key']['PRIMARY KEY'],',')===false
@@ -60,8 +62,9 @@ 
 	#if (!$sql_desc) $sql_desc = false;
 	#spip_log("table=$table sql_desc:$sql_desc","dbinstall"._LOG_INFO_IMPORTANTE);
 	if (!$sql_desc) {
-		if ($autoinc==='auto')
-			$autoinc = base_determine_autoinc($table,$desc);
+		if ($autoinc==='auto') {
+					$autoinc = base_determine_autoinc($table,$desc);
+		}
 		#spip_log("sql_create $table autoinc=$autoinc","dbinstall"._LOG_INFO_IMPORTANTE);
 		sql_create($table, $desc['field'], $desc['key'], $autoinc, false, $serveur);
 		// verifier la bonne installation de la table (php-fpm es-tu la ?)
@@ -79,23 +82,24 @@ 
 				spip_log("Echec creation table $table","maj"._LOG_CRITIQUE);
 			}
 		}
-	}
-	else {
+	} else {
 		#spip_log("sql_alter $table ... (on s'en fiche)","dbinstall"._LOG_INFO_IMPORTANTE);
 		// ajouter les champs manquants
 		// on ne supprime jamais les champs, car c'est dangereux
 		// c'est toujours a faire manuellement
 		$last = '';
 		foreach($desc['field'] as $field=>$type){
-			if (!isset($sql_desc['field'][$field]))
-				sql_alter("TABLE $table ADD $field $type".($last?" AFTER $last":""),$serveur);
+			if (!isset($sql_desc['field'][$field])) {
+							sql_alter("TABLE $table ADD $field $type".($last?" AFTER $last":""),$serveur);
+			}
 			$last = $field;
 		}
 		foreach($desc['key'] as $key=>$type){
 			// Ne pas oublier les cas des cles non nommees dans la declaration et qui sont retournees
 			// par le showtable sous la forme d'un index de tableau "KEY $type" et non "KEY"
-			if (!isset($sql_desc['key'][$key]) AND !isset($sql_desc['key']["$key $type"]))
-				sql_alter("TABLE $table ADD $key ($type)",$serveur);
+			if (!isset($sql_desc['key'][$key]) AND !isset($sql_desc['key']["$key $type"])) {
+							sql_alter("TABLE $table ADD $key ($type)",$serveur);
+			}
 			$last = $field;
 		}
 
@@ -125,16 +129,20 @@ 
 		$up = array();
 	} else {
 		$old = true;
-		if (!is_array($up)) $up = array($up);
+		if (!is_array($up)) {
+		    $up = array($up);
+		}
 	}
-	foreach($tables_inc as $k => $v)
-		if (!$old OR in_array($k, $up))
+	foreach($tables_inc as $k => $v) {
+			if (!$old OR in_array($k, $up))
 			creer_ou_upgrader_table($k,$v,true,$old,$serveur);
+	}
 
-	foreach($tables_noinc as $k => $v)
-		if (!$old OR in_array($k, $up))
+	foreach($tables_noinc as $k => $v) {
+			if (!$old OR in_array($k, $up))
 			creer_ou_upgrader_table($k,$v,false,$old,$serveur);
-}
+	}
+	}
 
 /**
  * Creer une base de donnee

ecrire/base/auxiliaires.php

Braces +3 added lines, -1 removed lines

@@ -20,7 +20,9 @@
  * @package SPIP\SQL\Tables
 **/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('base/objets');
 lister_tables_objets_sql();

ecrire/base/objets.php

Braces +208 added lines, -138 removed lines

@@ -16,7 +16,9 @@ 
  * @package SPIP\SQL\Tables
 **/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 /**
  * Merge dans un tableau une de ses clés avec une valeur
@@ -30,11 +32,12 @@ 
  * @return void
 **/
 function array_set_merge(&$table,$index,$valeur){
-	if (!isset($table[$index]))
-		$table[$index] = $valeur;
-	else
-		$table[$index] = array_merge($table[$index],$valeur);
-}
+	if (!isset($table[$index])) {
+			$table[$index] = $valeur;
+	} else {
+			$table[$index] = array_merge($table[$index],$valeur);
+	}
+	}
 
 /**
  * Lister les infos de toutes les tables sql declarées
@@ -342,30 +345,32 @@ 
 			if (is_numeric($t)) {
 				$all = array_merge_recursive($all,$infos_tables[$t]);
 				unset($infos_tables[$t]);
+			} else {
+							$infos_tables[$t] = renseigner_table_objet_sql($t,$infos_tables[$t]);
 			}
-			else
-				$infos_tables[$t] = renseigner_table_objet_sql($t,$infos_tables[$t]);
 		}
 
 		// repercuter les proprietes generales communes a tous les objets
 		foreach(array_keys($infos_tables) as $t) {
-			foreach($all as $i=>$v)
-				if (in_array($i,array('tables_jointures','champs_versionnes'))){
+			foreach($all as $i=>$v) {
+							if (in_array($i,array('tables_jointures','champs_versionnes'))){
 					$add = $all[$i];
+			}
 					// eviter les doublons de declaration de table jointure (ex des mots sur auteurs)
 					// pour les declarations generiques avec cles numeriques
 					if ($i=='tables_jointures' AND isset($infos_tables[$t][$i]) AND count($infos_tables[$t][$i])) {
 						$doublons = array_intersect($infos_tables[$t][$i],$add);
 						foreach($doublons as $d){
 							if (is_numeric(array_search($d,$infos_tables[$t][$i]))
-								AND is_numeric($k=array_search($d,$add)))
-								unset($add[$k]);
+								AND is_numeric($k=array_search($d,$add))) {
+															unset($add[$k]);
+							}
 						}
 					}
 					$infos_tables[$t][$i] = array_merge(isset($infos_tables[$t][$i])?$infos_tables[$t][$i]:array(),$add);
+				} else {
+									$infos_tables[$t][$i] = array_merge_recursive(isset($infos_tables[$t][$i])?$infos_tables[$t][$i]:array(),$all[$i]);
 				}
-				else
-					$infos_tables[$t][$i] = array_merge_recursive(isset($infos_tables[$t][$i])?$infos_tables[$t][$i]:array(),$all[$i]);
 		}
 
 		// completer les tables principales et auxiliaires
@@ -379,21 +384,23 @@ 
 			// l'ajouter au tableau
 			$GLOBALS[$principale_ou_auxiliaire][$table] = array();
 			if (isset($infos['field']) AND isset($infos['key'])){
-				foreach(array('field','key','join') as $k)
-					if (isset($infos_tables[$table][$k]))
+				foreach(array('field','key','join') as $k) {
+									if (isset($infos_tables[$table][$k]))
 						$GLOBALS[$principale_ou_auxiliaire][$table][$k] = &$infos_tables[$table][$k];
-			}
-			else {
+				}
+			} else {
 				// ici on ne renvoie que les declarations, donc RIEN
 				// pour avoir la vrai description en base, il faut passer par trouver_table
 				$GLOBALS[$principale_ou_auxiliaire][$table] = array();
 			}
 			if (count($mem)){
-				foreach(array_keys($mem) as $k)
-					if (isset($GLOBALS[$principale_ou_auxiliaire][$table][$k]))
+				foreach(array_keys($mem) as $k) {
+									if (isset($GLOBALS[$principale_ou_auxiliaire][$table][$k]))
 						$GLOBALS[$principale_ou_auxiliaire][$table][$k] = array_merge($GLOBALS[$principale_ou_auxiliaire][$table][$k],$mem[$k]);
-					else
-						$GLOBALS[$principale_ou_auxiliaire][$table][$k] = $mem[$k];
+				}
+					else {
+											$GLOBALS[$principale_ou_auxiliaire][$table][$k] = $mem[$k];
+					}
 			}
 		}
 
@@ -409,15 +416,17 @@ 
 		// signature
 		$md5 = md5(serialize($infos_tables));
 	}
-	if ($table_sql==="::md5")
-		return $md5;
+	if ($table_sql==="::md5") {
+			return $md5;
+	}
 	if ($table_sql AND !isset($infos_tables[$table_sql])){
 		#$desc = renseigner_table_objet_sql($table_sql,$desc);
 		$desc = renseigner_table_objet_interfaces($table_sql,$desc);
 		return $desc;
 	}
-	if ($table_sql)
-		return isset($infos_tables[$table_sql])?$infos_tables[$table_sql]:array();
+	if ($table_sql) {
+			return isset($infos_tables[$table_sql])?$infos_tables[$table_sql]:array();
+	}
 
 	return $infos_tables;
 }
@@ -587,20 +596,24 @@ 
 			$primary = explode(',',$primary);
 			$primary = reset($primary);
 			$infos['type'] = preg_replace(',^spip_|^id_|s$,', '', $primary);
+		} else {
+					$infos['type'] = preg_replace(',^spip_|s$,', '', $table_sql);
 		}
-		else
-			$infos['type'] = preg_replace(',^spip_|s$,', '', $table_sql);
 	}
-	if (!isset($infos['type_surnoms']))
-		$infos['type_surnoms'] = array();
+	if (!isset($infos['type_surnoms'])) {
+			$infos['type_surnoms'] = array();
+	}
 
-	if (!isset($infos['table_objet']))
-		$infos['table_objet'] = preg_replace(',^spip_,', '', $table_sql);
-	if (!isset($infos['table_objet_surnoms']))
-		$infos['table_objet_surnoms'] = array();
+	if (!isset($infos['table_objet'])) {
+			$infos['table_objet'] = preg_replace(',^spip_,', '', $table_sql);
+	}
+	if (!isset($infos['table_objet_surnoms'])) {
+			$infos['table_objet_surnoms'] = array();
+	}
 
-	if (!isset($infos['principale']))
-		$infos['principale'] = (isset($GLOBALS['tables_principales'][$table_sql])?'oui':false);
+	if (!isset($infos['principale'])) {
+			$infos['principale'] = (isset($GLOBALS['tables_principales'][$table_sql])?'oui':false);
+	}
 
 	// normaliser pour pouvoir tester en php $infos['principale']?
 	// et dans une boucle {principale=oui}
@@ -608,67 +621,95 @@ 
 
 	// declarer et normaliser pour pouvoir tester en php $infos['editable']?
 	// et dans une boucle {editable=oui}
-	if (!isset($infos['editable'])) $infos['editable'] = 'oui';
+	if (!isset($infos['editable'])) {
+	    $infos['editable'] = 'oui';
+	}
 	$infos['editable'] = (($infos['editable'] AND $infos['editable']!='non')?'oui':false);
 
 	// les urls publiques sont par defaut page=type pour les tables principales, et rien pour les autres
 	// seules les exceptions sont donc a declarer
-	if (!isset($infos['page']))
-		$infos['page'] = ($infos['principale']?$infos['type']:'');
+	if (!isset($infos['page'])) {
+			$infos['page'] = ($infos['principale']?$infos['type']:'');
+	}
 
-	if (!isset($infos['url_voir']))
-		$infos['url_voir'] = $infos['type'];
-	if (!isset($infos['url_edit']))
-		$infos['url_edit'] = $infos['url_voir'].($infos['editable']?"_edit":'');
-	if (!isset($infos['icone_objet']))
-		$infos['icone_objet'] = $infos['type'];
+	if (!isset($infos['url_voir'])) {
+			$infos['url_voir'] = $infos['type'];
+	}
+	if (!isset($infos['url_edit'])) {
+			$infos['url_edit'] = $infos['url_voir'].($infos['editable']?"_edit":'');
+	}
+	if (!isset($infos['icone_objet'])) {
+			$infos['icone_objet'] = $infos['type'];
+	}
 
 	// chaines de langue
 	// par defaut : objet:icone_xxx_objet
-	if (!isset($infos['texte_retour']))
-		$infos['texte_retour'] = 'icone_retour';
-	if (!isset($infos['texte_modifier']))
-		$infos['texte_modifier'] = $infos['type'].':'.'icone_modifier_'.$infos['type'];
-	if (!isset($infos['texte_creer']))
-		$infos['texte_creer'] = $infos['type'].':'.'icone_creer_'.$infos['type'];
-	if (!isset($infos['texte_creer_associer']))
-		$infos['texte_creer_associer'] = $infos['type'].':'.'texte_creer_associer_'.$infos['type'];
-	if (!isset($infos['texte_ajouter'])) // Ajouter un X
+	if (!isset($infos['texte_retour'])) {
+			$infos['texte_retour'] = 'icone_retour';
+	}
+	if (!isset($infos['texte_modifier'])) {
+			$infos['texte_modifier'] = $infos['type'].':'.'icone_modifier_'.$infos['type'];
+	}
+	if (!isset($infos['texte_creer'])) {
+			$infos['texte_creer'] = $infos['type'].':'.'icone_creer_'.$infos['type'];
+	}
+	if (!isset($infos['texte_creer_associer'])) {
+			$infos['texte_creer_associer'] = $infos['type'].':'.'texte_creer_associer_'.$infos['type'];
+	}
+	if (!isset($infos['texte_ajouter'])) {
+	    // Ajouter un X
 		$infos['texte_ajouter'] = $infos['type'].':'.'texte_ajouter_'.$infos['type'];
-	if (!isset($infos['texte_objets']))
-		$infos['texte_objets'] = $infos['type'].':'.'titre_'.$infos['table_objet'];
-	if (!isset($infos['texte_objet']))
-		$infos['texte_objet'] = $infos['type'].':'.'titre_'.$infos['type'];
-	if (!isset($infos['texte_logo_objet']))  // objet:titre_logo_objet "Logo de ce X"
+	}
+	if (!isset($infos['texte_objets'])) {
+			$infos['texte_objets'] = $infos['type'].':'.'titre_'.$infos['table_objet'];
+	}
+	if (!isset($infos['texte_objet'])) {
+			$infos['texte_objet'] = $infos['type'].':'.'titre_'.$infos['type'];
+	}
+	if (!isset($infos['texte_logo_objet'])) {
+	    // objet:titre_logo_objet "Logo de ce X"
 		$infos['texte_logo_objet'] = $infos['type'].':'.'titre_logo_'.$infos['type'];
-	if (!isset($infos['texte_langue_objet']))  // objet:texte_langue_objet "Langue de ce X"
+	}
+	if (!isset($infos['texte_langue_objet'])) {
+	    // objet:texte_langue_objet "Langue de ce X"
 		$infos['texte_langue_objet'] = $infos['type'].':'.'titre_langue_'.$infos['type'];
-	if (!isset($infos['texte_definir_comme_traduction_objet']))  // "Ce X est une traduction du X numéro :"
+	}
+	if (!isset($infos['texte_definir_comme_traduction_objet'])) {
+	    // "Ce X est une traduction du X numéro :"
 		$infos['texte_definir_comme_traduction_objet'] = $infos['type'].':'.'texte_definir_comme_traduction_'.$infos['type'];
+	}
 
 
 	// objet:info_aucun_objet
-	if (!isset($infos['info_aucun_objet']))
-		$infos['info_aucun_objet'] = $infos['type'].':'.'info_aucun_'.$infos['type'];
+	if (!isset($infos['info_aucun_objet'])) {
+			$infos['info_aucun_objet'] = $infos['type'].':'.'info_aucun_'.$infos['type'];
+	}
 	// objet:info_1_objet
-	if (!isset($infos['info_1_objet']))
-		$infos['info_1_objet'] = $infos['type'].':'.'info_1_'.$infos['type'];
+	if (!isset($infos['info_1_objet'])) {
+			$infos['info_1_objet'] = $infos['type'].':'.'info_1_'.$infos['type'];
+	}
 	// objet:info_nb_objets
-	if (!isset($infos['info_nb_objets']))
-		$infos['info_nb_objets'] = $infos['type'].':'.'info_nb_'.$infos['table_objet'];
+	if (!isset($infos['info_nb_objets'])) {
+			$infos['info_nb_objets'] = $infos['type'].':'.'info_nb_'.$infos['table_objet'];
+	}
 
 
-	if (!isset($infos['champs_editables']))
-		$infos['champs_editables'] = array();
-	if (!isset($infos['champs_versionnes']))
-		$infos['champs_versionnes'] = array();
-	if (!isset($infos['rechercher_champs']))
-		$infos['rechercher_champs'] = array();
-	if (!isset($infos['rechercher_jointures']))
-		$infos['rechercher_jointures'] = array();
+	if (!isset($infos['champs_editables'])) {
+			$infos['champs_editables'] = array();
+	}
+	if (!isset($infos['champs_versionnes'])) {
+			$infos['champs_versionnes'] = array();
+	}
+	if (!isset($infos['rechercher_champs'])) {
+			$infos['rechercher_champs'] = array();
+	}
+	if (!isset($infos['rechercher_jointures'])) {
+			$infos['rechercher_jointures'] = array();
+	}
 
-	if (!isset($infos['modeles']))
-		$infos['modeles'] = array($infos['type']);
+	if (!isset($infos['modeles'])) {
+			$infos['modeles'] = array($infos['type']);
+	}
 
 	return $infos;
 }
@@ -688,25 +729,29 @@ 
  */
 function renseigner_table_objet_interfaces($table_sql,&$infos){
 	if (!isset($infos['titre'])){
-		if (isset($infos['table_objet']) AND isset($GLOBALS['table_titre'][$infos['table_objet']]))
-			$infos['titre'] = $GLOBALS['table_titre'][$infos['table_objet']];
-		else {
+		if (isset($infos['table_objet']) AND isset($GLOBALS['table_titre'][$infos['table_objet']])) {
+					$infos['titre'] = $GLOBALS['table_titre'][$infos['table_objet']];
+		} else {
 			$infos['titre'] = ((isset($infos['field']['titre']))?"titre,":"'' as titre,");
 			$infos['titre'] .= ((isset($infos['field']['lang']))?"lang":"'' as lang");
 		}
 	}
 	if (!isset($infos['date'])){
-		if (isset($infos['table_objet']) and isset($GLOBALS['table_date'][$infos['table_objet']]))
-			$infos['date'] = $GLOBALS['table_date'][$infos['table_objet']];
-		else
-			$infos['date'] = ((isset($infos['field']['date']))?"date":'');
+		if (isset($infos['table_objet']) and isset($GLOBALS['table_date'][$infos['table_objet']])) {
+					$infos['date'] = $GLOBALS['table_date'][$infos['table_objet']];
+		} else {
+					$infos['date'] = ((isset($infos['field']['date']))?"date":'');
+		}
+	}
+	if (!isset($infos['statut'])) {
+			$infos['statut'] = isset($GLOBALS['table_statut'][$table_sql]) ? $GLOBALS['table_statut'][$table_sql] : '';
+	}
+	if (!isset($infos['tables_jointures'])) {
+			$infos['tables_jointures'] = array();
+	}
+	if (isset($GLOBALS['tables_jointures'][$table_sql])) {
+			$infos['tables_jointures'] = array_merge($infos['tables_jointures'],$GLOBALS['tables_jointures'][$table_sql]);
 	}
-	if (!isset($infos['statut']))
-		$infos['statut'] = isset($GLOBALS['table_statut'][$table_sql]) ? $GLOBALS['table_statut'][$table_sql] : '';
-	if (!isset($infos['tables_jointures']))
-		$infos['tables_jointures'] = array();
-	if (isset($GLOBALS['tables_jointures'][$table_sql]))
-		$infos['tables_jointures'] = array_merge($infos['tables_jointures'],$GLOBALS['tables_jointures'][$table_sql]);
 	return $infos;
 }
 
@@ -771,9 +816,10 @@ 
 				//$surnoms[$infos['type']] = $infos['table_objet'];
 				$surnoms[preg_replace(',^spip_|^id_|s$,', '', $infos['table_objet'])] = $infos['table_objet'];
 				$surnoms[preg_replace(',^spip_|^id_|s$,', '', $infos['type'])] = $infos['table_objet'];
-				if (is_array($infos['table_objet_surnoms']) AND count($infos['table_objet_surnoms']))
-					foreach($infos['table_objet_surnoms'] as $surnom)
+				if (is_array($infos['table_objet_surnoms']) AND count($infos['table_objet_surnoms'])) {
+									foreach($infos['table_objet_surnoms'] as $surnom)
 						$surnoms[$surnom] = $infos['table_objet'];
+				}
 			}
 		}
 		$md5 = lister_tables_objets_sql('::md5');
@@ -802,9 +848,10 @@ 
 				$surnoms[preg_replace(',^spip_|^id_|s$,', '', $infos['table_objet'])] = $infos['type'];
 				$surnoms[preg_replace(',^spip_|^id_|s$,', '', $infos['type'])] = $infos['type'];
 				// surnoms declares
-				if (is_array($infos['type_surnoms']) AND count($infos['type_surnoms']))
-					foreach($infos['type_surnoms'] as $surnom)
+				if (is_array($infos['type_surnoms']) AND count($infos['type_surnoms'])) {
+									foreach($infos['type_surnoms'] as $surnom)
 						$surnoms[$surnom] = $infos['type'];
+				}
 			}
 		}
 		$md5 = lister_tables_objets_sql('::md5');
@@ -816,8 +863,9 @@ 
 	static $tables = array();
 	if (!isset($tables[$serveur])){
 		$tables[$serveur] = array();
-		if (!function_exists("sql_alltable"))
-			include_spip("base/abstract_sql");
+		if (!function_exists("sql_alltable")) {
+					include_spip("base/abstract_sql");
+		}
 		$ts = sql_alltable(null,$serveur); // toutes les tables "spip_" (ou prefixe perso)
 		$connexion = $GLOBALS['connexions'][$serveur ? $serveur : 0];
 		$spip = $connexion['prefixe'] . '_';
@@ -833,8 +881,9 @@ 
 	static $tables = array();
 	if (!isset($tables[$serveur])){
 		$tables[$serveur] = array();
-		if (!function_exists("sql_alltable"))
-			include_spip("base/abstract_sql");
+		if (!function_exists("sql_alltable")) {
+					include_spip("base/abstract_sql");
+		}
 		$ts = sql_alltable('%',$serveur); // toutes les tables
 		foreach ($ts as $t){
 			$tables[$serveur][$t] = $t;
@@ -862,9 +911,12 @@ 
 function table_objet($type,$serveur='') {
 	$surnoms = lister_tables_objets_surnoms();
 	$type = preg_replace(',^spip_|^id_|s$,', '', $type);
-	if (!$type) return;
-	if (isset($surnoms[$type]))
-		return $surnoms[$type];
+	if (!$type) {
+	    return;
+	}
+	if (isset($surnoms[$type])) {
+			return $surnoms[$type];
+	}
 
 	if ($serveur!==false){
 		$t=lister_tables_spip($serveur);
@@ -873,13 +925,14 @@ 
 		if (
 		  (isset($t[$typetrim]) OR in_array($typetrim,$t))
 		  AND ($desc = $trouver_table(rtrim($type,'s')."s",$serveur))
-		  )
-			return $desc['id_table'];
-		elseif (
+		  ) {
+					return $desc['id_table'];
+		} elseif (
 			(isset($t[$type]) OR in_array($type,$t))
 			AND ($desc = $trouver_table($type,$serveur))
-		  )
-			return $desc['id_table'];
+		  ) {
+					return $desc['id_table'];
+		}
 
 		spip_log( 'table_objet('.$type.') calculee sans verification');
 	}
@@ -906,22 +959,24 @@ 
 function table_objet_sql($type,$serveur='') {
 	global $table_des_tables;
 	$nom = table_objet($type, $serveur);
-	if (!isset($table_des_tables['articles'])) // eviter de multiples inclusions
+	if (!isset($table_des_tables['articles'])) {
+	    // eviter de multiples inclusions
 		include_spip('public/interfaces');
+	}
 	if (isset($table_des_tables[$nom])) {
 		$nom = $table_des_tables[$nom];
 		$nom = "spip_$nom";
-	}
-	else {
+	} else {
 		$infos_tables = lister_tables_objets_sql();
-		if (isset($infos_tables["spip_$nom"]))
-			$nom = "spip_$nom";
-		elseif($serveur!==false) {
+		if (isset($infos_tables["spip_$nom"])) {
+					$nom = "spip_$nom";
+		} elseif($serveur!==false) {
 			$t=lister_tables_spip($serveur);
 			if (isset($t[$nom]) OR in_array($nom,$t)){
 				$trouver_table = charger_fonction('trouver_table', 'base');
-				if ($desc = $trouver_table($nom,$serveur))
-					return $desc['table_sql'];
+				if ($desc = $trouver_table($nom,$serveur)) {
+									return $desc['table_sql'];
+				}
 			}
 		}
 	}
@@ -947,19 +1002,24 @@ 
 function id_table_objet($type,$serveur='') {
 	static $trouver_table = null;
 	$type = objet_type($type,$serveur);
-	if (!$type) return;
+	if (!$type) {
+	    return;
+	}
 	$t = table_objet($type);
-	if (!$trouver_table)
-		$trouver_table = charger_fonction('trouver_table', 'base');
+	if (!$trouver_table) {
+			$trouver_table = charger_fonction('trouver_table', 'base');
+	}
 
 	$ts=lister_tables_spip($serveur);
 	if (in_array($t,$ts)
 	  OR in_array($t,lister_toutes_tables($serveur))){
 		$desc = $trouver_table($t,$serveur);
-		if (isset($desc['key']['PRIMARY KEY']))
-			return $desc['key']['PRIMARY KEY'];
-		if (!$desc OR isset($desc['field']["id_$type"]))
-			return "id_$type";
+		if (isset($desc['key']['PRIMARY KEY'])) {
+					return $desc['key']['PRIMARY KEY'];
+		}
+		if (!$desc OR isset($desc['field']["id_$type"])) {
+					return "id_$type";
+		}
 		// sinon renvoyer le premier champ de la table...
 		$keys = array_keys($desc['field']);
 		return array_shift($keys);
@@ -983,7 +1043,9 @@ 
  *     Type de l'objet
 **/
 function objet_type($table_objet, $serveur=''){
-	if (!$table_objet) return;
+	if (!$table_objet) {
+	    return;
+	}
 	$surnoms = lister_types_surnoms();
 
 	// scenario de base
@@ -991,8 +1053,9 @@ 
 	// et la marque du pluriel
 	// on accepte id_xx en entree aussi
 	$type = preg_replace(',^spip_|^id_|s$,', '', $table_objet);
-	if (isset($surnoms[$type]))
-		return $surnoms[$type];
+	if (isset($surnoms[$type])) {
+			return $surnoms[$type];
+	}
 
 	// securite : eliminer les caracteres non \w
 	$type = preg_replace(',[^\w-],','',$type);
@@ -1001,12 +1064,14 @@ 
 	// oui si table_objet ressemblait deja a un type
 	if ( $type==$table_objet
 		OR (table_objet($type,$serveur)==$table_objet)
-	  OR (table_objet_sql($type,$serveur)==$table_objet))
-	  return $type;
+	  OR (table_objet_sql($type,$serveur)==$table_objet)) {
+		  return $type;
+	}
 
 	// si on ne veut pas chercher en base
-	if ($serveur===false)
-		return $type;
+	if ($serveur===false) {
+			return $type;
+	}
 
 	// sinon on passe par la cle primaire id_xx pour trouver le type
 	// car le s a la fin est incertain
@@ -1016,13 +1081,16 @@ 
 	// dans le pipeline declarer_tables_objets_surnoms
 	$trouver_table = charger_fonction('trouver_table', 'base');
 	$ts=lister_tables_spip($serveur);
-	if (in_array($table_objet,$ts))
-		$desc = $trouver_table($table_objet);
-	if (!$desc AND in_array($table_objet=table_objet($type,$serveur),$ts))
-		$desc = $trouver_table($table_objet,$serveur);
+	if (in_array($table_objet,$ts)) {
+			$desc = $trouver_table($table_objet);
+	}
+	if (!$desc AND in_array($table_objet=table_objet($type,$serveur),$ts)) {
+			$desc = $trouver_table($table_objet,$serveur);
+	}
 	// si le type est declare : bingo !
-	if ($desc AND isset($desc['type']))
-			return $desc['type'];
+	if ($desc AND isset($desc['type'])) {
+				return $desc['type'];
+	}
 
 	// on a fait ce qu'on a pu
 	return $type;
@@ -1043,8 +1111,9 @@ 
 function objet_test_si_publie($objet,$id_objet, $serveur=''){
 	// voir si une fonction est definie pour faire le boulot
 	// elle a la priorite dans ce cas
-	if ($f = charger_fonction($objet."_test_si_publie","base",true))
-		return $f($objet,$id_objet, $serveur);
+	if ($f = charger_fonction($objet."_test_si_publie","base",true)) {
+			return $f($objet,$id_objet, $serveur);
+	}
 
 	// sinon on se fie a la declaration de l'objet si presente
 	$id_table = $table_objet = table_objet($objet);
@@ -1067,8 +1136,9 @@ 
 		include_spip('public/composer');
 		instituer_boucle($boucle, false, true);
 		$res = calculer_select($boucle->select,$boucle->from,$boucle->from_type,$boucle->where,$boucle->join,$boucle->group,$boucle->order,$boucle->limit,$boucle->having,$table_objet,$id_table,$serveur);
-		if (sql_fetch($res))
-			return true;
+		if (sql_fetch($res)) {
+					return true;
+		}
 		return false;
 	}