Inspection of "Merge pull request 'phpcs pour spip5' (#5534) from..." - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ad4f09...6d727b )

by cam

created 2023-03-22 21:39 UTC

Status

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

@@ -48,15 +48,15 @@  discard block
 block discarded – undo
 		// Correction Create Database
 		// Create Database -> requete ignoree
 		if (str_starts_with((string) $this->query, 'CREATE DATABASE')) {
-			spip_log("Sqlite : requete non executee -> $this->query", 'sqlite.' . _LOG_AVERTISSEMENT);
+			spip_log("Sqlite : requete non executee -> $this->query", 'sqlite.'._LOG_AVERTISSEMENT);
 			$this->query = 'SELECT 1';
 		}
 
 		// Correction Insert Ignore
 		// INSERT IGNORE -> insert (tout court et pas 'insert or replace')
 		if (str_starts_with((string) $this->query, 'INSERT IGNORE')) {
-			spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.' . _LOG_DEBUG);
-			$this->query = 'INSERT ' . substr((string) $this->query, '13');
+			spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.'._LOG_DEBUG);
+			$this->query = 'INSERT '.substr((string) $this->query, '13');
 		}
 
 		// Correction des dates avec INTERVAL
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
 		if (($this->sqlite_version == 2) && (str_contains((string) $this->query, 'USING'))) {
 			spip_log(
 				"'USING (champ)' n'est pas reconnu en SQLite 2. Utilisez 'ON table1.champ = table2.champ'",
-				'sqlite.' . _LOG_ERREUR
+				'sqlite.'._LOG_ERREUR
 			);
 			$this->query = preg_replace('/USING\s*\([^\)]*\)/', '', (string) $this->query);
 		}
@@ -107,8 +107,8 @@  discard block
 block discarded – undo
 		} else {
 			$suite = '';
 		}
-		$pref = ($this->prefixe) ? $this->prefixe . '_' : '';
-		$this->query = preg_replace('/([,\s])spip_/S', '\1' . $pref, (string) $this->query) . $suite;
+		$pref = ($this->prefixe) ? $this->prefixe.'_' : '';
+		$this->query = preg_replace('/([,\s])spip_/S', '\1'.$pref, (string) $this->query).$suite;
 
 		// Correction zero AS x
 		// pg n'aime pas 0+x AS alias, sqlite, dans le meme style,

Please login to merge, or discard this patch.

ecrire/src/Texte/Collecteur/AbstractCollecteur.php 1 patch

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

@@ -109,13 +109,13 @@
 block discarded – undo
 				// generer un marqueur qui n'existe pas dans le texte
 				do {
 					$this->markId = substr(md5(uniqid(static::class, 1)), 0, 7);
-					$this->markId = '@|' . static::$markPrefix . $this->markId . '|';
+					$this->markId = '@|'.static::$markPrefix.$this->markId.'|';
 				} while (str_contains($texte, $this->markId));
 			}
 
 			$offset_pos = 0;
 			foreach ($collection as $c) {
-				$rempl = $this->markId . base64_encode((string) $c['raw']) . '|@';
+				$rempl = $this->markId.base64_encode((string) $c['raw']).'|@';
 				$texte = substr_replace($texte, $rempl, $c['pos'] + $offset_pos, $c['length']);
 				$offset_pos += strlen($rempl) - $c['length'];
 			}

Please login to merge, or discard this patch.

ecrire/src/Texte/Collecteur/Multis.php 1 patch

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

@@ -205,9 +205,9 @@
 block discarded – undo
 								// il ne faut pas echapper en div si propre produit un seul paragraphe
 								include_spip('inc/texte');
 								$trad_propre = preg_replace(',(^<p[^>]*>|</p>$),Uims', '', (string) propre($trad));
-								$mode = preg_match(',</?(' . _BALISES_BLOCS . ')[>[:space:]],iS', $trad_propre) ? 'div' : 'span';
+								$mode = preg_match(',</?('._BALISES_BLOCS.')[>[:space:]],iS', $trad_propre) ? 'div' : 'span';
 								if ($mode === 'div') {
-									$trad = rtrim((string) $trad) . "\n\n";
+									$trad = rtrim((string) $trad)."\n\n";
 								}
 								$trad = code_echappement($trad, 'multi', false, $mode);
 								$trad = str_replace("'", '"', (string) inserer_attribut($trad, 'lang', $l));

Please login to merge, or discard this patch.

ecrire/src/Afficher/Minipage/AbstractPage.php 1 patch

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

@@ -65,17 +65,17 @@  discard block
 block discarded – undo
 
 		$page_title = ($options['page_title'] ?? $GLOBALS['meta']['nom_site']);
 		$doctype = ($options['doctype'] ?? '<!DOCTYPE html>');
-		$doctype = trim((string) $doctype) . "\n";
+		$doctype = trim((string) $doctype)."\n";
 		$charset = ($options['charset'] ?? 'utf-8');
 		$all_inline = ($options['all_inline'] ?? true);
 		$onLoad = ($options['onLoad'] ?? '');
 		if ($onLoad) {
-			$onLoad = ' onload="' . attribut_html($onLoad) . '"';
+			$onLoad = ' onload="'.attribut_html($onLoad).'"';
 		}
 
 		# envoyer le charset
 		if (!headers_sent()) {
-			header('Content-Type: text/html; charset=' . $charset);
+			header('Content-Type: text/html; charset='.$charset);
 		}
 
 		$css = '';
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
 			. "--minipage-color-theme--s: $s;"
 			. "--minipage-color-theme--l: $l;}";
 		$vars = file_get_contents(find_in_theme('minipage.vars.css'));
