Inspection of "Fix la navigation par initiale sur les listes aute..." - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2e2223...f512ef )

by cam

created 2022-04-28 13:16 UTC

Status

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

@@ -16,7 +16,7 @@  discard block
 block discarded – undo
  * @package SPIP\Core\Fonctions
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -29,29 +29,29 @@  discard block
 block discarded – undo
  * @param bool $left
  */
 function critere_compteur_articles_filtres_dist($idb, &$boucles, $crit, $left = false) {
-	$boucle = &$boucles[$idb];
+    $boucle = &$boucles[$idb];
 
-	$_statut = calculer_liste($crit->param[0], [], $boucles, $boucle->id_parent);
+    $_statut = calculer_liste($crit->param[0], [], $boucles, $boucle->id_parent);
 
-	$not = '';
-	if ($crit->not) {
-		$not = ", 'NOT'";
-	}
-	$boucle->from['LAA'] = 'spip_auteurs_liens';
-	$boucle->from_type['LAA'] = 'left';
-	$boucle->join['LAA'] = ["'auteurs'", "'id_auteur'", "'id_auteur'", "'LAA.objet=\'article\''"];
+    $not = '';
+    if ($crit->not) {
+        $not = ", 'NOT'";
+    }
+    $boucle->from['LAA'] = 'spip_auteurs_liens';
+    $boucle->from_type['LAA'] = 'left';
+    $boucle->join['LAA'] = ["'auteurs'", "'id_auteur'", "'id_auteur'", "'LAA.objet=\'article\''"];
 
-	$boucle->from['articles'] = 'spip_articles';
-	$boucle->from_type['articles'] = 'left';
-	$boucle->join['articles'] = [
-		"'LAA'",
-		"'id_article'",
-		"'id_objet'",
-		"'(articles.statut IS NULL OR '.sql_in_quote('articles.statut',[$_statut]$not).')'"
-	];
+    $boucle->from['articles'] = 'spip_articles';
+    $boucle->from_type['articles'] = 'left';
+    $boucle->join['articles'] = [
+        "'LAA'",
+        "'id_article'",
+        "'id_objet'",
+        "'(articles.statut IS NULL OR '.sql_in_quote('articles.statut',[$_statut]$not).')'"
+    ];
 
-	$boucle->select[] = 'COUNT(articles.id_article) AS compteur_articles';
-	$boucle->group[] = 'auteurs.id_auteur';
+    $boucle->select[] = 'COUNT(articles.id_article) AS compteur_articles';
+    $boucle->group[] = 'auteurs.id_auteur';
 }
 
 /**
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
  *     Pile complétée par le code à générer
  */
 function balise_COMPTEUR_ARTICLES_dist($p) {
-	return rindex_pile($p, 'compteur_articles', 'compteur_articles_filtres');
+    return rindex_pile($p, 'compteur_articles', 'compteur_articles_filtres');
 }
 
 
