Inspection of "[Salvatore] [source:ecrire/lang/ ecrire] Export de..." - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e003b8...5b64d4 )

by cam

created 2022-06-04 23:27 UTC

Status

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -53,7 +53,7 @@  discard block
 block discarded – undo
 		$class = ' blocreplie';
 	}
 
-	return '<div ' . ($id ? "id='$id' " : '') . "class='bloc_depliable$class'>";
+	return '<div '.($id ? "id='$id' " : '')."class='bloc_depliable$class'>";
 }
 
 function fin_block() {
@@ -64,12 +64,12 @@  discard block
 block discarded – undo
 // $deplie : true (deplie) ou false (plie) ou -1 (inactif) ou 'incertain' pour que le bouton s'auto init au chargement de la page
 // $ids : id des div lies au bouton (facultatif, par defaut c'est le div.bloc_depliable qui suit)
 function bouton_block_depliable($texte, $deplie, $ids = '') {
-	$bouton_id = 'b' . substr(md5($texte . microtime()), 0, 8);
+	$bouton_id = 'b'.substr(md5($texte.microtime()), 0, 8);
 
 	$class = ($deplie === true) ? ' deplie' : (($deplie == -1) ? ' impliable' : ' replie');
 	if (strlen($ids)) {
 		$cible = explode(',', $ids);
-		$cible = '#' . implode(',#', $cible);
+		$cible = '#'.implode(',#', $cible);
 	} else {
 		$cible = "#$bouton_id + div.bloc_depliable";
 	}

Please login to merge, or discard this patch.

ecrire/inc/lien.php 1 patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

@@ -98,7 +98,7 @@  discard block
 block discarded – undo
 	return $r ?: traiter_lien_explicite($ref, $texte, $pour, $connect, $echappe_typo);
 }
 
-define('_EXTRAIRE_LIEN', ',^\s*(?:' . _PROTOCOLES_STD . '):?/?/?\s*$,iS');
+define('_EXTRAIRE_LIEN', ',^\s*(?:'._PROTOCOLES_STD.'):?/?/?\s*$,iS');
 
 function traiter_lien_explicite($ref, $texte = '', $pour = 'url', string $connect = '', $echappe_typo = true) {
 	if (preg_match(_EXTRAIRE_LIEN, $ref)) {
@@ -114,19 +114,19 @@  discard block
 block discarded – undo
 		$lien_court = charger_fonction('lien_court', 'inc');
 		$texte = $lien_court($texte);
 		if ($echappe_typo) {
-			$texte = '<html>' . quote_amp($texte) . '</html>';
+			$texte = '<html>'.quote_amp($texte).'</html>';
 		}
 	}
 
 	// petites corrections d'URL
 	if (preg_match('/^www\.[^@]+$/S', $lien)) {
-		$lien = 'http://' . $lien;
+		$lien = 'http://'.$lien;
 	} else {
 		if (strpos($lien, '@') && email_valide($lien)) {
 			if (!$texte) {
 				$texte = $lien;
 			}
-			$lien = 'mailto:' . $lien;
+			$lien = 'mailto:'.$lien;
 		}
 	}
 
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
 }
 
 function liens_implicite_glose_dist($texte, $id, $type, $args, $ancre, string $connect = '') {
-	if (function_exists($f = 'glossaire_' . $ancre)) {
+	if (function_exists($f = 'glossaire_'.$ancre)) {
 		$url = $f($texte, $id);
 	} else {
 		$url = glossaire_std($texte);
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
 	if (!($match = typer_raccourci($ref))) {
 		return false;
 	}
-	[$type, , $id, , $args, , $ancre] = array_pad($match, 7, null);
+	[$type,, $id,, $args,, $ancre] = array_pad($match, 7, null);
 	// attention dans le cas des sites le lien doit pointer non pas sur
 	// la page locale du site, mais directement sur le site lui-meme
 	if ($f = charger_fonction("implicite_$type", 'liens', true)) {
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
 		$r['titre'] = $texte;
 	}
 	if (!@$r['titre']) {
-		$r['titre'] = _T($type) . " $id";
+		$r['titre'] = _T($type)." $id";
 	}
 	if ($pour == 'titre') {
 		return $r['titre'];
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
 		and $mime = sql_getfetsel(
 			'mime_type',
 			'spip_types_documents',
-			'extension IN (' . sql_get_select('extension', 'spip_documents', 'id_document=' . sql_quote($id)) . ')',
+			'extension IN ('.sql_get_select('extension', 'spip_documents', 'id_document='.sql_quote($id)).')',
 			'',
 			'',
 			'',
@@ -297,11 +297,11 @@  discard block
 block discarded – undo
 	'(<([a-z_-]{3,})' # <modele
 	. '\s*([0-9]*)\s*' # id
 	. '([|](?:<[^<>]*>|[^>])*?)?' # |arguments (y compris des tags <...>)
-	. '\s*/?' . '>)' # fin du modele >
+	. '\s*/?'.'>)' # fin du modele >
 	. '\s*(<\/a>)?' # eventuel </a>
 );
 
-define('_RACCOURCI_MODELE_DEBUT', '@^' . _RACCOURCI_MODELE . '@isS');
+define('_RACCOURCI_MODELE_DEBUT', '@^'._RACCOURCI_MODELE.'@isS');
 
 function traiter_modeles($texte, $doublons = false, $echap = '', string $connect = '', $liens = null, $env = []) {
 	// preserver la compatibilite : true = recherche des documents
@@ -380,7 +380,7 @@  discard block
 block discarded – undo
 						if (test_espace_prive()) {
 							$modele = entites_html(substr($texte, $a, $cherche));
 							if (!is_null($liens)) {
-								$modele = '<pre>' . str_replace($liens[0], $liens[1], $modele) . '</pre>';
+								$modele = '<pre>'.str_replace($liens[0], $liens[1], $modele).'</pre>';
 							}
 						}
 					}

Please login to merge, or discard this patch.

ecrire/inc/texte_mini.php 1 patch

Spacing +25 added lines, -25 removed lines patch added patch discarded remove patch

@@ -44,13 +44,13 @@  discard block
 block discarded – undo
 	// celle du texte) et public (spip_lang est la langue du texte)
 	$dir = _DIR_RESTREINT ? lang_dir() : lang_dir($GLOBALS['spip_lang']);
 
-	$p = 'puce' . (test_espace_prive() ? '_prive' : '');
+	$p = 'puce'.(test_espace_prive() ? '_prive' : '');
 	if ($dir == 'rtl') {
 		$p .= '_rtl';
 	}
 
 	if (!isset($GLOBALS[$p])) {
-		$GLOBALS[$p] = '<span class="spip-puce ' . $dir . '"><b>–</b></span>';
+		$GLOBALS[$p] = '<span class="spip-puce '.$dir.'"><b>–</b></span>';
 	}
 
 	return $GLOBALS[$p];
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
 }
 
 if (!defined('_BALISES_BLOCS_REGEXP')) {
-	define('_BALISES_BLOCS_REGEXP', ',</?(' . _BALISES_BLOCS . ')[>[:space:]],iS');
+	define('_BALISES_BLOCS_REGEXP', ',</?('._BALISES_BLOCS.')[>[:space:]],iS');
 }
 
 //
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
 
 	// Tester si on echappe en span ou en div
 	if (is_null($mode) or !in_array($mode, ['div', 'span'])) {
-		$mode = preg_match(',</?(' . _BALISES_BLOCS . ')[>[:space:]],iS', $rempl) ? 'div' : 'span';
+		$mode = preg_match(',</?('._BALISES_BLOCS.')[>[:space:]],iS', $rempl) ? 'div' : 'span';
 	}
 
 	// Decouper en morceaux, base64 a des probleme selon la taille de la pile
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
 	) {
 		foreach ($matches as $m) {
 			if ($m[1] === 'code') {
-				$code = '<code' . $m[2] . '>' . spip_htmlspecialchars($m[3]) . '</code>';
+				$code = '<code'.$m[2].'>'.spip_htmlspecialchars($m[3]).'</code>';
 				$pre = str_replace($m[0], $code, $pre);
 			}
 		}
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
 
 // Echapper les <code>...</ code>
 function traiter_echap_code_dist($regs) {
-	[, , $att, $corps] = $regs;
+	[,, $att, $corps] = $regs;
 	$echap = spip_htmlspecialchars($corps); // il ne faut pas passer dans entites_html, ne pas transformer les &#xxx; du code !
 
 	// ne pas mettre le <div...> s'il n'y a qu'une ligne
@@ -141,9 +141,9 @@  discard block
 block discarded – undo
 		$echap = nl2br($echap);
 		$echap = "<div style='text-align: left;' "
 			. "class='spip_code' dir='ltr'><code$att>"
-			. $echap . '</code></div>';
+			. $echap.'</code></div>';
 	} else {
-		$echap = "<code$att class='spip_code' dir='ltr'>" . $echap . '</code>';
+		$echap = "<code$att class='spip_code' dir='ltr'>".$echap.'</code>';
 	}
 
 	$echap = str_replace("\t", '&nbsp; &nbsp; &nbsp; &nbsp; ', $echap);
@@ -229,8 +229,8 @@  discard block
 block discarded – undo
 			} // sinon les traiter selon le cas
 			else {
 				if (
-					function_exists($f = $callback_prefix . 'traiter_echap_' . strtolower($regs[1]))
-					or function_exists($f = $f . '_dist')
+					function_exists($f = $callback_prefix.'traiter_echap_'.strtolower($regs[1]))
+					or function_exists($f = $f.'_dist')
 				) {
 					$echap = $f($regs);
 				}
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
 	// dans une callback autonommee
 	if (strpos($preg ?: _PROTEGE_BLOCS, 'script') !== false) {
 		if (
-			strpos($letexte, '<' . '?') !== false and preg_match_all(
+			strpos($letexte, '<'.'?') !== false and preg_match_all(
 				',<[?].*($|[?]>),UisS',
 				$letexte,
 				$matches,
@@ -283,7 +283,7 @@  discard block
 block discarded – undo
 			strpos($letexte, '<') !== false
 			and
 			preg_match_all(
-				',<(span|div)\sclass=[\'"]base64' . $source . '[\'"]\s(.*)>\s*</\1>,UmsS',
+				',<(span|div)\sclass=[\'"]base64'.$source.'[\'"]\s(.*)>\s*</\1>,UmsS',
 				$letexte,
 				$regs,
 				PREG_SET_ORDER
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
 					}
 				}
 				if ($at) {
-					$rempl = '<' . $reg[1] . '>' . $rempl . '</' . $reg[1] . '>';
+					$rempl = '<'.$reg[1].'>'.$rempl.'</'.$reg[1].'>';
 					foreach ($at as $attr => $a) {
 						$rempl = inserer_attribut($rempl, $attr, $a);
 					}
@@ -381,8 +381,8 @@  discard block
 block discarded – undo
 	$texte = nettoyer_raccourcis_typo($texte);
 
 	// balises de sauts de ligne et paragraphe
-	$texte = preg_replace('/<p( [^>]*)?' . '>/', "\r", $texte);
-	$texte = preg_replace('/<br( [^>]*)?' . '>/', "\n", $texte);
+	$texte = preg_replace('/<p( [^>]*)?'.'>/', "\r", $texte);
+	$texte = preg_replace('/<br( [^>]*)?'.'>/', "\n", $texte);
 
 	// on repasse les doubles \n en \r que nettoyer_raccourcis_typo() a pu modifier
 	$texte = str_replace("\n\n", "\r", $texte);
@@ -390,7 +390,7 @@  discard block
 block discarded – undo
 	// supprimer les tags
 	$texte = supprimer_tags($texte);
 	$texte = trim(str_replace("\n", ' ', $texte));
-	$texte .= "\n";  // marquer la fin
+	$texte .= "\n"; // marquer la fin
 
 	// corriger la longueur de coupe
 	// en fonction de la presence de caracteres utf
@@ -405,7 +405,7 @@  discard block
 block discarded – undo
 	// couper au mot precedent
 	$long = spip_substr($texte, 0, max($taille - 4, 1));
 	$u = $GLOBALS['meta']['pcre_u'];
-	$court = preg_replace("/([^\s][\s]+)[^\s]*\n?$/" . $u, "\\1", $long);
+	$court = preg_replace("/([^\s][\s]+)[^\s]*\n?$/".$u, "\\1", $long);
 	if (is_null($suite)) {
 		$suite = (defined('_COUPER_SUITE') ? _COUPER_SUITE : '&nbsp;(...)');
 	}
@@ -415,7 +415,7 @@  discard block
 block discarded – undo
 	if (spip_strlen($court) < max(0.75 * $taille, 2)) {
 		$points = '';
 		$long = spip_substr($texte, 0, $taille);
-		$texte = preg_replace("/([^\s][\s]+)[^\s]*\n?$/" . $u, "\\1", $long);
+		$texte = preg_replace("/([^\s][\s]+)[^\s]*\n?$/".$u, "\\1", $long);
 		// encore trop court ? couper au caractere
 		if (spip_strlen($texte) < 0.75 * $taille) {
 			$texte = $long;
@@ -434,7 +434,7 @@  discard block
 block discarded – undo
 	// supprimer l'eventuelle entite finale mal coupee
 	$texte = preg_replace('/&#?[a-z0-9]*$/S', '', $texte);
 
-	return quote_amp(trim($texte)) . $points;
+	return quote_amp(trim($texte)).$points;
 }
 
 
@@ -446,16 +446,16 @@  discard block
 block discarded – undo
 				define('_PROTEGE_JS_MODELES', creer_uniqid());
 			}
 			foreach ($r as $regs) {
-				$t = str_replace($regs[0], code_echappement($regs[0], 'javascript' . _PROTEGE_JS_MODELES), $t);
+				$t = str_replace($regs[0], code_echappement($regs[0], 'javascript'._PROTEGE_JS_MODELES), $t);
 			}
 		}
-		if (preg_match_all(',<\?php.*?($|\?' . '>),isS', $t, $r, PREG_SET_ORDER)) {
+		if (preg_match_all(',<\?php.*?($|\?'.'>),isS', $t, $r, PREG_SET_ORDER)) {
 			if (!defined('_PROTEGE_PHP_MODELES')) {
 				include_spip('inc/acces');
 				define('_PROTEGE_PHP_MODELES', creer_uniqid());
 			}
 			foreach ($r as $regs) {
-				$t = str_replace($regs[0], code_echappement($regs[0], 'php' . _PROTEGE_PHP_MODELES), $t);
+				$t = str_replace($regs[0], code_echappement($regs[0], 'php'._PROTEGE_PHP_MODELES), $t);
 			}
 		}
 	}
@@ -528,7 +528,7 @@  discard block
 block discarded – undo
 		if (!function_exists('attribut_html')) {
 			include_spip('inc/filtres');
 		}
-		$texte = "<mark class='danger-js' title='" . attribut_html(_T('erreur_contenu_suspect')) . "'>⚠️</mark> " . $texte;
+		$texte = "<mark class='danger-js' title='".attribut_html(_T('erreur_contenu_suspect'))."'>⚠️</mark> ".$texte;
 	}
 
 	return $texte;
@@ -598,11 +598,11 @@  discard block
 block discarded – undo
  **/
 function supprime_img($letexte, $message = null) {
 	if ($message === null) {
-		$message = '(' . _T('img_indisponible') . ')';
+		$message = '('._T('img_indisponible').')';
 	}
 
 	return preg_replace(
-		',<(img|doc|emb)([0-9]+)(\|([^>]*))?' . '\s*/?' . '>,i',
+		',<(img|doc|emb)([0-9]+)(\|([^>]*))?'.'\s*/?'.'>,i',
 		$message,
 		$letexte
 	);

Please login to merge, or discard this patch.

ecrire/inc/lang.php 1 patch

Spacing +10 added lines, -12 removed lines patch added patch discarded remove patch

@@ -40,14 +40,14 @@  discard block
 block discarded – undo
 function changer_langue($lang, $liste_langues = null) {
 
 	if (is_null($liste_langues)) {
-		$liste_langues = @$GLOBALS['meta']['langues_proposees'] . ',' . @$GLOBALS['meta']['langues_multilingue'];
+		$liste_langues = @$GLOBALS['meta']['langues_proposees'].','.@$GLOBALS['meta']['langues_multilingue'];
 	}
 	else {
 		if (is_array($liste_langues)) {
 			$liste_langues = implode(',', $liste_langues);
 		}
 	}
-	$liste_langues = ',' . $liste_langues . ',';
+	$liste_langues = ','.$liste_langues.',';
 
 	// Si la langue demandee n'existe pas, on essaie d'autres variantes
 	// Exemple : 'pt-br' => 'pt_br' => 'pt'
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
 	}
 	foreach ($langues as $l) {
 		$selected = ($l == $default) ? ' selected=\'selected\'' : '';
-		$ret .= "<option value='$l'$selected>[" . $l . '] ' . traduire_nom_langue($l) . "</option>\n";
+		$ret .= "<option value='$l'$selected>[".$l.'] '.traduire_nom_langue($l)."</option>\n";
 	}
 
 	if (!test_espace_prive()) {
@@ -211,23 +211,21 @@  discard block
 block discarded – undo
 		$base,
 		$cible,
 		(select_langues($nom_select, $change, $ret)
-			. "<noscript><div style='display:inline'><input type='submit' class='fondo' value='" . _T('bouton_changer') . "' /></div></noscript>"),
+			. "<noscript><div style='display:inline'><input type='submit' class='fondo' value='"._T('bouton_changer')."' /></div></noscript>"),
 		" method='post'"
 	);
 }
 
 function select_langues($nom_select, $change, $options, $label = '') {
 	static $cpt = 0;
-	$id = 'menu_langues' . $cpt++;
+	$id = 'menu_langues'.$cpt++;
 
 	return
-		"<label for='$id'>" . ($label ?: _T('info_langues')) . '</label> ' .
+		"<label for='$id'>".($label ?: _T('info_langues')).'</label> '.
 		"<select name='$nom_select' id='$id' "
 		. ((!test_espace_prive()) ?
-			("class='forml menu_langues'") :
-			(($nom_select == 'var_lang_ecrire') ?
-				("class='lang_ecrire'") :
-				"class='fondl'"))
+			("class='forml menu_langues'") : (($nom_select == 'var_lang_ecrire') ?
+				("class='lang_ecrire'") : "class='fondl'"))
 		. $change
 		. ">\n"
 		. $options
@@ -353,7 +351,7 @@  discard block
 block discarded – undo
 		and (!isset($GLOBALS['spip_lang'])
 			or $GLOBALS['spip_lang'] != $GLOBALS['meta']['langue_site'])
 	) {
-		return changer_langue($GLOBALS['meta']['langue_site'], $liste_langues);//@:install
+		return changer_langue($GLOBALS['meta']['langue_site'], $liste_langues); //@:install
 	}
 	// en theorie là, la globale est définie, sinon c'est un problème.
 	if (!isset($GLOBALS['spip_lang'])) {
@@ -456,7 +454,7 @@  discard block
 block discarded – undo
 	if (!isset($GLOBALS['meta']['langue_site'])) {
 		// Initialisation : le francais si dispo, sinon la premiere langue trouvee
 		$GLOBALS['meta']['langue_site'] = $tout =
-			(!$all_langs or (strpos(',' . _LANGUE_PAR_DEFAUT . ',', (string) ",$all_langs,") !== false))
+			(!$all_langs or (strpos(','._LANGUE_PAR_DEFAUT.',', (string) ",$all_langs,") !== false))
 				? _LANGUE_PAR_DEFAUT : substr($all_langs, 0, strpos($all_langs, ','));
 		ecrire_meta('langue_site', $tout);
 	}

Please login to merge, or discard this patch.

ecrire/xml/valider.php 1 patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

@@ -61,7 +61,7 @@  discard block
 block discarded – undo
 							. $pere
 							. '</b>'
 							. (!$bons_peres ? ''
-								: ('<p style="font-size: 80%"> ' . _T('zxml_mais_de') . ' <b>' . $bons_peres . '</b></p>')));
+								: ('<p style="font-size: 80%"> '._T('zxml_mais_de').' <b>'.$bons_peres.'</b></p>')));
 					} elseif ($this->dtc->regles[$pere][0] == '/') {
 						$frat = substr($depth, 2);
 						if (!isset($this->fratrie[$frat])) {
@@ -98,15 +98,15 @@  discard block
 block discarded – undo
 		if (!isset($a[$name])) {
 			$bons = join(', ', array_keys($a));
 			if ($bons) {
-				$bons = " title=' " .
-					_T('zxml_connus_attributs') .
-					'&nbsp;: ' .
-					$bons .
+				$bons = " title=' ".
+					_T('zxml_connus_attributs').
+					'&nbsp;: '.
+					$bons.
 					"'";
 			}
 			$bons .= " style='font-weight: bold'";
 			coordonnees_erreur($this, " <b>$name</b> "
-				. _T('zxml_inconnu_attribut') . ' ' . _T('zxml_de')
+				. _T('zxml_inconnu_attribut').' '._T('zxml_de')
 				. " <a$bons>$bal</a> ("
 				. _T('zxml_survoler')
 				. ')');
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
 			if (!preg_match('/^\w+$/', $type)) {
 				$this->valider_motif($phraseur, $name, $val, $bal, $type);
 			} else {
-				if (method_exists($this, $f = 'validerAttribut_' . $type)) {
+				if (method_exists($this, $f = 'validerAttribut_'.$type)) {
 					$this->$f($phraseur, $name, $val, $bal);
 				}
 			}
@@ -164,13 +164,13 @@  discard block
 block discarded – undo
 				. " <b>$bal</b> "
 				. _T('zxml_non_conforme')
 				. '</p><p>'
-				. '<b>' . $motif . '</b>');
+				. '<b>'.$motif.'</b>');
 		}
 	}
 
 	public function valider_idref($nom, $ligne, $col) {
 		if (!isset($this->ids[$nom])) {
-			$this->err[] = [" <p><b>$nom</b> " . _T('zxml_inconnu_id'), $ligne, $col];
+			$this->err[] = [" <p><b>$nom</b> "._T('zxml_inconnu_id'), $ligne, $col];
 		}
 	}
 
@@ -217,7 +217,7 @@  discard block
 block discarded – undo
 		// controler que les balises devant etre vides le sont
 		if ($vide) {
 			if ($n <> ($k + $c)) {
-				coordonnees_erreur($this, " <p><b>$name</b> " . _T('zxml_nonvide_balise'));
+				coordonnees_erreur($this, " <p><b>$name</b> "._T('zxml_nonvide_balise'));
 			}
 			// pour les regles PCDATA ou iteration de disjonction, tout est fait
 		} elseif ($regle and ($regle != '*')) {
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
 			$d = $this->ouvrant[$d];
 			preg_match('/^\s*(\S+)/', $d, $m);
 			if (isset($this->dtc->pcdata[$m[1]]) and ($this->dtc->pcdata[$m[1]])) {
-				coordonnees_erreur($this, ' <p><b>' . $m[1] . '</b> '
+				coordonnees_erreur($this, ' <p><b>'.$m[1].'</b> '
 					. _T('zxml_nonvide_balise')); // message a affiner
 			}
 		}
@@ -296,7 +296,7 @@  discard block
 block discarded – undo
 	public function phraserTout($phraseur, $data) {
 		xml_parsestring($this, $data);
 
-		if (!$this->dtc or preg_match(',^' . _MESSAGE_DOCTYPE . ',', $data)) {
+		if (!$this->dtc or preg_match(',^'._MESSAGE_DOCTYPE.',', $data)) {
 			$this->err[] = ['DOCTYPE ?', 0, 0];
 		} else {
 			$this->valider_passe2();

Please login to merge, or discard this patch.

ecrire/public/compiler.php 1 patch

Spacing +108 added lines, -117 removed lines patch added patch discarded remove patch

@@ -120,7 +120,7 @@  discard block
 block discarded – undo
 						: 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;
 					}
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
 	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 @@  discard block
 block discarded – undo
 	$_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);
 		if ($code and preg_match("/^'([^']*)'/s", $code, $r)) {
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
 		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)";
@@ -235,16 +235,16 @@  discard block
 block discarded – undo
 			$_options[] = $ajax;
 		}
 		$code = " ' . argumenter_squelette($code) . '";
-		$code = 'echo ' . sprintf(
+		$code = 'echo '.sprintf(
 			CODE_RECUPERER_FOND,
 			$code,
 			$contexte,
 			implode(',', $_options),
 			"_request(\\'connect\\') ?? \\'\\'"
-		) . ';';
+		).';';
 	}
 
-	return "\n'<'.'" . '?php ' . $code . "\n?'." . "'>'";
+	return "\n'<'.'".'?php '.$code."\n?'."."'>'";
 }
 
 