-		$inline .= "\n" . trim($vars);
+		$inline .= "\n".trim($vars);
 		if (function_exists('minifier')) {
 			$inline = minifier($inline, 'css');
 		}
@@ -121,31 +121,31 @@  discard block
 block discarded – undo
 			}
 			$css = "$inline\n$css";
 			if (!empty($options['css'])) {
-				$css .= "\n" . $options['css'];
+				$css .= "\n".$options['css'];
 			}
 			$css = "<style type='text/css'>$css</style>";
 		} else {
 			$css = "<style type='text/css'>$inline</style>";
 			foreach ($files as $name) {
 				$file = timestamp(direction_css($name));
-				$css .= "<link rel='stylesheet' href='" . attribut_html($file) . "' type='text/css' />\n";
+				$css .= "<link rel='stylesheet' href='".attribut_html($file)."' type='text/css' />\n";
 			}
 			if (!empty($options['css'])) {
-				$css .= "<style type='text/css'>" . $options['css'] . '</style>';
+				$css .= "<style type='text/css'>".$options['css'].'</style>';
 			}
 		}
 
-		return $doctype .
-			html_lang_attributes() .
-			"<head>\n" .
-			'<title>' .
-			textebrut($page_title) .
-			"</title>\n" .
-			"<meta name=\"viewport\" content=\"width=device-width\" />\n" .
-			$css .
-			(empty($options['head']) ? '' : $options['head']) .
-			"</head>\n" .
-			"<body{$onLoad} class=\"minipage" . ($this::TYPE ? ' minipage--' . $this::TYPE : '') . "\">\n" .
+		return $doctype.
+			html_lang_attributes().
+			"<head>\n".
+			'<title>'.
+			textebrut($page_title).
+			"</title>\n".
+			"<meta name=\"viewport\" content=\"width=device-width\" />\n".
+			$css.
+			(empty($options['head']) ? '' : $options['head']).
+			"</head>\n".
+			"<body{$onLoad} class=\"minipage".($this::TYPE ? ' minipage--'.$this::TYPE : '')."\">\n".
 			"\t<div class=\"minipage-bloc\">\n";
 	}
 
