spip / SPIP

ecrire/auth/sha256.inc.php

Doc Comments +33 added lines

@@ -135,15 +135,41 @@ 
             return (int)$x >> (int)$n;
         }
 
+        /**
+         * @param integer $n
+         */
         function ROTR($x, $n) { return (int)(($this->SHR($x, $n) | ($x << (32-$n)) & 0xFFFFFFFF)); }
+
+        /**
+         * @param integer $x
+         * @param integer $y
+         * @param integer $z
+         */
         function Ch($x, $y, $z) { return ($x & $y) ^ ((~$x) & $z); }
+
+        /**
+         * @param integer $x
+         * @param integer $y
+         * @param integer $z
+         */
         function Maj($x, $y, $z) { return ($x & $y) ^ ($x & $z) ^ ($y & $z); }
+
+        /**
+         * @param integer $x
+         */
         function Sigma0($x) { return (int) ($this->ROTR($x, 2)^$this->ROTR($x, 13)^$this->ROTR($x, 22)); }
+
+        /**
+         * @param integer $x
+         */
         function Sigma1($x) { return (int) ($this->ROTR($x, 6)^$this->ROTR($x, 11)^$this->ROTR($x, 25)); }
         function sigma_0($x) { return (int) ($this->ROTR($x, 7)^$this->ROTR($x, 18)^$this->SHR($x, 3)); }
         function sigma_1($x) { return (int) ($this->ROTR($x, 17)^$this->ROTR($x, 19)^$this->SHR($x, 10)); }
 
 
+				/**
+				 * @param integer $byteSize
+				 */
 				function string2ordUTF8($s,&$byteSize){
 					$chars = array();
 					// par defaut sur 8bits
@@ -219,6 +245,9 @@ 
 					return $bin;
 				}
 
