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

Indentation +111 added lines, -111 removed lines

@@ -73,7 +73,7 @@ 
         // php 4 - 5 compatable class properties
         var     $toUpper;
         var     $platform;
-				var		  $bytesString = 16;
+                var		  $bytesString = 16;
 
         // Php 4 - 6 compatable constructor
         function nanoSha2($toUpper = false) {
@@ -144,94 +144,94 @@ 
         function sigma_1($x) { return (int) ($this->ROTR($x, 17)^$this->ROTR($x, 19)^$this->SHR($x, 10)); }
 
 
-				function string2ordUTF8($s,&$byteSize){
-					$chars = array();
-					// par defaut sur 8bits
-					$byteSize = 8;
-					$i = 0;
-					while ($i<strlen($s)){
-						$chars[] = $this->ordUTF8($s, $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;
-					}
-					return $chars;
-				}
-
-				function ordUTF8($c, $index = 0, &$bytes)
-				{
-					$len = strlen($c);
-					$bytes = 0;
-
-					if ($index >= $len)
-						return false;
-
-					$h = ord($c{$index});
-
-					if ($h <= 0x7F) {
-						$bytes = 1;
-						return $h;
-					}
-					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);
-					}
-					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) {
-						$bytes = 4;
-						return ($h & 0x0F) << 18 | (ord($c{$index + 1}) & 0x3F) << 12
-																		 | (ord($c{$index + 2}) & 0x3F) << 6
-																		 | (ord($c{$index + 3}) & 0x3F);
-					}
-					else {
-						// pas utf mais renvoyer quand meme ce qu'on a
-						$bytes = 1;
-						return $h;
-					}
-				}
-
-				function string2binint ($str,$npad=512) {
-					$bin = array();
-					$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);
-					$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);
-					}
-					$bin[] = $length*$this->bytesString;
-					return $bin;
-				}
-
-				function array_split($a, $n) {
-					$split = array();
-					while (count($a)>$n) {
-						$s = array();
-						for($i = 0;$i<$n;$i++)
-							$s[] = array_shift($a);
-						$split[] = $s;
-					}
-					if (count($a)){
-						$a = array_pad($a,$n,0);
-						$split[] = $a;
-					}
-					return $split;
-				}
+                function string2ordUTF8($s,&$byteSize){
+                    $chars = array();
+                    // par defaut sur 8bits
+                    $byteSize = 8;
+                    $i = 0;
+                    while ($i<strlen($s)){
+                        $chars[] = $this->ordUTF8($s, $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;
+                    }
+                    return $chars;
+                }
+
+                function ordUTF8($c, $index = 0, &$bytes)
+                {
+                    $len = strlen($c);
+                    $bytes = 0;
+
+                    if ($index >= $len)
+                        return false;
+
+                    $h = ord($c{$index});
+
+                    if ($h <= 0x7F) {
+                        $bytes = 1;
+                        return $h;
+                    }
+                    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);
+                    }
+                    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) {
+                        $bytes = 4;
+                        return ($h & 0x0F) << 18 | (ord($c{$index + 1}) & 0x3F) << 12
+                                                                            | (ord($c{$index + 2}) & 0x3F) << 6
+                                                                            | (ord($c{$index + 3}) & 0x3F);
+                    }
+                    else {
+                        // pas utf mais renvoyer quand meme ce qu'on a
+                        $bytes = 1;
+                        return $h;
+                    }
+                }
+
+                function string2binint ($str,$npad=512) {
+                    $bin = array();
+                    $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);
+                    $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);
+                    }
+                    $bin[] = $length*$this->bytesString;
+                    return $bin;
+                }
+
+                function array_split($a, $n) {
+                    $split = array();
+                    while (count($a)>$n) {
+                        $s = array();
+                        for($i = 0;$i<$n;$i++)
+                            $s[] = array_shift($a);
+                        $split[] = $s;
+                    }
+                    if (count($a)){
+                        $a = array_pad($a,$n,0);
+                        $split[] = $a;
+                    }
+                    return $split;
+                }
 
         /**
          * Process and return the hash.
@@ -261,27 +261,27 @@ 
              *  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);
+                        (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);
@@ -382,7 +382,7 @@ 
      */
     function str_split($string, $split_length = 1)
     {
-	      $result = array();
+            $result = array();
         $sign = ($split_length < 0) ? -1 : 1;
         $strlen = strlen($string);
         $split_length = abs($split_length);

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

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

Indentation +87 added lines, -87 removed lines

@@ -20,102 +20,102 @@
 // http://doc.spip.org/@balise_LOGO__dist
 function balise_LOGO__dist ($p) {
 
-	preg_match(",^LOGO_([A-Z_]+?)(|_NORMAL|_SURVOL|_RUBRIQUE)$,i", $p->nom_champ, $regs);
-	$type = strtolower($regs[1]);
-	$suite_logo = $regs[2];
-
-	// cas de #LOGO_SITE_SPIP
-	if ($type == 'site_spip') {
-		$type = 'site';
-		$_id_objet = "\"'0'\"";
-	}
-
-	$id_objet = id_table_objet($type);
-	if (!isset($_id_objet) OR !$_id_objet)
-		$_id_objet = champ_sql($id_objet, $p);
-
-	$fichier = ($p->etoile === '**') ? -1 : 0;
-	$coord = array();
-	$align = $lien = '';
-	$mode_logo = '';
-
-	if ($p->param AND !$p->param[0][0]) {
-		$params = $p->param[0];
-		array_shift($params);
-		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;
-			}
-			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));
+    preg_match(",^LOGO_([A-Z_]+?)(|_NORMAL|_SURVOL|_RUBRIQUE)$,i", $p->nom_champ, $regs);
+    $type = strtolower($regs[1]);
+    $suite_logo = $regs[2];
+
+    // cas de #LOGO_SITE_SPIP
+    if ($type == 'site_spip') {
+        $type = 'site';
+        $_id_objet = "\"'0'\"";
+    }
+
+    $id_objet = id_table_objet($type);
+    if (!isset($_id_objet) OR !$_id_objet)
+        $_id_objet = champ_sql($id_objet, $p);
+
+    $fichier = ($p->etoile === '**') ? -1 : 0;
+    $coord = array();
+    $align = $lien = '';
+    $mode_logo = '';
+
+    if ($p->param AND !$p->param[0][0]) {
+        $params = $p->param[0];
+        array_shift($params);
+        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;
+            }
+            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));
 	
