spip / SPIP

ecrire/base/dump.php

Doc Comments +1 added lines, -1 removed lines

@@ -311,7 +311,7 @@
  * peut etre utilise pour l'import depuis xml,
  * ou la copie de base a base (mysql<->sqlite par exemple)
  *
- * @param array $tables
+ * @param string[] $tables
  * @param array $exclure_tables
  * @param string $serveur
  */

Spacing +36 added lines, -36 removed lines

@@ -30,7 +30,7 @@ 
 // donc il faut l'inclure "en globals"
 if ($f = find_in_path('mes_fonctions.php')) {
 	global $dossier_squelettes;
-	@include_once(_ROOT_CWD . $f);
+	@include_once(_ROOT_CWD.$f);
 }
 
 if (@is_readable(_CACHE_PLUGINS_FCT)) {
@@ -48,7 +48,7 @@ 
  * @return string
  **/
 function base_dump_meta_name($rub) {
-	return $meta = "status_dump_{$rub}_" . abs($GLOBALS['visiteur_session']['id_auteur']);
+	return $meta = "status_dump_{$rub}_".abs($GLOBALS['visiteur_session']['id_auteur']);
 }
 
 /**
@@ -94,7 +94,7 @@ 
 	$connexion = $GLOBALS['connexions'][$serveur ? $serveur : 0];
 	$prefixe = $connexion['prefixe'];
 
-	$p = '/^' . $prefixe . '/';
+	$p = '/^'.$prefixe.'/';
 	$res = $tables;
 	foreach (sql_alltable(null, $serveur) as $t) {
 		if (preg_match($p, $t)) {
@@ -353,8 +353,8 @@ 
 function base_vider_tables_destination_copie($tables, $exclure_tables = array(), $serveur = '') {
 	$trouver_table = charger_fonction('trouver_table', 'base');
 
-	spip_log('Vider ' . count($tables) . " tables sur serveur '$serveur' : " . join(', ', $tables),
-		'base.' . _LOG_INFO_IMPORTANTE);
+	spip_log('Vider '.count($tables)." tables sur serveur '$serveur' : ".join(', ', $tables),
+		'base.'._LOG_INFO_IMPORTANTE);
 	foreach ($tables as $table) {
 		if (!in_array($table, $exclure_tables)) {
 			// sur le serveur principal, il ne faut pas supprimer l'auteur loge !
@@ -391,14 +391,14 @@ 
 function base_conserver_copieur($move = true, $serveur = '') {
 	// s'asurer qu'on a pas deja fait la manip !
 	if ($GLOBALS['visiteur_session']['id_auteur'] > 0 and sql_countsel("spip_auteurs", "id_auteur>0")) {
-		spip_log('Conserver copieur dans id_auteur=' . $GLOBALS['visiteur_session']['id_auteur'] . " pour le serveur '$serveur'",
-			'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log('Conserver copieur dans id_auteur='.$GLOBALS['visiteur_session']['id_auteur']." pour le serveur '$serveur'",
+			'dump.'._LOG_INFO_IMPORTANTE);
 		sql_delete("spip_auteurs", "id_auteur<0", $serveur);
 		if ($move) {
 			sql_updateq('spip_auteurs', array('id_auteur' => -$GLOBALS['visiteur_session']['id_auteur']),
-				"id_auteur=" . intval($GLOBALS['visiteur_session']['id_auteur']), array(), $serveur);
+				"id_auteur=".intval($GLOBALS['visiteur_session']['id_auteur']), array(), $serveur);
 		} else {
-			$row = sql_fetsel('*', 'spip_auteurs', 'id_auteur=' . $GLOBALS['visiteur_session']['id_auteur'], '', '', '', '',
+			$row = sql_fetsel('*', 'spip_auteurs', 'id_auteur='.$GLOBALS['visiteur_session']['id_auteur'], '', '', '', '',
 				$serveur);
 			$row['id_auteur'] = -$GLOBALS['visiteur_session']['id_auteur'];
 			sql_insertq('spip_auteurs', $row, array(), $serveur);
@@ -421,15 +421,15 @@ 
 	// rien a faire si ce n'est pas le serveur principal !
 	if ($serveur == '') {
 		if (sql_countsel("spip_auteurs", "id_auteur>0")) {
-			spip_log("Detruire copieur id_auteur<0 pour le serveur '$serveur'", 'dump.' . _LOG_INFO_IMPORTANTE);
+			spip_log("Detruire copieur id_auteur<0 pour le serveur '$serveur'", 'dump.'._LOG_INFO_IMPORTANTE);
 			sql_delete("spip_auteurs", "id_auteur<0", $serveur);
 		} else {
 			spip_log("Restaurer copieur id_auteur<0 pour le serveur '$serveur' (aucun autre auteur en base)",
-				'dump.' . _LOG_INFO_IMPORTANTE);
+				'dump.'._LOG_INFO_IMPORTANTE);
 			sql_update('spip_auteurs', array('id_auteur' => '-id_auteur'), "id_auteur<0");
 		}
 	} else {
-		spip_log("Pas de destruction copieur sur serveur '$serveur'", 'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log("Pas de destruction copieur sur serveur '$serveur'", 'dump.'._LOG_INFO_IMPORTANTE);
 	}
 }
 
@@ -463,19 +463,19 @@ 
 			}
 		} else {
 			sql_drop_table($table, '', $serveur_dest);
-			spip_log("drop table '$table' sur serveur '$serveur_dest'", 'dump.' . _LOG_INFO_IMPORTANTE);
+			spip_log("drop table '$table' sur serveur '$serveur_dest'", 'dump.'._LOG_INFO_IMPORTANTE);
 		}
 		$desc_dest = false;
 	}
 	// si la table n'existe pas dans la destination, la creer a l'identique !
 	if (!$desc_dest) {
-		spip_log("creation '$table' sur serveur '$serveur_dest'", 'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log("creation '$table' sur serveur '$serveur_dest'", 'dump.'._LOG_INFO_IMPORTANTE);
 		include_spip('base/create');
 		creer_ou_upgrader_table($table, $desc, 'auto', $upgrade, $serveur_dest);
 		$desc_dest = sql_showtable($table, true, $serveur_dest);
 	}
 	if (!$desc_dest) {
-		spip_log("Erreur creation '$table' sur serveur '$serveur_dest'" . var_export($desc, 1), 'dump.' . _LOG_ERREUR);
+		spip_log("Erreur creation '$table' sur serveur '$serveur_dest'".var_export($desc, 1), 'dump.'._LOG_ERREUR);
 	}
 
 	return $desc_dest;
@@ -534,16 +534,16 @@ 
 	$racine_fonctions = (isset($options['racine_fonctions_dest']) ? $options['racine_fonctions_dest'] : 'base');
 	$data_pool = (isset($options['data_pool']) ? $options['data_pool'] : 50 * 1024);
 
-	spip_log("Copier " . count($tables) . " tables de '$serveur_source' vers '$serveur_dest'",
-		'dump.' . _LOG_INFO_IMPORTANTE);
+	spip_log("Copier ".count($tables)." tables de '$serveur_source' vers '$serveur_dest'",
+		'dump.'._LOG_INFO_IMPORTANTE);
 
 	if (!$inserer_copie = charger_fonction($fonction_base_inserer, $racine_fonctions, true)) {
-		spip_log("Fonction '{$racine_fonctions}_$fonction_base_inserer' inconnue. Abandon", 'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log("Fonction '{$racine_fonctions}_$fonction_base_inserer' inconnue. Abandon", 'dump.'._LOG_INFO_IMPORTANTE);
 
 		return true; // echec mais on a fini, donc true
 	}
 	if (!$preparer_table_dest = charger_fonction('preparer_table_dest', $racine_fonctions, true)) {
-		spip_log("Fonction '{$racine_fonctions}_$preparer_table_dest' inconnue. Abandon", 'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log("Fonction '{$racine_fonctions}_$preparer_table_dest' inconnue. Abandon", 'dump.'._LOG_INFO_IMPORTANTE);
 
 		return true; // echec mais on a fini, donc true
 	}
@@ -565,7 +565,7 @@ 
 			true)
 		) {
 			spip_log("Fonction '{$racine_fonctions}_vider_tables_destination_copie' inconnue. Abandon",
-				'dump.' . _LOG_INFO_IMPORTANTE);
+				'dump.'._LOG_INFO_IMPORTANTE);
 
 			return true; // echec mais on a fini, donc true
 		}
@@ -583,7 +583,7 @@ 
 		$tables = array_diff($tables, array('spip_meta'));
 		$tables[] = 'spip_meta';
 	}
-	spip_log("Tables a copier :" . implode(", ", $tables), 'dump.' . _LOG_INFO);
+	spip_log("Tables a copier :".implode(", ", $tables), 'dump.'._LOG_INFO);
 
 	$trouver_table = charger_fonction('trouver_table', 'base');
 
@@ -637,7 +637,7 @@ 
 						// mais si ca renvoie false c'est une erreur fatale => abandon
 						if ($inserer_copie($table, $rows, $desc_dest, $serveur_dest) === false) {
 							// forcer la sortie, charge a l'appelant de gerer l'echec
-							spip_log("Erreur fatale dans $inserer_copie table $table", "dump" . _LOG_ERREUR);
+							spip_log("Erreur fatale dans $inserer_copie table $table", "dump"._LOG_ERREUR);
 							$status['errors'][] = "Erreur fatale  lors de la copie de la table $table";
 							ecrire_fichier($status_file, serialize($status));
 
@@ -652,7 +652,7 @@ 
 					if ($n == $status['tables_copiees'][$table]) {
 						break;
 					}
-					spip_log("recopie $table " . $status['tables_copiees'][$table], 'dump.' . _LOG_INFO_IMPORTANTE);
+					spip_log("recopie $table ".$status['tables_copiees'][$table], 'dump.'._LOG_INFO_IMPORTANTE);
 					if ($callback_progression) {
 						$callback_progression($status['tables_copiees'][$table], 0, $table);
 					}
@@ -663,43 +663,43 @@ 
 				}
 				if ($drop_source) {
 					sql_drop_table($table, '', $serveur_source);
-					spip_log("drop $table sur serveur source '$serveur_source'", 'dump.' . _LOG_INFO_IMPORTANTE);
+					spip_log("drop $table sur serveur source '$serveur_source'", 'dump.'._LOG_INFO_IMPORTANTE);
 				}
 				$status['tables_copiees'][$table] = ($status['tables_copiees'][$table] ? -$status['tables_copiees'][$table] : "zero");
 				ecrire_fichier($status_file, serialize($status));
-				spip_log("tables_recopiees " . implode(',', array_keys($status['tables_copiees'])), 'dump.' . _LOG_INFO);
+				spip_log("tables_recopiees ".implode(',', array_keys($status['tables_copiees'])), 'dump.'._LOG_INFO);
 				if ($callback_progression) {
 					$callback_progression($status['tables_copiees'][$table], $status['tables_copiees'][$table], $table);
 				}
 			} else {
 				if ($status['tables_copiees'][$table] < 0) {
-					spip_log("Table $table deja copiee : " . $status['tables_copiees'][$table], "dump." . _LOG_INFO);
+					spip_log("Table $table deja copiee : ".$status['tables_copiees'][$table], "dump."._LOG_INFO);
 				}
 				if ($callback_progression) {
 					$callback_progression(0, $status['tables_copiees'][$table],
-						"$table" . ((is_numeric($status['tables_copiees'][$table]) and $status['tables_copiees'][$table] >= 0) ? "[Echec]" : ""));
+						"$table".((is_numeric($status['tables_copiees'][$table]) and $status['tables_copiees'][$table] >= 0) ? "[Echec]" : ""));
 				}
 			}
 		} else {
 			$status['errors'][] = "Impossible de lire la description de la table $table";
 			ecrire_fichier($status_file, serialize($status));
-			spip_log("Impossible de lire la description de la table $table", "dump." . _LOG_ERREUR);
+			spip_log("Impossible de lire la description de la table $table", "dump."._LOG_ERREUR);
 		}
 	}
 
 	// si le nombre de tables envoyees n'est pas egal au nombre de tables demandees
 	// abandonner
 	if (count($status['tables_copiees']) < count($tables)) {
-		spip_log("Nombre de tables copiees incorrect : " . count($status['tables_copiees']) . "/" . count($tables),
-			"dump." . _LOG_ERREUR);
-		$status['errors'][] = "Nombre de tables copiees incorrect : " . count($status['tables_copiees']) . "/" . count($tables);
+		spip_log("Nombre de tables copiees incorrect : ".count($status['tables_copiees'])."/".count($tables),
+			"dump."._LOG_ERREUR);
+		$status['errors'][] = "Nombre de tables copiees incorrect : ".count($status['tables_copiees'])."/".count($tables);
 		ecrire_fichier($status_file, serialize($status));
 	}
 
 	if ($detruire_copieur_si_besoin = charger_fonction('detruire_copieur_si_besoin', $racine_fonctions, true)) {
 		$detruire_copieur_si_besoin($serveur_dest);
 	} else {
-		spip_log("Fonction '{$racine_fonctions}_detruire_copieur_si_besoin' inconnue.", 'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log("Fonction '{$racine_fonctions}_detruire_copieur_si_besoin' inconnue.", 'dump.'._LOG_INFO_IMPORTANTE);
 	}
 
 	// OK, copie complete
@@ -723,8 +723,8 @@ 
 	$r = sql_insertq_multi($table, $rows, $desc_dest, $serveur_dest);
 	$nb = sql_countsel($table, '', '', '', $serveur_dest);
 	if ($nb - $nb1 < count($rows)) {
-		spip_log("base_inserer_copie : " . ($nb - $nb1) . " insertions au lieu de " . count($rows) . ". On retente 1 par 1",
-			"dump" . _LOG_INFO_IMPORTANTE);
+		spip_log("base_inserer_copie : ".($nb - $nb1)." insertions au lieu de ".count($rows).". On retente 1 par 1",
+			"dump"._LOG_INFO_IMPORTANTE);
 		foreach ($rows as $row) {
 			// si l'enregistrement est deja en base, ca fera un echec ou un doublon
 			$r = sql_insertq($table, $row, $desc_dest, $serveur_dest);
@@ -733,8 +733,8 @@ 
 		$r = 0;
 		$nb = sql_countsel($table, '', '', '', $serveur_dest);
 		if ($nb - $nb1 < count($rows)) {
-			spip_log("base_inserer_copie : " . ($nb - $nb1) . " insertions au lieu de " . count($rows) . " apres insertion 1 par 1",
-				"dump" . _LOG_ERREUR);
+			spip_log("base_inserer_copie : ".($nb - $nb1)." insertions au lieu de ".count($rows)." apres insertion 1 par 1",
+				"dump"._LOG_ERREUR);
 			$r = false;
 		}
 	}

ecrire/inc/cookie.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
  *     Chemin sur lequel le cookie sera disponible
  * @param string $domain
  *     Domaine à partir duquel le cookie est disponible
- * @param bool $secure
+ * @param string|boolean $secure
  *     cookie sécurisé ou non ?
  * @return bool
  *     true si le cookie a été posé, false sinon.

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
 	// liste des cookies en httponly (a passer en define si besoin)
 	$httponly = in_array($name, explode(' ', 'spip_session'));
 
-	$name = preg_replace('/^spip_/', $GLOBALS['cookie_prefix'] . '_', $name);
+	$name = preg_replace('/^spip_/', $GLOBALS['cookie_prefix'].'_', $name);
 	if ($path == 'AUTO') {
 		$path = defined('_COOKIE_PATH') ? _COOKIE_PATH : preg_replace(',^\w+://[^/]*,', '', url_de_base());
 	}
@@ -123,7 +123,7 @@ 
 	}
 	foreach ($_COOKIE as $name => $value) {
 		if (substr($name, 0, $prefix_long) == $cookie_prefix) {
-			$spipname = preg_replace('/^' . $cookie_prefix . '_/', 'spip_', $name);
+			$spipname = preg_replace('/^'.$cookie_prefix.'_/', 'spip_', $name);
 			$_COOKIE[$spipname] = $value;
 			$GLOBALS[$spipname] = $value;
 		}

ecrire/inc/modifier.php

Doc Comments +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
  *     Couples champ/valeur à modifier
  * @param string $serveur
  *     Nom du connecteur à la base de données
- * @return bool|string
+ * @return string|false
  *     - false  : Aucune modification, aucun champ n'est à modifier
  *     - chaîne vide : Vide si tout s'est bien passé
  *     - chaîne : Texte d'un message d'erreur

Spacing +10 added lines, -10 removed lines

@@ -113,7 +113,7 @@ 
 
 	// Appels incomplets (sans $c)
 	if (!is_array($c)) {
-		spip_log('erreur appel objet_modifier_champs(' . $objet . '), manque $c');
+		spip_log('erreur appel objet_modifier_champs('.$objet.'), manque $c');
 
 		return _T('erreur_technique_enregistrement_impossible');
 	}
@@ -181,7 +181,7 @@ 
 
 	// Verifier si les mises a jour sont pertinentes, datees, en conflit etc
 	include_spip('inc/editer');
-	if (!isset($options['data']) or is_null($options['data'])){
+	if (!isset($options['data']) or is_null($options['data'])) {
 		$options['data'] = &$_POST;
 	}
 	$conflits = controler_md5($champs, $options['data'], $objet, $id_objet, $serveur);
@@ -199,7 +199,7 @@ 
 				$id_rubrique = 0;
 				if (isset($desc['field']['id_rubrique'])) {
 					$parent = ($objet == 'rubrique') ? 'id_parent' : 'id_rubrique';
-					$id_rubrique = sql_getfetsel($parent, $spip_table_objet, "$id_table_objet=" . intval($id_objet));
+					$id_rubrique = sql_getfetsel($parent, $spip_table_objet, "$id_table_objet=".intval($id_objet));
 				}
 				$instituer_langue_objet = charger_fonction('instituer_langue_objet', 'action');
 				$champs['lang'] = $instituer_langue_objet($objet, $id_objet, $id_rubrique, $changer_lang, $serveur);
@@ -220,10 +220,10 @@ 
 		}
 
 		// allez on commit la modif
-		sql_updateq($spip_table_objet, $champs, "$id_table_objet=" . intval($id_objet), $serveur);
+		sql_updateq($spip_table_objet, $champs, "$id_table_objet=".intval($id_objet), $serveur);
 
 		// on verifie si elle est bien passee
-		$moof = sql_fetsel(array_keys($champs), $spip_table_objet, "$id_table_objet=" . intval($id_objet), array(), array(),
+		$moof = sql_fetsel(array_keys($champs), $spip_table_objet, "$id_table_objet=".intval($id_objet), array(), array(),
 			'', array(), $serveur);
 		// si difference entre les champs, reperer les champs mal enregistres
 		if ($moof != $champs) {
@@ -250,11 +250,11 @@ 
 			// si un champ n'a pas ete correctement enregistre, loger et retourner une erreur
 			// c'est un cas exceptionnel
 			if (count($liste)) {
-				spip_log("Erreur enregistrement en base $objet/$id_objet champs :" . var_export($conflits, true),
-					'modifier.' . _LOG_CRITIQUE);
+				spip_log("Erreur enregistrement en base $objet/$id_objet champs :".var_export($conflits, true),
+					'modifier.'._LOG_CRITIQUE);
 
 				return _T('erreur_technique_enregistrement_champs',
-					array('champs' => "<i>'" . implode("'</i>,<i>'", $liste) . "'</i>"));
+					array('champs' => "<i>'".implode("'</i>,<i>'", $liste)."'</i>"));
 			}
 		}
 
@@ -291,8 +291,8 @@ 
 	// message a affiner :-)
 	include_spip('inc/filtres_mini');
 	$qui = isset($GLOBALS['visiteur_session']['nom']) and $GLOBALS['visiteur_session']['nom'] ? $GLOBALS['visiteur_session']['nom'] : $GLOBALS['ip'];
-	journal(_L($qui . ' a &#233;dit&#233; l&#8217;' . $objet . ' ' . $id_objet . ' (' . join('+',
-			array_diff(array_keys($champs), array('date_modif'))) . ')'), array(
+	journal(_L($qui.' a &#233;dit&#233; l&#8217;'.$objet.' '.$id_objet.' ('.join('+',
+			array_diff(array_keys($champs), array('date_modif'))).')'), array(
 		'faire' => 'modifier',
 		'quoi' => $objet,
 		'id' => $id_objet

ecrire/auth/sha256.inc.php

Doc Comments +2 added lines, -2 removed lines

@@ -328,7 +328,7 @@ 
 
 		/** Array split
 		 *
-		 * @param array $a
+		 * @param integer[] $a
 		 * @param int $n
 		 * @return array
 		 **/
@@ -352,7 +352,7 @@ 
 		/**
 		 * Process and return the hash.
 		 *
-		 * @param $str Input string to hash
+		 * @param string $str Input string to hash
 		 * @param $ig_func Option param to ignore checking for php > 5.1.2
 		 * @return string Hexadecimal representation of the message digest
 		 */

Spacing +114 added lines, -114 removed lines

@@ -98,7 +98,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;
 		}
 
