Inspection of "Améliorer action_purger_dist() quand on vide le ca..." - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 254e8f...14072a )

by cam

created 2020-02-16 12:45 UTC

Status

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

@@ -16,7 +16,7 @@  discard block
 block discarded – undo
  * @package SPIP\Core\Cache
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -39,57 +39,57 @@  discard block
 block discarded – undo
  *     de l'action sécurisée.
  */
 function action_purger_dist($arg = null) {
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
 
-	include_spip('inc/invalideur');
+    include_spip('inc/invalideur');
 
-	spip_log("purger $arg");
+    spip_log("purger $arg");
 
-	switch ($arg) {
-		case 'inhibe_cache':
-			// inhiber le cache pendant 24h
-			ecrire_meta('cache_inhib', $_SERVER['REQUEST_TIME'] + 24 * 3600);
-			break;
-		case 'reactive_cache':
-			effacer_meta('cache_inhib');
-			break;
+    switch ($arg) {
+        case 'inhibe_cache':
+            // inhiber le cache pendant 24h
+            ecrire_meta('cache_inhib', $_SERVER['REQUEST_TIME'] + 24 * 3600);
+            break;
+        case 'reactive_cache':
+            effacer_meta('cache_inhib');
+            break;
 
-		case 'cache':
-			supprime_invalideurs();
-			@spip_unlink(_CACHE_RUBRIQUES);
-			@spip_unlink(_CACHE_CHEMIN);
-			@spip_unlink(_DIR_TMP . "plugin_xml_cache.gz");
-			// on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
-			// _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
-			// pour eviter des problemes de concurence
-			// cf http://core.spip.net/issues/2989
-			//@spip_unlink(_CACHE_PIPELINES);
-			//@spip_unlink(_CACHE_PLUGINS_PATH);
-			//@spip_unlink(_CACHE_PLUGINS_FCT);
-			@spip_unlink(_CACHE_PLUGINS_OPT);
-			purger_repertoire(_DIR_CACHE, array('subdir' => true));
-			purger_repertoire(_DIR_AIDE);
-			purger_repertoire(_DIR_VAR . 'cache-css');
-			purger_repertoire(_DIR_VAR . 'cache-js');
-			break;
+        case 'cache':
+            supprime_invalideurs();
+            @spip_unlink(_CACHE_RUBRIQUES);
+            @spip_unlink(_CACHE_CHEMIN);
+            @spip_unlink(_DIR_TMP . "plugin_xml_cache.gz");
+            // on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
+            // _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
+            // pour eviter des problemes de concurence
+            // cf http://core.spip.net/issues/2989
+            //@spip_unlink(_CACHE_PIPELINES);
+            //@spip_unlink(_CACHE_PLUGINS_PATH);
+            //@spip_unlink(_CACHE_PLUGINS_FCT);
+            @spip_unlink(_CACHE_PLUGINS_OPT);
+            purger_repertoire(_DIR_CACHE, array('subdir' => true));
+            purger_repertoire(_DIR_AIDE);
+            purger_repertoire(_DIR_VAR . 'cache-css');
+            purger_repertoire(_DIR_VAR . 'cache-js');
+            break;
 
-		case 'squelettes':
-			purger_repertoire(_DIR_SKELS);
-			break;
+        case 'squelettes':
+            purger_repertoire(_DIR_SKELS);
+            break;
 
-		case 'vignettes':
-			purger_repertoire(_DIR_VAR . 'cache-gd2', array('subdir' => true));
-			purger_repertoire(_DIR_VAR . 'cache-texte', array('subdir' => true));
-			purger_repertoire(_DIR_VAR . 'cache-vignettes', array('subdir' => true));
-			purger_repertoire(_DIR_VAR . 'cache-TeX', array('subdir' => true));
-			supprime_invalideurs();
-			purger_repertoire(_DIR_CACHE, array('subdir' => true));
-			break;
-	}
+        case 'vignettes':
+            purger_repertoire(_DIR_VAR . 'cache-gd2', array('subdir' => true));
+            purger_repertoire(_DIR_VAR . 'cache-texte', array('subdir' => true));
+            purger_repertoire(_DIR_VAR . 'cache-vignettes', array('subdir' => true));
+            purger_repertoire(_DIR_VAR . 'cache-TeX', array('subdir' => true));
+            supprime_invalideurs();
+            purger_repertoire(_DIR_CACHE, array('subdir' => true));
+            break;
+    }
 
-	// le faire savoir aux plugins
-	pipeline('trig_purger', $arg);
+    // le faire savoir aux plugins
+    pipeline('trig_purger', $arg);
 }

Please login to merge, or discard this patch.

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

@@ -61,7 +61,7 @@  discard block
 block discarded – undo
 			supprime_invalideurs();
 			@spip_unlink(_CACHE_RUBRIQUES);
 			@spip_unlink(_CACHE_CHEMIN);
