spip / SPIP

ecrire/public/fonctions.php

Spacing +10 added lines, -10 removed lines

@@ -84,7 +84,7 @@ 
 		$texte = $intro;
 	} else {
 		if (
-			strpos("\n" . $texte, "\n|") === false
+			strpos("\n".$texte, "\n|") === false
 			and strlen($texte) > 2.5 * $longueur
 		) {
 			if (strpos($texte, '<multi') !== false) {
@@ -168,12 +168,12 @@ 
 	if ($pas < 1) {
 		return '';
 	}
-	$ancre = 'pagination' . $nom; // #pagination_articles
-	$debut = 'debut' . $nom; // 'debut_articles'
+	$ancre = 'pagination'.$nom; // #pagination_articles
+	$debut = 'debut'.$nom; // 'debut_articles'
 
 	// n'afficher l'ancre qu'une fois
 	if (!isset($ancres[$ancre])) {
-		$bloc_ancre = $ancres[$ancre] = "<a id='" . $ancre . "' class='pagination_ancre'></a>";
+		$bloc_ancre = $ancres[$ancre] = "<a id='".$ancre."' class='pagination_ancre'></a>";
 	} else {
 		$bloc_ancre = '';
 	}
@@ -205,8 +205,8 @@ 
 
 	if ($modele) {
 		$pagination['type_pagination'] = $modele;
-		if (trouver_fond('pagination_' . $modele, 'modeles')) {
-			$modele = '_' . $modele;
+		if (trouver_fond('pagination_'.$modele, 'modeles')) {
+			$modele = '_'.$modele;
 		}
 		else {
 			$modele = '';
@@ -291,7 +291,7 @@ 
 function lister_objets_avec_logos($type) {
 
 	$objet = objet_type($type);
-	$ids = sql_allfetsel('L.id_objet', 'spip_documents AS D JOIN spip_documents_liens AS L ON L.id_document=D.id_document', 'D.mode=' . sql_quote('logoon') . ' AND L.objet=' . sql_quote($objet));
+	$ids = sql_allfetsel('L.id_objet', 'spip_documents AS D JOIN spip_documents_liens AS L ON L.id_document=D.id_document', 'D.mode='.sql_quote('logoon').' AND L.objet='.sql_quote($objet));
 	if ($ids) {
 		$ids = array_column($ids, 'id_objet');
 		return implode(',', $ids);
@@ -484,7 +484,7 @@ 
 
 	// Classes : on indique le sens de tri et l'item exposé
 	if (!$is_sens_fixe) {
-		$classe .= ' item-tri item-tri_' . ($tri_sens_actuel === 1 ? 'asc' : 'desc');
+		$classe .= ' item-tri item-tri_'.($tri_sens_actuel === 1 ? 'asc' : 'desc');
 	}
 	if ($champ_ou_sens === $tri_champ) {
 		$classe .= ' item-tri_actif';
@@ -523,7 +523,7 @@ 
  */
 function tri_champ_order($t, $from = null, $senstri = '') {
 	if (strncmp($t, 'multi ', 6) == 0) {
-		return 'multi' . $senstri;
+		return 'multi'.$senstri;
 	}
 
 	$champ = $t;
@@ -557,7 +557,7 @@ 
 		case 'sinum ':
 			return "CASE( 0+$champ ) WHEN 0 THEN 1 ELSE 0 END{$senstri}";
 		default:
-			return $champ . $senstri;
+			return $champ.$senstri;
 	}
 }
 

ecrire/req/mysql.php

Spacing +81 added lines, -81 removed lines

@@ -62,12 +62,12 @@ 
 			$link = @mysqli_connect($host, $login, $pass);
 		}
 	} catch (\mysqli_sql_exception $e) {
-		spip_log('mysqli_sql_exception: ' . $e->getMessage(), 'mysql.' . _LOG_DEBUG);
+		spip_log('mysqli_sql_exception: '.$e->getMessage(), 'mysql.'._LOG_DEBUG);
 		$link = false;
 	}
 
 	if (!$link) {
-		spip_log('Echec mysqli_connect. Erreur : ' . mysqli_connect_error(), 'mysql.' . _LOG_HS);
+		spip_log('Echec mysqli_connect. Erreur : '.mysqli_connect_error(), 'mysql.'._LOG_HS);
 
 		return false;
 	}
@@ -86,7 +86,7 @@ 
 	}
 
 	spip_log(
-		"Connexion MySQLi vers $host, base $db, prefixe $prefixe " . ($ok ? 'operationnelle' : 'impossible'),
+		"Connexion MySQLi vers $host, base $db, prefixe $prefixe ".($ok ? 'operationnelle' : 'impossible'),
 		_LOG_DEBUG
 	);
 
@@ -177,9 +177,9 @@ 
  */
 function spip_mysql_set_charset($charset, $serveur = '', $requeter = true) {
 	$connexion = &$GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
-	spip_log('changement de charset sql : ' . 'SET NAMES ' . _q($charset), _LOG_DEBUG);
+	spip_log('changement de charset sql : '.'SET NAMES '._q($charset), _LOG_DEBUG);
 
-	return mysqli_query($connexion['link'], $connexion['last'] = 'SET NAMES ' . _q($charset));
+	return mysqli_query($connexion['link'], $connexion['last'] = 'SET NAMES '._q($charset));
 }
 
 
@@ -194,7 +194,7 @@ 
 function spip_mysql_get_charset($charset = [], $serveur = '', $requeter = true) {
 	$connexion = &$GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
 	$connexion['last'] = $c = 'SHOW CHARACTER SET'
-		. (!$charset ? '' : (' LIKE ' . _q($charset['charset'])));
+		. (!$charset ? '' : (' LIKE '._q($charset['charset'])));
 
 	return spip_mysql_fetch(mysqli_query($connexion['link'], $c), null, $serveur);
 }
@@ -238,21 +238,21 @@ 
 	$debug = '';
 	if (defined('_DEBUG_SLOW_QUERIES') and _DEBUG_SLOW_QUERIES) {
 		if (isset($GLOBALS['debug']['aucasou'])) {
-			[, $id, , $infos] = $GLOBALS['debug']['aucasou'];
-			$debug .= "BOUCLE$id @ " . ($infos[0] ?? '') . ' | ';
+			[, $id,, $infos] = $GLOBALS['debug']['aucasou'];
+			$debug .= "BOUCLE$id @ ".($infos[0] ?? '').' | ';
 		}
 		if (isset($_SERVER['REQUEST_URI'])) {
 			$debug .= $_SERVER['REQUEST_URI'];
 		}
 		if (!empty($GLOBALS['ip'])) {
-			$debug .= ' + ' . $GLOBALS['ip'];
+			$debug .= ' + '.$GLOBALS['ip'];
 		}
-		$debug = ' /* ' . mysqli_real_escape_string($link, str_replace('*/', '@/', $debug)) . ' */';
+		$debug = ' /* '.mysqli_real_escape_string($link, str_replace('*/', '@/', $debug)).' */';
 	}
 	try {
-		$r = mysqli_query($link, $query . $debug);
+		$r = mysqli_query($link, $query.$debug);
 	} catch (\mysqli_sql_exception $e) {
-		spip_log('mysqli_sql_exception: ' . $e->getMessage(), 'mysql.' . _LOG_DEBUG);
+		spip_log('mysqli_sql_exception: '.$e->getMessage(), 'mysql.'._LOG_DEBUG);
 		$r = false;
 		// TODO: utiliser l’exception ensuite plutôt que les appels à spip_mysql_errno()
 		// mais il faut pour php < 8.1 forcer les exeptions via mysqli_report().
@@ -270,9 +270,9 @@ 
 			$link = $connexion['link'];
 			//On retente au cas où
 			try {
-				$r = mysqli_query($link, $query . $debug);
+				$r = mysqli_query($link, $query.$debug);
 			} catch (\mysqli_sql_exception $e) {
-				spip_log('mysqli_sql_exception: ' . $e->getMessage(), 'mysql.' . _LOG_DEBUG);
+				spip_log('mysqli_sql_exception: '.$e->getMessage(), 'mysql.'._LOG_DEBUG);
 				$r = false;
 				// TODO: utiliser l’exception ensuite plutôt que les appels à spip_mysql_errno()
 				// mais il faut pour php < 8.1 forcer les exeptions via mysqli_report().
@@ -305,7 +305,7 @@ 
 	// d'utiliser ceux-ci, copie-colle de phpmyadmin
 	$query = preg_replace(',^TABLE\s*`([^`]*)`,i', "TABLE \\1", $query);
 
-	return spip_mysql_query('ALTER ' . $query, $serveur, $requeter); # i.e. que PG se debrouille
+	return spip_mysql_query('ALTER '.$query, $serveur, $requeter); # i.e. que PG se debrouille
 }
 
 
@@ -318,7 +318,7 @@ 
  * @return bool            Toujours true
  */
 function spip_mysql_optimize($table, $serveur = '', $requeter = true) {
-	spip_mysql_query('OPTIMIZE TABLE ' . $table);
+	spip_mysql_query('OPTIMIZE TABLE '.$table);
 
 	return true;
 }
@@ -341,7 +341,7 @@ 
 	$link = $connexion['link'];
 	$db = $connexion['db'];
 
-	$query = 'EXPLAIN ' . _mysql_traite_query($query, $db, $prefixe);
+	$query = 'EXPLAIN '._mysql_traite_query($query, $db, $prefixe);
 	$r = mysqli_query($link, $query);
 
 	return spip_mysql_fetch($r, null, $serveur);
@@ -392,7 +392,7 @@ 
 		. calculer_mysql_expression('WHERE', $where)
 		. calculer_mysql_expression('GROUP BY', $groupby, ',')
 		. calculer_mysql_expression('HAVING', $having)
-		. ($orderby ? ("\nORDER BY " . spip_mysql_order($orderby)) : '')
+		. ($orderby ? ("\nORDER BY ".spip_mysql_order($orderby)) : '')
 		. ($limit ? "\nLIMIT $limit" : '');
 
 	// renvoyer la requete inerte si demandee
@@ -482,12 +482,12 @@ 
 	$exp = "\n$expression ";
 
 	if (!is_array($v)) {
-		return $exp . $v;
+		return $exp.$v;
 	} else {
 		if (strtoupper($join) === 'AND') {
-			return $exp . join("\n\t$join ", array_map('calculer_mysql_where', $v));
+			return $exp.join("\n\t$join ", array_map('calculer_mysql_where', $v));
 		} else {
-			return $exp . join($join, $v);
+			return $exp.join($join, $v);
 		}
 	}
 }
@@ -505,17 +505,17 @@ 
 		if (substr($k, -1) == '@') {
 			// c'est une jointure qui se refere au from precedent
 			// pas de virgule
-			$res .= '  ' . $v;
+			$res .= '  '.$v;
 		} else {
 			if (!is_numeric($k)) {
 				$p = strpos($v, ' ');
 				if ($p) {
-					$v = substr($v, 0, $p) . " AS `$k`" . substr($v, $p);
+					$v = substr($v, 0, $p)." AS `$k`".substr($v, $p);
 				} else {
 					$v .= " AS `$k`";
 				}
 			}
-			$res .= ', ' . $v;
+			$res .= ', '.$v;
 		}
 	}
 
@@ -545,13 +545,13 @@ 
 function _mysql_traite_query($query, $db = '', $prefixe = '', $echappe_textes = true) {
 
 	if ($GLOBALS['mysql_rappel_nom_base'] and $db) {
-		$pref = '`' . $db . '`.';
+		$pref = '`'.$db.'`.';
 	} else {
 		$pref = '';
 	}
 
 	if ($prefixe) {
-		$pref .= $prefixe . '_';
+		$pref .= $prefixe.'_';
 	}
 
 	if (!preg_match('/\s(SET|VALUES|WHERE|DATABASE)\s/i', $query, $regs)) {
@@ -570,7 +570,7 @@ 
 				$suite_echap = $suite;
 			}
 			if (preg_match('/^(.*?)([(]\s*SELECT\b.*)$/si', $suite_echap, $r)) {
-				$suite_echap = $r[1] . _mysql_traite_query($r[2], $db, $prefixe, false);
+				$suite_echap = $r[1]._mysql_traite_query($r[2], $db, $prefixe, false);
 				if ($echappe_textes) {
 					$suite = query_reinjecte_textes($suite_echap, $textes);
 				}
@@ -580,7 +580,7 @@ 
 			}
 		}
 	}
-	$r = preg_replace(_SQL_PREFIXE_TABLE_MYSQL, '\1' . $pref, $query) . $suite;
+	$r = preg_replace(_SQL_PREFIXE_TABLE_MYSQL, '\1'.$pref, $query).$suite;
 
 	// en option, remplacer les emoji (que mysql ne sait pas gérer) en &#128169;
 	// remplacer les emoji (que mysql ne sait pas gérer) en &#128169;
@@ -620,7 +620,7 @@ 
 		$ok = false;
 	}
 	if (!$ok) {
-		spip_log('Echec mysqli_selectdb. Erreur : ' . mysqli_error($link), 'mysql.' . _LOG_CRITIQUE);
+		spip_log('Echec mysqli_selectdb. Erreur : '.mysqli_error($link), 'mysql.'._LOG_CRITIQUE);
 	}
 
 	return $ok;
@@ -710,10 +710,10 @@ 
 
 	$character_set = '';
 	if (@$GLOBALS['meta']['charset_sql_base']) {
-		$character_set .= ' CHARACTER SET ' . $GLOBALS['meta']['charset_sql_base'];
+		$character_set .= ' CHARACTER SET '.$GLOBALS['meta']['charset_sql_base'];
 	}
 	if (@$GLOBALS['meta']['charset_collation_sql_base']) {
-		$character_set .= ' COLLATE ' . $GLOBALS['meta']['charset_collation_sql_base'];
+		$character_set .= ' COLLATE '.$GLOBALS['meta']['charset_collation_sql_base'];
 	}
 
 	foreach ($champs as $k => $v) {
@@ -723,7 +723,7 @@ 
 				preg_match(',(char|text),i', $defs[1])
 				and !preg_match(',(binary|CHARACTER|COLLATE),i', $v)
 			) {
-				$v = $defs[1] . $character_set . ' ' . substr($v, strlen($defs[1]));
+				$v = $defs[1].$character_set.' '.substr($v, strlen($defs[1]));
 			}
 		}
 
@@ -735,8 +735,8 @@ 
 		$s = ',';
 	}
 	$temporary = $temporary ? 'TEMPORARY' : '';
