spip / SPIP

ecrire/public/compiler.php

Spacing +115 added lines, -123 removed lines

@@ -97,7 +97,7 @@ 
 				preg_match(",^([^=]*)(=?)(.*)$,m", $var->texte, $m);
 				$m = array_pad($m, 3, null);
 				$var = $m[1];
-				$auto = false;;
+				$auto = false; ;
 				if ($m[2]) {
 					$v = $m[3];
 					if (preg_match(',^[\'"](.*)[\'"]$,', $v, $m)) {
@@ -121,7 +121,7 @@ 
 						: calculer_liste($val, $p->descr, $boucles, $id_boucle);
 					if ($var !== 1) {
 						$val = ($echap ? "\'$var\' => ' . argumenter_squelette(" : "'$var' => ")
-							. $val . ($echap ? ") . '" : " ");
+							. $val.($echap ? ") . '" : " ");
 					} else {
 						$val = $echap ? "'.$val.'" : $val;
 					}
@@ -142,7 +142,7 @@ 
 	if (!$lang) {
 		$lang = '$GLOBALS["spip_lang"]';
 	}
-	$l['lang'] = ($echap ? "\'lang\' => ' . argumenter_squelette(" : "'lang' => ") . $lang . ($echap ? ") . '" : " ");
+	$l['lang'] = ($echap ? "\'lang\' => ' . argumenter_squelette(" : "'lang' => ").$lang.($echap ? ") . '" : " ");
 
 	return $l;
 }
@@ -171,7 +171,7 @@ 
 	$_contexte = argumenter_inclure($p->param, false, $p, $boucles, $id_boucle, true, '', true);
 	if (is_string($p->texte)) {
 		$fichier = $p->texte;
-		$code = "\"".str_replace('"','\"',$fichier)."\"";
+		$code = "\"".str_replace('"', '\"', $fichier)."\"";
 
 	} else {
 		$code = calculer_liste($p->texte, $p->descr, $boucles, $id_boucle);
@@ -185,7 +185,7 @@ 
 		$trace = $p->fonctions;
 		while (is_array($trace)
 		  and $trace = array_filter($trace)
-			and count($trace)==1) {
+			and count($trace) == 1) {
 			$trace = reset($trace);
 		}
 		$erreur_p_i_i = array(
@@ -219,7 +219,7 @@ 
 		return false;
 	} // j'aurais voulu toucher le fond ...
 
-	$contexte = 'array(' . $_contexte . ')';
+	$contexte = 'array('.$_contexte.')';
 
 	if ($env) {
 		$contexte = "array_merge('.var_export(\$Pile[0],1).',$contexte)";
@@ -234,11 +234,11 @@ 
 			$_options[] = $ajax;
 		}
 		$code = " ' . argumenter_squelette($code) . '";
-		$code = "echo " . sprintf(CODE_RECUPERER_FOND, $code, $contexte, implode(',', $_options),
-				"_request(\"connect\")") . ';';
+		$code = "echo ".sprintf(CODE_RECUPERER_FOND, $code, $contexte, implode(',', $_options),
+				"_request(\"connect\")").';';
 	}
 
-	return "\n'<'.'" . "?php " . $code . "\n?'." . "'>'";
+	return "\n'<'.'"."?php ".$code."\n?'."."'>'";
 }
 
 
@@ -318,17 +318,17 @@ 
 					$id = $id_table;
 					$statut = preg_replace(',\W,', '', $s['champ']); // securite
 				}
-				$mstatut = $id . '.' . $statut;
+				$mstatut = $id.'.'.$statut;
 
 				$arg_ignore_previsu = ($ignore_previsu ? ",true" : '');
 				include_spip('public/quete');
 				if (isset($s['post_date']) and $s['post_date']
 					and $GLOBALS['meta']["post_dates"] == 'non'
 				) {
-					$date = $id . '.' . preg_replace(',\W,', '', $s['post_date']); // securite
+					$date = $id.'.'.preg_replace(',\W,', '', $s['post_date']); // securite
 					array_unshift($boucle->where,
 						$echapper ?
-							"\nquete_condition_postdates('$date'," . _q($boucle->sql_serveur) . "$arg_ignore_previsu)"
+							"\nquete_condition_postdates('$date',"._q($boucle->sql_serveur)."$arg_ignore_previsu)"
 							:
 							quete_condition_postdates($date, $boucle->sql_serveur, $ignore_previsu)
 					);
@@ -336,9 +336,9 @@ 
 				array_unshift($boucle->where,
 					$echapper ?
 						"\nquete_condition_statut('$mstatut',"
-						. _q($s['previsu']) . ","
-						. _q($s['publie']) . ","
-						. _q($boucle->sql_serveur) . "$arg_ignore_previsu)"
+						. _q($s['previsu']).","
+						. _q($s['publie']).","
+						. _q($boucle->sql_serveur)."$arg_ignore_previsu)"
 						:
 						quete_condition_statut($mstatut, $s['previsu'], $s['publie'], $boucle->sql_serveur, $ignore_previsu)
 				);
@@ -372,14 +372,14 @@ 
 	if (_request('var_mode_affiche') != 'resultat') {
 		$trace = '';
 	} else {
-		$_trace = $boucles[$id_boucle]->descr['nom'] . $id_boucle;
+		$_trace = $boucles[$id_boucle]->descr['nom'].$id_boucle;
 		$_trace = "\$GLOBALS['debug_objets']['resultat']['$_trace']";
 		$trace = "
 		if (empty($_trace)) { 
 			$_trace = []; 
 		}
 		if (count($_trace) < 3) { 
-			$_trace" . "[] = \$t0; 
+			$_trace"."[] = \$t0; 
 		}";
 	}
 
@@ -413,7 +413,7 @@ 
 	return
 		// Numrows[$nom] peut ne pas être encore defini
 		"\n\t\$save_numrows = (isset(\$Numrows['$nom']) ? \$Numrows['$nom'] : array());"
-		. "\n\t\$t0 = " . $boucles[$id_boucle]->return . ";"
+		. "\n\t\$t0 = ".$boucles[$id_boucle]->return.";"
 		. "\n\t\$Numrows['$nom'] = (\$save_numrows);"
 		. $trace
 		. "\n\treturn \$t0;";
@@ -481,7 +481,7 @@ 
 	// faudrait expanser le foreach a la compil, car y en a souvent qu'un 
 	// et puis faire un [] plutot qu'un "','."
 	if ($boucle->doublons) {
-		$corps .= "\n\t\t\tforeach(" . $boucle->doublons . ' as $k) $doublons[$k] .= "," . ' .
+		$corps .= "\n\t\t\tforeach(".$boucle->doublons.' as $k) $doublons[$k] .= "," . '.
 			index_pile($id_boucle, $primary, $boucles)
 			. "; // doublons\n";
 	}
@@ -508,13 +508,13 @@ 
 		$corps .=
 			"\n\t\tlang_select_public("
 			. index_pile($id_boucle, 'lang', $boucles)
-			. ", '" . $boucle->lang_select . "'"
+			. ", '".$boucle->lang_select."'"
 			. (in_array($type_boucle, array(
 				'articles',
 				'rubriques',
 				'hierarchie',
 				'breves'
-			)) ? ', ' . index_pile($id_boucle, 'titre', $boucles) : '')
+			)) ? ', '.index_pile($id_boucle, 'titre', $boucles) : '')
 			. ');';
 	} else {
 		$init_lang = '';
@@ -535,26 +535,22 @@ 
 
 	// gestion optimale des separateurs et des boucles constantes
 	if (count($boucle->separateur)) {
-		$code_sep = ("'" . str_replace("'", "\'", join('', $boucle->separateur)) . "'");
+		$code_sep = ("'".str_replace("'", "\'", join('', $boucle->separateur))."'");
 	}
 
 	$corps .=
 		((!$boucle->separateur) ?
-			(($constant && !$corps && !$flag_cpt) ? $return :
-				(($return === "''") ? '' :
-					("\n\t\t" . '$t0 .= ' . $return . ";"))) :
-			("\n\t\t\$t1 " .
+			(($constant && !$corps && !$flag_cpt) ? $return : (($return === "''") ? '' : ("\n\t\t".'$t0 .= '.$return.";"))) : ("\n\t\t\$t1 ".
 				((strpos($return, '$t1.') === 0) ?
-					(".=" . substr($return, 4)) :
-					('= ' . $return)) .
-				";\n\t\t" .
-				'$t0 .= ((strlen($t1) && strlen($t0)) ? ' . $code_sep . " : '') . \$t1;"));
+					(".=".substr($return, 4)) : ('= '.$return)).
+				";\n\t\t".
+				'$t0 .= ((strlen($t1) && strlen($t0)) ? '.$code_sep." : '') . \$t1;"));
 
 	// Calculer les invalideurs si c'est une boucle non constante et si on
 	// souhaite invalider ces elements
 	if (!$constant and $primary) {
 		include_spip('inc/invalideur');
-		$corps = calcul_invalideurs($corps, $primary,$boucles, $id_boucle);
+		$corps = calcul_invalideurs($corps, $primary, $boucles, $id_boucle);
 	}
 
 	// gerer le compteur de boucle 