-			@spip_unlink(_DIR_TMP . "plugin_xml_cache.gz");
+			@spip_unlink(_DIR_TMP."plugin_xml_cache.gz");
 			// on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
 			// _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
 			// pour eviter des problemes de concurence
@@ -72,8 +72,8 @@  discard block
 block discarded – undo
 			@spip_unlink(_CACHE_PLUGINS_OPT);
 			purger_repertoire(_DIR_CACHE, array('subdir' => true));
 			purger_repertoire(_DIR_AIDE);
-			purger_repertoire(_DIR_VAR . 'cache-css');
-			purger_repertoire(_DIR_VAR . 'cache-js');
+			purger_repertoire(_DIR_VAR.'cache-css');
+			purger_repertoire(_DIR_VAR.'cache-js');
 			break;
 
 		case 'squelettes':
@@ -81,10 +81,10 @@  discard block
 block discarded – undo
 			break;
 
 		case 'vignettes':
-			purger_repertoire(_DIR_VAR . 'cache-gd2', array('subdir' => true));
-			purger_repertoire(_DIR_VAR . 'cache-texte', array('subdir' => true));
-			purger_repertoire(_DIR_VAR . 'cache-vignettes', array('subdir' => true));
-			purger_repertoire(_DIR_VAR . 'cache-TeX', array('subdir' => true));
+			purger_repertoire(_DIR_VAR.'cache-gd2', array('subdir' => true));
+			purger_repertoire(_DIR_VAR.'cache-texte', array('subdir' => true));
+			purger_repertoire(_DIR_VAR.'cache-vignettes', array('subdir' => true));
+			purger_repertoire(_DIR_VAR.'cache-TeX', array('subdir' => true));
 			supprime_invalideurs();
 			purger_repertoire(_DIR_CACHE, array('subdir' => true));
 			break;

Please login to merge, or discard this patch.

		@@ -16,7 +16,7 @@ discard block
		block discarded – undo
16	16	* @package SPIP\Core\Cache
17	17	**/
18	18	if (!defined('_ECRIRE_INC_VERSION')) {
19		- return;
	19	+ return;
20	20	}
21	21
22	22	/**
		@@ -39,57 +39,57 @@ discard block
		block discarded – undo
39	39	* de l'action sécurisée.
40	40	*/
41	41	function action_purger_dist($arg = null) {
42		- if (is_null($arg)) {
43		- $securiser_action = charger_fonction('securiser_action', 'inc');
44		- $arg = $securiser_action();
45		- }
	42	+ if (is_null($arg)) {
	43	+ $securiser_action = charger_fonction('securiser_action', 'inc');
	44	+ $arg = $securiser_action();
	45	+ }
46	46
47		- include_spip('inc/invalideur');
	47	+ include_spip('inc/invalideur');
48	48
49		- spip_log("purger $arg");
	49	+ spip_log("purger $arg");
50	50
51		- switch ($arg) {
52		- case 'inhibe_cache':
53		- // inhiber le cache pendant 24h
54		- ecrire_meta('cache_inhib', $_SERVER['REQUEST_TIME'] + 24 * 3600);
55		- break;
56		- case 'reactive_cache':
57		- effacer_meta('cache_inhib');
58		- break;
	51	+ switch ($arg) {
	52	+ case 'inhibe_cache':
	53	+ // inhiber le cache pendant 24h
	54	+ ecrire_meta('cache_inhib', $_SERVER['REQUEST_TIME'] + 24 * 3600);
	55	+ break;
	56	+ case 'reactive_cache':
	57	+ effacer_meta('cache_inhib');
	58	+ break;
59	59
60		- case 'cache':
61		- supprime_invalideurs();
62		- @spip_unlink(_CACHE_RUBRIQUES);
63		- @spip_unlink(_CACHE_CHEMIN);
64		- @spip_unlink(_DIR_TMP . "plugin_xml_cache.gz");
65		- // on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
66		- // _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
67		- // pour eviter des problemes de concurence
68		- // cf http://core.spip.net/issues/2989
69		- //@spip_unlink(_CACHE_PIPELINES);
70		- //@spip_unlink(_CACHE_PLUGINS_PATH);
71		- //@spip_unlink(_CACHE_PLUGINS_FCT);
72		- @spip_unlink(_CACHE_PLUGINS_OPT);
73		- purger_repertoire(_DIR_CACHE, array('subdir' => true));
74		- purger_repertoire(_DIR_AIDE);
75		- purger_repertoire(_DIR_VAR . 'cache-css');
76		- purger_repertoire(_DIR_VAR . 'cache-js');
77		- break;
	60	+ case 'cache':
	61	+ supprime_invalideurs();
	62	+ @spip_unlink(_CACHE_RUBRIQUES);
	63	+ @spip_unlink(_CACHE_CHEMIN);
	64	+ @spip_unlink(_DIR_TMP . "plugin_xml_cache.gz");
	65	+ // on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
	66	+ // _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
	67	+ // pour eviter des problemes de concurence
	68	+ // cf http://core.spip.net/issues/2989
	69	+ //@spip_unlink(_CACHE_PIPELINES);
	70	+ //@spip_unlink(_CACHE_PLUGINS_PATH);
	71	+ //@spip_unlink(_CACHE_PLUGINS_FCT);
	72	+ @spip_unlink(_CACHE_PLUGINS_OPT);
	73	+ purger_repertoire(_DIR_CACHE, array('subdir' => true));
	74	+ purger_repertoire(_DIR_AIDE);
	75	+ purger_repertoire(_DIR_VAR . 'cache-css');
	76	+ purger_repertoire(_DIR_VAR . 'cache-js');
	77	+ break;
78	78
79		- case 'squelettes':
80		- purger_repertoire(_DIR_SKELS);
81		- break;
	79	+ case 'squelettes':
	80	+ purger_repertoire(_DIR_SKELS);
	81	+ break;
82	82
83		- case 'vignettes':
84		- purger_repertoire(_DIR_VAR . 'cache-gd2', array('subdir' => true));
85		- purger_repertoire(_DIR_VAR . 'cache-texte', array('subdir' => true));
86		- purger_repertoire(_DIR_VAR . 'cache-vignettes', array('subdir' => true));
87		- purger_repertoire(_DIR_VAR . 'cache-TeX', array('subdir' => true));
88		- supprime_invalideurs();
89		- purger_repertoire(_DIR_CACHE, array('subdir' => true));
90		- break;
91		- }
	83	+ case 'vignettes':
	84	+ purger_repertoire(_DIR_VAR . 'cache-gd2', array('subdir' => true));
	85	+ purger_repertoire(_DIR_VAR . 'cache-texte', array('subdir' => true));
	86	+ purger_repertoire(_DIR_VAR . 'cache-vignettes', array('subdir' => true));
	87	+ purger_repertoire(_DIR_VAR . 'cache-TeX', array('subdir' => true));
	88	+ supprime_invalideurs();
	89	+ purger_repertoire(_DIR_CACHE, array('subdir' => true));
	90	+ break;
	91	+ }
92	92
93		- // le faire savoir aux plugins
94		- pipeline('trig_purger', $arg);
	93	+ // le faire savoir aux plugins
	94	+ pipeline('trig_purger', $arg);
95	95	}

		@@ -61,7 +61,7 @@ discard block
		block discarded – undo