-	$q = "CREATE $temporary TABLE IF NOT EXISTS $nom ($query" . ($keys ? ",$keys" : '') . ')'
-		. ' ENGINE=' . _MYSQL_ENGINE
+	$q = "CREATE $temporary TABLE IF NOT EXISTS $nom ($query".($keys ? ",$keys" : '').')'
+		. ' ENGINE='._MYSQL_ENGINE
 		. ($character_set ? " DEFAULT $character_set" : '')
 		. "\n";
 
@@ -815,7 +815,7 @@ 
 		return false;
 	}
 
-	$query = "CREATE VIEW $nom AS " . $query_select;
+	$query = "CREATE VIEW $nom AS ".$query_select;
 
 	return spip_mysql_query($query, $serveur, $requeter);
 }
@@ -873,7 +873,7 @@ 
  *     Ressource à utiliser avec sql_fetch()
  **/
 function spip_mysql_showbase($match, $serveur = '', $requeter = true) {
-	return spip_mysql_query('SHOW TABLES LIKE ' . _q($match), $serveur, $requeter);
+	return spip_mysql_query('SHOW TABLES LIKE '._q($match), $serveur, $requeter);
 }
 
 /**
@@ -889,7 +889,7 @@ 
  *     - true si la requête a réussie, false sinon
  */
 function spip_mysql_repair($table, $serveur = '', $requeter = true) {
-	$table_status = spip_mysql_fetch(spip_mysql_query('SHOW TABLE STATUS WHERE Name = ' . _q($table), $serveur, true));
+	$table_status = spip_mysql_fetch(spip_mysql_query('SHOW TABLE STATUS WHERE Name = '._q($table), $serveur, true));
 	$engine = $table_status['Engine'];
 	if ($engine == 'InnoDB') {
 		if (spip_mysql_alter("TABLE $table ENGINE = InnoDB", $serveur, $requeter)) {
@@ -898,7 +898,7 @@ 
 	} elseif ($engine == 'MyISAM') {
 		return spip_mysql_query("REPAIR TABLE `$table`", $serveur, $requeter);
 	} else {
-		spip_log("spip_mysql_repair impossible pour la table $table engine $engine", 'mysql.' . _LOG_DEBUG);
+		spip_log("spip_mysql_repair impossible pour la table $table engine $engine", 'mysql.'._LOG_DEBUG);
 	}
 }
 
@@ -917,7 +917,7 @@ 
  *     - string : requete sql, si $requeter = true
  **/
 function spip_mysql_table_exists(string $table, $serveur = '', $requeter = true) {
-	$r = spip_mysql_query('SHOW TABLES LIKE ' . _q($table), $serveur, $requeter);
+	$r = spip_mysql_query('SHOW TABLES LIKE '._q($table), $serveur, $requeter);
 	if (!$requeter) {
 		return $r;
 	}
@@ -999,22 +999,22 @@ 
 			}
 			if ($val['Default'] === '0' || $val['Default']) {
 				if (preg_match('/[A-Z_]/', $val['Default'])) {
-					$nfields[$val['Field']] .= ' DEFAULT ' . $val['Default'];
+					$nfields[$val['Field']] .= ' DEFAULT '.$val['Default'];
 				} else {
-					$nfields[$val['Field']] .= " DEFAULT '" . $val['Default'] . "'";
+					$nfields[$val['Field']] .= " DEFAULT '".$val['Default']."'";
 				}
 			}
 			if ($val['Extra']) {
-				$nfields[$val['Field']] .= ' ' . $val['Extra'];
+				$nfields[$val['Field']] .= ' '.$val['Extra'];
 			}
 			if ($val['Key'] == 'PRI') {
 				$nkeys['PRIMARY KEY'] = $val['Field'];
 			} else {
 				if ($val['Key'] == 'MUL') {
-					$nkeys['KEY ' . $val['Field']] = $val['Field'];
+					$nkeys['KEY '.$val['Field']] = $val['Field'];
 				} else {
 					if ($val['Key'] == 'UNI') {
-						$nkeys['UNIQUE KEY ' . $val['Field']] = $val['Field'];
+						$nkeys['UNIQUE KEY '.$val['Field']] = $val['Field'];
 					}
 				}
 			}
@@ -1085,7 +1085,7 @@ 
 	$serveur = '',
 	$requeter = true
 ) {
-	$c = !$groupby ? '*' : ('DISTINCT ' . (is_string($groupby) ? $groupby : join(',', $groupby)));
+	$c = !$groupby ? '*' : ('DISTINCT '.(is_string($groupby) ? $groupby : join(',', $groupby)));
 
 	$r = spip_mysql_select("COUNT($c)", $from, $where, '', '', '', $having, $serveur, $requeter);
 	if (!$requeter) {
@@ -1125,7 +1125,7 @@ 
 	if ($s) {
 		$trace = debug_backtrace();
 		if ($trace[0]['function'] != 'spip_mysql_error') {
-			spip_log("$s - $query - " . sql_error_backtrace(), 'mysql.' . _LOG_ERREUR);
+			spip_log("$s - $query - ".sql_error_backtrace(), 'mysql.'._LOG_ERREUR);
 		}
 	}
 
@@ -1251,7 +1251,7 @@ 
 	try {
 		$insert = mysqli_query($link, $query);
 	} catch (\mysqli_sql_exception $e) {
-		spip_log('mysqli_sql_exception: ' . $e->getMessage(), 'mysql.' . _LOG_DEBUG);
+		spip_log('mysqli_sql_exception: '.$e->getMessage(), 'mysql.'._LOG_DEBUG);
 		// TODO: utiliser l’exception ensuite plutôt que les appels à spip_mysql_errno()
 		// mais il faut pour php < 8.1 forcer les exeptions via mysqli_report().
 	}
@@ -1306,8 +1306,8 @@ 
 
 	return spip_mysql_insert(
 		$table,
-		'(' . join(',', array_keys($couples)) . ')',
-		'(' . join(',', $couples) . ')',
+		'('.join(',', array_keys($couples)).')',
+		'('.join(',', $couples).')',
 		$desc,
 		$serveur,
 		$requeter
@@ -1344,7 +1344,7 @@ 
 	}
 	$fields = $desc['field'] ?? [];
 
-	$cles = '(' . join(',', array_keys(reset($tab_couples))) . ')';
+	$cles = '('.join(',', array_keys(reset($tab_couples))).')';
 	$valeurs = [];
 	$r = false;
 
@@ -1353,7 +1353,7 @@ 
 		foreach ($couples as $champ => $val) {
 			$couples[$champ] = spip_mysql_cite($val, $fields[$champ]);
 		}
-		$valeurs[] = '(' . join(',', $couples) . ')';
+		$valeurs[] = '('.join(',', $couples).')';
 		if (count($valeurs) >= 100) {
 			$r = spip_mysql_insert($table, $cles, join(', ', $valeurs), $desc, $serveur, $requeter);
 			$valeurs = [];
@@ -1390,7 +1390,7 @@ 
 function spip_mysql_update($table, $champs, $where = '', $desc = [], $serveur = '', $requeter = true) {
 	$set = [];
 	foreach ($champs as $champ => $val) {
-		$set[] = $champ . "=$val";
+		$set[] = $champ."=$val";
 	}
 	if (!empty($set)) {
 		return spip_mysql_query(
@@ -1446,7 +1446,7 @@ 
 	}
 	$set = [];
 	foreach ($champs as $champ => $val) {
-		$set[] = $champ . '=' . spip_mysql_cite($val, @$fields[$champ]);
+		$set[] = $champ.'='.spip_mysql_cite($val, @$fields[$champ]);
 	}
 
 	return spip_mysql_query(
@@ -1515,10 +1515,10 @@ 
  *     - False en cas d'erreur.
  **/
 function spip_mysql_replace($table, $couples, $desc = [], $serveur = '', $requeter = true) {
-	return spip_mysql_query("REPLACE $table (" . join(',', array_keys($couples)) . ') VALUES (' . join(
+	return spip_mysql_query("REPLACE $table (".join(',', array_keys($couples)).') VALUES ('.join(
 		',',
 		array_map('_q', $couples)
-	) . ')', $serveur, $requeter);
+	).')', $serveur, $requeter);
 }
 
 
@@ -1547,10 +1547,10 @@ 
  *     - False en cas d'erreur.
  **/
 function spip_mysql_replace_multi($table, $tab_couples, $desc = [], $serveur = '', $requeter = true) {
-	$cles = '(' . join(',', array_keys($tab_couples[0])) . ')';
+	$cles = '('.join(',', array_keys($tab_couples[0])).')';
 	$valeurs = [];
 	foreach ($tab_couples as $couples) {
-		$valeurs[] = '(' . join(',', array_map('_q', $couples)) . ')';
+		$valeurs[] = '('.join(',', array_map('_q', $couples)).')';
 	}
 	$valeurs = implode(', ', $valeurs);
 
@@ -1570,28 +1570,28 @@ 
  */
 function spip_mysql_multi($objet, $lang) {
 	$lengthlang = strlen("[$lang]");
-	$posmulti = 'INSTR(' . $objet . ", '<multi>')";
-	$posfinmulti = 'INSTR(' . $objet . ", '</multi>')";
-	$debutchaine = 'LEFT(' . $objet . ", $posmulti-1)";
-	$finchaine = 'RIGHT(' . $objet . ', CHAR_LENGTH(' . $objet . ") -(7+$posfinmulti))";
-	$chainemulti = 'TRIM(SUBSTRING(' . $objet . ", $posmulti+7, $posfinmulti -(7+$posmulti)))";
-	$poslang = "INSTR($chainemulti,'[" . $lang . "]')";
+	$posmulti = 'INSTR('.$objet.", '<multi>')";
+	$posfinmulti = 'INSTR('.$objet.", '</multi>')";
+	$debutchaine = 'LEFT('.$objet.", $posmulti-1)";
+	$finchaine = 'RIGHT('.$objet.', CHAR_LENGTH('.$objet.") -(7+$posfinmulti))";
+	$chainemulti = 'TRIM(SUBSTRING('.$objet.", $posmulti+7, $posfinmulti -(7+$posmulti)))";
+	$poslang = "INSTR($chainemulti,'[".$lang."]')";
 	$poslang = "IF($poslang=0,INSTR($chainemulti,']')+1,$poslang+$lengthlang)";
-	$chainelang = 'TRIM(SUBSTRING(' . $objet . ", $posmulti+7+$poslang-1,$posfinmulti -($posmulti+7+$poslang-1) ))";
-	$posfinlang = 'INSTR(' . $chainelang . ", '[')";
+	$chainelang = 'TRIM(SUBSTRING('.$objet.", $posmulti+7+$poslang-1,$posfinmulti -($posmulti+7+$poslang-1) ))";
+	$posfinlang = 'INSTR('.$chainelang.", '[')";
 	$chainelang = "IF($posfinlang>0,LEFT($chainelang,$posfinlang-1),$chainelang)";
 	//$chainelang = "LEFT($chainelang,$posfinlang-1)";
-	$retour = "(TRIM(IF($posmulti = 0 , " .
-		'     TRIM(' . $objet . '), ' .
-		'     CONCAT( ' .
-		"          $debutchaine, " .
-		'          IF( ' .
-		"               $poslang = 0, " .
-		"                     $chainemulti, " .
-		"               $chainelang" .
-		'          ), ' .
-		"          $finchaine" .
-		'     ) ' .
+	$retour = "(TRIM(IF($posmulti = 0 , ".
+		'     TRIM('.$objet.'), '.
+		'     CONCAT( '.
+		"          $debutchaine, ".
+		'          IF( '.
+		"               $poslang = 0, ".
+		"                     $chainemulti, ".
+		"               $chainelang".
+		'          ), '.
+		"          $finchaine".
+		'     ) '.
 		'))) AS multi';
 
 	return $retour;
@@ -1608,7 +1608,7 @@ 
  *     Valeur hexadécimale pour MySQL
  **/
 function spip_mysql_hex($v) {
-	return '0x' . $v;
+	return '0x'.$v;
 }
 
 /**
@@ -1648,7 +1648,7 @@ 
  *     Expression SQL
  **/
 function spip_mysql_date_proche($champ, $interval, $unite) {
-	$use_now = ( ($champ === 'maj' or strpos($champ, '.maj')) ? true : false );
+	$use_now = (($champ === 'maj' or strpos($champ, '.maj')) ? true : false);
 	return '('
 	. $champ
 	. (($interval <= 0) ? '>' : '<')
@@ -1703,7 +1703,7 @@ 
 		} elseif ($v === null) {
 			return "''";
 		}
-		return "'" . addslashes($v) . "'";
+		return "'".addslashes($v)."'";
 	}
 
 	if ($v === null) {
@@ -1727,7 +1727,7 @@ 
 		}
 	}
 
-	return ("'" . addslashes($v) . "'");
+	return ("'".addslashes($v)."'");
 }
 
 /**
@@ -1761,7 +1761,7 @@ 
 	} else {
 		$GLOBALS['mysql_rappel_nom_base'] = false;
 
-		return "\$GLOBALS['mysql_rappel_nom_base'] = false; " .
+		return "\$GLOBALS['mysql_rappel_nom_base'] = false; ".
 		"/* echec de test_rappel_nom_base_mysql a l'installation. */\n";
 	}
 }

ecrire/inc/modifier.php

Spacing +14 added lines, -14 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');
 	}
@@ -140,7 +140,7 @@ 
 	// N'accepter que les champs qui existent dans la table
 	$champs = array_intersect_key($c, $desc['field']);
 	// et dont la valeur n'est pas null
-	$champs = array_filter($champs, static function($var){return $var !== null;} );
+	$champs = array_filter($champs, static function($var) {return $var !== null; } );
 	// TODO: ici aussi on peut valider les contenus
 	// en fonction du type
 
@@ -148,7 +148,7 @@ 
 	$champs = array_map('corriger_caracteres', $champs);
 
 	// On récupère l'état avant toute modification
-	$row = sql_fetsel('*', $spip_table_objet, $id_table_objet . '=' . $id_objet);
+	$row = sql_fetsel('*', $spip_table_objet, $id_table_objet.'='.$id_objet);
 
 	// Envoyer aux plugins
 	$champs = pipeline(
@@ -203,7 +203,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);
@@ -225,13 +225,13 @@ 
 		}
 
 		// 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),
+			"$id_table_objet=".intval($id_objet),
 			[],
 			[],
 			'',
@@ -267,13 +267,13 @@ 
 			// 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
+					"Erreur enregistrement en base $objet/$id_objet champs :".var_export($conflits, true),
+					'modifier.'._LOG_CRITIQUE
 				);
 
 				return _T(
 					'erreur_technique_enregistrement_champs',
-					['champs' => "<i>'" . implode("'</i>,<i>'", $liste) . "'</i>"]
+					['champs' => "<i>'".implode("'</i>,<i>'", $liste)."'</i>"]
 				);
 			}
 		}
