spip / SPIP

ecrire/plugins/afficher_liste.php

Spacing +4 added lines, -4 removed lines

@@ -63,7 +63,7 @@ 
 	asort($liste_plugins);
 	$exposed = urldecode(_request('plugin') ?? '');
 
-	$block_par_lettre = false;//count($liste_plugins)>10;
+	$block_par_lettre = false; //count($liste_plugins)>10;
 	$fast_liste_plugins_actifs = [];
 	$fast_liste_plugins_checked = [];
 	if (is_array($liste_plugins_actifs)) {
@@ -88,11 +88,11 @@ 
 		$actif = isset($fast_liste_plugins_actifs[$plug]);
 		$checked = isset($fast_liste_plugins_checked[$plug]);
 		$block_actif |= $actif;
-		$expose = ($exposed && ($exposed == $plug || $exposed === $dir_plugins . $plug || $exposed === substr(
+		$expose = ($exposed && ($exposed == $plug || $exposed === $dir_plugins.$plug || $exposed === substr(
 			$dir_plugins,
 			strlen(_DIR_RACINE)
-		) . $plug));
-		$block .= $ligne_plug($url_page, $plug, $checked, $actif, $expose, 'item', $dir_plugins) . "\n";
+		).$plug));
+		$block .= $ligne_plug($url_page, $plug, $checked, $actif, $expose, 'item', $dir_plugins)."\n";
 	}
 	$res .= $block_par_lettre ? affiche_block_initiale($initiale, $block, $block_actif) : $block;
 	$class = basename($dir_plugins);

Indentation +75 added lines, -75 removed lines

@@ -10,7 +10,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 include_spip('inc/charsets');
 
@@ -26,90 +26,90 @@ 
  * @return string
  */
 function plugins_afficher_liste_dist(
-	$url_page,
-	$liste_plugins,
-	$liste_plugins_checked,
-	$liste_plugins_actifs,
-	$dir_plugins = _DIR_PLUGINS,
-	$afficher_un = 'afficher_plugin'
+    $url_page,
+    $liste_plugins,
+    $liste_plugins_checked,
+    $liste_plugins_actifs,
+    $dir_plugins = _DIR_PLUGINS,
+    $afficher_un = 'afficher_plugin'
 ) {
-	$get_infos = charger_fonction('get_infos', 'plugins');
-	$ligne_plug = charger_fonction($afficher_un, 'plugins');
+    $get_infos = charger_fonction('get_infos', 'plugins');
+    $ligne_plug = charger_fonction($afficher_un, 'plugins');
 
-	$all_infos = $get_infos($liste_plugins, false, $dir_plugins);
+    $all_infos = $get_infos($liste_plugins, false, $dir_plugins);
 
-	$all_infos = pipeline(
-		'filtrer_liste_plugins',
-		[
-			'args' => [
-				'liste_plugins' => $liste_plugins,
-				'liste_plugins_checked' => $liste_plugins_checked,
-				'liste_plugins_actifs' => $liste_plugins_actifs,
-				'dir_plugins' => $dir_plugins
-			],
-			'data' => $all_infos
-		]
-	);
+    $all_infos = pipeline(
+        'filtrer_liste_plugins',
+        [
+            'args' => [
+                'liste_plugins' => $liste_plugins,
+                'liste_plugins_checked' => $liste_plugins_checked,
+                'liste_plugins_actifs' => $liste_plugins_actifs,
+                'dir_plugins' => $dir_plugins
+            ],
+            'data' => $all_infos
+        ]
+    );
 
-	$liste_plugins = array_flip($liste_plugins);
-	foreach (array_keys($liste_plugins) as $chemin) {
-		// des plugins ont pu etre enleves de la liste par le pipeline. On en tient compte.
-		if (isset($all_infos[$chemin])) {
-			$liste_plugins[$chemin] = strtoupper(trim((string) typo(translitteration(unicode2charset(html2unicode($all_infos[$chemin]['nom']))))));
-		} else {
-			unset($liste_plugins[$chemin]);
-		}
-	}
-	asort($liste_plugins);
-	$exposed = urldecode(_request('plugin') ?? '');
+    $liste_plugins = array_flip($liste_plugins);
+    foreach (array_keys($liste_plugins) as $chemin) {
+        // des plugins ont pu etre enleves de la liste par le pipeline. On en tient compte.
+        if (isset($all_infos[$chemin])) {
+            $liste_plugins[$chemin] = strtoupper(trim((string) typo(translitteration(unicode2charset(html2unicode($all_infos[$chemin]['nom']))))));
+        } else {
+            unset($liste_plugins[$chemin]);
+        }
+    }
+    asort($liste_plugins);
+    $exposed = urldecode(_request('plugin') ?? '');
 
-	$block_par_lettre = false;//count($liste_plugins)>10;
-	$fast_liste_plugins_actifs = [];
-	$fast_liste_plugins_checked = [];
-	if (is_array($liste_plugins_actifs)) {
-		$fast_liste_plugins_actifs = array_flip($liste_plugins_actifs);
-	}
-	if (is_array($liste_plugins_checked)) {
-		$fast_liste_plugins_checked = array_flip($liste_plugins_checked);
-	}
+    $block_par_lettre = false;//count($liste_plugins)>10;
+    $fast_liste_plugins_actifs = [];
+    $fast_liste_plugins_checked = [];
+    if (is_array($liste_plugins_actifs)) {
+        $fast_liste_plugins_actifs = array_flip($liste_plugins_actifs);
+    }
+    if (is_array($liste_plugins_checked)) {
+        $fast_liste_plugins_checked = array_flip($liste_plugins_checked);
+    }
 
-	$res = '';
-	$block = '';
-	$initiale = '';
-	$block_actif = false;
-	foreach ($liste_plugins as $plug => $nom) {
-		if (($i = substr($nom, 0, 1)) !== $initiale) {
-			$res .= $block_par_lettre ? affiche_block_initiale($initiale, $block, $block_actif) : $block;
-			$initiale = $i;
-			$block = '';
-			$block_actif = false;
-		}
-		// le rep suivant
-		$actif = isset($fast_liste_plugins_actifs[$plug]);
-		$checked = isset($fast_liste_plugins_checked[$plug]);
-		$block_actif |= $actif;
-		$expose = ($exposed && ($exposed == $plug || $exposed === $dir_plugins . $plug || $exposed === substr(
-			$dir_plugins,
-			strlen(_DIR_RACINE)
-		) . $plug));
-		$block .= $ligne_plug($url_page, $plug, $checked, $actif, $expose, 'item', $dir_plugins) . "\n";
-	}
-	$res .= $block_par_lettre ? affiche_block_initiale($initiale, $block, $block_actif) : $block;
-	$class = basename($dir_plugins);
+    $res = '';
+    $block = '';
+    $initiale = '';
+    $block_actif = false;
+    foreach ($liste_plugins as $plug => $nom) {
+        if (($i = substr($nom, 0, 1)) !== $initiale) {
+            $res .= $block_par_lettre ? affiche_block_initiale($initiale, $block, $block_actif) : $block;
+            $initiale = $i;
+            $block = '';
+            $block_actif = false;
+        }
+        // le rep suivant
+        $actif = isset($fast_liste_plugins_actifs[$plug]);
+        $checked = isset($fast_liste_plugins_checked[$plug]);
+        $block_actif |= $actif;
+        $expose = ($exposed && ($exposed == $plug || $exposed === $dir_plugins . $plug || $exposed === substr(
+            $dir_plugins,
+            strlen(_DIR_RACINE)
+        ) . $plug));
+        $block .= $ligne_plug($url_page, $plug, $checked, $actif, $expose, 'item', $dir_plugins) . "\n";
+    }
+    $res .= $block_par_lettre ? affiche_block_initiale($initiale, $block, $block_actif) : $block;
+    $class = basename($dir_plugins);
 
-	return $res ? "<ul class='liste-items plugins $class'>$res</ul>" : '';
+    return $res ? "<ul class='liste-items plugins $class'>$res</ul>" : '';
 }
 
 
 function affiche_block_initiale($initiale, $block, $block_actif) {
-	if (strlen((string) $block)) {
-		return "<li class='item'>"
-		. bouton_block_depliable($initiale, (bool) $block_actif)
-		. debut_block_depliable($block_actif)
-		. "<ul>$block</ul>"
-		. fin_block()
-		. '</li>';
-	}
+    if (strlen((string) $block)) {
+        return "<li class='item'>"
+        . bouton_block_depliable($initiale, (bool) $block_actif)
+        . debut_block_depliable($block_actif)
+        . "<ul>$block</ul>"
+        . fin_block()
+        . '</li>';
+    }
 
-	return '';
+    return '';
 }

prive/formulaires/configurer_reducteur.php

Indentation +78 added lines, -78 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -27,24 +27,24 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_configurer_reducteur_charger_dist() {
-	$valeurs = [];
-	foreach (
-		[
-			'image_process',
-			'formats_graphiques',
-			'creer_preview',
-			'taille_preview',
-		] as $m
-	) {
-		$valeurs[$m] = $GLOBALS['meta'][$m] ?? null;
-	}
-
-	$valeurs['taille_preview'] = (int) $valeurs['taille_preview'];
-	if ($valeurs['taille_preview'] < 10) {
-		$valeurs['taille_preview'] = 120;
-	}
-
-	return $valeurs;
+    $valeurs = [];
+    foreach (
+        [
+            'image_process',
+            'formats_graphiques',
+            'creer_preview',
+            'taille_preview',
+        ] as $m
+    ) {
+        $valeurs[$m] = $GLOBALS['meta'][$m] ?? null;
+    }
+
+    $valeurs['taille_preview'] = (int) $valeurs['taille_preview'];
+    if ($valeurs['taille_preview'] < 10) {
+        $valeurs['taille_preview'] = 120;
+    }
+
+    return $valeurs;
 }
 
 