-	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 :'';
-	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)";
-		// (x=non-faux ? y : '') pour affecter x en retournant y
-		if ($p->descr['documents'])
-		  $code = '(($doublons["documents"] .= ",". '
-		    . $_id_objet
-		    . ") ? $code : '')";
-	}
-	elseif ($connect) {
-		$code = "''";
-		spip_log("Les logos distants ne sont pas prevus");
-	} else {
-		$code = logo_survol($id_objet, $_id_objet, $type, $align, $fichier, $lien, $p, $suite_logo);
-	}
-
-	// demande de reduction sur logo avec ecriture spip 2.1 : #LOGO_xxx{200, 0}
-	if ($coord_x OR $coord_y) {
-		$code = "filtrer('image_graver',filtrer('image_reduire',".$code.", '$coord_x', '$coord_y'))"; 
-	} 
-
-	$p->code = $code;
-	$p->interdire_scripts = false;
-	return $p;
+    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 :'';
+    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)";
+        // (x=non-faux ? y : '') pour affecter x en retournant y
+        if ($p->descr['documents'])
+            $code = '(($doublons["documents"] .= ",". '
+            . $_id_objet
+            . ") ? $code : '')";
+    }
+    elseif ($connect) {
+        $code = "''";
+        spip_log("Les logos distants ne sont pas prevus");
+    } else {
+        $code = logo_survol($id_objet, $_id_objet, $type, $align, $fichier, $lien, $p, $suite_logo);
+    }
+
+    // demande de reduction sur logo avec ecriture spip 2.1 : #LOGO_xxx{200, 0}
+    if ($coord_x OR $coord_y) {
+        $code = "filtrer('image_graver',filtrer('image_reduire',".$code.", '$coord_x', '$coord_y'))"; 
+    } 
+
+    $p->code = $code;
+    $p->interdire_scripts = false;
+    return $p;
 }
 
 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," .
-		(($suite == '_RUBRIQUE') ? 
-		champ_sql("id_rubrique", $p) :
-		(($type == 'rubrique') ? "quete_parent($_id_objet)" : "''")) .
-		", " . intval($fichier) . ")";
+    $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) . ")";
 
-	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]\'\"" : "") . \' />\'))';
+    $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>\':\'\')');
+    return ('(strlen($logo='.$code.')?\'<a href="\' .' . $lien . ' . \'">\' . $logo . \'</a>\':\'\')');
 
 }
 

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>\':\'\')');
 
 }
 

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

Indentation +253 added lines, -253 removed lines

@@ -30,38 +30,38 @@ 
  */
 function base_upgrade_dist($titre='', $reprise='')
 {
-	if (!$titre) return; // anti-testeur automatique
-	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);
-			creer_base();
-		}
+    if (!$titre) return; // anti-testeur automatique
+    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);
+            creer_base();
+        }
 		
-		// quand on rentre par ici, c'est toujours une mise a jour de SPIP
-		// lancement de l'upgrade SPIP
-		$res = maj_base();
+        // quand on rentre par ici, c'est toujours une mise a jour de SPIP
+        // lancement de l'upgrade SPIP
+        $res = maj_base();
 
