Inspection of "[Salvatore] [source:ecrire/lang/ ecrire] Export de..." - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c76770...babdec )

by cam

created 2023-02-28 00:27 UTC

Status

Indentation +17 added lines, -17 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,23 +27,23 @@  discard block
 block discarded – undo
  * @uses ajax_retour()
  **/
 function exec_plonger_dist() {
-	include_spip('inc/actions');
+    include_spip('inc/actions');
 
-	$rac = preg_replace(',[^\w\,/#&;-]+,', ' ', _request('rac'));
-	$id = intval(_request('id'));
-	$exclus = intval(_request('exclus'));
-	$col = intval(_request('col'));
-	$do = _request('do');
-	if (preg_match('/^\w*$/', $do)) {
-		if (!$do) {
-			$do = 'aff';
-		}
+    $rac = preg_replace(',[^\w\,/#&;-]+,', ' ', _request('rac'));
+    $id = intval(_request('id'));
+    $exclus = intval(_request('exclus'));
+    $col = intval(_request('col'));
+    $do = _request('do');
+    if (preg_match('/^\w*$/', $do)) {
+        if (!$do) {
+            $do = 'aff';
+        }
 
-		$plonger = charger_fonction('plonger', 'inc');
-		$r = $plonger($id, spip_htmlentities($rac), [], $col, $exclus, $do);
-	} else {
-		$r = '';
-	}
+        $plonger = charger_fonction('plonger', 'inc');
+        $r = $plonger($id, spip_htmlentities($rac), [], $col, $exclus, $do);
+    } else {
+        $r = '';
+    }
 
-	ajax_retour($r);
+    ajax_retour($r);
 }

Please login to merge, or discard this patch.

ecrire/exec/install.php 1 patch

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

@@ -17,7 +17,7 @@  discard block
 block discarded – undo
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/minipres');
@@ -42,27 +42,27 @@  discard block
 block discarded – undo
  *
  **/
 function exec_install_dist() {
-	$etape = _request('etape');
-	$deja = (_FILE_CONNECT and analyse_fichier_connection(_FILE_CONNECT));
+    $etape = _request('etape');
+    $deja = (_FILE_CONNECT and analyse_fichier_connection(_FILE_CONNECT));
 
-	// Si deja installe, on n'a plus le droit qu'a l'etape chmod
-	// pour chgt post-install ou aux etapes supplementaires
-	// de declaration de base externes.
-	// Mais alors il faut authentifier car ecrire/index.php l'a omis
+    // Si deja installe, on n'a plus le droit qu'a l'etape chmod
+    // pour chgt post-install ou aux etapes supplementaires
+    // de declaration de base externes.
+    // Mais alors il faut authentifier car ecrire/index.php l'a omis
 
-	if ($deja and in_array($etape, ['chmod', 'sup1', 'sup2'])) {
-		$auth = charger_fonction('auth', 'inc');
-		if (!$auth()) {
-			verifier_visiteur();
-			$deja = (!autoriser('configurer'));
-		}
-	}
-	if ($deja) {
-		// Rien a faire ici
-		echo minipres();
-	} else {
-		include_spip('base/create');
-		$fonc = charger_fonction("etape_$etape", 'install');
-		$fonc();
-	}
+    if ($deja and in_array($etape, ['chmod', 'sup1', 'sup2'])) {
+        $auth = charger_fonction('auth', 'inc');
+        if (!$auth()) {
+            verifier_visiteur();
+            $deja = (!autoriser('configurer'));
+        }
+    }
+    if ($deja) {
+        // Rien a faire ici
+        echo minipres();
+    } else {
+        include_spip('base/create');
+        $fonc = charger_fonction("etape_$etape", 'install');
+        $fonc();
+    }
 }

Please login to merge, or discard this patch.

ecrire/exec/upgrade.php 1 patch

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