@@ -629,7 +625,7 @@ 
 		$corps,
 		$fin_lang,
 		$trace,
-		'BOUCLE' . $id_boucle . ' @ ' . ($boucle->descr['sourcefile'])
+		'BOUCLE'.$id_boucle.' @ '.($boucle->descr['sourcefile'])
 	);
 
 #	var_dump($a);exit;
@@ -650,20 +646,20 @@ 
  **/
 function calculer_requete_sql($boucle) {
 	$init = array();
-	$init[] = calculer_dec('table', "'" . $boucle->id_table . "'");
-	$init[] = calculer_dec('id', "'" . $boucle->id_boucle . "'");
+	$init[] = calculer_dec('table', "'".$boucle->id_table."'");
+	$init[] = calculer_dec('id', "'".$boucle->id_boucle."'");
 	# En absence de champ c'est un decompte :
 	$init[] = calculer_dec('from', calculer_from($boucle));
 	$init[] = calculer_dec('type', calculer_from_type($boucle));
 	$init[] = calculer_dec('groupby',
-		'array(' . (($g = join("\",\n\t\t\"", $boucle->group)) ? '"' . $g . '"' : '') . ")");
-	$init[] = calculer_dec('select', 'array("' . join("\",\n\t\t\"", $boucle->select) . "\")");
-	$init[] = calculer_dec('orderby', 'array(' . calculer_order($boucle) . ")");
+		'array('.(($g = join("\",\n\t\t\"", $boucle->group)) ? '"'.$g.'"' : '').")");
+	$init[] = calculer_dec('select', 'array("'.join("\",\n\t\t\"", $boucle->select)."\")");
+	$init[] = calculer_dec('orderby', 'array('.calculer_order($boucle).")");
 	$init[] = calculer_dec('where', calculer_dump_array($boucle->where));
 	$init[] = calculer_dec('join', calculer_dump_join($boucle->join));
 	$init[] = calculer_dec('limit',
 		(strpos($boucle->limit, 'intval') === false ?
-			"'" . $boucle->limit . "'"
+			"'".$boucle->limit."'"
 			:
 			$boucle->limit));
 	$init[] = calculer_dec('having', calculer_dump_array($boucle->having));
@@ -672,17 +668,17 @@ 
 	// ou recalculée à chaque passage (vide)
 	foreach ($init as $i) {
 		if (reset($i)) {
-			$s .= "\n\t\t" . end($i);
+			$s .= "\n\t\t".end($i);
 		} # statique
 		else {
-			$d .= "\n\t" . end($i);
+			$d .= "\n\t".end($i);
 		} # dynamique
 	}
 
 	return ($boucle->hierarchie ? "\n\t$boucle->hierarchie" : '')
 	. $boucle->in
 	. $boucle->hash
-	. "\n\t" . 'if (!isset($command[\'table\'])) {'
+	. "\n\t".'if (!isset($command[\'table\'])) {'
 	. $s
 	. "\n\t}"
 	. $d;