-		if ($res) {
-			// on arrete tout ici !
-			exit;
-		}
-	}
-	spip_log("Fin de mise a jour SQL. Debut m-a-j acces et config","maj."._LOG_INFO_IMPORTANTE);
+        if ($res) {
+            // on arrete tout ici !
+            exit;
+        }
+    }
+    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);
-	@spip_unlink(_CACHE_PIPELINES);
-	@spip_unlink(_CACHE_PLUGINS_PATH);
-	@spip_unlink(_CACHE_PLUGINS_OPT);
-	@spip_unlink(_CACHE_PLUGINS_FCT);
-	@spip_unlink(_CACHE_CHEMIN);
-	@spip_unlink(_DIR_TMP."plugin_xml_cache.gz");
+    // supprimer quelques fichiers temporaires qui peuvent se retrouver invalides
+    @spip_unlink(_CACHE_RUBRIQUES);
+    @spip_unlink(_CACHE_PIPELINES);
+    @spip_unlink(_CACHE_PLUGINS_PATH);
+    @spip_unlink(_CACHE_PLUGINS_OPT);
+    @spip_unlink(_CACHE_PLUGINS_FCT);
+    @spip_unlink(_CACHE_CHEMIN);
+    @spip_unlink(_DIR_TMP."plugin_xml_cache.gz");
 
-	include_spip('inc/auth');
-	auth_synchroniser_distant();
-	$config = charger_fonction('config', 'inc');
-	$config();
+    include_spip('inc/auth');
+    auth_synchroniser_distant();
+    $config = charger_fonction('config', 'inc');
+    $config();
 }
 
 /**
@@ -74,63 +74,63 @@ 
  * @return array|bool
  */
 function maj_base($version_cible = 0, $redirect = '') {
-	global $spip_version_base;
+    global $spip_version_base;
 
-	$version_installee = @$GLOBALS['meta']['version_installee'];
-	//
-	// Si version nulle ou inexistante, c'est une nouvelle installation
-	//   => ne pas passer par le processus de mise a jour.
-	// De meme en cas de version superieure: ca devait etre un test,
-	// il y a eu le message d'avertissement il doit savoir ce qu'il fait
-	//
-	// version_installee = 1.702; quand on a besoin de forcer une MAJ
+    $version_installee = @$GLOBALS['meta']['version_installee'];
+    //
+    // Si version nulle ou inexistante, c'est une nouvelle installation
+    //   => ne pas passer par le processus de mise a jour.
+    // De meme en cas de version superieure: ca devait etre un test,
+    // il y a eu le message d'avertissement il doit savoir ce qu'il fait
+    //
+    // 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);
-	if (!$version_installee OR ($spip_version_base < $version_installee)) {
-		sql_replace('spip_meta', 
-		      array('nom' => 'version_installee',
-			    'valeur' => $spip_version_base,
-			    'impt' => 'non'));
-		return false;
-	}
-	if (!upgrade_test()) return true;
+    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',
+                'valeur' => $spip_version_base,
+                'impt' => 'non'));
+        return false;
+    }
+    if (!upgrade_test()) return true;
 	
-	$cible = ($version_cible ? $version_cible : $spip_version_base);
+    $cible = ($version_cible ? $version_cible : $spip_version_base);
 
-	if ($version_installee <= 1.926) {
-		$n = floor($version_installee * 10);
-		while ($n < 19) {
-			$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);
-				$f($version_installee, $spip_version_base);
-			} 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;
-		include_spip('maj/v019');
-	}
-	if ($cible < 2)
-		$cible = $cible*1000;
+    if ($version_installee <= 1.926) {
+        $n = floor($version_installee * 10);
+        while ($n < 19) {
+            $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);
+                $f($version_installee, $spip_version_base);
+            } 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;
+        include_spip('maj/v019');
+    }
+    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 {
-			echo _T('avis_operation_echec') . ' ' . join(' ', $res);
-			echo install_fin_html();
-		}
-	}
-	return $res;
+    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 {
+            echo _T('avis_operation_echec') . ' ' . join(' ', $res);
+            echo install_fin_html();
+        }
+    }
+    return $res;
 }
 
 /**
@@ -166,48 +166,48 @@ 
  */
 function maj_plugin($nom_meta_base_version, $version_cible, $maj, $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 ($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,'='))){
 
-		// $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])){
-				// 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');
-				$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'])){
-			include_spip('inc/plugin'); // pour spip_version_compare
-			uksort($maj,'spip_version_compare');
-		}
+        // $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])){
+                // 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');
+                $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'])){
+            include_spip('inc/plugin'); // pour 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'));
-		if (defined('_ECRIRE_INSTALL')) {
-			$redirect = parametre_url(generer_url_ecrire('install'),'etape', _request('etape'));
-		}
+        // 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'));
+        if (defined('_ECRIRE_INSTALL')) {
+            $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);
-			else {
-				echo "<p>"._T('avis_operation_echec') . ' ' . join(' ', $res)."</p>";
-			}
-		}
-	}
+        $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 {
+                echo "<p>"._T('avis_operation_echec') . ' ' . join(' ', $res)."</p>";
+            }
+        }
+    }
 }
 
 /**
@@ -221,17 +221,17 @@ 
  * @return void
  */
 function relance_maj($meta,$table,$redirect=''){
-	include_spip('inc/headers');
-	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);
-	}
-	echo redirige_formulaire($redirect);
-	exit();
+    include_spip('inc/headers');
+    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);
+    }
+    echo redirige_formulaire($redirect);
+    exit();
 }
 
 /**
@@ -244,21 +244,21 @@ 
  * @return
  */
 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;