@@ -324,7 +324,7 @@  discard block
 block discarded – undo
 					$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');
@@ -332,11 +332,11 @@  discard block
 block discarded – undo
 					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)
 					);
@@ -345,9 +345,9 @@  discard block
 block discarded – undo
 					$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)
 				);
@@ -381,14 +381,14 @@  discard block
 block discarded – undo
 	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; 
 		}';
 	}
 
@@ -422,7 +422,7 @@  discard block
 block discarded – undo
 	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;";
@@ -490,7 +490,7 @@  discard block
 block discarded – undo
 	// 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";
 	}
@@ -518,13 +518,13 @@  discard block
 block discarded – undo
 		$corps .=
 			"\n\t\tlang_select_public("
 			. index_pile($id_boucle, 'lang', $boucles)
-			. ", '" . $boucle->lang_select . "'"
+			. ", '".$boucle->lang_select."'"
 			. (in_array($type_boucle, [
 				'articles',
 				'rubriques',
 				'hierarchie',
 				'breves'
-			]) ? ', ' . index_pile($id_boucle, 'titre', $boucles) : '')
+			]) ? ', '.index_pile($id_boucle, 'titre', $boucles) : '')
 			. ');';
 	} else {
 		$init_lang = '';
@@ -546,20 +546,16 @@  discard block
 block discarded – undo
 
 	// gestion optimale des separateurs et des boucles constantes
 	if (is_countable($boucle->separateur) ? count($boucle->separateur) : 0) {
-		$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
@@ -641,7 +637,7 @@  discard block
 block discarded – undo
 		$corps,
 		$fin_lang,
 		$trace,
-		'BOUCLE' . $id_boucle . ' @ ' . ($boucle->descr['sourcefile'])
+		'BOUCLE'.$id_boucle.' @ '.($boucle->descr['sourcefile'])
 	);
 
 #	var_dump($a);exit;
@@ -662,25 +658,24 @@  discard block
 block discarded – undo
  **/
 function calculer_requete_sql($boucle) {
 	$init = [];
-	$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 . '"' : '') . ')'
+		'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('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)."'" : $boucle->limit
 		)
 	);
 	$init[] = calculer_dec('having', calculer_dump_array($boucle->having));