@@ -55,52 +55,52 @@ 
  *     Retours des traitements
  **/
 function formulaires_configurer_reducteur_traiter_dist() {
-	$res = ['editable' => true];
-
-	if (is_array($image_process = _request('image_process_'))) {
-		$image_process = array_keys($image_process);
-		$image_process = reset($image_process);
-
-		// application du choix de vignette
-		if ($image_process) {
-			// mettre a jour les formats graphiques lisibles
-			switch ($image_process) {
-				case 'gd2':
-					$formats_graphiques = $GLOBALS['meta']['gd_formats_read'];
-					break;
-				case 'netpbm':
-					$formats_graphiques = $GLOBALS['meta']['netpbm_formats'];
-					break;
-				case 'convert':
-				case 'imagick':
-					$formats_graphiques = 'gif,jpg,png,webp';
-					break;
-				default: #debug
-					$formats_graphiques = '';
-					$image_process = 'non';
-					break;
-			}
-			ecrire_meta('formats_graphiques', $formats_graphiques, 'non');
-			ecrire_meta('image_process', $image_process, 'non');
-		}
-	}
-
-	foreach (
-		[
-			'creer_preview'
-		] as $m
-	) {
-		if (!is_null($v = _request($m))) {
-			ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
-		}
-	}
-	if (!is_null($v = _request('taille_preview'))) {
-		ecrire_meta('taille_preview', (int) $v);
-	}
-
-	$res['message_ok'] = _T('config_info_enregistree');
-
-	return $res;
+    $res = ['editable' => true];
+
+    if (is_array($image_process = _request('image_process_'))) {
+        $image_process = array_keys($image_process);
+        $image_process = reset($image_process);
+
+        // application du choix de vignette
+        if ($image_process) {
+            // mettre a jour les formats graphiques lisibles
+            switch ($image_process) {
+                case 'gd2':
+                    $formats_graphiques = $GLOBALS['meta']['gd_formats_read'];
+                    break;
+                case 'netpbm':
+                    $formats_graphiques = $GLOBALS['meta']['netpbm_formats'];
+                    break;
+                case 'convert':
+                case 'imagick':
+                    $formats_graphiques = 'gif,jpg,png,webp';
+                    break;
+                default: #debug
+                    $formats_graphiques = '';
+                    $image_process = 'non';
+                    break;
+            }
+            ecrire_meta('formats_graphiques', $formats_graphiques, 'non');
+            ecrire_meta('image_process', $image_process, 'non');
+        }
+    }
+
+    foreach (
+        [
+            'creer_preview'
+        ] as $m
+    ) {
+        if (!is_null($v = _request($m))) {
+            ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
+        }
+    }
+    if (!is_null($v = _request('taille_preview'))) {
+        ecrire_meta('taille_preview', (int) $v);
+    }
+
+    $res['message_ok'] = _T('config_info_enregistree');
+
+    return $res;
 }
 
 /**
@@ -113,17 +113,17 @@ 
  *     URL d'action pour tester la librairie graphique en créant une vignette
  **/
 function url_vignette_choix(string $process): string {
-	$ok = match ($process) {
-		'gd2' => function_exists('ImageCreateTrueColor'),
-		'netpbm' => !(defined('_PNMSCALE_COMMAND') && _PNMSCALE_COMMAND == ''),
-		'imagick' => method_exists(\Imagick::class, 'readImage'),
-		'convert' => !(defined('_CONVERT_COMMAND') && _CONVERT_COMMAND == ''),
-		default => false,
-	};
-
-	if (!$ok) {
-		return '';
-	}
-
-	return generer_url_action('tester', "arg=$process&time=" . time());
+    $ok = match ($process) {
+        'gd2' => function_exists('ImageCreateTrueColor'),
+        'netpbm' => !(defined('_PNMSCALE_COMMAND') && _PNMSCALE_COMMAND == ''),
+        'imagick' => method_exists(\Imagick::class, 'readImage'),
+        'convert' => !(defined('_CONVERT_COMMAND') && _CONVERT_COMMAND == ''),
+        default => false,
+    };
+
+    if (!$ok) {
+        return '';
+    }
+
+    return generer_url_action('tester', "arg=$process&time=" . time());
 }

Spacing +1 added lines, -1 removed lines

@@ -125,5 +125,5 @@
 		return '';
 	}
 
-	return generer_url_action('tester', "arg=$process&time=" . time());
+	return generer_url_action('tester', "arg=$process&time=".time());
 }

prive/formulaires/declarer_bases.php

Indentation +181 added lines, -181 removed lines