@@ -17,7 +17,7 @@  discard block
 block discarded – undo
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -30,70 +30,70 @@  discard block
 block discarded – undo
  **/
 function exec_upgrade_dist() {
 
-	if (!_FILE_CONNECT) {
-		redirige_url_ecrire('install');
-	}
+    if (!_FILE_CONNECT) {
+        redirige_url_ecrire('install');
+    }
 
-	// Si reinstallation necessaire, message ad hoc
-	if (_request('reinstall') == 'oui') {
-		include_spip('inc/minipres');
-		$r = minipres(
-			_T('titre_page_upgrade'),
-			'<p><b>'
-			. _T('texte_nouvelle_version_spip_1')
-			. '</b><p> '
-			. _T(
-				'texte_nouvelle_version_spip_2',
-				['connect' => '<tt>' . _FILE_CONNECT . '</tt>']
-			)
-			. generer_form_ecrire(
-				'upgrade',
-				"<input type='hidden' name='reinstall' value='non' />",
-				'',
-				_T('bouton_relancer_installation')
-			)
-		);
-		echo $r;
-	} elseif (_request('fin')) {
-		include_spip('inc/plugin');
-		actualise_plugins_actifs();
-		include_spip('inc/headers');
-		$res = generer_url_ecrire('admin_plugin', 'var_mode=recalcul');
-		echo redirige_formulaire($res);
-	} else {
-		if (!isset($GLOBALS['meta']['version_installee'])) {
-			$GLOBALS['meta']['version_installee'] = 0.0;
-		} else {
-			$GLOBALS['meta']['version_installee'] =
-				(double)str_replace(',', '.', $GLOBALS['meta']['version_installee']);
-		}
-		# NB: str_replace car, sur club-internet, il semble que version_installe soit
-		# enregistree au format '1,812' et non '1.812'
+    // Si reinstallation necessaire, message ad hoc
+    if (_request('reinstall') == 'oui') {
+        include_spip('inc/minipres');
+        $r = minipres(
+            _T('titre_page_upgrade'),
+            '<p><b>'
+            . _T('texte_nouvelle_version_spip_1')
+            . '</b><p> '
+            . _T(
+                'texte_nouvelle_version_spip_2',
+                ['connect' => '<tt>' . _FILE_CONNECT . '</tt>']
+            )
+            . generer_form_ecrire(
+                'upgrade',
+                "<input type='hidden' name='reinstall' value='non' />",
+                '',
+                _T('bouton_relancer_installation')
+            )
+        );
+        echo $r;
+    } elseif (_request('fin')) {
+        include_spip('inc/plugin');
+        actualise_plugins_actifs();
+        include_spip('inc/headers');
+        $res = generer_url_ecrire('admin_plugin', 'var_mode=recalcul');
+        echo redirige_formulaire($res);
+    } else {
+        if (!isset($GLOBALS['meta']['version_installee'])) {
+            $GLOBALS['meta']['version_installee'] = 0.0;
+        } else {
+            $GLOBALS['meta']['version_installee'] =
+                (double)str_replace(',', '.', $GLOBALS['meta']['version_installee']);
+        }
+        # NB: str_replace car, sur club-internet, il semble que version_installe soit
+        # enregistree au format '1,812' et non '1.812'
 
-		// Erreur downgrade
-		// (cas de double installation de fichiers SPIP sur une meme base)
-		if ($GLOBALS['spip_version_base'] < $GLOBALS['meta']['version_installee']) {
-			$commentaire = _T('info_mise_a_niveau_base_2');
-		} // Commentaire standard upgrade
-		else {
-			$commentaire = _T('texte_mise_a_niveau_base_1');
-		}
+        // Erreur downgrade
+        // (cas de double installation de fichiers SPIP sur une meme base)
+        if ($GLOBALS['spip_version_base'] < $GLOBALS['meta']['version_installee']) {
+            $commentaire = _T('info_mise_a_niveau_base_2');
+        } // Commentaire standard upgrade
+        else {
+            $commentaire = _T('texte_mise_a_niveau_base_1');
+        }
 
-		$commentaire .= '<br />[' . $GLOBALS['meta']['version_installee'] . '/' . $GLOBALS['spip_version_base'] . ']';
+        $commentaire .= '<br />[' . $GLOBALS['meta']['version_installee'] . '/' . $GLOBALS['spip_version_base'] . ']';
 
-		$_POST['reinstall'] = 'non'; // pour copy_request dans admin
-		include_spip('inc/headers');
-		$admin = charger_fonction('admin', 'inc');
-		$res = $admin('upgrade', _T('info_mise_a_niveau_base'), $commentaire);
-		if ($res) {
-			echo $res;
-		} else {
-			// effacer les alea pour forcer leur relecture
-			// si jamais ils ont change pendant l'upgrade
-			unset($GLOBALS['meta']['alea_ephemere']);
-			unset($GLOBALS['meta']['alea_ephemere_ancien']);
-			$res = redirige_action_auteur('purger', 'cache', 'upgrade', 'fin=oui', true);
-			echo redirige_formulaire($res);
-		}
-	}
+        $_POST['reinstall'] = 'non'; // pour copy_request dans admin
+        include_spip('inc/headers');
+        $admin = charger_fonction('admin', 'inc');
+        $res = $admin('upgrade', _T('info_mise_a_niveau_base'), $commentaire);
+        if ($res) {
+            echo $res;
+        } else {
+            // effacer les alea pour forcer leur relecture
+            // si jamais ils ont change pendant l'upgrade
+            unset($GLOBALS['meta']['alea_ephemere']);
+            unset($GLOBALS['meta']['alea_ephemere_ancien']);
+            $res = redirige_action_auteur('purger', 'cache', 'upgrade', 'fin=oui', true);
+            echo redirige_formulaire($res);
+        }
+    }
 }

