Inspection of "Fix un bug sur l'autosave qui faisait parfois reve..." - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9be2d3...04cde7 )

by cam

created 2021-05-27 09:18 UTC

Status

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

@@ -17,7 +17,7 @@  discard block
 block discarded – undo
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
  * @return string
  */
 function autosave_clean_value($val) {
-	return stripslashes(urldecode($val));
+    return stripslashes(urldecode($val));
 }
 
 /**
@@ -38,56 +38,56 @@  discard block
 block discarded – undo
  * @return array
  */
 function cvtautosave_formulaire_charger($flux) {
-	if (is_array($flux['data'])
-		and isset($flux['data']['_autosave_id'])
-		and $cle_autosave = $flux['data']['_autosave_id']
-	) {
-
-		$form = $flux['args']['form'];
-		$je_suis_poste = $flux['args']['je_suis_poste'];
-
-		$cle_autosave = serialize($cle_autosave);
-		$cle_autosave = $form . "_" . md5($cle_autosave);
-
-		// si on a un backup en session et qu'on est au premier chargement, non poste
-		// on restitue les donnees
-		if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])
-			and !$je_suis_poste
-		) {
-			parse_str($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave], $vars);
-			foreach ($vars as $key => $val) {
-				if (isset($flux['data'][$key])) {
-					$flux['data'][$key] = (is_string($val) ? autosave_clean_value($val) : array_map('autosave_clean_value',
-						$val));
-				}
-			}
-		}
-
-		// si on est dans le charger() qui suit le traiter(), l'autosave a normalement ete vide
-		// mais si il y a plusieurs sessions il peut y avoir concurrence et un retour de l'autosave
-		if ($je_suis_poste and _request('autosave') === $cle_autosave and function_exists('terminer_actualiser_sessions')) {
-			terminer_actualiser_sessions();
-			// et verifions si jamais l'autosave a fait un come back, dans ce cas on le revide
-			if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])) {
-				session_set('session_autosave_' . $cle_autosave, null);
-				// en court sleep pour etre certain que la concurrence est finie
-				sleep(1);
-				terminer_actualiser_sessions();
-			}
-		}
-
-
-		/**
-		 * Envoyer le input hidden et le bout de js qui l'utilisera
-		 */
-		$flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
-			. '<script type="text/javascript">/*<![CDATA[*/if (window.jQuery) jQuery(function(){
+    if (is_array($flux['data'])
+        and isset($flux['data']['_autosave_id'])
+        and $cle_autosave = $flux['data']['_autosave_id']
+    ) {
+
+        $form = $flux['args']['form'];
+        $je_suis_poste = $flux['args']['je_suis_poste'];
+
+        $cle_autosave = serialize($cle_autosave);
+        $cle_autosave = $form . "_" . md5($cle_autosave);
+
+        // si on a un backup en session et qu'on est au premier chargement, non poste
+        // on restitue les donnees
+        if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])
+            and !$je_suis_poste
+        ) {
+            parse_str($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave], $vars);
+            foreach ($vars as $key => $val) {
+                if (isset($flux['data'][$key])) {
+                    $flux['data'][$key] = (is_string($val) ? autosave_clean_value($val) : array_map('autosave_clean_value',
+                        $val));
+                }
+            }
+        }
+
+        // si on est dans le charger() qui suit le traiter(), l'autosave a normalement ete vide
+        // mais si il y a plusieurs sessions il peut y avoir concurrence et un retour de l'autosave
+        if ($je_suis_poste and _request('autosave') === $cle_autosave and function_exists('terminer_actualiser_sessions')) {
+            terminer_actualiser_sessions();
+            // et verifions si jamais l'autosave a fait un come back, dans ce cas on le revide
+            if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])) {
+                session_set('session_autosave_' . $cle_autosave, null);
+                // en court sleep pour etre certain que la concurrence est finie
+                sleep(1);
+                terminer_actualiser_sessions();
+            }
+        }
+
+
+        /**
+         * Envoyer le input hidden et le bout de js qui l'utilisera
+         */
+        $flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
+            . '<script type="text/javascript">/*<![CDATA[*/if (window.jQuery) jQuery(function(){
 		  $("input.autosaveactive").closest("form:not(.autosaveon)").autosave({url:"' . $GLOBALS['meta']['adresse_site'] . '/"}).addClass("autosaveon");
 			});/*]]>*/</script>';
 