+				/**
+				 * @param integer $n
+				 */
 				function array_split($a, $n) {
 					$split = array();
 					while (count($a)>$n) {
@@ -434,6 +463,10 @@ 
 if (!function_exists('hash'))
 {
     define('_NO_HASH_DEFINED',true);
+
+    /**
+     * @param string $algo
+     */
     function hash($algo, $data)
     {
         if (empty($algo) || !is_string($algo) || !is_string($data)) {

Braces +14 added lines, -15 removed lines

@@ -155,7 +155,9 @@ 
 						// mais si un char necessite 16bits, on passe tout sur 16
 						// sinon on ne concorde pas avec le lecture de la chaine en js
 						// et le sha256 js
-						if ($bytes>1) $byteSize = 16;
+						if ($bytes>1) {
+						    $byteSize = 16;
+						}
 					}
 					return $chars;
 				}
@@ -165,36 +167,32 @@ 
 					$len = strlen($c);
 					$bytes = 0;
 
-					if ($index >= $len)
-						return false;
+					if ($index >= $len) {
+											return false;
+					}
 
 					$h = ord($c{$index});
 
 					if ($h <= 0x7F) {
 						$bytes = 1;
 						return $h;
-					}
-					else if ($h < 0xC2){
+					} else if ($h < 0xC2){
 						// pas utf mais renvoyer quand meme ce qu'on a
 						$bytes = 1;
 						return $h;
-					}
-					else if ($h <= 0xDF && $index < $len - 1) {
+					} else if ($h <= 0xDF && $index < $len - 1) {
 						$bytes = 2;
 						return ($h & 0x1F) <<  6 | (ord($c{$index + 1}) & 0x3F);
-					}
-					else if ($h <= 0xEF && $index < $len - 2) {
+					} else if ($h <= 0xEF && $index < $len - 2) {
 						$bytes = 3;
 						return ($h & 0x0F) << 12 | (ord($c{$index + 1}) & 0x3F) << 6
 																		 | (ord($c{$index + 2}) & 0x3F);
-					}
-					else if ($h <= 0xF4 && $index < $len - 3) {
+					} else if ($h <= 0xF4 && $index < $len - 3) {
 						$bytes = 4;
 						return ($h & 0x0F) << 18 | (ord($c{$index + 1}) & 0x3F) << 12
 																		 | (ord($c{$index + 2}) & 0x3F) << 6
 																		 | (ord($c{$index + 3}) & 0x3F);
-					}
-					else {
+					} else {
 						// pas utf mais renvoyer quand meme ce qu'on a
 						$bytes = 1;
 						return $h;
@@ -222,8 +220,9 @@ 
 					$split = array();
 					while (count($a)>$n) {
 						$s = array();
-						for($i = 0;$i<$n;$i++)
-							$s[] = array_shift($a);
+						for($i = 0;$i<$n;$i++) {
+													$s[] = array_shift($a);
+						}
 						$split[] = $s;
 					}
 					if (count($a)){

ecrire/balise/logo_.php

Doc Comments +7 added lines

@@ -96,6 +96,13 @@
 	return $p;
 }
 
+/**
+ * @param string $id_objet
+ * @param string $_id_objet
+ * @param string $type
+ * @param integer $fichier
+ * @param string $suite
+ */
 function logo_survol($id_objet, $_id_objet, $type, $align, $fichier, $lien, $p, $suite)
 {
 	$code = "quete_logo('$id_objet', '" .

Braces +30 added lines, -19 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 //
 // Fonction des balises #LOGO_XXXX
@@ -31,8 +33,9 @@ 
 	}
 
 	$id_objet = id_table_objet($type);
-	if (!isset($_id_objet) OR !$_id_objet)
-		$_id_objet = champ_sql($id_objet, $p);
+	if (!isset($_id_objet) OR !$_id_objet) {
+			$_id_objet = champ_sql($id_objet, $p);
+	}
 
 	$fichier = ($p->etoile === '**') ? -1 : 0;
 	$coord = array();
@@ -45,14 +48,16 @@ 
 		foreach($params as $a) {
 			if ($a[0]->type === 'texte') {
 				$n = $a[0]->texte;
-				if (is_numeric($n))
-					$coord[]= $n;
-				elseif (in_array($n,array('top','left','right','center','bottom')))
-					$align = $n;
-				elseif (in_array($n,array('auto','icone','apercu','vignette')))
-					$mode_logo = $n;
+				if (is_numeric($n)) {
+									$coord[]= $n;
+				} elseif (in_array($n,array('top','left','right','center','bottom'))) {
+									$align = $n;
+				} elseif (in_array($n,array('auto','icone','apercu','vignette'))) {
+									$mode_logo = $n;
+				}
+			} else {
+			    $lien =  calculer_liste($a, $p->descr, $p->boucles, $p->id_boucle);
 			}
-			else $lien =  calculer_liste($a, $p->descr, $p->boucles, $p->id_boucle);
 
 		}
 	}
@@ -69,16 +74,18 @@ 
 	if ($type == 'document') {
 		$qconnect = _q($connect);
 		$doc = "quete_document($_id_objet, $qconnect)";
-		if ($fichier)
-			$code = "quete_logo_file($doc, $qconnect)";
-		else $code = "quete_logo_document($doc, " . ($lien ? $lien : "''") . ", '$align', '$mode_logo', $coord_x, $coord_y, $qconnect)";
+		if ($fichier) {
+					$code = "quete_logo_file($doc, $qconnect)";
+		} else {
+		    $code = "quete_logo_document($doc, " . ($lien ? $lien : "''") . ", '$align', '$mode_logo', $coord_x, $coord_y, $qconnect)";
+		}
 		// (x=non-faux ? y : '') pour affecter x en retournant y
-		if ($p->descr['documents'])
-		  $code = '(($doublons["documents"] .= ",". '
+		if ($p->descr['documents']) {
+				  $code = '(($doublons["documents"] .= ",". '
 		    . $_id_objet
 		    . ") ? $code : '')";
-	}
-	elseif ($connect) {
+		}
+	} elseif ($connect) {
 		$code = "''";
 		spip_log("Les logos distants ne sont pas prevus");
 	} else {
@@ -106,14 +113,18 @@ 
 		(($type == 'rubrique') ? "quete_parent($_id_objet)" : "''")) .
 		", " . intval($fichier) . ")";
 
-	if ($fichier) return $code;
+	if ($fichier) {
+	    return $code;
+	}
 
 	$code = "\n((!is_array(\$l = $code)) ? '':\n (" .
 		     '"<img class=\"spip_logos\" alt=\"\"' .
 		    ($align ? " align=\\\"$align\\\"" : '')
 		    . ' src=\"$l[0]\"" . $l[2] .  ($l[1] ? " onmouseover=\"this.src=\'$l[1]\'\" onmouseout=\"this.src=\'$l[0]\'\"" : "") . \' />\'))';
 
-	if (!$lien) return $code;
+	if (!$lien) {
+	    return $code;
+	}
 
 	return ('(strlen($logo='.$code.')?\'<a href="\' .' . $lien . ' . \'">\' . $logo . \'</a>\':\'\')');
 

ecrire/balise/menu_lang_ecrire.php

Doc Comments +3 added lines

@@ -37,6 +37,9 @@
 }
 
 // http://doc.spip.org/@menu_lang_pour_tous
+/**
+ * @param string $nom
+ */
 function menu_lang_pour_tous($nom, $default) {
 	include_spip('inc/lang');
 

ecrire/base/upgrade.php

Doc Comments +3 added lines

@@ -231,6 +231,9 @@
 
 // pour versions <= 1.926
 // http://doc.spip.org/@upgrade_vers
+/**
+ * @param double $version
+ */
 function upgrade_vers($version, $version_installee, $version_cible = 0){
 	return ($version_installee<$version
 		AND (($version_cible>=$version) OR ($version_cible==0))

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/genie/mise_a_jour.php

Doc Comments +9 added lines

@@ -34,6 +34,10 @@ 
 define('_VERSIONS_SERVEUR', 'http://files.spip.org/');
 define('_VERSIONS_LISTE', 'archives.xml');
 
+/**
+ * @param string $dir
+ * @param string $file
+ */
 function info_maj ($dir, $file, $version){
 	include_spip('inc/plugin');
 	
@@ -67,6 +71,11 @@ 
 // on teste la nouveaute par If-Modified-Since,
 // et seulement quand celui-ci a change' pour limiter les acces HTTP
 
+/**
+ * @param string $nom
+ *
+ * @return string
+ */
 function info_maj_cache($nom, $dir, $page='')
 {
 	$re = '<archives id="a' . $GLOBALS['meta']["alea_ephemere"] . '">';

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/inc/filtres_images_mini.php

Doc Comments +3 added lines

@@ -14,6 +14,9 @@
 include_spip('inc/filtres_images_lib_mini'); // par precaution
 
 // http://doc.spip.org/@couleur_html_to_hex
+/**
+ * @param string $couleur
+ */
 function couleur_html_to_hex($couleur){
 	$couleurs_html=array(
 		'aqua'=>'00FFFF','black'=>'000000','blue'=>'0000FF','fuchsia'=>'FF00FF','gray'=>'808080','green'=>'008000','lime'=>'00FF00','maroon'=>'800000',

Braces +56 added lines, -31 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/filtres_images_lib_mini'); // par precaution
 
 // http://doc.spip.org/@couleur_html_to_hex
@@ -18,8 +20,9 @@ 
 	$couleurs_html=array(
 		'aqua'=>'00FFFF','black'=>'000000','blue'=>'0000FF','fuchsia'=>'FF00FF','gray'=>'808080','green'=>'008000','lime'=>'00FF00','maroon'=>'800000',
 		'navy'=>'000080','olive'=>'808000','purple'=>'800080','red'=>'FF0000','silver'=>'C0C0C0','teal'=>'008080','white'=>'FFFFFF','yellow'=>'FFFF00');
-	if (isset($couleurs_html[$lc=strtolower($couleur)]))
-		return $couleurs_html[$lc];
+	if (isset($couleurs_html[$lc=strtolower($couleur)])) {
+			return $couleurs_html[$lc];
+	}
 	return $couleur;
 }
 
@@ -55,11 +58,14 @@ 
 // dans la fonction image_filtrer
 // http://doc.spip.org/@image_select
 function image_select($img,$width_min=0, $height_min=0, $width_max=10000, $height_max=1000){
-	if (!$img) return $img;
+	if (!$img) {
+	    return $img;
+	}
 	list ($h,$l) = taille_image($img);
 	$select = true;
-	if ($l<$width_min OR $l>$width_max OR $h<$height_min OR $h>$height_max)
-		$select = false;
+	if ($l<$width_min OR $l>$width_max OR $h<$height_min OR $h>$height_max) {
+			$select = false;
+	}
 
 	$class = extraire_attribut($img,'class');
 	$p = strpos($class,'no_image_filtrer');
@@ -77,19 +83,28 @@ 
 
 // http://doc.spip.org/@image_passe_partout
 function image_passe_partout($img,$taille_x = -1, $taille_y = -1,$force = false,$cherche_image=false,$process='AUTO'){
-	if (!$img) return '';
+	if (!$img) {
+	    return '';
+	}
 	list ($hauteur,$largeur) = taille_image($img);
-	if ($taille_x == -1)
-		$taille_x = isset($GLOBALS['meta']['taille_preview'])?$GLOBALS['meta']['taille_preview']:150;
-	if ($taille_y == -1)
-		$taille_y = $taille_x;
-
-	if ($taille_x == 0 AND $taille_y > 0)
-		$taille_x = 1; # {0,300} -> c'est 300 qui compte
-	elseif ($taille_x > 0 AND $taille_y == 0)
-		$taille_y = 1; # {300,0} -> c'est 300 qui compte
-	elseif ($taille_x == 0 AND $taille_y == 0)
-		return '';
+	if ($taille_x == -1) {
+			$taille_x = isset($GLOBALS['meta']['taille_preview'])?$GLOBALS['meta']['taille_preview']:150;
+	}
+	if ($taille_y == -1) {
+			$taille_y = $taille_x;
+	}
+
+	if ($taille_x == 0 AND $taille_y > 0) {
+			$taille_x = 1;
+	}
+	# {0,300} -> c'est 300 qui compte
+	elseif ($taille_x > 0 AND $taille_y == 0) {
+			$taille_y = 1;
+	}
+	# {300,0} -> c'est 300 qui compte
+	elseif ($taille_x == 0 AND $taille_y == 0) {
+			return '';
+	}
 	
 	list($destWidth,$destHeight,$ratio) = ratio_passe_partout($largeur,$hauteur,$taille_x,$taille_y);
 	$fonction = array('image_passe_partout', func_get_args());
@@ -100,17 +115,24 @@ 
 function image_reduire($img, $taille = -1, $taille_y = -1, $force=false, $cherche_image=false, $process='AUTO') {
 	// Determiner la taille x,y maxi
 	// prendre le reglage de previsu par defaut
-	if ($taille == -1)
-		$taille = (isset($GLOBALS['meta']['taille_preview']) AND intval($GLOBALS['meta']['taille_preview']))?intval($GLOBALS['meta']['taille_preview']):150;
-	if ($taille_y == -1)
-		$taille_y = $taille;
-
-	if ($taille == 0 AND $taille_y > 0)
-		$taille = 10000; # {0,300} -> c'est 300 qui compte
-	elseif ($taille > 0 AND $taille_y == 0)
-		$taille_y = 10000; # {300,0} -> c'est 300 qui compte
-	elseif ($taille == 0 AND $taille_y == 0)
-		return '';
+	if ($taille == -1) {
+			$taille = (isset($GLOBALS['meta']['taille_preview']) AND intval($GLOBALS['meta']['taille_preview']))?intval($GLOBALS['meta']['taille_preview']):150;
+	}
+	if ($taille_y == -1) {
+			$taille_y = $taille;
+	}
+
+	if ($taille == 0 AND $taille_y > 0) {
+			$taille = 10000;
+	}
+	# {0,300} -> c'est 300 qui compte
+	elseif ($taille > 0 AND $taille_y == 0) {
+			$taille_y = 10000;
+	}
+	# {300,0} -> c'est 300 qui compte
+	elseif ($taille == 0 AND $taille_y == 0) {
+			return '';
+	}
 
 	$fonction = array('image_reduire', func_get_args());
 	return process_image_reduire($fonction,$img,$taille,$taille_y,$force,$cherche_image,$process);
@@ -124,8 +146,11 @@ 
 	$l = round($largeur/$val);
 	$h = round($hauteur/$val);
 	
-	if ($l > $h) $h = 0;
-	else $l = 0;
+	if ($l > $h) {
+	    $h = 0;
+	} else {
+	    $l = 0;
+	}
 	
 	$img = image_reduire($img, $l, $h, $force);
 

ecrire/inc/headers.php

Doc Comments +3 added lines

@@ -125,6 +125,9 @@
 }
 
 // http://doc.spip.org/@http_status
+/**
+ * @param integer $status
+ */
 function http_status($status) {
 	global $REDIRECT_STATUS, $flag_sapi_name;
 	static $status_string = array(

Braces +42 added lines, -28 removed lines

@@ -11,23 +11,28 @@ 
 \***************************************************************************/
 
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // envoyer le navigateur sur une nouvelle adresse
 // en evitant les attaques par la redirection (souvent indique par 1 $_GET)
 
 // http://doc.spip.org/@redirige_par_entete
 function redirige_par_entete($url, $equiv='', $status = 302) {
-	if (!in_array($status,array(301,302)))
-		$status = 302;
+	if (!in_array($status,array(301,302))) {
+			$status = 302;
+	}
 
 	$url = trim(strtr($url, "\n\r", "  "));
 	# en theorie on devrait faire ca tout le temps, mais quand la chaine
 	# commence par ? c'est imperatif, sinon l'url finale n'est pas la bonne
-	if ($url[0]=='?')
-		$url = url_de_base().$url;
-	if ($url[0]=='#')
-		$url = self('&').$url;
+	if ($url[0]=='?') {
+			$url = url_de_base().$url;
+	}
+	if ($url[0]=='#') {
+			$url = self('&').$url;
+	}
 	# si profondeur non nulle et url relative, il faut la passer en absolue
 	if ($GLOBALS['profondeur_url']>(_DIR_RESTREINT?1:2)
 		AND !preg_match(",^(\w+:)?//,",$url)){
@@ -35,11 +40,13 @@ 
 		$url = url_absolue($url);
 	}
 
-	if ($x = _request('transformer_xml'))
-		$url = parametre_url($url, 'transformer_xml', $x, '&');
+	if ($x = _request('transformer_xml')) {
+			$url = parametre_url($url, 'transformer_xml', $x, '&');
+	}
 
-	if (defined('_AJAX') AND _AJAX)
-		$url = parametre_url($url, 'var_ajax_redir', 1, '&');
+	if (defined('_AJAX') AND _AJAX) {
+			$url = parametre_url($url, 'var_ajax_redir', 1, '&');
+	}
 		
 	// ne pas laisser passer n'importe quoi dans l'url
 	$url = str_replace(array('<','"'),array('&lt;','&quot;'),$url);
@@ -47,8 +54,9 @@ 
 	if (
 		(preg_match(",data:,i",$url) AND preg_match("/base64\s*,/i",$url))
 		OR preg_match(",(javascript|mailto):,i",$url)
-		)
-		$url ="./";
+		) {
+			$url ="./";
+	}
 
 	// Il n'y a que sous Apache que setcookie puis redirection fonctionne
   include_spip('inc/cookie');
@@ -60,8 +68,9 @@ 
 		$equiv = "<meta http-equiv='Refresh' content='0; url=$url'>";
 	}
 	include_spip('inc/lang');
-	if ($status!=302)
-		http_status($status);
+	if ($status!=302) {
+			http_status($status);
+	}
 	echo '<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">',"\n",
 	  html_lang_attributes(),'
 <head>',
@@ -95,33 +104,35 @@ 
 		"<a href='$url' name='ajax_ancre' style='display:none;'>anchor</a>",
 		// et rien dans le message ok
 		'');
-	}
-	else {
+	} else {
 		// ne pas laisser passer n'importe quoi dans l'url
 		$url = str_replace(array('<','"'),array('&lt;','&quot;'),$url);
 
 		$url = strtr($url, "\n\r", "  ");
 		# en theorie on devrait faire ca tout le temps, mais quand la chaine
 		# commence par ? c'est imperatif, sinon l'url finale n'est pas la bonne
-		if ($url[0]=='?')
-			$url = url_de_base().$url;
+		if ($url[0]=='?') {
+					$url = url_de_base().$url;
+		}
 		$url = str_replace('&amp;','&',$url);
 		spip_log("redirige formulaire ajax: $url");
 		include_spip('inc/filtres');
-		if ($format=='ajaxform')
-			return array(
+		if ($format=='ajaxform') {
+					return array(
 			// on renvoie un lien masque qui sera traite par ajaxCallback.js
 			'<a href="'.quote_amp($url).'" name="ajax_redirect"  style="display:none;">'._T('navigateur_pas_redirige').'</a>',
 			// et un message au cas ou
 			'<br /><a href="'.quote_amp($url).'">'._T('navigateur_pas_redirige').'</a>'
 			);
-		else // format message texte, tout en js inline
+		} else {
+		    // format message texte, tout en js inline
 			return
 		// ie poste les formulaires dans une iframe, il faut donc rediriger son parent
 		"<script type='text/javascript'>if (parent.window){parent.window.document.location.replace(\"$url\");} else {document.location.replace(\"$url\");}</script>"
 		. http_img_pack('searching.gif','')
 		. '<br />'
 		. '<a href="'.quote_amp($url).'">'._T('navigateur_pas_redirige').'</a>';
+		}
 	}
 }
 
@@ -145,14 +156,17 @@ 
 		503 => '503 Service Unavailable'
 	);
 
-	if ($REDIRECT_STATUS && $REDIRECT_STATUS == $status) return;
+	if ($REDIRECT_STATUS && $REDIRECT_STATUS == $status) {
+	    return;
+	}
 
 	$php_cgi = ($flag_sapi_name AND preg_match(",cgi,i", @php_sapi_name()));
-	if ($php_cgi)
-		header("Status: ".$status_string[$status]);
-	else
-		header("HTTP/1.0 ".$status_string[$status]);
-}
+	if ($php_cgi) {
+			header("Status: ".$status_string[$status]);
+	} else {
+			header("HTTP/1.0 ".$status_string[$status]);
+	}
+	}
 
 // Retourne ce qui va bien pour que le navigateur ne mette pas la page en cache
 // http://doc.spip.org/@http_no_cache

ecrire/inc/meta.php

Doc Comments +3 added lines

@@ -107,6 +107,9 @@
 }
 
 // http://doc.spip.org/@ecrire_meta
+/**
+ * @param string $importable
+ */
 function ecrire_meta($nom, $valeur, $importable = NULL, $table='meta') {
 
 	static $touch = array();

Braces +44 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;
+}
 
 // Les parametres generaux du site sont dans une table SQL;
 // Recopie dans le tableau PHP global meta, car on en a souvent besoin
@@ -28,14 +30,18 @@ 
 	if ((_request('exec')!=='install' OR !test_espace_prive())
 	AND $new = jeune_fichier($cache, _META_CACHE_TIME)
 	AND lire_fichier_securise($cache, $meta)
-	AND $meta = @unserialize($meta))
-		$GLOBALS[$table] = $meta;
+	AND $meta = @unserialize($meta)) {
+			$GLOBALS[$table] = $meta;
+	}
 
 	if (isset($GLOBALS[$table]['touch']) 
-	AND ($GLOBALS[$table]['touch']<time()-_META_CACHE_TIME))
-		$GLOBALS[$table] = array();
+	AND ($GLOBALS[$table]['touch']<time()-_META_CACHE_TIME)) {
+			$GLOBALS[$table] = array();
+	}
 	// sinon lire en base
-	if (!$GLOBALS[$table]) $new = !lire_metas($table);
+	if (!$GLOBALS[$table]) {
+	    $new = !lire_metas($table);
+	}
 
 	// renouveller l'alea general si trop vieux ou sur demande explicite
 	if ((test_espace_prive() || isset($_GET['renouvelle_alea']))
@@ -47,11 +53,15 @@ 
 			include_spip('inc/acces');
 			renouvelle_alea();
 			$new = false; 
-		} else spip_log("impossible d'ecrire dans " . $cache);
+		} else {
+		    spip_log("impossible d'ecrire dans " . $cache);
+		}
 	}
 	// et refaire le cache si on a du lire en base
-	if (!$new) touch_meta(false, $table);
-}
+	if (!$new) {
+	    touch_meta(false, $table);
+	}
+	}
 
 // fonctions aussi appelees a l'install ==> spip_query en premiere requete 
 // pour eviter l'erreur fatale (serveur non encore configure)
