Inspection of "Correction deprecated php 7.4 : ordre de join inve..." - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 92ba1a...d4b189 )

by cam

created 2021-03-19 14:44 UTC

Status

Indentation +93 added lines, -93 removed lines patch added patch discarded remove patch

@@ -13,7 +13,7 @@  discard block
 block discarded – undo
 
 //
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 //
@@ -23,56 +23,56 @@  discard block
 block discarded – undo
 // https://code.spip.net/@image_math
 function produire_image_math($tex) {
 
-	switch ($GLOBALS['traiter_math']) {
-		// Attention: mathml desactiv'e pour l'instant
-		case 'mathml':
-			$ext = '.xhtml';
-			$server = $GLOBALS['mathml_server'];
-			break;
-		case 'tex':
-			$ext = '.png';
-			$server = $GLOBALS['tex_server'];
-			break;
-		default:
-			return $tex;
-	}
-
-	// Regarder dans le repertoire local des images TeX et blocs MathML
-	if (!@is_dir($dir_tex = _DIR_VAR . 'cache-TeX/')) {
-		@mkdir($dir_tex, _SPIP_CHMOD);
-	}
-	$fichier = $dir_tex . md5(trim($tex)) . $ext;
-
-
-	if (!@file_exists($fichier)) {
-		// Aller chercher l'image sur le serveur
-		if ($server) {
-			spip_log($url = $server . '?' . rawurlencode($tex));
-			include_spip('inc/distant');
-			recuperer_page($url, $fichier);
-		}
-	}
-
-
-	// Composer la reponse selon presence ou non de l'image
-	$tex = entites_html($tex);
-	if (@file_exists($fichier)) {
-
-		// MathML
-		if ($GLOBALS['traiter_math'] == 'mathml') {
-			return implode("", file($fichier));
-		} // TeX
-		else {
-			list(, , , $size) = @spip_getimagesize($fichier);
-			$alt = "alt=\"$tex\" title=\"$tex\"";
-
-			return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
-		}
-
-	} else // pas de fichier
-	{
-		return "<tt><span class='spip_code' dir='ltr'>$tex</span></tt>";
-	}
+    switch ($GLOBALS['traiter_math']) {
+        // Attention: mathml desactiv'e pour l'instant
+        case 'mathml':
+            $ext = '.xhtml';
+            $server = $GLOBALS['mathml_server'];
+            break;
+        case 'tex':
+            $ext = '.png';
+            $server = $GLOBALS['tex_server'];
+            break;
+        default:
+            return $tex;
+    }
+
+    // Regarder dans le repertoire local des images TeX et blocs MathML
+    if (!@is_dir($dir_tex = _DIR_VAR . 'cache-TeX/')) {
+        @mkdir($dir_tex, _SPIP_CHMOD);
+    }
+    $fichier = $dir_tex . md5(trim($tex)) . $ext;
+
+
+    if (!@file_exists($fichier)) {
+        // Aller chercher l'image sur le serveur
+        if ($server) {
+            spip_log($url = $server . '?' . rawurlencode($tex));
+            include_spip('inc/distant');
+            recuperer_page($url, $fichier);
+        }
+    }
+
+
+    // Composer la reponse selon presence ou non de l'image
+    $tex = entites_html($tex);
+    if (@file_exists($fichier)) {
+
+        // MathML
+        if ($GLOBALS['traiter_math'] == 'mathml') {
+            return implode("", file($fichier));
+        } // TeX
+        else {
+            list(, , , $size) = @spip_getimagesize($fichier);
+            $alt = "alt=\"$tex\" title=\"$tex\"";
+
+            return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
+        }
+
+    } else // pas de fichier
+    {
+        return "<tt><span class='spip_code' dir='ltr'>$tex</span></tt>";
+    }
 
 }
 