@@ -765,7 +761,7 @@ 
  *    - index 1 : Code de l'affectation
  **/
 function calculer_dec($nom, $val) {
-	$static = 'if (!isset($command[\'' . $nom . '\'])) ';
+	$static = 'if (!isset($command[\''.$nom.'\'])) ';
 	// si une variable apparait dans le calcul de la clause
 	// il faut la re-evaluer a chaque passage
 	if (
@@ -782,7 +778,7 @@ 
 		$static = "";
 	}
 
-	return array($static, '$command[\'' . $nom . '\'] = ' . $val . ';');
+	return array($static, '$command[\''.$nom.'\'] = '.$val.';');
 }
 
 /**
@@ -807,17 +803,17 @@ 
 	}
 	$res = "";
 	if ($a and $a[0] == "'?'") {
-		return ("(" . calculer_dump_array($a[1]) .
-			" ? " . calculer_dump_array($a[2]) .
-			" : " . calculer_dump_array($a[3]) .
+		return ("(".calculer_dump_array($a[1]).
+			" ? ".calculer_dump_array($a[2]).
+			" : ".calculer_dump_array($a[3]).
 			")");
 	} else {
 		foreach ($a as $k => $v) {
-			$showk = (is_numeric($k) ? '' : sql_quote($k) . ' => ');
-			$res .= ", " . $showk . calculer_dump_array($v);
+			$showk = (is_numeric($k) ? '' : sql_quote($k).' => ');
+			$res .= ", ".$showk.calculer_dump_array($v);
 		}
 
-		return "\n\t\t\tarray(" . substr($res, 2) . ')';
+		return "\n\t\t\tarray(".substr($res, 2).')';
 	}
 }
 
@@ -825,10 +821,10 @@ 
 function calculer_dump_join($a) {
 	$res = "";
 	foreach ($a as $k => $v) {
-		$res .= ", '$k' => array(" . implode(',', $v) . ")";
+		$res .= ", '$k' => array(".implode(',', $v).")";
 	}
 
-	return 'array(' . substr($res, 2) . ')';
+	return 'array('.substr($res, 2).')';
 }
 
 /**
@@ -845,7 +841,7 @@ 
 		$res .= ",'$k' => '$v'";
 	}
 
-	return 'array(' . substr($res, 1) . ')';
+	return 'array('.substr($res, 1).')';
 }
 
 /**
@@ -863,7 +859,7 @@ 
 		$res .= ",'$k' => '$v'";
 	}
 
-	return 'array(' . substr($res, 1) . ')';
+	return 'array('.substr($res, 1).')';
 }
 
 // https://code.spip.net/@calculer_order
@@ -932,17 +928,17 @@ 
 				) {
 					$res .= " .\n$tab$code";
 				} else {
-					$res = substr($res, 0, -1) . substr($code, 1);
+					$res = substr($res, 0, -1).substr($code, 1);
 				}
 			}
 
-			return '(' . substr($res, 2 + $descr['niv']) . ')';
+			return '('.substr($res, 2 + $descr['niv']).')';
 		}
 	} else {
-		$nom = $descr['nom'] . $id_boucle . ($descr['niv'] ? $descr['niv'] : '');
+		$nom = $descr['nom'].$id_boucle.($descr['niv'] ? $descr['niv'] : '');
 
-		return "join('', array_map('array_shift', \$GLOBALS['debug_objets']['sequence']['$nom'] = array(" . join(" ,\n$tab",
-			$codes) . ")))";
+		return "join('', array_map('array_shift', \$GLOBALS['debug_objets']['sequence']['$nom'] = array(".join(" ,\n$tab",
+			$codes).")))";
 	}
 }
 
@@ -971,7 +967,7 @@ 
 			// texte seul
 			case 'texte':
 				$code = sandbox_composer_texte($p->texte, $p);
-				$commentaire = strlen($p->texte) . " signes";
+				$commentaire = strlen($p->texte)." signes";
 				$avant = '';
 				$apres = '';
 				$altern = "''";
@@ -980,14 +976,14 @@ 
 			case 'polyglotte':
 				$code = "";
 				foreach ($p->traductions as $k => $v) {
-					$code .= ",'" .
-						str_replace(array("\\", "'"), array("\\\\", "\\'"), $k) .
-						"' => '" .
-						str_replace(array("\\", "'"), array("\\\\", "\\'"), $v) .
+					$code .= ",'".
+						str_replace(array("\\", "'"), array("\\\\", "\\'"), $k).
+						"' => '".
+						str_replace(array("\\", "'"), array("\\\\", "\\'"), $v).
 						"'";
 				}
-				$code = "choisir_traduction(array(" .
-					substr($code, 1) .
+				$code = "choisir_traduction(array(".
+					substr($code, 1).
 					"))";
 				$commentaire = '&';
 				$avant = '';
@@ -1003,7 +999,7 @@ 
 					$err_e_c = true;
 					$code = "''";
 				} else {
-					$commentaire = '<INCLURE ' . addslashes(str_replace("\n", ' ', $code)) . '>';
+					$commentaire = '<INCLURE '.addslashes(str_replace("\n", ' ', $code)).'>';
 					$avant = '';
 					$apres = '';
 					$altern = "''";
@@ -1030,8 +1026,8 @@ 
 					$err_e_c = true;
 					$code = "''";
 				} else {
-					$code = 'BOUCLE' .
-						str_replace("-", "_", $nom) . $descr['nom'] .
+					$code = 'BOUCLE'.
+						str_replace("-", "_", $nom).$descr['nom'].
 						'($Cache, $Pile, $doublons, $Numrows, $SP)';
 					$commentaire = "?$nom";
 					if (!$boucles[$nom]->milieu
@@ -1072,24 +1068,22 @@ 
 				foreach ($p->arg as $k => $v) {
 					$_v = calculer_liste($v, $descr, $boucles, $id_boucle);
 					if ($k) {
-						$l[] = _q($k) . ' => ' . $_v;
+						$l[] = _q($k).' => '.$_v;
 					} else {
 						$code = $_v;
 					}
 				}
 				// Si le module n'est pas fourni, l'expliciter sauf si calculé
 				if ($p->module) {
-					$m = $p->module . ':' . $p->nom_champ;
+					$m = $p->module.':'.$p->nom_champ;
 				} elseif ($p->nom_champ) {
-					$m = MODULES_IDIOMES . ':' . $p->nom_champ;
+					$m = MODULES_IDIOMES.':'.$p->nom_champ;
 				} else {
 					$m = '';
 				}
 
-				$code = (!$code ? "'$m'" :
-						($m ? "'$m' . $code" :
-							("(strpos(\$x=$code, ':') ? \$x : ('" . MODULES_IDIOMES . ":' . \$x))")))
-					. (!$l ? '' : (", array(" . implode(",\n", $l) . ")"));
+				$code = (!$code ? "'$m'" : ($m ? "'$m' . $code" : ("(strpos(\$x=$code, ':') ? \$x : ('".MODULES_IDIOMES.":' . \$x))")))
+					. (!$l ? '' : (", array(".implode(",\n", $l).")"));
 				$code = "_T($code)";
 				if ($p->param) {
 					$p->id_boucle = $id_boucle;
@@ -1112,7 +1106,7 @@ 
 				$p->type_requete = $type;
 
 				$code = calculer_champ($p);
-				$commentaire = '#' . $p->nom_champ . $p->etoile;
+				$commentaire = '#'.$p->nom_champ.$p->etoile;
 				$avant = calculer_liste($p->avant,
 					$descr, $boucles, $id_boucle);
 				$apres = calculer_liste($p->apres,
@@ -1143,10 +1137,9 @@ 
 		if ($code != "''") {
 			$code = compile_retour($code, $avant, $apres, $altern, $tab, $descr['niv']);
 			$codes[] = (($mode == 'validation') ?
-				"array($code, '$commentaire', " . $p->ligne . ")"
+				"array($code, '$commentaire', ".$p->ligne.")"
 				: (($mode == 'code') ?
-					"\n// $commentaire\n$code" :
-					$code));
+					"\n// $commentaire\n$code" : $code));
 		}
 	} // foreach
 
@@ -1197,28 +1190,28 @@ 
 	if ($apres === "''") {
 		$apres = '';
 	}
-	if ($avant or $apres or ($altern !== "''")){
-		if (preg_match(_REGEXP_CONCAT_NON_VIDE, $code)){
+	if ($avant or $apres or ($altern !== "''")) {
+		if (preg_match(_REGEXP_CONCAT_NON_VIDE, $code)) {
 			$t = $code;
 			$cond = '';
 		} elseif (preg_match(_REGEXP_COND_VIDE_NONVIDE, $code, $r)) {
 			$t = $r[2];
-			$cond = '!' . $r[1];
+			$cond = '!'.$r[1];
 		} else {
-			if (preg_match(_REGEXP_COND_NONVIDE_VIDE, $code, $r)){
+			if (preg_match(_REGEXP_COND_NONVIDE_VIDE, $code, $r)) {
 				$t = $r[2];
 				$cond = $r[1];
 			} else {
-				$t = '$t' . $n;
+				$t = '$t'.$n;
 				$cond = "($t = $code)!==''";
 			}
 		}
 
-		$res = (!$avant ? "" : "$avant . ") .
-			$t .
+		$res = (!$avant ? "" : "$avant . ").
+			$t.
 			(!$apres ? "" : " . $apres");
 
-		if ($res!==$t){
+		if ($res !== $t) {
 			$res = "($res)";
 		}
 
@@ -1268,12 +1261,12 @@ 
 
 	// rendre inertes les echappements de #[](){}<>
 	$i = 0;
-	while (false !== strpos($squelette, $inerte = '-INERTE' . $i)) {
+	while (false !== strpos($squelette, $inerte = '-INERTE'.$i)) {
 		$i++;
 	}
 	$squelette = preg_replace_callback(',\\\\([#[()\]{}<>]),',
 		function($a) use ($inerte) {
-			return "$inerte-" . ord($a[1]) . '-';
+			return "$inerte-".ord($a[1]).'-';
 		},
 		$squelette,
 		-1,
@@ -1290,7 +1283,7 @@ 
 	// Phraser le squelette, selon sa grammaire
 
 	$boucles = array();
-	$f = charger_fonction('phraser_' . $gram, 'public');
+	$f = charger_fonction('phraser_'.$gram, 'public');
 
 	$squelette = $f($squelette, '', $boucles, $descr);
 
@@ -1309,7 +1302,7 @@ 
 			$boucles[$i]->descr['squelette'] = preg_replace_callback(
 				",$inerte-(\d+)-,",
 				function($a) {
-					return "\\\\" . chr($a[1]);
+					return "\\\\".chr($a[1]);
 				},
 				$boucle->descr['squelette']
 			);
@@ -1321,19 +1314,19 @@ 
 		include_spip('public/decompiler');
 		foreach ($boucles as $id => $boucle) {
 			if ($id) {
-				$decomp = "\n/* BOUCLE " .
-					$boucle->type_requete .
-					" " .
-					str_replace('*/', '* /', public_decompiler($boucle, $gram, 0, 'criteres')) .
-					($boucle->debug ? "\n *\n * " . implode("\n * ", $boucle->debug) . "\n" : '') .
+				$decomp = "\n/* BOUCLE ".
+					$boucle->type_requete.
+					" ".
+					str_replace('*/', '* /', public_decompiler($boucle, $gram, 0, 'criteres')).
+					($boucle->debug ? "\n *\n * ".implode("\n * ", $boucle->debug)."\n" : '').
 					" */\n";
 			} else {
-				$decomp = ("\n/*\n" .
+				$decomp = ("\n/*\n".
 					str_replace('*/', '* /', public_decompiler($squelette, $gram))
 					. "\n*/");
 			}
-			$boucles[$id]->return = $decomp . $boucle->return;
-			$GLOBALS['debug_objets']['code'][$nom . $id] = $boucle->return;
+			$boucles[$id]->return = $decomp.$boucle->return;
+			$GLOBALS['debug_objets']['code'][$nom.$id] = $boucle->return;
 		}
 	}
 