@@ -62,20 +72,23 @@ 
 	if ($result = spip_query("SELECT nom,valeur FROM spip_$table")) {
 		include_spip('base/abstract_sql');
 		$GLOBALS[$table] = array();
-		while ($row = sql_fetch($result))
-			$GLOBALS[$table][$row['nom']] = $row['valeur'];
+		while ($row = sql_fetch($result)) {
+					$GLOBALS[$table][$row['nom']] = $row['valeur'];
+		}
         sql_free($result);
 
 		if (!$GLOBALS[$table]['charset']
 		  OR $GLOBALS[$table]['charset']=='_DEFAULT_CHARSET' // hum, correction d'un bug ayant abime quelques install
-		)
-			ecrire_meta('charset', _DEFAULT_CHARSET, NULL, $table);
+		) {
+					ecrire_meta('charset', _DEFAULT_CHARSET, NULL, $table);
+		}
 
 		// noter cette table de configuration dans les meta de SPIP
 		if ($table!=='meta') {
 			$liste = unserialize($GLOBALS['meta']['tables_config']);
-			if (!$liste)
-				$liste = array();
+			if (!$liste) {
+							$liste = array();
+			}
 			if (!in_array($table, $liste)) {
 				$liste[] = $table;
 				ecrire_meta('tables_config', serialize($liste));
@@ -98,7 +111,9 @@ 
 		// mais le sortir deu cache meta implique une requete sql des qu'on a un form dynamique
 		// meme si son squelette est en cache
 		//unset($r['secret_du_site']);
-		if ($antidate) $r['touch']= $antidate;
+		if ($antidate) {
+		    $r['touch']= $antidate;
+		}
 		ecrire_fichier_securise($file, serialize($r));
 	}
 }
