Inspection of "Des corrections suite au passage de Rector." - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6c9939...4ccfda )

by cam

created 2022-01-14 20:07 UTC

Status

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

@@ -11,53 +11,53 @@  discard block
 block discarded – undo
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function formulaires_configurer_multilinguisme_charger_dist() {
-	$valeurs = [];
-	$valeurs['multi_secteurs'] = $GLOBALS['meta']['multi_secteurs'];
-	foreach (['multi_objets', 'gerer_trad_objets'] as $m) {
-		$valeurs[$m] = explode(',', $GLOBALS['meta'][$m] ?? '');
-	}
-
-	if (
-		is_countable($valeurs['multi_objets']) ? count($valeurs['multi_objets']) : 0
-		or count(explode(',', $GLOBALS['meta']['langues_utilisees'])) > 1
-	) {
-		$selection = (is_null(_request('multi_objets')) ?
-			explode(',', $GLOBALS['meta']['langues_multilingue']) : _request('langues_auth'));
-		$valeurs['_langues'] = saisie_langues_utiles('langues_auth', $selection ?: []);
-		$valeurs['_nb_langues_selection'] = is_countable($selection) ? count($selection) : 0;
-	}
-
-	return $valeurs;
+    $valeurs = [];
+    $valeurs['multi_secteurs'] = $GLOBALS['meta']['multi_secteurs'];
+    foreach (['multi_objets', 'gerer_trad_objets'] as $m) {
+        $valeurs[$m] = explode(',', $GLOBALS['meta'][$m] ?? '');
+    }
+
+    if (
+        is_countable($valeurs['multi_objets']) ? count($valeurs['multi_objets']) : 0
+        or count(explode(',', $GLOBALS['meta']['langues_utilisees'])) > 1
+    ) {
+        $selection = (is_null(_request('multi_objets')) ?
+            explode(',', $GLOBALS['meta']['langues_multilingue']) : _request('langues_auth'));
+        $valeurs['_langues'] = saisie_langues_utiles('langues_auth', $selection ?: []);
+        $valeurs['_nb_langues_selection'] = is_countable($selection) ? count($selection) : 0;
+    }
+
+    return $valeurs;
 }
 
 
 function formulaires_configurer_multilinguisme_traiter_dist() {
-	$res = ['editable' => true];
-	// un checkbox seul de name X non coche n'est pas poste.
-	// on verifie le champ X_check qui indique que la checkbox etait presente dans le formulaire.
-	foreach (['multi_secteurs'] as $m) {
-		if (!is_null(_request($m . '_check'))) {
-			ecrire_meta($m, _request($m) ? 'oui' : 'non');
-		}
-	}
-	foreach (['multi_objets', 'gerer_trad_objets'] as $m) {
-		if (!is_null($v = _request($m))) {
-			// join et enlever la valeur vide ''
-			ecrire_meta($m, implode(',', array_diff($v, [''])));
-		}
-	}
-
-	if ($i = _request('langues_auth') and is_array($i)) {
-		$i = array_unique(array_merge($i, explode(',', $GLOBALS['meta']['langues_utilisees'])));
-		ecrire_meta('langues_multilingue', implode(',', $i));
-	}
-	$res['message_ok'] = _T('config_info_enregistree');
-
-	return $res;
+    $res = ['editable' => true];
+    // un checkbox seul de name X non coche n'est pas poste.
+    // on verifie le champ X_check qui indique que la checkbox etait presente dans le formulaire.
+    foreach (['multi_secteurs'] as $m) {
+        if (!is_null(_request($m . '_check'))) {
+            ecrire_meta($m, _request($m) ? 'oui' : 'non');
+        }
+    }
+    foreach (['multi_objets', 'gerer_trad_objets'] as $m) {
+        if (!is_null($v = _request($m))) {
+            // join et enlever la valeur vide ''
+            ecrire_meta($m, implode(',', array_diff($v, [''])));
+        }
+    }
+
+    if ($i = _request('langues_auth') and is_array($i)) {
+        $i = array_unique(array_merge($i, explode(',', $GLOBALS['meta']['langues_utilisees'])));
+        ecrire_meta('langues_multilingue', implode(',', $i));
+    }
+    $res['message_ok'] = _T('config_info_enregistree');
+
+    return $res;
 }
 
 /**
@@ -67,13 +67,13 @@  discard block
 block discarded – undo
  * @return string
  */
 function table_supporte_lang($table_sql) {
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_sql);
-	if (!$desc or !isset($desc['field']['lang'])) {
-		return '';
-	}
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_sql);
+    if (!$desc or !isset($desc['field']['lang'])) {
+        return '';
+    }
 