@@ -108,46 +108,46 @@  discard block
 block discarded – undo
  */
 function traiter_math($letexte, $source = '', $defaire_amp = false) {
 
-	$texte_a_voir = $letexte;
-	while (($debut = strpos($texte_a_voir, "<math>")) !== false) {
-		if (!$fin = strpos($texte_a_voir, "</math>")) {
-			$fin = strlen($texte_a_voir);
-		}
-
-		$texte_debut = substr($texte_a_voir, 0, $debut);
-		$texte_milieu = substr($texte_a_voir,
-			$debut + strlen("<math>"), $fin - $debut - strlen("<math>"));
-		$texte_fin = substr($texte_a_voir,
-			$fin + strlen("</math>"), strlen($texte_a_voir));
-
-		// Les doubles $$x^2$$ en mode 'div'
-		while ((preg_match(",[$][$]([^$]+)[$][$],", $texte_milieu, $regs))) {
-			$expression = $regs[1];
-			if ($defaire_amp) {
-				$expression = str_replace('&amp;', '&', $expression);
-			}
-			$echap = "\n<p class=\"spip\" style=\"text-align: center;\">" . produire_image_math($expression) . "</p>\n";
-			$pos = strpos($texte_milieu, $regs[0]);
-			$texte_milieu = substr($texte_milieu, 0, $pos)
-				. code_echappement($echap, $source)
-				. substr($texte_milieu, $pos + strlen($regs[0]));
-		}
-
-		// Les simples $x^2$ en mode 'span'
-		while ((preg_match(",[$]([^$]+)[$],", $texte_milieu, $regs))) {
-			$expression = $regs[1];
-			if ($defaire_amp) {
-				$expression = str_replace('&amp;', '&', $expression);
-			}
-			$echap = produire_image_math($expression);
-			$pos = strpos($texte_milieu, $regs[0]);
-			$texte_milieu = substr($texte_milieu, 0, $pos)
-				. code_echappement($echap, $source)
-				. substr($texte_milieu, $pos + strlen($regs[0]));
-		}
-
-		$texte_a_voir = $texte_debut . $texte_milieu . $texte_fin;
-	}
-
-	return $texte_a_voir;
+    $texte_a_voir = $letexte;
+    while (($debut = strpos($texte_a_voir, "<math>")) !== false) {
+        if (!$fin = strpos($texte_a_voir, "</math>")) {
+            $fin = strlen($texte_a_voir);
+        }
+
+        $texte_debut = substr($texte_a_voir, 0, $debut);
+        $texte_milieu = substr($texte_a_voir,
+            $debut + strlen("<math>"), $fin - $debut - strlen("<math>"));
+        $texte_fin = substr($texte_a_voir,
+            $fin + strlen("</math>"), strlen($texte_a_voir));
+
+        // Les doubles $$x^2$$ en mode 'div'
+        while ((preg_match(",[$][$]([^$]+)[$][$],", $texte_milieu, $regs))) {
+            $expression = $regs[1];
+            if ($defaire_amp) {
+                $expression = str_replace('&amp;', '&', $expression);
+            }
+            $echap = "\n<p class=\"spip\" style=\"text-align: center;\">" . produire_image_math($expression) . "</p>\n";
+            $pos = strpos($texte_milieu, $regs[0]);
+            $texte_milieu = substr($texte_milieu, 0, $pos)
+                . code_echappement($echap, $source)
+                . substr($texte_milieu, $pos + strlen($regs[0]));
+        }
+
+        // Les simples $x^2$ en mode 'span'
+        while ((preg_match(",[$]([^$]+)[$],", $texte_milieu, $regs))) {
+            $expression = $regs[1];
+            if ($defaire_amp) {
+                $expression = str_replace('&amp;', '&', $expression);
+            }
+            $echap = produire_image_math($expression);
+            $pos = strpos($texte_milieu, $regs[0]);
+            $texte_milieu = substr($texte_milieu, 0, $pos)
+                . code_echappement($echap, $source)
+                . substr($texte_milieu, $pos + strlen($regs[0]));
+        }
+
+        $texte_a_voir = $texte_debut . $texte_milieu . $texte_fin;
+    }
+
+    return $texte_a_voir;
 }

Please login to merge, or discard this patch.

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

@@ -38,16 +38,16 @@  discard block
 block discarded – undo
 	}
 
 	// Regarder dans le repertoire local des images TeX et blocs MathML
