spip / SPIP

ecrire/install/etape_fin.php

Indentation +48 added lines, -48 removed lines

@@ -12,7 +12,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/headers');
@@ -21,61 +21,61 @@ 
 // Mise en place des fichiers de configuration si ce n'est fait
 
 function install_etape_fin_dist() {
-	ecrire_acces();
+    ecrire_acces();
 
-	$f = str_replace(_FILE_TMP_SUFFIX, '.php', (string) _FILE_CHMOD_TMP);
-	if (file_exists(_FILE_CHMOD_TMP) && !@rename(_FILE_CHMOD_TMP, $f) && @copy(_FILE_CHMOD_TMP, $f)) {
-		spip_unlink(_FILE_CHMOD_TMP);
-	}
+    $f = str_replace(_FILE_TMP_SUFFIX, '.php', (string) _FILE_CHMOD_TMP);
+    if (file_exists(_FILE_CHMOD_TMP) && !@rename(_FILE_CHMOD_TMP, $f) && @copy(_FILE_CHMOD_TMP, $f)) {
+        spip_unlink(_FILE_CHMOD_TMP);
+    }
 
-	$f = str_replace(_FILE_TMP_SUFFIX, '.php', (string) _FILE_CONNECT_TMP);
-	if (file_exists(_FILE_CONNECT_TMP)) {
-		spip_log("renomme $f");
-		if (!@rename(_FILE_CONNECT_TMP, $f) && @copy(_FILE_CONNECT_TMP, $f)) {
-			@spip_unlink(_FILE_CONNECT_TMP);
-		}
-	}
+    $f = str_replace(_FILE_TMP_SUFFIX, '.php', (string) _FILE_CONNECT_TMP);
+    if (file_exists(_FILE_CONNECT_TMP)) {
+        spip_log("renomme $f");
+        if (!@rename(_FILE_CONNECT_TMP, $f) && @copy(_FILE_CONNECT_TMP, $f)) {
+            @spip_unlink(_FILE_CONNECT_TMP);
+        }
+    }
 
-	// creer le repertoire cache, qui sert partout !
-	// deja fait en etape 4 en principe, on garde au cas ou
-	if (!@file_exists(_DIR_CACHE)) {
-		$rep = preg_replace(',' . _DIR_TMP . ',', '', (string) _DIR_CACHE);
-		$rep = sous_repertoire(_DIR_TMP, $rep, true, true);
-	}
+    // creer le repertoire cache, qui sert partout !
+    // deja fait en etape 4 en principe, on garde au cas ou
+    if (!@file_exists(_DIR_CACHE)) {
+        $rep = preg_replace(',' . _DIR_TMP . ',', '', (string) _DIR_CACHE);
+        $rep = sous_repertoire(_DIR_TMP, $rep, true, true);
+    }
 
-	// Verifier la securite des htaccess
-	// Si elle ne fonctionne pas, prevenir
-	$msg = install_verifier_htaccess();
-	if ($msg) {
-		$cible = _T('public:accueil_site');
-		$cible = generer_form_ecrire('accueil', '', '', $cible);
-		$minipage = new Installation();
-		echo $minipage->page($msg . $cible);
-		// ok, deboucher dans l'espace prive
-	} else {
-		redirige_url_ecrire('accueil');
-	}
+    // Verifier la securite des htaccess
+    // Si elle ne fonctionne pas, prevenir
+    $msg = install_verifier_htaccess();
+    if ($msg) {
+        $cible = _T('public:accueil_site');
+        $cible = generer_form_ecrire('accueil', '', '', $cible);
+        $minipage = new Installation();
+        echo $minipage->page($msg . $cible);
+        // ok, deboucher dans l'espace prive
+    } else {
+        redirige_url_ecrire('accueil');
+    }
 }
 
 function install_verifier_htaccess() {
-	if (
-		verifier_htaccess(_DIR_TMP, true)
-		&& verifier_htaccess(_DIR_CONNECT, true)
-		&& verifier_htaccess(_DIR_VENDOR, true)
-	) {
-		return '';
-	}
+    if (
+        verifier_htaccess(_DIR_TMP, true)
+        && verifier_htaccess(_DIR_CONNECT, true)
+        && verifier_htaccess(_DIR_VENDOR, true)
+    ) {
+        return '';
+    }
 
-	$titre = _T('htaccess_inoperant');
+    $titre = _T('htaccess_inoperant');
 
-	$averti = _T(
-		'htaccess_a_simuler',
-		[
-			'htaccess' => '<tt>' . _ACCESS_FILE_NAME . '</tt>',
-			'constantes' => '<tt>_DIR_TMP &amp; _DIR_CONNECT</tt>',
-			'document_root' => '<tt>' . $_SERVER['DOCUMENT_ROOT'] . '</tt>'
-		]
-	);
+    $averti = _T(
+        'htaccess_a_simuler',
+        [
+            'htaccess' => '<tt>' . _ACCESS_FILE_NAME . '</tt>',
+            'constantes' => '<tt>_DIR_TMP &amp; _DIR_CONNECT</tt>',
+            'document_root' => '<tt>' . $_SERVER['DOCUMENT_ROOT'] . '</tt>'
+        ]
+    );
 
-	return "<div class='error'><h3>$titre</h3><p>$averti</p></div>";
+    return "<div class='error'><h3>$titre</h3><p>$averti</p></div>";
 }

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
 	// creer le repertoire cache, qui sert partout !
 	// deja fait en etape 4 en principe, on garde au cas ou
 	if (!@file_exists(_DIR_CACHE)) {
-		$rep = preg_replace(',' . _DIR_TMP . ',', '', (string) _DIR_CACHE);
+		$rep = preg_replace(','._DIR_TMP.',', '', (string) _DIR_CACHE);
 		$rep = sous_repertoire(_DIR_TMP, $rep, true, true);
 	}
 
@@ -50,7 +50,7 @@ 
 		$cible = _T('public:accueil_site');
 		$cible = generer_form_ecrire('accueil', '', '', $cible);
 		$minipage = new Installation();