Please login to merge, or discard this patch.

ecrire/exec/selectionner.php 1 patch

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

@@ -16,7 +16,7 @@  discard block
 block discarded – undo
  * @package SPIP\Core\Rubriques
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -28,21 +28,21 @@  discard block
 block discarded – undo
  * @uses ajax_retour()
  **/
 function exec_selectionner_dist() {
-	$id = intval(_request('id'));
-	$exclus = intval(_request('exclus'));
-	$type = _request('type');
-	$rac = _request('racine');
-	$do = _request('do');
-	if (preg_match('/^\w*$/', $do)) {
-		if (!$do) {
-			$do = 'aff';
-		}
+    $id = intval(_request('id'));
+    $exclus = intval(_request('exclus'));
+    $type = _request('type');
+    $rac = _request('racine');
+    $do = _request('do');
+    if (preg_match('/^\w*$/', $do)) {
+        if (!$do) {
+            $do = 'aff';
+        }
 
-		$selectionner = charger_fonction('selectionner', 'inc');
+        $selectionner = charger_fonction('selectionner', 'inc');
 
-		$r = $selectionner($id, 'choix_parent', $exclus, $rac, $type != 'breve', $do);
-	} else {
-		$r = '';
-	}
-	ajax_retour($r);
+        $r = $selectionner($id, 'choix_parent', $exclus, $rac, $type != 'breve', $do);
+    } else {
+        $r = '';
+    }
+    ajax_retour($r);
 }

Please login to merge, or discard this patch.

ecrire/exec/base_delete_all.php 1 patch

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

@@ -17,40 +17,40 @@
 block discarded – undo
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
  * Exec de la page de destruction des tables de SPIP
  **/
 function exec_base_delete_all_dist() {
-	include_spip('inc/autoriser');
-	if (!autoriser('detruire')) {
-		include_spip('inc/minipres');
-		echo minipres();
-	} else {
-		include_spip('base/dump');
-		$res = base_lister_toutes_tables('', [], [], true);
-		if (!$res) {
-			include_spip('inc/minipres');
-			spip_log('Erreur base de donnees');
-			echo minipres(
-				_T('info_travaux_titre'),
-				_T('titre_probleme_technique') . '<p><tt>' . sql_errno() . ' ' . sql_error() . '</tt></p>'
-			);
-		} else {
-			$res = base_saisie_tables('delete', $res);
-			include_spip('inc/headers');
-			$res = "\n<ol style='text-align:left'><li>\n" .
-				join("</li>\n<li>", $res) .
-				'</li></ol>';
-			$admin = charger_fonction('admin', 'inc');
-			$res = $admin('delete_all', _T('titre_page_delete_all'), $res);
-			if (!$res) {
-				redirige_url_ecrire('install', '');
-			} else {
-				echo $res;
-			}
-		}
-	}
+    include_spip('inc/autoriser');
+    if (!autoriser('detruire')) {
+        include_spip('inc/minipres');
+        echo minipres();
+    } else {
+        include_spip('base/dump');
+        $res = base_lister_toutes_tables('', [], [], true);
+        if (!$res) {
+            include_spip('inc/minipres');
+            spip_log('Erreur base de donnees');
+            echo minipres(
+                _T('info_travaux_titre'),
+                _T('titre_probleme_technique') . '<p><tt>' . sql_errno() . ' ' . sql_error() . '</tt></p>'
+            );
+        } else {
+            $res = base_saisie_tables('delete', $res);
+            include_spip('inc/headers');
+            $res = "\n<ol style='text-align:left'><li>\n" .
+                join("</li>\n<li>", $res) .
+                '</li></ol>';
+            $admin = charger_fonction('admin', 'inc');
+            $res = $admin('delete_all', _T('titre_page_delete_all'), $res);
+            if (!$res) {
+                redirige_url_ecrire('install', '');
+            } else {
+                echo $res;
+            }
+        }
+    }
 }

Please login to merge, or discard this patch.

ecrire/exec/fond.php 1 patch

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

@@ -21,7 +21,7 @@  discard block
 block discarded – undo
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -29,16 +29,16 @@  discard block
 block discarded – undo
 $GLOBALS['delais'] = 0;// pas de cache !
 // Securite
 if (strstr($fond, '/')) {
-	if (
-		!include_spip('inc/autoriser')
-		or !autoriser('webmestre')
-	) {
-		include_spip('inc/minipres');
-		echo minipres();
-		exit;
-	}
+    if (
+        !include_spip('inc/autoriser')
+        or !autoriser('webmestre')
+    ) {
+        include_spip('inc/minipres');
+        echo minipres();
+        exit;
+    }
 } else {
-	$fond = "prive/squelettes/$fond";
+    $fond = "prive/squelettes/$fond";
 }
 
 // quelques inclusions et ini prealables