@@ -10,210 +10,210 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/install');
 
 function formulaires_declarer_bases_charger_dist() {
 
-	if (!autoriser('detruire')) {
-		return false;
-	}
-
-	[$adresse_db, $login_db, $pass_db, $sel, $server_db] = analyse_fichier_connection(_FILE_CONNECT);
-
-	$deja = bases_referencees(_FILE_CONNECT);
-	// proposer un nom de connect si pas encore saisi
-	$nom_connect = '';
-	if (defined('_DECLARER_CHOIX_DB')) {
-		$nom_connect = _DECLARER_CHOIX_DB;
-		$n = '';
-		while (in_array($nom_connect . $n, $deja)) {
-			$n = ($n ? $n + 1 : 1);
-		}
-		$nom_connect .= $n;
-	}
-
-	return [
-		'_etapes' => 3,
-		'_bases_deja' => $deja,
-		'_bases_prop' => defined('_DECLARER_SERVEUR_DB') ? liste_bases(_DECLARER_SERVEUR_DB) : '',
-		'_tables' => (defined('_DECLARER_SERVEUR_DB') && defined('_DECLARER_CHOIX_DB')) ?
-			$tables = sql_alltable('%', _DECLARER_SERVEUR_DB)
-			:
-			[],
-		'main_db' => '',
-		'_serveurs' => liste_serveurs(),
-		'sql_serveur_db' => 'sqlite3', // valeur par defaut
-		'adresse_db' => $adresse_db,
-		'login_db' => '',
-		'pass_db' => '',
-		'choix_db' => '',
-		'table_new' => '',
-		'nom_connect' => $nom_connect,
-	];
+    if (!autoriser('detruire')) {
+        return false;
+    }
+
+    [$adresse_db, $login_db, $pass_db, $sel, $server_db] = analyse_fichier_connection(_FILE_CONNECT);
+
+    $deja = bases_referencees(_FILE_CONNECT);
+    // proposer un nom de connect si pas encore saisi
+    $nom_connect = '';
+    if (defined('_DECLARER_CHOIX_DB')) {
+        $nom_connect = _DECLARER_CHOIX_DB;
+        $n = '';
+        while (in_array($nom_connect . $n, $deja)) {
+            $n = ($n ? $n + 1 : 1);
+        }
+        $nom_connect .= $n;
+    }
+
+    return [
+        '_etapes' => 3,
+        '_bases_deja' => $deja,
+        '_bases_prop' => defined('_DECLARER_SERVEUR_DB') ? liste_bases(_DECLARER_SERVEUR_DB) : '',
+        '_tables' => (defined('_DECLARER_SERVEUR_DB') && defined('_DECLARER_CHOIX_DB')) ?
+            $tables = sql_alltable('%', _DECLARER_SERVEUR_DB)
+            :
+            [],
+        'main_db' => '',
+        '_serveurs' => liste_serveurs(),
+        'sql_serveur_db' => 'sqlite3', // valeur par defaut
+        'adresse_db' => $adresse_db,
+        'login_db' => '',
+        'pass_db' => '',
+        'choix_db' => '',
+        'table_new' => '',
+        'nom_connect' => $nom_connect,
+    ];
 }
 
 function liste_serveurs() {
-	$options = [];
-	$dir = _DIR_RESTREINT . 'req/';
-	$d = opendir($dir);
-	if (!$d) {
-		return [];
-	}
-	while ($f = readdir($d)) {
-		if (
-			preg_match('/^(.*)[.]php$/', $f, $s)
-			&& is_readable($f = $dir . $f)
-		) {
-			require_once($f);
-			$s = $s[1];
-			$v = 'spip_versions_' . $s;
-			if (function_exists($v) && $v()) {
-				$options[$s] = "install_select_type_$s";
-			} else {
-				spip_log("$s: portage indisponible");
-			}
-		}
-	}
-	ksort($options);
-
-	return $options;
+    $options = [];
+    $dir = _DIR_RESTREINT . 'req/';
+    $d = opendir($dir);
+    if (!$d) {
+        return [];
+    }
+    while ($f = readdir($d)) {
+        if (
+            preg_match('/^(.*)[.]php$/', $f, $s)
+            && is_readable($f = $dir . $f)
+        ) {
+            require_once($f);
+            $s = $s[1];
+            $v = 'spip_versions_' . $s;
+            if (function_exists($v) && $v()) {
+                $options[$s] = "install_select_type_$s";
+            } else {
+                spip_log("$s: portage indisponible");
+            }
+        }
+    }
+    ksort($options);
+
+    return $options;
 }
 
 function liste_bases($server_db) {
-	if (
-		is_null($server_db)
-		|| !$result = sql_listdbs($server_db)
-	) {
-		return '';
-	}
-
-	$noms = [];
-
-	// si sqlite : result est deja un tableau
-	if (is_array($result)) {
-		$noms = $result;
-	} else {
-		while ($row = sql_fetch($result, $server_db)) {
-			$noms[] = reset($row);
-		}
-	}
-
-	return $noms;
+    if (
+        is_null($server_db)
+        || !$result = sql_listdbs($server_db)
+    ) {
+        return '';
+    }
+
+    $noms = [];
+
+    // si sqlite : result est deja un tableau
+    if (is_array($result)) {
+        $noms = $result;
+    } else {
+        while ($row = sql_fetch($result, $server_db)) {
+            $noms[] = reset($row);
+        }
+    }
+
+    return $noms;
 }
 
 function formulaires_declarer_bases_verifier_1_dist() {
-	$erreurs = [];
-	[$def_adresse_db, $def_login_db, $def_pass_db, $sel_db, $def_serveur_db] = analyse_fichier_connection(_FILE_CONNECT);
-
-	if (!$adresse_db = _request('adresse_db')) {
-		$adresse_db = defined('_INSTALL_HOST_DB') ? _INSTALL_HOST_DB : $def_adresse_db;
-	}
-	if (!$serveur_db = _request('sql_serveur_db')) {
-		$serveur_db = defined('_INSTALL_SERVER_DB') ? _INSTALL_SERVER_DB : $def_serveur_db;
-	}
-
-	$login_db = $pass_db = '';
-	if (!preg_match(',^sqlite,i', (string) $serveur_db)) {
-		if (!$login_db = _request('login_db')) {
-			$login_db = defined('_INSTALL_USER_DB') ? _INSTALL_USER_DB : $def_login_db;
-		}
-		if (!$pass_db = _request('pass_db')) {
-			$pass_db = defined('_INSTALL_PASS_DB') ? _INSTALL_PASS_DB : $def_pass_db;
-		}
-	}
-
-	$link = spip_connect_db($adresse_db, '', $login_db, $pass_db, '@test@', $serveur_db);
-	if ($link) {
-		$GLOBALS['connexions'][$serveur_db][$GLOBALS['spip_sql_version']] = $GLOBALS['spip_' . $serveur_db . '_functions_' . $GLOBALS['spip_sql_version']];
-		$GLOBALS['connexions'][$serveur_db] = $link;
-		define('_DECLARER_SERVEUR_DB', $serveur_db);
-		define('_DECLARER_ADRESSE_DB', $adresse_db);
-		define('_DECLARER_LOGIN_DB', $login_db);
-		define('_DECLARER_PASS_DB', $pass_db);
-		// si on est sur le meme serveur que connect.php
-		// indiquer quelle est la db utilisee pour l'exclure des choix possibles
-		if ($serveur_db == $def_serveur_db && $adresse_db == $def_adresse_db) {
-			set_request('main_db', $sel_db);
-		} else {
-			set_request('main_db', '');
-		}
-	} else {
-		$erreurs['message_erreur'] = _T('avis_connexion_echec_1');
-	}
-
-	return $erreurs;
+    $erreurs = [];
+    [$def_adresse_db, $def_login_db, $def_pass_db, $sel_db, $def_serveur_db] = analyse_fichier_connection(_FILE_CONNECT);
+
+    if (!$adresse_db = _request('adresse_db')) {
+        $adresse_db = defined('_INSTALL_HOST_DB') ? _INSTALL_HOST_DB : $def_adresse_db;
+    }
+    if (!$serveur_db = _request('sql_serveur_db')) {
+        $serveur_db = defined('_INSTALL_SERVER_DB') ? _INSTALL_SERVER_DB : $def_serveur_db;
+    }
+
+    $login_db = $pass_db = '';
+    if (!preg_match(',^sqlite,i', (string) $serveur_db)) {
+        if (!$login_db = _request('login_db')) {
+            $login_db = defined('_INSTALL_USER_DB') ? _INSTALL_USER_DB : $def_login_db;
+        }
+        if (!$pass_db = _request('pass_db')) {
+            $pass_db = defined('_INSTALL_PASS_DB') ? _INSTALL_PASS_DB : $def_pass_db;
+        }
+    }
+
+    $link = spip_connect_db($adresse_db, '', $login_db, $pass_db, '@test@', $serveur_db);
+    if ($link) {
+        $GLOBALS['connexions'][$serveur_db][$GLOBALS['spip_sql_version']] = $GLOBALS['spip_' . $serveur_db . '_functions_' . $GLOBALS['spip_sql_version']];
+        $GLOBALS['connexions'][$serveur_db] = $link;
+        define('_DECLARER_SERVEUR_DB', $serveur_db);
+        define('_DECLARER_ADRESSE_DB', $adresse_db);
+        define('_DECLARER_LOGIN_DB', $login_db);
+        define('_DECLARER_PASS_DB', $pass_db);
+        // si on est sur le meme serveur que connect.php
+        // indiquer quelle est la db utilisee pour l'exclure des choix possibles
+        if ($serveur_db == $def_serveur_db && $adresse_db == $def_adresse_db) {
+            set_request('main_db', $sel_db);
+        } else {
+            set_request('main_db', '');
+        }
+    } else {
+        $erreurs['message_erreur'] = _T('avis_connexion_echec_1');
+    }
+
+    return $erreurs;
 }
 
 function formulaires_declarer_bases_verifier_2_dist() {
-	$erreurs = [];
-	$choix_db = _request('choix_db');
-	if ($choix_db == '-1') {
-		$choix_db = _request('table_new');
-	}
-	if (!$choix_db) {
-		$erreurs['choix_db'] = _T('info_obligatoire');
-	} else {
-		define('_ECRIRE_INSTALL', 1); // hackons sqlite
-		if (!sql_selectdb($choix_db, _DECLARER_SERVEUR_DB)) {
-			$erreurs['choix_db'] = _T('avis_base_inaccessible', ['base' => $choix_db]);
-		} else {
-			define('_DECLARER_CHOIX_DB', $choix_db);
-		}
-	}
-
-	return $erreurs;
+    $erreurs = [];
+    $choix_db = _request('choix_db');
+    if ($choix_db == '-1') {
+        $choix_db = _request('table_new');
+    }
+    if (!$choix_db) {
+        $erreurs['choix_db'] = _T('info_obligatoire');
+    } else {
+        define('_ECRIRE_INSTALL', 1); // hackons sqlite
+        if (!sql_selectdb($choix_db, _DECLARER_SERVEUR_DB)) {
+            $erreurs['choix_db'] = _T('avis_base_inaccessible', ['base' => $choix_db]);
+        } else {
+            define('_DECLARER_CHOIX_DB', $choix_db);
+        }
+    }
+
+    return $erreurs;
 }
 
 function formulaires_declarer_bases_verifier_3_dist() {
-	$erreurs = [];
-	$nom_connect = _request('nom_connect');
-	if (!$nom_connect) {
-		$erreurs['nom_connect'] = _T('info_obligatoire');
-	} else {
-		// securite : le nom doit etre un mot sans caracteres speciaux
-		$f = preg_replace(',[^\w],', '', (string) $nom_connect);
-		if ($f !== $nom_connect) {
-			$erreurs['nom_connect'] = _T('erreur_nom_connect_incorrect');
-		} elseif (file_exists(_DIR_CONNECT . $nom_connect . '.php')) {
-			$erreurs['nom_connect'] = _T('erreur_connect_deja_existant');
-		} else {
-			define('_DECLARER_NOM_CONNECT', $nom_connect);
-		}
-	}
-
-	return $erreurs;
+    $erreurs = [];
+    $nom_connect = _request('nom_connect');
+    if (!$nom_connect) {
+        $erreurs['nom_connect'] = _T('info_obligatoire');
+    } else {
+        // securite : le nom doit etre un mot sans caracteres speciaux
+        $f = preg_replace(',[^\w],', '', (string) $nom_connect);
+        if ($f !== $nom_connect) {
+            $erreurs['nom_connect'] = _T('erreur_nom_connect_incorrect');
+        } elseif (file_exists(_DIR_CONNECT . $nom_connect . '.php')) {
+            $erreurs['nom_connect'] = _T('erreur_connect_deja_existant');
+        } else {
+            define('_DECLARER_NOM_CONNECT', $nom_connect);
+        }
+    }
+
+    return $erreurs;
 }
 
 function formulaires_declarer_bases_traiter_dist() {
 
-	$adresse_db = _DECLARER_ADRESSE_DB;
-	if (preg_match(',(.*):(.*),', (string) $adresse_db, $r)) {
-		[, $adresse_db, $port] = $r;
-	} else {
-		$port = '';
-	}
-
-	$server_db = addcslashes((string) _DECLARER_SERVEUR_DB, "'\\");
-
-	$conn = install_mode_appel($server_db)
-		. install_connexion(
-			$adresse_db,
-			$port,
-			_DECLARER_LOGIN_DB,
-			_DECLARER_PASS_DB,
-			_DECLARER_CHOIX_DB,
-			_DECLARER_SERVEUR_DB,
-			'',
-			'',
-			''
-		);
-
-	install_fichier_connexion(_DIR_CONNECT . _DECLARER_NOM_CONNECT . '.php', $conn);
-
-	return [
-		'message_ok' => _T('install_connect_ok', ['connect' => '<strong>' . _DECLARER_NOM_CONNECT . '</strong>'])
-	];
+    $adresse_db = _DECLARER_ADRESSE_DB;
+    if (preg_match(',(.*):(.*),', (string) $adresse_db, $r)) {
+        [, $adresse_db, $port] = $r;
+    } else {
+        $port = '';
+    }
+
+    $server_db = addcslashes((string) _DECLARER_SERVEUR_DB, "'\\");
+
+    $conn = install_mode_appel($server_db)
+        . install_connexion(
+            $adresse_db,
+            $port,
+            _DECLARER_LOGIN_DB,
+            _DECLARER_PASS_DB,
+            _DECLARER_CHOIX_DB,
+            _DECLARER_SERVEUR_DB,
+            '',
+            '',
+            ''
+        );
+
+    install_fichier_connexion(_DIR_CONNECT . _DECLARER_NOM_CONNECT . '.php', $conn);
+
+    return [
+        'message_ok' => _T('install_connect_ok', ['connect' => '<strong>' . _DECLARER_NOM_CONNECT . '</strong>'])
+    ];
 }

Spacing +8 added lines, -8 removed lines