-		echo $minipage->page($msg . $cible);
+		echo $minipage->page($msg.$cible);
 		// ok, deboucher dans l'espace prive
 	} else {
 		redirige_url_ecrire('accueil');
@@ -71,9 +71,9 @@ 
 	$averti = _T(
 		'htaccess_a_simuler',
 		[
-			'htaccess' => '<tt>' . _ACCESS_FILE_NAME . '</tt>',
+			'htaccess' => '<tt>'._ACCESS_FILE_NAME.'</tt>',
 			'constantes' => '<tt>_DIR_TMP &amp; _DIR_CONNECT</tt>',
-			'document_root' => '<tt>' . $_SERVER['DOCUMENT_ROOT'] . '</tt>'
+			'document_root' => '<tt>'.$_SERVER['DOCUMENT_ROOT'].'</tt>'
 		]
 	);
 

ecrire/install/etape_.php

Indentation +15 added lines, -15 removed lines

@@ -18,7 +18,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -27,20 +27,20 @@ 
  * @uses info_copyright()
  **/
 function install_etape__dist() {
-	utiliser_langue_visiteur();
-	$menu_langues = menu_langues('var_lang_ecrire');
-	if (!$menu_langues) {
-		redirige_url_ecrire('install', 'etape=chmod');
-	} else {
-		include_spip('inc/presentation'); // pour info_copyright
+    utiliser_langue_visiteur();
+    $menu_langues = menu_langues('var_lang_ecrire');
+    if (!$menu_langues) {
+        redirige_url_ecrire('install', 'etape=chmod');
+    } else {
+        include_spip('inc/presentation'); // pour info_copyright
 
-		$res = "<div class='petit-centre'><img alt='SPIP' class='logo' src='" . chemin_image('logo-spip.png') . "' />\n" .
-			"<p class='small'>" . info_copyright() . "</p></div>\n" .
-			'<p>' . _T('install_select_langue') . '</p>' .
-			'<div>' . $menu_langues . "</div>\n" .
-			generer_form_ecrire('install', "<input type='hidden' name='etape' value='chmod' />" . bouton_suivant());
+        $res = "<div class='petit-centre'><img alt='SPIP' class='logo' src='" . chemin_image('logo-spip.png') . "' />\n" .
+            "<p class='small'>" . info_copyright() . "</p></div>\n" .
+            '<p>' . _T('install_select_langue') . '</p>' .
+            '<div>' . $menu_langues . "</div>\n" .
+            generer_form_ecrire('install', "<input type='hidden' name='etape' value='chmod' />" . bouton_suivant());
 
-		$minipage = new Installation();
-		echo $minipage->page($res);
-	}
+        $minipage = new Installation();
+        echo $minipage->page($res);
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -34,11 +34,11 @@
 	} else {
 		include_spip('inc/presentation'); // pour info_copyright
 
-		$res = "<div class='petit-centre'><img alt='SPIP' class='logo' src='" . chemin_image('logo-spip.png') . "' />\n" .
-			"<p class='small'>" . info_copyright() . "</p></div>\n" .
-			'<p>' . _T('install_select_langue') . '</p>' .
-			'<div>' . $menu_langues . "</div>\n" .
-			generer_form_ecrire('install', "<input type='hidden' name='etape' value='chmod' />" . bouton_suivant());
+		$res = "<div class='petit-centre'><img alt='SPIP' class='logo' src='".chemin_image('logo-spip.png')."' />\n".
+			"<p class='small'>".info_copyright()."</p></div>\n".
+			'<p>'._T('install_select_langue').'</p>'.
+			'<div>'.$menu_langues."</div>\n".
+			generer_form_ecrire('install', "<input type='hidden' name='etape' value='chmod' />".bouton_suivant());
 
 		$minipage = new Installation();
 		echo $minipage->page($res);

ecrire/install/etape_2.php

Indentation +121 added lines, -121 removed lines

@@ -12,167 +12,167 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('base/abstract_sql');
 
 function install_etape_2_dist() {
-	$adresse_db = defined('_INSTALL_HOST_DB')
-		? _INSTALL_HOST_DB
-		: _request('adresse_db');
+    $adresse_db = defined('_INSTALL_HOST_DB')
+        ? _INSTALL_HOST_DB
+        : _request('adresse_db');
 
-	if (preg_match(',(.*):(.*),', (string) $adresse_db, $r)) {
-		[, $adresse_db, $port] = $r;
-	} else {
-		$port = '';
-	}
+    if (preg_match(',(.*):(.*),', (string) $adresse_db, $r)) {
+        [, $adresse_db, $port] = $r;
+    } else {
+        $port = '';
+    }
 
-	$login_db = defined('_INSTALL_USER_DB')
-		? _INSTALL_USER_DB
-		: _request('login_db');
+    $login_db = defined('_INSTALL_USER_DB')
+        ? _INSTALL_USER_DB
+        : _request('login_db');
 
-	$pass_db = defined('_INSTALL_PASS_DB')
-		? _INSTALL_PASS_DB
-		: _request('pass_db');
+    $pass_db = defined('_INSTALL_PASS_DB')
+        ? _INSTALL_PASS_DB
+        : _request('pass_db');
 
-	$server_db = defined('_INSTALL_SERVER_DB')
-		? _INSTALL_SERVER_DB
-		: _request('server_db');
+    $server_db = defined('_INSTALL_SERVER_DB')
+        ? _INSTALL_SERVER_DB
+        : _request('server_db');
 
-	$name_db = defined('_INSTALL_NAME_DB')
-		? _INSTALL_NAME_DB
-		: '';
+    $name_db = defined('_INSTALL_NAME_DB')
+        ? _INSTALL_NAME_DB
+        : '';
 
-	$chmod = _request('chmod');
+    $chmod = _request('chmod');
 
-	$link = spip_connect_db($adresse_db, $port, $login_db, $pass_db, $name_db, $server_db);
-	$GLOBALS['connexions'][$server_db] = $link;
+    $link = spip_connect_db($adresse_db, $port, $login_db, $pass_db, $name_db, $server_db);
+    $GLOBALS['connexions'][$server_db] = $link;
 
-	$GLOBALS['connexions'][$server_db][$GLOBALS['spip_sql_version']]
-		= $GLOBALS['spip_' . $server_db . '_functions_' . $GLOBALS['spip_sql_version']];
+    $GLOBALS['connexions'][$server_db][$GLOBALS['spip_sql_version']]
+        = $GLOBALS['spip_' . $server_db . '_functions_' . $GLOBALS['spip_sql_version']];
 
-	$minipage = new Installation();
-	echo $minipage->installDebutPage();
+    $minipage = new Installation();
+    echo $minipage->installDebutPage();
 
 
 // prenons toutes les dispositions possibles pour que rien ne s'affiche !
 
-	/*
+    /*
 	 * /!\ sqlite3/PDO : erreur sur join(', ', $link)
 	 * L'objet PDO ne peut pas etre transformee en chaine
 	 * Un echo $link ne fonctionne pas non plus
 	 * Il faut utiliser par exemple print_r($link)
 	 */
-	//echo "\n<!--\n", join(', ', $link), " $login_db ";
-	$db_connect = 0; // revoirfunction_exists($ferrno) ? $ferrno() : 0;
-	//echo join(', ', $GLOBALS['connexions'][$server_db]);
-	//echo "\n-->\n";
+    //echo "\n<!--\n", join(', ', $link), " $login_db ";
+    $db_connect = 0; // revoirfunction_exists($ferrno) ? $ferrno() : 0;
+    //echo join(', ', $GLOBALS['connexions'][$server_db]);
+    //echo "\n-->\n";
 
-	if (($db_connect == '0') && $link) {
-		echo "<div class='success'><b>" . _T('info_connexion_ok') . '</b></div>';
-		echo info_progression_etape(2, 'etape_', 'install/');
+    if (($db_connect == '0') && $link) {
+        echo "<div class='success'><b>" . _T('info_connexion_ok') . '</b></div>';
+        echo info_progression_etape(2, 'etape_', 'install/');
 
-		echo info_etape(_T('menu_aide_installation_choix_base') . aider('install2', true));
+        echo info_etape(_T('menu_aide_installation_choix_base') . aider('install2', true));
 
-		echo "\n", '<!-- ', sql_version($server_db), ' -->';
-		[$checked, $res] = install_etape_2_bases($login_db, $server_db);
+        echo "\n", '<!-- ', sql_version($server_db), ' -->';
+        [$checked, $res] = install_etape_2_bases($login_db, $server_db);
 
-		$hidden = (defined('_SPIP_CHMOD')
-				? ''
-				: ("\n<input type='hidden' name='chmod' value='" . spip_htmlspecialchars($chmod) . "' />"))
-			. predef_ou_cache($adresse_db . ($port ? ':' . $port : ''), $login_db, $pass_db, $server_db);
+        $hidden = (defined('_SPIP_CHMOD')
+                ? ''
+                : ("\n<input type='hidden' name='chmod' value='" . spip_htmlspecialchars($chmod) . "' />"))
+            . predef_ou_cache($adresse_db . ($port ? ':' . $port : ''), $login_db, $pass_db, $server_db);
 
-		echo install_etape_2_form($hidden, $checked, $res, 3);
-	} else {
-		echo info_progression_etape(1, 'etape_', 'install/', true);
+        echo install_etape_2_form($hidden, $checked, $res, 3);
+    } else {
+        echo info_progression_etape(1, 'etape_', 'install/', true);
 
-		echo "<div class='error'>";
-		echo info_etape(_T('info_connexion_base'));
-		echo '<h3>' . _T('avis_connexion_echec_1') . '</h3>';
-		echo '<p>' . _T('avis_connexion_echec_2') . '</p>';
+        echo "<div class='error'>";
+        echo info_etape(_T('info_connexion_base'));
+        echo '<h3>' . _T('avis_connexion_echec_1') . '</h3>';
+        echo '<p>' . _T('avis_connexion_echec_2') . '</p>';
 
-		echo "<p style='font-size: small;'>",
-		_T('avis_connexion_echec_3'),
-		'</p></div>';
-	}
+        echo "<p style='font-size: small;'>",
+        _T('avis_connexion_echec_3'),
+        '</p></div>';
+    }
 
-	echo $minipage->installFinPage();
+    echo $minipage->installFinPage();
 }
 
 // Liste les bases accessibles,
 // avec une heuristique pour preselectionner la plus probable
 
 function install_etape_2_bases($login_db, $server_db) {
-	$res = install_etape_liste_bases($server_db, $login_db);
-	if ($res) {
-		[$checked, $bases] = $res;
-
-		return [
-			$checked,
-			"<label for='choix_db'><b>"
-			. _T('texte_choix_base_2')
-			. '</b><br />'
-			. _T('texte_choix_base_3')
-			. '</label>'
-			. "<ul>\n<li>"
-			. implode("</li>\n<li>", $bases)
-			. "</li>\n</ul><p>"
-			. _T('info_ou')
-			. ' '
-		];
-	}
-	$res = '<b>' . _T('avis_lecture_noms_bases_1') . '</b>
+    $res = install_etape_liste_bases($server_db, $login_db);
+    if ($res) {
+        [$checked, $bases] = $res;
+
+        return [
+            $checked,
+            "<label for='choix_db'><b>"
+            . _T('texte_choix_base_2')
+            . '</b><br />'
+            . _T('texte_choix_base_3')
+            . '</label>'
+            . "<ul>\n<li>"
+            . implode("</li>\n<li>", $bases)
+            . "</li>\n</ul><p>"
+            . _T('info_ou')
+            . ' '
+        ];
+    }
+    $res = '<b>' . _T('avis_lecture_noms_bases_1') . '</b>
 		' . _T('avis_lecture_noms_bases_2') . '<p>';
 
-	$checked = false;
-	if ($login_db) {
-		// Si un login comporte un point, le nom de la base est plus
-		// probablement le login sans le point -- testons pour savoir
-		$test_base = $login_db;
-		$ok = sql_selectdb($test_base, $server_db);
-		$test_base2 = str_replace('.', '_', (string) $test_base);
-		if (sql_selectdb($test_base2, $server_db)) {
-			$test_base = $test_base2;
-			$ok = true;
-		}
-
-		if ($ok) {
-			$res .= _T('avis_lecture_noms_bases_3')
-				. '<ul>'
-				. '<li><input name="choix_db" value="' . $test_base . "\" type='radio' id='stand' checked='checked' />"
-				. "<label for='stand'>" . $test_base . "</label></li>\n"
-				. '</ul>'
-				. '<p>' . _T('info_ou') . ' ';
-			$checked = true;
-		}
-	}
-
-	return [$checked, $res];
+    $checked = false;
+    if ($login_db) {
+        // Si un login comporte un point, le nom de la base est plus
+        // probablement le login sans le point -- testons pour savoir
+        $test_base = $login_db;
+        $ok = sql_selectdb($test_base, $server_db);
+        $test_base2 = str_replace('.', '_', (string) $test_base);
+        if (sql_selectdb($test_base2, $server_db)) {
+            $test_base = $test_base2;
+            $ok = true;
+        }
+
+        if ($ok) {
+            $res .= _T('avis_lecture_noms_bases_3')
+                . '<ul>'
+                . '<li><input name="choix_db" value="' . $test_base . "\" type='radio' id='stand' checked='checked' />"
+                . "<label for='stand'>" . $test_base . "</label></li>\n"
+                . '</ul>'
+                . '<p>' . _T('info_ou') . ' ';
+            $checked = true;
+        }
+    }
+
+    return [$checked, $res];
 }
 
 function install_etape_2_form($hidden, $checked, $res, $etape) {
-	return generer_form_ecrire('install', (
-		"\n<input type='hidden' name='etape' value='$etape' />"
-		. $hidden
-		. (defined('_INSTALL_NAME_DB')
-			? '<h3>' . _T('install_nom_base_hebergeur') . ' <tt>' . _INSTALL_NAME_DB . '</tt>' . '</h3>'
-			: "\n<fieldset><legend>" . _T('texte_choix_base_1') . "</legend>\n"
-			. $res
-			. "\n<input name=\"choix_db\" value=\"new_spip\" type='radio' id='nou'"
-			. ($checked ? '' : " checked='checked'")
-			. " />\n<label for='nou'>" . _T('info_creer_base') . "</label></p>\n<p>"
-			. "\n<input type='text' name='table_new' class='text' value=\"spip\" size='20' /></p></fieldset>\n"
-		)
-
-		. ((defined('_INSTALL_TABLE_PREFIX') || $GLOBALS['table_prefix'] != 'spip')
-			? '<h3>' . _T('install_table_prefix_hebergeur') . '  <tt>' . $GLOBALS['table_prefix'] . '</tt>' . '</h3>'
-			: '<fieldset><legend>' . _T('texte_choix_table_prefix') . "</legend>\n"
-			. "<p><label for='table_prefix'>" . _T('info_table_prefix') . '</label></p><p>'
-			. "\n<input type='text' id='tprefix' name='tprefix' class='text' value='"
-			. 'spip' # valeur par defaut
-			. "' size='20' /></p></fieldset>"
-		)
-
-		. bouton_suivant()));
+    return generer_form_ecrire('install', (
+        "\n<input type='hidden' name='etape' value='$etape' />"
+        . $hidden
+        . (defined('_INSTALL_NAME_DB')
+            ? '<h3>' . _T('install_nom_base_hebergeur') . ' <tt>' . _INSTALL_NAME_DB . '</tt>' . '</h3>'
+            : "\n<fieldset><legend>" . _T('texte_choix_base_1') . "</legend>\n"
+            . $res
+            . "\n<input name=\"choix_db\" value=\"new_spip\" type='radio' id='nou'"
+            . ($checked ? '' : " checked='checked'")
+            . " />\n<label for='nou'>" . _T('info_creer_base') . "</label></p>\n<p>"
+            . "\n<input type='text' name='table_new' class='text' value=\"spip\" size='20' /></p></fieldset>\n"
+        )
+
+        . ((defined('_INSTALL_TABLE_PREFIX') || $GLOBALS['table_prefix'] != 'spip')
+            ? '<h3>' . _T('install_table_prefix_hebergeur') . '  <tt>' . $GLOBALS['table_prefix'] . '</tt>' . '</h3>'
+            : '<fieldset><legend>' . _T('texte_choix_table_prefix') . "</legend>\n"
+            . "<p><label for='table_prefix'>" . _T('info_table_prefix') . '</label></p><p>'
+            . "\n<input type='text' id='tprefix' name='tprefix' class='text' value='"
+            . 'spip' # valeur par defaut
+            . "' size='20' /></p></fieldset>"
+        )
+
+        . bouton_suivant()));
 }

Spacing +18 added lines, -18 removed lines

@@ -50,7 +50,7 @@ 
 	$GLOBALS['connexions'][$server_db] = $link;
 
 	$GLOBALS['connexions'][$server_db][$GLOBALS['spip_sql_version']]
-		= $GLOBALS['spip_' . $server_db . '_functions_' . $GLOBALS['spip_sql_version']];
+		= $GLOBALS['spip_'.$server_db.'_functions_'.$GLOBALS['spip_sql_version']];
 
 	$minipage = new Installation();
 	echo $minipage->installDebutPage();
@@ -70,18 +70,18 @@ 
 	//echo "\n-->\n";
 
 	if (($db_connect == '0') && $link) {
-		echo "<div class='success'><b>" . _T('info_connexion_ok') . '</b></div>';
+		echo "<div class='success'><b>"._T('info_connexion_ok').'</b></div>';
 		echo info_progression_etape(2, 'etape_', 'install/');
 
-		echo info_etape(_T('menu_aide_installation_choix_base') . aider('install2', true));
+		echo info_etape(_T('menu_aide_installation_choix_base').aider('install2', true));
 
 		echo "\n", '<!-- ', sql_version($server_db), ' -->';
 		[$checked, $res] = install_etape_2_bases($login_db, $server_db);
 
 		$hidden = (defined('_SPIP_CHMOD')
 				? ''
-				: ("\n<input type='hidden' name='chmod' value='" . spip_htmlspecialchars($chmod) . "' />"))
-			. predef_ou_cache($adresse_db . ($port ? ':' . $port : ''), $login_db, $pass_db, $server_db);
+				: ("\n<input type='hidden' name='chmod' value='".spip_htmlspecialchars($chmod)."' />"))
+			. predef_ou_cache($adresse_db.($port ? ':'.$port : ''), $login_db, $pass_db, $server_db);
 
 		echo install_etape_2_form($hidden, $checked, $res, 3);
 	} else {
@@ -89,8 +89,8 @@ 
 
 		echo "<div class='error'>";
 		echo info_etape(_T('info_connexion_base'));
-		echo '<h3>' . _T('avis_connexion_echec_1') . '</h3>';
-		echo '<p>' . _T('avis_connexion_echec_2') . '</p>';
+		echo '<h3>'._T('avis_connexion_echec_1').'</h3>';
+		echo '<p>'._T('avis_connexion_echec_2').'</p>';
 
 		echo "<p style='font-size: small;'>",
 		_T('avis_connexion_echec_3'),
@@ -122,8 +122,8 @@ 
 			. ' '
 		];
 	}
-	$res = '<b>' . _T('avis_lecture_noms_bases_1') . '</b>
-		' . _T('avis_lecture_noms_bases_2') . '<p>';
+	$res = '<b>'._T('avis_lecture_noms_bases_1').'</b>
+		' . _T('avis_lecture_noms_bases_2').'<p>';
 
 	$checked = false;
 	if ($login_db) {
@@ -140,10 +140,10 @@ 
 		if ($ok) {
 			$res .= _T('avis_lecture_noms_bases_3')
 				. '<ul>'
-				. '<li><input name="choix_db" value="' . $test_base . "\" type='radio' id='stand' checked='checked' />"
-				. "<label for='stand'>" . $test_base . "</label></li>\n"
+				. '<li><input name="choix_db" value="'.$test_base."\" type='radio' id='stand' checked='checked' />"
+				. "<label for='stand'>".$test_base."</label></li>\n"
 				. '</ul>'
-				. '<p>' . _T('info_ou') . ' ';
+				. '<p>'._T('info_ou').' ';
 			$checked = true;
 		}
 	}
@@ -156,19 +156,19 @@ 
 		"\n<input type='hidden' name='etape' value='$etape' />"
 		. $hidden
 		. (defined('_INSTALL_NAME_DB')
-			? '<h3>' . _T('install_nom_base_hebergeur') . ' <tt>' . _INSTALL_NAME_DB . '</tt>' . '</h3>'
-			: "\n<fieldset><legend>" . _T('texte_choix_base_1') . "</legend>\n"
+			? '<h3>'._T('install_nom_base_hebergeur').' <tt>'._INSTALL_NAME_DB.'</tt>'.'</h3>'
+			: "\n<fieldset><legend>"._T('texte_choix_base_1')."</legend>\n"
 			. $res
 			. "\n<input name=\"choix_db\" value=\"new_spip\" type='radio' id='nou'"
 			. ($checked ? '' : " checked='checked'")
-			. " />\n<label for='nou'>" . _T('info_creer_base') . "</label></p>\n<p>"
+			. " />\n<label for='nou'>"._T('info_creer_base')."</label></p>\n<p>"
 			. "\n<input type='text' name='table_new' class='text' value=\"spip\" size='20' /></p></fieldset>\n"
 		)
 
 		. ((defined('_INSTALL_TABLE_PREFIX') || $GLOBALS['table_prefix'] != 'spip')
-			? '<h3>' . _T('install_table_prefix_hebergeur') . '  <tt>' . $GLOBALS['table_prefix'] . '</tt>' . '</h3>'
-			: '<fieldset><legend>' . _T('texte_choix_table_prefix') . "</legend>\n"
-			. "<p><label for='table_prefix'>" . _T('info_table_prefix') . '</label></p><p>'
+			? '<h3>'._T('install_table_prefix_hebergeur').'  <tt>'.$GLOBALS['table_prefix'].'</tt>'.'</h3>'
+			: '<fieldset><legend>'._T('texte_choix_table_prefix')."</legend>\n"
+			. "<p><label for='table_prefix'>"._T('info_table_prefix').'</label></p><p>'
 			. "\n<input type='text' id='tprefix' name='tprefix' class='text' value='"
 			. 'spip' # valeur par defaut
 			. "' size='20' /></p></fieldset>"

ecrire/src/Chiffrer/SpipCles.php

Indentation +165 added lines, -165 removed lines

@@ -13,169 +13,169 @@
 
 /** Gestion des clés d’authentification / chiffrement de SPIP */
 final class SpipCles {
-	private static array $instances = [];
-
-	private string $file = _DIR_ETC . 'cles.php';
-	private readonly Cles $cles;
-
-	public static function instance(string $file = ''): self {
-		if (empty(self::$instances[$file])) {
-			self::$instances[$file] = new self($file);
-		}
-		return self::$instances[$file];
-	}
-
-	/**
-	 * Retourne le secret du site (shorthand)
-	 * @uses self::getSecretSite()
-	 */
-	public static function secret_du_site(): ?string {
-		return (self::instance())->getSecretSite();
-	}
-
-	private function __construct(string $file = '') {
-		if ($file) {
-			$this->file = $file;
-		}
-		$this->cles = new Cles($this->read());
-	}
-
-	/**
-	 * Renvoyer le secret du site
-	 *
-	 * Le secret du site doit rester aussi secret que possible, et est eternel
-	 * On ne doit pas l'exporter
-	 *
-	 * Le secret est partagé entre une clé disque et une clé bdd
-	 *
-	 * @return string
-	 */
-	public function getSecretSite(bool $autoInit = true): ?string {
-		$key = $this->getKey('secret_du_site', $autoInit);
-		$meta = $this->getMetaKey('secret_du_site', $autoInit);
-		// conserve la même longeur.
-		return $key ^ $meta;
-	}
-
-	/** Renvoyer le secret des authentifications */
-	public function getSecretAuth(bool $autoInit = false): ?string {
-		return $this->getKey('secret_des_auth', $autoInit);
-	}
-	public function save(): bool {
-		return ecrire_fichier_securise($this->file, $this->cles->toJson());
-	}
-
-	/**
-	 * Fournir une sauvegarde chiffree des cles (a l'aide d'une autre clé, comme le pass d'un auteur)
-	 *
-	 * @param string $withKey Clé de chiffrage de la sauvegarde
-	 * @return string Contenu de la sauvegarde chiffrée générée
-	 */
-	public function backup(
-		#[\SensitiveParameter]
-		string $withKey
-	): string {
-		if (count($this->cles)) {
-			return Chiffrement::chiffrer($this->cles->toJson(), $withKey);
-		}
-		return '';
-	}
-
-	/**
-	 * Restaurer les cles manquantes depuis une sauvegarde chiffree des cles
-	 * (si la sauvegarde est bien valide)
-	 */
-	public function restore(
-		/** Sauvegarde chiffrée (générée par backup()) */
-		string $backup,
-		#[\SensitiveParameter]
-		string $password_clair,
-		#[\SensitiveParameter]
-		string $password_hash,
-		int $id_auteur
-	): bool {
-		if (empty($backup)) {
-			return false;
-		}
-
-		$sauvegarde = Chiffrement::dechiffrer($backup, $password_clair);
-		$json = json_decode($sauvegarde, true, 512, JSON_THROW_ON_ERROR);
-		if (!$json) {
-			return false;
-		}
-
-		// cela semble une sauvegarde valide
-		$cles_potentielles = array_map('base64_decode', $json);
-
-		// il faut faire une double verif sur secret_des_auth
-		// pour s'assurer qu'elle permet bien de decrypter le pass de l'auteur qui fournit la sauvegarde
-		// et par extension tous les passwords
-		if (
-			!empty($cles_potentielles['secret_des_auth'])
-			&& !Password::verifier($password_clair, $password_hash, $cles_potentielles['secret_des_auth'])
-		) {
-			spip_log("Restauration de la cle `secret_des_auth` par id_auteur $id_auteur erronnee, on ignore", 'chiffrer' . _LOG_INFO_IMPORTANTE);
-			unset($cles_potentielles['secret_des_auth']);
-		}
-
-		// on merge les cles pour recuperer les cles manquantes
-		$restauration = false;
-		foreach ($cles_potentielles as $name => $key) {
-			if (!$this->cles->has($name)) {
-				$this->cles->set($name, $key);
-				spip_log("Restauration de la cle $name par id_auteur $id_auteur", 'chiffrer' . _LOG_INFO_IMPORTANTE);
-				$restauration = true;
-			}
-		}
-		return $restauration;
-	}
-
-	private function getKey(string $name, bool $autoInit): ?string {
-		if ($this->cles->has($name)) {
-			return $this->cles->get($name);
-		}
-		if ($autoInit) {
-			$this->cles->generate($name);
-			// si l'ecriture de fichier a bien marche on peut utiliser la cle
-			if ($this->save()) {
-				return $this->cles->get($name);
-			}
-			// sinon loger et annule la cle generee car il ne faut pas l'utiliser
-			spip_log('Echec ecriture du fichier cle ' . $this->file . " ; impossible de generer une cle $name", 'chiffrer' . _LOG_ERREUR);
-			$this->cles->delete($name);
-		}
-		return null;
-	}
-
-	private function getMetaKey(string $name, bool $autoInit = true): ?string {
-		if (!isset($GLOBALS['meta'][$name])) {
-			include_spip('base/abstract_sql');
-			$GLOBALS['meta'][$name] = sql_getfetsel('valeur', 'spip_meta', 'nom = ' . sql_quote($name, '', 'string'));
-		}
-		$key = base64_decode($GLOBALS['meta'][$name] ?? '');
-		if (strlen($key) === \SODIUM_CRYPTO_SECRETBOX_KEYBYTES) {
-			return $key;
-		}
-		if (!$autoInit) {
-			return null;
-		}
-		$key = Chiffrement::keygen();
-		ecrire_meta($name, base64_encode($key), 'non');
-		lire_metas(); // au cas ou ecrire_meta() ne fonctionne pas
-
-		return $key;
-	}
-
-	private function read(): array {
-		$json = null;
-		lire_fichier_securise($this->file, $json);
-		if (
-			$json
-			&& ($json = \json_decode($json, true, 512, JSON_THROW_ON_ERROR))
-			&& is_array($json)
-		) {
-			return array_map('base64_decode', $json);
-		}
-		return [];
-	}
+    private static array $instances = [];
+
+    private string $file = _DIR_ETC . 'cles.php';
+    private readonly Cles $cles;
+
+    public static function instance(string $file = ''): self {
+        if (empty(self::$instances[$file])) {
+            self::$instances[$file] = new self($file);
+        }
+        return self::$instances[$file];
+    }
+
+    /**
+     * Retourne le secret du site (shorthand)
+     * @uses self::getSecretSite()
+     */
+    public static function secret_du_site(): ?string {
+        return (self::instance())->getSecretSite();
+    }
+
+    private function __construct(string $file = '') {
+        if ($file) {
+            $this->file = $file;
+        }
+        $this->cles = new Cles($this->read());
+    }
+
+    /**
+     * Renvoyer le secret du site
+     *
+     * Le secret du site doit rester aussi secret que possible, et est eternel
+     * On ne doit pas l'exporter
+     *
+     * Le secret est partagé entre une clé disque et une clé bdd
+     *
+     * @return string
+     */
+    public function getSecretSite(bool $autoInit = true): ?string {
+        $key = $this->getKey('secret_du_site', $autoInit);
+        $meta = $this->getMetaKey('secret_du_site', $autoInit);
+        // conserve la même longeur.
+        return $key ^ $meta;
+    }
+
+    /** Renvoyer le secret des authentifications */
+    public function getSecretAuth(bool $autoInit = false): ?string {
+        return $this->getKey('secret_des_auth', $autoInit);
+    }
+    public function save(): bool {
+        return ecrire_fichier_securise($this->file, $this->cles->toJson());
+    }
+
+    /**
+     * Fournir une sauvegarde chiffree des cles (a l'aide d'une autre clé, comme le pass d'un auteur)
+     *
+     * @param string $withKey Clé de chiffrage de la sauvegarde
+     * @return string Contenu de la sauvegarde chiffrée générée
+     */
+    public function backup(
+        #[\SensitiveParameter]
+        string $withKey
+    ): string {
+        if (count($this->cles)) {
+            return Chiffrement::chiffrer($this->cles->toJson(), $withKey);
+        }
+        return '';
+    }
+
+    /**
+     * Restaurer les cles manquantes depuis une sauvegarde chiffree des cles
+     * (si la sauvegarde est bien valide)
+     */
+    public function restore(
+        /** Sauvegarde chiffrée (générée par backup()) */
+        string $backup,
+        #[\SensitiveParameter]
+        string $password_clair,
+        #[\SensitiveParameter]
+        string $password_hash,
+        int $id_auteur
+    ): bool {
+        if (empty($backup)) {
+            return false;
+        }
+
+        $sauvegarde = Chiffrement::dechiffrer($backup, $password_clair);
+        $json = json_decode($sauvegarde, true, 512, JSON_THROW_ON_ERROR);
+        if (!$json) {
+            return false;
+        }
+
+        // cela semble une sauvegarde valide
+        $cles_potentielles = array_map('base64_decode', $json);
+
+        // il faut faire une double verif sur secret_des_auth
+        // pour s'assurer qu'elle permet bien de decrypter le pass de l'auteur qui fournit la sauvegarde
+        // et par extension tous les passwords
+        if (
+            !empty($cles_potentielles['secret_des_auth'])
+            && !Password::verifier($password_clair, $password_hash, $cles_potentielles['secret_des_auth'])
+        ) {
+            spip_log("Restauration de la cle `secret_des_auth` par id_auteur $id_auteur erronnee, on ignore", 'chiffrer' . _LOG_INFO_IMPORTANTE);
+            unset($cles_potentielles['secret_des_auth']);
+        }
+
+        // on merge les cles pour recuperer les cles manquantes
+        $restauration = false;
+        foreach ($cles_potentielles as $name => $key) {
+            if (!$this->cles->has($name)) {
+                $this->cles->set($name, $key);
+                spip_log("Restauration de la cle $name par id_auteur $id_auteur", 'chiffrer' . _LOG_INFO_IMPORTANTE);
+                $restauration = true;
+            }
+        }
+        return $restauration;
+    }
+
+    private function getKey(string $name, bool $autoInit): ?string {
+        if ($this->cles->has($name)) {
+            return $this->cles->get($name);
+        }
+        if ($autoInit) {
+            $this->cles->generate($name);
+            // si l'ecriture de fichier a bien marche on peut utiliser la cle
+            if ($this->save()) {
+                return $this->cles->get($name);
+            }
+            // sinon loger et annule la cle generee car il ne faut pas l'utiliser
+            spip_log('Echec ecriture du fichier cle ' . $this->file . " ; impossible de generer une cle $name", 'chiffrer' . _LOG_ERREUR);
+            $this->cles->delete($name);
+        }
+        return null;
+    }
+
+    private function getMetaKey(string $name, bool $autoInit = true): ?string {
+        if (!isset($GLOBALS['meta'][$name])) {
+            include_spip('base/abstract_sql');
+            $GLOBALS['meta'][$name] = sql_getfetsel('valeur', 'spip_meta', 'nom = ' . sql_quote($name, '', 'string'));
+        }
+        $key = base64_decode($GLOBALS['meta'][$name] ?? '');
+        if (strlen($key) === \SODIUM_CRYPTO_SECRETBOX_KEYBYTES) {
+            return $key;
+        }
+        if (!$autoInit) {
+            return null;
+        }
+        $key = Chiffrement::keygen();
+        ecrire_meta($name, base64_encode($key), 'non');
+        lire_metas(); // au cas ou ecrire_meta() ne fonctionne pas
+
+        return $key;
+    }
+
+    private function read(): array {
+        $json = null;
+        lire_fichier_securise($this->file, $json);
+        if (
+            $json
+            && ($json = \json_decode($json, true, 512, JSON_THROW_ON_ERROR))
+            && is_array($json)
+        ) {
+            return array_map('base64_decode', $json);
+        }
+        return [];
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
 final class SpipCles {
 	private static array $instances = [];
 
-	private string $file = _DIR_ETC . 'cles.php';
+	private string $file = _DIR_ETC.'cles.php';
 	private readonly Cles $cles;
 
 	public static function instance(string $file = ''): self {
@@ -114,7 +114,7 @@ 
 			!empty($cles_potentielles['secret_des_auth'])
 			&& !Password::verifier($password_clair, $password_hash, $cles_potentielles['secret_des_auth'])
 		) {
-			spip_log("Restauration de la cle `secret_des_auth` par id_auteur $id_auteur erronnee, on ignore", 'chiffrer' . _LOG_INFO_IMPORTANTE);
+			spip_log("Restauration de la cle `secret_des_auth` par id_auteur $id_auteur erronnee, on ignore", 'chiffrer'._LOG_INFO_IMPORTANTE);
 			unset($cles_potentielles['secret_des_auth']);
 		}
 
@@ -123,7 +123,7 @@ 
 		foreach ($cles_potentielles as $name => $key) {
 			if (!$this->cles->has($name)) {
 				$this->cles->set($name, $key);
-				spip_log("Restauration de la cle $name par id_auteur $id_auteur", 'chiffrer' . _LOG_INFO_IMPORTANTE);
+				spip_log("Restauration de la cle $name par id_auteur $id_auteur", 'chiffrer'._LOG_INFO_IMPORTANTE);
 				$restauration = true;
 			}
 		}
@@ -141,7 +141,7 @@ 
 				return $this->cles->get($name);
 			}
 			// sinon loger et annule la cle generee car il ne faut pas l'utiliser
-			spip_log('Echec ecriture du fichier cle ' . $this->file . " ; impossible de generer une cle $name", 'chiffrer' . _LOG_ERREUR);
+			spip_log('Echec ecriture du fichier cle '.$this->file." ; impossible de generer une cle $name", 'chiffrer'._LOG_ERREUR);
 			$this->cles->delete($name);
 		}
 		return null;
@@ -150,7 +150,7 @@ 
 	private function getMetaKey(string $name, bool $autoInit = true): ?string {
 		if (!isset($GLOBALS['meta'][$name])) {
 			include_spip('base/abstract_sql');
-			$GLOBALS['meta'][$name] = sql_getfetsel('valeur', 'spip_meta', 'nom = ' . sql_quote($name, '', 'string'));
+			$GLOBALS['meta'][$name] = sql_getfetsel('valeur', 'spip_meta', 'nom = '.sql_quote($name, '', 'string'));
 		}
 		$key = base64_decode($GLOBALS['meta'][$name] ?? '');
 		if (strlen($key) === \SODIUM_CRYPTO_SECRETBOX_KEYBYTES) {

ecrire/src/ErrorHandler.php

Indentation +24 added lines, -24 removed lines

@@ -7,33 +7,33 @@
  * @internal
  */
 final class ErrorHandler {
-	static bool $done = false;
+    static bool $done = false;
 
-	public static function setup(?int $error_level = null): void {
-		if (!self::$done) {
-			self::$done = true;
-			error_reporting($error_level);
-			set_error_handler(self::user_deprecated(...), E_USER_DEPRECATED);
-		}
-	}
+    public static function setup(?int $error_level = null): void {
+        if (!self::$done) {
+            self::$done = true;
+            error_reporting($error_level);
+            set_error_handler(self::user_deprecated(...), E_USER_DEPRECATED);
+        }
+    }
 
-	/** Loger les `trigger_deprecated()` */
-	private static function user_deprecated(int $errno, string $errstr, string $errfile, int $errline): bool {
-		if (!(\E_USER_DEPRECATED & $errno)) {
-			return false;
-		}
+    /** Loger les `trigger_deprecated()` */
+    private static function user_deprecated(int $errno, string $errstr, string $errfile, int $errline): bool {
+        if (!(\E_USER_DEPRECATED & $errno)) {
+            return false;
+        }
 
-		$backtrace = debug_backtrace();
-		array_shift($backtrace);
-		do {
-			$t = array_shift($backtrace);
-			$fqdn = ($t['class'] ?? '') . ($t['type'] ?? '') . ($t['function'] ?? '');
-		} while (in_array($fqdn, ['trigger_error', 'trigger_deprecation']));
+        $backtrace = debug_backtrace();
+        array_shift($backtrace);
+        do {
+            $t = array_shift($backtrace);
+            $fqdn = ($t['class'] ?? '') . ($t['type'] ?? '') . ($t['function'] ?? '');
+        } while (in_array($fqdn, ['trigger_error', 'trigger_deprecation']));
 
-		$errfile = $t['file'];
-		$errline = $t['line'];
+        $errfile = $t['file'];
+        $errline = $t['line'];
 
-		spip_log(sprintf('%s in %s on line %s', $errstr, $errfile, $errline), 'deprecated.' . _LOG_INFO);
-		return false;
-	}
+        spip_log(sprintf('%s in %s on line %s', $errstr, $errfile, $errline), 'deprecated.' . _LOG_INFO);
+        return false;
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -27,13 +27,13 @@
 		array_shift($backtrace);
 		do {
 			$t = array_shift($backtrace);
-			$fqdn = ($t['class'] ?? '') . ($t['type'] ?? '') . ($t['function'] ?? '');
+			$fqdn = ($t['class'] ?? '').($t['type'] ?? '').($t['function'] ?? '');
 		} while (in_array($fqdn, ['trigger_error', 'trigger_deprecation']));
 
 		$errfile = $t['file'];
 		$errline = $t['line'];
 
-		spip_log(sprintf('%s in %s on line %s', $errstr, $errfile, $errline), 'deprecated.' . _LOG_INFO);
+		spip_log(sprintf('%s in %s on line %s', $errstr, $errfile, $errline), 'deprecated.'._LOG_INFO);
 		return false;
 	}
 }

ecrire/src/Texte/Collecteur/HtmlTag.php

Indentation +206 added lines, -206 removed lines

@@ -17,210 +17,210 @@
  * @see extraire_balises()
  */
 class HtmlTag extends AbstractCollecteur {
-	protected static string $markPrefix = 'HTMLTAG';
-
-	/**
-	 * La preg pour découper et collecter les modèles
-	 * @var string
-	 */
-	protected string $preg_openingtag;
-	protected string $preg_closingtag;
-	protected string $tag;
-
-	public static array $listeBalisesAProteger = ['html', 'pre', 'code', 'cadre', 'frame', 'script', 'style'];
-
-	public function __construct(string $tag, ?string $preg_openingtag = null, ?string $preg_closingtag = null) {
-
-		$tag = strtolower($tag);
-		$this->tag = $tag;
-		$this->preg_openingtag = ($preg_openingtag ?: "@<{$tag}\b([^>]*?)(/?)>@isS");
-		$this->preg_closingtag = ($preg_closingtag ?? "@</{$tag}\b[^>]*>@isS");
-	}
-
-	/**
-	 * @param string $texte
-	 * @param array $options
-	 *   bool $detecter_presence
-	 *   bool $nb_max
-	 *   int  $profondeur
-	 * @return array
-	 */
-	public function collecter(string $texte, array $options = []): array {
-		if (!$texte) {
-			return [];
-		}
-
-		$upperTag = strtoupper($this->tag);
-		$hasUpperCaseTags = ($upperTag !== $this->tag && (str_contains($texte, '<' . $upperTag) || str_contains($texte, '</' . $upperTag)));
-
-		// collecter les balises ouvrantes
-		$opening = static::collecteur($texte, '', $hasUpperCaseTags ? '<' : '<' . $this->tag, $this->preg_openingtag, empty($options['detecter_presence']) ? 0 : 1);
-		if (!$opening) {
-			return [];
-		}
-
-		// si c'est un tag autofermant ou vide qui se repère avec une seule regexp, on va plus vite
-		if (!$this->preg_closingtag) {
-			return $opening;
-		}
-
-		// collecter les balises fermantes
-		$closing = static::collecteur($texte, '', $hasUpperCaseTags ? '</' : '</' . $this->tag, $this->preg_closingtag);
-
-		$profondeur = ($options['profondeur'] ?? 1);
-		$tags = [];
-		while (!empty($opening)) {
-			$first_opening = array_shift($opening);
-			// self closing ?
-			if (str_contains($first_opening['raw'], '/>')) {
-				$tag = $first_opening;
-				$tag['opening'] = $tag['raw'];
-				$tag['closing'] = '';
-				$tag['innerHtml'] = '';
-				$tag['attributs'] = trim(substr($tag['opening'], strlen($this->tag) + 1, -2));
-				$tags[] = $tag;
-			}
-			else {
-				// enlever les closing qui sont avant le premier opening, car ils n'ont pas de sens
-				while (
-					!empty($closing)
-					&& ($first_closing = reset($closing))
-					&& $first_closing['pos'] < $first_opening['pos']
-				) {
-					array_shift($closing);
-				}
-
-				$need_closing = 0;
-				$next_closing = reset($closing);
-				$next_opening = reset($opening);
-				// certaines balises comme <code> neutralisent le contenant, donc tout ce qui est avant le prochain closing doit etre ignoré
-				if (in_array($this->tag, ['code'])) {
-					while ($next_opening && $next_closing && $next_opening['pos'] < $next_closing['pos']) {
-						array_shift($opening);
-						$next_opening = reset($opening);
-					}
-				}
-				else {
-					while ($next_opening && $next_closing && $next_opening['pos'] < $next_closing['pos']) {
-						while ($next_opening && $next_opening['pos'] < $next_closing['pos']) {
-							// si pas self closing, il faut un closing de plus
-							if (!str_contains($next_opening['raw'], '/>')) {
-								$need_closing++;
-							}
-							array_shift($opening);
-							$next_opening = reset($opening);
-						}
-						// il faut depiler les balises fermantes autant de fois que nécessaire et tant qu'on a pas une nouvelle balise ouvrante
-						while ($need_closing && $next_closing && (!$next_opening || $next_closing['pos'] < $next_opening['pos'])) {
-							array_shift($closing);
-							$need_closing--;
-							$next_closing = reset($closing);
-						}
-					}
-				}
-				// si pas de fermeture, c'est une autofermante mal fermée...
-				if (!$next_closing || $need_closing) {
-					$tag = $first_opening;
-					$tag['opening'] = $tag['raw'];
-					$tag['closing'] = '';
-					$tag['innerHtml'] = '';
-					$tag['attributs'] = trim(substr($tag['opening'], strlen($this->tag) + 1, -1));
-					$tags[] = $tag;
-				}
-				else {
-					$tag = $first_opening;
-					$next_closing = array_shift($closing);
-					$innerHtml = substr($texte, $tag['pos'] + $tag['length'], $next_closing['pos'] - $tag['pos'] - $tag['length']);
-					$tag['length'] = $next_closing['pos'] - $tag['pos'] + $next_closing['length'];
-					$tag['opening'] = $tag['raw'];
-					$tag['raw'] = substr($texte, $tag['pos'], $tag['length']);
-					$tag['innerHtml'] = $innerHtml;
-					$tag['closing'] = $next_closing['raw'];
-					$tag['attributs'] = trim(substr($tag['opening'], strlen($this->tag) + 1, -1));
-					$tags[] = $tag;
-				}
-			}
-			if ((!empty($options['detecter_presence']) && count($tags))) {
-				return $tags;
-			}
-			if (($profondeur == 1 && !empty($options['nb_max']) && count($tags) >= $options['nb_max'])) {
-				break;
-			}
-		}
-
-		while (--$profondeur > 0) {
-			$outerTags = $tags;
-			$tags = [];
-			$options['profondeur'] = 1;
-			foreach ($outerTags as $outerTag) {
-				if (!empty($outerTag['innerHtml'])) {
-					$offsetPos = $outerTag['pos'] + strlen($outerTag['opening']);
-					$innerTags = $this->collecter($outerTag['innerHtml'], $options);
-					if (!empty($innerTags)) {
-						foreach ($innerTags as $tag) {
-							$tag['pos'] += $offsetPos;
-							$tags[] = $tag;
-						}
-						if (($profondeur == 1 && !empty($options['nb_max']) && count($tags) >= $options['nb_max'])) {
-							return $tags;
-						}
-					}
-				}
-			}
-		}
-
-
-		return $tags;
-	}
-
-	/**
-	 * @param callable|null $callback_function
-	 */
-	public function echapper_enHtmlBase64(string $texte, string $source = '', $callback_function = null, array $callback_options = []): string {
-		if ($callback_function) {
-			$legacy_callback = $callback_function;
-			// si on est dans un cas evident de preg perso, ne pas essayer de mapper le match car on ne sait pas ce qu'il contient
-			// et on aura pas non plus de innerHtml si pas de preg_closingtag
-			if ($this->preg_closingtag) {
-				$tag = $this->tag;
-				$legacy_callback = function ($c, $options) use ($tag, $callback_function) {
-					// legacy : renseigner les infos correspondantes aux matchs de l'ancienne regexp
-					$regs = [
-						0 => $c['raw'],
-						1 => $tag,
-						2 => $c['match'][1] . $c['match'][2],
-						3 => $c['innerHtml'],
-						'tag' => $this->tag,
-					] + $c;
-					return $callback_function($regs, $options);
-				};
-			}
-		}
-		return parent::echapper_enHtmlBase64($texte, $source, $callback_function ? $legacy_callback : null, $callback_options);
-	}
-
-
-	/**
-	 * pour $source voir commentaire infra (echappe_retour)
-	 * pour $no_transform voir le filtre post_autobr dans inc/filtres
-	 */
-	public static function proteger_balisesHtml(string $texte, string $source = '', ?array $html_tags = null, array $callbacks_function = [], array $callback_options = []): string {
-		if ($texte === '') {
-			return '';
-		}
-
-		$html_tags = $html_tags ?: self::$listeBalisesAProteger;
-
-		$tags_todo = $html_tags;
-		while (
-			!empty($tags_todo)
-			&& ($tag = array_shift($tags_todo))
-			&& str_contains($texte, '<')
-		) {
-			$htmlTagCollecteur = new self($tag);
-			$texte = $htmlTagCollecteur->echapper_enHtmlBase64($texte, $source, $callbacks_function[$tag] ?? null, $callback_options);
-		}
-
-		return $texte;
-	}
+    protected static string $markPrefix = 'HTMLTAG';
+
+    /**
+     * La preg pour découper et collecter les modèles
+     * @var string
+     */
+    protected string $preg_openingtag;
+    protected string $preg_closingtag;
+    protected string $tag;
+
+    public static array $listeBalisesAProteger = ['html', 'pre', 'code', 'cadre', 'frame', 'script', 'style'];
+
+    public function __construct(string $tag, ?string $preg_openingtag = null, ?string $preg_closingtag = null) {
+
+        $tag = strtolower($tag);
+        $this->tag = $tag;
+        $this->preg_openingtag = ($preg_openingtag ?: "@<{$tag}\b([^>]*?)(/?)>@isS");
+        $this->preg_closingtag = ($preg_closingtag ?? "@</{$tag}\b[^>]*>@isS");
+    }
+
+    /**
+     * @param string $texte
+     * @param array $options
+     *   bool $detecter_presence
+     *   bool $nb_max
+     *   int  $profondeur
+     * @return array
+     */
+    public function collecter(string $texte, array $options = []): array {
+        if (!$texte) {
+            return [];
+        }
+
+        $upperTag = strtoupper($this->tag);
+        $hasUpperCaseTags = ($upperTag !== $this->tag && (str_contains($texte, '<' . $upperTag) || str_contains($texte, '</' . $upperTag)));
+
+        // collecter les balises ouvrantes
+        $opening = static::collecteur($texte, '', $hasUpperCaseTags ? '<' : '<' . $this->tag, $this->preg_openingtag, empty($options['detecter_presence']) ? 0 : 1);
+        if (!$opening) {
+            return [];
+        }
+
+        // si c'est un tag autofermant ou vide qui se repère avec une seule regexp, on va plus vite
+        if (!$this->preg_closingtag) {
+            return $opening;
+        }
+
+        // collecter les balises fermantes
+        $closing = static::collecteur($texte, '', $hasUpperCaseTags ? '</' : '</' . $this->tag, $this->preg_closingtag);
+
+        $profondeur = ($options['profondeur'] ?? 1);
+        $tags = [];
+        while (!empty($opening)) {
+            $first_opening = array_shift($opening);
+            // self closing ?
+            if (str_contains($first_opening['raw'], '/>')) {
+                $tag = $first_opening;
+                $tag['opening'] = $tag['raw'];
+                $tag['closing'] = '';
+                $tag['innerHtml'] = '';
+                $tag['attributs'] = trim(substr($tag['opening'], strlen($this->tag) + 1, -2));
+                $tags[] = $tag;
+            }
+            else {
+                // enlever les closing qui sont avant le premier opening, car ils n'ont pas de sens
+                while (
+                    !empty($closing)
+                    && ($first_closing = reset($closing))
+                    && $first_closing['pos'] < $first_opening['pos']
+                ) {
+                    array_shift($closing);
+                }
+
+                $need_closing = 0;
+                $next_closing = reset($closing);
+                $next_opening = reset($opening);
+                // certaines balises comme <code> neutralisent le contenant, donc tout ce qui est avant le prochain closing doit etre ignoré
+                if (in_array($this->tag, ['code'])) {
+                    while ($next_opening && $next_closing && $next_opening['pos'] < $next_closing['pos']) {
+                        array_shift($opening);
+                        $next_opening = reset($opening);
+                    }
+                }
+                else {
+                    while ($next_opening && $next_closing && $next_opening['pos'] < $next_closing['pos']) {
+                        while ($next_opening && $next_opening['pos'] < $next_closing['pos']) {
+                            // si pas self closing, il faut un closing de plus
+                            if (!str_contains($next_opening['raw'], '/>')) {
+                                $need_closing++;
+                            }
+                            array_shift($opening);
+                            $next_opening = reset($opening);
+                        }
+                        // il faut depiler les balises fermantes autant de fois que nécessaire et tant qu'on a pas une nouvelle balise ouvrante
+                        while ($need_closing && $next_closing && (!$next_opening || $next_closing['pos'] < $next_opening['pos'])) {
+                            array_shift($closing);
+                            $need_closing--;
+                            $next_closing = reset($closing);
+                        }
+                    }
+                }
+                // si pas de fermeture, c'est une autofermante mal fermée...
+                if (!$next_closing || $need_closing) {
+                    $tag = $first_opening;
+                    $tag['opening'] = $tag['raw'];
+                    $tag['closing'] = '';
+                    $tag['innerHtml'] = '';
+                    $tag['attributs'] = trim(substr($tag['opening'], strlen($this->tag) + 1, -1));
+                    $tags[] = $tag;
+                }
+                else {
+                    $tag = $first_opening;
+                    $next_closing = array_shift($closing);
+                    $innerHtml = substr($texte, $tag['pos'] + $tag['length'], $next_closing['pos'] - $tag['pos'] - $tag['length']);
+                    $tag['length'] = $next_closing['pos'] - $tag['pos'] + $next_closing['length'];
+                    $tag['opening'] = $tag['raw'];
+                    $tag['raw'] = substr($texte, $tag['pos'], $tag['length']);
+                    $tag['innerHtml'] = $innerHtml;
+                    $tag['closing'] = $next_closing['raw'];
+                    $tag['attributs'] = trim(substr($tag['opening'], strlen($this->tag) + 1, -1));
+                    $tags[] = $tag;
+                }
+            }
+            if ((!empty($options['detecter_presence']) && count($tags))) {
+                return $tags;
+            }
+            if (($profondeur == 1 && !empty($options['nb_max']) && count($tags) >= $options['nb_max'])) {
+                break;
+            }
+        }
+
+        while (--$profondeur > 0) {
+            $outerTags = $tags;
+            $tags = [];
+            $options['profondeur'] = 1;
+            foreach ($outerTags as $outerTag) {
+                if (!empty($outerTag['innerHtml'])) {
+                    $offsetPos = $outerTag['pos'] + strlen($outerTag['opening']);
+                    $innerTags = $this->collecter($outerTag['innerHtml'], $options);
+                    if (!empty($innerTags)) {
+                        foreach ($innerTags as $tag) {
+                            $tag['pos'] += $offsetPos;
+                            $tags[] = $tag;
+                        }
+                        if (($profondeur == 1 && !empty($options['nb_max']) && count($tags) >= $options['nb_max'])) {
+                            return $tags;
+                        }
+                    }
+                }
+            }
+        }
+
+
+        return $tags;
+    }
+
+    /**
+     * @param callable|null $callback_function
+     */
+    public function echapper_enHtmlBase64(string $texte, string $source = '', $callback_function = null, array $callback_options = []): string {
+        if ($callback_function) {
+            $legacy_callback = $callback_function;
+            // si on est dans un cas evident de preg perso, ne pas essayer de mapper le match car on ne sait pas ce qu'il contient
+            // et on aura pas non plus de innerHtml si pas de preg_closingtag
+            if ($this->preg_closingtag) {
+                $tag = $this->tag;
+                $legacy_callback = function ($c, $options) use ($tag, $callback_function) {
+                    // legacy : renseigner les infos correspondantes aux matchs de l'ancienne regexp
+                    $regs = [
+                        0 => $c['raw'],
+                        1 => $tag,
+                        2 => $c['match'][1] . $c['match'][2],
+                        3 => $c['innerHtml'],
+                        'tag' => $this->tag,
+                    ] + $c;
+                    return $callback_function($regs, $options);
+                };
+            }
+        }
+        return parent::echapper_enHtmlBase64($texte, $source, $callback_function ? $legacy_callback : null, $callback_options);
+    }
+
+
+    /**
+     * pour $source voir commentaire infra (echappe_retour)
+     * pour $no_transform voir le filtre post_autobr dans inc/filtres
+     */
+    public static function proteger_balisesHtml(string $texte, string $source = '', ?array $html_tags = null, array $callbacks_function = [], array $callback_options = []): string {
+        if ($texte === '') {
+            return '';
+        }
+
+        $html_tags = $html_tags ?: self::$listeBalisesAProteger;
+
+        $tags_todo = $html_tags;
+        while (
+            !empty($tags_todo)
+            && ($tag = array_shift($tags_todo))
+            && str_contains($texte, '<')
+        ) {
+            $htmlTagCollecteur = new self($tag);
+            $texte = $htmlTagCollecteur->echapper_enHtmlBase64($texte, $source, $callbacks_function[$tag] ?? null, $callback_options);
+        }
+
+        return $texte;
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -51,10 +51,10 @@ 
 		}
 
 		$upperTag = strtoupper($this->tag);
-		$hasUpperCaseTags = ($upperTag !== $this->tag && (str_contains($texte, '<' . $upperTag) || str_contains($texte, '</' . $upperTag)));
+		$hasUpperCaseTags = ($upperTag !== $this->tag && (str_contains($texte, '<'.$upperTag) || str_contains($texte, '</'.$upperTag)));
 
 		// collecter les balises ouvrantes
-		$opening = static::collecteur($texte, '', $hasUpperCaseTags ? '<' : '<' . $this->tag, $this->preg_openingtag, empty($options['detecter_presence']) ? 0 : 1);
+		$opening = static::collecteur($texte, '', $hasUpperCaseTags ? '<' : '<'.$this->tag, $this->preg_openingtag, empty($options['detecter_presence']) ? 0 : 1);
 		if (!$opening) {
 			return [];
 		}
@@ -65,7 +65,7 @@ 
 		}
 
 		// collecter les balises fermantes
-		$closing = static::collecteur($texte, '', $hasUpperCaseTags ? '</' : '</' . $this->tag, $this->preg_closingtag);
+		$closing = static::collecteur($texte, '', $hasUpperCaseTags ? '</' : '</'.$this->tag, $this->preg_closingtag);
 
 		$profondeur = ($options['profondeur'] ?? 1);
 		$tags = [];
@@ -183,12 +183,12 @@ 
 			// et on aura pas non plus de innerHtml si pas de preg_closingtag
 			if ($this->preg_closingtag) {
 				$tag = $this->tag;
-				$legacy_callback = function ($c, $options) use ($tag, $callback_function) {
+				$legacy_callback = function($c, $options) use ($tag, $callback_function) {
 					// legacy : renseigner les infos correspondantes aux matchs de l'ancienne regexp
 					$regs = [
 						0 => $c['raw'],
 						1 => $tag,
-						2 => $c['match'][1] . $c['match'][2],
+						2 => $c['match'][1].$c['match'][2],
 						3 => $c['innerHtml'],
 						'tag' => $this->tag,
 					] + $c;

Braces +3 added lines, -6 removed lines

@@ -79,8 +79,7 @@ 
 				$tag['innerHtml'] = '';
 				$tag['attributs'] = trim(substr($tag['opening'], strlen($this->tag) + 1, -2));
 				$tags[] = $tag;
-			}
-			else {
+			} else {
 				// enlever les closing qui sont avant le premier opening, car ils n'ont pas de sens
 				while (
 					!empty($closing)
@@ -99,8 +98,7 @@ 
 						array_shift($opening);
 						$next_opening = reset($opening);
 					}
-				}
-				else {
+				} else {
 					while ($next_opening && $next_closing && $next_opening['pos'] < $next_closing['pos']) {
 						while ($next_opening && $next_opening['pos'] < $next_closing['pos']) {
 							// si pas self closing, il faut un closing de plus
@@ -126,8 +124,7 @@ 
 					$tag['innerHtml'] = '';
 					$tag['attributs'] = trim(substr($tag['opening'], strlen($this->tag) + 1, -1));
 					$tags[] = $tag;
-				}
-				else {
+				} else {
 					$tag = $first_opening;
 					$next_closing = array_shift($closing);
 					$innerHtml = substr($texte, $tag['pos'] + $tag['length'], $next_closing['pos'] - $tag['pos'] - $tag['length']);

ecrire/src/Compilateur/Iterateur/Factory.php

Indentation +53 added lines, -53 removed lines

@@ -12,60 +12,60 @@
  */
 class Factory
 {
-	public static function create($iterateur, $command, $info = null) {
-		$iter = null;
-		// cas des SI {si expression} analises tres tot
-		// pour eviter le chargement de tout iterateur
-		if (isset($command['si'])) {
-			foreach ($command['si'] as $si) {
-				if (!$si) {
-					// $command pour boucle SQL peut generer des erreurs de compilation
-					// s'il est transmis alors qu'on est dans un iterateur vide
-					return new Decorator(new EmptyIterator(), [], $info);
-				}
-			}
-		}
+    public static function create($iterateur, $command, $info = null) {
+        $iter = null;
+        // cas des SI {si expression} analises tres tot
+        // pour eviter le chargement de tout iterateur
+        if (isset($command['si'])) {
+            foreach ($command['si'] as $si) {
+                if (!$si) {
+                    // $command pour boucle SQL peut generer des erreurs de compilation
+                    // s'il est transmis alors qu'on est dans un iterateur vide
+                    return new Decorator(new EmptyIterator(), [], $info);
+                }
+            }
+        }
 
-		// chercher un iterateur PHP existant (par exemple dans SPL)
-		// (il faudrait passer l'argument ->sql_serveur
-		// pour etre certain qu'on est sur un "php:")
-		if (class_exists($iterateur)) {
-			$a = $command['args'] ?? [];
+        // chercher un iterateur PHP existant (par exemple dans SPL)
+        // (il faudrait passer l'argument ->sql_serveur
+        // pour etre certain qu'on est sur un "php:")
+        if (class_exists($iterateur)) {
+            $a = $command['args'] ?? [];
 
-			// permettre de passer un Iterateur directement {args #ITERATEUR} :
-			// si on recoit deja un iterateur en argument, on l'utilise
-			if ((is_countable($a) ? count($a) : 0) == 1 && is_object($a[0]) && is_subclass_of($a[0], \Iterator::class)) {
-				$iter = $a[0];
-			} else {
-				// sinon, on cree un iterateur du type donne
-				// arguments de creation de l'iterateur...
-				try {
-					$iter = new $iterateur(...$a);
-				} catch (Exception $e) {
-					spip_log("Erreur de chargement de l'iterateur {$iterateur}");
-					spip_log($e->getMessage());
-					$iter = new EmptyIterator();
-				}
-			}
-		} else {
-			// chercher la classe d'iterateur Iterateur/XXX
-			// definie dans le fichier src/Compilateur/Iterateur/xxx.php
-			// FIXME: déclarer quelque part les iterateurs supplémentaires
-			$class = __NAMESPACE__ . '\\' . ucfirst(strtolower((string) $iterateur));
-			if (!class_exists($class)) {
-				// historique
-				// Chercher IterateurXXX
-				include_spip('iterateur/' . strtolower($iterateur));
-				$class = 'Iterateur' . $iterateur;
-				if (!class_exists($class)) {
-					exit("Iterateur {$iterateur} non trouvé");
-					// si l'iterateur n'existe pas, on se rabat sur le generique
-					// $iter = new EmptyIterator();
-				}
-			}
-			$iter = new $class($command, $info);
-		}
+            // permettre de passer un Iterateur directement {args #ITERATEUR} :
+            // si on recoit deja un iterateur en argument, on l'utilise
+            if ((is_countable($a) ? count($a) : 0) == 1 && is_object($a[0]) && is_subclass_of($a[0], \Iterator::class)) {
+                $iter = $a[0];
+            } else {
+                // sinon, on cree un iterateur du type donne
+                // arguments de creation de l'iterateur...
+                try {
+                    $iter = new $iterateur(...$a);
+                } catch (Exception $e) {
+                    spip_log("Erreur de chargement de l'iterateur {$iterateur}");
+                    spip_log($e->getMessage());
+                    $iter = new EmptyIterator();
+                }
+            }
+        } else {
+            // chercher la classe d'iterateur Iterateur/XXX
+            // definie dans le fichier src/Compilateur/Iterateur/xxx.php
+            // FIXME: déclarer quelque part les iterateurs supplémentaires
+            $class = __NAMESPACE__ . '\\' . ucfirst(strtolower((string) $iterateur));
+            if (!class_exists($class)) {
+                // historique
+                // Chercher IterateurXXX
+                include_spip('iterateur/' . strtolower($iterateur));
+                $class = 'Iterateur' . $iterateur;
+                if (!class_exists($class)) {
+                    exit("Iterateur {$iterateur} non trouvé");
+                    // si l'iterateur n'existe pas, on se rabat sur le generique
+                    // $iter = new EmptyIterator();
+                }
+            }
+            $iter = new $class($command, $info);
+        }
 
-		return new Decorator($iter, $command, $info);
-	}
+        return new Decorator($iter, $command, $info);
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -51,12 +51,12 @@
 			// chercher la classe d'iterateur Iterateur/XXX
 			// definie dans le fichier src/Compilateur/Iterateur/xxx.php
 			// FIXME: déclarer quelque part les iterateurs supplémentaires
-			$class = __NAMESPACE__ . '\\' . ucfirst(strtolower((string) $iterateur));
+			$class = __NAMESPACE__.'\\'.ucfirst(strtolower((string) $iterateur));
 			if (!class_exists($class)) {
 				// historique
 				// Chercher IterateurXXX
-				include_spip('iterateur/' . strtolower($iterateur));
-				$class = 'Iterateur' . $iterateur;
+				include_spip('iterateur/'.strtolower($iterateur));
+				$class = 'Iterateur'.$iterateur;
 				if (!class_exists($class)) {
 					exit("Iterateur {$iterateur} non trouvé");
 					// si l'iterateur n'existe pas, on se rabat sur le generique

ecrire/src/Compilateur/Iterateur/Sql.php

Indentation +176 added lines, -176 removed lines

@@ -11,187 +11,187 @@
  */
 class Sql extends AbstractIterateur implements Iterator
 {
-	/**
-	 * Ressource sql.
-	 *
-	 * @var bool|object
-	 */
-	protected $sqlresult = false;
+    /**
+     * Ressource sql.
+     *
+     * @var bool|object
+     */
+    protected $sqlresult = false;
 
-	/**
-	 * row sql courante.
-	 *
-	 * @var null|array
-	 */
-	protected $row;
+    /**
+     * row sql courante.
+     *
+     * @var null|array
+     */
+    protected $row;
 
-	protected bool $firstseek = false;
+    protected bool $firstseek = false;
 
-	protected int $pos = -1;
+    protected int $pos = -1;
 
-	/*
+    /*
 	 * array command: les commandes d'initialisation
 	 * array info: les infos sur le squelette
 	 */
-	public function __construct(array $command, array $info = []) {
-		$this->type = 'SQL';
-		parent::__construct($command, $info);
-
-		$this->select();
-	}
-
-	/**
-	 * Rembobiner.
-	 *
-	 * @return bool
-	 */
-	public function rewind(): void {
-		if ($this->pos > 0) {
-			$this->seek(0);
-		}
-	}
-
-	/**
-	 * Verifier l'etat de l'iterateur.
-	 */
-	public function valid(): bool {
-		if ($this->err) {
-			return false;
-		}
-		if (!$this->firstseek) {
-			$this->next();
-		}
-
-		return is_array($this->row);
-	}
-
-	/**
-	 * Valeurs sur la position courante.
-	 *
-	 * @return array
-	 */
-	public function current(): ?array {
-		return $this->row;
-	}
-
-	public function key(): int {
-		return $this->pos;
-	}
-
-	/**
-	 * Sauter a une position absolue.
-	 *
-	 * @param int         $n
-	 * @param null|string $continue
-	 */
-	public function seek($n = 0, $continue = null): bool {
-		if (!sql_seek($this->sqlresult, $n, $this->command['connect'], $continue)) {
-			// SQLite ne sait pas seek(), il faut relancer la query
-			// si la position courante est apres la position visee
-			// il faut relancer la requete
-			if ($this->pos > $n) {
-				$this->free();
-				$this->select();
-				$this->valid();
-			}
-			// et utiliser la methode par defaut pour se deplacer au bon endroit
-			// (sera fait en cas d'echec de cette fonction)
-			return false;
-		}
-		$this->row = sql_fetch($this->sqlresult, $this->command['connect']);
-		$this->pos = min($n, $this->count());
-
-		return true;
-	}
-
-	/**
-	 * Avancer d'un cran.
-	 */
-	public function next(): void {
-		$this->row = sql_fetch($this->sqlresult, $this->command['connect']);
-		++$this->pos;
-		$this->firstseek = true;
-	}
-
-	/**
-	 * Avancer et retourner les donnees pour le nouvel element.
-	 *
-	 * @return null|array|bool
-	 */
-	public function fetch() {
-		if ($this->valid()) {
-			$r = $this->current();
-			$this->next();
-		} else {
-			$r = false;
-		}
-
-		return $r;
-	}
-
-	/**
-	 * liberer les ressources.
-	 *
-	 * @return bool
-	 */
-	public function free() {
-		if (!$this->sqlresult) {
-			return true;
-		}
-		$a = sql_free($this->sqlresult, $this->command['connect']);
-		$this->sqlresult = null;
-
-		return $a;
-	}
-
-	/**
-	 * Compter le nombre de resultats.
-	 */
-	public function count(): int {
-		if (is_null($this->total)) {
-			if (!$this->sqlresult) {
-				$this->total = 0;
-			} else {
-				// cas count(*)
-				if (in_array('count(*)', $this->command['select'])) {
-					$this->valid();
-					$s = $this->current();
-					$this->total = (int) $s['count(*)'];
-				} else {
-					$this->total = (int) sql_count($this->sqlresult, $this->command['connect']);
-				}
-			}
-		}
-
-		return $this->total;
-	}
-
-	/**
-	 * selectionner les donnees, ie faire la requete SQL.
-	 */
-	protected function select() {
-		$this->row = null;
-		$v = &$this->command;
-		$this->sqlresult = calculer_select(
-			$v['select'],
-			$v['from'],
-			$v['type'],
-			$v['where'],
-			$v['join'],
-			$v['groupby'],
-			$v['orderby'],
-			$v['limit'],
-			$v['having'],
-			$v['table'],
-			$v['id'],
-			$v['connect'],
-			$this->info
-		);
-		$this->err = !$this->sqlresult;
-		$this->firstseek = false;
-		$this->pos = -1;
-
-		// pas d'init a priori, le calcul ne sera fait qu'en cas de besoin (provoque une double requete souvent inutile en sqlite)
-		//$this->total = $this->count();
-	}
+    public function __construct(array $command, array $info = []) {
+        $this->type = 'SQL';
+        parent::__construct($command, $info);
+
+        $this->select();
+    }
+
+    /**
+     * Rembobiner.
+     *
+     * @return bool
+     */
+    public function rewind(): void {
+        if ($this->pos > 0) {
+            $this->seek(0);
+        }
+    }
+
+    /**
+     * Verifier l'etat de l'iterateur.
+     */
+    public function valid(): bool {
+        if ($this->err) {
+            return false;
+        }
+        if (!$this->firstseek) {
+            $this->next();
+        }
+
+        return is_array($this->row);
+    }
+
+    /**
+     * Valeurs sur la position courante.
+     *
+     * @return array
+     */
+    public function current(): ?array {
+        return $this->row;
+    }
+
+    public function key(): int {
+        return $this->pos;
+    }
+
+    /**
+     * Sauter a une position absolue.
+     *
+     * @param int         $n
+     * @param null|string $continue
+     */
+    public function seek($n = 0, $continue = null): bool {
+        if (!sql_seek($this->sqlresult, $n, $this->command['connect'], $continue)) {
+            // SQLite ne sait pas seek(), il faut relancer la query
+            // si la position courante est apres la position visee
+            // il faut relancer la requete
+            if ($this->pos > $n) {
+                $this->free();
+                $this->select();
+                $this->valid();
+            }
+            // et utiliser la methode par defaut pour se deplacer au bon endroit
+            // (sera fait en cas d'echec de cette fonction)
+            return false;
+        }
+        $this->row = sql_fetch($this->sqlresult, $this->command['connect']);
+        $this->pos = min($n, $this->count());
+
+        return true;
+    }
+
+    /**
+     * Avancer d'un cran.
+     */
+    public function next(): void {
+        $this->row = sql_fetch($this->sqlresult, $this->command['connect']);
+        ++$this->pos;
+        $this->firstseek = true;
+    }
+
+    /**
+     * Avancer et retourner les donnees pour le nouvel element.
+     *
+     * @return null|array|bool
+     */
+    public function fetch() {
+        if ($this->valid()) {
+            $r = $this->current();
+            $this->next();
+        } else {
+            $r = false;
+        }
+
+        return $r;
+    }
+
+    /**
+     * liberer les ressources.
+     *
+     * @return bool
+     */
+    public function free() {
+        if (!$this->sqlresult) {
+            return true;
+        }
+        $a = sql_free($this->sqlresult, $this->command['connect']);
+        $this->sqlresult = null;
+
+        return $a;
+    }
+
+    /**
+     * Compter le nombre de resultats.
+     */
+    public function count(): int {
+        if (is_null($this->total)) {
+            if (!$this->sqlresult) {
+                $this->total = 0;
+            } else {
+                // cas count(*)
+                if (in_array('count(*)', $this->command['select'])) {
+                    $this->valid();
+                    $s = $this->current();
+                    $this->total = (int) $s['count(*)'];
+                } else {
+                    $this->total = (int) sql_count($this->sqlresult, $this->command['connect']);
+                }
+            }
+        }
+
+        return $this->total;
+    }
+
+    /**
+     * selectionner les donnees, ie faire la requete SQL.
+     */
+    protected function select() {
+        $this->row = null;
+        $v = &$this->command;
+        $this->sqlresult = calculer_select(
+            $v['select'],
+            $v['from'],
+            $v['type'],
+            $v['where'],
+            $v['join'],
+            $v['groupby'],
+            $v['orderby'],
+            $v['limit'],
+            $v['having'],
+            $v['table'],
+            $v['id'],
+            $v['connect'],
+            $this->info
+        );
+        $this->err = !$this->sqlresult;
+        $this->firstseek = false;
+        $this->pos = -1;
+
+        // pas d'init a priori, le calcul ne sera fait qu'en cas de besoin (provoque une double requete souvent inutile en sqlite)
+        //$this->total = $this->count();
+    }
 }

ecrire/src/Compilateur/Noeud/Boucle.php

Indentation +364 added lines, -364 removed lines

@@ -7,368 +7,368 @@
  */
 class Boucle
 {
-	/** Type de noeud */
-	public string $type = 'boucle';
-
-	/** Identifiant de la boucle */
-	public string $id_boucle;
-
-	/** Identifiant de la boucle parente */
-	public string $id_parent = '';
-
-	/** Un nom explicite qui peut être affecté manuellement à certaines boucles générées */
-	public string $nom = '';
-
-	/**
-	 * Partie avant toujours affichee
-	 *
-	 * @var string|array
-	 */
-	public $preaff = '';
-
-	/**
-	 * Partie optionnelle avant
-	 *
-	 * @var string|array
-	 */
-	public $avant = '';
-
-	/**
-	 * Pour chaque élément
-	 *
-	 * @var string|array
-	 */
-	public $milieu = '';
-
-	/**
-	 * Partie optionnelle après
-	 *
-	 * @var string|array
-	 */
-	public $apres = '';
-
-	/**
-	 * Partie alternative, si pas de résultat dans la boucle
-	 *
-	 * @var string|array
-	 */
-	public $altern = '';
-
-	/**
-	 * Partie apres toujours affichee
-	 *
-	 * @var string|array
-	 */
-	public $postaff = '';
-
-
-	/**
-	 * La boucle doit-elle sélectionner la langue ?
-	 *
-	 * Valeurs : '', 'oui', 'non'
-	 */
-	public string $lang_select = '';
-
-	/**
-	 * Alias de table d'application de la requête ou nom complet de la table SQL
-	 *
-	 * FIXME: un seul typage (string ?)
-	 *
-	 * @var string|false|null
-	 */
-	public $type_requete = null;
-
-	/**
-	 * La table est elle optionnelle ?
-	 *
-	 * Si oui, aucune erreur ne sera générée si la table demandée n'est pas présente
-	 */
-	public bool $table_optionnelle = false;
-	public string $type_table_optionnelle = '';
-
-	/**
-	 * Nom du fichier de connexion
-	 */
-	public string $sql_serveur = '';
-
-	/**
-	 * Paramètres de la boucle
-	 *
-	 * Description des paramètres passés à la boucle, qui servent ensuite
-	 * au calcul des critères
-	 *
-	 * FIXME: type unique.
-	 * @var false|array
-	 *     - false: erreur de syntaxe
-	 */
-	public $param = [];
-
-	/**
-	 * Critères de la boucle
-	 *
-	 * FIXME: type array unique.
-	 *
-	 * @var string|Critere[]
-	 * - string: phrasage (code brut). Il reste si erreur de critère
-	 * - array: analyse correcte des critères...
-	 */
-	public $criteres = [];
-
-	/**
-	 * Textes insérés entre 2 éléments de boucle (critère inter)
-	 *
-	 * @var string[]
-	 */
-	public array $separateur = [];
-
-	/**
-	 * Liste des jointures possibles avec cette table
-	 *
-	 * Les jointures par défaut de la table sont complétées en priorité
-	 * des jointures déclarées explicitement sur la boucle
-	 *
-	 * @see base_trouver_table_dist()
-	 */
-	public array $jointures = [];
-
-	/**
-	 * Jointures explicites avec cette table
-	 *
-	 * Ces jointures sont utilisées en priorité par rapport aux jointures
-	 * normales possibles pour retrouver les colonnes demandées extérieures
-	 * à la boucle.
-	 *
-	 * @var string|bool
-	 */
-	public $jointures_explicites = false;
-
-	/**
-	 * Nom de la variable PHP stockant le noms de doublons utilisés "$doublons_index"
-	 */
-	public string $doublons = '';
-
-	/**
-	 * Code PHP ajouté au début de chaque itération de boucle.
-	 *
-	 * Utilisé entre autre par les critères {pagination}, {n-a,b}, {a/b}...
-	 */
-	public string $partie = '';
-
-	/**
-	 * Nombre de divisions de la boucle, d'éléments à afficher,
-	 * ou de soustractions d'éléments à faire
-	 *
-	 * Dans les critères limitant le nombre d'éléments affichés
-	 * {a,b}, {a,n-b}, {a/b}, {pagination b}, b est affecté à total_parties.
-	 */
-	public string $total_parties = '';
-
-	/**
-	 * Code PHP ajouté avant l'itération de boucle.
-	 *
-	 * Utilisé entre autre par les critères {pagination}, {a,b}, {a/b}
-	 * pour initialiser les variables de début et de fin d'itération.
-	 */
-	public string $mode_partie = '';
-
-	/**
-	 * Identifiant d'une boucle qui appelle celle-ci de manière récursive
-	 *
-	 * Si une boucle est appelée de manière récursive quelque part par
-	 * une autre boucle comme <BOUCLE_rec(boucle_identifiant) />, cette
-	 * boucle (identifiant) reçoit dans cette propriété l'identifiant
-	 * de l'appelant (rec)
-	 */
-	public string $externe = '';
-
-	// champs pour la construction de la requete SQL
-
-	/**
-	 * Liste des champs à récupérer par la boucle
-	 *
-	 * Expression 'table.nom_champ' ou calculée 'nom_champ AS x'
-	 *
-	 * @var string[]
-	 */
-	public array $select = [];
-
-	/**
-	 * Liste des alias / tables SQL utilisées dans la boucle
-	 *
-	 * L'index est un identifiant (xx dans spip_xx assez souvent) qui servira
-	 * d'alias au nom de la table ; la valeur est le nom de la table SQL désirée.
-	 *
-	 * L'index 0 peut définir le type de sources de données de l'itérateur DATA
-	 *
-	 * @var string[]
-	 */
-	public array $from = [];
-
-	/**
-	 * Liste des alias / type de jointures utilisées dans la boucle
-	 *
-	 * L'index est le nom d'alias (comme pour la propriété $from), et la valeur
-	 * un type de jointure parmi 'INNER', 'LEFT', 'RIGHT', 'OUTER'.
-	 *
-	 * Lorsque le type n'est pas déclaré pour un alias, c'est 'INNER'
-	 * qui sera utilisé par défaut (créant donc un INNER JOIN).
-	 *
-	 * @var string[]
-	 */
-	public array $from_type = [];
-
-	/**
-	 * Liste des conditions WHERE de la boucle
-	 *
-	 * Permet de restreindre les éléments retournés par une boucle
-	 * en fonctions des conditions transmises dans ce tableau.
-	 *
-	 * Ce tableau peut avoir plusieurs niveaux de profondeur.
-	 *
-	 * Les éléments du premier niveau sont reliés par des AND, donc
-	 * chaque élément ajouté directement au where par
-	 * $boucle->where[] = array(...) ou $boucle->where[] = "'expression'"
-	 * est une condition AND en plus.
-	 *
-	 * Par contre, lorsqu'on indique un tableau, il peut décrire des relations
-	 * internes différentes. Soit $expr un tableau d'expressions quelconques de 3 valeurs :
-	 * $expr = array(operateur, val1, val2)
-	 *
-	 * Ces 3 valeurs sont des expressions PHP. L'index 0 désigne l'opérateur
-	 * à réaliser tel que :
-	 *
-	 * - "'='" , "'>='", "'<'", "'IN'", "'REGEXP'", "'LIKE'", ... :
-	 *    val1 et val2 sont des champs et valeurs à utiliser dans la comparaison
-	 *    suivant cet ordre : "val1 operateur val2".
-	 *    Exemple : $boucle->where[] = array("'='", "'articles.statut'", "'\"publie\"'");
-	 * - "'AND'", "'OR'", "'NOT'" :
-	 *    dans ce cas val1 et val2 sont également des expressions
-	 *    de comparaison complètes, et peuvent être eux-même des tableaux comme $expr
-	 *    Exemples :
-	 *    $boucle->where[] = array("'OR'", $expr1, $expr2);
-	 *    $boucle->where[] = array("'NOT'", $expr); // val2 n'existe pas avec NOT
-	 *
-	 * D'autres noms sont possibles pour l'opérateur (le nombre de valeurs diffère) :
-	 * - "'SELF'", "'SUBSELECT'" : indiquent des sous requêtes
-	 * - "'?'" : indique une condition à faire évaluer (val1 ? val2 : val3)
-	 */
-	public array $where = [];
-
-	public array $join = [];
-	public array $having = [];
-	public $limit = '';
-	public array $group = [];
-	public array $order = [];
-	public array $default_order = [];
-	public string $date = 'date';
-	public string $hash = '';
-	public $in = '';
-	public bool $sous_requete = false;
-
-	/**
-	 * Code PHP qui sera ajouté en tout début de la fonction de boucle
-	 *
-	 * Il sert à insérer le code calculant une hierarchie
-	 */
-	public string $hierarchie = '';
-
-	// champs pour la construction du corps PHP
-
-	/**
-	 * Description des sources de données de la boucle
-	 *
-	 * Description des données de la boucle issu de trouver_table
-	 * dans le cadre de l'itérateur SQL et contenant au moins l'index 'field'.
-	 *
-	 * @see base_trouver_table_dist()
-	 */
-	public array $show = [];
-
-	/**
-	 * Nom de la table SQL principale de la boucle, sans son préfixe
-	 */
-	public string $id_table = '';
-
-	/**
-	 * Nom de la clé primaire de la table SQL principale de la boucle
-	 */
-	public string $primary = '';
-
-	/**
-	 * Code PHP compilé de la boucle
-	 *
-	 * FIXME: un seul type (string ?)
-	 *
-	 * - false: boucle fautive ?
-	 *
-	 * @var string|false
-	 */
-	public $return = '';
-
-	public $numrows = false;
-	public $cptrows = false;
-
-	/**
-	 * Description du squelette
-	 *
-	 * Sert pour la gestion d'erreur et la production de code dependant du contexte
-	 *
-	 * Peut contenir les index :
-	 *
-	 * - nom : Nom du fichier de cache
-	 * - gram : Nom de la grammaire du squelette (détermine le phraseur à utiliser)
-	 * - sourcefile : Chemin du squelette
-	 * - squelette : Code du squelette
-	 * - id_mere : Identifiant de la boucle parente
-	 * - documents : Pour embed et img dans les textes
-	 * - session : Pour un cache sessionné par auteur
-	 * - niv : Niveau de tabulation
-	 */
-	public array $descr = [];
-
-	/** Numéro de ligne dans le code source du squelette */
-	public int $ligne = 0;
-
-
-	/**
-	 * table pour stocker les modificateurs de boucle tels que tout, plat ...,
-	 * utilisable par les plugins egalement
-	 *
-	 * @var array<string, mixed>
-	 */
-	public array $modificateur = [];
-
-	/**
-	 * Type d'itérateur utilisé pour cette boucle
-	 *
-	 * - 'SQL' dans le cadre d'une boucle sur une table SQL
-	 * - 'DATA' pour l'itérateur DATA, ...
-	 *
-	 * @var string
-	 */
-	public string $iterateur = ''; // type d'iterateur
-
-	/**
-	 * @var array $debug Textes qui seront insérés dans l’entête de boucle du mode debug
-	 */
-	public array $debug = [];
-
-	/**
-	 * Index de la boucle dont le champ présent dans cette boucle est originaire,
-	 * notamment si le champ a été trouve dans une boucle parente
-	 *
-	 * Tableau nom du champ => index de boucle
-	*/
-	public array $index_champ = [];
-
-	/** Résultat de la compilation (?) (sert au débusqueur) */
-	public string $code = '';
-
-	/** Source des filtres (compatibilité) (?) */
-	public array $fonctions = [];
-
-	// obsoletes, conserves provisoirement pour compatibilite
-	public $tout = false;
-	public $plat = false;
-	public $lien = false;
+    /** Type de noeud */
+    public string $type = 'boucle';
+
+    /** Identifiant de la boucle */
+    public string $id_boucle;
+
+    /** Identifiant de la boucle parente */
+    public string $id_parent = '';
+
+    /** Un nom explicite qui peut être affecté manuellement à certaines boucles générées */
+    public string $nom = '';
+
+    /**
+     * Partie avant toujours affichee
+     *
+     * @var string|array
+     */
+    public $preaff = '';
+
+    /**
+     * Partie optionnelle avant
+     *
+     * @var string|array
+     */
+    public $avant = '';
+
+    /**
+     * Pour chaque élément
+     *
+     * @var string|array
+     */
+    public $milieu = '';
+
+    /**
+     * Partie optionnelle après
+     *
+     * @var string|array
+     */
+    public $apres = '';
+
+    /**
+     * Partie alternative, si pas de résultat dans la boucle
+     *
+     * @var string|array
+     */
+    public $altern = '';
+
+    /**
+     * Partie apres toujours affichee
+     *
+     * @var string|array
+     */
+    public $postaff = '';
+
+
+    /**
+     * La boucle doit-elle sélectionner la langue ?
+     *
+     * Valeurs : '', 'oui', 'non'
+     */
+    public string $lang_select = '';
+
+    /**
+     * Alias de table d'application de la requête ou nom complet de la table SQL
+     *
+     * FIXME: un seul typage (string ?)
+     *
+     * @var string|false|null
+     */
+    public $type_requete = null;
+
+    /**
+     * La table est elle optionnelle ?
+     *
+     * Si oui, aucune erreur ne sera générée si la table demandée n'est pas présente
+     */
+    public bool $table_optionnelle = false;
+    public string $type_table_optionnelle = '';
+
+    /**
+     * Nom du fichier de connexion
+     */
+    public string $sql_serveur = '';
+
+    /**
+     * Paramètres de la boucle
+     *
+     * Description des paramètres passés à la boucle, qui servent ensuite
+     * au calcul des critères
+     *
+     * FIXME: type unique.
+     * @var false|array
+     *     - false: erreur de syntaxe
+     */
+    public $param = [];
+
+    /**
+     * Critères de la boucle
+     *
+     * FIXME: type array unique.
+     *
+     * @var string|Critere[]
+     * - string: phrasage (code brut). Il reste si erreur de critère
+     * - array: analyse correcte des critères...
+     */
+    public $criteres = [];
+
+    /**
+     * Textes insérés entre 2 éléments de boucle (critère inter)
+     *
+     * @var string[]
+     */
+    public array $separateur = [];
+
+    /**
+     * Liste des jointures possibles avec cette table
+     *
+     * Les jointures par défaut de la table sont complétées en priorité
+     * des jointures déclarées explicitement sur la boucle
+     *
+     * @see base_trouver_table_dist()
+     */
+    public array $jointures = [];
+
+    /**
+     * Jointures explicites avec cette table
+     *
+     * Ces jointures sont utilisées en priorité par rapport aux jointures
+     * normales possibles pour retrouver les colonnes demandées extérieures
+     * à la boucle.
+     *
+     * @var string|bool
+     */
+    public $jointures_explicites = false;
+
+    /**
+     * Nom de la variable PHP stockant le noms de doublons utilisés "$doublons_index"
+     */
+    public string $doublons = '';
+
+    /**
+     * Code PHP ajouté au début de chaque itération de boucle.
+     *
+     * Utilisé entre autre par les critères {pagination}, {n-a,b}, {a/b}...
+     */
+    public string $partie = '';
+
+    /**
+     * Nombre de divisions de la boucle, d'éléments à afficher,
+     * ou de soustractions d'éléments à faire
+     *
+     * Dans les critères limitant le nombre d'éléments affichés
+     * {a,b}, {a,n-b}, {a/b}, {pagination b}, b est affecté à total_parties.
+     */
+    public string $total_parties = '';
+
+    /**
+     * Code PHP ajouté avant l'itération de boucle.
+     *
+     * Utilisé entre autre par les critères {pagination}, {a,b}, {a/b}
+     * pour initialiser les variables de début et de fin d'itération.
+     */
+    public string $mode_partie = '';
+
+    /**
+     * Identifiant d'une boucle qui appelle celle-ci de manière récursive
+     *
+     * Si une boucle est appelée de manière récursive quelque part par
+     * une autre boucle comme <BOUCLE_rec(boucle_identifiant) />, cette
+     * boucle (identifiant) reçoit dans cette propriété l'identifiant
+     * de l'appelant (rec)
+     */
+    public string $externe = '';
+
+    // champs pour la construction de la requete SQL
+
+    /**
+     * Liste des champs à récupérer par la boucle
+     *
+     * Expression 'table.nom_champ' ou calculée 'nom_champ AS x'
+     *
+     * @var string[]
+     */
+    public array $select = [];
+
+    /**
+     * Liste des alias / tables SQL utilisées dans la boucle
+     *
+     * L'index est un identifiant (xx dans spip_xx assez souvent) qui servira
+     * d'alias au nom de la table ; la valeur est le nom de la table SQL désirée.
+     *
+     * L'index 0 peut définir le type de sources de données de l'itérateur DATA
+     *
+     * @var string[]
+     */
+    public array $from = [];
+
+    /**
+     * Liste des alias / type de jointures utilisées dans la boucle
+     *
+     * L'index est le nom d'alias (comme pour la propriété $from), et la valeur
+     * un type de jointure parmi 'INNER', 'LEFT', 'RIGHT', 'OUTER'.
+     *
+     * Lorsque le type n'est pas déclaré pour un alias, c'est 'INNER'
+     * qui sera utilisé par défaut (créant donc un INNER JOIN).
+     *
+     * @var string[]
+     */
+    public array $from_type = [];
+
+    /**
+     * Liste des conditions WHERE de la boucle
+     *
+     * Permet de restreindre les éléments retournés par une boucle
+     * en fonctions des conditions transmises dans ce tableau.
+     *
+     * Ce tableau peut avoir plusieurs niveaux de profondeur.
+     *
+     * Les éléments du premier niveau sont reliés par des AND, donc
+     * chaque élément ajouté directement au where par
+     * $boucle->where[] = array(...) ou $boucle->where[] = "'expression'"
+     * est une condition AND en plus.
+     *
+     * Par contre, lorsqu'on indique un tableau, il peut décrire des relations
+     * internes différentes. Soit $expr un tableau d'expressions quelconques de 3 valeurs :
+     * $expr = array(operateur, val1, val2)
+     *
+     * Ces 3 valeurs sont des expressions PHP. L'index 0 désigne l'opérateur
+     * à réaliser tel que :
+     *
+     * - "'='" , "'>='", "'<'", "'IN'", "'REGEXP'", "'LIKE'", ... :
+     *    val1 et val2 sont des champs et valeurs à utiliser dans la comparaison
+     *    suivant cet ordre : "val1 operateur val2".
+     *    Exemple : $boucle->where[] = array("'='", "'articles.statut'", "'\"publie\"'");
+     * - "'AND'", "'OR'", "'NOT'" :
+     *    dans ce cas val1 et val2 sont également des expressions
+     *    de comparaison complètes, et peuvent être eux-même des tableaux comme $expr
+     *    Exemples :
+     *    $boucle->where[] = array("'OR'", $expr1, $expr2);
+     *    $boucle->where[] = array("'NOT'", $expr); // val2 n'existe pas avec NOT
+     *
+     * D'autres noms sont possibles pour l'opérateur (le nombre de valeurs diffère) :
+     * - "'SELF'", "'SUBSELECT'" : indiquent des sous requêtes
+     * - "'?'" : indique une condition à faire évaluer (val1 ? val2 : val3)
+     */
+    public array $where = [];
+
+    public array $join = [];
+    public array $having = [];
+    public $limit = '';
+    public array $group = [];
+    public array $order = [];
+    public array $default_order = [];
+    public string $date = 'date';
+    public string $hash = '';
+    public $in = '';
+    public bool $sous_requete = false;
+
+    /**
+     * Code PHP qui sera ajouté en tout début de la fonction de boucle
+     *
+     * Il sert à insérer le code calculant une hierarchie
+     */
+    public string $hierarchie = '';
+
+    // champs pour la construction du corps PHP
+
+    /**
+     * Description des sources de données de la boucle
+     *
+     * Description des données de la boucle issu de trouver_table
+     * dans le cadre de l'itérateur SQL et contenant au moins l'index 'field'.
+     *
+     * @see base_trouver_table_dist()
+     */
+    public array $show = [];
+
+    /**
+     * Nom de la table SQL principale de la boucle, sans son préfixe
+     */
+    public string $id_table = '';
+
+    /**
+     * Nom de la clé primaire de la table SQL principale de la boucle
+     */
+    public string $primary = '';
+
+    /**
+     * Code PHP compilé de la boucle
+     *
+     * FIXME: un seul type (string ?)
+     *
+     * - false: boucle fautive ?
+     *
+     * @var string|false
+     */
+    public $return = '';
+
+    public $numrows = false;
+    public $cptrows = false;
+
+    /**
+     * Description du squelette
+     *
+     * Sert pour la gestion d'erreur et la production de code dependant du contexte
+     *
+     * Peut contenir les index :
+     *
+     * - nom : Nom du fichier de cache
+     * - gram : Nom de la grammaire du squelette (détermine le phraseur à utiliser)
+     * - sourcefile : Chemin du squelette
+     * - squelette : Code du squelette
+     * - id_mere : Identifiant de la boucle parente
+     * - documents : Pour embed et img dans les textes
+     * - session : Pour un cache sessionné par auteur
+     * - niv : Niveau de tabulation
+     */
+    public array $descr = [];
+
+    /** Numéro de ligne dans le code source du squelette */
+    public int $ligne = 0;
+
+
+    /**
+     * table pour stocker les modificateurs de boucle tels que tout, plat ...,
+     * utilisable par les plugins egalement
+     *
+     * @var array<string, mixed>
+     */
+    public array $modificateur = [];
+
+    /**
+     * Type d'itérateur utilisé pour cette boucle
+     *
+     * - 'SQL' dans le cadre d'une boucle sur une table SQL
+     * - 'DATA' pour l'itérateur DATA, ...
+     *
+     * @var string
+     */
+    public string $iterateur = ''; // type d'iterateur
+
+    /**
+     * @var array $debug Textes qui seront insérés dans l’entête de boucle du mode debug
+     */
+    public array $debug = [];
+
+    /**
+     * Index de la boucle dont le champ présent dans cette boucle est originaire,
+     * notamment si le champ a été trouve dans une boucle parente
+     *
+     * Tableau nom du champ => index de boucle
+     */
+    public array $index_champ = [];
+
+    /** Résultat de la compilation (?) (sert au débusqueur) */
+    public string $code = '';
+
+    /** Source des filtres (compatibilité) (?) */
+    public array $fonctions = [];
+
+    // obsoletes, conserves provisoirement pour compatibilite
+    public $tout = false;
+    public $plat = false;
+    public $lien = false;
 }