@@ -116,15 +116,15 @@ 
 
 			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;
+			$r = $x + $y;
 
 			if ($r >= $n) {
 				while ($r >= $n) {
@@ -132,7 +132,7 @@ 
 				}
 			}
 
-			return (int)$r;
+			return (int) $r;
 		}
 
 		/**
@@ -145,23 +145,23 @@ 
 		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;
 		}
 
 		/** ROTR
@@ -170,7 +170,7 @@ 
 		 * @param int $n
 		 * @return int
 		 */
-		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)); }
 
 		/** Ch
 		 *
@@ -195,28 +195,28 @@ 
 		 * @param int $x
 		 * @return int
 		 */
-		function Sigma0($x) { return (int)($this->ROTR($x, 2) ^ $this->ROTR($x, 13) ^ $this->ROTR($x, 22)); }
+		function Sigma0($x) { return (int) ($this->ROTR($x, 2) ^ $this->ROTR($x, 13) ^ $this->ROTR($x, 22)); }
 
 		/** Sigma1
 		 *
 		 * @param int $x
 		 * @return int
 		 */
-		function Sigma1($x) { return (int)($this->ROTR($x, 6) ^ $this->ROTR($x, 11) ^ $this->ROTR($x, 25)); }
+		function Sigma1($x) { return (int) ($this->ROTR($x, 6) ^ $this->ROTR($x, 11) ^ $this->ROTR($x, 25)); }
 
 		/** Sigma_0
 		 *
 		 * @param int $x
 		 * @return int
 		 */