-	}
+    }
 
-	return $flux;
+    return $flux;
 }
 
 /**
@@ -101,33 +101,33 @@  discard block
 block discarded – undo
  * @return array
  */
 function cvtautosave_formulaire_traiter($flux) {
-	// si on poste 'autosave' c'est qu'on n'a plus besoin de sauvegarder :
-	// on elimine les donnees de la session
-	if ($cle_autosave = _request('autosave')) {
-		include_spip('inc/session');
-		session_set('session_autosave_' . $cle_autosave, null);
-	}
-
-	if (isset($GLOBALS['visiteur_session']) and $GLOBALS['visiteur_session']) {
-		// delai par defaut avant purge d'un backup de form : 72H
-		if (!defined('_AUTOSAVE_GB_DELAY')) {
-			define('_AUTOSAVE_GB_DELAY', 72 * 3600);
-		}
-		$time_too_old = time() - _AUTOSAVE_GB_DELAY;
-		// purger aussi toutes les vieilles autosave
-		$session = $GLOBALS['visiteur_session'];
-		foreach ($session as $k => $v) {
-			if (strncmp($k, 'session_autosave_', 17) == 0) {
-				$timestamp = 0;
-				if (preg_match(",&__timestamp=(\d+)$,", $v, $m)) {
-					$timestamp = intval($m[1]);
-				}
-				if ($timestamp < $time_too_old) {
-					session_set($k, null);
-				}
-			}
-		}
-	}
-
-	return $flux;
+    // si on poste 'autosave' c'est qu'on n'a plus besoin de sauvegarder :
+    // on elimine les donnees de la session
+    if ($cle_autosave = _request('autosave')) {
+        include_spip('inc/session');
+        session_set('session_autosave_' . $cle_autosave, null);
+    }
+
+    if (isset($GLOBALS['visiteur_session']) and $GLOBALS['visiteur_session']) {
+        // delai par defaut avant purge d'un backup de form : 72H
+        if (!defined('_AUTOSAVE_GB_DELAY')) {
+            define('_AUTOSAVE_GB_DELAY', 72 * 3600);
+        }
+        $time_too_old = time() - _AUTOSAVE_GB_DELAY;
+        // purger aussi toutes les vieilles autosave
+        $session = $GLOBALS['visiteur_session'];
+        foreach ($session as $k => $v) {
+            if (strncmp($k, 'session_autosave_', 17) == 0) {
+                $timestamp = 0;
+                if (preg_match(",&__timestamp=(\d+)$,", $v, $m)) {
+                    $timestamp = intval($m[1]);
+                }
+                if ($timestamp < $time_too_old) {
+                    session_set($k, null);
+                }
+            }
+        }
+    }
+
+    return $flux;
 }

Please login to merge, or discard this patch.

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

@@ -47,14 +47,14 @@  discard block
 block discarded – undo
 		$je_suis_poste = $flux['args']['je_suis_poste'];
 
 		$cle_autosave = serialize($cle_autosave);
-		$cle_autosave = $form . "_" . md5($cle_autosave);
+		$cle_autosave = $form."_".md5($cle_autosave);
 
 		// si on a un backup en session et qu'on est au premier chargement, non poste
 		// on restitue les donnees