@@ -315,18 +315,18 @@ 
 	include_spip('inc/filtres_mini');
 	$qui = '';
 	if (!empty($GLOBALS['visiteur_session']['id_auteur'])) {
-		$qui .= ' #id_auteur:' . $GLOBALS['visiteur_session']['id_auteur'] . '#';
+		$qui .= ' #id_auteur:'.$GLOBALS['visiteur_session']['id_auteur'].'#';
 	}
 	if (!empty($GLOBALS['visiteur_session']['nom'])) {
-		$qui .= ' #nom:' . $GLOBALS['visiteur_session']['nom'] . '#';
+		$qui .= ' #nom:'.$GLOBALS['visiteur_session']['nom'].'#';
 	}
 	if ($qui == '') {
-		$qui = '#ip:' . $GLOBALS['ip'] . '#';
+		$qui = '#ip:'.$GLOBALS['ip'].'#';
 	}
-	journal(_L($qui . ' a édité ' . $objet . ' ' . $id_objet . ' (' . join(
+	journal(_L($qui.' a édité '.$objet.' '.$id_objet.' ('.join(
 		'+',
 		array_diff(array_keys($champs), ['date_modif'])
-	) . ')'), [
+	).')'), [
 		'faire' => 'modifier',
 		'quoi' => $objet,
 		'id' => $id_objet

ecrire/inc/filtres_images_lib_mini.php

Spacing +67 added lines, -67 removed lines

@@ -43,13 +43,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";
@@ -67,7 +67,7 @@ 
 	$couleur = couleur_html_to_hex($couleur);
 	$couleur = ltrim($couleur, '#');
 	if (strlen($couleur) === 3) {
-		$couleur = $couleur[0] . $couleur[0] . $couleur[1] . $couleur[1] . $couleur[2] . $couleur[2];
+		$couleur = $couleur[0].$couleur[0].$couleur[1].$couleur[1].$couleur[2].$couleur[2];
 	}
 	$retour = [];
 	$retour['red'] = hexdec(substr($couleur, 0, 2));
@@ -125,9 +125,9 @@ 
 	$var_G = ($G / 255);
 	$var_B = ($B / 255);
 
-	$var_Min = min($var_R, $var_G, $var_B);   //Min. value of RGB
-	$var_Max = max($var_R, $var_G, $var_B);   //Max. value of RGB
-	$del_Max = $var_Max - $var_Min;           //Delta RGB value
+	$var_Min = min($var_R, $var_G, $var_B); //Min. value of RGB
+	$var_Max = max($var_R, $var_G, $var_B); //Max. value of RGB
+	$del_Max = $var_Max - $var_Min; //Delta RGB value
 
 	$L = ($var_Max + $var_Min) / 2;
 
@@ -189,7 +189,7 @@ 
  */
 function _couleur_hsl_to_rgb($H, $S, $L) {
 	// helper
-	$hue_2_rgb = function ($v1, $v2, $vH) {
+	$hue_2_rgb = function($v1, $v2, $vH) {
 		if ($vH < 0) {
 			$vH += 1;
 		}
@@ -321,11 +321,11 @@ 
 		$img = "<img src='$source' />";
 	} elseif (
 		preg_match('@^data:image/([^;]*);base64,(.*)$@isS', $source, $regs)
-		and $extension = _image_trouver_extension_depuis_mime('image/' . $regs[1])
+		and $extension = _image_trouver_extension_depuis_mime('image/'.$regs[1])
 		and in_array($extension, _image_extensions_acceptees_en_entree())
 	) {
 		# gerer img src="data:....base64"
-		$local = sous_repertoire(_DIR_VAR, 'image-data') . md5($regs[2]) . '.' . _image_extension_normalisee($extension);
+		$local = sous_repertoire(_DIR_VAR, 'image-data').md5($regs[2]).'.'._image_extension_normalisee($extension);
 		if (!file_exists($local)) {
 			ecrire_fichier($local, base64_decode($regs[2]));
 		}
@@ -342,7 +342,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 '';
 		}
@@ -440,9 +440,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);
@@ -453,7 +453,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;
 
@@ -491,15 +491,15 @@ 
 
 	if ($creer) {
 		spip_log(
-			'filtre image ' . ($fonction_creation ? reset($fonction_creation) : '') . "[$effet] sur $fichier",
-			'images' . _LOG_DEBUG
+			'filtre image '.($fonction_creation ? reset($fonction_creation) : '')."[$effet] sur $fichier",
+			'images'._LOG_DEBUG
 		);
 	}
 
 	$term_fonction = _image_trouver_extension_pertinente($fichier);
-	$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'] = _image_extension_normalisee($terminaison);
 	$ret['format_dest'] = $terminaison_dest;
@@ -648,7 +648,7 @@ 
 
 	$_terminaison = _image_trouver_extension_depuis_mime($mime);
 	if ($_terminaison and $_terminaison !== $terminaison) {
-		spip_log("Mauvaise extension du fichier : $path . Son type mime est : $mime", 'images.' . _LOG_INFO_IMPORTANTE);
+		spip_log("Mauvaise extension du fichier : $path . Son type mime est : $mime", 'images.'._LOG_INFO_IMPORTANTE);
 		$terminaison = $_terminaison;
 	}
 	return $terminaison;
@@ -805,7 +805,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);
@@ -840,7 +840,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);
@@ -880,7 +880,7 @@ 
 	if (!function_exists('imagejpeg')) {
 		return false;
 	}
-	$tmp = $fichier . '.tmp';
+	$tmp = $fichier.'.tmp';
 
 	// Enable interlancing
 	imageinterlace($img, true);
@@ -941,7 +941,7 @@ 
 	if (!function_exists('imagewebp')) {
 		return false;
 	}
-	$tmp = $fichier . '.tmp';
+	$tmp = $fichier.'.tmp';
 	$ret = imagewebp($img, $tmp, $qualite);
 	if (file_exists($tmp)) {
 		$taille_test = getimagesize($tmp);
@@ -975,7 +975,7 @@ 
  */
 function _image_imagesvg($img, $fichier) {
 
-	$tmp = $fichier . '.tmp';
+	$tmp = $fichier.'.tmp';
 	if (strpos($img, '<') === false) {
 		$img = supprimer_timestamp($img);
 		if (!file_exists($img)) {
@@ -1032,13 +1032,13 @@ 
  */
 function _image_gd_output($img, $valeurs, $qualite = _IMG_GD_QUALITE, $fonction = null) {
 	if (is_null($fonction)) {
-		$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
@@ -1049,7 +1049,7 @@ 
 			// dans tous les cas mettre a jour la taille de l'image finale
 			[$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);
 		}
 	}
 
@@ -1224,7 +1224,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', '');
 	}
 
@@ -1351,7 +1351,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'];
 
@@ -1383,14 +1383,14 @@ 
 
 	// 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);
 	}
 
 	elseif ($valeurs['format_source'] === 'svg') {
 		if ($svg = svg_redimensionner($valeurs['fichier'], $destWidth, $destHeight)) {
 			$format_sortie = 'svg';
-			$vignette = $destination . '.' . $format_sortie;
+			$vignette = $destination.'.'.$format_sortie;
 			$valeurs['fichier_dest'] = $vignette;
 			_image_gd_output($svg, $valeurs);
 		}
@@ -1402,9 +1402,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(
 			['%x', '%y', '%src', '%dest'],
 			[
@@ -1420,7 +1420,7 @@ 
 		if (!@file_exists($vignette)) {
 			spip_log("echec convert sur $vignette");
 
-			return;  // echec commande
+			return; // echec commande
 		}
 	}
 
@@ -1438,7 +1438,7 @@ 
 		if (!$output) {
 			return;
 		}
-		$vignette = $output . DIRECTORY_SEPARATOR . basename($destination) . '.' . $format_sortie;
+		$vignette = $output.DIRECTORY_SEPARATOR.basename($destination).'.'.$format_sortie;
 
 		$imagick = new Imagick();
 		$imagick->readImage(realpath($image));
@@ -1447,7 +1447,7 @@ 
 			$destHeight,
 			Imagick::FILTER_LANCZOS,
 			1
-		);//, IMAGICK_FILTER_LANCZOS, _IMG_IMAGICK_QUALITE / 100);
+		); //, IMAGICK_FILTER_LANCZOS, _IMG_IMAGICK_QUALITE / 100);
 		$imagick->writeImage($vignette);
 
 		if (!@file_exists($vignette)) {
@@ -1456,7 +1456,7 @@ 
 			return;
 		}
 		// remettre le chemin relatif car c'est ce qu'attend SPIP pour la suite (en particlier action/tester)
-		$vignette = $destination . '.' . $format_sortie;
+		$vignette = $destination.'.'.$format_sortie;
 	}
 
 	// netpbm
@@ -1467,11 +1467,11 @@ 
 		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);
 			}
@@ -1483,7 +1483,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);
 				}
@@ -1495,7 +1495,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);
 					}
@@ -1517,7 +1517,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;
 		}
@@ -1829,7 +1829,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, ['src' => "$logo$date", 'width' => $destWidth, 'height' => $destHeight]);
 	}
@@ -1875,7 +1875,7 @@ 
 	public static function LittleEndian2String($number, $minbytes = 1) {
 		$intstring = '';
 		while ($number > 0) {
-			$intstring = $intstring . chr($number & 255);
+			$intstring = $intstring.chr($number & 255);
 			$number >>= 8;
 		}
 
@@ -1907,9 +1907,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) {
@@ -1936,48 +1936,48 @@ 
 
 			// 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?
+			); // dwBytesInRes;	// How many bytes in this resource?
 
 			$icondata .= phpthumb_functions::LittleEndian2String(
 				$dwImageOffset,
 				4
-			);    // dwImageOffset;   // Where in the file is this image?
+			); // dwImageOffset;   // Where in the file is this image?
 			$dwImageOffset += strlen($BitmapInfoHeader[$key]);
 			$dwImageOffset += strlen($icXOR[$key]);
 			$dwImageOffset += strlen($icAND[$key]);