@@ -121,7 +136,9 @@ 
 function ecrire_meta($nom, $valeur, $importable = NULL, $table='meta') {
 
 	static $touch = array();
-	if (!$nom) return;
+	if (!$nom) {
+	    return;
+	}
 	include_spip('base/abstract_sql');
 	$res = sql_select("*",'spip_' . $table,"nom=" . sql_quote($nom),'','','','','','continue');
 	// table pas encore installee, travailler en php seulement
@@ -134,7 +151,9 @@ 
 
 	// ne pas invalider le cache si affectation a l'identique
 	// (tant pis si impt aurait du changer)
-	if ($row AND $valeur == $row['valeur'] AND $GLOBALS[$table][$nom] == $valeur) return;
+	if ($row AND $valeur == $row['valeur'] AND $GLOBALS[$table][$nom] == $valeur) {
+	    return;
+	}
 
 	$GLOBALS[$table][$nom] = $valeur;
 	// cf effacer pour comprendre le double touch
@@ -142,7 +161,9 @@ 
 	if (!isset($touch[$table])) {touch_meta($antidate, $table);}
 	$r = array('nom' => $nom, 'valeur' => $valeur);
 	// Gaffe aux tables sans impt (vieilles versions de SPIP notamment)
-	if ($importable AND isset($row['impt'])) $r['impt'] = $importable;
+	if ($importable AND isset($row['impt'])) {
+	    $r['impt'] = $importable;
+	}
 	if ($row) {
 		sql_updateq('spip_' . $table, $r,"nom=" . sql_quote($nom));
 	} else {
@@ -179,7 +200,10 @@ 
  * @param bool $force
  */
 function supprimer_table_meta($table, $force=false) {
-	if ($table=='meta') return; // interdit !
+	if ($table=='meta') {
+	    return;
+	}
+	// interdit !
 
 	if ($force OR !sql_countsel("spip_$table")) {
 		unset($GLOBALS[$table]);

ecrire/inc/selectionner.php

Doc Comments +6 added lines

@@ -58,6 +58,12 @@
 }
 
 // http://doc.spip.org/@construire_selectionner_hierarchie
+/**
+ * @param string $idom
+ * @param string|boolean $racine
+ * @param string $url
+ * @param string $name
+ */
 function construire_selectionner_hierarchie($idom, $liste, $racine, $url, $name, $url_init='')
 {
 	global $spip_lang_right;

Braces +11 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;
+}
 
 include_spip('inc/filtres');
 
@@ -21,7 +23,11 @@ 
 // http://doc.spip.org/@inc_selectionner_dist
 function inc_selectionner_dist ($sel, $idom="", $exclus=0, $aff_racine=false, $recur=true, $do='aff') {
 
-	if ($recur) $recur = mini_hier($sel); else $sel = 0;
+	if ($recur) {
+	    $recur = mini_hier($sel);
+	} else {
+	    $sel = 0;
+	}
 
 	if ($aff_racine) {
 		$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
@@ -118,8 +124,9 @@ 
 	
 	$liste = $id_rubrique;
 	$id_rubrique = intval($id_rubrique);
-	while ($id_rubrique = sql_getfetsel("id_parent", "spip_rubriques", "id_rubrique = " . $id_rubrique))
-		$liste = $id_rubrique . ",$liste";
+	while ($id_rubrique = sql_getfetsel("id_parent", "spip_rubriques", "id_rubrique = " . $id_rubrique)) {
+			$liste = $id_rubrique . ",$liste";
+	}
 	return explode(',',"0,$liste");
 }