@@ -50,8 +50,8 @@  discard block
 block discarded – undo
  * @todo supprimer cette fonction vide ?
  **/
 function shutdown_error() {
-	// si on arrive ici avec un tampon non ferme : erreur fatale
-	/*	if (ob_get_level()){
+    // si on arrive ici avec un tampon non ferme : erreur fatale
+    /*	if (ob_get_level()){
 			// envoyer tous les tampons
 			while (ob_get_level())
 				ob_end_flush();

Please login to merge, or discard this patch.

ecrire/action/activer_plugins.php 1 patch

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

@@ -17,7 +17,7 @@  discard block
 block discarded – undo
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -28,49 +28,49 @@  discard block
 block discarded – undo
  * @return void
  */
 function enregistre_modif_plugin() {
-	include_spip('inc/plugin');
-	// recuperer les plugins dans l'ordre des $_POST
-	$test = [];
-	foreach (liste_plugin_files() as $file) {
-		$test['s' . substr(md5(_DIR_PLUGINS . $file), 0, 16)] = $file;
-	}
-	if (defined('_DIR_PLUGINS_SUPPL')) {
-		foreach (liste_plugin_files(_DIR_PLUGINS_SUPPL) as $file) {
-			$test['s' . substr(md5(_DIR_PLUGINS_SUPPL . $file), 0, 16)] = $file;
-		}
-	}
+    include_spip('inc/plugin');
+    // recuperer les plugins dans l'ordre des $_POST
+    $test = [];
+    foreach (liste_plugin_files() as $file) {
+        $test['s' . substr(md5(_DIR_PLUGINS . $file), 0, 16)] = $file;
+    }
+    if (defined('_DIR_PLUGINS_SUPPL')) {
+        foreach (liste_plugin_files(_DIR_PLUGINS_SUPPL) as $file) {
+            $test['s' . substr(md5(_DIR_PLUGINS_SUPPL . $file), 0, 16)] = $file;
+        }
+    }
 
-	$plugin = [];
+    $plugin = [];
 
-	foreach ($_POST as $choix => $val) {
-		if (isset($test[$choix]) && $val == 'O') {
-			$plugin[] = $test[$choix];
-		}
-	}
+    foreach ($_POST as $choix => $val) {
+        if (isset($test[$choix]) && $val == 'O') {
+            $plugin[] = $test[$choix];
+        }
+    }
 
-	spip_log("Changement des plugins actifs par l'auteur " . $GLOBALS['visiteur_session']['id_auteur'] . ': ' . join(
-		',',
-		$plugin
-	));
-	ecrire_plugin_actifs($plugin);
+    spip_log("Changement des plugins actifs par l'auteur " . $GLOBALS['visiteur_session']['id_auteur'] . ': ' . join(
+        ',',
+        $plugin
+    ));
+    ecrire_plugin_actifs($plugin);
 
-	// Chaque fois que l'on valide des plugins, on memorise la liste de ces plugins comme etant "interessants", avec un score initial, qui sera decremente a chaque tour : ainsi un plugin active pourra reter visible a l'ecran, jusqu'a ce qu'il tombe dans l'oubli.
-	$plugins_interessants = @unserialize($GLOBALS['meta']['plugins_interessants']);
-	if (!is_array($plugins_interessants)) {
-		$plugins_interessants = [];
-	}
+    // Chaque fois que l'on valide des plugins, on memorise la liste de ces plugins comme etant "interessants", avec un score initial, qui sera decremente a chaque tour : ainsi un plugin active pourra reter visible a l'ecran, jusqu'a ce qu'il tombe dans l'oubli.
+    $plugins_interessants = @unserialize($GLOBALS['meta']['plugins_interessants']);
+    if (!is_array($plugins_interessants)) {
+        $plugins_interessants = [];
+    }
 
-	$plugins_interessants2 = [];
+    $plugins_interessants2 = [];
 
-	foreach ($plugins_interessants as $plug => $score) {
-		if ($score > 1) {
-			$plugins_interessants2[$plug] = $score - 1;
-		}
-	}
-	foreach ($plugin as $plug) {
-		$plugins_interessants2[$plug] = 10;
-	} // score initial
-	ecrire_meta('plugins_interessants', serialize($plugins_interessants2));
+    foreach ($plugins_interessants as $plug => $score) {
+        if ($score > 1) {
+            $plugins_interessants2[$plug] = $score - 1;
+        }
+    }
+    foreach ($plugin as $plug) {
+        $plugins_interessants2[$plug] = 10;
+    } // score initial
+    ecrire_meta('plugins_interessants', serialize($plugins_interessants2));
 }
 
 /**
@@ -82,13 +82,13 @@  discard block
 block discarded – undo
  */
 function action_activer_plugins_dist() {
 
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $securiser_action();
 
-	if (!autoriser('configurer', '_plugins')) {
-		die('erreur');
-	}
-	// forcer la maj des meta pour les cas de modif de numero de version base via phpmyadmin
-	lire_metas();
-	enregistre_modif_plugin();
+    if (!autoriser('configurer', '_plugins')) {
+        die('erreur');
+    }
+    // forcer la maj des meta pour les cas de modif de numero de version base via phpmyadmin
+    lire_metas();
+    enregistre_modif_plugin();
 }

Please login to merge, or discard this patch.

ecrire/action/calculer_taille_cache.php 1 patch

Indentation +57 added lines, -57 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;
 }
 
 
