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

Spacing +80 added lines, -80 removed lines

@@ -83,7 +83,7 @@ 
                            : ((defined('_NANO_SHA2_UPPER')) ? true : false);
 
             // Deteremine if the system is 32 or 64 bit.
-            $tmpInt = (int)4294967295;
+            $tmpInt = (int) 4294967295;
             $this->platform = ($tmpInt > 0) ? 64 : 32;
         }
 
@@ -94,12 +94,12 @@ 
 
             if ($x < 0) {
                 $x &= 0x7FFFFFFF;
-                $x = (float)$x + $mask;
+                $x = (float) $x + $mask;
             }
 
             if ($y < 0) {
                 $y &= 0x7FFFFFFF;
-                $y = (float)$y + $mask;
+                $y = (float) $y + $mask;
             }
 
             $r = $x + $y;
@@ -110,52 +110,52 @@ 
                 }
             }
 
-            return (int)$r;
+            return (int) $r;
         }
 
         // Logical bitwise right shift (PHP default is arithmetic shift)
         function SHR($x, $n)        // x >> n
         {
             if ($n >= 32) {      // impose some limits to keep it 32-bit
-                return (int)0;
+                return (int) 0;
             }
 
             if ($n <= 0) {
-                return (int)$x;
+                return (int) $x;
             }
 
             $mask = 0x40000000;
 
             if ($x < 0) {
                 $x &= 0x7FFFFFFF;
-                $mask = $mask >> ($n-1);
+                $mask = $mask >> ($n - 1);
                 return ($x >> $n) | $mask;
             }
 
-            return (int)$x >> (int)$n;
+            return (int) $x >> (int) $n;
         }
 
-        function ROTR($x, $n) { return (int)(($this->SHR($x, $n) | ($x << (32-$n)) & 0xFFFFFFFF)); }
+        function ROTR($x, $n) { return (int) (($this->SHR($x, $n) | ($x << (32 - $n)) & 0xFFFFFFFF)); }
         function Ch($x, $y, $z) { return ($x & $y) ^ ((~$x) & $z); }
         function Maj($x, $y, $z) { return ($x & $y) ^ ($x & $z) ^ ($y & $z); }