-		if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])
+		if (isset($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave])
 			and !$je_suis_poste
 		) {
-			parse_str($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave], $vars);
+			parse_str($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave], $vars);
 			foreach ($vars as $key => $val) {
 				if (isset($flux['data'][$key])) {
 					$flux['data'][$key] = (is_string($val) ? autosave_clean_value($val) : array_map('autosave_clean_value',
@@ -68,8 +68,8 @@  discard block
 block discarded – undo
 		if ($je_suis_poste and _request('autosave') === $cle_autosave and function_exists('terminer_actualiser_sessions')) {
 			terminer_actualiser_sessions();
 			// et verifions si jamais l'autosave a fait un come back, dans ce cas on le revide
-			if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])) {
-				session_set('session_autosave_' . $cle_autosave, null);
+			if (isset($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave])) {
+				session_set('session_autosave_'.$cle_autosave, null);
 				// en court sleep pour etre certain que la concurrence est finie
 				sleep(1);
 				terminer_actualiser_sessions();
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
 		 */
 		$flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
 			. '<script type="text/javascript">/*<![CDATA[*/if (window.jQuery) jQuery(function(){
-		  $("input.autosaveactive").closest("form:not(.autosaveon)").autosave({url:"' . $GLOBALS['meta']['adresse_site'] . '/"}).addClass("autosaveon");
+		  $("input.autosaveactive").closest("form:not(.autosaveon)").autosave({url:"' . $GLOBALS['meta']['adresse_site'].'/"}).addClass("autosaveon");
 			});/*]]>*/</script>';
 
 	}
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
 	// on elimine les donnees de la session
 	if ($cle_autosave = _request('autosave')) {
 		include_spip('inc/session');
-		session_set('session_autosave_' . $cle_autosave, null);
+		session_set('session_autosave_'.$cle_autosave, null);
 	}
 
 	if (isset($GLOBALS['visiteur_session']) and $GLOBALS['visiteur_session']) {

Please login to merge, or discard this patch.

		@@ -17,7 +17,7 @@ discard block
		block discarded – undo
17	17	*/
18	18
19	19	if (!defined('_ECRIRE_INC_VERSION')) {
20		- return;
	20	+ return;
21	21	}
22	22
23	23	/**
		@@ -27,7 +27,7 @@ discard block
		block discarded – undo
27	27	* @return string
28	28	*/
29	29	function autosave_clean_value($val) {
30		- return stripslashes(urldecode($val));
	30	+ return stripslashes(urldecode($val));
31	31	}
32	32
33	33	/**
		@@ -38,56 +38,56 @@ discard block
		block discarded – undo
38	38	* @return array
39	39	*/
40	40	function cvtautosave_formulaire_charger($flux) {
41		- if (is_array($flux['data'])
42		- and isset($flux['data']['_autosave_id'])
43		- and $cle_autosave = $flux['data']['_autosave_id']
44		- ) {
45		-
46		- $form = $flux['args']['form'];
47		- $je_suis_poste = $flux['args']['je_suis_poste'];
48		-
49		- $cle_autosave = serialize($cle_autosave);
50		- $cle_autosave = $form . "_" . md5($cle_autosave);
51		-
52		- // si on a un backup en session et qu'on est au premier chargement, non poste
53		- // on restitue les donnees
54		- if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])
55		- and !$je_suis_poste
56		- ) {
57		- parse_str($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave], $vars);
58		- foreach ($vars as $key => $val) {
59		- if (isset($flux['data'][$key])) {
60		- $flux['data'][$key] = (is_string($val) ? autosave_clean_value($val) : array_map('autosave_clean_value',
61		- $val));
62		- }
63		- }
64		- }
65		-
66		- // si on est dans le charger() qui suit le traiter(), l'autosave a normalement ete vide
67		- // mais si il y a plusieurs sessions il peut y avoir concurrence et un retour de l'autosave
68		- if ($je_suis_poste and _request('autosave') === $cle_autosave and function_exists('terminer_actualiser_sessions')) {
69		- terminer_actualiser_sessions();
70		- // et verifions si jamais l'autosave a fait un come back, dans ce cas on le revide
71		- if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])) {
72		- session_set('session_autosave_' . $cle_autosave, null);
73		- // en court sleep pour etre certain que la concurrence est finie
74		- sleep(1);
75		- terminer_actualiser_sessions();
76		- }
77		- }
78		-
79		-
80		- /**
81		- * Envoyer le input hidden et le bout de js qui l'utilisera
82		- */
83		- $flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
84		- . '<script type="text/javascript">/<![CDATA[/if (window.jQuery) jQuery(function(){
	41	+ if (is_array($flux['data'])
	42	+ and isset($flux['data']['_autosave_id'])
	43	+ and $cle_autosave = $flux['data']['_autosave_id']
	44	+ ) {
	45	+
	46	+ $form = $flux['args']['form'];
	47	+ $je_suis_poste = $flux['args']['je_suis_poste'];
	48	+
	49	+ $cle_autosave = serialize($cle_autosave);
	50	+ $cle_autosave = $form . "_" . md5($cle_autosave);
	51	+
	52	+ // si on a un backup en session et qu'on est au premier chargement, non poste
	53	+ // on restitue les donnees
	54	+ if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])
	55	+ and !$je_suis_poste
	56	+ ) {
	57	+ parse_str($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave], $vars);
	58	+ foreach ($vars as $key => $val) {
	59	+ if (isset($flux['data'][$key])) {
	60	+ $flux['data'][$key] = (is_string($val) ? autosave_clean_value($val) : array_map('autosave_clean_value',
	61	+ $val));
	62	+ }
	63	+ }
	64	+ }
	65	+
	66	+ // si on est dans le charger() qui suit le traiter(), l'autosave a normalement ete vide
	67	+ // mais si il y a plusieurs sessions il peut y avoir concurrence et un retour de l'autosave
	68	+ if ($je_suis_poste and _request('autosave') === $cle_autosave and function_exists('terminer_actualiser_sessions')) {
	69	+ terminer_actualiser_sessions();
	70	+ // et verifions si jamais l'autosave a fait un come back, dans ce cas on le revide
	71	+ if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])) {
	72	+ session_set('session_autosave_' . $cle_autosave, null);
	73	+ // en court sleep pour etre certain que la concurrence est finie
	74	+ sleep(1);
	75	+ terminer_actualiser_sessions();
	76	+ }
	77	+ }
	78	+
	79	+
	80	+ /**
	81	+ * Envoyer le input hidden et le bout de js qui l'utilisera
	82	+ */
	83	+ $flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
	84	+ . '<script type="text/javascript">/<![CDATA[/if (window.jQuery) jQuery(function(){