@@ -156,16 +156,16 @@  discard block
 block discarded – undo
 	 */
 	protected function ouvreCorps($options = []) {
 		$url_site = url_de_base();
-		$header = "<header>\n" .
-			'<h1><a href="' . attribut_html($url_site) . '">' . interdire_scripts($GLOBALS['meta']['nom_site'] ?? '') . "</a></h1>\n";
+		$header = "<header>\n".
+			'<h1><a href="'.attribut_html($url_site).'">'.interdire_scripts($GLOBALS['meta']['nom_site'] ?? '')."</a></h1>\n";
 
 		$titre = ($options['titre'] ?? '');
 		if ($titre) {
-			$header .= '<h2>' . interdire_scripts($titre) . '</h2>';
+			$header .= '<h2>'.interdire_scripts($titre).'</h2>';
 		}
 		$header .= '</header>';
 
-		return $header . "<div class='corps'>\n";
+		return $header."<div class='corps'>\n";
 	}
 
 	/**
@@ -179,13 +179,13 @@  discard block
 block discarded – undo
 		if (isset($options['footer'])) {
 			$footer = $options['footer'];
 		} else {
-			$footer = '<a href="' . attribut_html($url_site) . '">' . _T('retour') . "</a>\n";
+			$footer = '<a href="'.attribut_html($url_site).'">'._T('retour')."</a>\n";
 		}
 		if (!empty($footer)) {
 			$footer = "<footer>\n{$footer}</footer>";
 		}
 
-		return "</div>\n" . $footer;
+		return "</div>\n".$footer;
 	}
 
 

Please login to merge, or discard this patch.

ecrire/src/Compilateur/Iterateur/Decorator.php 1 patch

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

@@ -135,7 +135,7 @@  discard block
 block discarded – undo
 				return $this->iter->{$nom}();
 			} catch (Exception) {
 				// #GETCHILDREN sur un fichier de DirectoryIterator ...
-				spip_log("Methode {$nom} en echec sur " . $this->iter::class);
+				spip_log("Methode {$nom} en echec sur ".$this->iter::class);
 				spip_log("Cela peut être normal : retour d'une ligne de resultat ne pouvant pas calculer cette methode");
 
 				return '';
@@ -356,7 +356,7 @@  discard block
 block discarded – undo
 			return null;
 		}
 
-		return '(' . implode(") {$operateur} (", $filtres_string) . ')';
+		return '('.implode(") {$operateur} (", $filtres_string).')';
 	}
 
 	/**
@@ -439,7 +439,7 @@  discard block
 block discarded – undo
 			return $this->composer_filtre($v[1], $v[0], $v[2]);
 		}
 
-		return null;  // sera ignore
+		return null; // sera ignore
 	}
 
 	/**
@@ -468,28 +468,28 @@  discard block
 block discarded – undo
 		// if (!in_array($cle, array('cle', 'valeur')))
 		//	return;
 
-		$a = '$this->get_select(\'' . $cle . '\')';
+		$a = '$this->get_select(\''.$cle.'\')';
 
 		$filtre = '';
 
 		if ('REGEXP' == $op) {
-			$filtre = 'filtrer("match", ' . $a . ', ' . str_replace('\"', '"', (string) $valeur) . ')';
+			$filtre = 'filtrer("match", '.$a.', '.str_replace('\"', '"', (string) $valeur).')';
 			$op = '';
 		} else {
 			if ('LIKE' == $op) {
 				$valeur = str_replace(['\"', '_', '%'], ['"', '.', '.*'], preg_quote((string) $valeur));
-				$filtre = 'filtrer("match", ' . $a . ', ' . $valeur . ')';
+				$filtre = 'filtrer("match", '.$a.', '.$valeur.')';
 				$op = '';
 			} else {
 				if ('=' == $op) {
 					$op = '==';
 				} else {
 					if ('IN' == $op) {
-						$filtre = 'in_array(' . $a . ', array' . $valeur . ')';
+						$filtre = 'in_array('.$a.', array'.$valeur.')';
 						$op = '';
 					} else {
 						if (!in_array($op, ['<', '<=', '>', '>='])) {
-							spip_log('operateur non reconnu ' . $op); // [todo] mettre une erreur de squelette
+							spip_log('operateur non reconnu '.$op); // [todo] mettre une erreur de squelette
 							$op = '';
 						}
 					}
@@ -498,7 +498,7 @@  discard block
 block discarded – undo
 		}
 
 		if ($op) {
-			$filtre = $a . $op . str_replace('\"', '"', (string) $valeur);
+			$filtre = $a.$op.str_replace('\"', '"', (string) $valeur);
 		}
 
 		if ($not) {
@@ -547,7 +547,7 @@  discard block
 block discarded – undo
 		// Creer la fonction de filtrage sur $this
 		if ($this->filtre) {
 			if ($filtres = $this->assembler_filtres($this->filtre)) {
-				$filtres = 'return ' . $filtres . ';';
+				$filtres = 'return '.$filtres.';';
 				$this->func_filtre = fn () => eval($filtres);
 			} else {
 				$this->func_filtre = null;

Please login to merge, or discard this patch.

ecrire/src/Compilateur/Iterateur/Factory.php 1 patch

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

@@ -51,12 +51,12 @@
 block discarded – undo
 			// chercher la classe d'iterateur Iterateur/XXX
 			// definie dans le fichier src/Compilateur/Iterateur/xxx.php
 			// FIXME: déclarer quelque part les iterateurs supplémentaires
-			$class = __NAMESPACE__ . '\\' . ucfirst(strtolower((string) $iterateur));
+			$class = __NAMESPACE__.'\\'.ucfirst(strtolower((string) $iterateur));
 			if (!class_exists($class)) {
 				// historique
 				// Chercher IterateurXXX
-				include_spip('iterateur/' . $iterateur);
-				$class = 'Iterateur' . $iterateur;
+				include_spip('iterateur/'.$iterateur);
+				$class = 'Iterateur'.$iterateur;
 				if (!class_exists($class)) {
 					exit("Iterateur {$iterateur} non trouv&#233;");
 					// si l'iterateur n'existe pas, on se rabat sur le generique

Please login to merge, or discard this patch.

ecrire/inc/editer.php 1 patch

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

@@ -99,7 +99,7 @@  discard block
 block discarded – undo
 		$res['message_ok'] = _T('info_modification_enregistree');
 		if ($retour) {
 			if (str_starts_with($retour, 'javascript:')) {
-				$res['message_ok'] .= '<script type="text/javascript">/*<![CDATA[*/' . substr($retour, 11) . '/*]]>*/</script>';
+				$res['message_ok'] .= '<script type="text/javascript">/*<![CDATA[*/'.substr($retour, 11).'/*]]>*/</script>';
 				$res['editable'] = true;
 			} else {
 				$res['redirect'] = parametre_url($retour, $id_table_objet, $id);
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
 				if (!isset($erreurs[$champ])) {
 					$erreurs[$champ] = '';
 				}
-				$erreurs[$champ] .= _T('alerte_modif_info_concourante') . "<br /><textarea readonly='readonly' class='forml'>" . entites_html($conflit['base']) . '</textarea>';
+				$erreurs[$champ] .= _T('alerte_modif_info_concourante')."<br /><textarea readonly='readonly' class='forml'>".entites_html($conflit['base']).'</textarea>';
 			}
 		}
 	}