@@ -85,44 +85,44 @@  discard block
 block discarded – undo
  * @return string
  */
 function afficher_initiale($url, $initiale, $compteur, $debut, $pas) {
-	static $memo = null;
-	static $res = [];
-	$out = '';
-	if (
-		!$memo
-		or (!$initiale and !$url)
-		or ($initiale !== $memo['initiale'])
-	) {
-		$newcompt = intval(floor(($compteur - 1) / $pas) * $pas);
-		// si fin de la pagination et une seule entree, ne pas l'afficher, ca ne sert a rien
-		if (!$initiale and !$url and !$memo['compteur']) {
-			$memo = null;
-		}
-		if ($memo) {
-			$on = (($memo['compteur'] <= $debut)
-				and (
-					$newcompt > $debut or ($newcompt == $debut and $newcompt == $memo['compteur'])
-				));
-			$res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . "</li>";
-		}
-		if ($initiale) {
-			$memo = [
-				'entree' => isset($memo['entree']) ? $memo['entree'] + 1 : 0,
-				'initiale' => $initiale,
-				'url' => parametre_url($url, 'i', $initiale),
-				'compteur' => $newcompt
-			];
-		}
-	}
-	if (!$initiale and !$url) {
-		if ((is_countable($res) ? count($res) : 0) > 1) {
-			$out = "<ul class='pagination-items'>" . implode(' ', $res) . "</ul>";
-		}
-		$memo = null;
-		$res = [];
-	}
+    static $memo = null;
+    static $res = [];
+    $out = '';
+    if (
+        !$memo
+        or (!$initiale and !$url)
+        or ($initiale !== $memo['initiale'])
+    ) {
+        $newcompt = intval(floor(($compteur - 1) / $pas) * $pas);
+        // si fin de la pagination et une seule entree, ne pas l'afficher, ca ne sert a rien
+        if (!$initiale and !$url and !$memo['compteur']) {
+            $memo = null;
+        }
+        if ($memo) {
+            $on = (($memo['compteur'] <= $debut)
+                and (
+                    $newcompt > $debut or ($newcompt == $debut and $newcompt == $memo['compteur'])
+                ));
+            $res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . "</li>";
+        }
+        if ($initiale) {
+            $memo = [
+                'entree' => isset($memo['entree']) ? $memo['entree'] + 1 : 0,
+                'initiale' => $initiale,
+                'url' => parametre_url($url, 'i', $initiale),
+                'compteur' => $newcompt
+            ];
+        }
+    }
+    if (!$initiale and !$url) {
+        if ((is_countable($res) ? count($res) : 0) > 1) {
+            $out = "<ul class='pagination-items'>" . implode(' ', $res) . "</ul>";
+        }
+        $memo = null;
+        $res = [];
+    }
 
-	return $out;
+    return $out;
 }
 
 /**
@@ -141,24 +141,24 @@  discard block
 block discarded – undo
  * @return string
  */
 function auteur_lien_messagerie($id_auteur, $en_ligne, $statut, $imessage, $email = '') {
-	static $time = null;
-	if (!in_array($statut, ['0minirezo', '1comite'])) {
-		return '';
-	}
+    static $time = null;
+    if (!in_array($statut, ['0minirezo', '1comite'])) {
+        return '';
+    }
 
-	if (is_null($time)) {
-		$time = time();
-	}
-	$parti = (($time - strtotime($en_ligne)) > 15 * 60);
+    if (is_null($time)) {
+        $time = time();
+    }
+    $parti = (($time - strtotime($en_ligne)) > 15 * 60);
 
-	if (
-		$imessage != 'non' and !$parti // historique : est-ce que ca a encore un sens de limiter vu qu'on a la notification par email ?
-		and $GLOBALS['meta']['messagerie_agenda'] != 'non'
-	) {
-		return parametre_url(parametre_url(generer_url_ecrire('message_edit', 'new=oui'), 'to', $id_auteur), 'redirect', self());
-	} elseif (strlen($email) and autoriser('voir', 'auteur', $id_auteur)) {
-		return 'mailto:' . $email;
-	} else {
-		return '';
-	}
+    if (
+        $imessage != 'non' and !$parti // historique : est-ce que ca a encore un sens de limiter vu qu'on a la notification par email ?
+        and $GLOBALS['meta']['messagerie_agenda'] != 'non'
+    ) {
+        return parametre_url(parametre_url(generer_url_ecrire('message_edit', 'new=oui'), 'to', $id_auteur), 'redirect', self());
+    } elseif (strlen($email) and autoriser('voir', 'auteur', $id_auteur)) {
+        return 'mailto:' . $email;
+    } else {
+        return '';
+    }
 }

Please login to merge, or discard this patch.

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

@@ -103,7 +103,7 @@  discard block
 block discarded – undo
 				and (
 					$newcompt > $debut or ($newcompt == $debut and $newcompt == $memo['compteur'])
 				));
-			$res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . "</li>";
+			$res[] = "<li class='pagination-item'>".lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination')."</li>";
 		}
 		if ($initiale) {
 			$memo = [
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
 	}
 	if (!$initiale and !$url) {
 		if ((is_countable($res) ? count($res) : 0) > 1) {
-			$out = "<ul class='pagination-items'>" . implode(' ', $res) . "</ul>";
+			$out = "<ul class='pagination-items'>".implode(' ', $res)."</ul>";
 		}
 		$memo = null;
 		$res = [];
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
 	) {
 		return parametre_url(parametre_url(generer_url_ecrire('message_edit', 'new=oui'), 'to', $id_auteur), 'redirect', self());
 	} elseif (strlen($email) and autoriser('voir', 'auteur', $id_auteur)) {
-		return 'mailto:' . $email;
+		return 'mailto:'.$email;
 	} else {
 		return '';
 	}

Please login to merge, or discard this patch.

		@@ -103,7 +103,7 @@ discard block
		block discarded – undo
103	103	and (
104	104	$newcompt > $debut or ($newcompt == $debut and $newcompt == $memo['compteur'])
105	105	));
106		- $res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . "</li>";
	106	+ $res[] = "<li class='pagination-item'>".lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination')."</li>";