-	if (!@is_dir($dir_tex = _DIR_VAR . 'cache-TeX/')) {
+	if (!@is_dir($dir_tex = _DIR_VAR.'cache-TeX/')) {
 		@mkdir($dir_tex, _SPIP_CHMOD);
 	}
-	$fichier = $dir_tex . md5(trim($tex)) . $ext;
+	$fichier = $dir_tex.md5(trim($tex)).$ext;
 
 
 	if (!@file_exists($fichier)) {
 		// Aller chercher l'image sur le serveur
 		if ($server) {
-			spip_log($url = $server . '?' . rawurlencode($tex));
+			spip_log($url = $server.'?'.rawurlencode($tex));
 			include_spip('inc/distant');
 			recuperer_page($url, $fichier);
 		}
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
 			return implode("", file($fichier));
 		} // TeX
 		else {
-			list(, , , $size) = @spip_getimagesize($fichier);
+			list(,,, $size) = @spip_getimagesize($fichier);
 			$alt = "alt=\"$tex\" title=\"$tex\"";
 
 			return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
 			if ($defaire_amp) {
 				$expression = str_replace('&amp;', '&', $expression);
 			}
-			$echap = "\n<p class=\"spip\" style=\"text-align: center;\">" . produire_image_math($expression) . "</p>\n";
+			$echap = "\n<p class=\"spip\" style=\"text-align: center;\">".produire_image_math($expression)."</p>\n";
 			$pos = strpos($texte_milieu, $regs[0]);
 			$texte_milieu = substr($texte_milieu, 0, $pos)
 				. code_echappement($echap, $source)
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
 				. substr($texte_milieu, $pos + strlen($regs[0]));
 		}
 
-		$texte_a_voir = $texte_debut . $texte_milieu . $texte_fin;
+		$texte_a_voir = $texte_debut.$texte_milieu.$texte_fin;
 	}
 
 	return $texte_a_voir;

Please login to merge, or discard this patch.

		@@ -38,16 +38,16 @@ discard block
		block discarded – undo
38	38	}
39	39
40	40	// Regarder dans le repertoire local des images TeX et blocs MathML
41		- if (!@is_dir($dir_tex = _DIR_VAR . 'cache-TeX/')) {
	41	+ if (!@is_dir($dir_tex = _DIR_VAR.'cache-TeX/')) {
42	42	@mkdir($dir_tex, _SPIP_CHMOD);
43	43	}
44		- $fichier = $dir_tex . md5(trim($tex)) . $ext;
	44	+ $fichier = $dir_tex.md5(trim($tex)).$ext;
45	45
46	46
47	47	if (!@file_exists($fichier)) {
48	48	// Aller chercher l'image sur le serveur
49	49	if ($server) {
50		- spip_log($url = $server . '?' . rawurlencode($tex));
	50	+ spip_log($url = $server.'?'.rawurlencode($tex));
51	51	include_spip('inc/distant');
52	52	recuperer_page($url, $fichier);
53	53	}
		@@ -63,7 +63,7 @@ discard block
		block discarded – undo
63	63	return implode("", file($fichier));
64	64	} // TeX
65	65	else {
66		- list(, , , $size) = @spip_getimagesize($fichier);
	66	+ list(,,, $size) = @spip_getimagesize($fichier);
67	67	$alt = "alt=\"$tex\" title=\"$tex\"";
68	68
69	69	return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
		@@ -126,7 +126,7 @@ discard block
		block discarded – undo
126	126	if ($defaire_amp) {
127	127	$expression = str_replace('&', '&', $expression);
128	128	}
129		- $echap = "\n<p class=\"spip\" style=\"text-align: center;\">" . produire_image_math($expression) . "</p>\n";
	129	+ $echap = "\n<p class=\"spip\" style=\"text-align: center;\">".produire_image_math($expression)."</p>\n";
130	130	$pos = strpos($texte_milieu, $regs[0]);
131	131	$texte_milieu = substr($texte_milieu, 0, $pos)
132	132	. code_echappement($echap, $source)
		@@ -146,7 +146,7 @@ discard block
		block discarded – undo
146	146	. substr($texte_milieu, $pos + strlen($regs[0]));
147	147	}
148	148
149		- $texte_a_voir = $texte_debut . $texte_milieu . $texte_fin;
	149	+ $texte_a_voir = $texte_debut.$texte_milieu.$texte_fin;
150	150	}
151	151
152	152	return $texte_a_voir;

		@@ -13,7 +13,7 @@ discard block
		block discarded – undo
13	13
14	14	//
15	15	if (!defined('_ECRIRE_INC_VERSION')) {
16		- return;
	16	+ return;
17	17	}
18	18
19	19	//
		@@ -23,56 +23,56 @@ discard block
		block discarded – undo