@@ -219,7 +219,7 @@  discard block
 block discarded – undo
 	if (
 		$config_fonc
 		&& !in_array($config_fonc, ['articles_edit_config', 'rubriques_edit_config', 'auteurs_edit_config'])
-		&& $config_fonc !== $table_objet . '_edit_config'
+		&& $config_fonc !== $table_objet.'_edit_config'
 		&& ($args = test_formulaire_inclus_par_modele())
 		&& in_array($config_fonc, $args)
 	) {
@@ -231,12 +231,12 @@  discard block
 block discarded – undo
 	// Appel direct dans un squelette
 	if (!$row) {
 		if (!$new || $lier_trad) {
-			if ($select = charger_fonction('precharger_' . $type, 'inc', true)) {
+			if ($select = charger_fonction('precharger_'.$type, 'inc', true)) {
 				$row = $select($id, $id_parent, $lier_trad);
 				// si on a une fonction precharger, elle pu faire un reglage de langue
 				$lang_default = (empty($row['lang']) ? null : $row['lang']);
 			} else {
-				$row = sql_fetsel('*', $table_objet_sql, $id_table_objet . '=' . (int) $id);
+				$row = sql_fetsel('*', $table_objet_sql, $id_table_objet.'='.(int) $id);
 			}
 			if (!$new) {
 				$md5 = controles_md5($row ?: []);
@@ -286,7 +286,7 @@  discard block
 block discarded – undo
 	if ($config_fonc) {
 		$contexte['config'] = $config = $config_fonc($contexte);
 		if (!$lang_default) {
-			$lang_default = $config['langue'] ?? session_get('lang') ;
+			$lang_default = $config['langue'] ?? session_get('lang');
 		}
 	}
 	$config += [
@@ -308,13 +308,13 @@  discard block
 block discarded – undo
 		unset($contexte['lang']);
 	}
 
-	$contexte['_hidden'] = "<input type='hidden' name='editer_$type' value='oui' />\n" .
+	$contexte['_hidden'] = "<input type='hidden' name='editer_$type' value='oui' />\n".
 		($lier_trad
-			 ? "\n<input type='hidden' name='lier_trad' value='" .
-				$lier_trad .
-				"' />" .
-				"\n<input type='hidden' name='changer_lang' value='" .
-				$lang_default .
+			 ? "\n<input type='hidden' name='lier_trad' value='".
+				$lier_trad.
+				"' />".
+				"\n<input type='hidden' name='changer_lang' value='".
+				$lang_default.
 				"' />"
 			: '')
 		. $hidden
@@ -355,14 +355,14 @@  discard block
 block discarded – undo
 	$aider = charger_fonction('aider', 'inc');
 	if (strlen($texte) > 28 * 1024) {
 		$texte = str_replace("\r\n", "\n", $texte);
-		$pos = strpos($texte, "\n\n", 28 * 1024);  // coupe para > 28 ko
+		$pos = strpos($texte, "\n\n", 28 * 1024); // coupe para > 28 ko
 		if ($pos > 0 && $pos < 32 * 1024) {
-			$debut = substr($texte, 0, $pos) . "\n\n<!--SPIP-->\n";
+			$debut = substr($texte, 0, $pos)."\n\n<!--SPIP-->\n";
 			$suite = substr($texte, $pos + 2);
 		} else {
-			$pos = strpos($texte, ' ', 28 * 1024);  // sinon coupe espace
+			$pos = strpos($texte, ' ', 28 * 1024); // sinon coupe espace
 			if (!($pos > 0 && $pos < 32 * 1024)) {
-				$pos = 28 * 1024;  // au pire (pas d'espace trouv'e)
+				$pos = 28 * 1024; // au pire (pas d'espace trouv'e)
 				$decalage = 0; // si y'a pas d'espace, il ne faut pas perdre le caract`ere
 			} else {
 				$decalage = 1;
@@ -449,7 +449,7 @@  discard block
 block discarded – undo
 	$ctr = [];
 	foreach ($data as $key => $val) {
 		$m = md5($val ?? '');
-		$k = $prefixe . $key;
+		$k = $prefixe.$key;
 
 		$ctr[$k] = match ($format) {
 			'html' => "<input type='hidden' value='$m' name='$k' />",
@@ -458,7 +458,7 @@  discard block
 block discarded – undo
 	}
 
 	if ($format === 'html') {
-		return "\n\n<!-- controles md5 -->\n" . implode("\n", $ctr) . "\n\n";
+		return "\n\n<!-- controles md5 -->\n".implode("\n", $ctr)."\n\n";
 	} else {
 		return $ctr;
 	}
@@ -611,8 +611,8 @@  discard block
 block discarded – undo
 	// potentiellement modifiees entre temps par un autre utilisateur)
 	foreach ($champs as $key => $val) {
 		if (
-			isset($ctr[$prefix . $key])
-			&& ($m = $ctr[$prefix . $key])
+			isset($ctr[$prefix.$key])
+			&& ($m = $ctr[$prefix.$key])
 			&& (is_scalar($val) && $m == md5($val))
 		) {
 			unset($champs[$key]);
@@ -641,7 +641,7 @@  discard block
 block discarded – undo
 	// de conflit.
 	$ctrh = $ctrq = $conflits = [];
 	foreach (array_keys($champs) as $key) {
-		if (isset($ctr[$prefix . $key]) && ($m = $ctr[$prefix . $key])) {
+		if (isset($ctr[$prefix.$key]) && ($m = $ctr[$prefix.$key])) {
 			$ctrh[$key] = $m;
 			$ctrq[] = $key;
 		}
@@ -675,8 +675,8 @@  discard block
 block discarded – undo
  */
 function display_conflit_champ($x) {
 	if (strstr($x, "\n") || strlen($x) > 80) {
-		return "<textarea style='width:99%; height:10em;'>" . entites_html($x) . "</textarea>\n";
+		return "<textarea style='width:99%; height:10em;'>".entites_html($x)."</textarea>\n";
 	} else {
-		return "<input type='text' size='40' style='width:99%' value=\"" . entites_html($x) . "\" />\n";
+		return "<input type='text' size='40' style='width:99%' value=\"".entites_html($x)."\" />\n";
 	}
 }

Please login to merge, or discard this patch.

		@@ -109,13 +109,13 @@
		block discarded – undo
109	109	// generer un marqueur qui n'existe pas dans le texte
110	110	do {
111	111	$this->markId = substr(md5(uniqid(static::class, 1)), 0, 7);
112		- $this->markId = '@\|' . static::$markPrefix . $this->markId . '\|';
	112	+ $this->markId = '@\|'.static::$markPrefix.$this->markId.'\|';
113	113	} while (str_contains($texte, $this->markId));
114	114	}
115	115
116	116	$offset_pos = 0;
117	117	foreach ($collection as $c) {
118		- $rempl = $this->markId . base64_encode((string) $c['raw']) . '\|@';
	118	+ $rempl = $this->markId.base64_encode((string) $c['raw']).'\|@';
119	119	$texte = substr_replace($texte, $rempl, $c['pos'] + $offset_pos, $c['length']);
120	120	$offset_pos += strlen($rempl) - $c['length'];
121	121	}

		@@ -205,9 +205,9 @@
		block discarded – undo
205	205	// il ne faut pas echapper en div si propre produit un seul paragraphe
206	206	include_spip('inc/texte');
207	207	$trad_propre = preg_replace(',(^<p[^>]*>\|</p>$),Uims', '', (string) propre($trad));
208		- $mode = preg_match(',</?(' . _BALISES_BLOCS . ')[>[:space:]],iS', $trad_propre) ? 'div' : 'span';
	208	+ $mode = preg_match(',</?('._BALISES_BLOCS.')[>[:space:]],iS', $trad_propre) ? 'div' : 'span';
209	209	if ($mode === 'div') {
210		- $trad = rtrim((string) $trad) . "\n\n";
	210	+ $trad = rtrim((string) $trad)."\n\n";
211	211	}
212	212	$trad = code_echappement($trad, 'multi', false, $mode);
213	213	$trad = str_replace("'", '"', (string) inserer_attribut($trad, 'lang', $l));

		@@ -65,17 +65,17 @@ discard block
		block discarded – undo
65	65
66	66	$page_title = ($options['page_title'] ?? $GLOBALS['meta']['nom_site']);
67	67	$doctype = ($options['doctype'] ?? '<!DOCTYPE html>');
68		- $doctype = trim((string) $doctype) . "\n";
	68	+ $doctype = trim((string) $doctype)."\n";
69	69	$charset = ($options['charset'] ?? 'utf-8');
70	70	$all_inline = ($options['all_inline'] ?? true);
71	71	$onLoad = ($options['onLoad'] ?? '');
72	72	if ($onLoad) {
73		- $onLoad = ' onload="' . attribut_html($onLoad) . '"';
	73	+ $onLoad = ' onload="'.attribut_html($onLoad).'"';
74	74	}
75	75
76	76	# envoyer le charset
77	77	if (!headers_sent()) {
78		- header('Content-Type: text/html; charset=' . $charset);
	78	+ header('Content-Type: text/html; charset='.$charset);
79	79	}
80	80
81	81	$css = '';
		@@ -94,7 +94,7 @@ discard block
		block discarded – undo
94	94	. "--minipage-color-theme--s: $s;"
95	95	. "--minipage-color-theme--l: $l;}";
96	96	$vars = file_get_contents(find_in_theme('minipage.vars.css'));
97		- $inline .= "\n" . trim($vars);
	97	+ $inline .= "\n".trim($vars);
98	98	if (function_exists('minifier')) {
99	99	$inline = minifier($inline, 'css');
100	100	}
		@@ -121,31 +121,31 @@ discard block
		block discarded – undo
121	121	}
122	122	$css = "$inline\n$css";
123	123	if (!empty($options['css'])) {
124		- $css .= "\n" . $options['css'];
	124	+ $css .= "\n".$options['css'];
125	125	}
126	126	$css = "<style type='text/css'>$css</style>";
127	127	} else {
128	128	$css = "<style type='text/css'>$inline</style>";
129	129	foreach ($files as $name) {
130	130	$file = timestamp(direction_css($name));
131		- $css .= "<link rel='stylesheet' href='" . attribut_html($file) . "' type='text/css' />\n";
	131	+ $css .= "<link rel='stylesheet' href='".attribut_html($file)."' type='text/css' />\n";
132	132	}
133	133	if (!empty($options['css'])) {
134		- $css .= "<style type='text/css'>" . $options['css'] . '</style>';
	134	+ $css .= "<style type='text/css'>".$options['css'].'</style>';
135	135	}
136	136	}
137	137
138		- return $doctype .
139		- html_lang_attributes() .
140		- "<head>\n" .
141		- '<title>' .
142		- textebrut($page_title) .
143		- "</title>\n" .
144		- "<meta name=\"viewport\" content=\"width=device-width\" />\n" .
145		- $css .
146		- (empty($options['head']) ? '' : $options['head']) .
147		- "</head>\n" .
148		- "<body{$onLoad} class=\"minipage" . ($this::TYPE ? ' minipage--' . $this::TYPE : '') . "\">\n" .
	138	+ return $doctype.
	139	+ html_lang_attributes().
	140	+ "<head>\n".
	141	+ '<title>'.
	142	+ textebrut($page_title).
	143	+ "</title>\n".
	144	+ "<meta name=\"viewport\" content=\"width=device-width\" />\n".
	145	+ $css.
	146	+ (empty($options['head']) ? '' : $options['head']).
	147	+ "</head>\n".
	148	+ "<body{$onLoad} class=\"minipage".($this::TYPE ? ' minipage--'.$this::TYPE : '')."\">\n".
149	149	"\t<div class=\"minipage-bloc\">\n";
150	150	}
151	151
		@@ -156,16 +156,16 @@ discard block
		block discarded – undo
156	156	*/
157	157	protected function ouvreCorps($options = []) {
158	158	$url_site = url_de_base();
159		- $header = "<header>\n" .
160		- '<h1><a href="' . attribut_html($url_site) . '">' . interdire_scripts($GLOBALS['meta']['nom_site'] ?? '') . "</a></h1>\n";
	159	+ $header = "<header>\n".
	160	+ '<h1><a href="'.attribut_html($url_site).'">'.interdire_scripts($GLOBALS['meta']['nom_site'] ?? '')."</a></h1>\n";
161	161
162	162	$titre = ($options['titre'] ?? '');
163	163	if ($titre) {
164		- $header .= '<h2>' . interdire_scripts($titre) . '</h2>';
	164	+ $header .= '<h2>'.interdire_scripts($titre).'</h2>';
165	165	}
166	166	$header .= '</header>';
167	167
168		- return $header . "<div class='corps'>\n";
	168	+ return $header."<div class='corps'>\n";
169	169	}
170	170
171	171	/**
		@@ -179,13 +179,13 @@ discard block
		block discarded – undo
179	179	if (isset($options['footer'])) {
180	180	$footer = $options['footer'];
181	181	} else {
182		- $footer = '<a href="' . attribut_html($url_site) . '">' . _T('retour') . "</a>\n";
	182	+ $footer = '<a href="'.attribut_html($url_site).'">'._T('retour')."</a>\n";
183	183	}
184	184	if (!empty($footer)) {
185	185	$footer = "<footer>\n{$footer}</footer>";
186	186	}
187	187
188		- return "</div>\n" . $footer;
	188	+ return "</div>\n".$footer;
189	189	}
190	190
191	191

		@@ -135,7 +135,7 @@ discard block
		block discarded – undo
135	135	return $this->iter->{$nom}();
136	136	} catch (Exception) {
137	137	// #GETCHILDREN sur un fichier de DirectoryIterator ...
138		- spip_log("Methode {$nom} en echec sur " . $this->iter::class);
	138	+ spip_log("Methode {$nom} en echec sur ".$this->iter::class);
139	139	spip_log("Cela peut être normal : retour d'une ligne de resultat ne pouvant pas calculer cette methode");
140	140
141	141	return '';
		@@ -356,7 +356,7 @@ discard block
		block discarded – undo
356	356	return null;
357	357	}
358	358
359		- return '(' . implode(") {$operateur} (", $filtres_string) . ')';
	359	+ return '('.implode(") {$operateur} (", $filtres_string).')';
360	360	}
361	361
362	362	/**
		@@ -439,7 +439,7 @@ discard block
		block discarded – undo
439	439	return $this->composer_filtre($v[1], $v[0], $v[2]);
440	440	}
441	441
442		- return null; // sera ignore
	442	+ return null; // sera ignore
443	443	}
444	444
445	445	/**
		@@ -468,28 +468,28 @@ discard block
		block discarded – undo
468	468	// if (!in_array($cle, array('cle', 'valeur')))
469	469	// return;
470	470
471		- $a = '$this->get_select(\'' . $cle . '\')';
	471	+ $a = '$this->get_select(\''.$cle.'\')';
472	472
473	473	$filtre = '';
474	474
475	475	if ('REGEXP' == $op) {
476		- $filtre = 'filtrer("match", ' . $a . ', ' . str_replace('\"', '"', (string) $valeur) . ')';
	476	+ $filtre = 'filtrer("match", '.$a.', '.str_replace('\"', '"', (string) $valeur).')';
477	477	$op = '';
478	478	} else {
479	479	if ('LIKE' == $op) {
480	480	$valeur = str_replace(['\"', '_', '%'], ['"', '.', '.*'], preg_quote((string) $valeur));
481		- $filtre = 'filtrer("match", ' . $a . ', ' . $valeur . ')';
	481	+ $filtre = 'filtrer("match", '.$a.', '.$valeur.')';
482	482	$op = '';
483	483	} else {
484	484	if ('=' == $op) {
485	485	$op = '==';
486	486	} else {
487	487	if ('IN' == $op) {
488		- $filtre = 'in_array(' . $a . ', array' . $valeur . ')';
	488	+ $filtre = 'in_array('.$a.', array'.$valeur.')';
489	489	$op = '';
490	490	} else {
491	491	if (!in_array($op, ['<', '<=', '>', '>='])) {
492		- spip_log('operateur non reconnu ' . $op); // [todo] mettre une erreur de squelette
	492	+ spip_log('operateur non reconnu '.$op); // [todo] mettre une erreur de squelette
493	493	$op = '';
494	494	}
495	495	}
		@@ -498,7 +498,7 @@ discard block
		block discarded – undo
498	498	}
499	499
500	500	if ($op) {
501		- $filtre = $a . $op . str_replace('\"', '"', (string) $valeur);
	501	+ $filtre = $a.$op.str_replace('\"', '"', (string) $valeur);
502	502	}
503	503
504	504	if ($not) {
		@@ -547,7 +547,7 @@ discard block
		block discarded – undo
547	547	// Creer la fonction de filtrage sur $this
548	548	if ($this->filtre) {
549	549	if ($filtres = $this->assembler_filtres($this->filtre)) {
550		- $filtres = 'return ' . $filtres . ';';
	550	+ $filtres = 'return '.$filtres.';';
551	551	$this->func_filtre = fn () => eval($filtres);
552	552	} else {
553	553	$this->func_filtre = null;

		@@ -99,7 +99,7 @@ discard block
		block discarded – undo
99	99	$res['message_ok'] = _T('info_modification_enregistree');
100	100	if ($retour) {
101	101	if (str_starts_with($retour, 'javascript:')) {
102		- $res['message_ok'] .= '<script type="text/javascript">/<![CDATA[/' . substr($retour, 11) . '/]]>/</script>';
	102	+ $res['message_ok'] .= '<script type="text/javascript">/<![CDATA[/'.substr($retour, 11).'/]]>/</script>';
103	103	$res['editable'] = true;
104	104	} else {
105	105	$res['redirect'] = parametre_url($retour, $id_table_objet, $id);
		@@ -139,7 +139,7 @@ discard block
		block discarded – undo
139	139	if (!isset($erreurs[$champ])) {
140	140	$erreurs[$champ] = '';
141	141	}
142		- $erreurs[$champ] .= _T('alerte_modif_info_concourante') . "<br /><textarea readonly='readonly' class='forml'>" . entites_html($conflit['base']) . '</textarea>';
	142	+ $erreurs[$champ] .= _T('alerte_modif_info_concourante')."<br /><textarea readonly='readonly' class='forml'>".entites_html($conflit['base']).'</textarea>';
143	143	}
144	144	}
145	145	}
		@@ -219,7 +219,7 @@ discard block
		block discarded – undo
219	219	if (
220	220	$config_fonc
221	221	&& !in_array($config_fonc, ['articles_edit_config', 'rubriques_edit_config', 'auteurs_edit_config'])
222		- && $config_fonc !== $table_objet . '_edit_config'
	222	+ && $config_fonc !== $table_objet.'_edit_config'
223	223	&& ($args = test_formulaire_inclus_par_modele())
224	224	&& in_array($config_fonc, $args)
225	225	) {
		@@ -231,12 +231,12 @@ discard block
		block discarded – undo
231	231	// Appel direct dans un squelette
232	232	if (!$row) {
233	233	if (!$new \|\| $lier_trad) {
234		- if ($select = charger_fonction('precharger_' . $type, 'inc', true)) {
	234	+ if ($select = charger_fonction('precharger_'.$type, 'inc', true)) {
235	235	$row = $select($id, $id_parent, $lier_trad);
236	236	// si on a une fonction precharger, elle pu faire un reglage de langue
237	237	$lang_default = (empty($row['lang']) ? null : $row['lang']);
238	238	} else {
239		- $row = sql_fetsel('*', $table_objet_sql, $id_table_objet . '=' . (int) $id);
	239	+ $row = sql_fetsel('*', $table_objet_sql, $id_table_objet.'='.(int) $id);
240	240	}
241	241	if (!$new) {
242	242	$md5 = controles_md5($row ?: []);
		@@ -286,7 +286,7 @@ discard block
		block discarded – undo
286	286	if ($config_fonc) {
287	287	$contexte['config'] = $config = $config_fonc($contexte);
288	288	if (!$lang_default) {
289		- $lang_default = $config['langue'] ?? session_get('lang') ;
	289	+ $lang_default = $config['langue'] ?? session_get('lang');
290	290	}
291	291	}
292	292	$config += [
		@@ -308,13 +308,13 @@ discard block
		block discarded – undo
308	308	unset($contexte['lang']);
309	309	}
310	310
311		- $contexte['_hidden'] = "<input type='hidden' name='editer_$type' value='oui' />\n" .
	311	+ $contexte['_hidden'] = "<input type='hidden' name='editer_$type' value='oui' />\n".
312	312	($lier_trad
313		- ? "\n<input type='hidden' name='lier_trad' value='" .
314		- $lier_trad .
315		- "' />" .
316		- "\n<input type='hidden' name='changer_lang' value='" .
317		- $lang_default .
	313	+ ? "\n<input type='hidden' name='lier_trad' value='".
	314	+ $lier_trad.
	315	+ "' />".
	316	+ "\n<input type='hidden' name='changer_lang' value='".
	317	+ $lang_default.
318	318	"' />"
319	319	: '')
320	320	. $hidden
		@@ -355,14 +355,14 @@ discard block
		block discarded – undo
355	355	$aider = charger_fonction('aider', 'inc');
356	356	if (strlen($texte) > 28 * 1024) {
357	357	$texte = str_replace("\r\n", "\n", $texte);
358		- $pos = strpos($texte, "\n\n", 28 * 1024); // coupe para > 28 ko
	358	+ $pos = strpos($texte, "\n\n", 28 * 1024); // coupe para > 28 ko
359	359	if ($pos > 0 && $pos < 32 * 1024) {
360		- $debut = substr($texte, 0, $pos) . "\n\n<!--SPIP-->\n";
	360	+ $debut = substr($texte, 0, $pos)."\n\n<!--SPIP-->\n";
361	361	$suite = substr($texte, $pos + 2);
362	362	} else {
363		- $pos = strpos($texte, ' ', 28 * 1024); // sinon coupe espace
	363	+ $pos = strpos($texte, ' ', 28 * 1024); // sinon coupe espace
364	364	if (!($pos > 0 && $pos < 32 * 1024)) {
365		- $pos = 28 * 1024; // au pire (pas d'espace trouv'e)
	365	+ $pos = 28 * 1024; // au pire (pas d'espace trouv'e)
366	366	$decalage = 0; // si y'a pas d'espace, il ne faut pas perdre le caract`ere
367	367	} else {
368	368	$decalage = 1;
		@@ -449,7 +449,7 @@ discard block
		block discarded – undo
449	449	$ctr = [];
450	450	foreach ($data as $key => $val) {
451	451	$m = md5($val ?? '');
452		- $k = $prefixe . $key;
	452	+ $k = $prefixe.$key;
453	453
454	454	$ctr[$k] = match ($format) {
455	455	'html' => "<input type='hidden' value='$m' name='$k' />",
		@@ -458,7 +458,7 @@ discard block
		block discarded – undo
458	458	}
459	459
460	460	if ($format === 'html') {
461		- return "\n\n<!-- controles md5 -->\n" . implode("\n", $ctr) . "\n\n";
	461	+ return "\n\n<!-- controles md5 -->\n".implode("\n", $ctr)."\n\n";
462	462	} else {
463	463	return $ctr;
464	464	}
		@@ -611,8 +611,8 @@ discard block
		block discarded – undo
611	611	// potentiellement modifiees entre temps par un autre utilisateur)
612	612	foreach ($champs as $key => $val) {
613	613	if (
614		- isset($ctr[$prefix . $key])
615		- && ($m = $ctr[$prefix . $key])
	614	+ isset($ctr[$prefix.$key])
	615	+ && ($m = $ctr[$prefix.$key])
616	616	&& (is_scalar($val) && $m == md5($val))
617	617	) {
618	618	unset($champs[$key]);
		@@ -641,7 +641,7 @@ discard block
		block discarded – undo
641	641	// de conflit.
642	642	$ctrh = $ctrq = $conflits = [];
643	643	foreach (array_keys($champs) as $key) {
644		- if (isset($ctr[$prefix . $key]) && ($m = $ctr[$prefix . $key])) {
	644	+ if (isset($ctr[$prefix.$key]) && ($m = $ctr[$prefix.$key])) {
645	645	$ctrh[$key] = $m;
646	646	$ctrq[] = $key;
647	647	}
		@@ -675,8 +675,8 @@ discard block
		block discarded – undo
675	675	*/
676	676	function display_conflit_champ($x) {
677	677	if (strstr($x, "\n") \|\| strlen($x) > 80) {
678		- return "<textarea style='width:99%; height:10em;'>" . entites_html($x) . "</textarea>\n";
	678	+ return "<textarea style='width:99%; height:10em;'>".entites_html($x)."</textarea>\n";
679	679	} else {
680		- return "<input type='text' size='40' style='width:99%' value=\"" . entites_html($x) . "\" />\n";
	680	+ return "<input type='text' size='40' style='width:99%' value=\"".entites_html($x)."\" />\n";
681	681	}
682	682	}

		@@ -48,15 +48,15 @@ discard block
		block discarded – undo
48	48	// Correction Create Database
49	49	// Create Database -> requete ignoree
50	50	if (str_starts_with((string) $this->query, 'CREATE DATABASE')) {
51		- spip_log("Sqlite : requete non executee -> $this->query", 'sqlite.' . _LOG_AVERTISSEMENT);
	51	+ spip_log("Sqlite : requete non executee -> $this->query", 'sqlite.'._LOG_AVERTISSEMENT);
52	52	$this->query = 'SELECT 1';
53	53	}
54	54
55	55	// Correction Insert Ignore
56	56	// INSERT IGNORE -> insert (tout court et pas 'insert or replace')
57	57	if (str_starts_with((string) $this->query, 'INSERT IGNORE')) {
58		- spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.' . _LOG_DEBUG);
59		- $this->query = 'INSERT ' . substr((string) $this->query, '13');
	58	+ spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.'._LOG_DEBUG);
	59	+ $this->query = 'INSERT '.substr((string) $this->query, '13');
60	60	}
61	61
62	62	// Correction des dates avec INTERVAL
		@@ -84,7 +84,7 @@ discard block
		block discarded – undo
84	84	if (($this->sqlite_version == 2) && (str_contains((string) $this->query, 'USING'))) {
85	85	spip_log(
86	86	"'USING (champ)' n'est pas reconnu en SQLite 2. Utilisez 'ON table1.champ = table2.champ'",
87		- 'sqlite.' . _LOG_ERREUR
	87	+ 'sqlite.'._LOG_ERREUR
88	88	);
89	89	$this->query = preg_replace('/USING\s\([^\)]\)/', '', (string) $this->query);
90	90	}
		@@ -107,8 +107,8 @@ discard block
		block discarded – undo
107	107	} else {
108	108	$suite = '';
109	109	}
110		- $pref = ($this->prefixe) ? $this->prefixe . '_' : '';
111		- $this->query = preg_replace('/([,\s])spip_/S', '\1' . $pref, (string) $this->query) . $suite;
	110	+ $pref = ($this->prefixe) ? $this->prefixe.'_' : '';
	111	+ $this->query = preg_replace('/([,\s])spip_/S', '\1'.$pref, (string) $this->query).$suite;
112	112
113	113	// Correction zero AS x
114	114	// pg n'aime pas 0+x AS alias, sqlite, dans le meme style,

		@@ -51,12 +51,12 @@
		block discarded – undo
51	51	// chercher la classe d'iterateur Iterateur/XXX
52	52	// definie dans le fichier src/Compilateur/Iterateur/xxx.php
53	53	// FIXME: déclarer quelque part les iterateurs supplémentaires
54		- $class = __NAMESPACE__ . '\\' . ucfirst(strtolower((string) $iterateur));
	54	+ $class = __NAMESPACE__.'\\'.ucfirst(strtolower((string) $iterateur));
55	55	if (!class_exists($class)) {
56	56	// historique
57	57	// Chercher IterateurXXX
58		- include_spip('iterateur/' . $iterateur);
59		- $class = 'Iterateur' . $iterateur;
	58	+ include_spip('iterateur/'.$iterateur);
	59	+ $class = 'Iterateur'.$iterateur;
60	60	if (!class_exists($class)) {
61	61	exit("Iterateur {$iterateur} non trouvé");
62	62	// si l'iterateur n'existe pas, on se rabat sur le generique

spip / SPIP

Push — master ( ad4f09...6d727b )

Status

Category

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

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

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

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

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

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

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