-	$titre = _T('titre_page_upgrade');
-	$balise_img = charger_filtre('balise_img');
-	$titre .= $balise_img(chemin_image('searching.gif'));
-	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).")");
-	echo "<div style='text-align: left'>\n";
-	ob_flush();flush();
-	$done = true;
+    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;
+    $titre = _T('titre_page_upgrade');
+    $balise_img = charger_filtre('balise_img');
+    $titre .= $balise_img(chemin_image('searching.gif'));
+    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).")");
+    echo "<div style='text-align: left'>\n";
+    ob_flush();flush();
+    $done = true;
 }
 
 define('_UPGRADE_TIME_OUT', 20);
@@ -298,46 +298,46 @@ 
  */
 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');
-	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);
+    # 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');
+    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);
 
-	reset($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 ($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);
-			$n = time() - $time;
-			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);
-		}
-	}
-	$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);
-	return array();
+    reset($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 ($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);
+            $n = time() - $time;
+            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);
+        }
+    }
+    $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);
+    return array();
 }
 
 /**
@@ -359,45 +359,45 @@ 
  * @return int
  */
 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);
-				// 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);
-				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);
-				}
-				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);
-				// en cas d'erreur serieuse, on s'arrete
-				// mais on permet de passer par dessus en rechargeant la page.
-				return $i+1;
-			}
-		}
-	}
-	effacer_meta($meta2, $table);
-	return 0;
+    $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);
+                // 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);
+                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);
+                }
+                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);
+                // en cas d'erreur serieuse, on s'arrete
+                // mais on permet de passer par dessus en rechargeant la page.
+                return $i+1;
+            }
+        }
+    }
+    effacer_meta($meta2, $table);
+    return 0;
 }
 
 
@@ -407,49 +407,49 @@ 
 
 // 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();
+    if (include_spip('base/medias')
+    AND function_exists('creer_base_types_doc'))
+        creer_base_types_doc();
 }
 
 // http://doc.spip.org/@upgrade_test
 function upgrade_test() {
-	sql_drop_table("spip_test", true);
-	sql_create("spip_test", array('a' => 'int'));
-	sql_alter("TABLE spip_test ADD b INT");
-	sql_insertq('spip_test', array('b' => 1), array('field'=>array('b' => 'int')));
-	$result = sql_select('b', "spip_test");
-	// 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; 
-	sql_alter("TABLE spip_test DROP b");
-	return $result;
+    sql_drop_table("spip_test", true);
+    sql_create("spip_test", array('a' => 'int'));
+    sql_alter("TABLE spip_test ADD b INT");
+    sql_insertq('spip_test', array('b' => 1), array('field'=>array('b' => 'int')));
+    $result = sql_select('b', "spip_test");
+    // 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; 
+    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) {
-	if ($test) {
-		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') );
-		}
-		spip_log( "mise a jour de la base en $version","maj."._LOG_INFO_IMPORTANTE);
-	} else {
-		echo _T('alerte_maj_impossible', array('version' => $version));
-		exit;
-	}
+    if ($test) {
+        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') );
+        }
+        spip_log( "mise a jour de la base en $version","maj."._LOG_INFO_IMPORTANTE);
+    } else {
+        echo _T('alerte_maj_impossible', array('version' => $version));
+        exit;
+    }
 }
 
 // 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))
-	);
+    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') );

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

Indentation +40 added lines, -40 removed lines

@@ -19,12 +19,12 @@ 
  * @return int
  */
 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');
+    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');
 
-	spip_log("Verification version SPIP : ".($maj?$maj:"version a jour"),"verifie_maj");
-	return 1;
+    spip_log("Verification version SPIP : ".($maj?$maj:"version a jour"),"verifie_maj");
+    return 1;
 }
 
 
@@ -35,32 +35,32 @@ 
 define('_VERSIONS_LISTE', 'archives.xml');
 
 function info_maj ($dir, $file, $version){
-	include_spip('inc/plugin');
+    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;
-	$page = !file_exists($nom) ? '' : file_get_contents($nom);
-	$page = info_maj_cache($nom, $dir, $page);
+    $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);
-	$page = '';
-	foreach ($m as $v) {
-		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;
-	}
+    // 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);
+    $page = '';
+    foreach ($m as $v) {
+        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;
+    }
 
-	if (!$page) return "";
-	return "<a class='info_maj_spip' href='"._VERSIONS_SERVEUR."$dir' title='$page'>" .
-		_T('nouvelle_version_spip',array('version'=>$page)) .
-	    '</a>';
+    if (!$page) return "";
+    return "<a class='info_maj_spip' href='"._VERSIONS_SERVEUR."$dir' title='$page'>" .
+        _T('nouvelle_version_spip',array('version'=>$page)) .
+        '</a>';
 }
 
 // Verifie que la liste $page des versions dans le fichier $nom est a jour