-	return ' ';
+    return ' ';
 }
 
 /**
@@ -83,67 +83,67 @@  discard block
 block discarded – undo
  * @return string
  */
 function table_supporte_trad($table_sql) {
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_sql);
-	if (!$desc or !isset($desc['field']['id_trad'])) {
-		return '';
-	}
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_sql);
+    if (!$desc or !isset($desc['field']['id_trad'])) {
+        return '';
+    }
 
-	return ' ';
+    return ' ';
 }
 
 
 function saisie_langues_utiles($name, $selection) {
-	include_spip('inc/lang_liste');
-	$langues = $GLOBALS['codes_langues'];
-
-	$langues_installees = explode(',', $GLOBALS['meta']['langues_proposees']);
-	$langues_trad = array_flip($langues_installees);
-
-	$langues_bloquees = explode(',', $GLOBALS['meta']['langues_utilisees']);
-
-	$res = '';
-
-	$i = 0;
-	foreach ($langues_bloquees as $code_langue) {
-		$nom_langue = $langues[$code_langue];
-		$res .= "<li class='choix "
-			. alterner(++$i, 'odd', 'even')
-			. (isset($langues_trad[$code_langue]) ? ' traduite' : '')
-			. "'>"
-			. "<input type='hidden' name='{$name}[]' value='$code_langue'>" // necessaire ...
-			. "<input type='checkbox' name='{$name}[]' id='{$name}_$code_langue' value='$code_langue' checked='checked' disabled='disabled' />"
-			. "<label for='{$name}_$code_langue'>" . $nom_langue . "&nbsp;&nbsp; <span class='code_langue'>[$code_langue]</span></label>"
-			. '</li>';
-	}
-
-	if ($res) {
-		$res = "<ul id='langues_bloquees'>" . $res . "</ul><div class='nettoyeur'></div>";
-	}
-
-	$res .= "<ul id='langues_proposees'>";
-
-	$i = 0;
-	$langues_bloquees = array_flip($langues_bloquees);
-	foreach ($langues as $code_langue => $nom_langue) {
-		if (!isset($langues_bloquees[$code_langue])) {
-			$checked = (in_array($code_langue, $selection) ? ' checked="checked"' : '');
-			$res .= "<li class='choix "
-				. alterner(++$i, 'odd', 'even')
-				. (isset($langues_trad[$code_langue]) ? ' traduite' : '')
-				. "'>"
-				. "<input type='checkbox' name='{$name}[]' id='{$name}_$code_langue' value='$code_langue'"
-				. $checked
-				. '/>'
-				. "<label for='{$name}_$code_langue'"
-				. ($checked ? " class='on'" : '')
-				. '>'
-				. $nom_langue . "&nbsp;&nbsp; <span class='code_langue'>[$code_langue]</span></label>"
-				. '</li>';
-		}
-	}
-
-	$res .= "</ul><div class='nettoyeur'></div>";
-
-	return $res;
+    include_spip('inc/lang_liste');
+    $langues = $GLOBALS['codes_langues'];
+
+    $langues_installees = explode(',', $GLOBALS['meta']['langues_proposees']);
+    $langues_trad = array_flip($langues_installees);
+
+    $langues_bloquees = explode(',', $GLOBALS['meta']['langues_utilisees']);
+
+    $res = '';
+
+    $i = 0;
+    foreach ($langues_bloquees as $code_langue) {
+        $nom_langue = $langues[$code_langue];
+        $res .= "<li class='choix "
+            . alterner(++$i, 'odd', 'even')
+            . (isset($langues_trad[$code_langue]) ? ' traduite' : '')
+            . "'>"
+            . "<input type='hidden' name='{$name}[]' value='$code_langue'>" // necessaire ...
+            . "<input type='checkbox' name='{$name}[]' id='{$name}_$code_langue' value='$code_langue' checked='checked' disabled='disabled' />"
+            . "<label for='{$name}_$code_langue'>" . $nom_langue . "&nbsp;&nbsp; <span class='code_langue'>[$code_langue]</span></label>"
+            . '</li>';
+    }
+
+    if ($res) {
+        $res = "<ul id='langues_bloquees'>" . $res . "</ul><div class='nettoyeur'></div>";
+    }
+
+    $res .= "<ul id='langues_proposees'>";
+
+    $i = 0;
+    $langues_bloquees = array_flip($langues_bloquees);
+    foreach ($langues as $code_langue => $nom_langue) {
+        if (!isset($langues_bloquees[$code_langue])) {
+            $checked = (in_array($code_langue, $selection) ? ' checked="checked"' : '');
+            $res .= "<li class='choix "
+                . alterner(++$i, 'odd', 'even')
+                . (isset($langues_trad[$code_langue]) ? ' traduite' : '')
+                . "'>"
+                . "<input type='checkbox' name='{$name}[]' id='{$name}_$code_langue' value='$code_langue'"
+                . $checked
+                . '/>'
+                . "<label for='{$name}_$code_langue'"
+                . ($checked ? " class='on'" : '')
+                . '>'
+                . $nom_langue . "&nbsp;&nbsp; <span class='code_langue'>[$code_langue]</span></label>"
+                . '</li>';
+        }
+    }
+
+    $res .= "</ul><div class='nettoyeur'></div>";
+
+    return $res;
 }