61	61	supprime_invalideurs();
62	62	@spip_unlink(_CACHE_RUBRIQUES);
63	63	@spip_unlink(_CACHE_CHEMIN);
64		- @spip_unlink(_DIR_TMP . "plugin_xml_cache.gz");
	64	+ @spip_unlink(_DIR_TMP."plugin_xml_cache.gz");
65	65	// on ne supprime que _CACHE_PLUGINS_OPT qui declenche la reconstruction des 3
66	66	// _CACHE_PIPELINES _CACHE_PLUGINS_PATH et _CACHE_PLUGINS_FCT
67	67	// pour eviter des problemes de concurence
		@@ -72,8 +72,8 @@ discard block
		block discarded – undo
72	72	@spip_unlink(_CACHE_PLUGINS_OPT);
73	73	purger_repertoire(_DIR_CACHE, array('subdir' => true));
74	74	purger_repertoire(_DIR_AIDE);
75		- purger_repertoire(_DIR_VAR . 'cache-css');
76		- purger_repertoire(_DIR_VAR . 'cache-js');
	75	+ purger_repertoire(_DIR_VAR.'cache-css');
	76	+ purger_repertoire(_DIR_VAR.'cache-js');
77	77	break;
78	78
79	79	case 'squelettes':
		@@ -81,10 +81,10 @@ discard block
		block discarded – undo
81	81	break;
82	82
83	83	case 'vignettes':
84		- purger_repertoire(_DIR_VAR . 'cache-gd2', array('subdir' => true));
85		- purger_repertoire(_DIR_VAR . 'cache-texte', array('subdir' => true));
86		- purger_repertoire(_DIR_VAR . 'cache-vignettes', array('subdir' => true));
87		- purger_repertoire(_DIR_VAR . 'cache-TeX', array('subdir' => true));
	84	+ purger_repertoire(_DIR_VAR.'cache-gd2', array('subdir' => true));
	85	+ purger_repertoire(_DIR_VAR.'cache-texte', array('subdir' => true));
	86	+ purger_repertoire(_DIR_VAR.'cache-vignettes', array('subdir' => true));
	87	+ purger_repertoire(_DIR_VAR.'cache-TeX', array('subdir' => true));
88	88	supprime_invalideurs();
89	89	purger_repertoire(_DIR_CACHE, array('subdir' => true));
90	90	break;

spip / SPIP

Push — master ( 254e8f...14072a )

Status

Category

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

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