23	23	// https://code.spip.net/@image_math
24	24	function produire_image_math($tex) {
25	25
26		- switch ($GLOBALS['traiter_math']) {
27		- // Attention: mathml desactiv'e pour l'instant
28		- case 'mathml':
29		- $ext = '.xhtml';
30		- $server = $GLOBALS['mathml_server'];
31		- break;
32		- case 'tex':
33		- $ext = '.png';
34		- $server = $GLOBALS['tex_server'];
35		- break;
36		- default:
37		- return $tex;
38		- }
39		-
40		- // Regarder dans le repertoire local des images TeX et blocs MathML
41		- if (!@is_dir($dir_tex = _DIR_VAR . 'cache-TeX/')) {
42		- @mkdir($dir_tex, _SPIP_CHMOD);
43		- }
44		- $fichier = $dir_tex . md5(trim($tex)) . $ext;
45		-
46		-
47		- if (!@file_exists($fichier)) {
48		- // Aller chercher l'image sur le serveur
49		- if ($server) {
50		- spip_log($url = $server . '?' . rawurlencode($tex));
51		- include_spip('inc/distant');
52		- recuperer_page($url, $fichier);
53		- }
54		- }
55		-
56		-
57		- // Composer la reponse selon presence ou non de l'image
58		- $tex = entites_html($tex);
59		- if (@file_exists($fichier)) {
60		-
61		- // MathML
62		- if ($GLOBALS['traiter_math'] == 'mathml') {
63		- return implode("", file($fichier));
64		- } // TeX
65		- else {
66		- list(, , , $size) = @spip_getimagesize($fichier);
67		- $alt = "alt=\"$tex\" title=\"$tex\"";
68		-
69		- return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
70		- }
71		-
72		- } else // pas de fichier
73		- {
74		- return "<tt><span class='spip_code' dir='ltr'>$tex</span></tt>";
75		- }
	26	+ switch ($GLOBALS['traiter_math']) {
	27	+ // Attention: mathml desactiv'e pour l'instant
	28	+ case 'mathml':
	29	+ $ext = '.xhtml';
	30	+ $server = $GLOBALS['mathml_server'];
	31	+ break;
	32	+ case 'tex':
	33	+ $ext = '.png';
	34	+ $server = $GLOBALS['tex_server'];
	35	+ break;
	36	+ default:
	37	+ return $tex;
	38	+ }
	39	+
	40	+ // Regarder dans le repertoire local des images TeX et blocs MathML
	41	+ if (!@is_dir($dir_tex = _DIR_VAR . 'cache-TeX/')) {
	42	+ @mkdir($dir_tex, _SPIP_CHMOD);
	43	+ }
	44	+ $fichier = $dir_tex . md5(trim($tex)) . $ext;
	45	+
	46	+
	47	+ if (!@file_exists($fichier)) {
	48	+ // Aller chercher l'image sur le serveur
	49	+ if ($server) {
	50	+ spip_log($url = $server . '?' . rawurlencode($tex));
	51	+ include_spip('inc/distant');
	52	+ recuperer_page($url, $fichier);
	53	+ }
	54	+ }
	55	+
	56	+
	57	+ // Composer la reponse selon presence ou non de l'image
	58	+ $tex = entites_html($tex);
	59	+ if (@file_exists($fichier)) {
	60	+
	61	+ // MathML
	62	+ if ($GLOBALS['traiter_math'] == 'mathml') {
	63	+ return implode("", file($fichier));
	64	+ } // TeX
	65	+ else {
	66	+ list(, , , $size) = @spip_getimagesize($fichier);
	67	+ $alt = "alt=\"$tex\" title=\"$tex\"";
	68	+
	69	+ return "<img src=\"$fichier\" style=\"vertical-align:middle;\" $size $alt />";
	70	+ }
	71	+
	72	+ } else // pas de fichier
	73	+ {
	74	+ return "<tt><span class='spip_code' dir='ltr'>$tex</span></tt>";
	75	+ }
76	76
77	77	}
78	78
		@@ -108,46 +108,46 @@ discard block
		block discarded – undo