@@ -70,20 +70,20 @@ 
 
 function info_maj_cache($nom, $dir, $page='')
 {
-	$re = '<archives id="a' . $GLOBALS['meta']["alea_ephemere"] . '">';
-	if (preg_match("/$re/", $page)) return $page;
+    $re = '<archives id="a' . $GLOBALS['meta']["alea_ephemere"] . '">';
+    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;
-	// Placer l'indicateur de fraicheur
-	$page = preg_replace('/^<archives.*?>/', $re, $page);
-	sous_repertoire(_DIR_CACHE_XML);
-	ecrire_fichier($nom, $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;
+    // Placer l'indicateur de fraicheur
+    $page = preg_replace('/^<archives.*?>/', $re, $page);
+    sous_repertoire(_DIR_CACHE_XML);
+    ecrire_fichier($nom, $page);
+    return $page;
 }
 
 ?>

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

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

Indentation +74 added lines, -74 removed lines

@@ -15,38 +15,38 @@ 
 
 // 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)]))
-		return $couleurs_html[$lc];
-	return $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) {
-	$couleurs = _couleur_hex_to_dec($couleur);
+    $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);
+    $couleur = _couleur_dec_to_hex($red, $green, $blue);
 	
-	return $couleur;
+    return $couleur;
 }
 
 // http://doc.spip.org/@couleur_eclaircir
 function couleur_eclaircir ($couleur, $coeff=0.5) {
-	$couleurs = _couleur_hex_to_dec($couleur);
+    $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);
+    $couleur = _couleur_dec_to_hex($red, $green, $blue);
 	
-	return $couleur;
+    return $couleur;
 
 }
 
@@ -55,81 +55,81 @@ 
 // 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;
-	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;
-
-	$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);
-	}
-	if (($select==true) AND ($p!==FALSE)){
-		$class = preg_replace(",\s*no_image_filtrer,","",$class);
-		$img = inserer_attribut($img,'class',$class);
-	}
-	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;
+
+    $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);
+    }
+    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'){
-	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 (!$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 '';
 	
-	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);
+    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);
 }
 
 // http://doc.spip.org/@image_reduire
 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 '';
-
-	$fonction = array('image_reduire', func_get_args());
-	return process_image_reduire($fonction,$img,$taille,$taille_y,$force,$cherche_image,$process);
+    // 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 '';
+
+    $fonction = array('image_reduire', func_get_args());
+    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);
+    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;
+    if ($l > $h) $h = 0;
+    else $l = 0;
 	
-	$img = image_reduire($img, $l, $h, $force);
+    $img = image_reduire($img, $l, $h, $force);
 
-	return $img;
+    return $img;
 }
 
 ?>

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

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(

Indentation +128 added lines, -128 removed lines

@@ -18,160 +18,160 @@
 
 // http://doc.spip.org/@redirige_par_entete
 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]=='?')
-		$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)){
-		include_spip("inc/filtres_mini");
-		$url = url_absolue($url);
-	}
-
-	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 (!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;
+    # 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)){
+        include_spip("inc/filtres_mini");
+        $url = url_absolue($url);
+    }
+
+    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, '&');
 		
-	// ne pas laisser passer n'importe quoi dans l'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)
-		)
-		$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="";
-	} else {
-		@header("Refresh: 0; url=" . $url);
-		$equiv = "<meta http-equiv='Refresh' content='0; url=$url'>";
-	}
-	include_spip('inc/lang');
-	if ($status!=302)
-		http_status($status);
-	echo '<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">',"\n",
-	  html_lang_attributes(),'
+    // ne pas laisser passer n'importe quoi dans l'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)
+        )
+        $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="";
+    } else {
+        @header("Refresh: 0; url=" . $url);
+        $equiv = "<meta http-equiv='Refresh' content='0; url=$url'>";
+    }
+    include_spip('inc/lang');
+    if ($status!=302)
+        http_status($status);
+    echo '<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">',"\n",
+        html_lang_attributes(),'
 <head>',
-	  $equiv,'
+        $equiv,'
 <title>HTTP '.$status.'</title>
 </head>
 <body>
 <h1>HTTP '.$status.'</h1>
 <a href="',
-	  quote_amp($url),
-	  '">',
-	  _T('navigateur_pas_redirige'),
-	  '</a></body></html>';
+        quote_amp($url),
+        '">',
+        _T('navigateur_pas_redirige'),
+        '</a></body></html>';
 
-	spip_log("redirige $status: $url");
+    spip_log("redirige $status: $url");
 
