spip / SPIP

ecrire/maj/v018.php

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 				"fille.id_parent = maman.id_forum AND fille.id_thread <> maman.id_thread", '', "thread");
 			while ($row = sql_fetch($r)) {
 				if ($row['thread'] == $precedent) {
-					$discussion .= "," . $row['id'];
+					$discussion .= ",".$row['id'];
 				} else {
 					if ($precedent) {
 						sql_updateq("spip_forum", array("id_thread" => $precedent), "id_forum IN ($discussion)");

ecrire/req/pg.exp.php

Spacing +81 added lines, -82 removed lines

@@ -65,7 +65,7 @@ 
 	if (!$link) {
 		$erreurs[] = pg_last_error();
 		foreach ($erreurs as $e) {
-			spip_log('Echec pg_connect. Erreur : ' . $e, 'pg.' . _LOG_HS);
+			spip_log('Echec pg_connect. Erreur : '.$e, 'pg.'._LOG_HS);
 		}
 
 		return false;
@@ -82,8 +82,8 @@ 
 		);
 	}
 
-	spip_log("Connexion vers $host, base $db, prefixe $prefixe " . ($link ? 'operationnelle' : 'impossible'),
-		'pg.' . _LOG_DEBUG);
+	spip_log("Connexion vers $host, base $db, prefixe $prefixe ".($link ? 'operationnelle' : 'impossible'),
+		'pg.'._LOG_DEBUG);
 
 	return !$link ? false : array(
 		'db' => $db,
@@ -171,7 +171,7 @@ 
 	} else {
 		$suite = '';
 	}
-	$query = preg_replace('/([,\s])spip_/', '\1' . $prefixe . '_', $query) . $suite;
+	$query = preg_replace('/([,\s])spip_/', '\1'.$prefixe.'_', $query).$suite;
 
 	// renvoyer la requete inerte si demandee
 	if (!$requeter) {
@@ -241,7 +241,7 @@ 
 	// ou revoir l'api de sql_alter en creant un 
 	// sql_alter_table($table,array($actions));
 	if (!preg_match("/\s*((\s*IGNORE)?\s*TABLE\s*([^\s]*))\s*(.*)?/is", $query, $regs)) {
-		spip_log("$query mal comprise", 'pg.' . _LOG_ERREUR);
+		spip_log("$query mal comprise", 'pg.'._LOG_ERREUR);
 
 		return false;
 	}
@@ -254,7 +254,7 @@ 
 	$i = 0;
 	$ouverte = false;
 	while ($do = array_shift($todo)) {
-		$todo2[$i] = isset($todo2[$i]) ? $todo2[$i] . "," . $do : $do;
+		$todo2[$i] = isset($todo2[$i]) ? $todo2[$i].",".$do : $do;
 		$o = (false !== strpos($do, "("));
 		$f = (false !== strpos($do, ")"));
 		if ($o and !$f) {
@@ -267,24 +267,24 @@ 
 		}
 	}
 	$todo = $todo2;
-	$query = $debut . ' ' . array_shift($todo);
+	$query = $debut.' '.array_shift($todo);
 
 	if (!preg_match('/^\s*(IGNORE\s*)?TABLE\s+(\w+)\s+(ADD|DROP|CHANGE|MODIFY|RENAME)\s*(.*)$/is', $query, $r)) {
-		spip_log("$query incompris", 'pg.' . _LOG_ERREUR);
+		spip_log("$query incompris", 'pg.'._LOG_ERREUR);
 	} else {
 		if ($r[1]) {
-			spip_log("j'ignore IGNORE dans $query", 'pg.' . _LOG_AVERTISSEMENT);
+			spip_log("j'ignore IGNORE dans $query", 'pg.'._LOG_AVERTISSEMENT);
 		}
-		$f = 'spip_pg_alter_' . strtolower($r[3]);
+		$f = 'spip_pg_alter_'.strtolower($r[3]);
 		if (function_exists($f)) {
 			$f($r[2], $r[4], $serveur, $requeter);
 		} else {
-			spip_log("$query non prevu", 'pg.' . _LOG_ERREUR);
+			spip_log("$query non prevu", 'pg.'._LOG_ERREUR);
 		}
 	}
 	// Alter a plusieurs args. Faudrait optimiser.
 	if ($todo) {
-		spip_pg_alter("TABLE $table " . join(',', $todo));
+		spip_pg_alter("TABLE $table ".join(',', $todo));
 	}
 
 }
@@ -292,10 +292,10 @@ 
 // http://code.spip.net/@spip_pg_alter_change
 function spip_pg_alter_change($table, $arg, $serveur = '', $requeter = true) {
 	if (!preg_match('/^`?(\w+)`?\s+`?(\w+)`?\s+(.*?)\s*(DEFAULT .*?)?(NOT\s+NULL)?\s*(DEFAULT .*?)?$/i', $arg, $r)) {
-		spip_log("alter change: $arg  incompris", 'pg.' . _LOG_ERREUR);
+		spip_log("alter change: $arg  incompris", 'pg.'._LOG_ERREUR);
 	} else {
 		list(, $old, $new, $type, $default, $null, $def2) = $r;
-		$actions = array("ALTER $old TYPE " . mysql2pg_type($type));
+		$actions = array("ALTER $old TYPE ".mysql2pg_type($type));
 		if ($null) {
 			$actions[] = "ALTER $old SET NOT NULL";
 		} else {
@@ -308,7 +308,7 @@ 
 			$actions[] = "ALTER $old DROP DEFAULT";
 		}
 
-		spip_pg_query("ALTER TABLE $table " . join(', ', $actions));
+		spip_pg_query("ALTER TABLE $table ".join(', ', $actions));
 
 		if ($old != $new) {
 			spip_pg_query("ALTER TABLE $table RENAME $old TO $new", $serveur);
@@ -319,7 +319,7 @@ 
 // http://code.spip.net/@spip_pg_alter_add
 function spip_pg_alter_add($table, $arg, $serveur = '', $requeter = true) {
 	if (!preg_match('/^(COLUMN|INDEX|KEY|PRIMARY\s+KEY|)\s*(.*)$/', $arg, $r)) {
-		spip_log("alter add $arg  incompris", 'pg.' . _LOG_ERREUR);
+		spip_log("alter add $arg  incompris", 'pg.'._LOG_ERREUR);
 
 		return null;
 	}
@@ -329,13 +329,13 @@ 
 			$m[2] = $n[1];
 		}
 
-		return spip_pg_query("ALTER TABLE $table ADD " . $m[1] . ' ' . mysql2pg_type($m[2]), $serveur, $requeter);
+		return spip_pg_query("ALTER TABLE $table ADD ".$m[1].' '.mysql2pg_type($m[2]), $serveur, $requeter);
 	} elseif ($r[1][0] == 'P') {
 		// la primary peut etre sur plusieurs champs
 		$r[2] = trim(str_replace('`', '', $r[2]));
 		$m = ($r[2][0] == '(') ? substr($r[2], 1, -1) : $r[2];
 
-		return spip_pg_query("ALTER TABLE $table ADD CONSTRAINT $table" . '_pkey PRIMARY KEY (' . $m . ')', $serveur,
+		return spip_pg_query("ALTER TABLE $table ADD CONSTRAINT $table".'_pkey PRIMARY KEY ('.$m.')', $serveur,
 			$requeter);
 	} else {
 		preg_match('/([^\s,]*)\s*(.*)?/', $r[2], $m);
@@ -369,23 +369,23 @@ 
 // http://code.spip.net/@spip_pg_alter_drop
 function spip_pg_alter_drop($table, $arg, $serveur = '', $requeter = true) {
 	if (!preg_match('/^(COLUMN|INDEX|KEY|PRIMARY\s+KEY|)\s*`?(\w*)`?/', $arg, $r)) {
-		spip_log("alter drop: $arg  incompris", 'pg.' . _LOG_ERREUR);
+		spip_log("alter drop: $arg  incompris", 'pg.'._LOG_ERREUR);
 	} else {
 		if (!$r[1] or $r[1] == 'COLUMN') {
-			return spip_pg_query("ALTER TABLE $table DROP " . $r[2], $serveur);
+			return spip_pg_query("ALTER TABLE $table DROP ".$r[2], $serveur);
 		} elseif ($r[1][0] == 'P') {
-			return spip_pg_query("ALTER TABLE $table DROP CONSTRAINT $table" . '_pkey', $serveur);
+			return spip_pg_query("ALTER TABLE $table DROP CONSTRAINT $table".'_pkey', $serveur);
 		} else {
-			return spip_pg_query("DROP INDEX " . $table . '_' . $r[2], $serveur);
+			return spip_pg_query("DROP INDEX ".$table.'_'.$r[2], $serveur);
 		}
 	}
 }
 
 function spip_pg_alter_modify($table, $arg, $serveur = '', $requeter = true) {
 	if (!preg_match('/^`?(\w+)`?\s+(.*)$/', $arg, $r)) {
-		spip_log("alter modify: $arg  incompris", 'pg.' . _LOG_ERREUR);
+		spip_log("alter modify: $arg  incompris", 'pg.'._LOG_ERREUR);
 	} else {
-		return spip_pg_alter_change($table, $r[1] . ' ' . $arg, $serveur = '', $requeter = true);
+		return spip_pg_alter_change($table, $r[1].' '.$arg, $serveur = '', $requeter = true);
 	}
 }
 
@@ -401,7 +401,7 @@ 
 	} elseif (preg_match('/^(TO)\s*`?(\w*)`?/', $arg, $r)) {
 		$rename = $r[2];
 	} else {
-		spip_log("alter rename: $arg  incompris", 'pg.' . _LOG_ERREUR);
+		spip_log("alter rename: $arg  incompris", 'pg.'._LOG_ERREUR);
 	}
 
 	return $rename ? spip_pg_query("ALTER TABLE $table RENAME TO $rename") : false;
@@ -421,8 +421,8 @@ 
  */
 function spip_pg_create_index($nom, $table, $champs, $serveur = '', $requeter = true) {
 	if (!($nom or $table or $champs)) {
-		spip_log("Champ manquant pour creer un index pg ($nom, $table, (" . @join(',', $champs) . "))",
-			'pg.' . _LOG_ERREUR);
+		spip_log("Champ manquant pour creer un index pg ($nom, $table, (".@join(',', $champs)."))",
+			'pg.'._LOG_ERREUR);
 
 		return false;
 	}
@@ -432,7 +432,7 @@ 
 
 	// PG ne differentie pas noms des index en fonction des tables
 	// il faut donc creer des noms uniques d'index pour une base pg
-	$nom = $table . '_' . $nom;
+	$nom = $table.'_'.$nom;
 	// enlever d'eventuelles parentheses deja presentes sur champs
 	if (!is_array($champs)) {
 		if ($champs[0] == "(") {
@@ -440,7 +440,7 @@ 
 		}
 		$champs = array($champs);
 	}
-	$query = "CREATE INDEX $nom ON $table (" . join(',', $champs) . ")";
+	$query = "CREATE INDEX $nom ON $table (".join(',', $champs).")";
 	if (!$requeter) {
 		return $query;
 	}
@@ -464,7 +464,7 @@ 
 	} else {
 		$suite = '';
 	}
-	$query = 'EXPLAIN ' . preg_replace('/([,\s])spip_/', '\1' . $prefixe . '_', $query) . $suite;
+	$query = 'EXPLAIN '.preg_replace('/([,\s])spip_/', '\1'.$prefixe.'_', $query).$suite;
 
 	if (!$requeter) {
 		return $query;
@@ -559,14 +559,14 @@ 
 		}
 	}
 	$from = spip_pg_from($from, $prefixe);
-	$query = "SELECT " . $select
+	$query = "SELECT ".$select
 		. (!$from ? '' : "\nFROM $from")
-		. (!$where ? '' : ("\nWHERE " . (!is_array($where) ? calculer_pg_where($where) : (join("\n\tAND ",
+		. (!$where ? '' : ("\nWHERE ".(!is_array($where) ? calculer_pg_where($where) : (join("\n\tAND ",
 				array_map('calculer_pg_where', $where))))))
 		. spip_pg_groupby($groupby, $from, $select)
 		. (!$having ? '' : "\nHAVING $having")
 		. ($orderby ? ("\nORDER BY $orderby") : '')
-		. (!$limit ? '' : (" LIMIT $count" . (!$offset ? '' : " OFFSET $offset")));
+		. (!$limit ? '' : (" LIMIT $count".(!$offset ? '' : " OFFSET $offset")));
 
 	// renvoyer la requete inerte si demandee
 	if ($requeter === false) {
@@ -575,7 +575,7 @@ 
 
 	$r = spip_pg_trace_query($query, $serveur);
 
-	return $r ? $r : $query;;
+	return $r ? $r : $query; ;
 }
 
 // Le traitement des prefixes de table dans un Select se limite au FROM
@@ -587,7 +587,7 @@ 
 		$from = spip_pg_select_as($from);
 	}
 
-	return !$prefixe ? $from : preg_replace('/(\b)spip_/', '\1' . $prefixe . '_', $from);
+	return !$prefixe ? $from : preg_replace('/(\b)spip_/', '\1'.$prefixe.'_', $from);
 }
 
 // http://code.spip.net/@spip_pg_orderby
@@ -596,7 +596,7 @@ 
 	$arg = (is_array($order) ? $order : preg_split('/\s*,\s*/', $order));
 
 	foreach ($arg as $v) {
-		if (preg_match('/(case\s+.*?else\s+0\s+end)\s*AS\s+' . $v . '/', $select, $m)) {
+		if (preg_match('/(case\s+.*?else\s+0\s+end)\s*AS\s+'.$v.'/', $select, $m)) {
 
 			$res[] = $m[1];
 		} else {
@@ -757,7 +757,7 @@ 
 			$n++;
 			$res .= "\nwhen $index=$v then $n";
 		}
-		$arg = $m[1] . "case $res else 0 end "
+		$arg = $m[1]."case $res else 0 end "
 			. substr($arg, strlen($m[0]));
 	}
 
@@ -802,9 +802,9 @@ 
 	}
 
 	if (strtoupper($join) === 'AND') {
-		return $exp . join("\n\t$join ", array_map('calculer_pg_where', $v));
+		return $exp.join("\n\t$join ", array_map('calculer_pg_where', $v));
 	} else {
-		return $exp . join($join, $v);
+		return $exp.join($join, $v);
 	}
 }
 
@@ -815,7 +815,7 @@ 
 		if (substr($k, -1) == '@') {
 			// c'est une jointure qui se refere au from precedent
 			// pas de virgule
-			$argsas .= '  ' . $v;
+			$argsas .= '  '.$v;
 		} else {
 			$as = '';
 			//  spip_log("$k : $v", _LOG_DEBUG);
@@ -825,7 +825,7 @@ 
 				} elseif ($v != $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 {
 						$as = " AS $k";
 					}
@@ -833,7 +833,7 @@ 
 			}
 			// spip_log("subs $k : $v avec $as", _LOG_DEBUG);
 			// if (strpos($v, 'JOIN') === false)  $argsas .= ', ';
-			$argsas .= ', ' . $v . $as;
+			$argsas .= ', '.$v.$as;
 		}
 	}
 
@@ -866,7 +866,7 @@ 
 	$serveur = '',
 	$requeter = true
 ) {
-	$c = !$groupby ? '*' : ('DISTINCT ' . (is_string($groupby) ? $groupby : join(',', $groupby)));
+	$c = !$groupby ? '*' : ('DISTINCT '.(is_string($groupby) ? $groupby : join(',', $groupby)));
 	$r = spip_pg_select("COUNT($c)", $from, $where, '', '', '', $having, $serveur, $requeter);
 	if (!$requeter) {
 		return $r;
@@ -970,7 +970,7 @@ 
 	// recherche de champs 'timestamp' pour mise a jour auto de ceux-ci
 	$couples = spip_pg_ajouter_champs_timestamp($table, $couples, $desc, $serveur);
 
-	return spip_pg_insert($table, "(" . join(',', array_keys($couples)) . ")", "(" . join(',', $couples) . ")", $desc,
+	return spip_pg_insert($table, "(".join(',', array_keys($couples)).")", "(".join(',', $couples).")", $desc,
 		$serveur, $requeter);
 }
 
@@ -991,7 +991,7 @@ 
 	$c = isset($tab_couples[0]) ? $tab_couples[0] : array();
 	$les_cles = spip_pg_ajouter_champs_timestamp($table, $c, $desc, $serveur);
 
-	$cles = "(" . join(',', array_keys($les_cles)) . ')';
+	$cles = "(".join(',', array_keys($les_cles)).')';
 	$valeurs = array();
 	foreach ($tab_couples as $couples) {
 		foreach ($couples as $champ => $val) {
@@ -1000,7 +1000,7 @@ 
 		// recherche de champs 'timestamp' pour mise a jour auto de ceux-ci
 		$couples = spip_pg_ajouter_champs_timestamp($table, $couples, $desc, $serveur);
 
-		$valeurs[] = '(' . join(',', $couples) . ')';
+		$valeurs[] = '('.join(',', $couples).')';
 	}
 	$valeurs = implode(', ', $valeurs);
 
@@ -1022,7 +1022,7 @@ 
 
 	$set = array();
 	foreach ($couples as $champ => $val) {
-		$set[] = $champ . '=' . $val;
+		$set[] = $champ.'='.$val;
 	}
 
 	$query = calculer_pg_expression('UPDATE', $table, ',')
@@ -1059,7 +1059,7 @@ 
 // http://code.spip.net/@spip_pg_replace
 function spip_pg_replace($table, $values, $desc, $serveur = '', $requeter = true) {
 	if (!$values) {
-		spip_log("replace vide $table", 'pg.' . _LOG_AVERTISSEMENT);
+		spip_log("replace vide $table", 'pg.'._LOG_AVERTISSEMENT);
 
 		return 0;
 	}
@@ -1091,7 +1091,7 @@ 
 
 	$where = join(' AND ', $prims);
 	if (!$where) {
-		return spip_pg_insert($table, "(" . join(',', array_keys($values)) . ")", "(" . join(',', $values) . ")", $desc,
+		return spip_pg_insert($table, "(".join(',', array_keys($values)).")", "(".join(',', $values).")", $desc,
 			$serveur);
 	}
 	$couples = join(',', $noprims);
@@ -1110,10 +1110,9 @@ 
 		$couples = pg_affected_rows($couples);
 	}
 	if (!$couples) {
-		$ret = !$seq ? '' :
-			(" RETURNING nextval('$seq') < $prim");
-		$connexion['last'] = $q = "INSERT INTO $table (" . join(',', array_keys($values)) . ') VALUES (' . join(',',
-				$values) . ")$ret";
+		$ret = !$seq ? '' : (" RETURNING nextval('$seq') < $prim");
+		$connexion['last'] = $q = "INSERT INTO $table (".join(',', array_keys($values)).') VALUES ('.join(',',
+				$values).")$ret";
 		$couples = spip_pg_query_simple($link, $q);
 		if (!$couples) {
 			return false;
@@ -1161,7 +1160,7 @@ 
 	) {
 		return '';
 	} else {
-		return $raw ? $prim : $table . '_' . $prim . "_seq";
+		return $raw ? $prim : $table.'_'.$prim."_seq";
 	}
 }
 
@@ -1179,22 +1178,22 @@ 
 			return spip_pg_frommysql($v);
 		} else {
 			if (strncmp($v, '0000', 4) == 0) {
-				$v = "0001" . substr($v, 4);
+				$v = "0001".substr($v, 4);
 			}
 			if (strpos($v, "-00-00") === 4) {
-				$v = substr($v, 0, 4) . "-01-01" . substr($v, 10);
+				$v = substr($v, 0, 4)."-01-01".substr($v, 10);
 			}
 
 			return "timestamp '$v'";
 		}
 	} elseif (!sql_test_int($t)) {
-		return ("'" . pg_escape_string($v) . "'");
+		return ("'".pg_escape_string($v)."'");
 	} elseif (is_numeric($v) or (strpos($v, 'CAST(') === 0)) {
 		return $v;
 	} elseif ($v[0] == '0' and $v[1] !== 'x' and ctype_xdigit(substr($v, 1))) {
 		return substr($v, 1);
 	} else {
-		spip_log("Warning: '$v'  n'est pas de type $t", 'pg.' . _LOG_AVERTISSEMENT);
+		spip_log("Warning: '$v'  n'est pas de type $t", 'pg.'._LOG_AVERTISSEMENT);
 
 		return intval($v);
 	}
@@ -1202,7 +1201,7 @@ 
 
 // http://code.spip.net/@spip_pg_hex
 function spip_pg_hex($v) {
-	return "CAST(x'" . $v . "' as bigint)";
+	return "CAST(x'".$v."' as bigint)";
 }
 
 function spip_pg_quote($v, $type = '') {
@@ -1241,15 +1240,15 @@ 
 		return $not ? '0=0' : '0=1';
 	}
 	if (strpos($valeurs, "CAST(x'") !== false) {
-		return "($val=" . join("OR $val=", explode(',', $valeurs)) . ')';
+		return "($val=".join("OR $val=", explode(',', $valeurs)).')';
 	}
 	$n = $i = 0;
 	$in_sql = "";
 	while ($n = strpos($valeurs, ',', $n + 1)) {
 		if ((++$i) >= 255) {
-			$in_sql .= "($val $not IN (" .
-				substr($valeurs, 0, $n) .
-				"))\n" .
+			$in_sql .= "($val $not IN (".
+				substr($valeurs, 0, $n).
+				"))\n".
 				($not ? "AND\t" : "OR\t");
 			$valeurs = substr($valeurs, $n + 1);
 			$i = $n = 0;
@@ -1266,7 +1265,7 @@ 
 	$s = $link ? pg_last_error($link) : pg_last_error();
 	if ($s) {
 		$s = str_replace('ERROR', 'errcode: 1000 ', $s);
-		spip_log("$s - $query", 'pg.' . _LOG_ERREUR);
+		spip_log("$s - $query", 'pg.'._LOG_ERREUR);
 	}
 
 	return $s;
@@ -1317,7 +1316,7 @@ 
 function spip_pg_showbase($match, $serveur = '', $requeter = true) {
 	$connexion = &$GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
 	$link = $connexion['link'];
-	$connexion['last'] = $q = "SELECT tablename FROM pg_tables WHERE tablename ILIKE " . _q($match);
+	$connexion['last'] = $q = "SELECT tablename FROM pg_tables WHERE tablename ILIKE "._q($match);
 
 	return spip_pg_query_simple($link, $q);
 }
@@ -1326,7 +1325,7 @@ 
 function spip_pg_showtable($nom_table, $serveur = '', $requeter = true) {
 	$connexion = &$GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
 	$link = $connexion['link'];
-	$connexion['last'] = $q = "SELECT column_name, column_default, data_type FROM information_schema.columns WHERE table_name ILIKE " . _q($nom_table);
+	$connexion['last'] = $q = "SELECT column_name, column_default, data_type FROM information_schema.columns WHERE table_name ILIKE "._q($nom_table);
 
 	$res = spip_pg_query_simple($link, $q);
 	if (!$res) {
@@ -1337,15 +1336,15 @@ 
 	// il faut en tenir compte dans le return
 	$fields = array();
 	while ($field = pg_fetch_array($res, null, PGSQL_NUM)) {
-		$fields[$field[0]] = $field[2] . (!$field[1] ? '' : (" DEFAULT " . $field[1]));
+		$fields[$field[0]] = $field[2].(!$field[1] ? '' : (" DEFAULT ".$field[1]));
 	}
-	$connexion['last'] = $q = "SELECT indexdef FROM pg_indexes WHERE tablename ILIKE " . _q($nom_table);
+	$connexion['last'] = $q = "SELECT indexdef FROM pg_indexes WHERE tablename ILIKE "._q($nom_table);
 	$res = spip_pg_query_simple($link, $q);
 	$keys = array();
 	while ($index = pg_fetch_array($res, null, PGSQL_NUM)) {
 		if (preg_match('/CREATE\s+(UNIQUE\s+)?INDEX\s([^\s]+).*\((.*)\)$/', $index[0], $r)) {
-			$nom = str_replace($nom_table . '_', '', $r[2]);
-			$keys[($r[1] ? "PRIMARY KEY" : ("KEY " . $nom))] = $r[3];
+			$nom = str_replace($nom_table.'_', '', $r[2]);
+			$keys[($r[1] ? "PRIMARY KEY" : ("KEY ".$nom))] = $r[3];
 		}
 	}
 
@@ -1378,16 +1377,16 @@ 
 		if (strpos($k, "KEY ") === 0) {
 			$n = str_replace('`', '', $k);
 			$v = str_replace('`', '"', $v);
-			$i = $nom . preg_replace("/KEY +/", '_', $n);
+			$i = $nom.preg_replace("/KEY +/", '_', $n);
 			if ($k != $n) {
 				$i = "\"$i\"";
 			}
 			$keys[] = "CREATE INDEX $i ON $nom ($v);";
 		} elseif (strpos($k, "UNIQUE ") === 0) {
 			$k = preg_replace("/^UNIQUE +/", '', $k);
-			$prim .= "$s\n\t\tCONSTRAINT " . str_replace('`', '"', $k) . " UNIQUE ($v)";
+			$prim .= "$s\n\t\tCONSTRAINT ".str_replace('`', '"', $k)." UNIQUE ($v)";
 		} else {
-			$prim .= "$s\n\t\t" . str_replace('`', '"', $k) . " ($v)";
+			$prim .= "$s\n\t\t".str_replace('`', '"', $k)." ($v)";
 		}
 		if ($k == "PRIMARY KEY") {
 			$prim_name = $v;
@@ -1398,17 +1397,17 @@ 
 
 	$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) {
 		$k = str_replace('`', '"', $k);
 		if (preg_match(',([a-z]*\s*(\(\s*[0-9]*\s*\))?(\s*binary)?),i', $v, $defs)) {
 			if (preg_match(',(char|text),i', $defs[1]) and !preg_match(',binary,i', $defs[1])) {
-				$v = $defs[1] . $character_set . ' ' . substr($v, strlen($defs[1]));
+				$v = $defs[1].$character_set.' '.substr($v, strlen($defs[1]));
 			}
 		}
 
@@ -1423,7 +1422,7 @@ 
 
 	// En l'absence de "if not exists" en PG, on neutralise les erreurs
 
-	$q = "CREATE $temporary TABLE $nom ($query" . ($prim ? ",$prim" : '') . ")" .
+	$q = "CREATE $temporary TABLE $nom ($query".($prim ? ",$prim" : '').")".
 		($character_set ? " DEFAULT $character_set" : "")
 		. "\n";
 
@@ -1434,7 +1433,7 @@ 
 	$r = @pg_query($link, $q);
 
 	if (!$r) {
-		spip_log("Impossible de creer cette table: $q", 'pg.' . _LOG_ERREUR);
+		spip_log("Impossible de creer cette table: $q", 'pg.'._LOG_ERREUR);
 	} else {
 		foreach ($keys as $index) {
 			pg_query($link, $index);
@@ -1458,13 +1457,13 @@ 
 	// vue deja presente
 	if (sql_showtable($nom, false, $serveur)) {
 		if ($requeter) {
-			spip_log("Echec creation d'une vue sql ($nom) car celle-ci existe deja (serveur:$serveur)", 'pg.' . _LOG_ERREUR);
+			spip_log("Echec creation d'une vue sql ($nom) car celle-ci existe deja (serveur:$serveur)", 'pg.'._LOG_ERREUR);
 		}
 
 		return false;
 	}
 
-	$query = "CREATE VIEW $nom AS " . $query_select;
+	$query = "CREATE VIEW $nom AS ".$query_select;
 
 	return spip_pg_query($query, $serveur, $requeter);
 }
@@ -1472,7 +1471,7 @@ 
 
 // http://code.spip.net/@spip_pg_set_connect_charset
 function spip_pg_set_connect_charset($charset, $serveur = '', $requeter = true) {
-	spip_log("changement de charset sql a ecrire en PG", 'pg.' . _LOG_ERREUR);
+	spip_log("changement de charset sql a ecrire en PG", 'pg.'._LOG_ERREUR);
 }
 
 
@@ -1486,7 +1485,7 @@ 
  **/
 // http://code.spip.net/@spip_sqlite_optimize
 function spip_pg_optimize($table, $serveur = '', $requeter = true) {
-	return spip_pg_query("VACUUM " . $table, $serveur, $requeter);
+	return spip_pg_query("VACUUM ".$table, $serveur, $requeter);
 }
 
 // Selectionner la sous-chaine dans $objet

ecrire/req/sqlite_fonctions.php

Spacing +3 added lines, -3 removed lines

@@ -258,7 +258,7 @@ 
 
 // http://code.spip.net/@_sqlite_func_preg_replace
 function _sqlite_func_preg_replace($quoi, $cherche, $remplace) {
-	$return = preg_replace('%' . $cherche . '%', $remplace, $quoi);
+	$return = preg_replace('%'.$cherche.'%', $remplace, $quoi);
 
 	#spip_log("preg_replace : $quoi, $cherche, $remplace, $return",'sqlite.'._LOG_DEBUG);
 	return $return;
@@ -314,7 +314,7 @@ 
 		return false;
 	}
 	$u = isset($GLOBALS['meta']['pcre_u']) ? $GLOBALS['meta']['pcre_u'] : 'u';
-	$return = preg_match('%' . $cherche . '%imsS' . $u, $quoi);
+	$return = preg_match('%'.$cherche.'%imsS'.$u, $quoi);
 
 	#spip_log("regexp_replace : $quoi, $cherche, $remplace, $return",'sqlite.'._LOG_DEBUG);
 	return $return;
@@ -335,7 +335,7 @@ 
  */
 function _sqlite_func_to_days($d) {
 	static $offset = 719528; // nb de jour entre 0000-00-00 et timestamp 0=1970-01-01
-	$result = $offset + (int)ceil(_sqlite_func_unix_timestamp($d) / (24 * 3600));
+	$result = $offset + (int) ceil(_sqlite_func_unix_timestamp($d) / (24 * 3600));
 
 	#spip_log("Passage avec TO_DAYS : $d, $result",'sqlite.'._LOG_DEBUG);
 	return $result;

ecrire/action/calculer_taille_cache.php

Spacing +4 added lines, -4 removed lines

@@ -42,16 +42,16 @@
 		$res = _T('ecrire:taille_cache_image',
 			array(
 				'dir' => joli_repertoire(_DIR_VAR),
-				'taille' => "<b>" . (taille_en_octets($taille) > 0 ? taille_en_octets($taille) : "0 octet") . "</b>"
+				'taille' => "<b>".(taille_en_octets($taille) > 0 ? taille_en_octets($taille) : "0 octet")."</b>"
 			)
 		);
 	} else {
 		include_spip('inc/invalideur');
 		$taille =
 			calculer_taille_dossier(_DIR_CACHE_XML)
-			+ calculer_taille_dossier(_DIR_CACHE . 'skel/')
-			+ calculer_taille_dossier(_DIR_CACHE . 'wheels/')
-			+ calculer_taille_dossier(_DIR_CACHE . 'contextes/');
+			+ calculer_taille_dossier(_DIR_CACHE.'skel/')
+			+ calculer_taille_dossier(_DIR_CACHE.'wheels/')
+			+ calculer_taille_dossier(_DIR_CACHE.'contextes/');
 		$taille += intval(taille_du_cache());
 		if ($taille <= 150000) {
 			$res = _T('taille_cache_vide');

ecrire/action/tester_taille.php

Spacing +4 added lines, -4 removed lines

@@ -76,7 +76,7 @@ 
 		$t = ($GLOBALS['taille_min'] * $GLOBALS['taille_min']);
 		if ($GLOBALS['taille_min'] !== $GLOBALS['taille_max']) {
 			$t = $t * 0.9; // marge de securite
-			echo round($t / 1000000, 3) . ' Mpx';
+			echo round($t / 1000000, 3).' Mpx';
 		} else {
 			// c'est un cas "on a reussi la borne max initiale, donc on a pas de limite connue"
 			$t = 0;
@@ -111,18 +111,18 @@ 
 	$i = _request('i') + 1;
 	$image_source = chemin_image("test.png");
 	$GLOBALS['redirect'] = generer_url_action("tester_taille",
-		"i=$i&arg=" . $GLOBALS['taille_min'] . "-" . $GLOBALS['taille_test']);
+		"i=$i&arg=".$GLOBALS['taille_min']."-".$GLOBALS['taille_test']);
 
 	ob_start('action_tester_taille_error_handler');
 	filtrer('image_recadre', $image_source, $taille, $taille);
-	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-" . $GLOBALS['taille_max']);
+	$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-".$GLOBALS['taille_max']);
 
 	// si la valeur intermediaire a reussi, on teste la valeur maxi qui est peut etre sous estimee
 	// si $GLOBALS['taille_min']==0 (car on est au premier coup)
 	if ($GLOBALS['taille_min'] == 0) {
 		$taille = $GLOBALS['taille_max'];
 		filtrer('image_recadre', $image_source, $taille, $taille);
-		$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-" . $GLOBALS['taille_max']);
+		$GLOBALS['redirect'] = generer_url_action("tester_taille", "i=$i&arg=$taille-".$GLOBALS['taille_max']);
 	}
 	ob_end_clean();
 

ecrire/action/editer_article.php

Spacing +8 added lines, -8 removed lines

@@ -106,7 +106,7 @@ 
 	);
 
 	// Si l'article est publie, invalider les caches et demander sa reindexation
-	$t = sql_getfetsel("statut", "spip_articles", "id_article=" . intval($id_article));
+	$t = sql_getfetsel("statut", "spip_articles", "id_article=".intval($id_article));
 	$invalideur = $indexation = false;
 	if ($t == 'publie') {
 		$invalideur = "id='article/$id_article'";
@@ -116,7 +116,7 @@ 
 	if ($err = objet_modifier_champs('article', $id_article,
 		array(
 			'data' => $set,
-			'nonvide' => array('titre' => _T('info_nouvel_article') . " " . _T('info_numero_abbreviation') . $id_article),
+			'nonvide' => array('titre' => _T('info_nouvel_article')." "._T('info_numero_abbreviation').$id_article),
 			'invalideur' => $invalideur,
 			'indexation' => $indexation,
 			'date_modif' => 'date_modif' // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
@@ -298,7 +298,7 @@ 
 			if (autoriser('modifier', 'article', $id_article) and $s != 'publie') {
 				$statut = $champs['statut'] = $s;
 			} else {
-				spip_log("editer_article $id_article refus " . join(' ', $c));
+				spip_log("editer_article $id_article refus ".join(' ', $c));
 			}
 		}
 
@@ -322,7 +322,7 @@ 
 	if (isset($c['id_parent'])
 		and $id_parent = $c['id_parent']
 		and $id_parent != $id_rubrique
-		and (sql_fetsel('1', "spip_rubriques", "id_rubrique=" . intval($id_parent)))
+		and (sql_fetsel('1', "spip_rubriques", "id_rubrique=".intval($id_parent)))
 	) {
 		$champs['id_rubrique'] = $id_parent;
 
@@ -419,12 +419,12 @@ 
 	//  changer aussi son secteur et sa langue (si heritee)
 	if (isset($champs['id_rubrique'])) {
 
-		$row_rub = sql_fetsel("id_secteur, lang", "spip_rubriques", "id_rubrique=" . sql_quote($champs['id_rubrique']));
+		$row_rub = sql_fetsel("id_secteur, lang", "spip_rubriques", "id_rubrique=".sql_quote($champs['id_rubrique']));
 
 		$langue = $row_rub['lang'];
 		$champs['id_secteur'] = $row_rub['id_secteur'];
 		if (sql_fetsel('1', 'spip_articles',
-			"id_article=" . intval($id_article) . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue))) {
+			"id_article=".intval($id_article)." AND langue_choisie<>'oui' AND lang<>".sql_quote($langue))) {
 			$champs['lang'] = $langue;
 		}
 	}
@@ -433,7 +433,7 @@ 
 		return;
 	}
 
-	sql_updateq('spip_articles', $champs, "id_article=" . intval($id_article));
+	sql_updateq('spip_articles', $champs, "id_article=".intval($id_article));
 
 	// Changer le statut des rubriques concernees 
 
@@ -454,7 +454,7 @@ 
 		foreach ($plus as $n => $t) {
 			$plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,", "", $t);
 		}
-		set_request('texte', join('', $plus) . _request('texte'));
+		set_request('texte', join('', $plus)._request('texte'));
 	}
 }
 

ecrire/action/session.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 	) {
 		if ($_SERVER['REQUEST_METHOD'] == 'POST') {
 			include_spip('inc/session');
-			session_set('session_' . $var, $val = _request('val'));
+			session_set('session_'.$var, $val = _request('val'));
 			#spip_log("autosave:$var:$val",'autosave');
 		}
 	}

ecrire/action/instituer_langue_objet.php

Spacing +6 added lines, -6 removed lines

@@ -30,7 +30,7 @@ 
  * @param string $serveur
  * @return string
  */
-function action_instituer_langue_objet_dist($objet, $id, $id_rubrique, $changer_lang, $serveur='') {
+function action_instituer_langue_objet_dist($objet, $id, $id_rubrique, $changer_lang, $serveur = '') {
 	if ($changer_lang) {
 		$table_objet_sql = table_objet_sql($objet);
 		$id_table_objet = id_table_objet($objet);
@@ -38,13 +38,13 @@ 
 		$desc = $trouver_table($table_objet_sql, $serveur);
 		
 		$set = array();
-		if (isset($desc['field']['langue_choisie'])){
+		if (isset($desc['field']['langue_choisie'])) {
 			$set['langue_choisie'] = 'oui';
 		}
 		
 		if ($changer_lang != "herit") {
 			$set['lang'] = $changer_lang;
-			sql_updateq($table_objet_sql, $set, "$id_table_objet=" . intval($id),'',$serveur);
+			sql_updateq($table_objet_sql, $set, "$id_table_objet=".intval($id), '', $serveur);
 			include_spip('inc/rubriques'); // pour calculer_langues_rubriques et calculer_langues_utilisees
 			if ($table_objet_sql == 'spip_rubriques') {
 				calculer_langues_rubriques();
@@ -52,16 +52,16 @@ 
 			$langues = calculer_langues_utilisees($serveur);
 			ecrire_meta('langues_utilisees', $langues);
 		} else {
-			$langue_parent = sql_getfetsel("lang", "spip_rubriques", "id_rubrique=" . intval($id_rubrique));
+			$langue_parent = sql_getfetsel("lang", "spip_rubriques", "id_rubrique=".intval($id_rubrique));
 			if (!$langue_parent) {
 				$langue_parent = $GLOBALS['meta']['langue_site'];
 			}
 			$changer_lang = $langue_parent;
 			$set['lang'] = $changer_lang;
-			if (isset($set['langue_choisie'])){
+			if (isset($set['langue_choisie'])) {
 				$set['langue_choisie'] = 'non';
 			}
-			sql_updateq($table_objet_sql, $set, "$id_table_objet=" . intval($id),'',$serveur);
+			sql_updateq($table_objet_sql, $set, "$id_table_objet=".intval($id), '', $serveur);
 			if ($table_objet_sql == 'spip_rubriques') {
 				include_spip('inc/rubriques');
 				calculer_langues_rubriques();

ecrire/action/logout.php

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 			ask_php_auth(_T('login_deconnexion_ok'),
 				_T('login_verifiez_navigateur'),
 				_T('login_retour_public'),
-				"redirect=" . _DIR_RESTREINT_ABS,
+				"redirect="._DIR_RESTREINT_ABS,
 				_T('login_test_navigateur'),
 				true);