107	107	}
108	108	if ($initiale) {
109	109	$memo = [
		@@ -116,7 +116,7 @@ discard block
		block discarded – undo
116	116	}
117	117	if (!$initiale and !$url) {
118	118	if ((is_countable($res) ? count($res) : 0) > 1) {
119		- $out = "<ul class='pagination-items'>" . implode(' ', $res) . "</ul>";
	119	+ $out = "<ul class='pagination-items'>".implode(' ', $res)."</ul>";
120	120	}
121	121	$memo = null;
122	122	$res = [];
		@@ -157,7 +157,7 @@ discard block
		block discarded – undo
157	157	) {
158	158	return parametre_url(parametre_url(generer_url_ecrire('message_edit', 'new=oui'), 'to', $id_auteur), 'redirect', self());
159	159	} elseif (strlen($email) and autoriser('voir', 'auteur', $id_auteur)) {
160		- return 'mailto:' . $email;
	160	+ return 'mailto:'.$email;
161	161	} else {
162	162	return '';
163	163	}

		@@ -16,7 +16,7 @@ discard block
		block discarded – undo
16	16	* @package SPIP\Core\Fonctions
17	17	**/
18	18	if (!defined('_ECRIRE_INC_VERSION')) {
19		- return;
	19	+ return;
20	20	}
21	21
22	22	/**
		@@ -29,29 +29,29 @@ discard block
		block discarded – undo
29	29	* @param bool $left
30	30	*/
31	31	function critere_compteur_articles_filtres_dist($idb, &$boucles, $crit, $left = false) {
32		- $boucle = &$boucles[$idb];
	32	+ $boucle = &$boucles[$idb];
33	33
34		- $_statut = calculer_liste($crit->param[0], [], $boucles, $boucle->id_parent);
	34	+ $_statut = calculer_liste($crit->param[0], [], $boucles, $boucle->id_parent);
35	35
36		- $not = '';
37		- if ($crit->not) {
38		- $not = ", 'NOT'";
39		- }
40		- $boucle->from['LAA'] = 'spip_auteurs_liens';
41		- $boucle->from_type['LAA'] = 'left';
42		- $boucle->join['LAA'] = ["'auteurs'", "'id_auteur'", "'id_auteur'", "'LAA.objet=\'article\''"];
	36	+ $not = '';
	37	+ if ($crit->not) {
	38	+ $not = ", 'NOT'";
	39	+ }
	40	+ $boucle->from['LAA'] = 'spip_auteurs_liens';
	41	+ $boucle->from_type['LAA'] = 'left';
	42	+ $boucle->join['LAA'] = ["'auteurs'", "'id_auteur'", "'id_auteur'", "'LAA.objet=\'article\''"];
43	43
44		- $boucle->from['articles'] = 'spip_articles';
45		- $boucle->from_type['articles'] = 'left';
46		- $boucle->join['articles'] = [
47		- "'LAA'",
48		- "'id_article'",
49		- "'id_objet'",
50		- "'(articles.statut IS NULL OR '.sql_in_quote('articles.statut',[$_statut]$not).')'"
51		- ];
	44	+ $boucle->from['articles'] = 'spip_articles';
	45	+ $boucle->from_type['articles'] = 'left';
	46	+ $boucle->join['articles'] = [
	47	+ "'LAA'",
	48	+ "'id_article'",
	49	+ "'id_objet'",
	50	+ "'(articles.statut IS NULL OR '.sql_in_quote('articles.statut',[$_statut]$not).')'"
	51	+ ];
52	52
53		- $boucle->select[] = 'COUNT(articles.id_article) AS compteur_articles';
54		- $boucle->group[] = 'auteurs.id_auteur';
	53	+ $boucle->select[] = 'COUNT(articles.id_article) AS compteur_articles';
	54	+ $boucle->group[] = 'auteurs.id_auteur';
55	55	}
56	56
57	57	/**
		@@ -69,7 +69,7 @@ discard block
		block discarded – undo
69	69	* Pile complétée par le code à générer
70	70	*/
71	71	function balise_COMPTEUR_ARTICLES_dist($p) {
72		- return rindex_pile($p, 'compteur_articles', 'compteur_articles_filtres');
	72	+ return rindex_pile($p, 'compteur_articles', 'compteur_articles_filtres');
73	73	}
74	74
75	75
		@@ -85,44 +85,44 @@ discard block
		block discarded – undo
85	85	* @return string
86	86	*/
87	87	function afficher_initiale($url, $initiale, $compteur, $debut, $pas) {
88		- static $memo = null;
89		- static $res = [];
90		- $out = '';
91		- if (
92		- !$memo
93		- or (!$initiale and !$url)
94		- or ($initiale !== $memo['initiale'])
95		- ) {
96		- $newcompt = intval(floor(($compteur - 1) / $pas) * $pas);
97		- // si fin de la pagination et une seule entree, ne pas l'afficher, ca ne sert a rien
98		- if (!$initiale and !$url and !$memo['compteur']) {
99		- $memo = null;
100		- }
101		- if ($memo) {
102		- $on = (($memo['compteur'] <= $debut)
103		- and (
104		- $newcompt > $debut or ($newcompt == $debut and $newcompt == $memo['compteur'])
105		- ));
106		- $res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . "</li>";
107		- }
108		- if ($initiale) {
109		- $memo = [
110		- 'entree' => isset($memo['entree']) ? $memo['entree'] + 1 : 0,
111		- 'initiale' => $initiale,
112		- 'url' => parametre_url($url, 'i', $initiale),
113		- 'compteur' => $newcompt
114		- ];
115		- }
116		- }
117		- if (!$initiale and !$url) {
118		- if ((is_countable($res) ? count($res) : 0) > 1) {
119		- $out = "<ul class='pagination-items'>" . implode(' ', $res) . "</ul>";
120		- }
121		- $memo = null;
122		- $res = [];
123		- }
	88	+ static $memo = null;
	89	+ static $res = [];
	90	+ $out = '';
	91	+ if (
	92	+ !$memo
	93	+ or (!$initiale and !$url)
	94	+ or ($initiale !== $memo['initiale'])
	95	+ ) {
	96	+ $newcompt = intval(floor(($compteur - 1) / $pas) * $pas);
	97	+ // si fin de la pagination et une seule entree, ne pas l'afficher, ca ne sert a rien
	98	+ if (!$initiale and !$url and !$memo['compteur']) {
	99	+ $memo = null;
	100	+ }
	101	+ if ($memo) {
	102	+ $on = (($memo['compteur'] <= $debut)
	103	+ and (
	104	+ $newcompt > $debut or ($newcompt == $debut and $newcompt == $memo['compteur'])
	105	+ ));
	106	+ $res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . "</li>";
	107	+ }
	108	+ if ($initiale) {
	109	+ $memo = [
	110	+ 'entree' => isset($memo['entree']) ? $memo['entree'] + 1 : 0,
	111	+ 'initiale' => $initiale,
	112	+ 'url' => parametre_url($url, 'i', $initiale),
	113	+ 'compteur' => $newcompt
	114	+ ];
	115	+ }
	116	+ }
	117	+ if (!$initiale and !$url) {
	118	+ if ((is_countable($res) ? count($res) : 0) > 1) {
	119	+ $out = "<ul class='pagination-items'>" . implode(' ', $res) . "</ul>";
	120	+ }
	121	+ $memo = null;
	122	+ $res = [];
	123	+ }
124	124
125		- return $out;
	125	+ return $out;
126	126	}
127	127
128	128	/**
		@@ -141,24 +141,24 @@ discard block
		block discarded – undo
141	141	* @return string
142	142	*/
143	143	function auteur_lien_messagerie($id_auteur, $en_ligne, $statut, $imessage, $email = '') {
144		- static $time = null;
145		- if (!in_array($statut, ['0minirezo', '1comite'])) {
146		- return '';
147		- }
	144	+ static $time = null;
	145	+ if (!in_array($statut, ['0minirezo', '1comite'])) {
	146	+ return '';
	147	+ }
148	148
149		- if (is_null($time)) {
150		- $time = time();
151		- }
152		- $parti = (($time - strtotime($en_ligne)) > 15 * 60);
	149	+ if (is_null($time)) {
	150	+ $time = time();
	151	+ }
	152	+ $parti = (($time - strtotime($en_ligne)) > 15 * 60);
153	153
154		- if (
155		- $imessage != 'non' and !$parti // historique : est-ce que ca a encore un sens de limiter vu qu'on a la notification par email ?
156		- and $GLOBALS['meta']['messagerie_agenda'] != 'non'
157		- ) {
158		- return parametre_url(parametre_url(generer_url_ecrire('message_edit', 'new=oui'), 'to', $id_auteur), 'redirect', self());
159		- } elseif (strlen($email) and autoriser('voir', 'auteur', $id_auteur)) {
160		- return 'mailto:' . $email;
161		- } else {
162		- return '';
163		- }
	154	+ if (
	155	+ $imessage != 'non' and !$parti // historique : est-ce que ca a encore un sens de limiter vu qu'on a la notification par email ?
	156	+ and $GLOBALS['meta']['messagerie_agenda'] != 'non'
	157	+ ) {
	158	+ return parametre_url(parametre_url(generer_url_ecrire('message_edit', 'new=oui'), 'to', $id_auteur), 'redirect', self());
	159	+ } elseif (strlen($email) and autoriser('voir', 'auteur', $id_auteur)) {
	160	+ return 'mailto:' . $email;
	161	+ } else {
	162	+ return '';
	163	+ }
164	164	}

spip / SPIP

Push — master ( 2e2223...f512ef )

Status

Category

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

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