108	108	*/
109	109	function traiter_math($letexte, $source = '', $defaire_amp = false) {
110	110
111		- $texte_a_voir = $letexte;
112		- while (($debut = strpos($texte_a_voir, "<math>")) !== false) {
113		- if (!$fin = strpos($texte_a_voir, "</math>")) {
114		- $fin = strlen($texte_a_voir);
115		- }
116		-
117		- $texte_debut = substr($texte_a_voir, 0, $debut);
118		- $texte_milieu = substr($texte_a_voir,
119		- $debut + strlen("<math>"), $fin - $debut - strlen("<math>"));
120		- $texte_fin = substr($texte_a_voir,
121		- $fin + strlen("</math>"), strlen($texte_a_voir));
122		-
123		- // Les doubles $$x^2$$ en mode 'div'
124		- while ((preg_match(",[$][$]([^$]+)[$][$],", $texte_milieu, $regs))) {
125		- $expression = $regs[1];
126		- if ($defaire_amp) {
127		- $expression = str_replace('&', '&', $expression);
128		- }
129		- $echap = "\n<p class=\"spip\" style=\"text-align: center;\">" . produire_image_math($expression) . "</p>\n";
130		- $pos = strpos($texte_milieu, $regs[0]);
131		- $texte_milieu = substr($texte_milieu, 0, $pos)
132		- . code_echappement($echap, $source)
133		- . substr($texte_milieu, $pos + strlen($regs[0]));
134		- }
135		-
136		- // Les simples $x^2$ en mode 'span'
137		- while ((preg_match(",[$]([^$]+)[$],", $texte_milieu, $regs))) {
138		- $expression = $regs[1];
139		- if ($defaire_amp) {
140		- $expression = str_replace('&', '&', $expression);
141		- }
142		- $echap = produire_image_math($expression);
143		- $pos = strpos($texte_milieu, $regs[0]);
144		- $texte_milieu = substr($texte_milieu, 0, $pos)
145		- . code_echappement($echap, $source)
146		- . substr($texte_milieu, $pos + strlen($regs[0]));
147		- }
148		-
149		- $texte_a_voir = $texte_debut . $texte_milieu . $texte_fin;
150		- }
151		-
152		- return $texte_a_voir;
	111	+ $texte_a_voir = $letexte;
	112	+ while (($debut = strpos($texte_a_voir, "<math>")) !== false) {
	113	+ if (!$fin = strpos($texte_a_voir, "</math>")) {
	114	+ $fin = strlen($texte_a_voir);
	115	+ }
	116	+
	117	+ $texte_debut = substr($texte_a_voir, 0, $debut);
	118	+ $texte_milieu = substr($texte_a_voir,
	119	+ $debut + strlen("<math>"), $fin - $debut - strlen("<math>"));
	120	+ $texte_fin = substr($texte_a_voir,
	121	+ $fin + strlen("</math>"), strlen($texte_a_voir));
	122	+
	123	+ // Les doubles $$x^2$$ en mode 'div'
	124	+ while ((preg_match(",[$][$]([^$]+)[$][$],", $texte_milieu, $regs))) {
	125	+ $expression = $regs[1];
	126	+ if ($defaire_amp) {
	127	+ $expression = str_replace('&', '&', $expression);
	128	+ }
	129	+ $echap = "\n<p class=\"spip\" style=\"text-align: center;\">" . produire_image_math($expression) . "</p>\n";
	130	+ $pos = strpos($texte_milieu, $regs[0]);
	131	+ $texte_milieu = substr($texte_milieu, 0, $pos)
	132	+ . code_echappement($echap, $source)
	133	+ . substr($texte_milieu, $pos + strlen($regs[0]));
	134	+ }
	135	+
	136	+ // Les simples $x^2$ en mode 'span'
	137	+ while ((preg_match(",[$]([^$]+)[$],", $texte_milieu, $regs))) {
	138	+ $expression = $regs[1];
	139	+ if ($defaire_amp) {
	140	+ $expression = str_replace('&', '&', $expression);
	141	+ }
	142	+ $echap = produire_image_math($expression);
	143	+ $pos = strpos($texte_milieu, $regs[0]);
	144	+ $texte_milieu = substr($texte_milieu, 0, $pos)
	145	+ . code_echappement($echap, $source)
	146	+ . substr($texte_milieu, $pos + strlen($regs[0]));
	147	+ }
	148	+
	149	+ $texte_a_voir = $texte_debut . $texte_milieu . $texte_fin;
	150	+ }
	151	+
	152	+ return $texte_a_voir;
153	153	}

spip / SPIP

Push — master ( 92ba1a...d4b189 )

Status

Category

Indentation +93 added lines, -93 removed lines patch added patch discarded remove patch

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