@@ -1356,7 +1349,7 @@ 
 		}
 	}
 	foreach ($boucles as $id => $boucle) {
-		$GLOBALS['debug_objets']['boucle'][$nom . $id] = $boucle;
+		$GLOBALS['debug_objets']['boucle'][$nom.$id] = $boucle;
 	}
 	$descr['documents'] = compile_inclure_doublons($squelette);
 
@@ -1440,8 +1433,7 @@ 
 						} else {
 							$boucles[$id]->type_requete = false;
 							$boucle = $boucles[$id];
-							$x = (!$boucle->sql_serveur ? '' :
-									($boucle->sql_serveur . ":")) .
+							$x = (!$boucle->sql_serveur ? '' : ($boucle->sql_serveur.":")).
 								$type;
 							$msg = array(
 								'zbug_table_inconnue',
@@ -1527,13 +1519,13 @@ 
 			if (
 				// fonction de boucle avec serveur & table
 				(!$serveur or
-					((!function_exists($f = "boucle_" . $serveur . "_" . $table))
-						and (!function_exists($f = $f . "_dist"))
+					((!function_exists($f = "boucle_".$serveur."_".$table))
+						and (!function_exists($f = $f."_dist"))
 					)
 				)
 				// fonction de boucle avec table
-				and (!function_exists($f = "boucle_" . $table))
-				and (!function_exists($f = $f . "_dist"))
+				and (!function_exists($f = "boucle_".$table))
+				and (!function_exists($f = $f."_dist"))
 			) {
 				// fonction de boucle standard 
 				if (!function_exists($f = 'boucle_DEFAUT')) {
@@ -1541,20 +1533,20 @@ 
 				}
 			}
 
-			$req = "\n\n\tstatic \$command = array();\n\t" .
-				"static \$connect;\n\t" .
-				"\$command['connect'] = \$connect = " .
-				_q($boucle->sql_serveur) .
-				";" .
+			$req = "\n\n\tstatic \$command = array();\n\t".
+				"static \$connect;\n\t".
+				"\$command['connect'] = \$connect = ".
+				_q($boucle->sql_serveur).
+				";".
 				$f($id, $boucles);
 		} else {
 			$req = ("\n\treturn '';");
 		}
 
 		$boucles[$id]->return =
-			"\n\nfunction BOUCLE" . strtr($id, "-", "_") . $nom .
-			'(&$Cache, &$Pile, &$doublons, &$Numrows, $SP) {' .
-			$req .
+			"\n\nfunction BOUCLE".strtr($id, "-", "_").$nom.
+			'(&$Cache, &$Pile, &$doublons, &$Numrows, $SP) {'.
+			$req.
 			"\n}\n";
 	}
 
@@ -1564,7 +1556,7 @@ 
 		return false;
 	}
 
-	$principal = "\nfunction " . $nom . '($Cache, $Pile, $doublons = array(), $Numrows = array(), $SP = 0) {
+	$principal = "\nfunction ".$nom.'($Cache, $Pile, $doublons = array(), $Numrows = array(), $SP = 0) {
 '
 		// reporter de maniere securisee les doublons inclus
 		. '
@@ -1572,15 +1564,15 @@ 
 		$doublons = nettoyer_env_doublons($Pile[0]["doublons"]);
 
 	$connect = ' .
-		_q($connect) . ';
+		_q($connect).';
 	$page = ' .
 		// ATTENTION, le calcul de l'expression $corps affectera $Cache
 		// c'est pourquoi on l'affecte a la variable auxiliaire $page.
 		// avant de referencer $Cache
-		$corps . ";
+		$corps.";
 
 	return analyse_resultat_skel(" . var_export($nom, true)
-		. ", \$Cache, \$page, " . var_export($sourcefile, true) . ");
+		. ", \$Cache, \$page, ".var_export($sourcefile, true).");
 }";
 
 	$secondes = spip_timer('calcul_skel');
@@ -1594,10 +1586,10 @@ 
 	$code->return = '
 //
 // Fonction principale du squelette ' .
-		$sourcefile .
-		($connect ? " pour $connect" : '') .
-		(!CODE_COMMENTE ? '' : "\n// Temps de compilation total: $secondes") .
-		"\n//\n" .
+		$sourcefile.
+		($connect ? " pour $connect" : '').
+		(!CODE_COMMENTE ? '' : "\n// Temps de compilation total: $secondes").
+		"\n//\n".
 		$principal;
 
 	$boucles[''] = $code;
@@ -1649,7 +1641,7 @@ 
  **/
 function requeteur_data_dist(&$boucles, &$boucle, &$id) {
 	include_spip('iterateur/data');
-	if ($h = charger_fonction($boucle->type_requete . '_to_array', 'inc', true)) {
+	if ($h = charger_fonction($boucle->type_requete.'_to_array', 'inc', true)) {
 		$g = charger_fonction('data', 'iterateur');
 		$boucles[$id] = $g($boucle);
 		// from[0] stocke le type de data (rss, yql, ...)

ecrire/inc/recherche_to_array.php

Spacing +8 added lines, -8 removed lines

@@ -68,7 +68,7 @@ 
 		$_id_table = reset($_id_table);
 	}
 
-	$requete['SELECT'][] = "t." . $_id_table;
+	$requete['SELECT'][] = "t.".$_id_table;
 	$a = array();
 	// Recherche fulltext
 	foreach ($champs as $champ => $poids) {
@@ -79,13 +79,13 @@ 
 				$champ = "t.$champ";
 			}
 			$requete['SELECT'][] = $champ;
-			$a[] = $champ . ' ' . $methode . ' ' . $q;
+			$a[] = $champ.' '.$methode.' '.$q;
 		}
 	}
 	if ($a) {
 		$requete['WHERE'][] = join(" OR ", $a);
 	}
-	$requete['FROM'][] = table_objet_sql($table) . ' AS t';
+	$requete['FROM'][] = table_objet_sql($table).' AS t';
 
 	$results = array();
 