-		function sigma_0($x) { return (int)($this->ROTR($x, 7) ^ $this->ROTR($x, 18) ^ $this->SHR($x, 3)); }
+		function sigma_0($x) { return (int) ($this->ROTR($x, 7) ^ $this->ROTR($x, 18) ^ $this->SHR($x, 3)); }
 
 		/** Sigma_1
 		 *
 		 * @param int $x
 		 * @return int
 		 */
-		function sigma_1($x) { return (int)($this->ROTR($x, 17) ^ $this->ROTR($x, 19) ^ $this->SHR($x, 10)); }
+		function sigma_1($x) { return (int) ($this->ROTR($x, 17) ^ $this->ROTR($x, 19) ^ $this->SHR($x, 10)); }
 
 		/** String 2 ord UTF8
 		 *
@@ -271,23 +271,23 @@ 
 
 					return $h;
 				} else {
-					if ($h <= 0xDF && $index < $len-1) {
+					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) {
+						if ($h <= 0xEF && $index < $len - 2) {
 							$bytes = 3;
 
-							return ($h & 0x0F) << 12 | (ord($c{$index+1}) & 0x3F) << 6
-							| (ord($c{$index+2}) & 0x3F);
+							return ($h & 0x0F) << 12 | (ord($c{$index + 1}) & 0x3F) << 6
+							| (ord($c{$index + 2}) & 0x3F);
 						} else {
-							if ($h <= 0xF4 && $index < $len-3) {
+							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);
+								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;
@@ -309,19 +309,19 @@ 
 		function string2binint($str, $npad = 512) {
 			$bin = array();
 			$ords = $this->string2ordUTF8($str, $this->bytesString);
-			$npad = $npad/$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;
+			$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) {
+			$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[$i >> 5] |= ($ords[$i / $this->bytesString] & $mask) << (24 - $i % 32);
 			}
-			$bin[] = $length*$this->bytesString;
+			$bin[] = $length * $this->bytesString;
 
 			return $bin;
 		}
@@ -357,8 +357,8 @@ 
 		 * @return string Hexadecimal representation of the message digest
 		 */
 		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) {
@@ -378,70 +378,70 @@ 
 			 *  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) 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