-	exit;
+    exit;
 }
 
 // http://doc.spip.org/@redirige_formulaire
 function redirige_formulaire($url, $equiv = '', $format='message') {
-	if (!_AJAX
-	AND !headers_sent()
-	AND !_request('var_ajax')) {
-		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)) {
-		return array(
-		// on renvoie un lien masque qui sera traite par ajaxCallback.js
-		"<a href='$url' name='ajax_ancre' style='display:none;'>anchor</a>",
-		// et rien dans le message ok
-		'');
-	}
-	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;
-		$url = str_replace('&amp;','&',$url);
-		spip_log("redirige formulaire ajax: $url");
-		include_spip('inc/filtres');
-		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
-			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>';
-	}
+    if (!_AJAX
+    AND !headers_sent()
+    AND !_request('var_ajax')) {
+        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)) {
+        return array(
+        // on renvoie un lien masque qui sera traite par ajaxCallback.js
+        "<a href='$url' name='ajax_ancre' style='display:none;'>anchor</a>",
+        // et rien dans le message ok
+        '');
+    }
+    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;
+        $url = str_replace('&amp;','&',$url);
+        spip_log("redirige formulaire ajax: $url");
+        include_spip('inc/filtres');
+        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
+            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>';
+    }
 }
 
 // http://doc.spip.org/@redirige_url_ecrire
 function redirige_url_ecrire($script='', $args='', $equiv='') {
-	return redirige_par_entete(generer_url_ecrire($script, $args, true), $equiv);
+    return redirige_par_entete(generer_url_ecrire($script, $args, true), $equiv);
 }
 
 // http://doc.spip.org/@http_status
 function http_status($status) {
-	global $REDIRECT_STATUS, $flag_sapi_name;
-	static $status_string = array(
-		200 => '200 OK',
-		204 => '204 No Content',
-		301 => '301 Moved Permanently',
-		302 => '302 Found',
-		304 => '304 Not Modified',
-		401 => '401 Unauthorized',
-		403 => '403 Forbidden',
-		404 => '404 Not Found',
-		503 => '503 Service Unavailable'
-	);
-
-	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]);
+    global $REDIRECT_STATUS, $flag_sapi_name;
+    static $status_string = array(
+        200 => '200 OK',
+        204 => '204 No Content',
+        301 => '301 Moved Permanently',
+        302 => '302 Found',
+        304 => '304 Not Modified',
+        401 => '401 Unauthorized',
+        403 => '403 Forbidden',
+        404 => '404 Not Found',
+        503 => '503 Service Unavailable'
+    );
+
+    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]);
 }
 
 // Retourne ce qui va bien pour que le navigateur ne mette pas la page en cache
 // http://doc.spip.org/@http_no_cache
 function http_no_cache() {
-	if (headers_sent())
-		{ 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
-	// il faudrait aussi pour Safari
-	// header("Cache-Control: post-check=0, pre-check=0", false)
-	// mais ca ne respecte pas
-	// http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.9
-
-	header("Content-Type: text/html; charset=$charset");
-	header("Expires: 0");
-	header("Last-Modified: " .gmdate("D, d M Y H:i:s"). " GMT");
-	header("Cache-Control: no-cache, must-revalidate");
-	header("Pragma: no-cache");
+    if (headers_sent())
+        { 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
+    // il faudrait aussi pour Safari
+    // header("Cache-Control: post-check=0, pre-check=0", false)
+    // mais ca ne respecte pas
+    // http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.9
+
+    header("Content-Type: text/html; charset=$charset");
+    header("Expires: 0");
+    header("Last-Modified: " .gmdate("D, d M Y H:i:s"). " GMT");
+    header("Cache-Control: no-cache, must-revalidate");
+    header("Pragma: no-cache");
 }
 
 ?>

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");
 }

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();

Indentation +119 added lines, -119 removed lines

@@ -21,36 +21,36 @@ 
 // http://doc.spip.org/@inc_meta_dist
 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())
-	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))
-		$GLOBALS[$table] = array();
-	// sinon lire en base
-	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']))
-	AND $GLOBALS[$table]
-	AND (time() > _RENOUVELLE_ALEA + $GLOBALS['meta']['alea_ephemere_date'])) {
-		// si on n'a pas l'acces en ecriture sur le cache,
-		// ne pas renouveller l'alea sinon le cache devient faux
-		if (supprimer_fichier($cache)) {
-			include_spip('inc/acces');
-			renouvelle_alea();
-			$new = false; 
-		} 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);
+    // 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())
+    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))
+        $GLOBALS[$table] = array();
+    // sinon lire en base
+    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']))
+    AND $GLOBALS[$table]
+    AND (time() > _RENOUVELLE_ALEA + $GLOBALS['meta']['alea_ephemere_date'])) {
+        // si on n'a pas l'acces en ecriture sur le cache,
+        // ne pas renouveller l'alea sinon le cache devient faux
+        if (supprimer_fichier($cache)) {
+            include_spip('inc/acces');
+            renouvelle_alea();
+            $new = false; 
+        } 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);
 }
 
 // fonctions aussi appelees a l'install ==> spip_query en premiere requete 
@@ -59,101 +59,101 @@ 
 // http://doc.spip.org/@lire_metas
 function lire_metas($table='meta') {
 
-	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'];
+    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'];
         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);