@@ -31,41 +31,41 @@  discard block
 block discarded – undo
  *     de l'action sécurisée.
  */
 function action_calculer_taille_cache_dist($arg = null) {
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
-	include_spip('inc/filtres');
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
+    include_spip('inc/filtres');
 
-	if ($arg == 'images') {
-		$taille = calculer_taille_dossier(_DIR_VAR);
-		$res = _T(
-			'ecrire:taille_cache_image',
-			[
-				'dir' => joli_repertoire(_DIR_VAR),
-				'taille' => '<b>' . (taille_en_octets($taille) > 0 ? taille_en_octets($taille) : '0 octet') . '</b>'
-			]
-		);
-	} else {
-		include_spip('inc/invalideur');
-		$taille =
-			calculer_taille_dossier(_DIR_CACHE_XML)
-			+ calculer_taille_dossier(_DIR_CACHE . 'skel/')
-			+ calculer_taille_dossier(_DIR_CACHE . 'wheels/')
-			+ calculer_taille_dossier(_DIR_CACHE . 'contextes/');
-		$taille += intval(taille_du_cache());
-		if ($taille <= 150000) {
-			$res = _T('taille_cache_vide');
-		} elseif ($taille <= 1024 * 1024) {
-			$res = _T('taille_cache_moins_de', ['octets' => taille_en_octets(1024 * 1024)]);
-		} else {
-			$res = _T('taille_cache_octets', ['octets' => taille_en_octets($taille)]);
-		}
-		$res = "<b>$res</b>";
-	}
+    if ($arg == 'images') {
+        $taille = calculer_taille_dossier(_DIR_VAR);
+        $res = _T(
+            'ecrire:taille_cache_image',
+            [
+                'dir' => joli_repertoire(_DIR_VAR),
+                'taille' => '<b>' . (taille_en_octets($taille) > 0 ? taille_en_octets($taille) : '0 octet') . '</b>'
+            ]
+        );
+    } else {
+        include_spip('inc/invalideur');
+        $taille =
+            calculer_taille_dossier(_DIR_CACHE_XML)
+            + calculer_taille_dossier(_DIR_CACHE . 'skel/')
+            + calculer_taille_dossier(_DIR_CACHE . 'wheels/')
+            + calculer_taille_dossier(_DIR_CACHE . 'contextes/');
+        $taille += intval(taille_du_cache());
+        if ($taille <= 150000) {
+            $res = _T('taille_cache_vide');
+        } elseif ($taille <= 1024 * 1024) {
+            $res = _T('taille_cache_moins_de', ['octets' => taille_en_octets(1024 * 1024)]);
+        } else {
+            $res = _T('taille_cache_octets', ['octets' => taille_en_octets($taille)]);
+        }
+        $res = "<b>$res</b>";
+    }
 
-	$res = "<p>$res</p>";
-	ajax_retour($res);
+    $res = "<p>$res</p>";
+    ajax_retour($res);
 }
 
 
@@ -76,28 +76,28 @@  discard block
 block discarded – undo
  * @return int Taille en octets
  */
 function calculer_taille_dossier($dir) {
-	if (!is_dir($dir) or !is_readable($dir)) {
-		return 0;
-	}
-	$handle = opendir($dir);
-	if (!$handle) {
-		return 0;
-	}
-	$taille = 0;
-	while (($fichier = @readdir($handle)) !== false) {
-		// Eviter ".", "..", ".htaccess", etc.
-		if ($fichier[0] == '.') {
-			continue;
-		}
-		if (is_file($d = "$dir/$fichier")) {
-			$taille += filesize($d);
-		} else {
-			if (is_dir($d)) {
-				$taille += calculer_taille_dossier($d);
-			}
-		}
-	}
-	closedir($handle);
+    if (!is_dir($dir) or !is_readable($dir)) {
+        return 0;
+    }
+    $handle = opendir($dir);
+    if (!$handle) {
+        return 0;
+    }
+    $taille = 0;
+    while (($fichier = @readdir($handle)) !== false) {
+        // Eviter ".", "..", ".htaccess", etc.
+        if ($fichier[0] == '.') {
+            continue;
+        }
+        if (is_file($d = "$dir/$fichier")) {
+            $taille += filesize($d);
+        } else {
+            if (is_dir($d)) {
+                $taille += calculer_taille_dossier($d);
+            }
+        }
+    }
+    closedir($handle);
 
-	return $taille;
+    return $taille;
 }