@@ -126,7 +126,7 @@ 
 					}
 					if ($options['score']) {
 						// compter les points avec un peu de discernement : on pondere par la longueur du match compte en chars
-						$score += $poids * strlen(implode('',array_column($regs, 0)));
+						$score += $poids * strlen(implode('', array_column($regs, 0)));
 					}
 
 					if ($options['matches']) {
@@ -210,8 +210,8 @@ 
 					$s = sql_select("$primary as $cle_depart, id_objet as $cle_arrivee", $table_liens,
 						array("objet='$table_liee'", sql_in('id_objet', array_keys($ids_trouves))), '', '', '', '', $serveur);
 				} // cas table de liaison generique spip_xxx_yyy
-				elseif ($t = $trouver_table($table_arrivee . "_" . $table_depart, $serveur)
-					or $t = $trouver_table($table_depart . "_" . $table_arrivee, $serveur)
+				elseif ($t = $trouver_table($table_arrivee."_".$table_depart, $serveur)
+					or $t = $trouver_table($table_depart."_".$table_arrivee, $serveur)
 				) {
 					$s = sql_select("$cle_depart,$cle_arrivee", $t["table_sql"], sql_in($cle_arrivee, array_keys($ids_trouves)),
 						'', '', '', '', $serveur);
@@ -238,12 +238,12 @@ 
 				}
 				if (isset($joint['champs']) and $joint['champs']) {
 					foreach ($joint['champs'] as $c => $val) {
-						$results[$id]['champs'][$table_liee . '.' . $c] = $val;
+						$results[$id]['champs'][$table_liee.'.'.$c] = $val;
 					}
 				}
 				if (isset($joint['matches']) and $joint['matches']) {
 					foreach ($joint['matches'] as $c => $val) {
-						$results[$id]['matches'][$table_liee . '.' . $c] = $val;
+						$results[$id]['matches'][$table_liee.'.'.$c] = $val;
 					}
 				}
 			}

config/ecran_securite.php

Spacing +25 added lines, -25 removed lines

@@ -15,10 +15,10 @@ 
  * Test utilisateur
  */
 if (isset($_GET['test_ecran_securite'])) {
-	$ecran_securite_raison = 'test ' . _ECRAN_SECURITE;
+	$ecran_securite_raison = 'test '._ECRAN_SECURITE;
 }
 
-if (file_exists($f = __DIR__ . DIRECTORY_SEPARATOR  . 'ecran_securite_options.php')) {
+if (file_exists($f = __DIR__.DIRECTORY_SEPARATOR.'ecran_securite_options.php')) {
 	include ($f);
 }
 
@@ -226,8 +226,8 @@ 
 						'yats',
 						'yeti',
 						'zeerch'
-					)) . ',i',
-				(string)$_SERVER['HTTP_USER_AGENT']
+					)).',i',
+				(string) $_SERVER['HTTP_USER_AGENT']
 			)
 	);
 }
@@ -236,13 +236,13 @@ 
 		'_IS_BOT_FRIEND',
 		isset($_SERVER['HTTP_USER_AGENT'])
 			and preg_match(
-				',' . implode('|', array(
+				','.implode('|', array(
 					'facebookexternalhit',
 					'twitterbot',
 					'flipboardproxy',
 					'wordpress'
-				)) . ',i',
-				(string)$_SERVER['HTTP_USER_AGENT']
+				)).',i',
+				(string) $_SERVER['HTTP_USER_AGENT']
 			)
 	);
 }
@@ -291,10 +291,10 @@ 
  */
 foreach (array('lang', 'var_recherche', 'aide', 'var_lang_r', 'lang_r', 'var_ajax_ancre', 'nom_fichier') as $var) {
 	if (isset($_GET[$var])) {
-		$_REQUEST[$var] = $GLOBALS[$var] = $_GET[$var] = preg_replace(',[^\w\,/#&;-]+,', ' ', (string)$_GET[$var]);
+		$_REQUEST[$var] = $GLOBALS[$var] = $_GET[$var] = preg_replace(',[^\w\,/#&;-]+,', ' ', (string) $_GET[$var]);
 	}
 	if (isset($_POST[$var])) {
-		$_REQUEST[$var] = $GLOBALS[$var] = $_POST[$var] = preg_replace(',[^\w\,/#&;-]+,', ' ', (string)$_POST[$var]);
+		$_REQUEST[$var] = $GLOBALS[$var] = $_POST[$var] = preg_replace(',[^\w\,/#&;-]+,', ' ', (string) $_POST[$var]);
 	}
 }
 
@@ -302,8 +302,8 @@ 
  * Filtre l'accès à spip_acces_doc (injection SQL en 1.8.2x)
  */
 if (isset($_SERVER['REQUEST_URI'])) {
-	if (preg_match(',^(.*/)?spip_acces_doc\.,', (string)$_SERVER['REQUEST_URI'])) {
-		$file = addslashes((string)$_GET['file']);
+	if (preg_match(',^(.*/)?spip_acces_doc\.,', (string) $_SERVER['REQUEST_URI'])) {
+		$file = addslashes((string) $_GET['file']);
 	}
 }
 
@@ -323,13 +323,13 @@ 
  */
 if (
 	isset($_REQUEST['partie_cal'])
-	and $_REQUEST['partie_cal'] !== htmlentities((string)$_REQUEST['partie_cal'])
+	and $_REQUEST['partie_cal'] !== htmlentities((string) $_REQUEST['partie_cal'])
 ) {
 	$ecran_securite_raison = "partie_cal";
 }
 if (
 	isset($_REQUEST['echelle'])
-	and $_REQUEST['echelle'] !== htmlentities((string)$_REQUEST['echelle'])
+	and $_REQUEST['echelle'] !== htmlentities((string) $_REQUEST['echelle'])
 ) {
 	$ecran_securite_raison = "echelle";
 }
@@ -339,13 +339,13 @@ 
  */
 if (
 	isset($_REQUEST['exec'])
-	and !preg_match(',^[\w-]+$,', (string)$_REQUEST['exec'])
+	and !preg_match(',^[\w-]+$,', (string) $_REQUEST['exec'])
 ) {
 	$ecran_securite_raison = "exec";
 }
 if (
 	isset($_REQUEST['cherche_auteur'])
-	and preg_match(',[<],', (string)$_REQUEST['cherche_auteur'])
+	and preg_match(',[<],', (string) $_REQUEST['cherche_auteur'])
 ) {
 	$ecran_securite_raison = "cherche_auteur";
 }
@@ -353,7 +353,7 @@ 
 	isset($_REQUEST['exec'])
 	and $_REQUEST['exec'] == 'auteurs'
 	and isset($_REQUEST['recherche'])
-	and preg_match(',[<],', (string)$_REQUEST['recherche'])
+	and preg_match(',[<],', (string) $_REQUEST['recherche'])
 ) {
 	$ecran_securite_raison = "recherche";
 }
@@ -361,7 +361,7 @@ 
 	isset($_REQUEST['exec'])
 	and $_REQUEST['exec'] == 'info_plugin'
 	and isset($_REQUEST['plugin'])
-	and preg_match(',[<],', (string)$_REQUEST['plugin'])
+	and preg_match(',[<],', (string) $_REQUEST['plugin'])
 ) {
 	$ecran_securite_raison = "plugin";
 }
@@ -388,7 +388,7 @@ 
 				echo minipres(_T('info_acces_interdit'));
 				exit;
 			}
-			require _DIR_RESTREINT . 'action/configurer.php';
+			require _DIR_RESTREINT.'action/configurer.php';
 			action_configurer_dist();
 		}
 	}