-
-		// noter cette table de configuration dans les meta de SPIP
-		if ($table!=='meta') {
-			$liste = unserialize($GLOBALS['meta']['tables_config']);
-			if (!$liste)
-				$liste = array();
-			if (!in_array($table, $liste)) {
-				$liste[] = $table;
-				ecrire_meta('tables_config', serialize($liste));
-			}
-		}
-	}
-	return $GLOBALS[$table];
+        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);
+
+        // noter cette table de configuration dans les meta de SPIP
+        if ($table!=='meta') {
+            $liste = unserialize($GLOBALS['meta']['tables_config']);
+            if (!$liste)
+                $liste = array();
+            if (!in_array($table, $liste)) {
+                $liste[] = $table;
+                ecrire_meta('tables_config', serialize($liste));
+            }
+        }
+    }
+    return $GLOBALS[$table];
 }
 
 // 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'){
-	$file = cache_meta($table);
-	if (!$antidate OR !@touch($file, $antidate)) {
-		$r = $GLOBALS[$table];
-		unset($r['alea_ephemere']);
-		unset($r['alea_ephemere_ancien']);
-		// le secret du site est utilise pour encoder les contextes ajax que l'on considere fiables
-		// 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;
-		ecrire_fichier_securise($file, serialize($r));
-	}
+    $file = cache_meta($table);
+    if (!$antidate OR !@touch($file, $antidate)) {
+        $r = $GLOBALS[$table];
+        unset($r['alea_ephemere']);
+        unset($r['alea_ephemere_ancien']);
+        // le secret du site est utilise pour encoder les contextes ajax que l'on considere fiables
+        // 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;
+        ecrire_fichier_securise($file, serialize($r));
+    }
 }
 
 // http://doc.spip.org/@effacer_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'");
-	unset($GLOBALS[$table][$nom]);
-	if (!isset($touch[$table])) {touch_meta($antidate, $table); $touch[$table] = false;}
+    // 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'");
+    unset($GLOBALS[$table][$nom]);
+    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') {
 
-	static $touch = array();
-	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
-	if (!$res) {
-		$GLOBALS[$table][$nom] = $valeur;
-		return;
-	}
-	$row = sql_fetch($res);
+    static $touch = array();
+    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
+    if (!$res) {
+        $GLOBALS[$table][$nom] = $valeur;
+        return;
+    }
+    $row = sql_fetch($res);
     sql_free($res);
 
-	// 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;
-
-	$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);}
-	$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));
-	} else {
-		sql_insertq('spip_' . $table, $r);
-	}
-	if (!isset($touch[$table])) {touch_meta($antidate, $table); $touch[$table] = false;}
+    // 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;
+
+    $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);}
+    $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));
+    } else {
+        sql_insertq('spip_' . $table, $r);
+    }
+    if (!isset($touch[$table])) {touch_meta($antidate, $table); $touch[$table] = false;}
 }
 
 function cache_meta($table='meta')
 {
-	return ($table=='meta') ? _FILE_META : (_DIR_CACHE . $table . '.php');
+    return ($table=='meta') ? _FILE_META : (_DIR_CACHE . $table . '.php');
 }
 
 /**
@@ -161,14 +161,14 @@ 
  * @param string $table
  */
 function installer_table_meta($table) {
-	$trouver_table = charger_fonction('trouver_table','base');
-	if (!$trouver_table("spip_$table")) {
-		include_spip('base/auxiliaires');
-		include_spip('base/create');
-		creer_ou_upgrader_table("spip_$table", $GLOBALS['tables_auxiliaires']['spip_meta'], false, false);
-		$trouver_table('');
-	}
-	lire_metas($table);
+    $trouver_table = charger_fonction('trouver_table','base');
+    if (!$trouver_table("spip_$table")) {
+        include_spip('base/auxiliaires');
+        include_spip('base/create');
+        creer_ou_upgrader_table("spip_$table", $GLOBALS['tables_auxiliaires']['spip_meta'], false, false);
+        $trouver_table('');
+    }
+    lire_metas($table);
 }
 
 /**
@@ -179,14 +179,14 @@ 
  * @param bool $force
  */
 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('');
-	}
+    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('');
+    }
 }
 ?>

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('');
 	}
 }

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;

Indentation +89 added lines, -89 removed lines

@@ -21,106 +21,106 @@
 // 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 ($aff_racine) {