@@ -689,17 +684,17 @@  discard block
 block discarded – undo
 	// 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;
@@ -782,7 +777,7 @@  discard block
 block discarded – undo
  *    - 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 (
@@ -799,7 +794,7 @@  discard block
 block discarded – undo
 		$static = '';
 	}
 
-	return [$static, '$command[\'' . $nom . '\'] = ' . $val . ';'];
+	return [$static, '$command[\''.$nom.'\'] = '.$val.';'];
 }
 
 /**
@@ -824,27 +819,27 @@  discard block
 block discarded – undo
 	}
 	$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).')';
 	}
 }
 
 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).')';
 }
 
 /**
@@ -861,7 +856,7 @@  discard block
 block discarded – undo
 		$res .= ",'$k' => '$v'";
 	}
 
-	return 'array(' . substr($res, 1) . ')';
+	return 'array('.substr($res, 1).')';
 }
 
 /**
@@ -879,7 +874,7 @@  discard block
 block discarded – undo
 		$res .= ",'$k' => '$v'";
 	}
 
-	return 'array(' . substr($res, 1) . ')';
+	return 'array('.substr($res, 1).')';
 }
 
 function calculer_order(&$boucle) {
@@ -948,19 +943,19 @@  discard block
 block discarded – undo
 				) {
 					$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'] ?: '');
+		$nom = $descr['nom'].$id_boucle.($descr['niv'] ?: '');
 
-		return "join('', array_map('array_shift', \$GLOBALS['debug_objets']['sequence']['$nom'] = array(" . join(
+		return "join('', array_map('array_shift', \$GLOBALS['debug_objets']['sequence']['$nom'] = array(".join(
 			" ,\n$tab",
 			$codes
-		) . ')))';
+		).')))';
 	}
 }
 
@@ -987,7 +982,7 @@  discard block
 block discarded – undo
 			// texte seul
 			case 'texte':
 				$code = sandbox_composer_texte($p->texte, $p);
-				$commentaire = strlen($p->texte) . ' signes';
+				$commentaire = strlen($p->texte).' signes';
 				$avant = '';
 				$apres = '';
 				$altern = "''";
@@ -996,14 +991,14 @@  discard block
 block discarded – undo
 			case 'polyglotte':
 				$code = '';
 				foreach ($p->traductions as $k => $v) {
-					$code .= ",'" .
-						str_replace(['\\', "'"], ['\\\\', "\\'"], $k) .
-						"' => '" .
-						str_replace(['\\', "'"], ['\\\\', "\\'"], $v) .
+					$code .= ",'".
+						str_replace(['\\', "'"], ['\\\\', "\\'"], $k).
+						"' => '".
+						str_replace(['\\', "'"], ['\\\\', "\\'"], $v).
 						"'";
 				}
-				$code = 'choisir_traduction(array(' .
-					substr($code, 1) .
+				$code = 'choisir_traduction(array('.
+					substr($code, 1).
 					'))';
 				$commentaire = '&';
 				$avant = '';
@@ -1019,7 +1014,7 @@  discard block
 block discarded – undo
 					$err_e_c = true;
 					$code = "''";
 				} else {
-					$commentaire = '<INCLURE ' . addslashes(str_replace("\n", ' ', $code)) . '>';
+					$commentaire = '<INCLURE '.addslashes(str_replace("\n", ' ', $code)).'>';
 					$avant = '';
 					$apres = '';
 					$altern = "''";
@@ -1048,8 +1043,8 @@  discard block
 block discarded – undo
 					$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 (
@@ -1091,24 +1086,22 @@  discard block
 block discarded – undo
 				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;
@@ -1130,7 +1123,7 @@  discard block
 block discarded – undo
 				$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,
@@ -1170,10 +1163,9 @@  discard block
 block discarded – undo
 		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
 
@@ -1229,19 +1221,19 @@  discard block
 block discarded – undo
 			$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)) {
 				$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) {
@@ -1292,12 +1284,12 @@  discard block
 block discarded – undo
 
 	// 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(
 		',\\\\([#[()\]{}<>]),',
-		fn($a) => "$inerte-" . ord($a[1]) . '-',
+		fn($a) => "$inerte-".ord($a[1]).'-',
 		$squelette,
 		-1,
 		$esc
@@ -1313,7 +1305,7 @@  discard block
 block discarded – undo
 	// Phraser le squelette, selon sa grammaire
 
 	$boucles = [];
-	$f = charger_fonction('phraser_' . $gram, 'public');
+	$f = charger_fonction('phraser_'.$gram, 'public');
 
 	$squelette = $f($squelette, '', $boucles, $descr);
 
@@ -1329,7 +1321,7 @@  discard block
 block discarded – undo
 			);
 			$boucles[$i]->descr['squelette'] = preg_replace_callback(
 				",$inerte-(\d+)-,",
-				fn($a) => '\\\\' . chr($a[1]),
+				fn($a) => '\\\\'.chr($a[1]),
 				$boucle->descr['squelette']
 			);
 		}
@@ -1340,19 +1332,19 @@  discard block
 block discarded – undo
 		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;
 		}
 	}
 
@@ -1375,7 +1367,7 @@  discard block
 block discarded – undo
 		}
 	}
 	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);
 
@@ -1466,8 +1458,7 @@  discard block
 block discarded – undo
 						} 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 = [
 								'zbug_table_inconnue',
@@ -1557,13 +1548,13 @@  discard block
 block discarded – undo
 			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')) {
@@ -1571,20 +1562,20 @@  discard block
 block discarded – undo
 				}
 			}
 
-			$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";
 	}
 
@@ -1594,7 +1585,7 @@  discard block
 block discarded – undo
 		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
 		. '
@@ -1602,15 +1593,15 @@  discard block
 block discarded – undo
 		$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');
@@ -1624,10 +1615,10 @@  discard block
 block discarded – undo
 	$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;
@@ -1679,7 +1670,7 @@  discard block
 block discarded – undo
  **/
 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, ...)

Please login to merge, or discard this patch.

ecrire/public/phraser_html.php 1 patch

Spacing +28 added lines, -28 removed lines patch added patch discarded remove patch

@@ -48,19 +48,19 @@  discard block
 block discarded – undo
  * Nom d'une balise #TOTO
  *
  * Écriture alambiquée pour rester compatible avec les hexadecimaux des vieux squelettes */
-define('NOM_DE_CHAMP', '#((' . NOM_DE_BOUCLE . "):)?(([A-F]*[G-Z_][A-Z_0-9]*)|[A-Z_]+)\b(\*{0,2})");
+define('NOM_DE_CHAMP', '#(('.NOM_DE_BOUCLE."):)?(([A-F]*[G-Z_][A-Z_0-9]*)|[A-Z_]+)\b(\*{0,2})");
 /** Balise complète [...(#TOTO) ... ] */
-define('CHAMP_ETENDU', '/\[([^]\[]*)\(' . NOM_DE_CHAMP . '([^[)]*\)[^]\[]*)\]/S');
+define('CHAMP_ETENDU', '/\[([^]\[]*)\('.NOM_DE_CHAMP.'([^[)]*\)[^]\[]*)\]/S');
 
 define('BALISE_INCLURE', '/<INCLU[DR]E[[:space:]]*(\(([^)]*)\))?/S');
 define('BALISE_POLYGLOTTE', ',<multi>(.*)</multi>,Uims');
 define('BALISE_IDIOMES', ',<:(([a-z0-9_]+):)?([a-z0-9_]*)({([^\|=>]*=[^\|>]*)})?((\|[^>]*)?:/?>),iS');
-define('BALISE_IDIOMES_ARGS', '@^\s*([^= ]*)\s*=\s*((' . NOM_DE_CHAMP . '[{][^}]*})?[^,]*)\s*,?\s*@s');
+define('BALISE_IDIOMES_ARGS', '@^\s*([^= ]*)\s*=\s*(('.NOM_DE_CHAMP.'[{][^}]*})?[^,]*)\s*,?\s*@s');
 
 /** Champ sql dans parenthèse ex: (id_article) */
 define('SQL_ARGS', '(\([^)]*\))');
 /** Fonction SQL sur un champ ex: SUM(visites) */