@@ -440,9 +440,9 @@ 
 if (_IS_BOT) {
 	if (
 		(isset($_REQUEST['echelle']) and isset($_REQUEST['partie_cal']) and isset($_REQUEST['type']))
-		or (strpos((string)$_SERVER['REQUEST_URI'], 'debut_') and preg_match(',[?&]debut_.*&debut_,', (string)$_SERVER['REQUEST_URI']))
-		or (isset($_REQUEST['calendrier_annee']) and strpos((string)$_SERVER['REQUEST_URI'], 'debut_'))
-		or (isset($_REQUEST['calendrier_annee']) and preg_match(',[?&]calendrier_annee=.*&calendrier_annee=,', (string)$_SERVER['REQUEST_URI']))
+		or (strpos((string) $_SERVER['REQUEST_URI'], 'debut_') and preg_match(',[?&]debut_.*&debut_,', (string) $_SERVER['REQUEST_URI']))
+		or (isset($_REQUEST['calendrier_annee']) and strpos((string) $_SERVER['REQUEST_URI'], 'debut_'))
+		or (isset($_REQUEST['calendrier_annee']) and preg_match(',[?&]calendrier_annee=.*&calendrier_annee=,', (string) $_SERVER['REQUEST_URI']))
 	) {
 		$ecran_securite_raison = "robot agenda/double pagination";
 	}
@@ -456,7 +456,7 @@ 
 	if ($_REQUEST['page'] == 'test_cfg') {
 		$ecran_securite_raison = "test_cfg";
 	}
-	if ($_REQUEST['page'] !== htmlspecialchars((string)$_REQUEST['page'])) {
+	if ($_REQUEST['page'] !== htmlspecialchars((string) $_REQUEST['page'])) {
 		$ecran_securite_raison = "xsspage";
 	}
 	if (
@@ -472,7 +472,7 @@ 
  */
 foreach (array('var_login') as $var) {
 	if (isset($_REQUEST[$var]) and is_array($_REQUEST[$var])) {
-		$ecran_securite_raison = "xss " . $var;
+		$ecran_securite_raison = "xss ".$var;
 	}
 }
 
@@ -520,13 +520,13 @@ 
  */
 if (
 	isset($_REQUEST['nom_sauvegarde'])
-	and strstr((string)$_REQUEST['nom_sauvegarde'], '/')
+	and strstr((string) $_REQUEST['nom_sauvegarde'], '/')
 ) {
 	$ecran_securite_raison = 'nom_sauvegarde manipulee';
 }
 if (
 	isset($_REQUEST['znom_sauvegarde'])
-	and strstr((string)$_REQUEST['znom_sauvegarde'], '/')
+	and strstr((string) $_REQUEST['znom_sauvegarde'], '/')
 ) {
 	$ecran_securite_raison = 'znom_sauvegarde manipulee';
 }

ecrire/inc/minipres.php

Spacing +17 added lines, -17 removed lines

@@ -74,7 +74,7 @@ 
 			lire_fichier($file, $c);
 			$css .= $c;
 		}
-		$css = "<style type='text/css'>" . $css . "</style>";
+		$css = "<style type='text/css'>".$css."</style>";
 	} else {
 		foreach ($files as $name) {
 			$file = direction_css(find_in_theme($name));
@@ -87,19 +87,19 @@ 
 		define('_DOCTYPE_ECRIRE', '');
 	}
 
-	return _DOCTYPE_ECRIRE .
-	html_lang_attributes() .
-	"<head>\n" .
-	"<title>" .
-	textebrut($titre) .
-	"</title>\n" .
-	"<meta name='viewport' content='width=device-width' />\n" .
-	$css .
+	return _DOCTYPE_ECRIRE.
+	html_lang_attributes().
+	"<head>\n".
+	"<title>".
+	textebrut($titre).
+	"</title>\n".
+	"<meta name='viewport' content='width=device-width' />\n".
+	$css.
 	"</head>
-<body" . $onLoad . " class='minipres'>
+<body" . $onLoad." class='minipres'>
 	<div id='minipres'>
 	<h1>" .
-	$titre .
+	$titre.
 	"</h1>
 	<div>\n";
 }
@@ -181,7 +181,7 @@ 
 
 		$titre = ($titre == 'install')
 			? _T('avis_espace_interdit')
-			: $titre . '&nbsp;: ' . _T('info_acces_interdit');
+			: $titre.'&nbsp;: '._T('info_acces_interdit');
 
 		$statut = isset($GLOBALS['visiteur_session']['statut']) ? $GLOBALS['visiteur_session']['statut'] : '';
 		$nom = isset($GLOBALS['visiteur_session']['nom']) ? $GLOBALS['visiteur_session']['nom'] : '';
@@ -191,16 +191,16 @@ 
 		}
 
 		if ($statut AND test_espace_prive()) {
-			$corps = bouton_action(_T('public:accueil_site'),generer_url_ecrire('accueil'));
+			$corps = bouton_action(_T('public:accueil_site'), generer_url_ecrire('accueil'));
 		}
 		elseif (!empty($_COOKIE['spip_admin'])) {
-			$corps = bouton_action(_T('public:lien_connecter'),generer_url_public('login'));
+			$corps = bouton_action(_T('public:lien_connecter'), generer_url_public('login'));
 		}
 		else {
-			$corps = bouton_action(_T('public:accueil_site'),$GLOBALS['meta']['adresse_site']);
+			$corps = bouton_action(_T('public:accueil_site'), $GLOBALS['meta']['adresse_site']);
 		}
 		$corps = "<div class='boutons'>$corps</div>";
-		spip_log($nom . " $titre " . $_SERVER['REQUEST_URI']);
+		spip_log($nom." $titre ".$_SERVER['REQUEST_URI']);
 	}
 
 	if (!_AJAX) {
@@ -218,6 +218,6 @@ 
 		foreach ($_POST as $v => $c) {
 			$url = parametre_url($url, $v, $c, '&');
 		}
-		ajax_retour("<div>" . $titre . redirige_formulaire($url) . "</div>", false);
+		ajax_retour("<div>".$titre.redirige_formulaire($url)."</div>", false);
 	}
 }

ecrire/inc/securiser_action.php

Spacing +14 added lines, -14 removed lines

@@ -79,7 +79,7 @@ 
  * @param string|null $url_action
  * @return bool
  */