@@ -451,14 +451,14 @@ 
 			$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);
@@ -467,14 +467,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);
@@ -504,12 +504,12 @@ 
 
 				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)),

ecrire/inc/filtres_images_lib_mini.php

Doc Comments +3 added lines, -3 removed lines

@@ -83,7 +83,7 @@ 
  *
  * @param bool|string $stat
  *     true, false ou le statut déjà défini si traitements enchaînés.
- * @return bool
+ * @return false|null
  *     true si il faut supprimer le fichier temporaire ; false sinon.
  */
 function statut_effacer_images_temporaires($stat) {
@@ -554,7 +554,7 @@ 
  *
  * @uses statut_effacer_images_temporaires()
  *
- * @param ressource $img
+ * @param resource $img
  *     Une ressource de type Image GD.
  * @param array $valeurs
  *     Un tableau des informations (tailles, traitement, path...) accompagnant
@@ -1400,7 +1400,7 @@ 
 	/**
 	 * Transforme une ressource GD en image au format ICO
 	 *
-	 * @param array $gd_image_array
+	 * @param Ressource[] $gd_image_array
 	 *     Tableau de ressources d'images GD
 	 * @return string
 	 *     Image au format ICO

Spacing +59 added lines, -59 removed lines

@@ -42,13 +42,13 @@ 
 	$blue = dechex($blue);
 
 	if (strlen($red) == 1) {
-		$red = "0" . $red;
+		$red = "0".$red;
 	}
 	if (strlen($green) == 1) {
-		$green = "0" . $green;
+		$green = "0".$green;
 	}
 	if (strlen($blue) == 1) {
-		$blue = "0" . $blue;
+		$blue = "0".$blue;
 	}
 
 	return "$red$green$blue";
@@ -148,7 +148,7 @@ 
 		$img = "<img src='$source' />";
 	} # gerer img src="data:....base64"
 	elseif (preg_match('@^data:image/(jpe?g|png|gif);base64,(.*)$@isS', $source, $regs)) {
-		$local = sous_repertoire(_DIR_VAR, 'image-data') . md5($regs[2]) . '.' . str_replace('jpeg', 'jpg', $regs[1]);
+		$local = sous_repertoire(_DIR_VAR, 'image-data').md5($regs[2]).'.'.str_replace('jpeg', 'jpg', $regs[1]);
 		if (!file_exists($local)) {
 			ecrire_fichier($local, base64_decode($regs[2]));
 		}
@@ -162,7 +162,7 @@ 
 	// les protocoles web prennent au moins 3 lettres
 	if (tester_url_absolue($source)) {
 		include_spip('inc/distant');
-		$fichier = _DIR_RACINE . copie_locale($source);
+		$fichier = _DIR_RACINE.copie_locale($source);
 		if (!$fichier) {
 			return "";
 		}
@@ -256,9 +256,9 @@ 
 			// on garde la terminaison initiale car image simplement copiee
 			// et on postfixe son nom avec un md5 du path
 			$terminaison_dest = $terminaison;
-			$fichier_dest .= '-' . substr(md5("$identifiant"), 0, 5);
+			$fichier_dest .= '-'.substr(md5("$identifiant"), 0, 5);
 		} else {
-			$fichier_dest .= '-' . substr(md5("$identifiant-$effet"), 0, 5);
+			$fichier_dest .= '-'.substr(md5("$identifiant-$effet"), 0, 5);
 		}
 		$cache = sous_repertoire(_DIR_VAR, $cache);
 		$cache = sous_repertoire($cache, $effet);
@@ -274,7 +274,7 @@ 
 		$fichier_dest = substr($fichier_dest, 2);
 	}
 
-	$fichier_dest = $cache . $fichier_dest . "." . $terminaison_dest;
+	$fichier_dest = $cache.$fichier_dest.".".$terminaison_dest;
 
 	$GLOBALS["images_calculees"][] = $fichier_dest;
 
@@ -310,14 +310,14 @@ 
 	}
 
 	if ($creer) {
-		spip_log("filtre image " . ($fonction_creation ? reset($fonction_creation) : '') . "[$effet] sur $fichier",
-			"images" . _LOG_DEBUG);
+		spip_log("filtre image ".($fonction_creation ? reset($fonction_creation) : '')."[$effet] sur $fichier",
+			"images"._LOG_DEBUG);
 	}
 
 	// TODO: si une image png est nommee .jpg, le reconnaitre avec le bon $f
-	$ret["fonction_imagecreatefrom"] = "_imagecreatefrom" . $term_fonction;
+	$ret["fonction_imagecreatefrom"] = "_imagecreatefrom".$term_fonction;
 	$ret["fichier"] = $fichier;
-	$ret["fonction_image"] = "_image_image" . $terminaison_dest;
+	$ret["fonction_image"] = "_image_image".$terminaison_dest;
 	$ret["fichier_dest"] = $fichier_dest;
 	$ret["format_source"] = ($terminaison != 'jpeg' ? $terminaison : 'jpg');
 	$ret["format_dest"] = $terminaison_dest;
@@ -441,7 +441,7 @@ 
 	if (!function_exists('imagepng')) {
 		return false;
 	}
-	$tmp = $fichier . ".tmp";
+	$tmp = $fichier.".tmp";
 	$ret = imagepng($img, $tmp);
 	if (file_exists($tmp)) {
 		$taille_test = getimagesize($tmp);
@@ -476,7 +476,7 @@ 
 	if (!function_exists('imagegif')) {
 		return false;
 	}
-	$tmp = $fichier . ".tmp";
+	$tmp = $fichier.".tmp";
 	$ret = imagegif($img, $tmp);
 	if (file_exists($tmp)) {
 		$taille_test = getimagesize($tmp);
@@ -516,7 +516,7 @@ 
 	if (!function_exists('imagejpeg')) {
 		return false;
 	}
-	$tmp = $fichier . ".tmp";
+	$tmp = $fichier.".tmp";
 
 	// Enable interlancing
 	imageinterlace($img, true);
@@ -582,12 +582,12 @@ 
  *     - false sinon.
  */
 function _image_gd_output($img, $valeurs, $qualite = _IMG_GD_QUALITE) {
-	$fonction = "_image_image" . $valeurs['format_dest'];
+	$fonction = "_image_image".$valeurs['format_dest'];
 	$ret = false;
 	#un flag pour reperer les images gravees
 	$lock =
 		!statut_effacer_images_temporaires('get') // si la fonction n'a pas ete activee, on grave tout
-	or (@file_exists($valeurs['fichier_dest']) and !@file_exists($valeurs['fichier_dest'] . '.src'));
+	or (@file_exists($valeurs['fichier_dest']) and !@file_exists($valeurs['fichier_dest'].'.src'));
 	if (
 		function_exists($fonction)
 		&& ($ret = $fonction($img, $valeurs['fichier_dest'], $qualite)) # on a reussi a creer l'image
@@ -598,7 +598,7 @@ 
 			// dans tous les cas mettre a jour la taille de l'image finale
 			list($valeurs["hauteur_dest"], $valeurs["largeur_dest"]) = taille_image($valeurs['fichier_dest']);
 			$valeurs['date'] = @filemtime($valeurs['fichier_dest']); // pour la retrouver apres disparition
-			ecrire_fichier($valeurs['fichier_dest'] . '.src', serialize($valeurs), true);
+			ecrire_fichier($valeurs['fichier_dest'].'.src', serialize($valeurs), true);
 		}
 	}
 
@@ -617,7 +617,7 @@ 
 function reconstruire_image_intermediaire($fichier_manquant) {
 	$reconstruire = array();
 	$fichier = $fichier_manquant;
-	while (strpos($fichier,"://")===false
+	while (strpos($fichier, "://") === false
 		and !@file_exists($fichier)
 		and lire_fichier($src = "$fichier.src", $source)
 		and $valeurs = unserialize($source)
@@ -654,7 +654,7 @@ 
  *     Chemin du fichier d'image calculé
  **/
 function ramasse_miettes($fichier) {
-	if (strpos($fichier,"://")!==false
+	if (strpos($fichier, "://") !== false
 		or !lire_fichier($src = "$fichier.src", $source)
 		or !$valeurs = unserialize($source)
 	) {
@@ -710,7 +710,7 @@ 
 	}
 	# si jamais le fichier final n'a pas ete calcule car suppose temporaire
 	# et qu'il ne s'agit pas d'une URL
-	if (strpos($fichier,"://")===false and !@file_exists($fichier)) {
+	if (strpos($fichier, "://") === false and !@file_exists($fichier)) {
 		reconstruire_image_intermediaire($fichier);
 	}
 	ramasse_miettes($fichier);
@@ -807,7 +807,7 @@ 
 
 	// attributs deprecies. Transformer en CSS
 	if ($espace = extraire_attribut($tag, 'hspace')) {
-		$style = "margin:${espace}px;" . $style;
+		$style = "margin:${espace}px;".$style;
 		$tag = inserer_attribut($tag, 'hspace', '');
 	}
 
@@ -931,7 +931,7 @@ 
 	$image = $valeurs['fichier'];
 	$format = $valeurs['format_source'];
 	$destdir = dirname($valeurs['fichier_dest']);
-	$destfile = basename($valeurs['fichier_dest'], "." . $valeurs["format_dest"]);
+	$destfile = basename($valeurs['fichier_dest'], ".".$valeurs["format_dest"]);
 
 	$format_sortie = $valeurs['format_dest'];
 
@@ -966,7 +966,7 @@ 
 
 	// Si l'image est de la taille demandee (ou plus petite), simplement la retourner
 	if ($srcWidth and $srcWidth <= $maxWidth and $srcHeight <= $maxHeight) {
-		$vignette = $destination . '.' . $format;
+		$vignette = $destination.'.'.$format;
 		@copy($image, $vignette);
 	} // imagemagick en ligne de commande
 	elseif ($process == 'convert') {
@@ -974,9 +974,9 @@ 
 			define('_CONVERT_COMMAND', 'convert');
 		} // Securite : mes_options.php peut preciser le chemin absolu
 		if (!defined('_RESIZE_COMMAND')) {
-			define('_RESIZE_COMMAND', _CONVERT_COMMAND . ' -quality ' . _IMG_CONVERT_QUALITE . ' -resize %xx%y! %src %dest');
+			define('_RESIZE_COMMAND', _CONVERT_COMMAND.' -quality '._IMG_CONVERT_QUALITE.' -resize %xx%y! %src %dest');
 		}
-		$vignette = $destination . "." . $format_sortie;
+		$vignette = $destination.".".$format_sortie;
 		$commande = str_replace(
 			array('%x', '%y', '%src', '%dest'),
 			array(
@@ -991,11 +991,11 @@ 
 		if (!@file_exists($vignette)) {
 			spip_log("echec convert sur $vignette");
 
-			return;  // echec commande
+			return; // echec commande
 		}
 	} // php5 imagemagick
 	elseif ($process == 'imagick') {
-		$vignette = "$destination." . $format_sortie;
+		$vignette = "$destination.".$format_sortie;
 
 		if (!class_exists('Imagick')) {
 			spip_log("Classe Imagick absente !", _LOG_ERREUR);
@@ -1005,7 +1005,7 @@ 
 		$imagick = new Imagick();
 		$imagick->readImage($image);
 		$imagick->resizeImage($destWidth, $destHeight, Imagick::FILTER_LANCZOS,
-			1);//, IMAGICK_FILTER_LANCZOS, _IMG_IMAGICK_QUALITE / 100);
+			1); //, IMAGICK_FILTER_LANCZOS, _IMG_IMAGICK_QUALITE / 100);
 		$imagick->writeImage($vignette);
 
 		if (!@file_exists($vignette)) {
@@ -1021,12 +1021,12 @@ 
 		if (_PNMSCALE_COMMAND == '') {
 			return;
 		}
-		$vignette = $destination . "." . $format_sortie;
+		$vignette = $destination.".".$format_sortie;
 		$pnmtojpeg_command = str_replace("pnmscale", "pnmtojpeg", _PNMSCALE_COMMAND);
 		if ($format == "jpg") {
 
 			$jpegtopnm_command = str_replace("pnmscale", "jpegtopnm", _PNMSCALE_COMMAND);
-			exec("$jpegtopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
+			exec("$jpegtopnm_command $image | "._PNMSCALE_COMMAND." -width $destWidth | $pnmtojpeg_command > $vignette");
 			if (!($s = @filesize($vignette))) {
 				spip_unlink($vignette);
 			}
@@ -1038,7 +1038,7 @@ 
 		} else {
 			if ($format == "gif") {
 				$giftopnm_command = str_replace("pnmscale", "giftopnm", _PNMSCALE_COMMAND);
-				exec("$giftopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
+				exec("$giftopnm_command $image | "._PNMSCALE_COMMAND." -width $destWidth | $pnmtojpeg_command > $vignette");
 				if (!($s = @filesize($vignette))) {
 					spip_unlink($vignette);
 				}
@@ -1050,7 +1050,7 @@ 
 			} else {
 				if ($format == "png") {
 					$pngtopnm_command = str_replace("pnmscale", "pngtopnm", _PNMSCALE_COMMAND);
-					exec("$pngtopnm_command $image | " . _PNMSCALE_COMMAND . " -width $destWidth | $pnmtojpeg_command > $vignette");
+					exec("$pngtopnm_command $image | "._PNMSCALE_COMMAND." -width $destWidth | $pnmtojpeg_command > $vignette");
 					if (!($s = @filesize($vignette))) {
 						spip_unlink($vignette);
 					}
@@ -1070,7 +1070,7 @@ 
 			return;
 		}
 		if (_IMG_GD_MAX_PIXELS && $srcWidth * $srcHeight > _IMG_GD_MAX_PIXELS) {
-			spip_log("vignette gd1/gd2 impossible : " . $srcWidth * $srcHeight . "pixels");
+			spip_log("vignette gd1/gd2 impossible : ".$srcWidth * $srcHeight."pixels");
 
 			return;
 		}
@@ -1357,7 +1357,7 @@ 
 		// de l'image, de facon a tromper le cache du navigateur
 		// quand on fait supprimer/reuploader un logo
 		// (pas de filemtime si SAFE MODE)
-		$date = test_espace_prive() ? ('?' . $date) : '';
+		$date = test_espace_prive() ? ('?'.$date) : '';
 
 		return _image_ecrire_tag($image, array('src' => "$logo$date", 'width' => $destWidth, 'height' => $destHeight));
 	} else # SVG par exemple ? BMP, tiff ... les redacteurs osent tout!
@@ -1404,7 +1404,7 @@ 
 	public static function LittleEndian2String($number, $minbytes = 1) {
 		$intstring = '';
 		while ($number > 0) {
-			$intstring = $intstring . chr($number & 255);
+			$intstring = $intstring.chr($number & 255);
 			$number >>= 8;
 		}
 
@@ -1437,9 +1437,9 @@ 
 					$b = $argb['blue'];
 
 					if ($bpp[$key] == 32) {
-						$icXOR[$key] .= chr($b) . chr($g) . chr($r) . chr($a);
+						$icXOR[$key] .= chr($b).chr($g).chr($r).chr($a);
 					} elseif ($bpp[$key] == 24) {
-						$icXOR[$key] .= chr($b) . chr($g) . chr($r);
+						$icXOR[$key] .= chr($b).chr($g).chr($r);
 					}
 
 					if ($a < 128) {
@@ -1467,44 +1467,44 @@ 
 
 			// BITMAPINFOHEADER - 40 bytes
 			$BitmapInfoHeader[$key] = '';
-			$BitmapInfoHeader[$key] .= "\x28\x00\x00\x00";                // DWORD  biSize;
-			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageWidths[$key], 4);    // LONG   biWidth;
+			$BitmapInfoHeader[$key] .= "\x28\x00\x00\x00"; // DWORD  biSize;
+			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageWidths[$key], 4); // LONG   biWidth;
 			// The biHeight member specifies the combined
 			// height of the XOR and AND masks.
 			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($ImageHeights[$key] * 2, 4); // LONG   biHeight;
-			$BitmapInfoHeader[$key] .= "\x01\x00";                    // WORD   biPlanes;
-			$BitmapInfoHeader[$key] .= chr($bpp[$key]) . "\x00";              // wBitCount;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biCompression;
-			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($biSizeImage, 4);      // DWORD  biSizeImage;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // LONG   biXPelsPerMeter;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // LONG   biYPelsPerMeter;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biClrUsed;
-			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00";                // DWORD  biClrImportant;
+			$BitmapInfoHeader[$key] .= "\x01\x00"; // WORD   biPlanes;
+			$BitmapInfoHeader[$key] .= chr($bpp[$key])."\x00"; // wBitCount;
+			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00"; // DWORD  biCompression;
+			$BitmapInfoHeader[$key] .= phpthumb_functions::LittleEndian2String($biSizeImage, 4); // DWORD  biSizeImage;
+			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00"; // LONG   biXPelsPerMeter;
+			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00"; // LONG   biYPelsPerMeter;
+			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00"; // DWORD  biClrUsed;
+			$BitmapInfoHeader[$key] .= "\x00\x00\x00\x00"; // DWORD  biClrImportant;
 		}
 
 
-		$icondata = "\x00\x00";                    // idReserved;   // Reserved (must be 0)
-		$icondata .= "\x01\x00";                    // idType;	   // Resource Type (1 for icons)
-		$icondata .= phpthumb_functions::LittleEndian2String(count($gd_image_array), 2);  // idCount;	  // How many images?
+		$icondata = "\x00\x00"; // idReserved;   // Reserved (must be 0)
+		$icondata .= "\x01\x00"; // idType;	   // Resource Type (1 for icons)
+		$icondata .= phpthumb_functions::LittleEndian2String(count($gd_image_array), 2); // idCount;	  // How many images?
 
 		$dwImageOffset = 6 + (count($gd_image_array) * 16);
 		foreach ($gd_image_array as $key => $gd_image) {
 			// ICONDIRENTRY   idEntries[1]; // An entry for each image (idCount of 'em)
 
-			$icondata .= chr($ImageWidths[$key]);           // bWidth;		  // Width, in pixels, of the image
-			$icondata .= chr($ImageHeights[$key]);          // bHeight;		 // Height, in pixels, of the image
-			$icondata .= chr($totalcolors[$key]);           // bColorCount;	 // Number of colors in image (0 if >=8bpp)
-			$icondata .= "\x00";                    // bReserved;	   // Reserved ( must be 0)
+			$icondata .= chr($ImageWidths[$key]); // bWidth;		  // Width, in pixels, of the image
+			$icondata .= chr($ImageHeights[$key]); // bHeight;		 // Height, in pixels, of the image
+			$icondata .= chr($totalcolors[$key]); // bColorCount;	 // Number of colors in image (0 if >=8bpp)
+			$icondata .= "\x00"; // bReserved;	   // Reserved ( must be 0)
 
-			$icondata .= "\x01\x00";                  // wPlanes;		 // Color Planes
-			$icondata .= chr($bpp[$key]) . "\x00";            // wBitCount;	   // Bits per pixel
+			$icondata .= "\x01\x00"; // wPlanes;		 // Color Planes
+			$icondata .= chr($bpp[$key])."\x00"; // wBitCount;	   // Bits per pixel
 
 			$dwBytesInRes = 40 + strlen($icXOR[$key]) + strlen($icAND[$key]);
 			$icondata .= phpthumb_functions::LittleEndian2String($dwBytesInRes,
-				4);     // dwBytesInRes;	// How many bytes in this resource?
+				4); // dwBytesInRes;	// How many bytes in this resource?
 
 			$icondata .= phpthumb_functions::LittleEndian2String($dwImageOffset,
-				4);    // dwImageOffset;   // Where in the file is this image?
+				4); // dwImageOffset;   // Where in the file is this image?
 			$dwImageOffset += strlen($BitmapInfoHeader[$key]);
 			$dwImageOffset += strlen($icXOR[$key]);
 			$dwImageOffset += strlen($icAND[$key]);

ecrire/inc/filtres_images_mini.php

Doc Comments +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
  * @param int $height_max
  *    Hauteur minimale de l'image à traiter (10000 par défaut)
  * @return
- *    Le tag html `<img src=... />` avec une class `filtre_inactif` ou pas
+ string    Le tag html `<img src=... />` avec une class `filtre_inactif` ou pas
  */
 function image_select($img, $width_min = 0, $height_min = 0, $width_max = 10000, $height_max = 1000) {
 	if (!$img) {

ecrire/inc/flock.php

Doc Comments +1 added lines, -1 removed lines

@@ -644,7 +644,7 @@
  *
  * @param string $dir
  *     Répertoire à parcourir
- * @param int|string $pattern
+ * @param integer $pattern
  *     Expression régulière pour trouver des fichiers, tel que `[.]lock$`
  * @param int $maxfiles
  *     Nombre de fichiers maximums retournés

Spacing +19 added lines, -19 removed lines

@@ -122,7 +122,7 @@ 
 		$contenu = @gzfile($fichier);
 	}
 
-	return is_array($contenu) ? join('', $contenu) : (string)$contenu;
+	return is_array($contenu) ? join('', $contenu) : (string) $contenu;
 }
 
 
@@ -315,9 +315,9 @@ 
  */
 function ecrire_fichier_securise($fichier, $contenu, $ecrire_quand_meme = false, $truncate = true) {
 	if (substr($fichier, -4) !== '.php') {
-		spip_log('Erreur de programmation: ' . $fichier . ' doit finir par .php');
+		spip_log('Erreur de programmation: '.$fichier.' doit finir par .php');
 	}
-	$contenu = "<" . "?php die ('Acces interdit'); ?" . ">\n" . $contenu;
+	$contenu = "<"."?php die ('Acces interdit'); ?".">\n".$contenu;
 
 	return ecrire_fichier($fichier, $contenu, $ecrire_quand_meme, $truncate);
 }
@@ -340,7 +340,7 @@ 
  */
 function lire_fichier_securise($fichier, &$contenu, $options = array()) {
 	if ($res = lire_fichier($fichier, $contenu, $options)) {
-		$contenu = substr($contenu, strlen("<" . "?php die ('Acces interdit'); ?" . ">\n"));
+		$contenu = substr($contenu, strlen("<"."?php die ('Acces interdit'); ?".">\n"));
 	}
 
 	return $res;
@@ -460,7 +460,7 @@ 
 		$invalidate = @opcache_invalidate($filepath, true);
 		// si l'invalidation a echoue lever un flag
 		if (!$invalidate and !defined('_spip_attend_invalidation_opcode_cache')) {
-			define('_spip_attend_invalidation_opcode_cache',true);
+			define('_spip_attend_invalidation_opcode_cache', true);
 		}
 	}
 	// APC.
@@ -501,11 +501,11 @@ 
 		$wait = $duree + 1;
 		if ($timestamp) {
 			$wait -= (time() - $timestamp);
-			if ($wait<0) {
+			if ($wait < 0) {
 				$wait = 0;
 			}
 		}
-		spip_log('Probleme de configuration opcache.revalidate_freq '. $duree .'s : on attend '.$wait.'s', _LOG_INFO_IMPORTANTE);
+		spip_log('Probleme de configuration opcache.revalidate_freq '.$duree.'s : on attend '.$wait.'s', _LOG_INFO_IMPORTANTE);
 		if ($wait) {
 			sleep($duree + 1);
 		}
@@ -533,9 +533,9 @@ 
 		if ($item == '.' || $item == '..') {
 			continue;
 		}
-		if (!supprimer_repertoire($dir . "/" . $item)) {
-			@chmod($dir . "/" . $item, 0777);
-			if (!supprimer_repertoire($dir . "/" . $item)) {
+		if (!supprimer_repertoire($dir."/".$item)) {
+			@chmod($dir."/".$item, 0777);
+			if (!supprimer_repertoire($dir."/".$item)) {
 				return false;
 			}
 		};
@@ -580,7 +580,7 @@ 
 	if (!strlen($subdir)) {
 		$n = strrpos($base, "/");
 		if ($n === false) {
-			return $nobase ? '' : ($base . '/');
+			return $nobase ? '' : ($base.'/');
 		}
 		$subdir = substr($base, $n + 1);
 		$base = substr($base, 0, $n + 1);
@@ -590,14 +590,14 @@ 
 	}
 
 	$baseaff = $nobase ? '' : $base;
-	if (isset($dirs[$base . $subdir])) {
-		return $baseaff . $dirs[$base . $subdir];
+	if (isset($dirs[$base.$subdir])) {
+		return $baseaff.$dirs[$base.$subdir];
 	}
 
-	$path = $base . $subdir; # $path = 'IMG/distant/pdf' ou 'IMG/distant_pdf'
+	$path = $base.$subdir; # $path = 'IMG/distant/pdf' ou 'IMG/distant_pdf'
 
 	if (file_exists("$path/.ok")) {
-		return $baseaff . ($dirs[$base . $subdir] = "$subdir/");
+		return $baseaff.($dirs[$base.$subdir] = "$subdir/");
 	}
 
 	@mkdir($path, _SPIP_CHMOD);
@@ -607,7 +607,7 @@ 
 		@touch("$path/.ok");
 		spip_log("creation $base$subdir/");
 
-		return $baseaff . ($dirs[$base . $subdir] = "$subdir/");
+		return $baseaff.($dirs[$base.$subdir] = "$subdir/");
 	}
 
 	// en cas d'echec c'est peut etre tout simplement que le disque est plein :
@@ -617,10 +617,10 @@ 
 		return '';
 	}
 	if (!_DIR_RESTREINT) {
-		$base = preg_replace(',^' . _DIR_RACINE . ',', '', $base);
+		$base = preg_replace(',^'._DIR_RACINE.',', '', $base);
 	}
 	$base .= $subdir;
-	raler_fichier($base . '/.ok');
+	raler_fichier($base.'/.ok');
 }
 
 
@@ -688,7 +688,7 @@ 
 							$beginning = $fichiers;
 							$end = preg_files("$f/", $pattern,
 								$maxfiles - $nbfiles, $recurs);
-							$fichiers = array_merge((array)$beginning, (array)$end);
+							$fichiers = array_merge((array) $beginning, (array) $end);
 							$nbfiles = count($fichiers);
 						}
 					}

ecrire/inc/importer_csv.php

Doc Comments +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
  * Importer le charset d'une ligne
  *
  * @param string $texte
- * @return array
+ * @return string
  */
 function importer_csv_importcharset($texte) {
 	// le plus frequent, en particulier avec les trucs de ms@@@
@@ -66,7 +66,7 @@ 
  * @param string $delim
  * @param string $enclos
  * @param int $len
- * @return array
+ * @return boolean
  */
 function inc_importer_csv_dist($file, $head = false, $delim = ", ", $enclos = '"', $len = 10000) {
 	$return = false;

ecrire/inc/lang.php

Doc Comments +11 added lines, -1 removed lines

@@ -31,7 +31,7 @@ 
  *
  * @param string $lang
  *     La langue à utiliser
- * @return string|bool
+ * @return false|string
  *     string : La langue qui a été utilisée si trouvée
  *     false : aucune langue ne correspondait à la demande
  **/
@@ -80,6 +80,9 @@ 
 // retourne son 2e argument si c'est un index du premier
 // ou un index approchant sinon et si possible, 
 // la langue X etant consideree comme une approche de X_Y
+/**
+ * @return string
+ */
 function approcher_langue($trads, $lang = '') {
 
 	if (!$lang) {
@@ -175,6 +178,9 @@ 
 // pour 'changer_lang' (langue de l'article, espace prive), c'est en Ajax
 // 
 // http://code.spip.net/@menu_langues
+/**
+ * @param string $nom_select
+ */
 function menu_langues($nom_select, $default = '') {
 	include_spip('inc/actions');
 
@@ -209,6 +215,10 @@ 
 }
 
 // http://code.spip.net/@select_langues
+/**
+ * @param string $change
+ * @param string $options
+ */
 function select_langues($nom_select, $change, $options, $label = "") {
 	static $cpt = 0;
 	$id = "menu_langues" . $cpt++;

Spacing +10 added lines, -12 removed lines

@@ -37,8 +37,8 @@ 
  **/
 function changer_langue($lang) {
 
-	$liste_langues = ',' . @$GLOBALS['meta']['langues_proposees']
-		. ',' . @$GLOBALS['meta']['langues_multilingue'] . ',';
+	$liste_langues = ','.@$GLOBALS['meta']['langues_proposees']
+		. ','.@$GLOBALS['meta']['langues_multilingue'].',';
 
 	// Si la langue demandee n'existe pas, on essaie d'autres variantes
 	// Exemple : 'pt-br' => 'pt_br' => 'pt'
@@ -189,7 +189,7 @@ 
 	}
 	foreach ($langues as $l) {
 		$selected = ($l == $default) ? ' selected=\'selected\'' : '';
-		$ret .= "<option value='$l'$selected>[" . $l . "] " . traduire_nom_langue($l) . "</option>\n";
+		$ret .= "<option value='$l'$selected>[".$l."] ".traduire_nom_langue($l)."</option>\n";
 	}
 
 	if (!test_espace_prive()) {
@@ -204,23 +204,21 @@ 
 
 	return generer_action_auteur('converser', $base, $cible,
 		(select_langues($nom_select, $change, $ret)
-			. "<noscript><div style='display:inline'><input type='submit' class='fondo' value='" . _T('bouton_changer') . "' /></div></noscript>"),
+			. "<noscript><div style='display:inline'><input type='submit' class='fondo' value='"._T('bouton_changer')."' /></div></noscript>"),
 		" method='post'");
 }
 
 // http://code.spip.net/@select_langues
 function select_langues($nom_select, $change, $options, $label = "") {
 	static $cpt = 0;
-	$id = "menu_langues" . $cpt++;
+	$id = "menu_langues".$cpt++;
 
 	return
-		"<label for='$id'>" . ($label ? $label : _T('info_langues')) . "</label> " .
+		"<label for='$id'>".($label ? $label : _T('info_langues'))."</label> ".
 		"<select name='$nom_select' id='$id' "
 		. ((!test_espace_prive()) ?
-			("class='forml menu_langues'") :
-			(($nom_select == 'var_lang_ecrire') ?
-				("class='lang_ecrire'") :
-				"class='fondl'"))
+			("class='forml menu_langues'") : (($nom_select == 'var_lang_ecrire') ?
+				("class='lang_ecrire'") : "class='fondl'"))
 		. $change
 		. ">\n"
 		. $options
@@ -342,7 +340,7 @@ 
 		and (!isset($GLOBALS['spip_lang'])
 			or $GLOBALS['spip_lang'] != $GLOBALS['meta']['langue_site'])
 	) {
-		return changer_langue($GLOBALS['meta']['langue_site']);//@:install
+		return changer_langue($GLOBALS['meta']['langue_site']); //@:install
 	}
 	// en theorie là, la globale est définie, sinon c'est un problème.
 	if (!isset($GLOBALS['spip_lang'])) {
@@ -443,7 +441,7 @@ 
 	if (!isset($GLOBALS['meta']['langue_site'])) {
 		// Initialisation : le francais si dispo, sinon la premiere langue trouvee
 		$GLOBALS['meta']['langue_site'] = $tout =
-			(!$all_langs or (strpos(',' . _LANGUE_PAR_DEFAUT . ',', ",$all_langs,") !== false))
+			(!$all_langs or (strpos(','._LANGUE_PAR_DEFAUT.',', ",$all_langs,") !== false))
 				? _LANGUE_PAR_DEFAUT : substr($all_langs, 0, strpos($all_langs, ','));
 		ecrire_meta('langue_site', $tout);
 	}