Please login to merge, or discard this patch.

ecrire/action/annuler_job.php 1 patch

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

@@ -17,7 +17,7 @@  discard block
 block discarded – undo
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -26,13 +26,13 @@  discard block
 block discarded – undo
  * @return void
  */
 function action_annuler_job_dist() {
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$id_job = $securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $id_job = $securiser_action();
 
-	if (
-		$id_job = intval($id_job)
-		and autoriser('annuler', 'job', $id_job)
-	) {
-		job_queue_remove($id_job);
-	}
+    if (
+        $id_job = intval($id_job)
+        and autoriser('annuler', 'job', $id_job)
+    ) {
+        job_queue_remove($id_job);
+    }
 }

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,23 +27,23 @@ discard block
		block discarded – undo
27	27	* @uses ajax_retour()
28	28	**/
29	29	function exec_plonger_dist() {
30		- include_spip('inc/actions');
	30	+ include_spip('inc/actions');
31	31
32		- $rac = preg_replace(',[^\w\,/#&;-]+,', ' ', _request('rac'));
33		- $id = intval(_request('id'));
34		- $exclus = intval(_request('exclus'));
35		- $col = intval(_request('col'));
36		- $do = _request('do');
37		- if (preg_match('/^\w*$/', $do)) {
38		- if (!$do) {
39		- $do = 'aff';
40		- }
	32	+ $rac = preg_replace(',[^\w\,/#&;-]+,', ' ', _request('rac'));
	33	+ $id = intval(_request('id'));
	34	+ $exclus = intval(_request('exclus'));
	35	+ $col = intval(_request('col'));
	36	+ $do = _request('do');
	37	+ if (preg_match('/^\w*$/', $do)) {
	38	+ if (!$do) {
	39	+ $do = 'aff';
	40	+ }
41	41
42		- $plonger = charger_fonction('plonger', 'inc');
43		- $r = $plonger($id, spip_htmlentities($rac), [], $col, $exclus, $do);
44		- } else {
45		- $r = '';
46		- }
	42	+ $plonger = charger_fonction('plonger', 'inc');
	43	+ $r = $plonger($id, spip_htmlentities($rac), [], $col, $exclus, $do);
	44	+ } else {
	45	+ $r = '';
	46	+ }
47	47
48		- ajax_retour($r);
	48	+ ajax_retour($r);
49	49	}

		@@ -16,7 +16,7 @@ discard block
		block discarded – undo
16	16	* @package SPIP\Core\Rubriques
17	17	**/
18	18	if (!defined('_ECRIRE_INC_VERSION')) {
19		- return;
	19	+ return;
20	20	}
21	21
22	22	include_spip('inc/actions');
		@@ -28,21 +28,21 @@ discard block
		block discarded – undo
28	28	* @uses ajax_retour()
29	29	**/
30	30	function exec_selectionner_dist() {
31		- $id = intval(_request('id'));
32		- $exclus = intval(_request('exclus'));
33		- $type = _request('type');
34		- $rac = _request('racine');
35		- $do = _request('do');
36		- if (preg_match('/^\w*$/', $do)) {
37		- if (!$do) {
38		- $do = 'aff';
39		- }
	31	+ $id = intval(_request('id'));
	32	+ $exclus = intval(_request('exclus'));
	33	+ $type = _request('type');
	34	+ $rac = _request('racine');
	35	+ $do = _request('do');
	36	+ if (preg_match('/^\w*$/', $do)) {
	37	+ if (!$do) {
	38	+ $do = 'aff';
	39	+ }
40	40
41		- $selectionner = charger_fonction('selectionner', 'inc');
	41	+ $selectionner = charger_fonction('selectionner', 'inc');
42	42
43		- $r = $selectionner($id, 'choix_parent', $exclus, $rac, $type != 'breve', $do);
44		- } else {
45		- $r = '';
46		- }
47		- ajax_retour($r);
	43	+ $r = $selectionner($id, 'choix_parent', $exclus, $rac, $type != 'breve', $do);
	44	+ } else {
	45	+ $r = '';
	46	+ }
	47	+ ajax_retour($r);
48	48	}

		@@ -17,40 +17,40 @@
		block discarded – undo