-function demander_confirmation_avant_action($titre, $titre_bouton, $url_action=null) {
+function demander_confirmation_avant_action($titre, $titre_bouton, $url_action = null) {
 
 	if (!$url_action) {
 		$url_action = self();
@@ -102,7 +102,7 @@ 
 	$corps = "<div style='text-align:center;'>$bouton_action</div>";
 
 	include_spip("inc/minipres");
-	echo minipres($titre,$corps);
+	echo minipres($titre, $corps);
 	exit;
 }
 
@@ -143,7 +143,7 @@ 
 		if ($mode === -1) {
 			return array('action' => $action, 'arg' => $arg, 'hash' => $hash);
 		} else {
-			return generer_url_action($action, "arg=" . rawurlencode($arg) . "&hash=$hash" . (!$r ? '' : "&redirect=$r"),
+			return generer_url_action($action, "arg=".rawurlencode($arg)."&hash=$hash".(!$r ? '' : "&redirect=$r"),
 				$mode, $public);
 		}
 	}
@@ -152,9 +152,9 @@ 
 	$hash = calculer_action_auteur("$action-$arg");
 	$att .= " style='margin: 0px; border: 0px'";
 	if ($redirect) {
-		$redirect = "\n\t\t<input name='redirect' type='hidden' value='" . str_replace("'", '&#39;', $redirect) . "' />";
+		$redirect = "\n\t\t<input name='redirect' type='hidden' value='".str_replace("'", '&#39;', $redirect)."' />";
 	}
-	$mode .= $redirect . "
+	$mode .= $redirect."
 <input name='hash' type='hidden' value='$hash' />
 <input name='arg' type='hidden' value='$arg' />";
 
@@ -224,12 +224,12 @@ 
  */
 function _action_auteur($action, $id_auteur, $pass, $alea) {
 	static $sha = array();
-	if (!isset($sha[$id_auteur . $pass . $alea])) {
+	if (!isset($sha[$id_auteur.$pass.$alea])) {
 		if (!isset($GLOBALS['meta'][$alea])) {
-			if (!$exec = _request('exec') or !autoriser_sans_cookie($exec)){
+			if (!$exec = _request('exec') or !autoriser_sans_cookie($exec)) {
 				include_spip('inc/acces');
 				charger_aleas();
-				if (empty($GLOBALS['meta'][$alea])){
+				if (empty($GLOBALS['meta'][$alea])) {
 					include_spip('inc/minipres');
 					echo minipres();
 					spip_log("$alea indisponible");
@@ -238,12 +238,12 @@ 
 			}
 		}
 		include_spip('auth/sha256.inc');
-		$sha[$id_auteur . $pass . $alea] = spip_sha256($id_auteur . $pass . @$GLOBALS['meta'][$alea]);
+		$sha[$id_auteur.$pass.$alea] = spip_sha256($id_auteur.$pass.@$GLOBALS['meta'][$alea]);
 	}
 	if (function_exists('sha1')) {
-		return sha1($action . $sha[$id_auteur . $pass . $alea]);
+		return sha1($action.$sha[$id_auteur.$pass.$alea]);
 	} else {
-		return md5($action . $sha[$id_auteur . $pass . $alea]);
+		return md5($action.$sha[$id_auteur.$pass.$alea]);
 	}
 }
 
@@ -323,9 +323,9 @@ 
  */
 function calculer_cle_action($action) {
 	if (function_exists('sha1')) {
-		return sha1($action . secret_du_site());
+		return sha1($action.secret_du_site());
 	} else {
-		return md5($action . secret_du_site());
+		return md5($action.secret_du_site());
 	}
 }
 
@@ -365,7 +365,7 @@ 
 	// On nettoie l’URL de tous les var_.
 	$url = nettoyer_uri_var($url);
 
-	$token = _action_auteur('previsualiser-' . $url, $id_auteur, null, $alea);
+	$token = _action_auteur('previsualiser-'.$url, $id_auteur, null, $alea);
 	return "$id_auteur-$token";
 }
 

ecrire/inc/headers.php

Spacing +15 added lines, -15 removed lines

@@ -78,20 +78,20 @@ 
 	if ((!$equiv and !spip_cookie_envoye()) or (
 			   (!empty($_SERVER['SERVER_SOFTWARE'])
 				   and _SERVEUR_SOFTWARE_ACCEPTE_LOCATION_APRES_COOKIE
-				   and preg_match("/"._SERVEUR_SOFTWARE_ACCEPTE_LOCATION_APRES_COOKIE."/i",$_SERVER['SERVER_SOFTWARE']))
+				   and preg_match("/"._SERVEUR_SOFTWARE_ACCEPTE_LOCATION_APRES_COOKIE."/i", $_SERVER['SERVER_SOFTWARE']))
 			or (!empty($_SERVER['SERVER_SIGNATURE'])
 				   and _SERVEUR_SIGNATURE_ACCEPTE_LOCATION_APRES_COOKIE
-				   and preg_match("/"._SERVEUR_SIGNATURE_ACCEPTE_LOCATION_APRES_COOKIE."/i",$_SERVER['SERVER_SIGNATURE']))
+				   and preg_match("/"._SERVEUR_SIGNATURE_ACCEPTE_LOCATION_APRES_COOKIE."/i", $_SERVER['SERVER_SIGNATURE']))
 			or function_exists('apache_getenv')
 			or defined('_SERVER_APACHE')
 		)
 	) {
-		@header("Location: " . $url);
+		@header("Location: ".$url);
 		$equiv = "";
 	} else {
-		@header("Refresh: 0; url=" . $url);
+		@header("Refresh: 0; url=".$url);
 		if (isset($GLOBALS['meta']['charset'])) {
-			@header("Content-Type: text/html; charset=" . $GLOBALS['meta']['charset']);
+			@header("Content-Type: text/html; charset=".$GLOBALS['meta']['charset']);
 		}
 		$equiv = "<meta http-equiv='Refresh' content='0; url=$url'>";
 	}
@@ -103,11 +103,11 @@ 
 	html_lang_attributes(), '
 <head>',
 	$equiv, '
-<title>HTTP ' . $status . '</title>
-' . ((isset($GLOBALS['meta']['charset'])) ? '<meta http-equiv="Content-Type" content="text/html;charset=' . $GLOBALS['meta']['charset'] . '">' : '') . '
+<title>HTTP ' . $status.'</title>
+' . ((isset($GLOBALS['meta']['charset'])) ? '<meta http-equiv="Content-Type" content="text/html;charset='.$GLOBALS['meta']['charset'].'">' : '').'
 </head>
 <body>
-<h1>HTTP ' . $status . '</h1>
+<h1>HTTP ' . $status.'</h1>
 <a href="',
 	quote_amp($url),
 	'">',
@@ -142,7 +142,7 @@ 
 		# en theorie on devrait faire ca tout le temps, mais quand la chaine
 		# commence par ? c'est imperatif, sinon l'url finale n'est pas la bonne
 		if ($url[0] == '?') {
-			$url = url_de_base() . $url;
+			$url = url_de_base().$url;
 		}
 		$url = str_replace('&amp;', '&', $url);
 		spip_log("redirige formulaire ajax: $url");
@@ -150,9 +150,9 @@ 
 		if ($format == 'ajaxform') {
 			return array(
 				// on renvoie un lien masque qui sera traite par ajaxCallback.js
-				'<a href="' . quote_amp($url) . '" name="ajax_redirect"  style="display:none;">' . _T('navigateur_pas_redirige') . '</a>',
+				'<a href="'.quote_amp($url).'" name="ajax_redirect"  style="display:none;">'._T('navigateur_pas_redirige').'</a>',
 				// et un message au cas ou
-				'<br /><a href="' . quote_amp($url) . '">' . _T('navigateur_pas_redirige') . '</a>'
+				'<br /><a href="'.quote_amp($url).'">'._T('navigateur_pas_redirige').'</a>'
 			);
 		} else // format message texte, tout en js inline
 		{
@@ -161,7 +161,7 @@ 
 				"<script type='text/javascript'>if (parent.window){parent.window.document.location.replace(\"$url\");} else {document.location.replace(\"$url\");}</script>"
 				. http_img_pack('loader.svg', '', " class='loader'")
 				. '<br />'
-				. '<a href="' . quote_amp($url) . '">' . _T('navigateur_pas_redirige') . '</a>';
+				. '<a href="'.quote_amp($url).'">'._T('navigateur_pas_redirige').'</a>';
 		}
 	}
 }
@@ -218,9 +218,9 @@ 
 
 	$php_cgi = ($GLOBALS['flag_sapi_name'] and preg_match(",cgi,i", @php_sapi_name()));
 	if ($php_cgi) {
-		header("Status: " . $status_string[$status]);
+		header("Status: ".$status_string[$status]);
 	} else {
-		header("HTTP/1.0 " . $status_string[$status]);
+		header("HTTP/1.0 ".$status_string[$status]);
 	}
 }
 