-define('CHAMP_SQL_PLUS_FONC', '`?([A-Z_\/][A-Z_\/0-9.]*)' . SQL_ARGS . '?`?');
+define('CHAMP_SQL_PLUS_FONC', '`?([A-Z_\/][A-Z_\/0-9.]*)'.SQL_ARGS.'?`?');
 
 function phraser_inclure($texte, $ligne, $result) {
 
@@ -215,7 +215,7 @@  discard block
 block discarded – undo
  * @return array
  **/
 function phraser_champs($texte, $ligne, $result) {
-	while (preg_match('/' . NOM_DE_CHAMP . '/S', $texte, $match)) {
+	while (preg_match('/'.NOM_DE_CHAMP.'/S', $texte, $match)) {
 		$p = strpos($texte, (string) $match[0]);
 		// texte après la balise
 		$suite = substr($texte, $p + strlen($match[0]));
@@ -363,7 +363,7 @@  discard block
 block discarded – undo
 			$collecte[] = $champ;
 			$args = ltrim($regs[count($regs) - 1]);
 		} else {
-			if (!preg_match('/' . NOM_DE_CHAMP . '([{|])/', $arg, $r)) {
+			if (!preg_match('/'.NOM_DE_CHAMP.'([{|])/', $arg, $r)) {
 				// 0 est un aveu d'impuissance. A completer
 				$arg = phraser_champs_exterieurs($arg, 0, $sep, $result);
 
@@ -447,7 +447,7 @@  discard block
 block discarded – undo
 function phraser_champs_exterieurs($texte, $ligne, $sep, $nested) {
 	$res = [];
 	while (($p = strpos($texte, (string) "%$sep")) !== false) {
-		if (!preg_match(',^%' . preg_quote($sep) . '([0-9]+)@,', substr($texte, $p), $m)) {
+		if (!preg_match(',^%'.preg_quote($sep).'([0-9]+)@,', substr($texte, $p), $m)) {
 			break;
 		}
 		$debut = substr($texte, 0, $p);
@@ -487,7 +487,7 @@  discard block
 block discarded – undo
 			$pos_apres = 0;
 			$result = phraser_args($match[7], ')', $sep, $result, $champ, $pos_apres);
 			phraser_vieux($champ);
-			$champ->avant =	phraser_champs_exterieurs($match[1], $n, $sep, $result);
+			$champ->avant = phraser_champs_exterieurs($match[1], $n, $sep, $result);
 			$debut = substr($match[7], $pos_apres + 1);
 			if (!empty($debut)) {
 				$n += substr_count(substr($texte, 0, strpos($texte, $debut)), "\n");
@@ -618,7 +618,7 @@  discard block
 block discarded – undo
 					// une maniere tres sale de supprimer les "' autour de {critere "xxx","yyy"}
 					if (preg_match(',^(["\'])(.*)\1$,', $m[4])) {
 						$c = null;
-						eval('$c = ' . $m[4] . ';');
+						eval('$c = '.$m[4].';');
 						if (isset($c)) {
 							$m[4] = $c;
 						}
@@ -698,7 +698,7 @@  discard block
 block discarded – undo
 					if (preg_match(',^ *([0-9-]+) *(/) *(.+) *$,', $param, $m)) {
 						$crit = phraser_critere_infixe($m[1], $m[3], $v, '/', '', '');
 					} elseif (
-						preg_match(',^([!]?)(' . CHAMP_SQL_PLUS_FONC .
+						preg_match(',^([!]?)('.CHAMP_SQL_PLUS_FONC.
 						')[[:space:]]*(\??)(!?)(<=?|>=?|==?|\b(?:IN|LIKE)\b)(.*)$,is', $param, $m)
 					) {
 						$a2 = trim($m[8]);
@@ -715,8 +715,8 @@  discard block
 block discarded – undo
 						);
 						$crit->exclus = $m[1];
 					} elseif (
-						preg_match('/^([!]?)\s*(' .
-						CHAMP_SQL_PLUS_FONC .
+						preg_match('/^([!]?)\s*('.
+						CHAMP_SQL_PLUS_FONC.
 						')\s*(\??)(.*)$/is', $param, $m)
 					) {
 						// contient aussi les comparaisons implicites !
@@ -861,7 +861,7 @@  discard block
 block discarded – undo
 			}
 
 			// trouver sa position de depart reelle : au <Bxx> ou au <BBxx>
-			$precond_boucle = BALISE_PRECOND_BOUCLE . $id_boucle . '>';
+			$precond_boucle = BALISE_PRECOND_BOUCLE.$id_boucle.'>';
 			$pos_precond = strpos($texte, $precond_boucle, $id_boucle ? $pos_debut_texte : $pos_derniere_boucle_anonyme);
 			if (
 				$pos_precond !== false
@@ -872,7 +872,7 @@  discard block
 block discarded – undo
 				$boucle['pos_precond_inside'] = $pos_precond + strlen($precond_boucle);
 			}
 
-			$preaff_boucle = BALISE_PREAFF_BOUCLE . $id_boucle . '>';
+			$preaff_boucle = BALISE_PREAFF_BOUCLE.$id_boucle.'>';
 			$pos_preaff = strpos($texte, $preaff_boucle, $id_boucle ? $pos_debut_texte : $pos_derniere_boucle_anonyme);
 			if (
 				$pos_preaff !== false
@@ -920,13 +920,13 @@  discard block
 block discarded – undo
 	$pos_anonyme_next = null;
 	// si c'est une boucle anonyme, chercher la position de la prochaine boucle anonyme
 	if (!strlen($id_boucle)) {
-		$pos_anonyme_next = strpos($texte, BALISE_BOUCLE . '(', $pos_courante);
+		$pos_anonyme_next = strpos($texte, BALISE_BOUCLE.'(', $pos_courante);
 	}
 
 	//
 	// 1. Recuperer la partie conditionnelle apres
 	//
-	$apres_boucle = BALISE_POSTCOND_BOUCLE . $id_boucle . '>';
+	$apres_boucle = BALISE_POSTCOND_BOUCLE.$id_boucle.'>';
 	$pos_apres = strpos($texte, $apres_boucle, $pos_courante);
 	if (
 		$pos_apres !== false
@@ -935,13 +935,13 @@  discard block
 block discarded – undo
 		$boucle['pos_postcond'] = $pos_apres;
 		$pos_apres += strlen($apres_boucle);
 		$boucle['pos_postcond_inside'] = $pos_apres;
-		$pos_courante = $pos_apres ;
+		$pos_courante = $pos_apres;
 	}
 
 	//
 	// 2. Récuperer la partie alternative apres
 	//
-	$altern_boucle = BALISE_ALT_BOUCLE . $id_boucle . '>';
+	$altern_boucle = BALISE_ALT_BOUCLE.$id_boucle.'>';
 	$pos_altern = strpos($texte, $altern_boucle, $pos_courante);
 	if (
 		$pos_altern !== false
@@ -956,7 +956,7 @@  discard block
 block discarded – undo
 	//
 	// 3. Recuperer la partie footer non alternative
 	//
-	$postaff_boucle = BALISE_POSTAFF_BOUCLE . $id_boucle . '>';
+	$postaff_boucle = BALISE_POSTAFF_BOUCLE.$id_boucle.'>';
 	$pos_postaff = strpos($texte, $postaff_boucle, $pos_courante);
 	if (
 		$pos_postaff !== false
@@ -965,7 +965,7 @@  discard block
 block discarded – undo
 		$boucle['pos_postaff'] = $pos_postaff;
 		$pos_postaff += strlen($postaff_boucle);
 		$boucle['pos_postaff_inside'] = $pos_postaff;
-		$pos_courante = $pos_postaff ;
+		$pos_courante = $pos_postaff;
 	}
 
 	return $boucle;
@@ -1005,7 +1005,7 @@  discard block
 block discarded – undo
  * @return string
  */
 function public_generer_boucle_placeholder($id_boucle, &$boucle, $boucle_placeholder, $nb_lignes) {
-	$placeholder = "[(#{$boucle_placeholder}{" . $id_boucle . '})' . str_pad('', $nb_lignes, "\n") . ']';
+	$placeholder = "[(#{$boucle_placeholder}{".$id_boucle.'})'.str_pad('', $nb_lignes, "\n").']';
 	//memoriser la boucle a reinjecter
 	$id_boucle = "$id_boucle";
 	phraser_boucle_placeholder($id_boucle, $boucle_placeholder, $boucle);
@@ -1018,7 +1018,7 @@  discard block
 block discarded – undo
 	// definir un placholder pour les boucles dont on est sur d'avoir aucune occurence dans le squelette
 	if (is_null($boucle_placeholder)) {
 		do {
-			$boucle_placeholder = 'BOUCLE_PLACEHOLDER_' . strtoupper(md5(uniqid()));
+			$boucle_placeholder = 'BOUCLE_PLACEHOLDER_'.strtoupper(md5(uniqid()));
 		} while (strpos($texte, $boucle_placeholder) !== false);
 	}
 
@@ -1038,7 +1038,7 @@  discard block
 block discarded – undo
 
 		// boucle anonyme ?
 		if (!strlen($id_boucle)) {
-			$id_boucle = '_anon_L' . $ligne_milieu . '_' . substr(md5('anonyme:' . $id_parent . ':' . json_encode($boucle, JSON_THROW_ON_ERROR)), 0, 8);
+			$id_boucle = '_anon_L'.$ligne_milieu.'_'.substr(md5('anonyme:'.$id_parent.':'.json_encode($boucle, JSON_THROW_ON_ERROR)), 0, 8);
 		}
 
 		$pos_debut_boucle = $pos_courante;
@@ -1051,7 +1051,7 @@  discard block
 block discarded – undo
 
 			$pos_avant = $boucle['pos_precond_inside'];
 			$result->avant = substr($texte, $pos_avant, $pos_courante - $pos_avant);
-			$ligne_avant = $ligne_debut_texte +  public_compte_ligne($texte, $pos_debut_texte, $pos_avant);
+			$ligne_avant = $ligne_debut_texte + public_compte_ligne($texte, $pos_debut_texte, $pos_avant);
 		}
 
 		// Regarder si on a une partie inconditionnelle avant <BB_xxx>
@@ -1060,7 +1060,7 @@  discard block
 block discarded – undo
 
 			$pos_preaff = $boucle['pos_preaff_inside'];
 			$result->preaff = substr($texte, $pos_preaff, $end_preaff - $pos_preaff);
-			$ligne_preaff = $ligne_debut_texte +  public_compte_ligne($texte, $pos_debut_texte, $pos_preaff);
+			$ligne_preaff = $ligne_debut_texte + public_compte_ligne($texte, $pos_debut_texte, $pos_preaff);
 		}
 
 		$result->id_boucle = $id_boucle;
@@ -1117,7 +1117,7 @@  discard block
 block discarded – undo
 		} else {
 			$pos_milieu += 1;
 
-			$fin_boucle = BALISE_FIN_BOUCLE . $id_boucle_search . '>';
+			$fin_boucle = BALISE_FIN_BOUCLE.$id_boucle_search.'>';
 			$pos_fin = strpos($texte, $fin_boucle, $pos_milieu);
 			if ($pos_fin === false) {
 				$err_b = [
@@ -1130,7 +1130,7 @@  discard block
 block discarded – undo
 			else {
 				// verifier une eventuelle imbrication d'une boucle homonyme
 				// (interdite, generera une erreur plus loin, mais permet de signaler la bonne erreur)
-				$search_debut_boucle = BALISE_BOUCLE . $id_boucle_search . '(';
+				$search_debut_boucle = BALISE_BOUCLE.$id_boucle_search.'(';
 				$search_from = $pos_milieu;
 				$nb_open = 1;
 				$nb_close = 1;
@@ -1170,7 +1170,7 @@  discard block
 block discarded – undo
 		if ($boucle['pos_postcond']) {
 			$result->apres = substr($texte, $pos_courante, $boucle['pos_postcond'] - $pos_courante);
 			$ligne_suite += public_compte_ligne($texte, $pos_courante, $boucle['pos_postcond_inside']);
-			$pos_courante = $boucle['pos_postcond_inside'] ;
+			$pos_courante = $boucle['pos_postcond_inside'];
 		}
 
 

Please login to merge, or discard this patch.

ecrire/install/etape_ldap4.php 1 patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

@@ -39,8 +39,8 @@  discard block
 block discarded – undo
 	if ($fail) {
 		echo info_etape(_T('info_chemin_acces_annuaire')),
 		info_progression_etape(3, 'etape_ldap', 'install/', true),
-			"<div class='error'><p><b>" . _T('avis_operation_echec') . '</b></p><p>' . _T('avis_chemin_invalide_1'),
-			' (<tt>' . spip_htmlspecialchars($base_ldap) . '</tt>) ' . _T('avis_chemin_invalide_2') . '</p></div>';
+			"<div class='error'><p><b>"._T('avis_operation_echec').'</b></p><p>'._T('avis_chemin_invalide_1'),
+			' (<tt>'.spip_htmlspecialchars($base_ldap).'</tt>) '._T('avis_chemin_invalide_2').'</p></div>';
 	} else {
 		info_etape(_T('info_reglage_ldap'));
 		echo info_progression_etape(4, 'etape_ldap', 'install/');
@@ -53,12 +53,12 @@  discard block
 block discarded – undo
 
 		$res = install_propager(['adresse_ldap', 'port_ldap', 'login_ldap', 'pass_ldap', 'protocole_ldap', 'tls_ldap'])
 			. "<input type='hidden' name='etape' value='ldap5' />"
-			. "<input type='hidden' name='base_ldap' value='" . spip_htmlentities($base_ldap) . "' />"
+			. "<input type='hidden' name='base_ldap' value='".spip_htmlentities($base_ldap)."' />"
 			. fieldset(
 				_T('info_statut_utilisateurs_1'),
 				[
 					'statut_ldap' => [
-						'label' => _T('info_statut_utilisateurs_2') . '<br />',
+						'label' => _T('info_statut_utilisateurs_2').'<br />',
 						'valeur' => $statut_ldap,
 						'alternatives' => $statuts
 					]
@@ -75,9 +75,9 @@  discard block
 block discarded – undo
 
 function liste_statuts_ldap() {
 	$recom = [
-		'info_administrateurs' => ('<b>' . _T('info_administrateur_1') . '</b> ' . _T('info_administrateur_2') . '<br />'),
-		'info_redacteurs' => ('<b>' . _T('info_redacteur_1') . '</b> ' . _T('info_redacteur_2') . '<br />'),
-		'info_visiteurs' => ('<b>' . _T('info_visiteur_1') . '</b> ' . _T('info_visiteur_2') . '<br />')
+		'info_administrateurs' => ('<b>'._T('info_administrateur_1').'</b> '._T('info_administrateur_2').'<br />'),
+		'info_redacteurs' => ('<b>'._T('info_redacteur_1').'</b> '._T('info_redacteur_2').'<br />'),
+		'info_visiteurs' => ('<b>'._T('info_visiteur_1').'</b> '._T('info_visiteur_2').'<br />')
 	];
 
 	$res = [];
@@ -93,14 +93,14 @@  discard block
 block discarded – undo
 function install_ldap_correspondances() {
 	$champs = [];
 	foreach (is_array($GLOBALS['ldap_attributes']) ? $GLOBALS['ldap_attributes'] : [] as $champ => $v) {
-		$nom = 'ldap_' . $champ;
+		$nom = 'ldap_'.$champ;
 		$val = is_array($v) ? join(',', $v) : strval($v);
 		$champs[$nom] = [
-			'label' => _T('ldap_correspondance', ['champ' => "<tt>$champ</tt>"]) . '<br />',
+			'label' => _T('ldap_correspondance', ['champ' => "<tt>$champ</tt>"]).'<br />',
 			'valeur' => $val
 		];
 	}
 
 	return !$champs ?
-		'' : fieldset(_T('ldap_correspondance_1'), $champs, '', _T('ldap_correspondance_2') . '<br /><br />');
+		'' : fieldset(_T('ldap_correspondance_1'), $champs, '', _T('ldap_correspondance_2').'<br /><br />');
 }

Please login to merge, or discard this patch.

ecrire/balise/formulaire_admin.php 1 patch

Spacing +10 added lines, -11 removed lines patch added patch discarded remove patch

@@ -101,8 +101,8 @@  discard block
 block discarded – undo
 			}
 			foreach ($debug['sourcefile'] as $k => $v) {
 				if (strpos($v, 'administration.') !== false) {
-					if (isset($debug['resultat'][$k . 'tout'])) {
-						return $debug['resultat'][$k . 'tout'];
+					if (isset($debug['resultat'][$k.'tout'])) {
+						return $debug['resultat'][$k.'tout'];
 					}
 				}
 			}
@@ -180,12 +180,12 @@  discard block
 block discarded – undo
 			and $id = intval($id)
 			and $desc = $trouver_table(table_objet_sql($type))
 		) {
-			$id = sql_getfetsel($_id_type, table_objet_sql($type), "$_id_type=" . intval($id));
+			$id = sql_getfetsel($_id_type, table_objet_sql($type), "$_id_type=".intval($id));
 			if ($id) {
 				$env[$_id_type] = $id;
 				$env['objet'] = $type;
 				$env['id_objet'] = $id;
-				$env['voir_' . $obj] =
+				$env['voir_'.$obj] =
 					str_replace('&amp;', '&', generer_objet_url($id, $obj, '', '', false));
 				if (
 					isset($desc['field']['id_rubrique'])
@@ -239,10 +239,10 @@  discard block
 block discarded – undo
 	$notpub = sql_in('statut', ['prop', 'prive']);
 
 	if ($type == 'article' and $GLOBALS['meta']['post_dates'] != 'oui') {
-		$notpub .= " OR (statut='publie' AND date>" . sql_quote(date('Y-m-d H:i:s')) . ')';
+		$notpub .= " OR (statut='publie' AND date>".sql_quote(date('Y-m-d H:i:s')).')';
 	}
 
-	return sql_fetsel('1', table_objet_sql($type), id_table_objet($type) . '=' . $id . " AND ($notpub)");
+	return sql_fetsel('1', table_objet_sql($type), id_table_objet($type).'='.$id." AND ($notpub)");
 }
 
 
@@ -256,9 +256,9 @@  discard block
 block discarded – undo
 	$alang = '';
 	if (!empty($_COOKIE['spip_admin'])) {
 		$email_or_login = preg_replace(',^@,', '', $_COOKIE['spip_admin']);
-		$alang = sql_getfetsel('lang', 'spip_auteurs', 'email=' . sql_quote($email_or_login));
+		$alang = sql_getfetsel('lang', 'spip_auteurs', 'email='.sql_quote($email_or_login));
 		if (!$alang) {
-			$alang = sql_getfetsel('lang', 'spip_auteurs', 'login=' . sql_quote($email_or_login));
+			$alang = sql_getfetsel('lang', 'spip_auteurs', 'login='.sql_quote($email_or_login));
 		}
 	}
 	if (!$alang) {
@@ -283,9 +283,8 @@  discard block
 block discarded – undo
 
 	return ((!isset($GLOBALS['xhtml']) or $GLOBALS['xhtml'] !== 'true') ?
 		(parametre_url(self(), 'var_mode', 'debug', '&')
-			. '&var_mode_affiche=validation') :
-		('http://validator.w3.org/check?uri='
-			. rawurlencode('http://' . $_SERVER['HTTP_HOST'] . nettoyer_uri())));
+			. '&var_mode_affiche=validation') : ('http://validator.w3.org/check?uri='
+			. rawurlencode('http://'.$_SERVER['HTTP_HOST'].nettoyer_uri())));
 }
 
 /**

Please login to merge, or discard this patch.

		@@ -53,7 +53,7 @@ discard block
		block discarded – undo
53	53	$class = ' blocreplie';
54	54	}
55	55
56		- return '<div ' . ($id ? "id='$id' " : '') . "class='bloc_depliable$class'>";
	56	+ return '<div '.($id ? "id='$id' " : '')."class='bloc_depliable$class'>";
57	57	}
58	58
59	59	function fin_block() {
		@@ -64,12 +64,12 @@ discard block
		block discarded – undo
64	64	// $deplie : true (deplie) ou false (plie) ou -1 (inactif) ou 'incertain' pour que le bouton s'auto init au chargement de la page
65	65	// $ids : id des div lies au bouton (facultatif, par defaut c'est le div.bloc_depliable qui suit)
66	66	function bouton_block_depliable($texte, $deplie, $ids = '') {
67		- $bouton_id = 'b' . substr(md5($texte . microtime()), 0, 8);
	67	+ $bouton_id = 'b'.substr(md5($texte.microtime()), 0, 8);
68	68
69	69	$class = ($deplie === true) ? ' deplie' : (($deplie == -1) ? ' impliable' : ' replie');
70	70	if (strlen($ids)) {
71	71	$cible = explode(',', $ids);
72		- $cible = '#' . implode(',#', $cible);
	72	+ $cible = '#'.implode(',#', $cible);
73	73	} else {
74	74	$cible = "#$bouton_id + div.bloc_depliable";
75	75	}

		@@ -98,7 +98,7 @@ discard block
		block discarded – undo
98	98	return $r ?: traiter_lien_explicite($ref, $texte, $pour, $connect, $echappe_typo);
99	99	}
100	100
101		-define('_EXTRAIRE_LIEN', ',^\s(?:' . _PROTOCOLES_STD . '):?/?/?\s$,iS');
	101	+define('_EXTRAIRE_LIEN', ',^\s(?:'._PROTOCOLES_STD.'):?/?/?\s$,iS');
102	102
103	103	function traiter_lien_explicite($ref, $texte = '', $pour = 'url', string $connect = '', $echappe_typo = true) {
104	104	if (preg_match(_EXTRAIRE_LIEN, $ref)) {
		@@ -114,19 +114,19 @@ discard block
		block discarded – undo
114	114	$lien_court = charger_fonction('lien_court', 'inc');
115	115	$texte = $lien_court($texte);
116	116	if ($echappe_typo) {
117		- $texte = '<html>' . quote_amp($texte) . '</html>';
	117	+ $texte = '<html>'.quote_amp($texte).'</html>';
118	118	}
119	119	}
120	120
121	121	// petites corrections d'URL
122	122	if (preg_match('/^www\.[^@]+$/S', $lien)) {
123		- $lien = 'http://' . $lien;
	123	+ $lien = 'http://'.$lien;
124	124	} else {
125	125	if (strpos($lien, '@') && email_valide($lien)) {
126	126	if (!$texte) {
127	127	$texte = $lien;
128	128	}
129		- $lien = 'mailto:' . $lien;
	129	+ $lien = 'mailto:'.$lien;
130	130	}
131	131	}
132	132
		@@ -142,7 +142,7 @@ discard block
		block discarded – undo
142	142	}
143	143
144	144	function liens_implicite_glose_dist($texte, $id, $type, $args, $ancre, string $connect = '') {
145		- if (function_exists($f = 'glossaire_' . $ancre)) {
	145	+ if (function_exists($f = 'glossaire_'.$ancre)) {
146	146	$url = $f($texte, $id);
147	147	} else {
148	148	$url = glossaire_std($texte);
		@@ -156,7 +156,7 @@ discard block
		block discarded – undo
156	156	if (!($match = typer_raccourci($ref))) {
157	157	return false;
158	158	}
159		- [$type, , $id, , $args, , $ancre] = array_pad($match, 7, null);
	159	+ [$type,, $id,, $args,, $ancre] = array_pad($match, 7, null);
160	160	// attention dans le cas des sites le lien doit pointer non pas sur
161	161	// la page locale du site, mais directement sur le site lui-meme
162	162	if ($f = charger_fonction("implicite_$type", 'liens', true)) {
		@@ -183,7 +183,7 @@ discard block
		block discarded – undo
183	183	$r['titre'] = $texte;
184	184	}
185	185	if (!@$r['titre']) {
186		- $r['titre'] = _T($type) . " $id";
	186	+ $r['titre'] = _T($type)." $id";
187	187	}
188	188	if ($pour == 'titre') {
189	189	return $r['titre'];
		@@ -196,7 +196,7 @@ discard block
		block discarded – undo
196	196	and $mime = sql_getfetsel(
197	197	'mime_type',
198	198	'spip_types_documents',
199		- 'extension IN (' . sql_get_select('extension', 'spip_documents', 'id_document=' . sql_quote($id)) . ')',
	199	+ 'extension IN ('.sql_get_select('extension', 'spip_documents', 'id_document='.sql_quote($id)).')',
200	200	'',
201	201	'',
202	202	'',
		@@ -297,11 +297,11 @@ discard block
		block discarded – undo
297	297	'(<([a-z_-]{3,})' # <modele
298	298	. '\s([0-9])\s*' # id
299	299	. '([\|](?:<[^<>]>\|[^>])?)?' # \|arguments (y compris des tags <...>)
300		- . '\s*/?' . '>)' # fin du modele >
	300	+ . '\s*/?'.'>)' # fin du modele >
301	301	. '\s*(<\/a>)?' # eventuel </a>
302	302	);
303	303
304		-define('_RACCOURCI_MODELE_DEBUT', '@^' . _RACCOURCI_MODELE . '@isS');
	304	+define('_RACCOURCI_MODELE_DEBUT', '@^'._RACCOURCI_MODELE.'@isS');
305	305
306	306	function traiter_modeles($texte, $doublons = false, $echap = '', string $connect = '', $liens = null, $env = []) {
307	307	// preserver la compatibilite : true = recherche des documents
		@@ -380,7 +380,7 @@ discard block
		block discarded – undo
380	380	if (test_espace_prive()) {
381	381	$modele = entites_html(substr($texte, $a, $cherche));
382	382	if (!is_null($liens)) {
383		- $modele = '<pre>' . str_replace($liens[0], $liens[1], $modele) . '</pre>';
	383	+ $modele = '<pre>'.str_replace($liens[0], $liens[1], $modele).'</pre>';
384	384	}
385	385	}
386	386	}

		@@ -44,13 +44,13 @@ discard block
		block discarded – undo
44	44	// celle du texte) et public (spip_lang est la langue du texte)
45	45	$dir = _DIR_RESTREINT ? lang_dir() : lang_dir($GLOBALS['spip_lang']);
46	46
47		- $p = 'puce' . (test_espace_prive() ? '_prive' : '');
	47	+ $p = 'puce'.(test_espace_prive() ? '_prive' : '');
48	48	if ($dir == 'rtl') {
49	49	$p .= '_rtl';
50	50	}
51	51
52	52	if (!isset($GLOBALS[$p])) {
53		- $GLOBALS[$p] = '<span class="spip-puce ' . $dir . '"><b>–</b></span>';
	53	+ $GLOBALS[$p] = '<span class="spip-puce '.$dir.'"><b>–</b></span>';
54	54	}
55	55
56	56	return $GLOBALS[$p];
		@@ -68,7 +68,7 @@ discard block
		block discarded – undo
68	68	}
69	69
70	70	if (!defined('_BALISES_BLOCS_REGEXP')) {
71		- define('_BALISES_BLOCS_REGEXP', ',</?(' . _BALISES_BLOCS . ')[>[:space:]],iS');
	71	+ define('_BALISES_BLOCS_REGEXP', ',</?('._BALISES_BLOCS.')[>[:space:]],iS');
72	72	}
73	73
74	74	//
		@@ -85,7 +85,7 @@ discard block
		block discarded – undo
85	85
86	86	// Tester si on echappe en span ou en div
87	87	if (is_null($mode) or !in_array($mode, ['div', 'span'])) {
88		- $mode = preg_match(',</?(' . _BALISES_BLOCS . ')[>[:space:]],iS', $rempl) ? 'div' : 'span';
	88	+ $mode = preg_match(',</?('._BALISES_BLOCS.')[>[:space:]],iS', $rempl) ? 'div' : 'span';
89	89	}
90	90
91	91	// Decouper en morceaux, base64 a des probleme selon la taille de la pile
		@@ -120,7 +120,7 @@ discard block
		block discarded – undo
120	120	) {
121	121	foreach ($matches as $m) {
122	122	if ($m[1] === 'code') {
123		- $code = '<code' . $m[2] . '>' . spip_htmlspecialchars($m[3]) . '</code>';
	123	+ $code = '<code'.$m[2].'>'.spip_htmlspecialchars($m[3]).'</code>';
124	124	$pre = str_replace($m[0], $code, $pre);
125	125	}
126	126	}
		@@ -130,7 +130,7 @@ discard block
		block discarded – undo
130	130
131	131	// Echapper les <code>...</ code>
132	132	function traiter_echap_code_dist($regs) {
133		- [, , $att, $corps] = $regs;
	133	+ [,, $att, $corps] = $regs;
134	134	$echap = spip_htmlspecialchars($corps); // il ne faut pas passer dans entites_html, ne pas transformer les &#xxx; du code !
135	135
136	136	// ne pas mettre le <div...> s'il n'y a qu'une ligne
		@@ -141,9 +141,9 @@ discard block
		block discarded – undo
141	141	$echap = nl2br($echap);
142	142	$echap = "<div style='text-align: left;' "
143	143	. "class='spip_code' dir='ltr'><code$att>"
144		- . $echap . '</code></div>';
	144	+ . $echap.'</code></div>';
145	145	} else {
146		- $echap = "<code$att class='spip_code' dir='ltr'>" . $echap . '</code>';
	146	+ $echap = "<code$att class='spip_code' dir='ltr'>".$echap.'</code>';
147	147	}
148	148
149	149	$echap = str_replace("\t", '        ', $echap);
		@@ -229,8 +229,8 @@ discard block
		block discarded – undo
229	229	} // sinon les traiter selon le cas
230	230	else {
231	231	if (
232		- function_exists($f = $callback_prefix . 'traiter_echap_' . strtolower($regs[1]))
233		- or function_exists($f = $f . '_dist')
	232	+ function_exists($f = $callback_prefix.'traiter_echap_'.strtolower($regs[1]))
	233	+ or function_exists($f = $f.'_dist')
234	234	) {
235	235	$echap = $f($regs);
236	236	}
		@@ -251,7 +251,7 @@ discard block
		block discarded – undo
251	251	// dans une callback autonommee
252	252	if (strpos($preg ?: _PROTEGE_BLOCS, 'script') !== false) {
253	253	if (
254		- strpos($letexte, '<' . '?') !== false and preg_match_all(
	254	+ strpos($letexte, '<'.'?') !== false and preg_match_all(
255	255	',<[?].*($\|[?]>),UisS',
256	256	$letexte,
257	257	$matches,
		@@ -283,7 +283,7 @@ discard block
		block discarded – undo
283	283	strpos($letexte, '<') !== false
284	284	and
285	285	preg_match_all(
286		- ',<(span\|div)\sclass=[\'"]base64' . $source . '[\'"]\s(.)>\s</\1>,UmsS',
	286	+ ',<(span\|div)\sclass=[\'"]base64'.$source.'[\'"]\s(.)>\s</\1>,UmsS',
287	287	$letexte,
288	288	$regs,
289	289	PREG_SET_ORDER
		@@ -300,7 +300,7 @@ discard block
		block discarded – undo
300	300	}
301	301	}
302	302	if ($at) {
303		- $rempl = '<' . $reg[1] . '>' . $rempl . '</' . $reg[1] . '>';
	303	+ $rempl = '<'.$reg[1].'>'.$rempl.'</'.$reg[1].'>';
304	304	foreach ($at as $attr => $a) {
305	305	$rempl = inserer_attribut($rempl, $attr, $a);
306	306	}
		@@ -381,8 +381,8 @@ discard block
		block discarded – undo
381	381	$texte = nettoyer_raccourcis_typo($texte);
382	382
383	383	// balises de sauts de ligne et paragraphe
384		- $texte = preg_replace('/<p( [^>]*)?' . '>/', "\r", $texte);
385		- $texte = preg_replace('/<br( [^>]*)?' . '>/', "\n", $texte);
	384	+ $texte = preg_replace('/<p( [^>]*)?'.'>/', "\r", $texte);
	385	+ $texte = preg_replace('/<br( [^>]*)?'.'>/', "\n", $texte);
386	386
387	387	// on repasse les doubles \n en \r que nettoyer_raccourcis_typo() a pu modifier
388	388	$texte = str_replace("\n\n", "\r", $texte);
		@@ -390,7 +390,7 @@ discard block
		block discarded – undo
390	390	// supprimer les tags
391	391	$texte = supprimer_tags($texte);
392	392	$texte = trim(str_replace("\n", ' ', $texte));
393		- $texte .= "\n"; // marquer la fin
	393	+ $texte .= "\n"; // marquer la fin
394	394
395	395	// corriger la longueur de coupe
396	396	// en fonction de la presence de caracteres utf
		@@ -405,7 +405,7 @@ discard block
		block discarded – undo
405	405	// couper au mot precedent
406	406	$long = spip_substr($texte, 0, max($taille - 4, 1));
407	407	$u = $GLOBALS['meta']['pcre_u'];
408		- $court = preg_replace("/([^\s][\s]+)[^\s]*\n?$/" . $u, "\\1", $long);
	408	+ $court = preg_replace("/([^\s][\s]+)[^\s]*\n?$/".$u, "\\1", $long);
409	409	if (is_null($suite)) {
410	410	$suite = (defined('_COUPER_SUITE') ? _COUPER_SUITE : ' (...)');
411	411	}
		@@ -415,7 +415,7 @@ discard block
		block discarded – undo
415	415	if (spip_strlen($court) < max(0.75 * $taille, 2)) {
416	416	$points = '';
417	417	$long = spip_substr($texte, 0, $taille);
418		- $texte = preg_replace("/([^\s][\s]+)[^\s]*\n?$/" . $u, "\\1", $long);
	418	+ $texte = preg_replace("/([^\s][\s]+)[^\s]*\n?$/".$u, "\\1", $long);
419	419	// encore trop court ? couper au caractere
420	420	if (spip_strlen($texte) < 0.75 * $taille) {
421	421	$texte = $long;
		@@ -434,7 +434,7 @@ discard block
		block discarded – undo
434	434	// supprimer l'eventuelle entite finale mal coupee
435	435	$texte = preg_replace('/&#?[a-z0-9]*$/S', '', $texte);
436	436
437		- return quote_amp(trim($texte)) . $points;
	437	+ return quote_amp(trim($texte)).$points;
438	438	}
439	439
440	440
		@@ -446,16 +446,16 @@ discard block
		block discarded – undo
446	446	define('_PROTEGE_JS_MODELES', creer_uniqid());
447	447	}
448	448	foreach ($r as $regs) {
449		- $t = str_replace($regs[0], code_echappement($regs[0], 'javascript' . _PROTEGE_JS_MODELES), $t);
	449	+ $t = str_replace($regs[0], code_echappement($regs[0], 'javascript'._PROTEGE_JS_MODELES), $t);
450	450	}
451	451	}
452		- if (preg_match_all(',<\?php.*?($\|\?' . '>),isS', $t, $r, PREG_SET_ORDER)) {
	452	+ if (preg_match_all(',<\?php.*?($\|\?'.'>),isS', $t, $r, PREG_SET_ORDER)) {
453	453	if (!defined('_PROTEGE_PHP_MODELES')) {
454	454	include_spip('inc/acces');
455	455	define('_PROTEGE_PHP_MODELES', creer_uniqid());
456	456	}
457	457	foreach ($r as $regs) {
458		- $t = str_replace($regs[0], code_echappement($regs[0], 'php' . _PROTEGE_PHP_MODELES), $t);
	458	+ $t = str_replace($regs[0], code_echappement($regs[0], 'php'._PROTEGE_PHP_MODELES), $t);
459	459	}
460	460	}
461	461	}
		@@ -528,7 +528,7 @@ discard block
		block discarded – undo
528	528	if (!function_exists('attribut_html')) {
529	529	include_spip('inc/filtres');
530	530	}
531		- $texte = "<mark class='danger-js' title='" . attribut_html(_T('erreur_contenu_suspect')) . "'>⚠️</mark> " . $texte;
	531	+ $texte = "<mark class='danger-js' title='".attribut_html(_T('erreur_contenu_suspect'))."'>⚠️</mark> ".$texte;
532	532	}
533	533
534	534	return $texte;
		@@ -598,11 +598,11 @@ discard block
		block discarded – undo
598	598	**/
599	599	function supprime_img($letexte, $message = null) {
600	600	if ($message === null) {
601		- $message = '(' . _T('img_indisponible') . ')';
	601	+ $message = '('._T('img_indisponible').')';
602	602	}
603	603
604	604	return preg_replace(
605		- ',<(img\|doc\|emb)([0-9]+)(\\|([^>]))?' . '\s/?' . '>,i',
	605	+ ',<(img\|doc\|emb)([0-9]+)(\\|([^>]))?'.'\s/?'.'>,i',
606	606	$message,
607	607	$letexte
608	608	);

		@@ -40,14 +40,14 @@ discard block
		block discarded – undo
40	40	function changer_langue($lang, $liste_langues = null) {
41	41
42	42	if (is_null($liste_langues)) {
43		- $liste_langues = @$GLOBALS['meta']['langues_proposees'] . ',' . @$GLOBALS['meta']['langues_multilingue'];
	43	+ $liste_langues = @$GLOBALS['meta']['langues_proposees'].','.@$GLOBALS['meta']['langues_multilingue'];
44	44	}
45	45	else {
46	46	if (is_array($liste_langues)) {
47	47	$liste_langues = implode(',', $liste_langues);
48	48	}
49	49	}
50		- $liste_langues = ',' . $liste_langues . ',';
	50	+ $liste_langues = ','.$liste_langues.',';
51	51
52	52	// Si la langue demandee n'existe pas, on essaie d'autres variantes
53	53	// Exemple : 'pt-br' => 'pt_br' => 'pt'
		@@ -193,7 +193,7 @@ discard block
		block discarded – undo
193	193	}
194	194	foreach ($langues as $l) {
195	195	$selected = ($l == $default) ? ' selected=\'selected\'' : '';
196		- $ret .= "<option value='$l'$selected>[" . $l . '] ' . traduire_nom_langue($l) . "</option>\n";
	196	+ $ret .= "<option value='$l'$selected>[".$l.'] '.traduire_nom_langue($l)."</option>\n";
197	197	}
198	198
199	199	if (!test_espace_prive()) {
		@@ -211,23 +211,21 @@ discard block
		block discarded – undo
211	211	$base,
212	212	$cible,
213	213	(select_langues($nom_select, $change, $ret)
214		- . "<noscript><div style='display:inline'><input type='submit' class='fondo' value='" . _T('bouton_changer') . "' /></div></noscript>"),
	214	+ . "<noscript><div style='display:inline'><input type='submit' class='fondo' value='"._T('bouton_changer')."' /></div></noscript>"),
215	215	" method='post'"
216	216	);
217	217	}
218	218
219	219	function select_langues($nom_select, $change, $options, $label = '') {
220	220	static $cpt = 0;
221		- $id = 'menu_langues' . $cpt++;
	221	+ $id = 'menu_langues'.$cpt++;
222	222
223	223	return
224		- "<label for='$id'>" . ($label ?: _T('info_langues')) . '</label> ' .
	224	+ "<label for='$id'>".($label ?: _T('info_langues')).'</label> '.
225	225	"<select name='$nom_select' id='$id' "
226	226	. ((!test_espace_prive()) ?
227		- ("class='forml menu_langues'") :
228		- (($nom_select == 'var_lang_ecrire') ?
229		- ("class='lang_ecrire'") :
230		- "class='fondl'"))
	227	+ ("class='forml menu_langues'") : (($nom_select == 'var_lang_ecrire') ?
	228	+ ("class='lang_ecrire'") : "class='fondl'"))
231	229	. $change
232	230	. ">\n"
233	231	. $options
		@@ -353,7 +351,7 @@ discard block
		block discarded – undo
353	351	and (!isset($GLOBALS['spip_lang'])
354	352	or $GLOBALS['spip_lang'] != $GLOBALS['meta']['langue_site'])
355	353	) {
356		- return changer_langue($GLOBALS['meta']['langue_site'], $liste_langues);//@:install
	354	+ return changer_langue($GLOBALS['meta']['langue_site'], $liste_langues); //@:install
357	355	}
358	356	// en theorie là, la globale est définie, sinon c'est un problème.
359	357	if (!isset($GLOBALS['spip_lang'])) {
		@@ -456,7 +454,7 @@ discard block
		block discarded – undo
456	454	if (!isset($GLOBALS['meta']['langue_site'])) {
457	455	// Initialisation : le francais si dispo, sinon la premiere langue trouvee
458	456	$GLOBALS['meta']['langue_site'] = $tout =
459		- (!$all_langs or (strpos(',' . _LANGUE_PAR_DEFAUT . ',', (string) ",$all_langs,") !== false))
	457	+ (!$all_langs or (strpos(','._LANGUE_PAR_DEFAUT.',', (string) ",$all_langs,") !== false))
460	458	? _LANGUE_PAR_DEFAUT : substr($all_langs, 0, strpos($all_langs, ','));
461	459	ecrire_meta('langue_site', $tout);
462	460	}

		@@ -61,7 +61,7 @@ discard block
		block discarded – undo
61	61	. $pere
62	62	. '</b>'
63	63	. (!$bons_peres ? ''
64		- : ('<p style="font-size: 80%"> ' . _T('zxml_mais_de') . ' <b>' . $bons_peres . '</b></p>')));
	64	+ : ('<p style="font-size: 80%"> '._T('zxml_mais_de').' <b>'.$bons_peres.'</b></p>')));
65	65	} elseif ($this->dtc->regles[$pere][0] == '/') {
66	66	$frat = substr($depth, 2);
67	67	if (!isset($this->fratrie[$frat])) {
		@@ -98,15 +98,15 @@ discard block
		block discarded – undo
98	98	if (!isset($a[$name])) {
99	99	$bons = join(', ', array_keys($a));
100	100	if ($bons) {
101		- $bons = " title=' " .
102		- _T('zxml_connus_attributs') .
103		- ' : ' .
104		- $bons .
	101	+ $bons = " title=' ".
	102	+ _T('zxml_connus_attributs').
	103	+ ' : '.
	104	+ $bons.
105	105	"'";
106	106	}
107	107	$bons .= " style='font-weight: bold'";
108	108	coordonnees_erreur($this, " <b>$name</b> "
109		- . _T('zxml_inconnu_attribut') . ' ' . _T('zxml_de')
	109	+ . _T('zxml_inconnu_attribut').' '._T('zxml_de')
110	110	. " <a$bons>$bal</a> ("
111	111	. _T('zxml_survoler')
112	112	. ')');
		@@ -115,7 +115,7 @@ discard block
		block discarded – undo
115	115	if (!preg_match('/^\w+$/', $type)) {
116	116	$this->valider_motif($phraseur, $name, $val, $bal, $type);
117	117	} else {
118		- if (method_exists($this, $f = 'validerAttribut_' . $type)) {
	118	+ if (method_exists($this, $f = 'validerAttribut_'.$type)) {
119	119	$this->$f($phraseur, $name, $val, $bal);
120	120	}
121	121	}
		@@ -164,13 +164,13 @@ discard block
		block discarded – undo
164	164	. " <b>$bal</b> "
165	165	. _T('zxml_non_conforme')
166	166	. '</p><p>'
167		- . '<b>' . $motif . '</b>');
	167	+ . '<b>'.$motif.'</b>');
168	168	}
169	169	}
170	170
171	171	public function valider_idref($nom, $ligne, $col) {
172	172	if (!isset($this->ids[$nom])) {
173		- $this->err[] = [" <p><b>$nom</b> " . _T('zxml_inconnu_id'), $ligne, $col];
	173	+ $this->err[] = [" <p><b>$nom</b> "._T('zxml_inconnu_id'), $ligne, $col];
174	174	}
175	175	}
176	176
		@@ -217,7 +217,7 @@ discard block
		block discarded – undo
217	217	// controler que les balises devant etre vides le sont
218	218	if ($vide) {
219	219	if ($n <> ($k + $c)) {
220		- coordonnees_erreur($this, " <p><b>$name</b> " . _T('zxml_nonvide_balise'));
	220	+ coordonnees_erreur($this, " <p><b>$name</b> "._T('zxml_nonvide_balise'));
221	221	}
222	222	// pour les regles PCDATA ou iteration de disjonction, tout est fait
223	223	} elseif ($regle and ($regle != '*')) {
		@@ -252,7 +252,7 @@ discard block
		block discarded – undo
252	252	$d = $this->ouvrant[$d];
253	253	preg_match('/^\s*(\S+)/', $d, $m);
254	254	if (isset($this->dtc->pcdata[$m[1]]) and ($this->dtc->pcdata[$m[1]])) {
255		- coordonnees_erreur($this, ' <p><b>' . $m[1] . '</b> '
	255	+ coordonnees_erreur($this, ' <p><b>'.$m[1].'</b> '
256	256	. _T('zxml_nonvide_balise')); // message a affiner
257	257	}
258	258	}
		@@ -296,7 +296,7 @@ discard block
		block discarded – undo
296	296	public function phraserTout($phraseur, $data) {
297	297	xml_parsestring($this, $data);
298	298
299		- if (!$this->dtc or preg_match(',^' . _MESSAGE_DOCTYPE . ',', $data)) {
	299	+ if (!$this->dtc or preg_match(',^'._MESSAGE_DOCTYPE.',', $data)) {
300	300	$this->err[] = ['DOCTYPE ?', 0, 0];
301	301	} else {
302	302	$this->valider_passe2();

		@@ -48,19 +48,19 @@ discard block
		block discarded – undo
48	48	* Nom d'une balise #TOTO
49	49	*
50	50	* Écriture alambiquée pour rester compatible avec les hexadecimaux des vieux squelettes */
51		-define('NOM_DE_CHAMP', '#((' . NOM_DE_BOUCLE . "):)?(([A-F][G-Z_][A-Z_0-9])\|[A-Z_]+)\b(\*{0,2})");
	51	+define('NOM_DE_CHAMP', '#(('.NOM_DE_BOUCLE."):)?(([A-F][G-Z_][A-Z_0-9])\|[A-Z_]+)\b(\*{0,2})");
52	52	/** Balise complète [...(#TOTO) ... ] */
53		-define('CHAMP_ETENDU', '/\[([^]\[])\(' . NOM_DE_CHAMP . '([^[)]\)[^]\[]*)\]/S');
	53	+define('CHAMP_ETENDU', '/\[([^]\[])\('.NOM_DE_CHAMP.'([^[)]\)[^]\[]*)\]/S');
54	54
55	55	define('BALISE_INCLURE', '/<INCLU[DR]E[[:space:]](\(([^)])\))?/S');
56	56	define('BALISE_POLYGLOTTE', ',<multi>(.*)</multi>,Uims');
57	57	define('BALISE_IDIOMES', ',<:(([a-z0-9_]+):)?([a-z0-9_])({([^\\|=>]=[^\\|>])})?((\\|[^>])?:/?>),iS');
58		-define('BALISE_IDIOMES_ARGS', '@^\s([^= ])\s=\s((' . NOM_DE_CHAMP . '[{][^}]})?[^,])\s,?\s@s');
	58	+define('BALISE_IDIOMES_ARGS', '@^\s([^= ])\s=\s(('.NOM_DE_CHAMP.'[{][^}]})?[^,])\s,?\s@s');
59	59
60	60	/** Champ sql dans parenthèse ex: (id_article) */
61	61	define('SQL_ARGS', '(\([^)]*\))');
62	62	/** Fonction SQL sur un champ ex: SUM(visites) */
63		-define('CHAMP_SQL_PLUS_FONC', '`?([A-Z_\/][A-Z_\/0-9.]*)' . SQL_ARGS . '?`?');
	63	+define('CHAMP_SQL_PLUS_FONC', '`?([A-Z_\/][A-Z_\/0-9.]*)'.SQL_ARGS.'?`?');
64	64
65	65	function phraser_inclure($texte, $ligne, $result) {
66	66
		@@ -215,7 +215,7 @@ discard block
		block discarded – undo
215	215	* @return array
216	216	**/
217	217	function phraser_champs($texte, $ligne, $result) {
218		- while (preg_match('/' . NOM_DE_CHAMP . '/S', $texte, $match)) {
	218	+ while (preg_match('/'.NOM_DE_CHAMP.'/S', $texte, $match)) {
219	219	$p = strpos($texte, (string) $match[0]);
220	220	// texte après la balise
221	221	$suite = substr($texte, $p + strlen($match[0]));
		@@ -363,7 +363,7 @@ discard block
		block discarded – undo
363	363	$collecte[] = $champ;
364	364	$args = ltrim($regs[count($regs) - 1]);
365	365	} else {
366		- if (!preg_match('/' . NOM_DE_CHAMP . '([{\|])/', $arg, $r)) {
	366	+ if (!preg_match('/'.NOM_DE_CHAMP.'([{\|])/', $arg, $r)) {
367	367	// 0 est un aveu d'impuissance. A completer
368	368	$arg = phraser_champs_exterieurs($arg, 0, $sep, $result);
369	369
		@@ -447,7 +447,7 @@ discard block
		block discarded – undo
447	447	function phraser_champs_exterieurs($texte, $ligne, $sep, $nested) {
448	448	$res = [];
449	449	while (($p = strpos($texte, (string) "%$sep")) !== false) {
450		- if (!preg_match(',^%' . preg_quote($sep) . '([0-9]+)@,', substr($texte, $p), $m)) {
	450	+ if (!preg_match(',^%'.preg_quote($sep).'([0-9]+)@,', substr($texte, $p), $m)) {
451	451	break;
452	452	}
453	453	$debut = substr($texte, 0, $p);
		@@ -487,7 +487,7 @@ discard block
		block discarded – undo
487	487	$pos_apres = 0;
488	488	$result = phraser_args($match[7], ')', $sep, $result, $champ, $pos_apres);
489	489	phraser_vieux($champ);
490		- $champ->avant = phraser_champs_exterieurs($match[1], $n, $sep, $result);
	490	+ $champ->avant = phraser_champs_exterieurs($match[1], $n, $sep, $result);
491	491	$debut = substr($match[7], $pos_apres + 1);
492	492	if (!empty($debut)) {
493	493	$n += substr_count(substr($texte, 0, strpos($texte, $debut)), "\n");
		@@ -618,7 +618,7 @@ discard block
		block discarded – undo
618	618	// une maniere tres sale de supprimer les "' autour de {critere "xxx","yyy"}
619	619	if (preg_match(',^(["\'])(.*)\1$,', $m[4])) {
620	620	$c = null;
621		- eval('$c = ' . $m[4] . ';');
	621	+ eval('$c = '.$m[4].';');
622	622	if (isset($c)) {
623	623	$m[4] = $c;
624	624	}
		@@ -698,7 +698,7 @@ discard block
		block discarded – undo
698	698	if (preg_match(',^ ([0-9-]+) (/) (.+) $,', $param, $m)) {
699	699	$crit = phraser_critere_infixe($m[1], $m[3], $v, '/', '', '');
700	700	} elseif (
701		- preg_match(',^([!]?)(' . CHAMP_SQL_PLUS_FONC .
	701	+ preg_match(',^([!]?)('.CHAMP_SQL_PLUS_FONC.
702	702	')[[:space:]](\??)(!?)(<=?\|>=?\|==?\|\b(?:IN\|LIKE)\b)(.)$,is', $param, $m)
703	703	) {
704	704	$a2 = trim($m[8]);
		@@ -715,8 +715,8 @@ discard block
		block discarded – undo
715	715	);
716	716	$crit->exclus = $m[1];
717	717	} elseif (
718		- preg_match('/^([!]?)\s*(' .
719		- CHAMP_SQL_PLUS_FONC .
	718	+ preg_match('/^([!]?)\s*('.
	719	+ CHAMP_SQL_PLUS_FONC.
720	720	')\s(\??)(.)$/is', $param, $m)
721	721	) {
722	722	// contient aussi les comparaisons implicites !
		@@ -861,7 +861,7 @@ discard block
		block discarded – undo
861	861	}
862	862
863	863	// trouver sa position de depart reelle : au <Bxx> ou au <BBxx>
864		- $precond_boucle = BALISE_PRECOND_BOUCLE . $id_boucle . '>';
	864	+ $precond_boucle = BALISE_PRECOND_BOUCLE.$id_boucle.'>';
865	865	$pos_precond = strpos($texte, $precond_boucle, $id_boucle ? $pos_debut_texte : $pos_derniere_boucle_anonyme);
866	866	if (
867	867	$pos_precond !== false
		@@ -872,7 +872,7 @@ discard block
		block discarded – undo
872	872	$boucle['pos_precond_inside'] = $pos_precond + strlen($precond_boucle);
873	873	}
874	874
875		- $preaff_boucle = BALISE_PREAFF_BOUCLE . $id_boucle . '>';
	875	+ $preaff_boucle = BALISE_PREAFF_BOUCLE.$id_boucle.'>';
876	876	$pos_preaff = strpos($texte, $preaff_boucle, $id_boucle ? $pos_debut_texte : $pos_derniere_boucle_anonyme);
877	877	if (
878	878	$pos_preaff !== false
		@@ -920,13 +920,13 @@ discard block
		block discarded – undo
920	920	$pos_anonyme_next = null;
921	921	// si c'est une boucle anonyme, chercher la position de la prochaine boucle anonyme
922	922	if (!strlen($id_boucle)) {
923		- $pos_anonyme_next = strpos($texte, BALISE_BOUCLE . '(', $pos_courante);
	923	+ $pos_anonyme_next = strpos($texte, BALISE_BOUCLE.'(', $pos_courante);
924	924	}
925	925
926	926	//
927	927	// 1. Recuperer la partie conditionnelle apres
928	928	//
929		- $apres_boucle = BALISE_POSTCOND_BOUCLE . $id_boucle . '>';
	929	+ $apres_boucle = BALISE_POSTCOND_BOUCLE.$id_boucle.'>';
930	930	$pos_apres = strpos($texte, $apres_boucle, $pos_courante);
931	931	if (
932	932	$pos_apres !== false
		@@ -935,13 +935,13 @@ discard block
		block discarded – undo
935	935	$boucle['pos_postcond'] = $pos_apres;
936	936	$pos_apres += strlen($apres_boucle);
937	937	$boucle['pos_postcond_inside'] = $pos_apres;
938		- $pos_courante = $pos_apres ;
	938	+ $pos_courante = $pos_apres;
939	939	}
940	940
941	941	//
942	942	// 2. Récuperer la partie alternative apres
943	943	//
944		- $altern_boucle = BALISE_ALT_BOUCLE . $id_boucle . '>';
	944	+ $altern_boucle = BALISE_ALT_BOUCLE.$id_boucle.'>';
945	945	$pos_altern = strpos($texte, $altern_boucle, $pos_courante);
946	946	if (
947	947	$pos_altern !== false
		@@ -956,7 +956,7 @@ discard block
		block discarded – undo
956	956	//
957	957	// 3. Recuperer la partie footer non alternative
958	958	//
959		- $postaff_boucle = BALISE_POSTAFF_BOUCLE . $id_boucle . '>';
	959	+ $postaff_boucle = BALISE_POSTAFF_BOUCLE.$id_boucle.'>';
960	960	$pos_postaff = strpos($texte, $postaff_boucle, $pos_courante);
961	961	if (
962	962	$pos_postaff !== false
		@@ -965,7 +965,7 @@ discard block
		block discarded – undo
965	965	$boucle['pos_postaff'] = $pos_postaff;
966	966	$pos_postaff += strlen($postaff_boucle);
967	967	$boucle['pos_postaff_inside'] = $pos_postaff;
968		- $pos_courante = $pos_postaff ;
	968	+ $pos_courante = $pos_postaff;
969	969	}
970	970
971	971	return $boucle;
		@@ -1005,7 +1005,7 @@ discard block
		block discarded – undo
1005	1005	* @return string
1006	1006	*/
1007	1007	function public_generer_boucle_placeholder($id_boucle, &$boucle, $boucle_placeholder, $nb_lignes) {
1008		- $placeholder = "[(#{$boucle_placeholder}{" . $id_boucle . '})' . str_pad('', $nb_lignes, "\n") . ']';
	1008	+ $placeholder = "[(#{$boucle_placeholder}{".$id_boucle.'})'.str_pad('', $nb_lignes, "\n").']';
1009	1009	//memoriser la boucle a reinjecter
1010	1010	$id_boucle = "$id_boucle";
1011	1011	phraser_boucle_placeholder($id_boucle, $boucle_placeholder, $boucle);
		@@ -1018,7 +1018,7 @@ discard block
		block discarded – undo
1018	1018	// definir un placholder pour les boucles dont on est sur d'avoir aucune occurence dans le squelette
1019	1019	if (is_null($boucle_placeholder)) {
1020	1020	do {
1021		- $boucle_placeholder = 'BOUCLE_PLACEHOLDER_' . strtoupper(md5(uniqid()));
	1021	+ $boucle_placeholder = 'BOUCLE_PLACEHOLDER_'.strtoupper(md5(uniqid()));
1022	1022	} while (strpos($texte, $boucle_placeholder) !== false);
1023	1023	}
1024	1024
		@@ -1038,7 +1038,7 @@ discard block
		block discarded – undo
1038	1038
1039	1039	// boucle anonyme ?
1040	1040	if (!strlen($id_boucle)) {
1041		- $id_boucle = '_anon_L' . $ligne_milieu . '_' . substr(md5('anonyme:' . $id_parent . ':' . json_encode($boucle, JSON_THROW_ON_ERROR)), 0, 8);
	1041	+ $id_boucle = '_anon_L'.$ligne_milieu.'_'.substr(md5('anonyme:'.$id_parent.':'.json_encode($boucle, JSON_THROW_ON_ERROR)), 0, 8);
1042	1042	}
1043	1043
1044	1044	$pos_debut_boucle = $pos_courante;
		@@ -1051,7 +1051,7 @@ discard block
		block discarded – undo
1051	1051
1052	1052	$pos_avant = $boucle['pos_precond_inside'];
1053	1053	$result->avant = substr($texte, $pos_avant, $pos_courante - $pos_avant);
1054		- $ligne_avant = $ligne_debut_texte + public_compte_ligne($texte, $pos_debut_texte, $pos_avant);
	1054	+ $ligne_avant = $ligne_debut_texte + public_compte_ligne($texte, $pos_debut_texte, $pos_avant);
1055	1055	}
1056	1056
1057	1057	// Regarder si on a une partie inconditionnelle avant <BB_xxx>
		@@ -1060,7 +1060,7 @@ discard block
		block discarded – undo
1060	1060
1061	1061	$pos_preaff = $boucle['pos_preaff_inside'];
1062	1062	$result->preaff = substr($texte, $pos_preaff, $end_preaff - $pos_preaff);
1063		- $ligne_preaff = $ligne_debut_texte + public_compte_ligne($texte, $pos_debut_texte, $pos_preaff);
	1063	+ $ligne_preaff = $ligne_debut_texte + public_compte_ligne($texte, $pos_debut_texte, $pos_preaff);
1064	1064	}
1065	1065
1066	1066	$result->id_boucle = $id_boucle;
		@@ -1117,7 +1117,7 @@ discard block
		block discarded – undo
1117	1117	} else {
1118	1118	$pos_milieu += 1;
1119	1119
1120		- $fin_boucle = BALISE_FIN_BOUCLE . $id_boucle_search . '>';
	1120	+ $fin_boucle = BALISE_FIN_BOUCLE.$id_boucle_search.'>';
1121	1121	$pos_fin = strpos($texte, $fin_boucle, $pos_milieu);
1122	1122	if ($pos_fin === false) {
1123	1123	$err_b = [
		@@ -1130,7 +1130,7 @@ discard block
		block discarded – undo
1130	1130	else {
1131	1131	// verifier une eventuelle imbrication d'une boucle homonyme
1132	1132	// (interdite, generera une erreur plus loin, mais permet de signaler la bonne erreur)
1133		- $search_debut_boucle = BALISE_BOUCLE . $id_boucle_search . '(';
	1133	+ $search_debut_boucle = BALISE_BOUCLE.$id_boucle_search.'(';
1134	1134	$search_from = $pos_milieu;
1135	1135	$nb_open = 1;
1136	1136	$nb_close = 1;
		@@ -1170,7 +1170,7 @@ discard block
		block discarded – undo
1170	1170	if ($boucle['pos_postcond']) {
1171	1171	$result->apres = substr($texte, $pos_courante, $boucle['pos_postcond'] - $pos_courante);
1172	1172	$ligne_suite += public_compte_ligne($texte, $pos_courante, $boucle['pos_postcond_inside']);
1173		- $pos_courante = $boucle['pos_postcond_inside'] ;
	1173	+ $pos_courante = $boucle['pos_postcond_inside'];
1174	1174	}
1175	1175
1176	1176

		@@ -39,8 +39,8 @@ discard block
		block discarded – undo
39	39	if ($fail) {
40	40	echo info_etape(_T('info_chemin_acces_annuaire')),
41	41	info_progression_etape(3, 'etape_ldap', 'install/', true),
42		- "<div class='error'><p><b>" . _T('avis_operation_echec') . '</b></p><p>' . _T('avis_chemin_invalide_1'),
43		- ' (<tt>' . spip_htmlspecialchars($base_ldap) . '</tt>) ' . _T('avis_chemin_invalide_2') . '</p></div>';
	42	+ "<div class='error'><p><b>"._T('avis_operation_echec').'</b></p><p>'._T('avis_chemin_invalide_1'),
	43	+ ' (<tt>'.spip_htmlspecialchars($base_ldap).'</tt>) '._T('avis_chemin_invalide_2').'</p></div>';
44	44	} else {
45	45	info_etape(_T('info_reglage_ldap'));
46	46	echo info_progression_etape(4, 'etape_ldap', 'install/');
		@@ -53,12 +53,12 @@ discard block
		block discarded – undo
53	53
54	54	$res = install_propager(['adresse_ldap', 'port_ldap', 'login_ldap', 'pass_ldap', 'protocole_ldap', 'tls_ldap'])
55	55	. "<input type='hidden' name='etape' value='ldap5' />"
56		- . "<input type='hidden' name='base_ldap' value='" . spip_htmlentities($base_ldap) . "' />"
	56	+ . "<input type='hidden' name='base_ldap' value='".spip_htmlentities($base_ldap)."' />"
57	57	. fieldset(
58	58	_T('info_statut_utilisateurs_1'),
59	59	[
60	60	'statut_ldap' => [
61		- 'label' => _T('info_statut_utilisateurs_2') . '<br />',
	61	+ 'label' => _T('info_statut_utilisateurs_2').'<br />',
62	62	'valeur' => $statut_ldap,
63	63	'alternatives' => $statuts
64	64	]
		@@ -75,9 +75,9 @@ discard block
		block discarded – undo
75	75
76	76	function liste_statuts_ldap() {
77	77	$recom = [
78		- 'info_administrateurs' => ('<b>' . _T('info_administrateur_1') . '</b> ' . _T('info_administrateur_2') . '<br />'),
79		- 'info_redacteurs' => ('<b>' . _T('info_redacteur_1') . '</b> ' . _T('info_redacteur_2') . '<br />'),
80		- 'info_visiteurs' => ('<b>' . _T('info_visiteur_1') . '</b> ' . _T('info_visiteur_2') . '<br />')
	78	+ 'info_administrateurs' => ('<b>'._T('info_administrateur_1').'</b> '._T('info_administrateur_2').'<br />'),
	79	+ 'info_redacteurs' => ('<b>'._T('info_redacteur_1').'</b> '._T('info_redacteur_2').'<br />'),
	80	+ 'info_visiteurs' => ('<b>'._T('info_visiteur_1').'</b> '._T('info_visiteur_2').'<br />')
81	81	];
82	82
83	83	$res = [];
		@@ -93,14 +93,14 @@ discard block
		block discarded – undo
93	93	function install_ldap_correspondances() {
94	94	$champs = [];
95	95	foreach (is_array($GLOBALS['ldap_attributes']) ? $GLOBALS['ldap_attributes'] : [] as $champ => $v) {
96		- $nom = 'ldap_' . $champ;
	96	+ $nom = 'ldap_'.$champ;
97	97	$val = is_array($v) ? join(',', $v) : strval($v);
98	98	$champs[$nom] = [
99		- 'label' => _T('ldap_correspondance', ['champ' => "<tt>$champ</tt>"]) . '<br />',
	99	+ 'label' => _T('ldap_correspondance', ['champ' => "<tt>$champ</tt>"]).'<br />',
100	100	'valeur' => $val
101	101	];
102	102	}
103	103
104	104	return !$champs ?
105		- '' : fieldset(_T('ldap_correspondance_1'), $champs, '', _T('ldap_correspondance_2') . '<br /><br />');
	105	+ '' : fieldset(_T('ldap_correspondance_1'), $champs, '', _T('ldap_correspondance_2').'<br /><br />');
106	106	}

		@@ -101,8 +101,8 @@ discard block
		block discarded – undo
101	101	}
102	102	foreach ($debug['sourcefile'] as $k => $v) {
103	103	if (strpos($v, 'administration.') !== false) {
104		- if (isset($debug['resultat'][$k . 'tout'])) {
105		- return $debug['resultat'][$k . 'tout'];
	104	+ if (isset($debug['resultat'][$k.'tout'])) {
	105	+ return $debug['resultat'][$k.'tout'];
106	106	}
107	107	}
108	108	}
		@@ -180,12 +180,12 @@ discard block
		block discarded – undo
180	180	and $id = intval($id)
181	181	and $desc = $trouver_table(table_objet_sql($type))
182	182	) {
183		- $id = sql_getfetsel($_id_type, table_objet_sql($type), "$_id_type=" . intval($id));
	183	+ $id = sql_getfetsel($_id_type, table_objet_sql($type), "$_id_type=".intval($id));
184	184	if ($id) {
185	185	$env[$_id_type] = $id;
186	186	$env['objet'] = $type;
187	187	$env['id_objet'] = $id;
188		- $env['voir_' . $obj] =
	188	+ $env['voir_'.$obj] =
189	189	str_replace('&', '&', generer_objet_url($id, $obj, '', '', false));
190	190	if (
191	191	isset($desc['field']['id_rubrique'])
		@@ -239,10 +239,10 @@ discard block
		block discarded – undo
239	239	$notpub = sql_in('statut', ['prop', 'prive']);
240	240
241	241	if ($type == 'article' and $GLOBALS['meta']['post_dates'] != 'oui') {
242		- $notpub .= " OR (statut='publie' AND date>" . sql_quote(date('Y-m-d H:i:s')) . ')';
	242	+ $notpub .= " OR (statut='publie' AND date>".sql_quote(date('Y-m-d H:i:s')).')';
243	243	}
244	244
245		- return sql_fetsel('1', table_objet_sql($type), id_table_objet($type) . '=' . $id . " AND ($notpub)");
	245	+ return sql_fetsel('1', table_objet_sql($type), id_table_objet($type).'='.$id." AND ($notpub)");
246	246	}
247	247
248	248
		@@ -256,9 +256,9 @@ discard block
		block discarded – undo
256	256	$alang = '';
257	257	if (!empty($_COOKIE['spip_admin'])) {
258	258	$email_or_login = preg_replace(',^@,', '', $_COOKIE['spip_admin']);
259		- $alang = sql_getfetsel('lang', 'spip_auteurs', 'email=' . sql_quote($email_or_login));
	259	+ $alang = sql_getfetsel('lang', 'spip_auteurs', 'email='.sql_quote($email_or_login));
260	260	if (!$alang) {
261		- $alang = sql_getfetsel('lang', 'spip_auteurs', 'login=' . sql_quote($email_or_login));
	261	+ $alang = sql_getfetsel('lang', 'spip_auteurs', 'login='.sql_quote($email_or_login));
262	262	}
263	263	}
264	264	if (!$alang) {
		@@ -283,9 +283,8 @@ discard block
		block discarded – undo
283	283
284	284	return ((!isset($GLOBALS['xhtml']) or $GLOBALS['xhtml'] !== 'true') ?
285	285	(parametre_url(self(), 'var_mode', 'debug', '&')
286		- . '&var_mode_affiche=validation') :
287		- ('http://validator.w3.org/check?uri='
288		- . rawurlencode('http://' . $_SERVER['HTTP_HOST'] . nettoyer_uri())));
	286	+ . '&var_mode_affiche=validation') : ('http://validator.w3.org/check?uri='
	287	+ . rawurlencode('http://'.$_SERVER['HTTP_HOST'].nettoyer_uri())));
289	288	}
290	289
291	290	/**

spip / SPIP

Push — master ( e003b8...5b64d4 )

Status

Category

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

Spacing +25 added lines, -25 removed lines patch added patch discarded remove patch

Spacing +10 added lines, -12 removed lines patch added patch discarded remove patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

Spacing +108 added lines, -117 removed lines patch added patch discarded remove patch

Spacing +28 added lines, -28 removed lines patch added patch discarded remove patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

Spacing +10 added lines, -11 removed lines patch added patch discarded remove patch