-        function Sigma0($x) { return (int) ($this->ROTR($x, 2)^$this->ROTR($x, 13)^$this->ROTR($x, 22)); }
-        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)); }
+        function Sigma0($x) { return (int) ($this->ROTR($x, 2) ^ $this->ROTR($x, 13) ^ $this->ROTR($x, 22)); }
+        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)); }
 
 
-				function string2ordUTF8($s,&$byteSize){
+				function string2ordUTF8($s, &$byteSize) {
 					$chars = array();
 					// par defaut sur 8bits
 					$byteSize = 8;
 					$i = 0;
-					while ($i<strlen($s)){
+					while ($i < strlen($s)) {
 						$chars[] = $this->ordUTF8($s, $i, $bytes);
-						$i+=$bytes;
+						$i += $bytes;
 						// 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;
 				}
@@ -174,14 +174,14 @@ 
 						$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) {
 						$bytes = 2;
-						return ($h & 0x1F) <<  6 | (ord($c{$index + 1}) & 0x3F);
+						return ($h & 0x1F) << 6 | (ord($c{$index + 1}) & 0x3F);
 					}
 					else if ($h <= 0xEF && $index < $len - 2) {
 						$bytes = 3;
@@ -201,33 +201,33 @@ 
 					}
 				}
 
-				function string2binint ($str,$npad=512) {
+				function string2binint($str, $npad = 512) {
 					$bin = array();
-					$ords = $this->string2ordUTF8($str,$this->bytesString);
-					$npad = $npad/$this->bytesString;
+					$ords = $this->string2ordUTF8($str, $this->bytesString);
+					$npad = $npad / $this->bytesString;
 					$length = count($ords);
 					$ords[] = 0x80; // append the "1" bit followed by 7 0's
-					$pad = ceil(($length+1+32/$this->bytesString)/$npad)*$npad-32/$this->bytesString;
-					$ords = array_pad($ords,$pad,0);
+					$pad = ceil(($length + 1 + 32 / $this->bytesString) / $npad) * $npad - 32 / $this->bytesString;
+					$ords = array_pad($ords, $pad, 0);
 					$mask = (1 << $this->bytesString) - 1;
-					for($i = 0; $i < count($ords) * $this->bytesString; $i += $this->bytesString) {
-						if (!isset($bin[$i>>5])) { $bin[$i>>5] = 0; } // pour eviter des notices.
-						$bin[$i>>5] |= ($ords[$i / $this->bytesString] & $mask) << (24 - $i%32);
+					for ($i = 0; $i < count($ords) * $this->bytesString; $i += $this->bytesString) {
+						if (!isset($bin[$i >> 5])) { $bin[$i >> 5] = 0; } // pour eviter des notices.
+						$bin[$i >> 5] |= ($ords[$i / $this->bytesString] & $mask) << (24 - $i % 32);
 					}
-					$bin[] = $length*$this->bytesString;
+					$bin[] = $length * $this->bytesString;
 					return $bin;
 				}
 
 				function array_split($a, $n) {
 					$split = array();
-					while (count($a)>$n) {
+					while (count($a) > $n) {
 						$s = array();
-						for($i = 0;$i<$n;$i++)
+						for ($i = 0; $i < $n; $i++)
 							$s[] = array_shift($a);
 						$split[] = $s;
 					}
-					if (count($a)){
-						$a = array_pad($a,$n,0);
+					if (count($a)) {
+						$a = array_pad($a, $n, 0);
 						$split[] = $a;
 					}
 					return $split;
@@ -242,12 +242,12 @@ 
          */
         function hash($str, $ig_func = true)
         {
-            unset($binStr);     // binary representation of input string
-            unset($hexStr);     // 256-bit message digest in readable hex format
+            unset($binStr); // binary representation of input string
+            unset($hexStr); // 256-bit message digest in readable hex format
 
             // check for php's internal sha256 function, ignore if ig_func==true
             if ($ig_func == false) {
-                if (version_compare(PHP_VERSION,'5.1.2','>=') AND !defined('_NO_HASH_DEFINED')) {
+                if (version_compare(PHP_VERSION, '5.1.2', '>=') AND !defined('_NO_HASH_DEFINED')) {
                     return hash("sha256", $str, false);
                 } else if (function_exists('mhash') && defined('MHASH_SHA256')) {
                     return base64_encode(bin2hex(mhash(MHASH_SHA256, $str)));
@@ -260,44 +260,44 @@ 
              *  first thirty-two bits of the fractional parts of the cube roots
              *  of the first sixtyfour prime numbers.
              */
-            $K = array((int)0x428a2f98, (int)0x71374491, (int)0xb5c0fbcf,
-                       (int)0xe9b5dba5, (int)0x3956c25b, (int)0x59f111f1,
-                       (int)0x923f82a4, (int)0xab1c5ed5, (int)0xd807aa98,
-                       (int)0x12835b01, (int)0x243185be, (int)0x550c7dc3,
-                       (int)0x72be5d74, (int)0x80deb1fe, (int)0x9bdc06a7,
-                       (int)0xc19bf174, (int)0xe49b69c1, (int)0xefbe4786,
-                       (int)0x0fc19dc6, (int)0x240ca1cc, (int)0x2de92c6f,
-                       (int)0x4a7484aa, (int)0x5cb0a9dc, (int)0x76f988da,
-                       (int)0x983e5152, (int)0xa831c66d, (int)0xb00327c8,
-                       (int)0xbf597fc7, (int)0xc6e00bf3, (int)0xd5a79147,
-                       (int)0x06ca6351, (int)0x14292967, (int)0x27b70a85,
-                       (int)0x2e1b2138, (int)0x4d2c6dfc, (int)0x53380d13,
-                       (int)0x650a7354, (int)0x766a0abb, (int)0x81c2c92e,
-                       (int)0x92722c85, (int)0xa2bfe8a1, (int)0xa81a664b,
-                       (int)0xc24b8b70, (int)0xc76c51a3, (int)0xd192e819,
-                       (int)0xd6990624, (int)0xf40e3585, (int)0x106aa070,
-                       (int)0x19a4c116, (int)0x1e376c08, (int)0x2748774c,
-                       (int)0x34b0bcb5, (int)0x391c0cb3, (int)0x4ed8aa4a,
-                       (int)0x5b9cca4f, (int)0x682e6ff3, (int)0x748f82ee,
-                       (int)0x78a5636f, (int)0x84c87814, (int)0x8cc70208,
-                       (int)0x90befffa, (int)0xa4506ceb, (int)0xbef9a3f7,
-                       (int)0xc67178f2);
+            $K = array((int) 0x428a2f98, (int) 0x71374491, (int) 0xb5c0fbcf,
+                       (int) 0xe9b5dba5, (int) 0x3956c25b, (int) 0x59f111f1,
+                       (int) 0x923f82a4, (int) 0xab1c5ed5, (int) 0xd807aa98,
+                       (int) 0x12835b01, (int) 0x243185be, (int) 0x550c7dc3,
+                       (int) 0x72be5d74, (int) 0x80deb1fe, (int) 0x9bdc06a7,
+                       (int) 0xc19bf174, (int) 0xe49b69c1, (int) 0xefbe4786,
+                       (int) 0x0fc19dc6, (int) 0x240ca1cc, (int) 0x2de92c6f,
+                       (int) 0x4a7484aa, (int) 0x5cb0a9dc, (int) 0x76f988da,
+                       (int) 0x983e5152, (int) 0xa831c66d, (int) 0xb00327c8,
+                       (int) 0xbf597fc7, (int) 0xc6e00bf3, (int) 0xd5a79147,
+                       (int) 0x06ca6351, (int) 0x14292967, (int) 0x27b70a85,
+                       (int) 0x2e1b2138, (int) 0x4d2c6dfc, (int) 0x53380d13,
+                       (int) 0x650a7354, (int) 0x766a0abb, (int) 0x81c2c92e,
+                       (int) 0x92722c85, (int) 0xa2bfe8a1, (int) 0xa81a664b,
+                       (int) 0xc24b8b70, (int) 0xc76c51a3, (int) 0xd192e819,
+                       (int) 0xd6990624, (int) 0xf40e3585, (int) 0x106aa070,
+                       (int) 0x19a4c116, (int) 0x1e376c08, (int) 0x2748774c,
+                       (int) 0x34b0bcb5, (int) 0x391c0cb3, (int) 0x4ed8aa4a,
+                       (int) 0x5b9cca4f, (int) 0x682e6ff3, (int) 0x748f82ee,
+                       (int) 0x78a5636f, (int) 0x84c87814, (int) 0x8cc70208,
+                       (int) 0x90befffa, (int) 0xa4506ceb, (int) 0xbef9a3f7,
+                       (int) 0xc67178f2);
 
             // Pre-processing: Padding the string
-            $binStr = $this->string2binint($str,512);
+            $binStr = $this->string2binint($str, 512);
 
             // Parsing the Padded Message (Break into N 512-bit blocks)
             $M = $this->array_split($binStr, 16);
 
             // Set the initial hash values
-            $h[0] = (int)0x6a09e667;
-            $h[1] = (int)0xbb67ae85;
-            $h[2] = (int)0x3c6ef372;
-            $h[3] = (int)0xa54ff53a;
-            $h[4] = (int)0x510e527f;
-            $h[5] = (int)0x9b05688c;
-            $h[6] = (int)0x1f83d9ab;
-            $h[7] = (int)0x5be0cd19;
+            $h[0] = (int) 0x6a09e667;
+            $h[1] = (int) 0xbb67ae85;
+            $h[2] = (int) 0x3c6ef372;
+            $h[3] = (int) 0xa54ff53a;
+            $h[4] = (int) 0x510e527f;
+            $h[5] = (int) 0x9b05688c;
+            $h[6] = (int) 0x1f83d9ab;
+            $h[7] = (int) 0x5be0cd19;
 
             // loop through message blocks and compute hash. ( For i=1 to N : )
             $N = count($M);
@@ -307,14 +307,14 @@ 
                 $MI = $M[$i];
 
                 // Initialize working variables
-                $_a = (int)$h[0];
-                $_b = (int)$h[1];
-                $_c = (int)$h[2];
-                $_d = (int)$h[3];
-                $_e = (int)$h[4];
-                $_f = (int)$h[5];
-                $_g = (int)$h[6];
-                $_h = (int)$h[7];
+                $_a = (int) $h[0];
+                $_b = (int) $h[1];
+                $_c = (int) $h[2];
+                $_d = (int) $h[3];
+                $_e = (int) $h[4];
+                $_f = (int) $h[5];
+                $_g = (int) $h[6];
+                $_h = (int) $h[7];
                 unset($_s0);
                 unset($_s1);
                 unset($_T1);
@@ -339,15 +339,15 @@ 
                 for (; $t < 64; $t++)
                 {
                     // Continue building the message schedule as we loop
-                    $_s0 = $W[($t+1)&0x0F];
+                    $_s0 = $W[($t + 1) & 0x0F];
                     $_s0 = $this->sigma_0($_s0);
-                    $_s1 = $W[($t+14)&0x0F];
+                    $_s1 = $W[($t + 14) & 0x0F];
                     $_s1 = $this->sigma_1($_s1);
 
-                    $W[$t&0xF] = $this->addmod2n($this->addmod2n($this->addmod2n($W[$t&0xF], $_s0), $_s1), $W[($t+9)&0x0F]);
+                    $W[$t & 0xF] = $this->addmod2n($this->addmod2n($this->addmod2n($W[$t & 0xF], $_s0), $_s1), $W[($t + 9) & 0x0F]);
 
                     // Compute hash
-                    $_T1 = $this->addmod2n($this->addmod2n($this->addmod2n($this->addmod2n($_h, $this->Sigma1($_e)), $this->Ch($_e, $_f, $_g)), $K[$t]), $W[$t&0xF]);
+                    $_T1 = $this->addmod2n($this->addmod2n($this->addmod2n($this->addmod2n($_h, $this->Sigma1($_e)), $this->Ch($_e, $_f, $_g)), $K[$t]), $W[$t & 0xF]);
                     $_T2 = $this->addmod2n($this->Sigma0($_a), $this->Maj($_a, $_b, $_c));
 
                     // Update working variables
@@ -397,7 +397,7 @@ 
             for ($i = 0; $i < $strlen; $i++)
             {
                 $i = (($sign < 0) ? $i + $length : $i);
-                $result[] = substr($string, $sign*$i, $length);
+                $result[] = substr($string, $sign * $i, $length);
                 $i--;
                 $i = (($sign < 0) ? $i : $i + $length);
 
@@ -434,7 +434,7 @@ 
 // support to give php4 the hash() routine which abstracts this code.
 if (!function_exists('hash'))
 {
-    define('_NO_HASH_DEFINED',true);
+    define('_NO_HASH_DEFINED', true);
     function hash($algo, $data)
     {
         if (empty($algo) || !is_string($algo) || !is_string($data)) {

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', '" .

Spacing +18 added lines, -20 removed lines

@@ -18,7 +18,7 @@ 
 //
 
 // http://doc.spip.org/@balise_LOGO__dist
-function balise_LOGO__dist ($p) {
+function balise_LOGO__dist($p) {
 
 	preg_match(",^LOGO_([A-Z_]+?)(|_NORMAL|_SURVOL|_RUBRIQUE)$,i", $p->nom_champ, $regs);
 	$type = strtolower($regs[1]);
@@ -42,36 +42,36 @@ 
 	if ($p->param AND !$p->param[0][0]) {
 		$params = $p->param[0];
 		array_shift($params);
-		foreach($params as $a) {
+		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')))
+					$coord[] = $n;
+				elseif (in_array($n, array('top', 'left', 'right', 'center', 'bottom')))
 					$align = $n;
-				elseif (in_array($n,array('auto','icone','apercu','vignette')))
+				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);
 
 		}
 	}
 
-	$coord_x = !$coord  ? 0 : intval(array_shift($coord));
-	$coord_y = !$coord  ? 0 : intval(array_shift($coord));
+	$coord_x = !$coord ? 0 : intval(array_shift($coord));
+	$coord_y = !$coord ? 0 : intval(array_shift($coord));
 	
 	if ($p->etoile === '*') {
 		include_spip('balise/url_');
 		$lien = generer_generer_url_arg($type, $p, $_id_objet);
 	}
 
-	$connect = $p->id_boucle ?$p->boucles[$p->id_boucle]->sql_serveur :'';
+	$connect = $p->id_boucle ? $p->boucles[$p->id_boucle]->sql_serveur : '';
 	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)";
+		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"] .= ",". '
@@ -97,25 +97,23 @@ 
 
 function logo_survol($id_objet, $_id_objet, $type, $align, $fichier, $lien, $p, $suite)
 {
-	$code = "quete_logo('$id_objet', '" .
-		(($suite == '_SURVOL') ? 'off' : 
-		(($suite == '_NORMAL') ? 'on' : 'ON')) .
-		"', $_id_objet," .
+	$code = "quete_logo('$id_objet', '".
+		(($suite == '_SURVOL') ? 'off' : (($suite == '_NORMAL') ? 'on' : 'ON')).
+		"', $_id_objet,".
 		(($suite == '_RUBRIQUE') ? 
-		champ_sql("id_rubrique", $p) :
-		(($type == 'rubrique') ? "quete_parent($_id_objet)" : "''")) .
-		", " . intval($fichier) . ")";
+		champ_sql("id_rubrique", $p) : (($type == 'rubrique') ? "quete_parent($_id_objet)" : "''")).
+		", ".intval($fichier).")";
 
 	if ($fichier) return $code;
 
-	$code = "\n((!is_array(\$l = $code)) ? '':\n (" .
-		     '"<img class=\"spip_logos\" alt=\"\"' .
+	$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;
 
-	return ('(strlen($logo='.$code.')?\'<a href="\' .' . $lien . ' . \'">\' . $logo . \'</a>\':\'\')');
+	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))

Spacing +68 added lines, -68 removed lines

@@ -28,13 +28,13 @@ 
  * @param string $reprise
  * @return
  */
-function base_upgrade_dist($titre='', $reprise='')
+function base_upgrade_dist($titre = '', $reprise = '')
 {
 	if (!$titre) return; // anti-testeur automatique
-	if ($GLOBALS['spip_version_base']!=$GLOBALS['meta']['version_installee']) {
+	if ($GLOBALS['spip_version_base'] != $GLOBALS['meta']['version_installee']) {
 		if (!is_numeric(_request('reinstall'))) {
 			include_spip('base/create');
-			spip_log("recree les tables eventuellement disparues","maj."._LOG_INFO_IMPORTANTE);
+			spip_log("recree les tables eventuellement disparues", "maj."._LOG_INFO_IMPORTANTE);
 			creer_base();
 		}
 		
@@ -47,7 +47,7 @@ 
 			exit;
 		}
 	}
-	spip_log("Fin de mise a jour SQL. Debut m-a-j acces et config","maj."._LOG_INFO_IMPORTANTE);
+	spip_log("Fin de mise a jour SQL. Debut m-a-j acces et config", "maj."._LOG_INFO_IMPORTANTE);
 	
 	// supprimer quelques fichiers temporaires qui peuvent se retrouver invalides
 	@spip_unlink(_CACHE_RUBRIQUES);
@@ -85,7 +85,7 @@ 
 	//
 	// version_installee = 1.702; quand on a besoin de forcer une MAJ
 	
-	spip_log("Version anterieure: $version_installee. Courante: $spip_version_base","maj."._LOG_INFO_IMPORTANTE);
+	spip_log("Version anterieure: $version_installee. Courante: $spip_version_base", "maj."._LOG_INFO_IMPORTANTE);
 	if (!$version_installee OR ($spip_version_base < $version_installee)) {
 		sql_replace('spip_meta', 
 		      array('nom' => 'version_installee',
@@ -100,12 +100,12 @@ 
 	if ($version_installee <= 1.926) {
 		$n = floor($version_installee * 10);
 		while ($n < 19) {
-			$nom  = sprintf("v%03d",$n);
+			$nom = sprintf("v%03d", $n);
 			$f = charger_fonction($nom, 'maj', true);
 			if ($f) {
-				spip_log( "$f repercute les modifications de la version " . ($n/10),"maj."._LOG_INFO_IMPORTANTE);
+				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');
@@ -113,20 +113,20 @@ 
 	}
 	if ($version_installee < 2000) {
 		if ($version_installee < 2)
-			$version_installee = $version_installee*1000;
+			$version_installee = $version_installee * 1000;
 		include_spip('maj/v019');
 	}
 	if ($cible < 2)
-		$cible = $cible*1000;
+		$cible = $cible * 1000;
 
 	include_spip('maj/svn10000');
 	ksort($GLOBALS['maj']);
-	$res = maj_while($version_installee, $cible, $GLOBALS['maj'], 'version_installee','meta', $redirect, true);
+	$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);
+			spip_log("Pb d'acces SQL a la mise a jour", "maj."._LOG_INFO_ERREUR);
 		else {
-			echo _T('avis_operation_echec') . ' ' . join(' ', $res);
+			echo _T('avis_operation_echec').' '.join(' ', $res);
 			echo install_fin_html();
 		}
 	}
@@ -164,47 +164,47 @@ 
  *     Nom de la table meta (sans le prefixe spip_) dans laquelle trouver la meta $nom_meta_base_version
  * @return void
  */
-function maj_plugin($nom_meta_base_version, $version_cible, $maj, $table_meta='meta'){
+function maj_plugin($nom_meta_base_version, $version_cible, $maj, $table_meta = 'meta') {
 
-	if ($table_meta!=='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,'='))){
+	if ((!isset($GLOBALS[$table_meta][$nom_meta_base_version]))
+			|| (!spip_version_compare($current_version = $GLOBALS[$table_meta][$nom_meta_base_version], $version_cible, '='))) {
 
 		// $maj['create'] contient les directives propres a la premiere creation de base
 		// c'est une operation derogatoire qui fait aboutir directement dans la version_cible
-		if (isset($maj['create'])){
-			if (!isset($GLOBALS[$table_meta][$nom_meta_base_version])){
+		if (isset($maj['create'])) {
+			if (!isset($GLOBALS[$table_meta][$nom_meta_base_version])) {
 				// installation : on ne fait que l'operation create
 				$maj = array("init"=>$maj['create']);
 				// et on lui ajoute un appel a inc/config
 				// pour creer les metas par defaut
-				$config = charger_fonction('config','inc');
+				$config = charger_fonction('config', 'inc');
 				$maj[$version_cible] = array(array($config));
 			}
 			// dans tous les cas enlever cet index du tableau
 			unset($maj['create']);
 		}
 		// si init, deja dans le bon ordre
-		if (!isset($maj['init'])){
+		if (!isset($maj['init'])) {
 			include_spip('inc/plugin'); // pour spip_version_compare
-			uksort($maj,'spip_version_compare');
+			uksort($maj, 'spip_version_compare');
 		}
 
 		// la redirection se fait par defaut sur la page d'administration des plugins
 		// sauf lorsque nous sommes sur l'installation de SPIP
 		// ou define _REDIRECT_MAJ_PLUGIN
-		$redirect = (defined('_REDIRECT_MAJ_PLUGIN')?_REDIRECT_MAJ_PLUGIN:generer_url_ecrire('admin_plugin'));
+		$redirect = (defined('_REDIRECT_MAJ_PLUGIN') ?_REDIRECT_MAJ_PLUGIN:generer_url_ecrire('admin_plugin'));
 		if (defined('_ECRIRE_INSTALL')) {
-			$redirect = parametre_url(generer_url_ecrire('install'),'etape', _request('etape'));
+			$redirect = parametre_url(generer_url_ecrire('install'), 'etape', _request('etape'));
 		}
 		
 		$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);
+				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>";
+				echo "<p>"._T('avis_operation_echec').' '.join(' ', $res)."</p>";
 			}
 		}
 	}
@@ -220,15 +220,15 @@ 
  * @param string $redirect
  * @return void
  */
-function relance_maj($meta,$table,$redirect=''){
+function relance_maj($meta, $table, $redirect = '') {
 	include_spip('inc/headers');
-	if (!$redirect){
+	if (!$redirect) {
 		// recuperer la valeur installee en cours
 		// on la tronque numeriquement, elle ne sert pas reellement
 		// sauf pour verifier que ce n'est pas oui ou non
 		// sinon is_numeric va echouer sur un numero de version 1.2.3
 		$installee = intval($GLOBALS[$table][$meta]);
-		$redirect = generer_url_ecrire('upgrade',"reinstall=$installee&meta=$meta&table=$table",true);
+		$redirect = generer_url_ecrire('upgrade', "reinstall=$installee&meta=$meta&table=$table", true);
 	}
 	echo redirige_formulaire($redirect);
 	exit();
@@ -243,21 +243,21 @@ 
  * @param string $table
  * @return
  */
-function maj_debut_page($installee,$meta,$table){
+function maj_debut_page($installee, $meta, $table) {
 	static $done = false;
 	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;
+	@ini_set("zlib.output_compression", "0"); // pour permettre l'affichage au fur et a mesure
+	$timeout = _UPGRADE_TIME_OUT * 2;
 	$titre = _T('titre_page_upgrade');
 	$balise_img = charger_filtre('balise_img');
 	$titre .= $balise_img(chemin_image('searching.gif'));
-	echo ( install_debut_html($titre));
+	echo (install_debut_html($titre));
 	// script de rechargement auto sur timeout
-	$redirect = generer_url_ecrire('upgrade',"reinstall=$installee&meta=$meta&table=$table",true);
-	echo http_script("window.setTimeout('location.href=\"".$redirect."\";',".($timeout*1000).")");
+	$redirect = generer_url_ecrire('upgrade', "reinstall=$installee&meta=$meta&table=$table", true);
+	echo http_script("window.setTimeout('location.href=\"".$redirect."\";',".($timeout * 1000).")");
 	echo "<div style='text-align: left'>\n";
-	ob_flush();flush();
+	ob_flush(); flush();
 	$done = true;
 }
 
@@ -296,27 +296,27 @@ 
  * @param bool $debut_page
  * @return array
  */
-function maj_while($installee, $cible, $maj, $meta='', $table='meta', $redirect='', $debut_page = false)
+function maj_while($installee, $cible, $maj, $meta = '', $table = 'meta', $redirect = '', $debut_page = false)
 {
 	# inclusions pour que les procedures d'upgrade disposent des fonctions de base
 	include_spip('base/create');
 	include_spip('base/abstract_sql');
-	$trouver_table = charger_fonction('trouver_table','base');
+	$trouver_table = charger_fonction('trouver_table', 'base');
 	include_spip('inc/plugin'); // pour spip_version_compare
 	$n = 0;
 	$time = time();
 	// definir le timeout qui peut etre utilise dans les fonctions
 	// de maj qui durent trop longtemps
-	define('_TIME_OUT',$time+_UPGRADE_TIME_OUT);
+	define('_TIME_OUT', $time + _UPGRADE_TIME_OUT);
 
 	reset($maj);
-	while (list($v,)=each($maj)) {
+	while (list($v,) = each($maj)) {
 		// si une maj pour cette version
-		if ($v=='init' OR
-			(spip_version_compare($v,$installee,'>')
-			AND spip_version_compare($v,$cible,'<='))) {
+		if ($v == 'init' OR
+			(spip_version_compare($v, $installee, '>')
+			AND spip_version_compare($v, $cible, '<='))) {
 			if ($debut_page)
-				maj_debut_page($v,$meta,$table);
+				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
@@ -324,19 +324,19 @@ 
 			# on sort
 			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);
+			spip_log("$table $meta: $v en $n secondes", 'maj.'._LOG_INFO_IMPORTANTE);
+			if ($meta) ecrire_meta($meta, $installee = $v, 'oui', $table);
 			echo "<br />";
 		}
 		if (time() >= _TIME_OUT) {
-			relance_maj($meta,$table,$redirect);
+			relance_maj($meta, $table, $redirect);
 		}
 	}
 	$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);
-	spip_log( "MAJ terminee. $meta: $installee",'maj.'._LOG_INFO_IMPORTANTE);
+	if ($meta) ecrire_meta($meta, $cible, 'oui', $table);
+	spip_log("MAJ terminee. $meta: $installee", 'maj.'._LOG_INFO_IMPORTANTE);
 	return array();
 }
 
@@ -358,41 +358,41 @@ 
  *   url de redirection en cas d'interruption
  * @return int
  */
-function serie_alter($serie, $q = array(), $meta='', $table='meta', $redirect='') {
-	$meta2 = $meta . '_maj_' . $serie;
+function serie_alter($serie, $q = array(), $meta = '', $table = 'meta', $redirect = '') {
+	$meta2 = $meta.'_maj_'.$serie;
 	$etape = intval(@$GLOBALS[$table][$meta2]);
 	foreach ($q as $i => $r) {
 		if ($i >= $etape) {
 			$msg = "maj $table $meta2 etape $i";
 			if (is_array($r)
 			  AND function_exists($f = array_shift($r))) {
-				spip_log( "$msg: $f " . join(',',$r),'maj.'._LOG_INFO_IMPORTANTE);
+				spip_log("$msg: $f ".join(',', $r), 'maj.'._LOG_INFO_IMPORTANTE);
 				// pour les fonctions atomiques sql_xx
 				// on enregistre le meta avant de lancer la fonction,
 				// de maniere a eviter de boucler sur timeout
 				// 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
 				// car on est peut etre sorti sur timeout si c'est une fonction longue
 				if (time() >= _TIME_OUT) {
-					relance_maj($meta,$table,$redirect);
+					relance_maj($meta, $table, $redirect);
 				}
-				ecrire_meta($meta2, $i+1, 'non', $table);
-				spip_log( "$meta2: ok", 'maj.'._LOG_INFO_IMPORTANTE);
+				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);
+					spip_log("maj $i format incorrect", "maj."._LOG_ERREUR);
 				else
-					spip_log("maj $i fonction $f non definie","maj."._LOG_ERREUR);
+					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;
+				return $i + 1;
 			}
 		}
 	}
@@ -422,23 +422,23 @@ 
 	// ne pas garder le resultat de la requete sinon sqlite3 
 	// ne peut pas supprimer la table spip_test lors du sql_alter qui suit
 	// car cette table serait alors 'verouillee'
-	$result = $result?true:false; 
+	$result = $result ?true:false; 
 	sql_alter("TABLE spip_test DROP b");
 	return $result;
 }
 
 // pour versions <= 1.926
 // http://doc.spip.org/@maj_version
-function maj_version ($version, $test = true) {
+function maj_version($version, $test = true) {
 	if ($test) {
-		if ($version>=1.922)
+		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') );
+			sql_updateq('spip_meta', array('valeur' => $version), "nom=".sql_quote('version_installee'));
 		}
-		spip_log( "mise a jour de la base en $version","maj."._LOG_INFO_IMPORTANTE);
+		spip_log("mise a jour de la base en $version", "maj."._LOG_INFO_IMPORTANTE);
 	} else {
 		echo _T('alerte_maj_impossible', array('version' => $version));
 		exit;
@@ -447,9 +447,9 @@ 
 
 // pour versions <= 1.926
 // http://doc.spip.org/@upgrade_vers
-function upgrade_vers($version, $version_installee, $version_cible = 0){
-	return ($version_installee<$version
-		AND (($version_cible>=$version) OR ($version_cible==0))
+function upgrade_vers($version, $version_installee, $version_cible = 0) {
+	return ($version_installee < $version
+		AND (($version_cible >= $version) OR ($version_cible == 0))
 	);
 }
 ?>

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"] . '">';

Spacing +15 added lines, -15 removed lines

@@ -20,10 +20,10 @@ 
  */
 function genie_mise_a_jour_dist($t) {
 	include_spip('inc/meta');
-	$maj = info_maj ('spip', 'SPIP', $GLOBALS['spip_version_branche']);
-	ecrire_meta('info_maj_spip',$maj?($GLOBALS['spip_version_branche']."|$maj"):"",'non');
+	$maj = info_maj('spip', 'SPIP', $GLOBALS['spip_version_branche']);
+	ecrire_meta('info_maj_spip', $maj ? ($GLOBALS['spip_version_branche']."|$maj") : "", 'non');
 
-	spip_log("Verification version SPIP : ".($maj?$maj:"version a jour"),"verifie_maj");
+	spip_log("Verification version SPIP : ".($maj ? $maj : "version a jour"), "verifie_maj");
 	return 1;
 }
 
@@ -34,32 +34,32 @@ 
 define('_VERSIONS_SERVEUR', 'http://files.spip.org/');
 define('_VERSIONS_LISTE', 'archives.xml');
 
-function info_maj ($dir, $file, $version){
+function info_maj($dir, $file, $version) {
 	include_spip('inc/plugin');
 	
-	list($maj,$min,$rev) = preg_split('/\D+/', $version);
+	list($maj, $min, $rev) = preg_split('/\D+/', $version);
 
-	$nom = _DIR_CACHE_XML . _VERSIONS_LISTE;
+	$nom = _DIR_CACHE_XML._VERSIONS_LISTE;
 	$page = !file_exists($nom) ? '' : file_get_contents($nom);
 	$page = info_maj_cache($nom, $dir, $page);
 
 	// reperer toutes les versions de numero majeur superieur ou egal
 	// (a revoir quand on arrivera a SPIP V10 ...)
 	$p = substr("0123456789", intval($maj));
-	$p = ',/' . $file . '\D+([' . $p . ']+)\D+(\d+)(\D+(\d+))?.*?[.]zip",i';
-	preg_match_all($p, $page, $m,  PREG_SET_ORDER);
+	$p = ',/'.$file.'\D+(['.$p.']+)\D+(\d+)(\D+(\d+))?.*?[.]zip",i';
+	preg_match_all($p, $page, $m, PREG_SET_ORDER);
 	$page = '';
 	foreach ($m as $v) {
 		list(, $maj2, $min2,, $rev2) = $v;
-		$version_maj = $maj2 . '.' . $min2 . '.' . $rev2;
+		$version_maj = $maj2.'.'.$min2.'.'.$rev2;
 		if ((spip_version_compare($version, $version_maj, '<'))
 		AND (spip_version_compare($page, $version_maj, '<')))
 			$page = $version_maj;
 	}
 
 	if (!$page) return "";
-	return "<a class='info_maj_spip' href='"._VERSIONS_SERVEUR."$dir' title='$page'>" .
-		_T('nouvelle_version_spip',array('version'=>$page)) .
+	return "<a class='info_maj_spip' href='"._VERSIONS_SERVEUR."$dir' title='$page'>".
+		_T('nouvelle_version_spip', array('version'=>$page)).
 	    '</a>';
 }
 
@@ -68,15 +68,15 @@ 
 // on teste la nouveaute par If-Modified-Since,
 // et seulement quand celui-ci a change' pour limiter les acces HTTP
 
-function info_maj_cache($nom, $dir, $page='')
+function info_maj_cache($nom, $dir, $page = '')
 {
-	$re = '<archives id="a' . $GLOBALS['meta']["alea_ephemere"] . '">';
+	$re = '<archives id="a'.$GLOBALS['meta']["alea_ephemere"].'">';
 	if (preg_match("/$re/", $page)) return $page;
 
-	$url = _VERSIONS_SERVEUR . $dir . '/' . _VERSIONS_LISTE;
+	$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);
+	$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;
 	// Placer l'indicateur de fraicheur

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',

Spacing +35 added lines, -35 removed lines

@@ -14,22 +14,22 @@ 
 include_spip('inc/filtres_images_lib_mini'); // par precaution
 
 // http://doc.spip.org/@couleur_html_to_hex
-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',
-		'navy'=>'000080','olive'=>'808000','purple'=>'800080','red'=>'FF0000','silver'=>'C0C0C0','teal'=>'008080','white'=>'FFFFFF','yellow'=>'FFFF00');
-	if (isset($couleurs_html[$lc=strtolower($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',
+		'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];
 	return $couleur;
 }
 
 // http://doc.spip.org/@couleur_foncer
-function couleur_foncer ($couleur, $coeff=0.5) {
+function couleur_foncer($couleur, $coeff = 0.5) {
 	$couleurs = _couleur_hex_to_dec($couleur);
 
-	$red = $couleurs["red"] - round(($couleurs["red"])*$coeff);
-	$green = $couleurs["green"] - round(($couleurs["green"])*$coeff);
-	$blue = $couleurs["blue"] - round(($couleurs["blue"])*$coeff);
+	$red = $couleurs["red"] - round(($couleurs["red"]) * $coeff);
+	$green = $couleurs["green"] - round(($couleurs["green"]) * $coeff);
+	$blue = $couleurs["blue"] - round(($couleurs["blue"]) * $coeff);
 
 	$couleur = _couleur_dec_to_hex($red, $green, $blue);
 	
@@ -37,12 +37,12 @@ 
 }
 
 // http://doc.spip.org/@couleur_eclaircir
-function couleur_eclaircir ($couleur, $coeff=0.5) {
+function couleur_eclaircir($couleur, $coeff = 0.5) {
 	$couleurs = _couleur_hex_to_dec($couleur);
 
-	$red = $couleurs["red"] + round((255 - $couleurs["red"])*$coeff);
-	$green = $couleurs["green"] + round((255 - $couleurs["green"])*$coeff);
-	$blue = $couleurs["blue"] + round((255 - $couleurs["blue"])*$coeff);
+	$red = $couleurs["red"] + round((255 - $couleurs["red"]) * $coeff);
+	$green = $couleurs["green"] + round((255 - $couleurs["green"]) * $coeff);
+	$blue = $couleurs["blue"] + round((255 - $couleurs["blue"]) * $coeff);
 
 	$couleur = _couleur_dec_to_hex($red, $green, $blue);
 	
@@ -54,33 +54,33 @@ 
 // ls images exclues sont marquees d'une class no_image_filtrer qui bloque les filtres suivants
 // 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){
+function image_select($img, $width_min = 0, $height_min = 0, $width_max = 10000, $height_max = 1000) {
 	if (!$img) return $img;
-	list ($h,$l) = taille_image($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)
+	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');
-	if (($select==false) AND ($p===FALSE)){
+	$class = extraire_attribut($img, 'class');
+	$p = strpos($class, 'no_image_filtrer');
+	if (($select == false) AND ($p === FALSE)) {
 		$class .= " no_image_filtrer";
-		$img = inserer_attribut($img,'class',$class);
+		$img = inserer_attribut($img, 'class', $class);
 	}
-	if (($select==true) AND ($p!==FALSE)){
-		$class = preg_replace(",\s*no_image_filtrer,","",$class);
-		$img = inserer_attribut($img,'class',$class);
+	if (($select == true) AND ($p !== FALSE)) {
+		$class = preg_replace(",\s*no_image_filtrer,", "", $class);
+		$img = inserer_attribut($img, 'class', $class);
 	}
 	return $img;
 }
 
 
 // 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'){
+function image_passe_partout($img, $taille_x = -1, $taille_y = -1, $force = false, $cherche_image = false, $process = 'AUTO') {
 	if (!$img) return '';
-	list ($hauteur,$largeur) = taille_image($img);
+	list ($hauteur, $largeur) = taille_image($img);
 	if ($taille_x == -1)
-		$taille_x = isset($GLOBALS['meta']['taille_preview'])?$GLOBALS['meta']['taille_preview']:150;
+		$taille_x = isset($GLOBALS['meta']['taille_preview']) ? $GLOBALS['meta']['taille_preview'] : 150;
 	if ($taille_y == -1)
 		$taille_y = $taille_x;
 
@@ -91,17 +91,17 @@ 
 	elseif ($taille_x == 0 AND $taille_y == 0)
 		return '';
 	
-	list($destWidth,$destHeight,$ratio) = ratio_passe_partout($largeur,$hauteur,$taille_x,$taille_y);
+	list($destWidth, $destHeight, $ratio) = ratio_passe_partout($largeur, $hauteur, $taille_x, $taille_y);
 	$fonction = array('image_passe_partout', func_get_args());
-	return process_image_reduire($fonction,$img,$destWidth,$destHeight,$force,$cherche_image,$process);
+	return process_image_reduire($fonction, $img, $destWidth, $destHeight, $force, $cherche_image, $process);
 }
 
 // http://doc.spip.org/@image_reduire
-function image_reduire($img, $taille = -1, $taille_y = -1, $force=false, $cherche_image=false, $process='AUTO') {
+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;
+		$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;
 
@@ -113,16 +113,16 @@ 
 		return '';
 
 	$fonction = array('image_reduire', func_get_args());
-	return process_image_reduire($fonction,$img,$taille,$taille_y,$force,$cherche_image,$process);
+	return process_image_reduire($fonction, $img, $taille, $taille_y, $force, $cherche_image, $process);
 }
 
 // Reduire une image d'un certain facteur
 // http://doc.spip.org/@image_reduire_par
-function image_reduire_par ($img, $val=1, $force=false) {
-	list ($hauteur,$largeur) = taille_image($img);
+function image_reduire_par($img, $val = 1, $force = false) {
+	list ($hauteur, $largeur) = taille_image($img);
 
-	$l = round($largeur/$val);
-	$h = round($hauteur/$val);
+	$l = round($largeur / $val);
+	$h = round($hauteur / $val);
 	
 	if ($l > $h) $h = 0;
 	else $l = 0;

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(

Spacing +29 added lines, -29 removed lines

@@ -17,20 +17,20 @@ 
 // 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)))
+function redirige_par_entete($url, $equiv = '', $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]=='?')
+	if ($url[0] == '?')
 		$url = url_de_base().$url;
-	if ($url[0]=='#')
+	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)){
+	if ($GLOBALS['profondeur_url'] > (_DIR_RESTREINT ? 1 : 2)
+		AND !preg_match(",^(\w+:)?//,", $url)) {
 		include_spip("inc/filtres_mini");
 		$url = url_absolue($url);
 	}
@@ -42,30 +42,30 @@ 
 		$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);
+	$url = str_replace(array('<', '"'), array('&lt;', '&quot;'), $url);
 	// interdire les url inline avec des pseudo-protocoles :
 	if (
-		(preg_match(",data:,i",$url) AND preg_match("/base64\s*,/i",$url))
-		OR preg_match(",(javascript|mailto):,i",$url)
+		(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');
-	if ((!$equiv AND !spip_cookie_envoye()) OR ((strncmp("Apache", $_SERVER['SERVER_SOFTWARE'],6)==0) OR defined('_SERVER_APACHE'))) {
-		@header("Location: " . $url);
-		$equiv="";
+	if ((!$equiv AND !spip_cookie_envoye()) OR ((strncmp("Apache", $_SERVER['SERVER_SOFTWARE'], 6) == 0) OR defined('_SERVER_APACHE'))) {
+		@header("Location: ".$url);
+		$equiv = "";
 	} else {
-		@header("Refresh: 0; url=" . $url);
+		@header("Refresh: 0; url=".$url);
 		$equiv = "<meta http-equiv='Refresh' content='0; url=$url'>";
 	}
 	include_spip('inc/lang');
-	if ($status!=302)
+	if ($status != 302)
 		http_status($status);
-	echo '<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">',"\n",
-	  html_lang_attributes(),'
+	echo '<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">', "\n",
+	  html_lang_attributes(), '
 <head>',
-	  $equiv,'
+	  $equiv, '
 <title>HTTP '.$status.'</title>
 </head>
 <body>
@@ -82,14 +82,14 @@ 
 }
 
 // http://doc.spip.org/@redirige_formulaire
-function redirige_formulaire($url, $equiv = '', $format='message') {
+function redirige_formulaire($url, $equiv = '', $format = 'message') {
 	if (!_AJAX
 	AND !headers_sent()
 	AND !_request('var_ajax')) {
-		redirige_par_entete(str_replace('&amp;','&',$url), $equiv);
+		redirige_par_entete(str_replace('&amp;', '&', $url), $equiv);
 	}
 	// si c'est une ancre, fixer simplement le window.location.hash
-	elseif($format=='ajaxform' AND preg_match(',^#[0-9a-z\-_]+$,i',$url)) {
+	elseif ($format == 'ajaxform' AND preg_match(',^#[0-9a-z\-_]+$,i', $url)) {
 		return array(
 		// on renvoie un lien masque qui sera traite par ajaxCallback.js
 		"<a href='$url' name='ajax_ancre' style='display:none;'>anchor</a>",
@@ -98,17 +98,17 @@ 
 	}
 	else {
 		// ne pas laisser passer n'importe quoi dans l'url
-		$url = str_replace(array('<','"'),array('&lt;','&quot;'),$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]=='?')
+		if ($url[0] == '?')
 			$url = url_de_base().$url;
-		$url = str_replace('&amp;','&',$url);
+		$url = str_replace('&amp;', '&', $url);
 		spip_log("redirige formulaire ajax: $url");
 		include_spip('inc/filtres');
-		if ($format=='ajaxform')
+		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>',
@@ -119,14 +119,14 @@ 
 			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','')
+		. http_img_pack('searching.gif', '')
 		. '<br />'
 		. '<a href="'.quote_amp($url).'">'._T('navigateur_pas_redirige').'</a>';
 	}
 }
 
 // http://doc.spip.org/@redirige_url_ecrire
-function redirige_url_ecrire($script='', $args='', $equiv='') {
+function redirige_url_ecrire($script = '', $args = '', $equiv = '') {
 	return redirige_par_entete(generer_url_ecrire($script, $args, true), $equiv);
 }
 
@@ -158,7 +158,7 @@ 
 // http://doc.spip.org/@http_no_cache
 function http_no_cache() {
 	if (headers_sent())
-		{ spip_log("http_no_cache arrive trop tard"); return;}
+		{ spip_log("http_no_cache arrive trop tard"); return; }
 	$charset = empty($GLOBALS['meta']['charset']) ? 'utf-8' : $GLOBALS['meta']['charset'];
 
 	// selon http://developer.apple.com/internet/safari/faq.html#anchor5
@@ -169,7 +169,7 @@ 
 
 	header("Content-Type: text/html; charset=$charset");
 	header("Expires: 0");
-	header("Last-Modified: " .gmdate("D, d M Y H:i:s"). " GMT");
+	header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
 	header("Cache-Control: no-cache, must-revalidate");
 	header("Pragma: 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();

Spacing +28 added lines, -28 removed lines

@@ -16,23 +16,23 @@ 
 // Recopie dans le tableau PHP global meta, car on en a souvent besoin
 
 // duree maximale du cache. Le double pour l'antidater
-define('_META_CACHE_TIME', 1<<24);
+define('_META_CACHE_TIME', 1 << 24);
 
 // http://doc.spip.org/@inc_meta_dist
-function inc_meta_dist($table='meta')
+function inc_meta_dist($table = 'meta')
 {
 	// Lire les meta, en cache si present, valide et lisible
 	// en cas d'install ne pas faire confiance au meta_cache eventuel
 	$cache = cache_meta($table);
 
-	if ((_request('exec')!=='install' OR !test_espace_prive())
+	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;
 
 	if (isset($GLOBALS[$table]['touch']) 
-	AND ($GLOBALS[$table]['touch']<time()-_META_CACHE_TIME))
+	AND ($GLOBALS[$table]['touch'] < time() - _META_CACHE_TIME))
 		$GLOBALS[$table] = array();
 	// sinon lire en base
 	if (!$GLOBALS[$table]) $new = !lire_metas($table);
@@ -47,7 +47,7 @@ 
 			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);
@@ -57,7 +57,7 @@ 
 // pour eviter l'erreur fatale (serveur non encore configure)
 
 // http://doc.spip.org/@lire_metas
-function lire_metas($table='meta') {
+function lire_metas($table = 'meta') {
 
 	if ($result = spip_query("SELECT nom,valeur FROM spip_$table")) {
 		include_spip('base/abstract_sql');
@@ -67,12 +67,12 @@ 
         sql_free($result);
 
 		if (!$GLOBALS[$table]['charset']
-		  OR $GLOBALS[$table]['charset']=='_DEFAULT_CHARSET' // hum, correction d'un bug ayant abime quelques install
+		  OR $GLOBALS[$table]['charset'] == '_DEFAULT_CHARSET' // hum, correction d'un bug ayant abime quelques install
 		)
 			ecrire_meta('charset', _DEFAULT_CHARSET, NULL, $table);
 
 		// noter cette table de configuration dans les meta de SPIP
-		if ($table!=='meta') {
+		if ($table !== 'meta') {
 			$liste = unserialize($GLOBALS['meta']['tables_config']);
 			if (!$liste)
 				$liste = array();
@@ -88,7 +88,7 @@ 
 // Mettre en cache la liste des meta, sauf les valeurs sensibles 
 // pour qu'elles ne soient pas visibiles dans un fichier.souvent en 777
 // http://doc.spip.org/@touch_meta
-function touch_meta($antidate= false, $table='meta'){
+function touch_meta($antidate = false, $table = 'meta') {
 	$file = cache_meta($table);
 	if (!$antidate OR !@touch($file, $antidate)) {
 		$r = $GLOBALS[$table];
@@ -98,32 +98,32 @@ 
 		// 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));
 	}
 }
 
 // http://doc.spip.org/@effacer_meta
-function effacer_meta($nom, $table='meta') {
+function effacer_meta($nom, $table = 'meta') {
 	// section critique sur le cache:
 	// l'invalider avant et apres la MAJ de la BD
 	// c'est un peu moins bien qu'un vrai verrou mais ca suffira
 	// et utiliser une statique pour eviter des acces disques a repetition
 	static $touch = array();
-	$antidate = time() - (_META_CACHE_TIME<<4);
-	if (!isset($touch[$table])) {touch_meta($antidate, $table);}
-	sql_delete('spip_' . $table, "nom='$nom'");
+	$antidate = time() - (_META_CACHE_TIME << 4);
+	if (!isset($touch[$table])) {touch_meta($antidate, $table); }
+	sql_delete('spip_'.$table, "nom='$nom'");
 	unset($GLOBALS[$table][$nom]);
-	if (!isset($touch[$table])) {touch_meta($antidate, $table); $touch[$table] = false;}
+	if (!isset($touch[$table])) {touch_meta($antidate, $table); $touch[$table] = false; }
 }
 
 // http://doc.spip.org/@ecrire_meta
-function ecrire_meta($nom, $valeur, $importable = NULL, $table='meta') {
+function ecrire_meta($nom, $valeur, $importable = NULL, $table = 'meta') {
 
 	static $touch = array();
 	if (!$nom) return;
 	include_spip('base/abstract_sql');
-	$res = sql_select("*",'spip_' . $table,"nom=" . sql_quote($nom),'','','','','','continue');
+	$res = sql_select("*", 'spip_'.$table, "nom=".sql_quote($nom), '', '', '', '', '', 'continue');
 	// table pas encore installee, travailler en php seulement
 	if (!$res) {
 		$GLOBALS[$table][$nom] = $valeur;
@@ -138,22 +138,22 @@ 
 
 	$GLOBALS[$table][$nom] = $valeur;
 	// cf effacer pour comprendre le double touch
-	$antidate = time() - (_META_CACHE_TIME<<1);
-	if (!isset($touch[$table])) {touch_meta($antidate, $table);}
+	$antidate = time() - (_META_CACHE_TIME << 1);
+	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 ($row) {
-		sql_updateq('spip_' . $table, $r,"nom=" . sql_quote($nom));
+		sql_updateq('spip_'.$table, $r, "nom=".sql_quote($nom));
 	} else {
-		sql_insertq('spip_' . $table, $r);
+		sql_insertq('spip_'.$table, $r);
 	}
-	if (!isset($touch[$table])) {touch_meta($antidate, $table); $touch[$table] = false;}
+	if (!isset($touch[$table])) {touch_meta($antidate, $table); $touch[$table] = false; }
 }
 
-function cache_meta($table='meta')
+function cache_meta($table = 'meta')
 {
-	return ($table=='meta') ? _FILE_META : (_DIR_CACHE . $table . '.php');
+	return ($table == 'meta') ? _FILE_META : (_DIR_CACHE.$table.'.php');
 }
 
 /**
@@ -161,7 +161,7 @@ 
  * @param string $table
  */
 function installer_table_meta($table) {
-	$trouver_table = charger_fonction('trouver_table','base');
+	$trouver_table = charger_fonction('trouver_table', 'base');
 	if (!$trouver_table("spip_$table")) {
 		include_spip('base/auxiliaires');
 		include_spip('base/create');
@@ -178,14 +178,14 @@ 
  * @param string $table
  * @param bool $force
  */
-function supprimer_table_meta($table, $force=false) {
-	if ($table=='meta') return; // interdit !
+function supprimer_table_meta($table, $force = false) {
+	if ($table == 'meta') return; // interdit !
 
 	if ($force OR !sql_countsel("spip_$table")) {
 		unset($GLOBALS[$table]);
 		sql_drop_table("spip_$table");
 		// vider le cache des tables
-		$trouver_table = charger_fonction('trouver_table','base');
+		$trouver_table = charger_fonction('trouver_table', 'base');
 		$trouver_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;

Spacing +16 added lines, -16 removed lines

@@ -19,13 +19,13 @@ 
 //
 
 // http://doc.spip.org/@inc_selectionner_dist
-function inc_selectionner_dist ($sel, $idom="", $exclus=0, $aff_racine=false, $recur=true, $do='aff') {
+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 ($aff_racine) {
 		$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
-		$idom3 = $idom . "_selection";
+		$idom3 = $idom."_selection";
 
 		$onClick = "jQuery(this).parent().addClass('on');jQuery('#choix_parent_principal .on').removeClass('on'); aff_selection(0, '$idom3', '$info', event);return false;";
 
@@ -34,7 +34,7 @@ 
 					textebrut(_T('info_racine_site')))),
 				"\n\r", "  ");
 
-		$js_func = $do . '_selection_titre';
+		$js_func = $do.'_selection_titre';
 		$ondbClick = "$js_func('$ondbClick',0,'selection_rubrique','id_parent');";
 
 		$aff_racine = "<div class='petite-racine item'>"
@@ -49,7 +49,7 @@ 
 		. "</a></div>";
 	}
 
-	$url_init = generer_url_ecrire('plonger',"rac=$idom&exclus=$exclus&id=0&col=1&do=$do");
+	$url_init = generer_url_ecrire('plonger', "rac=$idom&exclus=$exclus&id=0&col=1&do=$do");
 
 	$plonger = charger_fonction('plonger', 'inc');
 	$plonger_r = $plonger($sel, $idom, $recur, 1, $exclus, $do);
@@ -60,20 +60,20 @@ 
 }
 
 // http://doc.spip.org/@construire_selectionner_hierarchie
-function construire_selectionner_hierarchie($idom, $liste, $racine, $url, $name, $url_init='')
+function construire_selectionner_hierarchie($idom, $liste, $racine, $url, $name, $url_init = '')
 {
 	global $spip_lang_right;
 
-	$idom1 = $idom . "_champ_recherche";
-	$idom2 = $idom . "_principal";
-	$idom3 = $idom . "_selection";
-	$idom4 = $idom . "_col_1";
-	$idom5 = 'img_' . $idom4;
+	$idom1 = $idom."_champ_recherche";
+	$idom2 = $idom."_principal";
+	$idom3 = $idom."_selection";
+	$idom4 = $idom."_col_1";
+	$idom5 = 'img_'.$idom4;
 	$idom6 = $idom."_fonc";
 
 	return "<div id='$idom'>"
 	. "<a id='$idom6' style='visibility: hidden;'"
-	. ($url_init ?  "\nhref='$url_init'" : '')
+	. ($url_init ? "\nhref='$url_init'" : '')
 	. "></a>"
 	. "<div class='recherche_rapide_parent'>"
 	. http_img_pack("searching.gif", "*", "style='visibility: hidden;float:$spip_lang_right' id='$idom5'")
@@ -104,7 +104,7 @@ 
 	. "')\"" 
 	. " />"
 	. "\n</div>"
-	. ($racine?"<div>$racine</div>":"")
+	. ($racine ? "<div>$racine</div>" : "")
 	. "<div id='"
 	.  $idom2
 	.  "'><div id='$idom4'"
@@ -114,13 +114,13 @@ 
 }
 
 // http://doc.spip.org/@mini_hier
-function mini_hier ($id_rubrique) {
+function mini_hier($id_rubrique) {
 	
 	$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";
-	return explode(',',"0,$liste");
+	while ($id_rubrique = sql_getfetsel("id_parent", "spip_rubriques", "id_rubrique = ".$id_rubrique))
+		$liste = $id_rubrique.",$liste";
+	return explode(',', "0,$liste");
 }
 
 ?>