@@ -242,7 +242,7 @@ 
 
 	header("Content-Type: text/html; charset=$charset");
 	header("Expires: 0");
-	header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
+	header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
 	header("Cache-Control: no-cache, must-revalidate");
 	header("Pragma: no-cache");
 }

ecrire/action/inscrire_auteur.php

Spacing +7 added lines, -7 removed lines

@@ -56,7 +56,7 @@ 
 	}
 
 	include_spip('base/abstract_sql');
-	$res = sql_select("statut, id_auteur, login, email", "spip_auteurs", "email=" . sql_quote($desc['email']));
+	$res = sql_select("statut, id_auteur, login, email", "spip_auteurs", "email=".sql_quote($desc['email']));
 	// erreur ?
 	if (!$res) {
 		return _T('titre_probleme_technique');
@@ -209,7 +209,7 @@ 
 		if (!sql_countsel('spip_auteurs', "login='$login'")) {
 			return $login;
 		}
-		$login = $login_base . $i;
+		$login = $login_base.$i;
 	}
 
 	return $login;
@@ -242,7 +242,7 @@ 
 	}
 
 	$modele_mail = 'modeles/mail_inscription';
-	if (isset($options['modele_mail']) and $options['modele_mail']){
+	if (isset($options['modele_mail']) and $options['modele_mail']) {
 		$modele_mail = $options['modele_mail'];
 	}
 	$message = recuperer_fond($modele_mail, $contexte);
@@ -346,8 +346,8 @@ 
 	// s'assurer de l'unicite du jeton pour le couple (email,cookie)
 	do {
 		$jeton = creer_uniqid();
-		sql_updateq("spip_auteurs", array("cookie_oubli" => $jeton), "id_auteur=" . intval($id_auteur));
-	} while (sql_countsel("spip_auteurs", "cookie_oubli=" . sql_quote($jeton)) > 1);
+		sql_updateq("spip_auteurs", array("cookie_oubli" => $jeton), "id_auteur=".intval($id_auteur));
+	} while (sql_countsel("spip_auteurs", "cookie_oubli=".sql_quote($jeton)) > 1);
 
 	return $jeton;
 }
@@ -365,7 +365,7 @@ 
 	}
 
 	// on peut tomber sur un jeton compose uniquement de chiffres, il faut forcer le $type pour sql_quote pour eviter de planter
-	$desc = sql_fetsel('*', 'spip_auteurs', "cookie_oubli=" . sql_quote($jeton, '', 'string'));
+	$desc = sql_fetsel('*', 'spip_auteurs', "cookie_oubli=".sql_quote($jeton, '', 'string'));
 
 	return $desc;
 }
@@ -377,5 +377,5 @@ 
  * @return bool
  */
 function auteur_effacer_jeton($id_auteur) {
-	return sql_updateq("spip_auteurs", array("cookie_oubli" => ''), "id_auteur=" . intval($id_auteur));
+	return sql_updateq("spip_auteurs", array("cookie_oubli" => ''), "id_auteur=".intval($id_auteur));
 }

prive/formulaires/configurer_preferences.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 	$les_couleurs = $couleurs();
 	foreach ($les_couleurs as $k => $c) {
 		$valeurs['_couleurs_url'][$k] = generer_url_public('style_prive.css', 'ltr='
-			. $GLOBALS['spip_lang_left'] . '&'
+			. $GLOBALS['spip_lang_left'].'&'
 			. $couleurs($k));
 		$valeurs['couleurs'][$k] = $c;
 	}

ecrire/inc/filtres_images_lib_mini.php

Spacing +73 added lines, -73 removed lines

@@ -44,13 +44,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";
@@ -68,7 +68,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;
 
@@ -321,7 +321,7 @@ 
 	  and $extension = _image_trouver_extension_depuis_mime("image/".$regs[1])
 		and in_array($extension, _image_extensions_acceptees_en_entree())
 	) {
-		$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]));
 		}
@@ -335,7 +335,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 "";
 		}
@@ -360,7 +360,7 @@ 
 
 	if ($forcer_format !== false
 		// ignorer forcer_format si on a une image svg, que le filtre appelant ne supporte pas SVG, et que le forcage est un autre format image
-		and ($terminaison_dest !== 'svg' or $support_svg or !in_array($forcer_format,_image_extensions_acceptees_en_sortie()))) {
+		and ($terminaison_dest !== 'svg' or $support_svg or !in_array($forcer_format, _image_extensions_acceptees_en_sortie()))) {
 		$terminaison_dest = $forcer_format;
 	}
 
@@ -422,9 +422,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);
@@ -435,7 +435,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;
 
@@ -471,14 +471,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);
 	}
 
 	$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;
@@ -552,9 +552,9 @@ 
 /**
  * @return array|string[]|null
  */
-function _image_extensions_acceptees_en_sortie(){
+function _image_extensions_acceptees_en_sortie() {
 	static $extensions = null;
-	if (empty($extensions)){
+	if (empty($extensions)) {
 		$extensions = _image_extensions_acceptees_en_entree();
 		$extensions = array_diff($extensions, ['jpeg']);
 		if (in_array('gif', $extensions) and !function_exists('imagegif')) {
@@ -568,7 +568,7 @@ 
 	return $extensions;
 }
 
-function _image_extension_normalisee($extension){
+function _image_extension_normalisee($extension) {
 	$extension = strtolower($extension);
 	if ($extension === 'jpeg') {
 		$extension = 'jpg';
@@ -576,7 +576,7 @@ 
 	return $extension;
 }
 
-function _image_extensions_conservent_transparence(){
+function _image_extensions_conservent_transparence() {
 	return ['png', 'webp'];
 }
 
@@ -626,7 +626,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;
@@ -783,7 +783,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);
@@ -818,7 +818,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);
@@ -858,7 +858,7 @@ 
 	if (!function_exists('imagejpeg')) {
 		return false;
 	}
-	$tmp = $fichier . ".tmp";
+	$tmp = $fichier.".tmp";
 
 	// Enable interlancing
 	imageinterlace($img, true);
@@ -919,7 +919,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);
@@ -953,7 +953,7 @@ 
  */
 function _image_imagesvg($img, $fichier) {
 
-	$tmp = $fichier . ".tmp";
+	$tmp = $fichier.".tmp";
 	if (strpos($img, "<") === false) {
 		$img = supprimer_timestamp($img);
 		if (!file_exists($img)) {
@@ -1010,13 +1010,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
@@ -1027,7 +1027,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);
 		}
 	}
 
@@ -1046,7 +1046,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)
@@ -1083,7 +1083,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)
 	) {
@@ -1139,7 +1139,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);
@@ -1236,7 +1236,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', '');
 	}
 
@@ -1360,7 +1360,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'];
 
@@ -1392,14 +1392,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)){
+		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);
 		}
@@ -1411,9 +1411,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(
@@ -1428,13 +1428,13 @@ 
 		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);
@@ -1444,7 +1444,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)) {
@@ -1462,12 +1462,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);
 			}
@@ -1479,7 +1479,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);
 				}
@@ -1491,7 +1491,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);
 					}
@@ -1513,7 +1513,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;
 		}
@@ -1818,7 +1818,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));
 	}
@@ -1865,7 +1865,7 @@ 
 	public static function LittleEndian2String($number, $minbytes = 1) {
 		$intstring = '';
 		while ($number > 0) {
-			$intstring = $intstring . chr($number & 255);
+			$intstring = $intstring.chr($number & 255);
 			$number >>= 8;
 		}
 
@@ -1898,9 +1898,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) {
@@ -1928,44 +1928,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]);