spip / SPIP

ecrire/inc/auth.php

Spacing +14 added lines, -15 removed lines

@@ -59,7 +59,7 @@ 
 	}
 
 	$n = intval(sql_errno());
-	spip_log("Erreur base de donnees $n " . sql_error());
+	spip_log("Erreur base de donnees $n ".sql_error());
 
 	return $n ? $n : 1;
 }
@@ -85,11 +85,11 @@ 
 		// erreur SQL a afficher
 		$raison = minipres(
 			_T('info_travaux_titre'),
-			_T('titre_probleme_technique') . '<p><tt>' . sql_errno() . ' ' . sql_error() . '</tt></p>'
+			_T('titre_probleme_technique').'<p><tt>'.sql_errno().' '.sql_error().'</tt></p>'
 		);
 	} elseif (@$raison['statut']) {
 		// un simple visiteur n'a pas acces a l'espace prive
-		spip_log('connexion refusee a ' . @$raison['id_auteur']);
+		spip_log('connexion refusee a '.@$raison['id_auteur']);
 		$raison = minipres(_T('avis_erreur_connexion'), _T('avis_erreur_visiteur'));
 	} else {
 		// auteur en fin de droits ...
@@ -178,8 +178,7 @@ 
 	$where = (is_numeric($id_auteur)
 		/*AND $id_auteur>0*/ // reprise lors des restaurations
 	) ?
-		"id_auteur=$id_auteur" :
-		(!strlen($GLOBALS['connect_login']) ? '' : 'login=' . sql_quote($GLOBALS['connect_login'], '', 'text'));
+		"id_auteur=$id_auteur" : (!strlen($GLOBALS['connect_login']) ? '' : 'login='.sql_quote($GLOBALS['connect_login'], '', 'text'));
 
 	if (!$where) {
 		return '';
@@ -221,7 +220,7 @@ 
 	$GLOBALS['connect_login'] = $row['login'];
 	$GLOBALS['connect_statut'] = $row['statut'];
 
-	$GLOBALS['visiteur_session'] = array_merge((array)$GLOBALS['visiteur_session'], $row);
+	$GLOBALS['visiteur_session'] = array_merge((array) $GLOBALS['visiteur_session'], $row);
 
 	// au cas ou : ne pas memoriser les champs sensibles
 	unset($GLOBALS['visiteur_session']['pass']);
@@ -290,7 +289,7 @@ 
  * @return string
  */
 function auth_a_loger() {
-	$redirect = generer_url_public('login', 'url=' . rawurlencode(self('&', true)), '&');
+	$redirect = generer_url_public('login', 'url='.rawurlencode(self('&', true)), '&');
 
 	// un echec au "bonjour" (login initial) quand le statut est
 	// inconnu signale sans doute un probleme de cookies
@@ -328,7 +327,7 @@ 
 	}
 
 	if (abs(strtotime($date) - $connect_quand) >= 60) {
-		sql_updateq('spip_auteurs', array('en_ligne' => $date), 'id_auteur=' . intval($row['id_auteur']));
+		sql_updateq('spip_auteurs', array('en_ligne' => $date), 'id_auteur='.intval($row['id_auteur']));
 		$row['en_ligne'] = $date;
 	}
 
@@ -435,14 +434,14 @@ 
 function auth_informer_login($login, $serveur = '') {
 	if (!$login
 		or !$login_base = auth_retrouver_login($login, $serveur)
-		or !$row = sql_fetsel('*', 'spip_auteurs', 'login=' . sql_quote($login_base, $serveur, 'text'), '', '', '', '', $serveur)
+		or !$row = sql_fetsel('*', 'spip_auteurs', 'login='.sql_quote($login_base, $serveur, 'text'), '', '', '', '', $serveur)
 	) {
 
 		// generer de fausses infos, mais credibles, pour eviter une attaque
 		// https://core.spip.net/issues/1758 + https://core.spip.net/issues/3691
 		include_spip('inc/securiser_action');
-		$fauxalea1 = md5('fauxalea' . secret_du_site() . $login . floor(date('U') / 86400));
-		$fauxalea2 = md5('fauxalea' . secret_du_site() . $login . ceil(date('U') / 86400));
+		$fauxalea1 = md5('fauxalea'.secret_du_site().$login.floor(date('U') / 86400));
+		$fauxalea2 = md5('fauxalea'.secret_du_site().$login.ceil(date('U') / 86400));
 
 		$row = array(
 			'login' => $login,
@@ -564,7 +563,7 @@ 
 	sql_updateq(
 		'spip_auteurs',
 		array('prefs' => serialize($p)),
-		'id_auteur=' . intval($auteur['id_auteur'])
+		'id_auteur='.intval($auteur['id_auteur'])
 	);
 
 	//  bloquer ici le visiteur qui tente d'abuser de ses droits
@@ -728,7 +727,7 @@ 
 		return false;
 	}
 
-	$row = sql_fetsel('*', 'spip_auteurs', 'login=' . sql_quote($login, $serveur, 'text'), '', '', '', '', $serveur);
+	$row = sql_fetsel('*', 'spip_auteurs', 'login='.sql_quote($login, $serveur, 'text'), '', '', '', '', $serveur);
 
 	if (!$row) {
 		if (spip_connect_ldap($serveur)
@@ -780,11 +779,11 @@ 
 	$retour = $retour ? $retour : _T('icone_retour');
 	$corps .= "<p>$raison</p>[<a href='$public'>$retour</a>] ";
 	if ($url) {
-		$corps .= "[<a href='" . generer_url_action('cookie', "essai_auth_http=oui&$url") . "'>$re</a>]";
+		$corps .= "[<a href='".generer_url_action('cookie', "essai_auth_http=oui&$url")."'>$re</a>]";
 	}
 
 	if ($lien) {
-		$corps .= " [<a href='$ecrire'>" . _T('login_espace_prive') . '</a>]';
+		$corps .= " [<a href='$ecrire'>"._T('login_espace_prive').'</a>]';
 	}
 	include_spip('inc/minipres');
 	echo minipres($pb, $corps);

ecrire/inc/preselectionner_parent_nouvel_objet.php

Spacing +2 added lines, -2 removed lines

@@ -47,12 +47,12 @@
 		$id_rubrique = $row_rub['id_rubrique'];
 	}
 	// si le choix ne convient pas, on cherche dans un secteur
-	if (!autoriser('creer' . $objet . 'dans', 'rubrique', $id_rubrique)) {
+	if (!autoriser('creer'.$objet.'dans', 'rubrique', $id_rubrique)) {
 		$id_rubrique = '';
 		// manque de chance, la rubrique n'est pas autorisee, on cherche un des secteurs autorises
 		$res = sql_select("id_rubrique", "spip_rubriques", "id_parent=0");
 		while (!$id_rubrique and $row_rub = sql_fetch($res)) {
-			if (autoriser('creer' . $objet . 'dans', 'rubrique', $row_rub['id_rubrique'])) {
+			if (autoriser('creer'.$objet.'dans', 'rubrique', $row_rub['id_rubrique'])) {
 				$id_rubrique = $row_rub['id_rubrique'];
 			}
 		}

ecrire/inc/lang_liste.php

Spacing +5 added lines, -5 removed lines

@@ -51,7 +51,7 @@ 
 	'cpf_dom' => "Kreyòl",
 	'cpf_hat' => "Kreyòl (Peyi Dayiti)",
 	'cs' => "čeština",
-	'cy' => "Cymraeg",  # welsh, gallois
+	'cy' => "Cymraeg", # welsh, gallois
 	'da' => "dansk",
 	'de' => "Deutsch",
 	'dz' => "Bhutani",
@@ -92,7 +92,7 @@ 
 	'hi' => "हिंदी",
 	'hr' => "hrvatski",
 	'hu' => "magyar",
-	'hy' => "Հայերեն",// Arménien
+	'hy' => "Հայերեն", // Arménien
 	'ia' => "Interlingua",
 	'id' => "Indonesia",
 	'ie' => "Interlingue",
@@ -106,7 +106,7 @@ 
 	'ka' => "ქართული",
 	'kk' => "қазақ тілі", // Kazakh
 	'kl' => "kalaallisut",
-	'km' => "ភាសាខ្មែរ",// Khmer
+	'km' => "ភាសាខ្មែរ", // Khmer
 	'kn' => "Kannada",
 	'ko' => "한국어",
 	'kok' => "कोंकणी",
@@ -176,8 +176,8 @@ 
 	'sh_latn' => 'srpskohrvastski',
 	'sh_cyrl' => 'Српскохрватски',
 	'si' => "Sinhalese",
-	'sk' => "slovenčina",  // (Slovakia)
-	'sl' => "slovenščina",  // (Slovenia)
+	'sk' => "slovenčina", // (Slovakia)
+	'sl' => "slovenščina", // (Slovenia)
 	'sm' => "Samoan",
 	'sn' => "Shona",
 	'so' => "Somali",

ecrire/inc/json.php

Spacing +7 added lines, -7 removed lines

@@ -31,11 +31,11 @@ 
 		case is_null($var):
 			return 'null';
 		case is_string($var):
-			return '"' . addcslashes($var, "\"\\\n\r/") . '"';
+			return '"'.addcslashes($var, "\"\\\n\r/").'"';
 		case is_bool($var):
 			return $var ? 'true' : 'false';
 		case is_scalar($var):
-			return (string)$var;
+			return (string) $var;
 		case is_object($var):// blam
 			$var = get_object_vars($var);
 			$asso = true;
@@ -50,19 +50,19 @@ 
 			if ($asso) {
 				$ret = '{';
 				foreach ($var as $key => $elt) {
-					$ret .= $sep . '"' . $key . '":' . var2js($elt);
+					$ret .= $sep.'"'.$key.'":'.var2js($elt);
 					$sep = ',';
 				}
 
-				return $ret . '}';
+				return $ret.'}';
 			} else {
 				$ret = '[';
 				foreach ($var as $elt) {
-					$ret .= $sep . var2js($elt);
+					$ret .= $sep.var2js($elt);
 					$sep = ',';
 				}
 
-				return $ret . ']';
+				return $ret.']';
 			}
 	}
 
@@ -82,7 +82,7 @@ 
 	// flag indiquant qu'on est en iframe et qu'il faut proteger nos
 	// donnees dans un <textarea> ; attention $_FILES a ete vide par array_pop
 	if (defined('FILE_UPLOAD')) {
-		return '<textarea>' . spip_htmlspecialchars($var) . '</textarea>';
+		return '<textarea>'.spip_htmlspecialchars($var).'</textarea>';
 	} else {
 		return $var;
 	}

ecrire/inc/config.php

Spacing +8 added lines, -8 removed lines

@@ -340,7 +340,7 @@ 
 			if (strncmp($url, 'configurer_', 11) == 0) {
 				$deja[$url] = $b;
 			} elseif ($b['url'] == 'configurer' and preg_match(',cfg=([a-z0-9_]+),i', $b['args'], $match)) {
-				$deja["configurer_" . $match[1]] = $b;
+				$deja["configurer_".$match[1]] = $b;
 			}
 		}
 
@@ -354,10 +354,10 @@ 
 
 	// trouver toutes les pages configurer_xxx de l'espace prive
 	// et construire un tableau des entrees qui ne sont pas dans $deja
-	$pages = find_all_in_path("prive/squelettes/contenu/", "configurer_.*[.]" . _EXTENSION_SQUELETTES . '$');
+	$pages = find_all_in_path("prive/squelettes/contenu/", "configurer_.*[.]"._EXTENSION_SQUELETTES.'$');
 
 	foreach ($pages as $page) {
-		$configurer = basename($page, "." . _EXTENSION_SQUELETTES);
+		$configurer = basename($page, "."._EXTENSION_SQUELETTES);
 		if (!isset($exclure[$configurer])) {
 			$liste[$configurer] = array(
 				'parent' => 'bando_configuration',
@@ -377,9 +377,9 @@ 
 
 	// trouver tous les formulaires/configurer_
 	// et construire un tableau des entrees
-	$pages = find_all_in_path("formulaires/", "configurer_.*[.]" . _EXTENSION_SQUELETTES . '$');
+	$pages = find_all_in_path("formulaires/", "configurer_.*[.]"._EXTENSION_SQUELETTES.'$');
 	foreach ($pages as $page) {
-		$configurer = basename($page, "." . _EXTENSION_SQUELETTES);
+		$configurer = basename($page, "."._EXTENSION_SQUELETTES);
 		if (!isset($forms[$configurer])
 			and !isset($liste[$configurer])
 			and !isset($exclure[$configurer])
@@ -387,7 +387,7 @@ 
 			$liste[$configurer] = array(
 				'parent' => 'bando_configuration',
 				'url' => 'configurer',
-				'args' => 'cfg=' . substr($configurer, 11),
+				'args' => 'cfg='.substr($configurer, 11),
 				'titre' => _T("configurer:{$configurer}_titre"),
 				'icone' => find_in_theme($i = "images/{$configurer}-16.png") ? $i : $icone_defaut,
 			);
@@ -419,8 +419,8 @@ 
 
 	// evaluer le fond en lui passant un exec coherent pour que les pipelines le reconnaissent
 	// et reperer les formulaires CVT configurer_xx insereres par les plugins via pipeline
-	$config = basename(substr($file, 0, -strlen("." . _EXTENSION_SQUELETTES)));
-	spip_log('Calcul de ' . "prive/squelettes/contenu/$config");
+	$config = basename(substr($file, 0, -strlen("."._EXTENSION_SQUELETTES)));
+	spip_log('Calcul de '."prive/squelettes/contenu/$config");
 	$fond = recuperer_fond("prive/squelettes/contenu/$config", array("exec" => $config));
 
 	// passer dans le pipeline affiche_milieu pour que les plugins puissent ajouter leur formulaires...

ecrire/inc/lien_court.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 	$coupe_url = defined('_MAX_COUPE_URL') ? _MAX_COUPE_URL : 35;
 
 	if (strlen($url) > $long_url) {
-		$url = substr($url, 0, $coupe_url) . '...';
+		$url = substr($url, 0, $coupe_url).'...';
 	}
 
 	return $url;

ecrire/inc/log.php

Spacing +6 added lines, -6 removed lines

@@ -52,11 +52,11 @@ 
 
 	// si spip_log() dans mes_options, ou repertoire log/ non present, poser dans tmp/
 	if (!defined('_DIR_LOG') or !$test_repertoire[$d]) {
-		$logfile = _DIR_RACINE . _NOM_TEMPORAIRES_INACCESSIBLES . $logname . '.log';
+		$logfile = _DIR_RACINE._NOM_TEMPORAIRES_INACCESSIBLES.$logname.'.log';
 	}
 
 	$rotate = 0;
-	$pid = '(pid ' . @getmypid() . ')';
+	$pid = '(pid '.@getmypid().')';
 
 	// accepter spip_log( Array )
 	if (!is_string($message)) {
@@ -71,10 +71,10 @@ 
 			$fi = substr($fi, strlen(_ROOT_RACINE));
 		}
 		$fu = isset($debug[2]['function']) ? $debug[2]['function'] : '';
-		$debugverb = "$fi:L$l:$fu" . '():';
+		$debugverb = "$fi:L$l:$fu".'():';
 	}
 
-	$m = date('Y-m-d H:i:s') . ' ' . (isset($GLOBALS['ip']) ? $GLOBALS['ip'] : '') . ' ' . $pid . ' '
+	$m = date('Y-m-d H:i:s').' '.(isset($GLOBALS['ip']) ? $GLOBALS['ip'] : '').' '.$pid.' '
 		//distinguer les logs prives et publics dans les grep
 		. $debugverb
 		. (test_espace_prive() ? ':Pri:' : ':Pub:')
@@ -97,9 +97,9 @@ 
 	if ($rotate-- > 0
 		and function_exists('spip_unlink')
 	) {
-		spip_unlink($logfile . '.' . $rotate);
+		spip_unlink($logfile.'.'.$rotate);
 		while ($rotate--) {
-			@rename($logfile . ($rotate ? '.' . $rotate : ''), $logfile . '.' . ($rotate + 1));
+			@rename($logfile.($rotate ? '.'.$rotate : ''), $logfile.'.'.($rotate + 1));
 		}
 	}
 

ecrire/inc/nfslock.php

Spacing +7 added lines, -7 removed lines

@@ -106,7 +106,7 @@ 
 	if (!$max_age) {
 		$max_age = _DEFAULT_LOCKTIME;
 	}
-	$lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
+	$lock_file = _DIR_TMP._NAME_LOCK.'-'.substr(md5($fichier), 0, 8);
 
 
 	/*
@@ -115,7 +115,7 @@ 
 	 */
 
 	$id = creer_uniqid();
-	$tpath = _DIR_TMP . "slock.$id";
+	$tpath = _DIR_TMP."slock.$id";
 	$tmpfd = @fopen($tpath, 'w'); // hum, le 'x' necessite php4,3,2 ...
 	if (!$tmpfd) {  /* open failed */
 		@fclose($tmpfd);
@@ -241,7 +241,7 @@ 
 	 *    thinks the current time is (our_tmp.st_ctime)..
 	 */
 
-	$tpath = _DIR_TMP . "stime.$id";
+	$tpath = _DIR_TMP."stime.$id";
 	$tmpfd = @fopen($tpath, 'w');
 	if ((!$tmpfd)
 		or (@fputs($tmpfd, 'zz', 2) != 2)
@@ -254,14 +254,14 @@ 
 		return false; //(NFSL_SYSF);
 	}
 
-	@fclose($tmpfd);    /* We don't need this once we have our_tmp.st_ctime. */
+	@fclose($tmpfd); /* We don't need this once we have our_tmp.st_ctime. */
 	spip_unlink($tpath);
 
 	/*
 	 * 2. make fullpath, a buffer for the full pathname of the lock file
 	 */
 
-	$lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
+	$lock_file = _DIR_TMP._NAME_LOCK.'-'.substr(md5($fichier), 0, 8);
 
 	/*
 	 * 3. If the ctime hasn't been modified, unlink the file and return. If the
@@ -280,10 +280,10 @@ 
 			if (!$test) {
 				return false;
 			} //(NFSL_LOST);
-			sleep(1 + (random(0, 4)));    /* so sleep a bit */
+			sleep(1 + (random(0, 4))); /* so sleep a bit */
 		}
 
-		return true;//(NFSL_OK);			/* success */
+		return true; //(NFSL_OK);			/* success */
 	}
 
 	/*

ecrire/inc/roles.php

Spacing +3 added lines, -3 removed lines

@@ -149,7 +149,7 @@ 
 			$role = $roles['roles']['defaut'];
 		}
 		// where
-		$cond = array("$colonne_role=" . sql_quote($role));
+		$cond = array("$colonne_role=".sql_quote($role));
 	}
 
 	return array($role, $colonne_role, $cond);
@@ -184,7 +184,7 @@ 
 	if ($colonne_role) {
 		// on ajoute la condition du role aux autres conditions.
 		if ($role != '*') {
-			$cond[] = "$colonne_role=" . sql_quote($role);
+			$cond[] = "$colonne_role=".sql_quote($role);
 		}
 	}
 
@@ -368,7 +368,7 @@ 
 	$all = sql_allfetsel(
 		"DISTINCT $colone_role",
 		$l,
-		'objet=' . sql_quote(($objet_source == $objet_lien) ? $objet : $objet_source)
+		'objet='.sql_quote(($objet_source == $objet_lien) ? $objet : $objet_source)
 	);
 	$done[$hash] = array_map('reset', $all);