Please login to merge, or discard this patch.

		@@ -11,53 +11,53 @@ discard block
		block discarded – undo
11	11	\***************************************************************************/
12	12
13	13	if (!defined('_ECRIRE_INC_VERSION')) {
14		- return;
	14	+ return;
15	15	}
16	16
17	17	function formulaires_configurer_multilinguisme_charger_dist() {
18		- $valeurs = [];
19		- $valeurs['multi_secteurs'] = $GLOBALS['meta']['multi_secteurs'];
20		- foreach (['multi_objets', 'gerer_trad_objets'] as $m) {
21		- $valeurs[$m] = explode(',', $GLOBALS['meta'][$m] ?? '');
22		- }
23		-
24		- if (
25		- is_countable($valeurs['multi_objets']) ? count($valeurs['multi_objets']) : 0
26		- or count(explode(',', $GLOBALS['meta']['langues_utilisees'])) > 1
27		- ) {
28		- $selection = (is_null(_request('multi_objets')) ?
29		- explode(',', $GLOBALS['meta']['langues_multilingue']) : _request('langues_auth'));
30		- $valeurs['_langues'] = saisie_langues_utiles('langues_auth', $selection ?: []);
31		- $valeurs['_nb_langues_selection'] = is_countable($selection) ? count($selection) : 0;
32		- }
33		-
34		- return $valeurs;
	18	+ $valeurs = [];
	19	+ $valeurs['multi_secteurs'] = $GLOBALS['meta']['multi_secteurs'];
	20	+ foreach (['multi_objets', 'gerer_trad_objets'] as $m) {
	21	+ $valeurs[$m] = explode(',', $GLOBALS['meta'][$m] ?? '');
	22	+ }
	23	+
	24	+ if (
	25	+ is_countable($valeurs['multi_objets']) ? count($valeurs['multi_objets']) : 0
	26	+ or count(explode(',', $GLOBALS['meta']['langues_utilisees'])) > 1
	27	+ ) {
	28	+ $selection = (is_null(_request('multi_objets')) ?
	29	+ explode(',', $GLOBALS['meta']['langues_multilingue']) : _request('langues_auth'));
	30	+ $valeurs['_langues'] = saisie_langues_utiles('langues_auth', $selection ?: []);
	31	+ $valeurs['_nb_langues_selection'] = is_countable($selection) ? count($selection) : 0;
	32	+ }
	33	+
	34	+ return $valeurs;
35	35	}
36	36
37	37
38	38	function formulaires_configurer_multilinguisme_traiter_dist() {
39		- $res = ['editable' => true];
40		- // un checkbox seul de name X non coche n'est pas poste.
41		- // on verifie le champ X_check qui indique que la checkbox etait presente dans le formulaire.
42		- foreach (['multi_secteurs'] as $m) {
43		- if (!is_null(_request($m . '_check'))) {
44		- ecrire_meta($m, _request($m) ? 'oui' : 'non');
45		- }
46		- }
47		- foreach (['multi_objets', 'gerer_trad_objets'] as $m) {
48		- if (!is_null($v = _request($m))) {
49		- // join et enlever la valeur vide ''
50		- ecrire_meta($m, implode(',', array_diff($v, [''])));
51		- }
52		- }
53		-
54		- if ($i = _request('langues_auth') and is_array($i)) {
55		- $i = array_unique(array_merge($i, explode(',', $GLOBALS['meta']['langues_utilisees'])));
56		- ecrire_meta('langues_multilingue', implode(',', $i));
57		- }
58		- $res['message_ok'] = _T('config_info_enregistree');
59		-
60		- return $res;
	39	+ $res = ['editable' => true];
	40	+ // un checkbox seul de name X non coche n'est pas poste.
	41	+ // on verifie le champ X_check qui indique que la checkbox etait presente dans le formulaire.
	42	+ foreach (['multi_secteurs'] as $m) {
	43	+ if (!is_null(_request($m . '_check'))) {
	44	+ ecrire_meta($m, _request($m) ? 'oui' : 'non');
	45	+ }
	46	+ }
	47	+ foreach (['multi_objets', 'gerer_trad_objets'] as $m) {
	48	+ if (!is_null($v = _request($m))) {
	49	+ // join et enlever la valeur vide ''
	50	+ ecrire_meta($m, implode(',', array_diff($v, [''])));
	51	+ }
	52	+ }
	53	+
	54	+ if ($i = _request('langues_auth') and is_array($i)) {
	55	+ $i = array_unique(array_merge($i, explode(',', $GLOBALS['meta']['langues_utilisees'])));
	56	+ ecrire_meta('langues_multilingue', implode(',', $i));
	57	+ }
	58	+ $res['message_ok'] = _T('config_info_enregistree');
	59	+
	60	+ return $res;
61	61	}
62	62
63	63	/**
		@@ -67,13 +67,13 @@ discard block
		block discarded – undo
67	67	* @return string
68	68	*/
69	69	function table_supporte_lang($table_sql) {
70		- $trouver_table = charger_fonction('trouver_table', 'base');
71		- $desc = $trouver_table($table_sql);
72		- if (!$desc or !isset($desc['field']['lang'])) {
73		- return '';
74		- }
	70	+ $trouver_table = charger_fonction('trouver_table', 'base');
	71	+ $desc = $trouver_table($table_sql);
	72	+ if (!$desc or !isset($desc['field']['lang'])) {
	73	+ return '';
	74	+ }
75	75
76		- return ' ';
	76	+ return ' ';
77	77	}
78	78
79	79	/**
		@@ -83,67 +83,67 @@ discard block
		block discarded – undo
83	83	* @return string
84	84	*/
85	85	function table_supporte_trad($table_sql) {
86		- $trouver_table = charger_fonction('trouver_table', 'base');
87		- $desc = $trouver_table($table_sql);
88		- if (!$desc or !isset($desc['field']['id_trad'])) {
89		- return '';
90		- }
	86	+ $trouver_table = charger_fonction('trouver_table', 'base');
	87	+ $desc = $trouver_table($table_sql);
	88	+ if (!$desc or !isset($desc['field']['id_trad'])) {
	89	+ return '';
	90	+ }
91	91
92		- return ' ';
	92	+ return ' ';
93	93	}
94	94
95	95
96	96	function saisie_langues_utiles($name, $selection) {
97		- include_spip('inc/lang_liste');
98		- $langues = $GLOBALS['codes_langues'];
99		-
100		- $langues_installees = explode(',', $GLOBALS['meta']['langues_proposees']);
101		- $langues_trad = array_flip($langues_installees);
102		-
103		- $langues_bloquees = explode(',', $GLOBALS['meta']['langues_utilisees']);
104		-
105		- $res = '';
106		-
107		- $i = 0;
108		- foreach ($langues_bloquees as $code_langue) {
109		- $nom_langue = $langues[$code_langue];
110		- $res .= "<li class='choix "
111		- . alterner(++$i, 'odd', 'even')
112		- . (isset($langues_trad[$code_langue]) ? ' traduite' : '')
113		- . "'>"
114		- . "<input type='hidden' name='{$name}[]' value='$code_langue'>" // necessaire ...
115		- . "<input type='checkbox' name='{$name}[]' id='{$name}_$code_langue' value='$code_langue' checked='checked' disabled='disabled' />"
116		- . "<label for='{$name}_$code_langue'>" . $nom_langue . "   <span class='code_langue'>[$code_langue]</span></label>"
117		- . '</li>';
118		- }
119		-
120		- if ($res) {
121		- $res = "<ul id='langues_bloquees'>" . $res . "</ul><div class='nettoyeur'></div>";
122		- }
123		-
124		- $res .= "<ul id='langues_proposees'>";
125		-
126		- $i = 0;
127		- $langues_bloquees = array_flip($langues_bloquees);
128		- foreach ($langues as $code_langue => $nom_langue) {
129		- if (!isset($langues_bloquees[$code_langue])) {
130		- $checked = (in_array($code_langue, $selection) ? ' checked="checked"' : '');
131		- $res .= "<li class='choix "
132		- . alterner(++$i, 'odd', 'even')
133		- . (isset($langues_trad[$code_langue]) ? ' traduite' : '')
134		- . "'>"
135		- . "<input type='checkbox' name='{$name}[]' id='{$name}_$code_langue' value='$code_langue'"
136		- . $checked
137		- . '/>'
138		- . "<label for='{$name}_$code_langue'"
139		- . ($checked ? " class='on'" : '')
140		- . '>'
141		- . $nom_langue . "   <span class='code_langue'>[$code_langue]</span></label>"
142		- . '</li>';
143		- }
144		- }
145		-
146		- $res .= "</ul><div class='nettoyeur'></div>";
147		-
148		- return $res;
	97	+ include_spip('inc/lang_liste');
	98	+ $langues = $GLOBALS['codes_langues'];
	99	+
	100	+ $langues_installees = explode(',', $GLOBALS['meta']['langues_proposees']);
	101	+ $langues_trad = array_flip($langues_installees);
	102	+
	103	+ $langues_bloquees = explode(',', $GLOBALS['meta']['langues_utilisees']);
	104	+
	105	+ $res = '';
	106	+
	107	+ $i = 0;
	108	+ foreach ($langues_bloquees as $code_langue) {
	109	+ $nom_langue = $langues[$code_langue];
	110	+ $res .= "<li class='choix "
	111	+ . alterner(++$i, 'odd', 'even')
	112	+ . (isset($langues_trad[$code_langue]) ? ' traduite' : '')
	113	+ . "'>"
	114	+ . "<input type='hidden' name='{$name}[]' value='$code_langue'>" // necessaire ...
	115	+ . "<input type='checkbox' name='{$name}[]' id='{$name}_$code_langue' value='$code_langue' checked='checked' disabled='disabled' />"
	116	+ . "<label for='{$name}_$code_langue'>" . $nom_langue . "   <span class='code_langue'>[$code_langue]</span></label>"
	117	+ . '</li>';
	118	+ }
	119	+
	120	+ if ($res) {
	121	+ $res = "<ul id='langues_bloquees'>" . $res . "</ul><div class='nettoyeur'></div>";
	122	+ }
	123	+
	124	+ $res .= "<ul id='langues_proposees'>";
	125	+
	126	+ $i = 0;
	127	+ $langues_bloquees = array_flip($langues_bloquees);
	128	+ foreach ($langues as $code_langue => $nom_langue) {
	129	+ if (!isset($langues_bloquees[$code_langue])) {
	130	+ $checked = (in_array($code_langue, $selection) ? ' checked="checked"' : '');
	131	+ $res .= "<li class='choix "
	132	+ . alterner(++$i, 'odd', 'even')
	133	+ . (isset($langues_trad[$code_langue]) ? ' traduite' : '')
	134	+ . "'>"
	135	+ . "<input type='checkbox' name='{$name}[]' id='{$name}_$code_langue' value='$code_langue'"
	136	+ . $checked
	137	+ . '/>'
	138	+ . "<label for='{$name}_$code_langue'"
	139	+ . ($checked ? " class='on'" : '')
	140	+ . '>'
	141	+ . $nom_langue . "   <span class='code_langue'>[$code_langue]</span></label>"
	142	+ . '</li>';
	143	+ }
	144	+ }
	145	+
	146	+ $res .= "</ul><div class='nettoyeur'></div>";
	147	+
	148	+ return $res;
149	149	}

spip / SPIP

Push — master ( 6c9939...4ccfda )

Status

Category

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