17	17	*/
18	18
19	19	if (!defined('_ECRIRE_INC_VERSION')) {
20		- return;
	20	+ return;
21	21	}
22	22
23	23	/**
24	24	* Exec de la page de destruction des tables de SPIP
25	25	**/
26	26	function exec_base_delete_all_dist() {
27		- include_spip('inc/autoriser');
28		- if (!autoriser('detruire')) {
29		- include_spip('inc/minipres');
30		- echo minipres();
31		- } else {
32		- include_spip('base/dump');
33		- $res = base_lister_toutes_tables('', [], [], true);
34		- if (!$res) {
35		- include_spip('inc/minipres');
36		- spip_log('Erreur base de donnees');
37		- echo minipres(
38		- _T('info_travaux_titre'),
39		- _T('titre_probleme_technique') . '<p><tt>' . sql_errno() . ' ' . sql_error() . '</tt></p>'
40		- );
41		- } else {
42		- $res = base_saisie_tables('delete', $res);
43		- include_spip('inc/headers');
44		- $res = "\n<ol style='text-align:left'><li>\n" .
45		- join("</li>\n<li>", $res) .
46		- '</li></ol>';
47		- $admin = charger_fonction('admin', 'inc');
48		- $res = $admin('delete_all', _T('titre_page_delete_all'), $res);
49		- if (!$res) {
50		- redirige_url_ecrire('install', '');
51		- } else {
52		- echo $res;
53		- }
54		- }
55		- }
	27	+ include_spip('inc/autoriser');
	28	+ if (!autoriser('detruire')) {
	29	+ include_spip('inc/minipres');
	30	+ echo minipres();
	31	+ } else {
	32	+ include_spip('base/dump');
	33	+ $res = base_lister_toutes_tables('', [], [], true);
	34	+ if (!$res) {
	35	+ include_spip('inc/minipres');
	36	+ spip_log('Erreur base de donnees');
	37	+ echo minipres(
	38	+ _T('info_travaux_titre'),
	39	+ _T('titre_probleme_technique') . '<p><tt>' . sql_errno() . ' ' . sql_error() . '</tt></p>'
	40	+ );
	41	+ } else {
	42	+ $res = base_saisie_tables('delete', $res);
	43	+ include_spip('inc/headers');
	44	+ $res = "\n<ol style='text-align:left'><li>\n" .
	45	+ join("</li>\n<li>", $res) .
	46	+ '</li></ol>';
	47	+ $admin = charger_fonction('admin', 'inc');
	48	+ $res = $admin('delete_all', _T('titre_page_delete_all'), $res);
	49	+ if (!$res) {
	50	+ redirige_url_ecrire('install', '');
	51	+ } else {
	52	+ echo $res;
	53	+ }
	54	+ }
	55	+ }
56	56	}

		@@ -21,7 +21,7 @@ discard block
		block discarded – undo
21	21	*/
22	22
23	23	if (!defined('_ECRIRE_INC_VERSION')) {
24		- return;
	24	+ return;
25	25	}
26	26
27	27
		@@ -29,16 +29,16 @@ discard block
		block discarded – undo
29	29	$GLOBALS['delais'] = 0;// pas de cache !
30	30	// Securite
31	31	if (strstr($fond, '/')) {
32		- if (
33		- !include_spip('inc/autoriser')
34		- or !autoriser('webmestre')
35		- ) {
36		- include_spip('inc/minipres');
37		- echo minipres();
38		- exit;
39		- }
	32	+ if (
	33	+ !include_spip('inc/autoriser')
	34	+ or !autoriser('webmestre')
	35	+ ) {
	36	+ include_spip('inc/minipres');
	37	+ echo minipres();
	38	+ exit;
	39	+ }
40	40	} else {
41		- $fond = "prive/squelettes/$fond";
	41	+ $fond = "prive/squelettes/$fond";
42	42	}
43	43
44	44	// quelques inclusions et ini prealables
		@@ -50,8 +50,8 @@ discard block
		block discarded – undo