85	85	$("input.autosaveactive").closest("form:not(.autosaveon)").autosave({url:"' . $GLOBALS['meta']['adresse_site'] . '/"}).addClass("autosaveon");
86	86	});/]]>/</script>';
87	87
88		- }
	88	+ }
89	89
90		- return $flux;
	90	+ return $flux;
91	91	}
92	92
93	93	/**
		@@ -101,33 +101,33 @@ discard block
		block discarded – undo
101	101	* @return array
102	102	*/
103	103	function cvtautosave_formulaire_traiter($flux) {
104		- // si on poste 'autosave' c'est qu'on n'a plus besoin de sauvegarder :
105		- // on elimine les donnees de la session
106		- if ($cle_autosave = _request('autosave')) {
107		- include_spip('inc/session');
108		- session_set('session_autosave_' . $cle_autosave, null);
109		- }
110		-
111		- if (isset($GLOBALS['visiteur_session']) and $GLOBALS['visiteur_session']) {
112		- // delai par defaut avant purge d'un backup de form : 72H
113		- if (!defined('_AUTOSAVE_GB_DELAY')) {
114		- define('_AUTOSAVE_GB_DELAY', 72 * 3600);
115		- }
116		- $time_too_old = time() - _AUTOSAVE_GB_DELAY;
117		- // purger aussi toutes les vieilles autosave
118		- $session = $GLOBALS['visiteur_session'];
119		- foreach ($session as $k => $v) {
120		- if (strncmp($k, 'session_autosave_', 17) == 0) {
121		- $timestamp = 0;
122		- if (preg_match(",&__timestamp=(\d+)$,", $v, $m)) {
123		- $timestamp = intval($m[1]);
124		- }
125		- if ($timestamp < $time_too_old) {
126		- session_set($k, null);
127		- }
128		- }
129		- }
130		- }
131		-
132		- return $flux;
	104	+ // si on poste 'autosave' c'est qu'on n'a plus besoin de sauvegarder :
	105	+ // on elimine les donnees de la session
	106	+ if ($cle_autosave = _request('autosave')) {
	107	+ include_spip('inc/session');
	108	+ session_set('session_autosave_' . $cle_autosave, null);
	109	+ }
	110	+
	111	+ if (isset($GLOBALS['visiteur_session']) and $GLOBALS['visiteur_session']) {
	112	+ // delai par defaut avant purge d'un backup de form : 72H
	113	+ if (!defined('_AUTOSAVE_GB_DELAY')) {
	114	+ define('_AUTOSAVE_GB_DELAY', 72 * 3600);
	115	+ }
	116	+ $time_too_old = time() - _AUTOSAVE_GB_DELAY;
	117	+ // purger aussi toutes les vieilles autosave
	118	+ $session = $GLOBALS['visiteur_session'];
	119	+ foreach ($session as $k => $v) {
	120	+ if (strncmp($k, 'session_autosave_', 17) == 0) {
	121	+ $timestamp = 0;
	122	+ if (preg_match(",&__timestamp=(\d+)$,", $v, $m)) {
	123	+ $timestamp = intval($m[1]);
	124	+ }
	125	+ if ($timestamp < $time_too_old) {
	126	+ session_set($k, null);
	127	+ }
	128	+ }
	129	+ }
	130	+ }
	131	+
	132	+ return $flux;
133	133	}

		@@ -47,14 +47,14 @@ discard block
		block discarded – undo