-		$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
-		$idom3 = $idom . "_selection";
-
-		$onClick = "jQuery(this).parent().addClass('on');jQuery('#choix_parent_principal .on').removeClass('on'); aff_selection(0, '$idom3', '$info', event);return false;";
-
-		$ondbClick = strtr(str_replace("'", "’",
-				str_replace('"', """,
-					textebrut(_T('info_racine_site')))),
-				"\n\r", "  ");
-
-		$js_func = $do . '_selection_titre';
-		$ondbClick = "$js_func('$ondbClick',0,'selection_rubrique','id_parent');";
-
-		$aff_racine = "<div class='petite-racine item'>"
-		. "<a href='#'"
-			. "onclick=\""
-			. $onClick
-			. "\"\nondbclick=\""
-			. $ondbClick
-			. $onClick
-		. "\">"
-		. _T("info_racine_site")
-		. "</a></div>";
-	}
-
-	$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);
-
-	// url completee par la fonction JS onkeypress_rechercher
-	$url = generer_url_ecrire('rechercher', "exclus=$exclus&rac=$idom&do=$do&type=");
-	return construire_selectionner_hierarchie($idom, $plonger_r, $aff_racine, $url, 'id_parent', $url_init);
+    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";
+
+        $onClick = "jQuery(this).parent().addClass('on');jQuery('#choix_parent_principal .on').removeClass('on'); aff_selection(0, '$idom3', '$info', event);return false;";
+
+        $ondbClick = strtr(str_replace("'", "&#8217;",
+                str_replace('"', "&#34;",
+                    textebrut(_T('info_racine_site')))),
+                "\n\r", "  ");
+
+        $js_func = $do . '_selection_titre';
+        $ondbClick = "$js_func('$ondbClick',0,'selection_rubrique','id_parent');";
+
+        $aff_racine = "<div class='petite-racine item'>"
+        . "<a href='#'"
+            . "onclick=\""
+            . $onClick
+            . "\"\nondbclick=\""
+            . $ondbClick
+            . $onClick
+        . "\">"
+        . _T("info_racine_site")
+        . "</a></div>";
+    }
+
+    $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);
+
+    // url completee par la fonction JS onkeypress_rechercher
+    $url = generer_url_ecrire('rechercher', "exclus=$exclus&rac=$idom&do=$do&type=");
+    return construire_selectionner_hierarchie($idom, $plonger_r, $aff_racine, $url, 'id_parent', $url_init);
 }
 
 // http://doc.spip.org/@construire_selectionner_hierarchie
 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;
-	$idom6 = $idom."_fonc";
-
-	return "<div id='$idom'>"
-	. "<a id='$idom6' style='visibility: hidden;'"
-	. ($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'")
-	. ""
-	. "<input style='width: 100px;float:$spip_lang_right;' type='search' id='$idom1'"
-	  // eliminer Return car il provoque la soumission (balise unique)
-	  // ce serait encore mieux de ne le faire que s'il y a encore plusieurs
-	  // resultats retournes par la recherche
-	. "\nonkeypress=\"k=event.keyCode;if (k==13 || k==3){return false;}\""
-	  // lancer la recherche apres le filtrage ci-dessus
-	. "\nonkeyup=\"return onkey_rechercher(this.value,"
-	  // la destination de la recherche
-	. "'$idom4'"
+    global $spip_lang_right;
+
+    $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'" : '')
+    . "></a>"
+    . "<div class='recherche_rapide_parent'>"
+    . http_img_pack("searching.gif", "*", "style='visibility: hidden;float:$spip_lang_right' id='$idom5'")
+    . ""
+    . "<input style='width: 100px;float:$spip_lang_right;' type='search' id='$idom1'"
+        // eliminer Return car il provoque la soumission (balise unique)
+        // ce serait encore mieux de ne le faire que s'il y a encore plusieurs
+        // resultats retournes par la recherche
+    . "\nonkeypress=\"k=event.keyCode;if (k==13 || k==3){return false;}\""
+        // lancer la recherche apres le filtrage ci-dessus
+    . "\nonkeyup=\"return onkey_rechercher(this.value,"
+        // la destination de la recherche
+    . "'$idom4'"
 #	. "this.parentNode.parentNode.parentNode.parentNode.nextSibling.firstChild.id"
-	. ",'"
-	  // l'url effectuant la recherche
-	. $url
-	. "',"	
-	  // le noeud contenant un gif anime 
-	  // . "'idom5'"
-	. "this.parentNode.previousSibling.firstChild"
-	. ",'"
-	  // la valeur de l'attribut Name a remplir
-	.  $name
-	. "','"
-	  // noeud invisible memorisant l'URL initiale (pour re-initialisation)
-	. $idom6
-	. "')\"" 
-	. " />"
-	. "\n</div>"
-	. ($racine?"<div>$racine</div>":"")
-	. "<div id='"
-	.  $idom2
-	.  "'><div id='$idom4'"
-	. " class=''>"
-	. $liste
-	. "</div></div>\n<div id='$idom3'></div></div>\n";
+    . ",'"
+        // l'url effectuant la recherche
+    . $url
+    . "',"	
+        // le noeud contenant un gif anime 
+        // . "'idom5'"
+    . "this.parentNode.previousSibling.firstChild"
+    . ",'"
+        // la valeur de l'attribut Name a remplir
+    .  $name
+    . "','"
+        // noeud invisible memorisant l'URL initiale (pour re-initialisation)
+    . $idom6
+    . "')\"" 
+    . " />"
+    . "\n</div>"
+    . ($racine?"<div>$racine</div>":"")
+    . "<div id='"
+    .  $idom2
+    .  "'><div id='$idom4'"
+    . " class=''>"
+    . $liste
+    . "</div></div>\n<div id='$idom3'></div></div>\n";
 }
 
 // http://doc.spip.org/@mini_hier
 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");
+    $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");
 }
 
 ?>

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");
 }
 
 ?>

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");
 }