@@ -29,7 +29,7 @@ 
 	if (defined('_DECLARER_CHOIX_DB')) {
 		$nom_connect = _DECLARER_CHOIX_DB;
 		$n = '';
-		while (in_array($nom_connect . $n, $deja)) {
+		while (in_array($nom_connect.$n, $deja)) {
 			$n = ($n ? $n + 1 : 1);
 		}
 		$nom_connect .= $n;
@@ -57,7 +57,7 @@ 
 
 function liste_serveurs() {
 	$options = [];
-	$dir = _DIR_RESTREINT . 'req/';
+	$dir = _DIR_RESTREINT.'req/';
 	$d = opendir($dir);
 	if (!$d) {
 		return [];
@@ -65,11 +65,11 @@ 
 	while ($f = readdir($d)) {
 		if (
 			preg_match('/^(.*)[.]php$/', $f, $s)
-			&& is_readable($f = $dir . $f)
+			&& is_readable($f = $dir.$f)
 		) {
 			require_once($f);
 			$s = $s[1];
-			$v = 'spip_versions_' . $s;
+			$v = 'spip_versions_'.$s;
 			if (function_exists($v) && $v()) {
 				$options[$s] = "install_select_type_$s";
 			} else {
@@ -127,7 +127,7 @@ 
 
 	$link = spip_connect_db($adresse_db, '', $login_db, $pass_db, '@test@', $serveur_db);
 	if ($link) {
-		$GLOBALS['connexions'][$serveur_db][$GLOBALS['spip_sql_version']] = $GLOBALS['spip_' . $serveur_db . '_functions_' . $GLOBALS['spip_sql_version']];
+		$GLOBALS['connexions'][$serveur_db][$GLOBALS['spip_sql_version']] = $GLOBALS['spip_'.$serveur_db.'_functions_'.$GLOBALS['spip_sql_version']];
 		$GLOBALS['connexions'][$serveur_db] = $link;
 		define('_DECLARER_SERVEUR_DB', $serveur_db);
 		define('_DECLARER_ADRESSE_DB', $adresse_db);
@@ -177,7 +177,7 @@ 
 		$f = preg_replace(',[^\w],', '', (string) $nom_connect);
 		if ($f !== $nom_connect) {
 			$erreurs['nom_connect'] = _T('erreur_nom_connect_incorrect');
-		} elseif (file_exists(_DIR_CONNECT . $nom_connect . '.php')) {
+		} elseif (file_exists(_DIR_CONNECT.$nom_connect.'.php')) {
 			$erreurs['nom_connect'] = _T('erreur_connect_deja_existant');
 		} else {
 			define('_DECLARER_NOM_CONNECT', $nom_connect);
@@ -211,9 +211,9 @@ 
 			''
 		);
 
-	install_fichier_connexion(_DIR_CONNECT . _DECLARER_NOM_CONNECT . '.php', $conn);
+	install_fichier_connexion(_DIR_CONNECT._DECLARER_NOM_CONNECT.'.php', $conn);
 
 	return [
-		'message_ok' => _T('install_connect_ok', ['connect' => '<strong>' . _DECLARER_NOM_CONNECT . '</strong>'])
+		'message_ok' => _T('install_connect_ok', ['connect' => '<strong>'._DECLARER_NOM_CONNECT.'</strong>'])
 	];
 }

prive/formulaires/login.php

Spacing +5 added lines, -5 removed lines

@@ -143,7 +143,7 @@ 
 			# preparer un lien pour quand redirige_formulaire ne fonctionne pas
 			$m = redirige_formulaire($res['redirect']);
 			$valeurs['_deja_loge'] = inserer_attribut(
-				'<a>' . _T('login_par_ici') . "</a>$m",
+				'<a>'._T('login_par_ici')."</a>$m",
 				'href',
 				$res['redirect']
 			);
@@ -290,7 +290,7 @@ 
 function login_autoriser() {
 	include_spip('inc/autoriser');
 	if (!autoriser('ecrire')) {
-		$h = generer_url_action('logout', 'logout=prive&url=' . urlencode((string) self()));
+		$h = generer_url_action('logout', 'logout=prive&url='.urlencode((string) self()));
 
 		return [
 			'message_erreur' => '<h1>'
@@ -298,7 +298,7 @@ 
 				. '</h1><p>'
 				. _T('texte_erreur_visiteur')
 				. "</p><p class='retour'>[<a href='$h'>"
-				. _T('icone_deconnecter') . '</a>]</p>'
+				. _T('icone_deconnecter').'</a>]</p>'
 		];
 	}
 
@@ -350,7 +350,7 @@ 
 		// transformer la cible absolue en cible relative
 		// pour pas echouer quand la meta adresse_site est foireuse
 		if (strncmp((string) $cible, (string) ($u = url_de_base()), strlen((string) $u)) == 0) {
-			$cible = './' . substr((string) $cible, strlen((string) $u));
+			$cible = './'.substr((string) $cible, strlen((string) $u));
 		} elseif (tester_url_absolue($cible) && !defined('_AUTORISER_LOGIN_ABS_REDIRECT')) {
 			// si c'est une url absolue, refuser la redirection
 			// sauf si cette securite est levee volontairement par le webmestre
@@ -362,7 +362,7 @@ 
 	if ($cible && $cible != self('&') && $cible != self()) {
 		$res['redirect'] = $cible;
 		$res['message_ok'] = inserer_attribut(
-			'<a>' . _T('login_par_ici') . '</a>',
+			'<a>'._T('login_par_ici').'</a>',
 			'href',
 			$cible
 		);

Indentation +247 added lines, -247 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('base/abstract_sql');
@@ -30,11 +30,11 @@ 
  *     true si espace privé, false sinon.
  **/
 function is_url_prive($cible) {
-	include_spip('inc/filtres_mini');
-	$path = parse_url(tester_url_absolue($cible) ? $cible : url_absolue($cible));
-	$path = ($path['path'] ?? '');
+    include_spip('inc/filtres_mini');
+    $path = parse_url(tester_url_absolue($cible) ? $cible : url_absolue($cible));
+    $path = ($path['path'] ?? '');
 
-	return str_starts_with(substr($path, -strlen((string) _DIR_RESTREINT_ABS)), (string) _DIR_RESTREINT_ABS);
+    return str_starts_with(substr($path, -strlen((string) _DIR_RESTREINT_ABS)), (string) _DIR_RESTREINT_ABS);
 }
 
 /**
@@ -59,110 +59,110 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_login_charger_dist($cible = '', $options = [], $deprecated = null) {
-	$erreur = _request('var_erreur');
-
-	if (!is_array($options)) {
-		$options = [
-			'login' => $options,
-			'prive' => $deprecated
-		];
-	}
-
-	$login = (empty($options['login']) ? '' : $options['login']);
-	$prive = (empty($options['prive']) ? null : $options['prive']);
-	// conserver le fonctionnement historique
-	if (!isset($options['autofocus'])) {
-		$options['autofocus'] = true;
-	}
-
-	if (!$login) {
-		$login = (string) _request('var_login');
-	}
-	// si on est deja identifie
-	if (!$login && isset($GLOBALS['visiteur_session']['email'])) {
-		$login = $GLOBALS['visiteur_session']['email'];
-	}
-	if (!$login && isset($GLOBALS['visiteur_session']['login'])) {
-		$login = $GLOBALS['visiteur_session']['login'];
-	}
-	// ou si on a un cookie admin
-	if (
-		!$login
-		&& isset($_COOKIE['spip_admin'])
-		&& preg_match(',^@(.*)$,', (string) $_COOKIE['spip_admin'], $regs)
-	) {
-		$login = $regs[1];
-	}
-
-	$lang = $GLOBALS['spip_lang'];
-	include_spip('inc/auth');
-	$row = auth_informer_login($login);
-
-	// retablir la langue de l'URL si forcee (on ignore la langue de l'auteur dans ce cas)
-	if (_request('lang') === $lang && $GLOBALS['spip_lang'] !== $lang) {
-		changer_langue($lang);
-	}
-
-	// Construire l'environnement du squelette
-	// Ne pas proposer de "rester connecte quelques jours"
-	// si la duree de l'alea est inferieure a 12 h (valeur par defaut)
-
-	$valeurs = [
-		'var_login' => $login,
-		'editable' => !$row,
-		'cnx' => $row['cnx'] ?? '0',
-		'auth_http' => login_auth_http(),
-		'rester_connecte' => ((_RENOUVELLE_ALEA < 12 * 3600) ? '' : ' '),
-		'_logo' => $row['logo'] ?? '',
-		'_alea_actuel' => $row['alea_actuel'] ?? '',
-		'_alea_futur' => $row['alea_futur'] ?? '',
-		'_pipeline' => 'affiche_formulaire_login', // faire passer le formulaire dans un pipe dedie pour les methodes auth
-		'_autofocus' => ($options['autofocus'] && $options['autofocus'] !== 'non') ? ' ' : '',
-	];
-
-	if ($erreur || !isset($GLOBALS['visiteur_session']['id_auteur']) || !$GLOBALS['visiteur_session']['id_auteur']) {
-		$valeurs['editable'] = true;
-	}
-
-	if (is_null($prive) ? is_url_prive($cible) : $prive) {
-		include_spip('inc/autoriser');
-		$loge = autoriser('ecrire');
-	} else {
-		$loge = (isset($GLOBALS['visiteur_session']['auth']) && $GLOBALS['visiteur_session']['auth'] != '');
-	}
-
-	// Si on est connecte, appeler traiter()
-	// et lancer la redirection si besoin
-	if (!$valeurs['editable'] && $loge && _request('formulaire_action') !== 'login') {
-		$traiter = charger_fonction('traiter', 'formulaires/login');
-		$res = $traiter($cible, $login, $prive);
-		$valeurs = array_merge($valeurs, $res);
-
-		if (isset($res['redirect']) && $res['redirect']) {
-			include_spip('inc/headers');
-			# preparer un lien pour quand redirige_formulaire ne fonctionne pas
-			$m = redirige_formulaire($res['redirect']);
-			$valeurs['_deja_loge'] = inserer_attribut(
-				'<a>' . _T('login_par_ici') . "</a>$m",
-				'href',
-				$res['redirect']
-			);
-		}
-	}
-	// en cas d'echec de cookie, inc_auth a renvoye vers le script de
-	// pose de cookie ; s'il n'est pas la, c'est echec cookie
-	// s'il est la, c'est probablement un bookmark sur bonjour=oui,
-	// et pas un echec cookie.
-	if ($erreur == 'cookie') {
-		$valeurs['echec_cookie'] = ' ';
-	} elseif ($erreur) {
-		// une erreur d'un SSO indique dans la redirection vers ici
-		// mais il faut se proteger de toute tentative d'injection malveilante
-		include_spip('inc/filtres');
-		$valeurs['message_erreur'] = textebrut($erreur);
-	}
-
-	return $valeurs;
+    $erreur = _request('var_erreur');
+
+    if (!is_array($options)) {
+        $options = [
+            'login' => $options,
+            'prive' => $deprecated
+        ];
+    }
+
+    $login = (empty($options['login']) ? '' : $options['login']);
+    $prive = (empty($options['prive']) ? null : $options['prive']);
+    // conserver le fonctionnement historique
+    if (!isset($options['autofocus'])) {
+        $options['autofocus'] = true;
+    }
+
+    if (!$login) {
+        $login = (string) _request('var_login');
+    }
+    // si on est deja identifie
+    if (!$login && isset($GLOBALS['visiteur_session']['email'])) {
+        $login = $GLOBALS['visiteur_session']['email'];
+    }
+    if (!$login && isset($GLOBALS['visiteur_session']['login'])) {
+        $login = $GLOBALS['visiteur_session']['login'];
+    }
+    // ou si on a un cookie admin
+    if (
+        !$login
+        && isset($_COOKIE['spip_admin'])
+        && preg_match(',^@(.*)$,', (string) $_COOKIE['spip_admin'], $regs)
+    ) {
+        $login = $regs[1];
+    }
+
+    $lang = $GLOBALS['spip_lang'];
+    include_spip('inc/auth');
+    $row = auth_informer_login($login);
+
+    // retablir la langue de l'URL si forcee (on ignore la langue de l'auteur dans ce cas)
+    if (_request('lang') === $lang && $GLOBALS['spip_lang'] !== $lang) {
+        changer_langue($lang);
+    }
+
+    // Construire l'environnement du squelette
+    // Ne pas proposer de "rester connecte quelques jours"
+    // si la duree de l'alea est inferieure a 12 h (valeur par defaut)
+
+    $valeurs = [
+        'var_login' => $login,
+        'editable' => !$row,
+        'cnx' => $row['cnx'] ?? '0',
+        'auth_http' => login_auth_http(),
+        'rester_connecte' => ((_RENOUVELLE_ALEA < 12 * 3600) ? '' : ' '),
+        '_logo' => $row['logo'] ?? '',
+        '_alea_actuel' => $row['alea_actuel'] ?? '',
+        '_alea_futur' => $row['alea_futur'] ?? '',
+        '_pipeline' => 'affiche_formulaire_login', // faire passer le formulaire dans un pipe dedie pour les methodes auth
+        '_autofocus' => ($options['autofocus'] && $options['autofocus'] !== 'non') ? ' ' : '',
+    ];
+
+    if ($erreur || !isset($GLOBALS['visiteur_session']['id_auteur']) || !$GLOBALS['visiteur_session']['id_auteur']) {
+        $valeurs['editable'] = true;
+    }
+
+    if (is_null($prive) ? is_url_prive($cible) : $prive) {
+        include_spip('inc/autoriser');
+        $loge = autoriser('ecrire');
+    } else {
+        $loge = (isset($GLOBALS['visiteur_session']['auth']) && $GLOBALS['visiteur_session']['auth'] != '');
+    }
+
+    // Si on est connecte, appeler traiter()
+    // et lancer la redirection si besoin
+    if (!$valeurs['editable'] && $loge && _request('formulaire_action') !== 'login') {
+        $traiter = charger_fonction('traiter', 'formulaires/login');
+        $res = $traiter($cible, $login, $prive);
+        $valeurs = array_merge($valeurs, $res);
+
+        if (isset($res['redirect']) && $res['redirect']) {
+            include_spip('inc/headers');
+            # preparer un lien pour quand redirige_formulaire ne fonctionne pas
+            $m = redirige_formulaire($res['redirect']);
+            $valeurs['_deja_loge'] = inserer_attribut(
+                '<a>' . _T('login_par_ici') . "</a>$m",
+                'href',
+                $res['redirect']
+            );
+        }
+    }
+    // en cas d'echec de cookie, inc_auth a renvoye vers le script de
+    // pose de cookie ; s'il n'est pas la, c'est echec cookie
+    // s'il est la, c'est probablement un bookmark sur bonjour=oui,
+    // et pas un echec cookie.
+    if ($erreur == 'cookie') {
+        $valeurs['echec_cookie'] = ' ';
+    } elseif ($erreur) {
+        // une erreur d'un SSO indique dans la redirection vers ici
+        // mais il faut se proteger de toute tentative d'injection malveilante
+        include_spip('inc/filtres');
+        $valeurs['message_erreur'] = textebrut($erreur);
+    }
+
+    return $valeurs;
 }
 
 
@@ -177,20 +177,20 @@ 
  *     - chaîne vide sinon.
  **/
 function login_auth_http() {
-	if (!$GLOBALS['ignore_auth_http'] && _request('var_erreur') == 'cookie') {
-		include_spip('inc/session');
-		$cookie = lire_cookie_session(true);
-		if ($cookie !== 'test_echec_cookie'
-		  && (preg_match(',apache,', \PHP_SAPI) || preg_match(',^Apache.* PHP,', (string) $_SERVER['SERVER_SOFTWARE']))
-		  // Attention dans le cas 'intranet' la proposition de se loger
-		  // par auth_http peut conduire a l'echec.
-		  && !(isset($_SERVER['PHP_AUTH_USER']) && isset($_SERVER['PHP_AUTH_PW']))
-		) {
-			return generer_url_action('cookie', '', false, true);
-		}
-	}
-
-	return '';
+    if (!$GLOBALS['ignore_auth_http'] && _request('var_erreur') == 'cookie') {
+        include_spip('inc/session');
+        $cookie = lire_cookie_session(true);
+        if ($cookie !== 'test_echec_cookie'
+          && (preg_match(',apache,', \PHP_SAPI) || preg_match(',^Apache.* PHP,', (string) $_SERVER['SERVER_SOFTWARE']))
+            // Attention dans le cas 'intranet' la proposition de se loger
+            // par auth_http peut conduire a l'echec.
+          && !(isset($_SERVER['PHP_AUTH_USER']) && isset($_SERVER['PHP_AUTH_PW']))
+        ) {
+            return generer_url_action('cookie', '', false, true);
+        }
+    }
+
+    return '';
 }
 
 
@@ -216,64 +216,64 @@ 
  **/
 function formulaires_login_verifier_dist($cible = '', $options = [], $deprecated = null) {
 
-	$erreurs = [];
-	if (!is_array($options)) {
-		$options = [
-			'login' => $options,
-			'prive' => $deprecated
-		];
-	}
-
-	$prive = (empty($options['prive']) ? null : $options['prive']);
-
-	$session_login = _request('var_login');
-	$session_password = _request('password');
-	$session_remember = _request('session_remember');
-
-	if (!$session_login) {
-		# pas de login saisi !
-		return ['var_login' => _T('info_obligatoire')];
-	}
-
-	// appeler auth_identifier_login qui va :
-	// - renvoyer un string si echec (message d'erreur)
-	// - un array decrivant l'auteur identifie si possible
-	// - rediriger vers un SSO qui renverra in fine sur action/auth qui finira l'authentification
-	include_spip('inc/auth');
-	$auteur = auth_identifier_login($session_login, $session_password);
-	// on arrive ici si on ne s'est pas identifie avec un SSO
-	if (!is_array($auteur)) {
-		$erreurs = [];
-		if (is_string($auteur) && strlen($auteur)) {
-			$erreurs['var_login'] = $auteur;
-		}
-		include_spip('inc/cookie');
-		spip_setcookie('spip_admin', '', time() - 3600, httponly: true);
-		if (strlen((string) $session_password)) {
-			$erreurs['password'] = _T('login_erreur_pass');
-		} else {
-			// sinon c'est un login en deux passe old style (ou js en panne)
-			// pas de message d'erreur
-			$erreurs['password'] = ' ';
-			$erreurs['message_erreur'] = '';
-		}
-
-		return $erreurs;
-	}
-	// on a ete authentifie, construire la session
-	// en gerant la duree demandee pour son cookie
-	if ($session_remember !== null) {
-		$auteur['cookie'] = $session_remember;
-	}
-	// si la connexion est refusee on renvoi un message erreur de mot de passe
-	// car en donnant plus de detail on renseignerait un assaillant sur l'existence d'un compte
-	if (auth_loger($auteur) === false) {
-		$erreurs['message_erreur'] = _T('login_erreur_pass');
-		return $erreurs;
-	}
-
-	return (is_null($prive) ? is_url_prive($cible) : $prive)
-		? login_autoriser() : [];
+    $erreurs = [];
+    if (!is_array($options)) {
+        $options = [
+            'login' => $options,
+            'prive' => $deprecated
+        ];
+    }
+
+    $prive = (empty($options['prive']) ? null : $options['prive']);
+
+    $session_login = _request('var_login');
+    $session_password = _request('password');
+    $session_remember = _request('session_remember');
+
+    if (!$session_login) {
+        # pas de login saisi !
+        return ['var_login' => _T('info_obligatoire')];
+    }
+
+    // appeler auth_identifier_login qui va :
+    // - renvoyer un string si echec (message d'erreur)
+    // - un array decrivant l'auteur identifie si possible
+    // - rediriger vers un SSO qui renverra in fine sur action/auth qui finira l'authentification
+    include_spip('inc/auth');
+    $auteur = auth_identifier_login($session_login, $session_password);
+    // on arrive ici si on ne s'est pas identifie avec un SSO
+    if (!is_array($auteur)) {
+        $erreurs = [];
+        if (is_string($auteur) && strlen($auteur)) {
+            $erreurs['var_login'] = $auteur;
+        }
+        include_spip('inc/cookie');
+        spip_setcookie('spip_admin', '', time() - 3600, httponly: true);
+        if (strlen((string) $session_password)) {
+            $erreurs['password'] = _T('login_erreur_pass');
+        } else {
+            // sinon c'est un login en deux passe old style (ou js en panne)
+            // pas de message d'erreur
+            $erreurs['password'] = ' ';
+            $erreurs['message_erreur'] = '';
+        }
+
+        return $erreurs;
+    }
+    // on a ete authentifie, construire la session
+    // en gerant la duree demandee pour son cookie
+    if ($session_remember !== null) {
+        $auteur['cookie'] = $session_remember;
+    }
+    // si la connexion est refusee on renvoi un message erreur de mot de passe
+    // car en donnant plus de detail on renseignerait un assaillant sur l'existence d'un compte
+    if (auth_loger($auteur) === false) {
+        $erreurs['message_erreur'] = _T('login_erreur_pass');
+        return $erreurs;
+    }
+
+    return (is_null($prive) ? is_url_prive($cible) : $prive)
+        ? login_autoriser() : [];
 }
 
 /**
@@ -288,21 +288,21 @@ 
  *     - tableau vide sinon.
  **/
 function login_autoriser() {
-	include_spip('inc/autoriser');
-	if (!autoriser('ecrire')) {
-		$h = generer_url_action('logout', 'logout=prive&url=' . urlencode((string) self()));
-
-		return [
-			'message_erreur' => '<h1>'
-				. _T('avis_erreur_visiteur')
-				. '</h1><p>'
-				. _T('texte_erreur_visiteur')
-				. "</p><p class='retour'>[<a href='$h'>"
-				. _T('icone_deconnecter') . '</a>]</p>'
-		];
-	}
-
-	return [];
+    include_spip('inc/autoriser');
+    if (!autoriser('ecrire')) {
+        $h = generer_url_action('logout', 'logout=prive&url=' . urlencode((string) self()));
+
+        return [
+            'message_erreur' => '<h1>'
+                . _T('avis_erreur_visiteur')
+                . '</h1><p>'
+                . _T('texte_erreur_visiteur')
+                . "</p><p class='retour'>[<a href='$h'>"
+                . _T('icone_deconnecter') . '</a>]</p>'
+        ];
+    }
+
+    return [];
 }
 
 /**
@@ -323,55 +323,55 @@ 
  *     Retours du traitement
  **/
 function formulaires_login_traiter_dist($cible = '', $options = [], $deprecated = null) {
-	$res = [];
-
-	if (!is_array($options)) {
-		$options = [
-			'login' => $options,
-			'prive' => $deprecated
-		];
-	}
-
-	$login = (empty($options['login']) ? '' : $options['login']);
-	$prive = (empty($options['prive']) ? null : $options['prive']);
-
-	// Si on se connecte dans l'espace prive,
-	// ajouter "bonjour" (repere a peu pres les cookies desactives)
-	if (is_null($prive) ? is_url_prive($cible) : $prive) {
-		$cible = parametre_url($cible, 'bonjour', 'oui', '&');
-	}
-	if ($cible == '@page_auteur') {
-		$cible = generer_objet_url($GLOBALS['auteur_session']['id_auteur'], 'auteur');
-	}
-
-	if ($cible) {
-		$cible = parametre_url($cible, 'var_login', '', '&');
-
-		// transformer la cible absolue en cible relative
-		// pour pas echouer quand la meta adresse_site est foireuse
-		if (strncmp((string) $cible, (string) ($u = url_de_base()), strlen((string) $u)) == 0) {
-			$cible = './' . substr((string) $cible, strlen((string) $u));
-		} elseif (tester_url_absolue($cible) && !defined('_AUTORISER_LOGIN_ABS_REDIRECT')) {
-			// si c'est une url absolue, refuser la redirection
-			// sauf si cette securite est levee volontairement par le webmestre
-			$cible = '';
-		}
-	}
-
-	// Si on est connecte, envoyer vers la destination
-	if ($cible && $cible != self('&') && $cible != self()) {
-		$res['redirect'] = $cible;
-		$res['message_ok'] = inserer_attribut(
-			'<a>' . _T('login_par_ici') . '</a>',
-			'href',
-			$cible
-		);
-	}
-
-	// avant de rediriger il faut mettre a jour les sessions sur le disque si on a charge une session
-	if (function_exists('terminer_actualiser_sessions')) {
-		terminer_actualiser_sessions();
-	}
-
-	return $res;
+    $res = [];
+
+    if (!is_array($options)) {
+        $options = [
+            'login' => $options,
+            'prive' => $deprecated
+        ];
+    }
+
+    $login = (empty($options['login']) ? '' : $options['login']);
+    $prive = (empty($options['prive']) ? null : $options['prive']);
+
+    // Si on se connecte dans l'espace prive,
+    // ajouter "bonjour" (repere a peu pres les cookies desactives)
+    if (is_null($prive) ? is_url_prive($cible) : $prive) {
+        $cible = parametre_url($cible, 'bonjour', 'oui', '&');
+    }
+    if ($cible == '@page_auteur') {
+        $cible = generer_objet_url($GLOBALS['auteur_session']['id_auteur'], 'auteur');
+    }
+
+    if ($cible) {
+        $cible = parametre_url($cible, 'var_login', '', '&');
+
+        // transformer la cible absolue en cible relative
+        // pour pas echouer quand la meta adresse_site est foireuse
+        if (strncmp((string) $cible, (string) ($u = url_de_base()), strlen((string) $u)) == 0) {
+            $cible = './' . substr((string) $cible, strlen((string) $u));
+        } elseif (tester_url_absolue($cible) && !defined('_AUTORISER_LOGIN_ABS_REDIRECT')) {
+            // si c'est une url absolue, refuser la redirection
+            // sauf si cette securite est levee volontairement par le webmestre
+            $cible = '';
+        }
+    }
+
+    // Si on est connecte, envoyer vers la destination
+    if ($cible && $cible != self('&') && $cible != self()) {
+        $res['redirect'] = $cible;
+        $res['message_ok'] = inserer_attribut(
+            '<a>' . _T('login_par_ici') . '</a>',
+            'href',
+            $cible
+        );
+    }
+
+    // avant de rediriger il faut mettre a jour les sessions sur le disque si on a charge une session
+    if (function_exists('terminer_actualiser_sessions')) {
+        terminer_actualiser_sessions();
+    }
+
+    return $res;
 }

prive/formulaires/editer_rubrique.php

Indentation +77 added lines, -77 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -46,30 +46,30 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_editer_rubrique_charger_dist(
-	$id_rubrique = 'new',
-	$id_parent = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'rubriques_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_rubrique = 'new',
+    $id_parent = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'rubriques_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	$valeurs = formulaires_editer_objet_charger(
-		'rubrique',
-		$id_rubrique,
-		$id_parent,
-		$lier_trad,
-		$retour,
-		$config_fonc,
-		$row,
-		$hidden
-	);
+    $valeurs = formulaires_editer_objet_charger(
+        'rubrique',
+        $id_rubrique,
+        $id_parent,
+        $lier_trad,
+        $retour,
+        $config_fonc,
+        $row,
+        $hidden
+    );
 
-	if ((int) $id_rubrique && !autoriser('modifier', 'rubrique', (int) $id_rubrique)) {
-		$valeurs['editable'] = '';
-	}
+    if ((int) $id_rubrique && !autoriser('modifier', 'rubrique', (int) $id_rubrique)) {
+        $valeurs['editable'] = '';
+    }
 
-	return $valeurs;
+    return $valeurs;
 }
 
 /**
@@ -81,11 +81,11 @@ 
  *     Configuration pour le formulaire
  */
 function rubriques_edit_config(array $row): array {
-	return [
-		'lignes' => 8,
-		'langue' => $GLOBALS['spip_lang'],
-		'restreint' => !$GLOBALS['connect_toutes_rubriques']
-	];
+    return [
+        'lignes' => 8,
+        'langue' => $GLOBALS['spip_lang'],
+        'restreint' => !$GLOBALS['connect_toutes_rubriques']
+    ];
 }
 
 /**
@@ -110,15 +110,15 @@ 
  *     Hash du formulaire
  */
 function formulaires_editer_rubrique_identifier_dist(
-	$id_rubrique = 'new',
-	$id_parent = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'rubriques_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_rubrique = 'new',
+    $id_parent = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'rubriques_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	return serialize([(int) $id_rubrique, $lier_trad]);
+    return serialize([(int) $id_rubrique, $lier_trad]);
 }
 
 /**
@@ -144,34 +144,34 @@ 
  *     Erreurs du formulaire
  **/
 function formulaires_editer_rubrique_verifier_dist(
-	$id_rubrique = 'new',
-	$id_parent = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'rubriques_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_rubrique = 'new',
+    $id_parent = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'rubriques_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	// auto-renseigner le titre si il n'existe pas
-	titre_automatique('titre', ['descriptif', 'texte']);
-	// on ne demande pas le titre obligatoire : il sera rempli a la volee dans editer_rubrique si vide
-	$erreurs = formulaires_editer_objet_verifier('rubrique', $id_rubrique, []);
+    // auto-renseigner le titre si il n'existe pas
+    titre_automatique('titre', ['descriptif', 'texte']);
+    // on ne demande pas le titre obligatoire : il sera rempli a la volee dans editer_rubrique si vide
+    $erreurs = formulaires_editer_objet_verifier('rubrique', $id_rubrique, []);
 
-	// s'assurer qu'on ne s'auto-designe pas comme parent !
-	if (
-		(int) $id_rubrique
-		&& empty($erreurs['id_parent'])
-		&& ($id_parent = _request('id_parent'))
-	) {
-		include_spip('inc/rubriques');
-		$branche = calcul_branche_in($id_rubrique);
-		$branche = explode(',', (string) $branche);
-		if (in_array($id_parent, $branche)) {
-			$erreurs['id_parent'] = _L('Impossible de déplacer une rubrique dans sa propre branche, on tourne en rond !');
-		}
-	}
+    // s'assurer qu'on ne s'auto-designe pas comme parent !
+    if (
+        (int) $id_rubrique
+        && empty($erreurs['id_parent'])
+        && ($id_parent = _request('id_parent'))
+    ) {
+        include_spip('inc/rubriques');
+        $branche = calcul_branche_in($id_rubrique);
+        $branche = explode(',', (string) $branche);
+        if (in_array($id_parent, $branche)) {
+            $erreurs['id_parent'] = _L('Impossible de déplacer une rubrique dans sa propre branche, on tourne en rond !');
+        }
+    }
 
-	return $erreurs;
+    return $erreurs;
 }
 
 /**
@@ -197,22 +197,22 @@ 
  *     Retour des traitements
  **/
 function formulaires_editer_rubrique_traiter_dist(
-	$id_rubrique = 'new',
-	$id_parent = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'rubriques_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_rubrique = 'new',
+    $id_parent = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'rubriques_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	return formulaires_editer_objet_traiter(
-		'rubrique',
-		$id_rubrique,
-		$id_parent,
-		$lier_trad,
-		$retour,
-		$config_fonc,
-		$row,
-		$hidden
-	);
+    return formulaires_editer_objet_traiter(
+        'rubrique',
+        $id_rubrique,
+        $id_parent,
+        $lier_trad,
+        $retour,
+        $config_fonc,
+        $row,
+        $hidden
+    );
 }

prive/transmettre/rss/a_suivre_fonctions.php

Indentation +14 added lines, -14 removed lines

@@ -1,22 +1,22 @@
 <?php
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function trier_rss($texte) {
-	if (preg_match_all(',<item.*</item>\s*?,Uims', (string) $texte, $matches, PREG_SET_ORDER)) {
-		$placeholder = '<!--REINSERT-->';
-		$items = [];
-		foreach ($matches as $match) {
-			if (preg_match(',<dc:date>(.*)</dc:date>,Uims', $match[0], $r)) {
-				$items[strtotime($r[1])] = trim($match[0]);
-				$texte = str_replace($match[0], unique($placeholder), (string) $texte);
-			}
-		}
-		krsort($items);
-		$texte = str_replace($placeholder, implode("\n\t", $items) . "\n", (string) $texte);
-	}
+    if (preg_match_all(',<item.*</item>\s*?,Uims', (string) $texte, $matches, PREG_SET_ORDER)) {
+        $placeholder = '<!--REINSERT-->';
+        $items = [];
+        foreach ($matches as $match) {
+            if (preg_match(',<dc:date>(.*)</dc:date>,Uims', $match[0], $r)) {
+                $items[strtotime($r[1])] = trim($match[0]);
+                $texte = str_replace($match[0], unique($placeholder), (string) $texte);
+            }
+        }
+        krsort($items);
+        $texte = str_replace($placeholder, implode("\n\t", $items) . "\n", (string) $texte);
+    }
 
-	return $texte;
+    return $texte;
 }

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 			}
 		}
 		krsort($items);
-		$texte = str_replace($placeholder, implode("\n\t", $items) . "\n", (string) $texte);
+		$texte = str_replace($placeholder, implode("\n\t", $items)."\n", (string) $texte);
 	}
 
 	return $texte;

prive/objets/liste/articles_fonctions.php

Indentation +27 added lines, -27 removed lines

@@ -10,40 +10,40 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function defaut_tri_defined($defaut) {
-	if (!defined('_TRI_ARTICLES_RUBRIQUE')) {
-		return $defaut;
-	}
+    if (!defined('_TRI_ARTICLES_RUBRIQUE')) {
+        return $defaut;
+    }
 
-	$sens = 1;
-	$tri = trim((string) _TRI_ARTICLES_RUBRIQUE);
-	$tri = explode(' ', $tri);
-	if (strncasecmp(end($tri), 'DESC', 4) == 0) {
-		$sens = -1;
-		array_pop($tri);
-	} elseif (strncasecmp(end($tri), 'ASC', 3) == 0) {
-		$sens = 1;
-		array_pop($tri);
-	}
-	$tri = implode(' ', $tri);
-	$tri = [$tri => $sens];
-	foreach ($defaut as $n => $s) {
-		if (!isset($tri[$n])) {
-			$tri[$n] = $s;
-		}
-	}
+    $sens = 1;
+    $tri = trim((string) _TRI_ARTICLES_RUBRIQUE);
+    $tri = explode(' ', $tri);
+    if (strncasecmp(end($tri), 'DESC', 4) == 0) {
+        $sens = -1;
+        array_pop($tri);
+    } elseif (strncasecmp(end($tri), 'ASC', 3) == 0) {
+        $sens = 1;
+        array_pop($tri);
+    }
+    $tri = implode(' ', $tri);
+    $tri = [$tri => $sens];
+    foreach ($defaut as $n => $s) {
+        if (!isset($tri[$n])) {
+            $tri[$n] = $s;
+        }
+    }
 
-	return $tri;
+    return $tri;
 }
 
 function defaut_tri_par($par, $defaut) {
-	if (!defined('_TRI_ARTICLES_RUBRIQUE')) {
-		return $par;
-	}
-	$par = array_keys($defaut);
+    if (!defined('_TRI_ARTICLES_RUBRIQUE')) {
+        return $par;
+    }
+    $par = array_keys($defaut);
 
-	return reset($par);
+    return reset($par);
 }

ecrire/notifications/instituerarticle.php

Indentation +44 added lines, -44 removed lines

@@ -10,53 +10,53 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 // Fonction appelee par divers pipelines
 function notifications_instituerarticle_dist($quoi, $id_article, $options) {
 
-	// ne devrait jamais se produire
-	if ($options['statut'] == $options['statut_ancien']) {
-		spip_log('statut inchange', 'notifications');
-
-		return;
-	}
-
-	include_spip('inc/texte');
-
-	$modele = '';
-	if ($options['statut'] == 'publie') {
-		if (
-			$GLOBALS['meta']['post_dates'] == 'non'
-			&& strtotime((string) $options['date']) > time()
-		) {
-			$modele = 'notifications/article_valide';
-		} else {
-			$modele = 'notifications/article_publie';
-		}
-	}
-
-	if ($options['statut'] == 'prop' && $options['statut_ancien'] != 'publie') {
-		$modele = 'notifications/article_propose';
-	}
-
-	if ($modele) {
-		$destinataires = [];
-		if ($GLOBALS['meta']['suivi_edito'] == 'oui') {
-			$destinataires = explode(',', (string) $GLOBALS['meta']['adresse_suivi']);
-		}
-
-
-		$destinataires = pipeline(
-			'notifications_destinataires',
-			[
-				'args' => ['quoi' => $quoi, 'id' => $id_article, 'options' => $options],
-				'data' => $destinataires
-			]
-		);
-
-		$texte = email_notification_article($id_article, $modele);
-		notifications_envoyer_mails($destinataires, $texte);
-	}
+    // ne devrait jamais se produire
+    if ($options['statut'] == $options['statut_ancien']) {
+        spip_log('statut inchange', 'notifications');
+
+        return;
+    }
+
+    include_spip('inc/texte');
+
+    $modele = '';
+    if ($options['statut'] == 'publie') {
+        if (
+            $GLOBALS['meta']['post_dates'] == 'non'
+            && strtotime((string) $options['date']) > time()
+        ) {
+            $modele = 'notifications/article_valide';
+        } else {
+            $modele = 'notifications/article_publie';
+        }
+    }
+
+    if ($options['statut'] == 'prop' && $options['statut_ancien'] != 'publie') {
+        $modele = 'notifications/article_propose';
+    }
+
+    if ($modele) {
+        $destinataires = [];
+        if ($GLOBALS['meta']['suivi_edito'] == 'oui') {
+            $destinataires = explode(',', (string) $GLOBALS['meta']['adresse_suivi']);
+        }
+
+
+        $destinataires = pipeline(
+            'notifications_destinataires',
+            [
+                'args' => ['quoi' => $quoi, 'id' => $id_article, 'options' => $options],
+                'data' => $destinataires
+            ]
+        );
+
+        $texte = email_notification_article($id_article, $modele);
+        notifications_envoyer_mails($destinataires, $texte);
+    }
 }

ecrire/install/etape_chmod.php

Indentation +114 added lines, -114 removed lines

@@ -10,11 +10,11 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 if (defined('_TEST_DIRS')) {
-	return;
+    return;
 }
 define('_TEST_DIRS', '1');
 
@@ -26,38 +26,38 @@ 
 // Tente d'ecrire
 //
 function test_ecrire($my_dir) {
-	static $chmod = 0;
-
-	$ok = false;
-	$script = @file_exists('spip_loader.php') ? 'spip_loader.php' : $_SERVER['PHP_SELF'];
-	$self = basename((string) $script);
-	$uid = @fileowner('.');
-	$uid2 = @fileowner($self);
-	$gid = @filegroup('.');
-	$gid2 = @filegroup($self);
-	$perms = @fileperms($self);
-
-	// Comparer l'appartenance d'un fichier cree par PHP
-	// avec celle du script et du repertoire courant
-	if (!$chmod) {
-		@rmdir('test');
-		spip_unlink('test'); // effacer au cas ou
-		@touch('test');
-		if ($uid > 0 && $uid == $uid2 && @fileowner('test') == $uid) {
-			$chmod = 0700;
-		} else {
-			$chmod = $gid > 0 && $gid == $gid2 && @filegroup('test') == $gid ? 0770 : 0777;
-		}
-		// Appliquer de plus les droits d'acces du script
-		if ($perms > 0) {
-			$perms = ($perms & 0777) | (($perms & 0444) >> 2);
-			$chmod |= $perms;
-		}
-		spip_unlink('test');
-	}
-	$ok = is_dir($my_dir) && is_writable($my_dir);
-
-	return $ok ? $chmod : false;
+    static $chmod = 0;
+
+    $ok = false;
+    $script = @file_exists('spip_loader.php') ? 'spip_loader.php' : $_SERVER['PHP_SELF'];
+    $self = basename((string) $script);
+    $uid = @fileowner('.');
+    $uid2 = @fileowner($self);
+    $gid = @filegroup('.');
+    $gid2 = @filegroup($self);
+    $perms = @fileperms($self);
+
+    // Comparer l'appartenance d'un fichier cree par PHP
+    // avec celle du script et du repertoire courant
+    if (!$chmod) {
+        @rmdir('test');
+        spip_unlink('test'); // effacer au cas ou
+        @touch('test');
+        if ($uid > 0 && $uid == $uid2 && @fileowner('test') == $uid) {
+            $chmod = 0700;
+        } else {
+            $chmod = $gid > 0 && $gid == $gid2 && @filegroup('test') == $gid ? 0770 : 0777;
+        }
+        // Appliquer de plus les droits d'acces du script
+        if ($perms > 0) {
+            $perms = ($perms & 0777) | (($perms & 0444) >> 2);
+            $chmod |= $perms;
+        }
+        spip_unlink('test');
+    }
+    $ok = is_dir($my_dir) && is_writable($my_dir);
+
+    return $ok ? $chmod : false;
 }
 
 //
@@ -67,84 +67,84 @@ 
 
 function install_etape_chmod_dist() {
 
-	$continuer = null;
-	$test_dir = _request('test_dir');
-	$chmod = 0;
-
-	if ($test_dir && !str_contains((string) $test_dir, '..')) {
-		if (!str_ends_with((string) $test_dir, '/')) {
-			$test_dir .= '/';
-		}
-		if (!in_array($test_dir, $GLOBALS['test_dirs'])) {
-			$GLOBALS['test_dirs'][] = _DIR_RACINE . $test_dir;
-		}
-	} else {
-		if (!_FILE_CONNECT) {
-			$GLOBALS['test_dirs'][] = _DIR_CONNECT;
-			$GLOBALS['test_dirs'][] = _DIR_CHMOD;
-		}
-	}
-
-	$bad_dirs = [];
-	$absent_dirs = [];
-
-	foreach ($GLOBALS['test_dirs'] as $i => $my_dir) {
-		$test = test_ecrire($my_dir);
-		if (!$test) {
-			$m = preg_replace(',^' . _DIR_RACINE . ',', '', (string) $my_dir);
-			if (@file_exists($my_dir)) {
-				$bad_dirs['<li>' . $m . '</li>'] = 1;
-			} else {
-				$absent_dirs['<li>' . $m . '</li>'] = 1;
-			}
-		} else {
-			$chmod = max($chmod, $test);
-		}
-	}
-
-	if ($bad_dirs || $absent_dirs) {
-		if (!_FILE_CONNECT) {
-			$titre = _T('dirs_preliminaire');
-			$continuer = ' ' . _T('dirs_commencer') . '.';
-		} else {
-			$titre = _T('dirs_probleme_droits');
-		}
-
-
-		$res = "<div align='right'>" . menu_langues('var_lang_ecrire') . "</div>\n";
-
-		if ($bad_dirs) {
-			$res .=
-				_T(
-					'dirs_repertoires_suivants',
-					['bad_dirs' => implode("\n", array_keys($bad_dirs))]
-				) .
-				'<b>' . _T('login_recharger') . '</b>.';
-		}
-
-		if ($absent_dirs) {
-			$res .=
-				_T(
-					'dirs_repertoires_absents',
-					['bad_dirs' => implode("\n", array_keys($absent_dirs))]
-				) .
-				'<b>' . _T('login_recharger') . '</b>.';
-		}
-		$res = '<p>' . $continuer . $res . aider('install0', true) . '</p>';
-
-		$t = _T('login_recharger');
-		$t = ($test_dir ? "<input type='hidden' name='test_dir' value='" . spip_htmlspecialchars($test_dir, ENT_QUOTES) . "' />" :
-				'')
-			. "<input type='hidden' name='etape' value='chmod' />"
-			. "<div style='text-align: right'><input type='submit' value='" . attribut_html($t) . "' /></div>";
-
-		echo minipres($titre, $res . generer_form_ecrire('install', $t));
-	} else {
-		$deja = (_FILE_CONNECT && analyse_fichier_connection(_FILE_CONNECT));
-		if (!$deja) {
-			redirige_url_ecrire('install', 'etape=1&chmod=' . $chmod);
-		} else {
-			redirige_url_ecrire();
-		}
-	}
+    $continuer = null;
+    $test_dir = _request('test_dir');
+    $chmod = 0;
+
+    if ($test_dir && !str_contains((string) $test_dir, '..')) {
+        if (!str_ends_with((string) $test_dir, '/')) {
+            $test_dir .= '/';
+        }
+        if (!in_array($test_dir, $GLOBALS['test_dirs'])) {
+            $GLOBALS['test_dirs'][] = _DIR_RACINE . $test_dir;
+        }
+    } else {
+        if (!_FILE_CONNECT) {
+            $GLOBALS['test_dirs'][] = _DIR_CONNECT;
+            $GLOBALS['test_dirs'][] = _DIR_CHMOD;
+        }
+    }
+
+    $bad_dirs = [];
+    $absent_dirs = [];
+
+    foreach ($GLOBALS['test_dirs'] as $i => $my_dir) {
+        $test = test_ecrire($my_dir);
+        if (!$test) {
+            $m = preg_replace(',^' . _DIR_RACINE . ',', '', (string) $my_dir);
+            if (@file_exists($my_dir)) {
+                $bad_dirs['<li>' . $m . '</li>'] = 1;
+            } else {
+                $absent_dirs['<li>' . $m . '</li>'] = 1;
+            }
+        } else {
+            $chmod = max($chmod, $test);
+        }
+    }
+
+    if ($bad_dirs || $absent_dirs) {
+        if (!_FILE_CONNECT) {
+            $titre = _T('dirs_preliminaire');
+            $continuer = ' ' . _T('dirs_commencer') . '.';
+        } else {
+            $titre = _T('dirs_probleme_droits');
+        }
+
+
+        $res = "<div align='right'>" . menu_langues('var_lang_ecrire') . "</div>\n";
+
+        if ($bad_dirs) {
+            $res .=
+                _T(
+                    'dirs_repertoires_suivants',
+                    ['bad_dirs' => implode("\n", array_keys($bad_dirs))]
+                ) .
+                '<b>' . _T('login_recharger') . '</b>.';
+        }
+
+        if ($absent_dirs) {
+            $res .=
+                _T(
+                    'dirs_repertoires_absents',
+                    ['bad_dirs' => implode("\n", array_keys($absent_dirs))]
+                ) .
+                '<b>' . _T('login_recharger') . '</b>.';
+        }
+        $res = '<p>' . $continuer . $res . aider('install0', true) . '</p>';
+
+        $t = _T('login_recharger');
+        $t = ($test_dir ? "<input type='hidden' name='test_dir' value='" . spip_htmlspecialchars($test_dir, ENT_QUOTES) . "' />" :
+                '')
+            . "<input type='hidden' name='etape' value='chmod' />"
+            . "<div style='text-align: right'><input type='submit' value='" . attribut_html($t) . "' /></div>";
+
+        echo minipres($titre, $res . generer_form_ecrire('install', $t));
+    } else {
+        $deja = (_FILE_CONNECT && analyse_fichier_connection(_FILE_CONNECT));
+        if (!$deja) {
+            redirige_url_ecrire('install', 'etape=1&chmod=' . $chmod);
+        } else {
+            redirige_url_ecrire();
+        }
+    }
 }

Spacing +15 added lines, -16 removed lines

@@ -76,7 +76,7 @@ 
 			$test_dir .= '/';
 		}
 		if (!in_array($test_dir, $GLOBALS['test_dirs'])) {
-			$GLOBALS['test_dirs'][] = _DIR_RACINE . $test_dir;
+			$GLOBALS['test_dirs'][] = _DIR_RACINE.$test_dir;
 		}
 	} else {
 		if (!_FILE_CONNECT) {
@@ -91,11 +91,11 @@ 
 	foreach ($GLOBALS['test_dirs'] as $i => $my_dir) {
 		$test = test_ecrire($my_dir);
 		if (!$test) {
-			$m = preg_replace(',^' . _DIR_RACINE . ',', '', (string) $my_dir);
+			$m = preg_replace(',^'._DIR_RACINE.',', '', (string) $my_dir);
 			if (@file_exists($my_dir)) {
-				$bad_dirs['<li>' . $m . '</li>'] = 1;
+				$bad_dirs['<li>'.$m.'</li>'] = 1;
 			} else {
-				$absent_dirs['<li>' . $m . '</li>'] = 1;
+				$absent_dirs['<li>'.$m.'</li>'] = 1;
 			}
 		} else {
 			$chmod = max($chmod, $test);
@@ -105,21 +105,21 @@ 
 	if ($bad_dirs || $absent_dirs) {
 		if (!_FILE_CONNECT) {
 			$titre = _T('dirs_preliminaire');
-			$continuer = ' ' . _T('dirs_commencer') . '.';
+			$continuer = ' '._T('dirs_commencer').'.';
 		} else {
 			$titre = _T('dirs_probleme_droits');
 		}
 
 
-		$res = "<div align='right'>" . menu_langues('var_lang_ecrire') . "</div>\n";
+		$res = "<div align='right'>".menu_langues('var_lang_ecrire')."</div>\n";
 
 		if ($bad_dirs) {
 			$res .=
 				_T(
 					'dirs_repertoires_suivants',
 					['bad_dirs' => implode("\n", array_keys($bad_dirs))]
-				) .
-				'<b>' . _T('login_recharger') . '</b>.';
+				).
+				'<b>'._T('login_recharger').'</b>.';
 		}
 
 		if ($absent_dirs) {
@@ -127,22 +127,21 @@ 
 				_T(
 					'dirs_repertoires_absents',
 					['bad_dirs' => implode("\n", array_keys($absent_dirs))]
-				) .
-				'<b>' . _T('login_recharger') . '</b>.';
+				).
+				'<b>'._T('login_recharger').'</b>.';
 		}
-		$res = '<p>' . $continuer . $res . aider('install0', true) . '</p>';
+		$res = '<p>'.$continuer.$res.aider('install0', true).'</p>';
 
 		$t = _T('login_recharger');
-		$t = ($test_dir ? "<input type='hidden' name='test_dir' value='" . spip_htmlspecialchars($test_dir, ENT_QUOTES) . "' />" :
-				'')
+		$t = ($test_dir ? "<input type='hidden' name='test_dir' value='".spip_htmlspecialchars($test_dir, ENT_QUOTES)."' />" : '')
 			. "<input type='hidden' name='etape' value='chmod' />"
-			. "<div style='text-align: right'><input type='submit' value='" . attribut_html($t) . "' /></div>";
+			. "<div style='text-align: right'><input type='submit' value='".attribut_html($t)."' /></div>";
 
-		echo minipres($titre, $res . generer_form_ecrire('install', $t));
+		echo minipres($titre, $res.generer_form_ecrire('install', $t));
 	} else {
 		$deja = (_FILE_CONNECT && analyse_fichier_connection(_FILE_CONNECT));
 		if (!$deja) {
-			redirige_url_ecrire('install', 'etape=1&chmod=' . $chmod);
+			redirige_url_ecrire('install', 'etape=1&chmod='.$chmod);
 		} else {
 			redirige_url_ecrire();
 		}