47	47	$je_suis_poste = $flux['args']['je_suis_poste'];
48	48
49	49	$cle_autosave = serialize($cle_autosave);
50		- $cle_autosave = $form . "_" . md5($cle_autosave);
	50	+ $cle_autosave = $form."_".md5($cle_autosave);
51	51
52	52	// si on a un backup en session et qu'on est au premier chargement, non poste
53	53	// on restitue les donnees
54		- if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])
	54	+ if (isset($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave])
55	55	and !$je_suis_poste
56	56	) {
57		- parse_str($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave], $vars);
	57	+ parse_str($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave], $vars);
58	58	foreach ($vars as $key => $val) {
59	59	if (isset($flux['data'][$key])) {
60	60	$flux['data'][$key] = (is_string($val) ? autosave_clean_value($val) : array_map('autosave_clean_value',
		@@ -68,8 +68,8 @@ discard block
		block discarded – undo
68	68	if ($je_suis_poste and _request('autosave') === $cle_autosave and function_exists('terminer_actualiser_sessions')) {
69	69	terminer_actualiser_sessions();
70	70	// et verifions si jamais l'autosave a fait un come back, dans ce cas on le revide
71		- if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])) {
72		- session_set('session_autosave_' . $cle_autosave, null);
	71	+ if (isset($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave])) {
	72	+ session_set('session_autosave_'.$cle_autosave, null);
73	73	// en court sleep pour etre certain que la concurrence est finie
74	74	sleep(1);
75	75	terminer_actualiser_sessions();
		@@ -82,7 +82,7 @@ discard block
		block discarded – undo
82	82	*/
83	83	$flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
84	84	. '<script type="text/javascript">/<![CDATA[/if (window.jQuery) jQuery(function(){
85		- $("input.autosaveactive").closest("form:not(.autosaveon)").autosave({url:"' . $GLOBALS['meta']['adresse_site'] . '/"}).addClass("autosaveon");
	85	+ $("input.autosaveactive").closest("form:not(.autosaveon)").autosave({url:"' . $GLOBALS['meta']['adresse_site'].'/"}).addClass("autosaveon");
86	86	});/]]>/</script>';
87	87
88	88	}
		@@ -105,7 +105,7 @@ discard block
		block discarded – undo
105	105	// on elimine les donnees de la session
106	106	if ($cle_autosave = _request('autosave')) {
107	107	include_spip('inc/session');
108		- session_set('session_autosave_' . $cle_autosave, null);
	108	+ session_set('session_autosave_'.$cle_autosave, null);
109	109	}
110	110
111	111	if (isset($GLOBALS['visiteur_session']) and $GLOBALS['visiteur_session']) {

spip / SPIP

Push — master ( 9be2d3...04cde7 )

Status

Category

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

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