50	50	* @todo supprimer cette fonction vide ?
51	51	**/
52	52	function shutdown_error() {
53		- // si on arrive ici avec un tampon non ferme : erreur fatale
54		- /* if (ob_get_level()){
	53	+ // si on arrive ici avec un tampon non ferme : erreur fatale
	54	+ /* if (ob_get_level()){
55	55	// envoyer tous les tampons
56	56	while (ob_get_level())
57	57	ob_end_flush();

		@@ -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
		@@ -31,41 +31,41 @@ discard block
		block discarded – undo
31	31	* de l'action sécurisée.
32	32	*/
33	33	function action_calculer_taille_cache_dist($arg = null) {
34		- if (is_null($arg)) {
35		- $securiser_action = charger_fonction('securiser_action', 'inc');
36		- $arg = $securiser_action();
37		- }
38		- include_spip('inc/filtres');
	34	+ if (is_null($arg)) {
	35	+ $securiser_action = charger_fonction('securiser_action', 'inc');
	36	+ $arg = $securiser_action();
	37	+ }
	38	+ include_spip('inc/filtres');
39	39
40		- if ($arg == 'images') {
41		- $taille = calculer_taille_dossier(_DIR_VAR);
42		- $res = _T(
43		- 'ecrire:taille_cache_image',
44		- [
45		- 'dir' => joli_repertoire(_DIR_VAR),
46		- 'taille' => '<b>' . (taille_en_octets($taille) > 0 ? taille_en_octets($taille) : '0 octet') . '</b>'
47		- ]
48		- );
49		- } else {
50		- include_spip('inc/invalideur');
51		- $taille =
52		- calculer_taille_dossier(_DIR_CACHE_XML)
53		- + calculer_taille_dossier(_DIR_CACHE . 'skel/')
54		- + calculer_taille_dossier(_DIR_CACHE . 'wheels/')
55		- + calculer_taille_dossier(_DIR_CACHE . 'contextes/');
56		- $taille += intval(taille_du_cache());
57		- if ($taille <= 150000) {
58		- $res = _T('taille_cache_vide');
59		- } elseif ($taille <= 1024 * 1024) {
60		- $res = _T('taille_cache_moins_de', ['octets' => taille_en_octets(1024 * 1024)]);
61		- } else {
62		- $res = _T('taille_cache_octets', ['octets' => taille_en_octets($taille)]);
63		- }
64		- $res = "<b>$res</b>";
65		- }
	40	+ if ($arg == 'images') {
	41	+ $taille = calculer_taille_dossier(_DIR_VAR);
	42	+ $res = _T(
	43	+ 'ecrire:taille_cache_image',
	44	+ [
	45	+ 'dir' => joli_repertoire(_DIR_VAR),
	46	+ 'taille' => '<b>' . (taille_en_octets($taille) > 0 ? taille_en_octets($taille) : '0 octet') . '</b>'
	47	+ ]
	48	+ );
	49	+ } else {
	50	+ include_spip('inc/invalideur');
	51	+ $taille =
	52	+ calculer_taille_dossier(_DIR_CACHE_XML)
	53	+ + calculer_taille_dossier(_DIR_CACHE . 'skel/')
	54	+ + calculer_taille_dossier(_DIR_CACHE . 'wheels/')
	55	+ + calculer_taille_dossier(_DIR_CACHE . 'contextes/');
	56	+ $taille += intval(taille_du_cache());
	57	+ if ($taille <= 150000) {
	58	+ $res = _T('taille_cache_vide');
	59	+ } elseif ($taille <= 1024 * 1024) {
	60	+ $res = _T('taille_cache_moins_de', ['octets' => taille_en_octets(1024 * 1024)]);
	61	+ } else {
	62	+ $res = _T('taille_cache_octets', ['octets' => taille_en_octets($taille)]);
	63	+ }
	64	+ $res = "<b>$res</b>";
	65	+ }
66	66
67		- $res = "<p>$res</p>";
68		- ajax_retour($res);
	67	+ $res = "<p>$res</p>";
	68	+ ajax_retour($res);
69	69	}
70	70
71	71
		@@ -76,28 +76,28 @@ discard block
		block discarded – undo
76	76	* @return int Taille en octets
77	77	*/
78	78	function calculer_taille_dossier($dir) {
79		- if (!is_dir($dir) or !is_readable($dir)) {
80		- return 0;
81		- }
82		- $handle = opendir($dir);
83		- if (!$handle) {
84		- return 0;
85		- }
86		- $taille = 0;
87		- while (($fichier = @readdir($handle)) !== false) {
88		- // Eviter ".", "..", ".htaccess", etc.
89		- if ($fichier[0] == '.') {
90		- continue;
91		- }
92		- if (is_file($d = "$dir/$fichier")) {
93		- $taille += filesize($d);
94		- } else {
95		- if (is_dir($d)) {
96		- $taille += calculer_taille_dossier($d);
97		- }
98		- }
99		- }
100		- closedir($handle);
	79	+ if (!is_dir($dir) or !is_readable($dir)) {
	80	+ return 0;
	81	+ }
	82	+ $handle = opendir($dir);
	83	+ if (!$handle) {
	84	+ return 0;
	85	+ }
	86	+ $taille = 0;
	87	+ while (($fichier = @readdir($handle)) !== false) {
	88	+ // Eviter ".", "..", ".htaccess", etc.
	89	+ if ($fichier[0] == '.') {
	90	+ continue;
	91	+ }
	92	+ if (is_file($d = "$dir/$fichier")) {
	93	+ $taille += filesize($d);
	94	+ } else {
	95	+ if (is_dir($d)) {
	96	+ $taille += calculer_taille_dossier($d);
	97	+ }
	98	+ }
	99	+ }
	100	+ closedir($handle);
101	101
102		- return $taille;
	102	+ return $taille;
103	103	}

spip / SPIP

Push — master ( c76770...babdec )

Status

Category

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

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

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

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

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

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

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

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

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