spip / SPIP

ecrire/action/instituer_objet.php

Indentation +19 added lines, -19 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -28,24 +28,24 @@ 
  */
 function action_instituer_objet_dist($arg = null) {
 
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
 
-	[$objet, $id_objet, $statut] = preg_split('/\W/', (string) $arg);
-	if (!$statut) {
-		$statut = _request('statut_nouv');
-	} // cas POST
-	if (!$statut) {
-		return;
-	} // impossible mais sait-on jamais
+    [$objet, $id_objet, $statut] = preg_split('/\W/', (string) $arg);
+    if (!$statut) {
+        $statut = _request('statut_nouv');
+    } // cas POST
+    if (!$statut) {
+        return;
+    } // impossible mais sait-on jamais
 
-	if (
-		($id_objet = (int) $id_objet)
-		&& autoriser('instituer', $objet, $id_objet, '', ['statut' => $statut])
-	) {
-		include_spip('action/editer_objet');
-		objet_modifier($objet, $id_objet, ['statut' => $statut]);
-	}
+    if (
+        ($id_objet = (int) $id_objet)
+        && autoriser('instituer', $objet, $id_objet, '', ['statut' => $statut])
+    ) {
+        include_spip('action/editer_objet');
+        objet_modifier($objet, $id_objet, ['statut' => $statut]);
+    }
 }

ecrire/action/annuler_job.php

Indentation +9 added lines, -9 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -25,13 +25,13 @@ 
  * @return void
  */
 function action_annuler_job_dist() {
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$id_job = $securiser_action();
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $id_job = $securiser_action();
 
-	if (
-		($id_job = (int) $id_job)
-		&& autoriser('annuler', 'job', $id_job)
-	) {
-		job_queue_remove($id_job);
-	}
+    if (
+        ($id_job = (int) $id_job)
+        && autoriser('annuler', 'job', $id_job)
+    ) {
+        job_queue_remove($id_job);
+    }
 }

ecrire/action/activer_plugins.php

Indentation +46 added lines, -46 removed lines

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

Spacing +3 added lines, -3 removed lines

@@ -31,11 +31,11 @@ 
 	// recuperer les plugins dans l'ordre des $_POST
 	$test = [];
 	foreach (liste_plugin_files() as $file) {
-		$test['s' . substr(md5(_DIR_PLUGINS . $file), 0, 16)] = $file;
+		$test['s'.substr(md5(_DIR_PLUGINS.$file), 0, 16)] = $file;
 	}
 	if (defined('_DIR_PLUGINS_SUPPL')) {
 		foreach (liste_plugin_files(_DIR_PLUGINS_SUPPL) as $file) {
-			$test['s' . substr(md5(_DIR_PLUGINS_SUPPL . $file), 0, 16)] = $file;
+			$test['s'.substr(md5(_DIR_PLUGINS_SUPPL.$file), 0, 16)] = $file;
 		}
 	}
 
@@ -47,7 +47,7 @@ 
 		}
 	}
 
-	spip_log("Changement des plugins actifs par l'auteur " . $GLOBALS['visiteur_session']['id_auteur'] . ': ' . implode(
+	spip_log("Changement des plugins actifs par l'auteur ".$GLOBALS['visiteur_session']['id_auteur'].': '.implode(
 		',',
 		$plugin
 	));

ecrire/action/cookie.php

Spacing +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
 	if ($change_session == 'oui') {
 		$session = charger_fonction('session', 'inc');
 		$session(true);
-		spip_log('statut 204 pour ' . $_SERVER['REQUEST_URI']);
+		spip_log('statut 204 pour '.$_SERVER['REQUEST_URI']);
 		http_response_code(204); // No Content
 		return;
 	}
@@ -78,7 +78,7 @@ 
 				_T('info_connexion_refusee'),
 				_T('login_login_pass_incorrect'),
 				_T('login_retour_site'),
-				'url=' . rawurlencode((string) $redirect),
+				'url='.rawurlencode((string) $redirect),
 				_T('login_nouvelle_tentative'),
 				(str_contains((string) $url, (string) _DIR_RESTREINT_ABS))
 			);

Indentation +86 added lines, -86 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -32,94 +32,94 @@ 
  * @return void
  */
 function action_cookie_dist($set_cookie_admin = null, $change_session = null) {
-	$redirect_echec = $redirect = null;
-	$test_echec_cookie = null;
-	$url = '';
-	if (is_null($set_cookie_admin)) {
-		$set_cookie_admin = _request('cookie_admin');
-		$change_session = _request('change_session');
-		$test_echec_cookie = _request('test_echec_cookie');
+    $redirect_echec = $redirect = null;
+    $test_echec_cookie = null;
+    $url = '';
+    if (is_null($set_cookie_admin)) {
+        $set_cookie_admin = _request('cookie_admin');
+        $change_session = _request('change_session');
+        $test_echec_cookie = _request('test_echec_cookie');
 
-		// La cible de notre operation de connexion
-		$url = securiser_redirect_action(_request('url'));
-		$redirect = $url ?: generer_url_ecrire('accueil');
-		$redirect_echec = _request('url_echec');
-		if (!isset($redirect_echec)) {
-			if (str_contains((string) $redirect, (string) _DIR_RESTREINT_ABS)) {
-				$redirect_echec = generer_url_public('login', '', true);
-			} else {
-				$redirect_echec = $redirect;
-			}
-		}
-	}
+        // La cible de notre operation de connexion
+        $url = securiser_redirect_action(_request('url'));
+        $redirect = $url ?: generer_url_ecrire('accueil');
+        $redirect_echec = _request('url_echec');
+        if (!isset($redirect_echec)) {
+            if (str_contains((string) $redirect, (string) _DIR_RESTREINT_ABS)) {
+                $redirect_echec = generer_url_public('login', '', true);
+            } else {
+                $redirect_echec = $redirect;
+            }
+        }
+    }
 
 
-	// rejoue le cookie pour renouveler spip_session
-	if ($change_session == 'oui') {
-		$session = charger_fonction('session', 'inc');
-		$session(true);
-		spip_log('statut 204 pour ' . $_SERVER['REQUEST_URI']);
-		http_response_code(204); // No Content
-		return;
-	}
+    // rejoue le cookie pour renouveler spip_session
+    if ($change_session == 'oui') {
+        $session = charger_fonction('session', 'inc');
+        $session(true);
+        spip_log('statut 204 pour ' . $_SERVER['REQUEST_URI']);
+        http_response_code(204); // No Content
+        return;
+    }
 
-	// tentative de connexion en auth_http
-	if (_request('essai_auth_http') && !$GLOBALS['ignore_auth_http']) {
-		include_spip('inc/auth');
-		if (
-			@$_SERVER['PHP_AUTH_USER']
-			&& @$_SERVER['PHP_AUTH_PW']
-			&& ($auteur = lire_php_auth($_SERVER['PHP_AUTH_USER'], $_SERVER['PHP_AUTH_PW']))
-		) {
-			auth_loger($auteur);
-			redirige_par_entete(parametre_url($redirect, 't', time(), '&'));
-		} else {
-			ask_php_auth(
-				_T('info_connexion_refusee'),
-				_T('login_login_pass_incorrect'),
-				_T('login_retour_site'),
-				'url=' . rawurlencode((string) $redirect),
-				_T('login_nouvelle_tentative'),
-				(str_contains((string) $url, (string) _DIR_RESTREINT_ABS))
-			);
-		}
-	} else {
-		// en cas de login sur bonjour=oui, on tente de poser un cookie
-		// puis de passer au login qui diagnostiquera l'echec de cookie
-		// le cas echeant.
-		if ($test_echec_cookie == 'oui') {
-			spip_setcookie('spip_session', 'test_echec_cookie', httponly: true);
-			if ($redirect) {
-				$redirect = parametre_url(
-					parametre_url($redirect_echec, 'var_echec_cookie', 'oui', '&'),
-					'url',
-					rawurlencode((string) $redirect),
-					'&'
-				);
-			}
-		} else {
-			$cook = $_COOKIE['spip_admin'] ?? '';
-			// Suppression cookie d'admin ?
-			if ($set_cookie_admin == 'non') {
-				if ($cook) {
-					spip_setcookie('spip_admin', $cook, time() - 3600 * 24, httponly: true);
-				}
-			} // Ajout de cookie d'admin
-			else {
-				if ($set_cookie_admin && _DUREE_COOKIE_ADMIN) {
-					spip_setcookie(
-						'spip_admin',
-						$set_cookie_admin,
-						time() + max(_DUREE_COOKIE_ADMIN, 2 * _RENOUVELLE_ALEA),
-						httponly: true
-					);
-				}
-			}
-		}
-	}
+    // tentative de connexion en auth_http
+    if (_request('essai_auth_http') && !$GLOBALS['ignore_auth_http']) {
+        include_spip('inc/auth');
+        if (
+            @$_SERVER['PHP_AUTH_USER']
+            && @$_SERVER['PHP_AUTH_PW']
+            && ($auteur = lire_php_auth($_SERVER['PHP_AUTH_USER'], $_SERVER['PHP_AUTH_PW']))
+        ) {
+            auth_loger($auteur);
+            redirige_par_entete(parametre_url($redirect, 't', time(), '&'));
+        } else {
+            ask_php_auth(
+                _T('info_connexion_refusee'),
+                _T('login_login_pass_incorrect'),
+                _T('login_retour_site'),
+                'url=' . rawurlencode((string) $redirect),
+                _T('login_nouvelle_tentative'),
+                (str_contains((string) $url, (string) _DIR_RESTREINT_ABS))
+            );
+        }
+    } else {
+        // en cas de login sur bonjour=oui, on tente de poser un cookie
+        // puis de passer au login qui diagnostiquera l'echec de cookie
+        // le cas echeant.
+        if ($test_echec_cookie == 'oui') {
+            spip_setcookie('spip_session', 'test_echec_cookie', httponly: true);
+            if ($redirect) {
+                $redirect = parametre_url(
+                    parametre_url($redirect_echec, 'var_echec_cookie', 'oui', '&'),
+                    'url',
+                    rawurlencode((string) $redirect),
+                    '&'
+                );
+            }
+        } else {
+            $cook = $_COOKIE['spip_admin'] ?? '';
+            // Suppression cookie d'admin ?
+            if ($set_cookie_admin == 'non') {
+                if ($cook) {
+                    spip_setcookie('spip_admin', $cook, time() - 3600 * 24, httponly: true);
+                }
+            } // Ajout de cookie d'admin
+            else {
+                if ($set_cookie_admin && _DUREE_COOKIE_ADMIN) {
+                    spip_setcookie(
+                        'spip_admin',
+                        $set_cookie_admin,
+                        time() + max(_DUREE_COOKIE_ADMIN, 2 * _RENOUVELLE_ALEA),
+                        httponly: true
+                    );
+                }
+            }
+        }
+    }
 
-	// Redirection finale
-	if ($redirect) {
-		redirige_par_entete($redirect, true);
-	}
+    // Redirection finale
+    if ($redirect) {
+        redirige_par_entete($redirect, true);
+    }
 }

ecrire/src/Sql/Sqlite/Traducteur.php

Indentation +191 added lines, -191 removed lines

@@ -9,195 +9,195 @@
  */
 class Traducteur
 {
-	/** Pour les corrections à effectuer sur les requêtes : array(code=>'texte') trouvé
-	 *
-	 * @var array
-	 */
-	public $textes = [];
-
-	/**
-	 * Constructeur
-	 */
-	public function __construct(
-		/** Requête à préparer */
-		public string $query,
-		/** Prefixe des tables à utiliser */
-		public string $prefixe,
-		/** Version SQLite (2 ou 3) */
-		public string $sqlite_version
-	) {
-	}
-
-	/**
-	 * Transformer la requete pour SQLite
-	 *
-	 * Enlève les textes, transforme la requête pour quelle soit
-	 * bien interprétée par SQLite, puis remet les textes
-	 * la fonction affecte `$this->query`
-	 */
-	public function traduire_requete() {
-		//
-		// 1) Protection des textes en les remplacant par des codes
-		//
-		// enlever les 'textes' et initialiser avec
-		[$this->query, $textes] = query_echappe_textes($this->query);
-
-		//
-		// 2) Corrections de la requete
-		//
-		// Correction Create Database
-		// Create Database -> requete ignoree
-		if (str_starts_with((string) $this->query, 'CREATE DATABASE')) {
-			spip_log("Sqlite : requete non executee -> $this->query", 'sqlite.' . _LOG_AVERTISSEMENT);
-			$this->query = 'SELECT 1';
-		}
-
-		// Correction Insert Ignore
-		// INSERT IGNORE -> insert (tout court et pas 'insert or replace')
-		if (str_starts_with((string) $this->query, 'INSERT IGNORE')) {
-			spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.' . _LOG_DEBUG);
-			$this->query = 'INSERT ' . substr((string) $this->query, '13');
-		}
-
-		// Correction des dates avec INTERVAL
-		// utiliser sql_date_proche() de preference
-		if (str_contains((string) $this->query, 'INTERVAL')) {
-			$this->query = preg_replace_callback(
-				'/DATE_(ADD|SUB)(.*)INTERVAL\s+(\d+)\s+([a-zA-Z]+)\)/U',
-				fn(array $matches): string => $this->_remplacerDateParTime($matches),
-				(string) $this->query
-			);
-		}
-
-		if (str_contains((string) $this->query, 'LEFT(')) {
-			$this->query = str_replace('LEFT(', '_LEFT(', (string) $this->query);
-		}
-
-		if (str_contains((string) $this->query, 'TIMESTAMPDIFF(')) {
-			$this->query = preg_replace('/TIMESTAMPDIFF\(\s*([^,]*)\s*,/Uims', "TIMESTAMPDIFF('\\1',", (string) $this->query);
-		}
-
-
-		// Correction Using
-		// USING (non reconnu en sqlite2)
-		// problematique car la jointure ne se fait pas du coup.
-		if (($this->sqlite_version == 2) && (str_contains((string) $this->query, 'USING'))) {
-			spip_log(
-				"'USING (champ)' n'est pas reconnu en SQLite 2. Utilisez 'ON table1.champ = table2.champ'",
-				'sqlite.' . _LOG_ERREUR
-			);
-			$this->query = preg_replace('/USING\s*\([^\)]*\)/', '', (string) $this->query);
-		}
-
-		// Correction Field
-		// remplace FIELD(table,i,j,k...) par CASE WHEN table=i THEN n ... ELSE 0 END
-		if (str_contains((string) $this->query, 'FIELD')) {
-			$this->query = preg_replace_callback(
-				'/FIELD\s*\(([^\)]*)\)/',
-				fn(array $matches): string => $this->_remplacerFieldParCase($matches),
-				(string) $this->query
-			);
-		}
-
-		// Correction des noms de tables FROM
-		// mettre les bons noms de table dans from, update, insert, replace...
-		if (preg_match('/\s(SET|VALUES|WHERE|DATABASE)\s/iS', (string) $this->query, $regs)) {
-			$suite = strstr((string) $this->query, $regs[0]);
-			$this->query = substr((string) $this->query, 0, -strlen($suite));
-		} else {
-			$suite = '';
-		}
-		$pref = ($this->prefixe) ? $this->prefixe . '_' : '';
-		$this->query = preg_replace('/([,\s])spip_/S', '\1' . $pref, (string) $this->query) . $suite;
-
-		// Correction zero AS x
-		// pg n'aime pas 0+x AS alias, sqlite, dans le meme style,
-		// n'apprecie pas du tout SELECT 0 as x ... ORDER BY x
-		// il dit que x ne doit pas être un integer dans le order by !
-		// on remplace du coup x par vide() dans ce cas uniquement
-		//
-		// apparait dans public/vertebrer.php et dans le plugin menu aussi qui genere aussi ce genre de requete via un {par num #GET{tri_num}}
-		// mais est-ce encore un soucis pour sqlite en 2021 ? (ie commenter le preg_replace marche très bien en sqlite 3.28)
-		// on ne remplace que dans ORDER BY ou GROUP BY
-		if (str_contains($this->query, '0 AS') && preg_match('/\s(ORDER|GROUP) BY\s/i', $this->query, $regs)) {
-			$suite = strstr($this->query, $regs[0]);
-			$this->query = substr($this->query, 0, -strlen($suite));
-			// on cherche les noms des x dans 0 AS x
-			// on remplace dans $suite le nom par vide()
-			preg_match_all('/\b0 AS\s*([^\s,]+)/', $this->query, $matches, PREG_PATTERN_ORDER);
-			foreach ($matches[1] as $m) {
-				if (str_contains($suite, $m)) {
-					$suite = preg_replace(",\b$m\b,", 'VIDE()', $suite);
-				}
-			}
-			$this->query .= $suite;
-		}
-
-		// Correction possible des divisions entieres
-		// Le standard SQL (lequel? ou?) semble indiquer que
-		// a/b=c doit donner c entier si a et b sont entiers 4/3=1.
-		// C'est ce que retournent effectivement SQL Server et SQLite
-		// Ce n'est pas ce qu'applique MySQL qui retourne un reel : 4/3=1.333...
-		//
-		// On peut forcer la conversion en multipliant par 1.0 avant la division
-		// /!\ SQLite 3.5.9 Debian/Ubuntu est victime d'un bug en plus !
-		// cf. https://bugs.launchpad.net/ubuntu/+source/sqlite3/+bug/254228
-		//     http://www.sqlite.org/cvstrac/tktview?tn=3202
-		// (4*1.0/3) n'est pas rendu dans ce cas !
-		# $this->query = str_replace('/','* 1.00 / ',$this->query);
-
-
-		// Correction critere REGEXP, non reconnu en sqlite2
-		if (($this->sqlite_version == 2) && (str_contains($this->query, 'REGEXP'))) {
-			$this->query = preg_replace('/([^\s\(]*)(\s*)REGEXP(\s*)([^\s\)]*)/', 'REGEXP($4, $1)', $this->query);
-		}
-
-		//
-		// 3) Remise en place des textes d'origine
-		//
-		// Correction Antiquotes et echappements
-		// ` => rien
-		if (str_contains($this->query, '`')) {
-			$this->query = str_replace('`', '', $this->query);
-		}
-
-		$this->query = query_reinjecte_textes($this->query, $textes);
-
-		return $this->query;
-	}
-
-	/**
-	 * Callback pour remplacer `DATE_` / `INTERVAL`
-	 * par `DATE ... strtotime`
-	 *
-	 * @param array $matches Captures
-	 * @return string texte de date compris par SQLite
-	 */
-	public function _remplacerDateParTime($matches) {
-		$op = strtoupper($matches[1] == 'ADD') ? '+' : '-';
-
-		return "datetime$matches[2] '$op$matches[3] $matches[4]')";
-	}
-
-	/**
-	 * Callback pour remplacer `FIELD(table,i,j,k...)`
-	 * par `CASE WHEN table=i THEN n ... ELSE 0 END`
-	 *
-	 * @param array $matches Captures
-	 * @return string texte de liste ordonnée compris par SQLite
-	 */
-	public function _remplacerFieldParCase($matches) {
-		$fields = substr((string) $matches[0], 6, -1); // ne recuperer que l'interieur X de field(X)
-		$t = explode(',', $fields);
-		$index = array_shift($t);
-
-		$res = '';
-		$n = 0;
-		foreach ($t as $v) {
-			$n++;
-			$res .= "\nWHEN $index=$v THEN $n";
-		}
-
-		return "CASE $res ELSE 0 END ";
-	}
+    /** Pour les corrections à effectuer sur les requêtes : array(code=>'texte') trouvé
+     *
+     * @var array
+     */
+    public $textes = [];
+
+    /**
+     * Constructeur
+     */
+    public function __construct(
+        /** Requête à préparer */
+        public string $query,
+        /** Prefixe des tables à utiliser */
+        public string $prefixe,
+        /** Version SQLite (2 ou 3) */
+        public string $sqlite_version
+    ) {
+    }
+
+    /**
+     * Transformer la requete pour SQLite
+     *
+     * Enlève les textes, transforme la requête pour quelle soit
+     * bien interprétée par SQLite, puis remet les textes
+     * la fonction affecte `$this->query`
+     */
+    public function traduire_requete() {
+        //
+        // 1) Protection des textes en les remplacant par des codes
+        //
+        // enlever les 'textes' et initialiser avec
+        [$this->query, $textes] = query_echappe_textes($this->query);
+
+        //
+        // 2) Corrections de la requete
+        //
+        // Correction Create Database
+        // Create Database -> requete ignoree
+        if (str_starts_with((string) $this->query, 'CREATE DATABASE')) {
+            spip_log("Sqlite : requete non executee -> $this->query", 'sqlite.' . _LOG_AVERTISSEMENT);
+            $this->query = 'SELECT 1';
+        }
+
+        // Correction Insert Ignore
+        // INSERT IGNORE -> insert (tout court et pas 'insert or replace')
+        if (str_starts_with((string) $this->query, 'INSERT IGNORE')) {
+            spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.' . _LOG_DEBUG);
+            $this->query = 'INSERT ' . substr((string) $this->query, '13');
+        }
+
+        // Correction des dates avec INTERVAL
+        // utiliser sql_date_proche() de preference
+        if (str_contains((string) $this->query, 'INTERVAL')) {
+            $this->query = preg_replace_callback(
+                '/DATE_(ADD|SUB)(.*)INTERVAL\s+(\d+)\s+([a-zA-Z]+)\)/U',
+                fn(array $matches): string => $this->_remplacerDateParTime($matches),
+                (string) $this->query
+            );
+        }
+
+        if (str_contains((string) $this->query, 'LEFT(')) {
+            $this->query = str_replace('LEFT(', '_LEFT(', (string) $this->query);
+        }
+
+        if (str_contains((string) $this->query, 'TIMESTAMPDIFF(')) {
+            $this->query = preg_replace('/TIMESTAMPDIFF\(\s*([^,]*)\s*,/Uims', "TIMESTAMPDIFF('\\1',", (string) $this->query);
+        }
+
+
+        // Correction Using
+        // USING (non reconnu en sqlite2)
+        // problematique car la jointure ne se fait pas du coup.
+        if (($this->sqlite_version == 2) && (str_contains((string) $this->query, 'USING'))) {
+            spip_log(
+                "'USING (champ)' n'est pas reconnu en SQLite 2. Utilisez 'ON table1.champ = table2.champ'",
+                'sqlite.' . _LOG_ERREUR
+            );
+            $this->query = preg_replace('/USING\s*\([^\)]*\)/', '', (string) $this->query);
+        }
+
+        // Correction Field
+        // remplace FIELD(table,i,j,k...) par CASE WHEN table=i THEN n ... ELSE 0 END
+        if (str_contains((string) $this->query, 'FIELD')) {
+            $this->query = preg_replace_callback(
+                '/FIELD\s*\(([^\)]*)\)/',
+                fn(array $matches): string => $this->_remplacerFieldParCase($matches),
+                (string) $this->query
+            );
+        }
+
+        // Correction des noms de tables FROM
+        // mettre les bons noms de table dans from, update, insert, replace...
+        if (preg_match('/\s(SET|VALUES|WHERE|DATABASE)\s/iS', (string) $this->query, $regs)) {
+            $suite = strstr((string) $this->query, $regs[0]);
+            $this->query = substr((string) $this->query, 0, -strlen($suite));
+        } else {
+            $suite = '';
+        }
+        $pref = ($this->prefixe) ? $this->prefixe . '_' : '';
+        $this->query = preg_replace('/([,\s])spip_/S', '\1' . $pref, (string) $this->query) . $suite;
+
+        // Correction zero AS x
+        // pg n'aime pas 0+x AS alias, sqlite, dans le meme style,
+        // n'apprecie pas du tout SELECT 0 as x ... ORDER BY x
+        // il dit que x ne doit pas être un integer dans le order by !
+        // on remplace du coup x par vide() dans ce cas uniquement
+        //
+        // apparait dans public/vertebrer.php et dans le plugin menu aussi qui genere aussi ce genre de requete via un {par num #GET{tri_num}}
+        // mais est-ce encore un soucis pour sqlite en 2021 ? (ie commenter le preg_replace marche très bien en sqlite 3.28)
+        // on ne remplace que dans ORDER BY ou GROUP BY
+        if (str_contains($this->query, '0 AS') && preg_match('/\s(ORDER|GROUP) BY\s/i', $this->query, $regs)) {
+            $suite = strstr($this->query, $regs[0]);
+            $this->query = substr($this->query, 0, -strlen($suite));
+            // on cherche les noms des x dans 0 AS x
+            // on remplace dans $suite le nom par vide()
+            preg_match_all('/\b0 AS\s*([^\s,]+)/', $this->query, $matches, PREG_PATTERN_ORDER);
+            foreach ($matches[1] as $m) {
+                if (str_contains($suite, $m)) {
+                    $suite = preg_replace(",\b$m\b,", 'VIDE()', $suite);
+                }
+            }
+            $this->query .= $suite;
+        }
+
+        // Correction possible des divisions entieres
+        // Le standard SQL (lequel? ou?) semble indiquer que
+        // a/b=c doit donner c entier si a et b sont entiers 4/3=1.
+        // C'est ce que retournent effectivement SQL Server et SQLite
+        // Ce n'est pas ce qu'applique MySQL qui retourne un reel : 4/3=1.333...
+        //
+        // On peut forcer la conversion en multipliant par 1.0 avant la division
+        // /!\ SQLite 3.5.9 Debian/Ubuntu est victime d'un bug en plus !
+        // cf. https://bugs.launchpad.net/ubuntu/+source/sqlite3/+bug/254228
+        //     http://www.sqlite.org/cvstrac/tktview?tn=3202
+        // (4*1.0/3) n'est pas rendu dans ce cas !
+        # $this->query = str_replace('/','* 1.00 / ',$this->query);
+
+
+        // Correction critere REGEXP, non reconnu en sqlite2
+        if (($this->sqlite_version == 2) && (str_contains($this->query, 'REGEXP'))) {
+            $this->query = preg_replace('/([^\s\(]*)(\s*)REGEXP(\s*)([^\s\)]*)/', 'REGEXP($4, $1)', $this->query);
+        }
+
+        //
+        // 3) Remise en place des textes d'origine
+        //
+        // Correction Antiquotes et echappements
+        // ` => rien
+        if (str_contains($this->query, '`')) {
+            $this->query = str_replace('`', '', $this->query);
+        }
+
+        $this->query = query_reinjecte_textes($this->query, $textes);
+
+        return $this->query;
+    }
+
+    /**
+     * Callback pour remplacer `DATE_` / `INTERVAL`
+     * par `DATE ... strtotime`
+     *
+     * @param array $matches Captures
+     * @return string texte de date compris par SQLite
+     */
+    public function _remplacerDateParTime($matches) {
+        $op = strtoupper($matches[1] == 'ADD') ? '+' : '-';
+
+        return "datetime$matches[2] '$op$matches[3] $matches[4]')";
+    }
+
+    /**
+     * Callback pour remplacer `FIELD(table,i,j,k...)`
+     * par `CASE WHEN table=i THEN n ... ELSE 0 END`
+     *
+     * @param array $matches Captures
+     * @return string texte de liste ordonnée compris par SQLite
+     */
+    public function _remplacerFieldParCase($matches) {
+        $fields = substr((string) $matches[0], 6, -1); // ne recuperer que l'interieur X de field(X)
+        $t = explode(',', $fields);
+        $index = array_shift($t);
+
+        $res = '';
+        $n = 0;
+        foreach ($t as $v) {
+            $n++;
+            $res .= "\nWHEN $index=$v THEN $n";
+        }
+
+        return "CASE $res ELSE 0 END ";
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -48,15 +48,15 @@ 
 		// Correction Create Database
 		// Create Database -> requete ignoree
 		if (str_starts_with((string) $this->query, 'CREATE DATABASE')) {
-			spip_log("Sqlite : requete non executee -> $this->query", 'sqlite.' . _LOG_AVERTISSEMENT);
+			spip_log("Sqlite : requete non executee -> $this->query", 'sqlite.'._LOG_AVERTISSEMENT);
 			$this->query = 'SELECT 1';
 		}
 
 		// Correction Insert Ignore
 		// INSERT IGNORE -> insert (tout court et pas 'insert or replace')
 		if (str_starts_with((string) $this->query, 'INSERT IGNORE')) {
-			spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.' . _LOG_DEBUG);
-			$this->query = 'INSERT ' . substr((string) $this->query, '13');
+			spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.'._LOG_DEBUG);
+			$this->query = 'INSERT '.substr((string) $this->query, '13');
 		}
 
 		// Correction des dates avec INTERVAL
@@ -84,7 +84,7 @@ 
 		if (($this->sqlite_version == 2) && (str_contains((string) $this->query, 'USING'))) {
 			spip_log(
 				"'USING (champ)' n'est pas reconnu en SQLite 2. Utilisez 'ON table1.champ = table2.champ'",
-				'sqlite.' . _LOG_ERREUR
+				'sqlite.'._LOG_ERREUR
 			);
 			$this->query = preg_replace('/USING\s*\([^\)]*\)/', '', (string) $this->query);
 		}
@@ -107,8 +107,8 @@ 
 		} else {
 			$suite = '';
 		}
-		$pref = ($this->prefixe) ? $this->prefixe . '_' : '';
-		$this->query = preg_replace('/([,\s])spip_/S', '\1' . $pref, (string) $this->query) . $suite;
+		$pref = ($this->prefixe) ? $this->prefixe.'_' : '';
+		$this->query = preg_replace('/([,\s])spip_/S', '\1'.$pref, (string) $this->query).$suite;
 
 		// Correction zero AS x
 		// pg n'aime pas 0+x AS alias, sqlite, dans le meme style,

ecrire/src/Texte/Collecteur/Liens.php

Indentation +94 added lines, -94 removed lines

@@ -15,98 +15,98 @@
  *    Collecte les raccourcis liens [titre->url] de SPIP
  */
 class Liens extends AbstractCollecteur {
-	protected static string $markPrefix = 'LIEN';
-
-	/**
-	 * La preg pour découper et collecter les modèles
-	 * @var string
-	 */
-	protected string $preg_lien;
-
-	public function __construct(?string $preg = null) {
-
-		// Regexp des raccourcis, aussi utilisee pour la fusion de sauvegarde Spip
-		// Laisser passer des paires de crochets pour la balise multi
-		// mais refuser plus d'imbrications ou de mauvaises imbrications
-		// sinon les crochets ne peuvent plus servir qu'a ce type de raccourci
-		$this->preg_lien = ($preg ?: '/\[([^][]*?([[][^]>-]*[]][^][]*)*)->(>?)([^]]*)\]/msS');
-	}
-
-
-	/**
-	 * Sanitizer une collection d'occurences de liens : il faut sanitizer le href et le texte uniquement
-	 *
-	 * @param array $collection
-	 * @param string $sanitize_callback
-	 * @return array
-	 */
-	protected function sanitizer_collection(array $collection, string $sanitize_callback): array {
-		foreach ($collection as &$lien) {
-			$t = $sanitize_callback($lien['texte']);
-			if ($t !== $lien['texte']) {
-				$lien['raw'] = str_replace($lien['texte'], $t, (string) $lien['raw']);
-				$lien['texte'] = $t;
-			}
-			$href = $sanitize_callback($lien['href']);
-			if ($href !== $lien['href']) {
-				$lien['raw'] = str_replace($lien['href'], $href, (string) $lien['raw']);
-				$lien['href'] = $href;
-			}
-		}
-
-		return $collection;
-	}
-
-	/**
-	 * @param string $texte
-	 * @param array $options
-	 *   bool $collecter_liens
-	 * @return array
-	 */
-	public function collecter(string $texte, array $options = []): array {
-		if (!$texte) {
-			return [];
-		}
-
-		$liens = [];
-		if (str_contains($texte, '->')) {
-			$desechappe_crochets = false;
-			// si il y a un crochet ouvrant échappé ou un crochet fermant échappé, les substituer pour les ignorer
-			if (str_contains($texte, '\[') || str_contains($texte, '\]')) {
-				$texte = str_replace(['\[', '\]'], ["\x1\x5", "\x1\x6"], $texte);
-				$desechappe_crochets = true;
-			}
-
-			// collecter les matchs de la preg
-			$liens = static::collecteur($texte, '->', '[', $this->preg_lien, empty($options['detecter_presence']) ? 0 : 1);
-
-			// si on veut seulement detecter la présence, on peut retourner tel quel
-			if (empty($options['detecter_presence'])) {
-				foreach ($liens as $k => &$lien) {
-					$lien['href'] = end($lien['match']);
-					$lien['texte'] = $lien['match'][1];
-					$lien['ouvrant'] = $lien['match'][3] ?? '';
-
-					// la mise en lien automatique est passee par la a tort !
-					// corrigeons pour eviter d'avoir un <a...> dans un href...
-					if (str_starts_with((string) $lien['href'], '<a')) {
-						$href = extraire_attribut($lien['href'], 'href');
-						// remplacons dans la source qui peut etre reinjectee dans les arguments
-						// d'un modele
-						$lien['raw'] = str_replace($lien['href'], $href, (string) $lien['raw']);
-						// et prenons le href comme la vraie url a linker
-						$lien['href'] = $href;
-					}
-
-					if ($desechappe_crochets && str_contains((string) $lien['raw'], "\x1")) {
-						$lien['raw'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], (string) $lien['raw']);
-						$lien['texte'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], (string) $lien['texte']);
-						$lien['href'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], (string) $lien['href']);
-					}
-				}
-			}
-		}
-
-		return $liens;
-	}
+    protected static string $markPrefix = 'LIEN';
+
+    /**
+     * La preg pour découper et collecter les modèles
+     * @var string
+     */
+    protected string $preg_lien;
+
+    public function __construct(?string $preg = null) {
+
+        // Regexp des raccourcis, aussi utilisee pour la fusion de sauvegarde Spip
+        // Laisser passer des paires de crochets pour la balise multi
+        // mais refuser plus d'imbrications ou de mauvaises imbrications
+        // sinon les crochets ne peuvent plus servir qu'a ce type de raccourci
+        $this->preg_lien = ($preg ?: '/\[([^][]*?([[][^]>-]*[]][^][]*)*)->(>?)([^]]*)\]/msS');
+    }
+
+
+    /**
+     * Sanitizer une collection d'occurences de liens : il faut sanitizer le href et le texte uniquement
+     *
+     * @param array $collection
+     * @param string $sanitize_callback
+     * @return array
+     */
+    protected function sanitizer_collection(array $collection, string $sanitize_callback): array {
+        foreach ($collection as &$lien) {
+            $t = $sanitize_callback($lien['texte']);
+            if ($t !== $lien['texte']) {
+                $lien['raw'] = str_replace($lien['texte'], $t, (string) $lien['raw']);
+                $lien['texte'] = $t;
+            }
+            $href = $sanitize_callback($lien['href']);
+            if ($href !== $lien['href']) {
+                $lien['raw'] = str_replace($lien['href'], $href, (string) $lien['raw']);
+                $lien['href'] = $href;
+            }
+        }
+
+        return $collection;
+    }
+
+    /**
+     * @param string $texte
+     * @param array $options
+     *   bool $collecter_liens
+     * @return array
+     */
+    public function collecter(string $texte, array $options = []): array {
+        if (!$texte) {
+            return [];
+        }
+
+        $liens = [];
+        if (str_contains($texte, '->')) {
+            $desechappe_crochets = false;
+            // si il y a un crochet ouvrant échappé ou un crochet fermant échappé, les substituer pour les ignorer
+            if (str_contains($texte, '\[') || str_contains($texte, '\]')) {
+                $texte = str_replace(['\[', '\]'], ["\x1\x5", "\x1\x6"], $texte);
+                $desechappe_crochets = true;
+            }
+
+            // collecter les matchs de la preg
+            $liens = static::collecteur($texte, '->', '[', $this->preg_lien, empty($options['detecter_presence']) ? 0 : 1);
+
+            // si on veut seulement detecter la présence, on peut retourner tel quel
+            if (empty($options['detecter_presence'])) {
+                foreach ($liens as $k => &$lien) {
+                    $lien['href'] = end($lien['match']);
+                    $lien['texte'] = $lien['match'][1];
+                    $lien['ouvrant'] = $lien['match'][3] ?? '';
+
+                    // la mise en lien automatique est passee par la a tort !
+                    // corrigeons pour eviter d'avoir un <a...> dans un href...
+                    if (str_starts_with((string) $lien['href'], '<a')) {
+                        $href = extraire_attribut($lien['href'], 'href');
+                        // remplacons dans la source qui peut etre reinjectee dans les arguments
+                        // d'un modele
+                        $lien['raw'] = str_replace($lien['href'], $href, (string) $lien['raw']);
+                        // et prenons le href comme la vraie url a linker
+                        $lien['href'] = $href;
+                    }
+
+                    if ($desechappe_crochets && str_contains((string) $lien['raw'], "\x1")) {
+                        $lien['raw'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], (string) $lien['raw']);
+                        $lien['texte'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], (string) $lien['texte']);
+                        $lien['href'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], (string) $lien['href']);
+                    }
+                }
+            }
+        }
+
+        return $liens;
+    }
 }

ecrire/src/Afficher/Minipage/AbstractPage.php

Spacing +24 added lines, -24 removed lines

@@ -65,17 +65,17 @@ 
 
 		$page_title = ($options['page_title'] ?? $GLOBALS['meta']['nom_site']);
 		$doctype = ($options['doctype'] ?? '<!DOCTYPE html>');
-		$doctype = trim((string) $doctype) . "\n";
+		$doctype = trim((string) $doctype)."\n";
 		$charset = ($options['charset'] ?? 'utf-8');
 		$all_inline = ($options['all_inline'] ?? true);
 		$onLoad = ($options['onLoad'] ?? '');
 		if ($onLoad) {
-			$onLoad = ' onload="' . attribut_html($onLoad) . '"';
+			$onLoad = ' onload="'.attribut_html($onLoad).'"';
 		}
 
 		# envoyer le charset
 		if (!headers_sent()) {
-			header('Content-Type: text/html; charset=' . $charset);
+			header('Content-Type: text/html; charset='.$charset);
 		}
 
 		$css = '';
@@ -94,7 +94,7 @@ 
 			. "--minipage-color-theme--s: $s;"
 			. "--minipage-color-theme--l: $l;}";
 		$vars = file_get_contents(find_in_theme('minipage.vars.css'));
-		$inline .= "\n" . trim($vars);
+		$inline .= "\n".trim($vars);
 		if (function_exists('minifier')) {
 			$inline = minifier($inline, 'css');
 		}
@@ -121,31 +121,31 @@ 
 			}
 			$css = "$inline\n$css";
 			if (!empty($options['css'])) {
-				$css .= "\n" . $options['css'];
+				$css .= "\n".$options['css'];
 			}
 			$css = "<style type='text/css'>$css</style>";
 		} else {
 			$css = "<style type='text/css'>$inline</style>";
 			foreach ($files as $name) {
 				$file = timestamp(direction_css($name));
-				$css .= "<link rel='stylesheet' href='" . attribut_html($file) . "' type='text/css' />\n";
+				$css .= "<link rel='stylesheet' href='".attribut_html($file)."' type='text/css' />\n";
 			}
 			if (!empty($options['css'])) {
-				$css .= "<style type='text/css'>" . $options['css'] . '</style>';
+				$css .= "<style type='text/css'>".$options['css'].'</style>';
 			}
 		}
 
-		return $doctype .
-			html_lang_attributes() .
-			"<head>\n" .
-			'<title>' .
-			textebrut($page_title) .
-			"</title>\n" .
-			"<meta name=\"viewport\" content=\"width=device-width\" />\n" .
-			$css .
-			(empty($options['head']) ? '' : $options['head']) .
-			"</head>\n" .
-			"<body{$onLoad} class=\"minipage" . ($this::TYPE ? ' minipage--' . $this::TYPE : '') . "\">\n" .
+		return $doctype.
+			html_lang_attributes().
+			"<head>\n".
+			'<title>'.
+			textebrut($page_title).
+			"</title>\n".
+			"<meta name=\"viewport\" content=\"width=device-width\" />\n".
+			$css.
+			(empty($options['head']) ? '' : $options['head']).
+			"</head>\n".
+			"<body{$onLoad} class=\"minipage".($this::TYPE ? ' minipage--'.$this::TYPE : '')."\">\n".
 			"\t<div class=\"minipage-bloc\">\n";
 	}
 
@@ -156,16 +156,16 @@ 
 	 */
 	protected function ouvreCorps($options = []) {
 		$url_site = url_de_base();
-		$header = "<header>\n" .
-			'<h1><a href="' . attribut_html($url_site) . '">' . interdire_scripts($GLOBALS['meta']['nom_site'] ?? '') . "</a></h1>\n";
+		$header = "<header>\n".
+			'<h1><a href="'.attribut_html($url_site).'">'.interdire_scripts($GLOBALS['meta']['nom_site'] ?? '')."</a></h1>\n";
 
 		$titre = ($options['titre'] ?? '');
 		if ($titre) {
-			$header .= '<h2>' . interdire_scripts($titre) . '</h2>';
+			$header .= '<h2>'.interdire_scripts($titre).'</h2>';
 		}
 		$header .= '</header>';
 
-		return $header . "<div class='corps'>\n";
+		return $header."<div class='corps'>\n";
 	}
 
 	/**
@@ -179,13 +179,13 @@ 
 		if (isset($options['footer'])) {
 			$footer = $options['footer'];
 		} else {
-			$footer = '<a href="' . attribut_html($url_site) . '">' . _T('retour') . "</a>\n";
+			$footer = '<a href="'.attribut_html($url_site).'">'._T('retour')."</a>\n";
 		}
 		if (!empty($footer)) {
 			$footer = "<footer>\n{$footer}</footer>";
 		}
 
-		return "</div>\n" . $footer;
+		return "</div>\n".$footer;
 	}
 
 

Indentation +236 added lines, -236 removed lines

@@ -15,265 +15,265 @@
  * Présentation des pages simplifiées
  **/
 abstract class AbstractPage {
-	public const TYPE = '';
+    public const TYPE = '';
 
-	public function __construct() {
-		include_fichiers_fonctions();
-		include_spip('inc/headers');
-		include_spip('inc/texte'); //inclue inc/lang et inc/filtres
-		include_spip('inc/filtres_images_mini');
-	}
+    public function __construct() {
+        include_fichiers_fonctions();
+        include_spip('inc/headers');
+        include_spip('inc/texte'); //inclue inc/lang et inc/filtres
+        include_spip('inc/filtres_images_mini');
+    }
 
-	/**
-	 * Retourne le début d'une page HTML minimale
-	 *
-	 * Le contenu de CSS minimales (reset.css, clear.css, minipage.css) est inséré
-	 * dans une balise script inline (compactée si possible)
-	 *
-	 * @param array $options
-	 *   string $lang : forcer la langue utilisateur
-	 *   string $page_title : titre éventuel de la page (nom du site par défaut)
-	 *   string $couleur_fond : pour la couleur dominante de la page (par défaut on reprend le réglage de la page de login)
-	 *   bool $all_inline : inliner les CSS pour envoyer toute la page en 1 hit
-	 *   string $doctype
-	 *   string $charset
-	 *   string $onload
-	 *   array $css_files : ajouter des fichiers css
-	 *   string $css : ajouter du CSS inline
-	 *   string $head : contenu à ajouter à la fin <head> (pour inclusion de JS ou JS inline...)
-	 * @return string
-	 *    Code HTML
-	 *
-	 * @uses html_lang_attributes()
-	 * @uses minifier() si le plugin compresseur est présent
-	 * @uses url_absolue_css()
-	 *
-	 * @uses utiliser_langue_visiteur()
-	 * @uses http_no_cache()
-	 */
-	protected function ouvreBody($options = []) {
-		$h = null;
-		$s = null;
-		$l = null;
-		if (empty($options['lang'])) {
-			// on se limite sur une langue de $GLOBALS['meta']['langues_multilingue'] car on est dans le public
-			utiliser_langue_visiteur($GLOBALS['meta']['langues_multilingue'] ?? null);
-		} else {
-			changer_langue($options['lang']);
-		}
-		http_no_cache();
+    /**
+     * Retourne le début d'une page HTML minimale
+     *
+     * Le contenu de CSS minimales (reset.css, clear.css, minipage.css) est inséré
+     * dans une balise script inline (compactée si possible)
+     *
+     * @param array $options
+     *   string $lang : forcer la langue utilisateur
+     *   string $page_title : titre éventuel de la page (nom du site par défaut)
+     *   string $couleur_fond : pour la couleur dominante de la page (par défaut on reprend le réglage de la page de login)
+     *   bool $all_inline : inliner les CSS pour envoyer toute la page en 1 hit
+     *   string $doctype
+     *   string $charset
+     *   string $onload
+     *   array $css_files : ajouter des fichiers css
+     *   string $css : ajouter du CSS inline
+     *   string $head : contenu à ajouter à la fin <head> (pour inclusion de JS ou JS inline...)
+     * @return string
+     *    Code HTML
+     *
+     * @uses html_lang_attributes()
+     * @uses minifier() si le plugin compresseur est présent
+     * @uses url_absolue_css()
+     *
+     * @uses utiliser_langue_visiteur()
+     * @uses http_no_cache()
+     */
+    protected function ouvreBody($options = []) {
+        $h = null;
+        $s = null;
+        $l = null;
+        if (empty($options['lang'])) {
+            // on se limite sur une langue de $GLOBALS['meta']['langues_multilingue'] car on est dans le public
+            utiliser_langue_visiteur($GLOBALS['meta']['langues_multilingue'] ?? null);
+        } else {
+            changer_langue($options['lang']);
+        }
+        http_no_cache();
 
-		$page_title = ($options['page_title'] ?? $GLOBALS['meta']['nom_site']);
-		$doctype = ($options['doctype'] ?? '<!DOCTYPE html>');
-		$doctype = trim((string) $doctype) . "\n";
-		$charset = ($options['charset'] ?? 'utf-8');
-		$all_inline = ($options['all_inline'] ?? true);
-		$onLoad = ($options['onLoad'] ?? '');
-		if ($onLoad) {
-			$onLoad = ' onload="' . attribut_html($onLoad) . '"';
-		}
+        $page_title = ($options['page_title'] ?? $GLOBALS['meta']['nom_site']);
+        $doctype = ($options['doctype'] ?? '<!DOCTYPE html>');
+        $doctype = trim((string) $doctype) . "\n";
+        $charset = ($options['charset'] ?? 'utf-8');
+        $all_inline = ($options['all_inline'] ?? true);
+        $onLoad = ($options['onLoad'] ?? '');
+        if ($onLoad) {
+            $onLoad = ' onload="' . attribut_html($onLoad) . '"';
+        }
 
-		# envoyer le charset
-		if (!headers_sent()) {
-			header('Content-Type: text/html; charset=' . $charset);
-		}
+        # envoyer le charset
+        if (!headers_sent()) {
+            header('Content-Type: text/html; charset=' . $charset);
+        }
 
-		$css = '';
+        $css = '';
 
-		if (function_exists('couleur_hex_to_hsl')) {
-			$couleur_fond = empty($options['couleur_fond'])
-				? lire_config('couleur_login', '#db1762')
-				: $options['couleur_fond'];
-			$h = couleur_hex_to_hsl($couleur_fond, 'h');
-			$s = couleur_hex_to_hsl($couleur_fond, 's');
-			$l = couleur_hex_to_hsl($couleur_fond, 'l');
-		}
+        if (function_exists('couleur_hex_to_hsl')) {
+            $couleur_fond = empty($options['couleur_fond'])
+                ? lire_config('couleur_login', '#db1762')
+                : $options['couleur_fond'];
+            $h = couleur_hex_to_hsl($couleur_fond, 'h');
+            $s = couleur_hex_to_hsl($couleur_fond, 's');
+            $l = couleur_hex_to_hsl($couleur_fond, 'l');
+        }
 
-		$inline = ':root {'
-			. "--minipage-color-theme--h: $h;"
-			. "--minipage-color-theme--s: $s;"
-			. "--minipage-color-theme--l: $l;}";
-		$vars = file_get_contents(find_in_theme('minipage.vars.css'));
-		$inline .= "\n" . trim($vars);
-		if (function_exists('minifier')) {
-			$inline = minifier($inline, 'css');
-		}
-		$files = [
-			find_in_theme('reset.css'),
-			find_in_theme('clear.css'),
-			find_in_theme('minipage.css'),
-		];
-		if (!empty($options['css_files'])) {
-			foreach ($options['css_files'] as $css_file) {
-				$files[] = $css_file;
-			}
-		}
-		if ($all_inline) {
-			// inliner les CSS (optimisation de la page minipage qui passe en un seul hit a la demande)
-			foreach ($files as $name) {
-				$file = direction_css($name);
-				if (function_exists('minifier')) {
-					$file = minifier($file);
-				} else {
-					$file = url_absolue_css($file); // precaution
-				}
-				$css .= file_get_contents($file);
-			}
-			$css = "$inline\n$css";
-			if (!empty($options['css'])) {
-				$css .= "\n" . $options['css'];
-			}
-			$css = "<style type='text/css'>$css</style>";
-		} else {
-			$css = "<style type='text/css'>$inline</style>";
-			foreach ($files as $name) {
-				$file = timestamp(direction_css($name));
-				$css .= "<link rel='stylesheet' href='" . attribut_html($file) . "' type='text/css' />\n";
-			}
-			if (!empty($options['css'])) {
-				$css .= "<style type='text/css'>" . $options['css'] . '</style>';
-			}
-		}
+        $inline = ':root {'
+            . "--minipage-color-theme--h: $h;"
+            . "--minipage-color-theme--s: $s;"
+            . "--minipage-color-theme--l: $l;}";
+        $vars = file_get_contents(find_in_theme('minipage.vars.css'));
+        $inline .= "\n" . trim($vars);
+        if (function_exists('minifier')) {
+            $inline = minifier($inline, 'css');
+        }
+        $files = [
+            find_in_theme('reset.css'),
+            find_in_theme('clear.css'),
+            find_in_theme('minipage.css'),
+        ];
+        if (!empty($options['css_files'])) {
+            foreach ($options['css_files'] as $css_file) {
+                $files[] = $css_file;
+            }
+        }
+        if ($all_inline) {
+            // inliner les CSS (optimisation de la page minipage qui passe en un seul hit a la demande)
+            foreach ($files as $name) {
+                $file = direction_css($name);
+                if (function_exists('minifier')) {
+                    $file = minifier($file);
+                } else {
+                    $file = url_absolue_css($file); // precaution
+                }
+                $css .= file_get_contents($file);
+            }
+            $css = "$inline\n$css";
+            if (!empty($options['css'])) {
+                $css .= "\n" . $options['css'];
+            }
+            $css = "<style type='text/css'>$css</style>";
+        } else {
+            $css = "<style type='text/css'>$inline</style>";
+            foreach ($files as $name) {
+                $file = timestamp(direction_css($name));
+                $css .= "<link rel='stylesheet' href='" . attribut_html($file) . "' type='text/css' />\n";
+            }
+            if (!empty($options['css'])) {
+                $css .= "<style type='text/css'>" . $options['css'] . '</style>';
+            }
+        }
 
-		return $doctype .
-			html_lang_attributes() .
-			"<head>\n" .
-			'<title>' .
-			textebrut($page_title) .
-			"</title>\n" .
-			"<meta name=\"viewport\" content=\"width=device-width\" />\n" .
-			$css .
-			(empty($options['head']) ? '' : $options['head']) .
-			"</head>\n" .
-			"<body{$onLoad} class=\"minipage" . ($this::TYPE ? ' minipage--' . $this::TYPE : '') . "\">\n" .
-			"\t<div class=\"minipage-bloc\">\n";
-	}
+        return $doctype .
+            html_lang_attributes() .
+            "<head>\n" .
+            '<title>' .
+            textebrut($page_title) .
+            "</title>\n" .
+            "<meta name=\"viewport\" content=\"width=device-width\" />\n" .
+            $css .
+            (empty($options['head']) ? '' : $options['head']) .
+            "</head>\n" .
+            "<body{$onLoad} class=\"minipage" . ($this::TYPE ? ' minipage--' . $this::TYPE : '') . "\">\n" .
+            "\t<div class=\"minipage-bloc\">\n";
+    }
 
-	/**
-	 * Ouvre le corps : affiche le header avec un éventuel titre + ouvre le div corps
-	 * @param array options
-	 * @return string
-	 */
-	protected function ouvreCorps($options = []) {
-		$url_site = url_de_base();
-		$header = "<header>\n" .
-			'<h1><a href="' . attribut_html($url_site) . '">' . interdire_scripts($GLOBALS['meta']['nom_site'] ?? '') . "</a></h1>\n";
+    /**
+     * Ouvre le corps : affiche le header avec un éventuel titre + ouvre le div corps
+     * @param array options
+     * @return string
+     */
+    protected function ouvreCorps($options = []) {
+        $url_site = url_de_base();
+        $header = "<header>\n" .
+            '<h1><a href="' . attribut_html($url_site) . '">' . interdire_scripts($GLOBALS['meta']['nom_site'] ?? '') . "</a></h1>\n";
 
-		$titre = ($options['titre'] ?? '');
-		if ($titre) {
-			$header .= '<h2>' . interdire_scripts($titre) . '</h2>';
-		}
-		$header .= '</header>';
+        $titre = ($options['titre'] ?? '');
+        if ($titre) {
+            $header .= '<h2>' . interdire_scripts($titre) . '</h2>';
+        }
+        $header .= '</header>';
 
-		return $header . "<div class='corps'>\n";
-	}
+        return $header . "<div class='corps'>\n";
+    }
 
-	/**
-	 * Ferme le corps : affiche le footer par défaut ou custom et ferme le div corps
-	 * @param array $options
-	 * @return string
-	 */
-	protected function fermeCorps($options = []) {
-		$url_site = url_de_base();
+    /**
+     * Ferme le corps : affiche le footer par défaut ou custom et ferme le div corps
+     * @param array $options
+     * @return string
+     */
+    protected function fermeCorps($options = []) {
+        $url_site = url_de_base();
 
-		if (isset($options['footer'])) {
-			$footer = $options['footer'];
-		} else {
-			$footer = '<a href="' . attribut_html($url_site) . '">' . _T('retour') . "</a>\n";
-		}
-		if (!empty($footer)) {
-			$footer = "<footer>\n{$footer}</footer>";
-		}
+        if (isset($options['footer'])) {
+            $footer = $options['footer'];
+        } else {
+            $footer = '<a href="' . attribut_html($url_site) . '">' . _T('retour') . "</a>\n";
+        }
+        if (!empty($footer)) {
+            $footer = "<footer>\n{$footer}</footer>";
+        }
 
-		return "</div>\n" . $footer;
-	}
+        return "</div>\n" . $footer;
+    }
 
 
-	/**
-	 * Retourne la fin d'une page HTML minimale
-	 *
-	 * @return string Code HTML
-	 */
-	protected function fermeBody() {
-		return "\n\t</div>\n</body>\n</html>";
-	}
+    /**
+     * Retourne la fin d'une page HTML minimale
+     *
+     * @return string Code HTML
+     */
+    protected function fermeBody() {
+        return "\n\t</div>\n</body>\n</html>";
+    }
 
 
-	/**
-	 * Retourne une page HTML contenant, dans une présentation minimale,
-	 * le contenu transmis dans `$corps`.
-	 *
-	 * Appelée pour afficher un message ou une demande de confirmation simple et rapide
-	 *
-	 * @param string $corps
-	 *   Corps de la page
-	 * @param array $options
-	 * @return string
-	 *   HTML de la page
-	 * @see  ouvreBody()
-	 * @see  ouvreCorps()
-	 *   string $titre : Titre à l'affichage (différent de $page_title)
-	 *   int $status : status de la page
-	 *   string $footer : pied de la box en remplacement du bouton retour par défaut
-	 * @uses ouvreBody()
-	 * @uses ouvreCorps()
-	 * @uses fermeCorps()
-	 * @uses fermeBody()
-	 *
-	 */
-	public function page($corps, $options = []) {
+    /**
+     * Retourne une page HTML contenant, dans une présentation minimale,
+     * le contenu transmis dans `$corps`.
+     *
+     * Appelée pour afficher un message ou une demande de confirmation simple et rapide
+     *
+     * @param string $corps
+     *   Corps de la page
+     * @param array $options
+     * @return string
+     *   HTML de la page
+     * @see  ouvreBody()
+     * @see  ouvreCorps()
+     *   string $titre : Titre à l'affichage (différent de $page_title)
+     *   int $status : status de la page
+     *   string $footer : pied de la box en remplacement du bouton retour par défaut
+     * @uses ouvreBody()
+     * @uses ouvreCorps()
+     * @uses fermeCorps()
+     * @uses fermeBody()
+     *
+     */
+    public function page($corps, $options = []) {
 
-		// par securite
-		if (!defined('_AJAX')) {
-			define('_AJAX', false);
-		}
+        // par securite
+        if (!defined('_AJAX')) {
+            define('_AJAX', false);
+        }
 
-		$status = ((int) ($options['status'] ?? 200)) ?: 200;
+        $status = ((int) ($options['status'] ?? 200)) ?: 200;
 
-		http_response_code($status);
+        http_response_code($status);
 
-		$html = $this->ouvreBody($options)
-			. $this->ouvreCorps($options)
-			. $corps
-			. $this->fermeCorps($options)
-			. $this->fermeBody();
+        $html = $this->ouvreBody($options)
+            . $this->ouvreCorps($options)
+            . $corps
+            . $this->fermeCorps($options)
+            . $this->fermeBody();
 
-		if (
-			$GLOBALS['profondeur_url'] >= (_DIR_RESTREINT ? 1 : 2)
-			&& empty($options['all_inline'])
-		) {
-			define('_SET_HTML_BASE', true);
-			include_spip('public/assembler');
-			$GLOBALS['html'] = true;
-			page_base_href($html);
-		}
-		return $html;
-	}
+        if (
+            $GLOBALS['profondeur_url'] >= (_DIR_RESTREINT ? 1 : 2)
+            && empty($options['all_inline'])
+        ) {
+            define('_SET_HTML_BASE', true);
+            include_spip('public/assembler');
+            $GLOBALS['html'] = true;
+            page_base_href($html);
+        }
+        return $html;
+    }
 
-	/**
-	 * Fonction helper pour les erreurs
-	 * @param ?string $message_erreur
-	 * @param array $options
-	 * @see page()
-	 * @return string
-	 *
-	 */
-	public function pageErreur($message_erreur = null, $options = []) {
+    /**
+     * Fonction helper pour les erreurs
+     * @param ?string $message_erreur
+     * @param array $options
+     * @see page()
+     * @return string
+     *
+     */
+    public function pageErreur($message_erreur = null, $options = []) {
 
-		if (empty($message_erreur)) {
-			if (empty($options['lang'])) {
-				utiliser_langue_visiteur();
-			} else {
-				changer_langue($options['lang']);
-			}
-			$message_erreur = _T('info_acces_interdit');
-		}
-		$corps = "<div class='msg-alert error'>"
-			. $message_erreur
-			. '</div>';
-		if (empty($options['status'])) {
-			$options['status'] = 403;
-		}
-		return $this->page($corps, $options);
-	}
+        if (empty($message_erreur)) {
+            if (empty($options['lang'])) {
+                utiliser_langue_visiteur();
+            } else {
+                changer_langue($options['lang']);
+            }
+            $message_erreur = _T('info_acces_interdit');
+        }
+        $corps = "<div class='msg-alert error'>"
+            . $message_erreur
+            . '</div>';
+        if (empty($options['status'])) {
+            $options['status'] = 403;
+        }
+        return $this->page($corps, $options);
+    }
 }

ecrire/src/Compilateur/Iterateur/Decorator.php

Spacing +10 added lines, -10 removed lines

@@ -135,7 +135,7 @@ 
 				return $this->iter->{$nom}();
 			} catch (Exception) {
 				// #GETCHILDREN sur un fichier de DirectoryIterator ...
-				spip_log("Methode {$nom} en echec sur " . $this->iter::class);
+				spip_log("Methode {$nom} en echec sur ".$this->iter::class);
 				spip_log("Cela peut être normal : retour d'une ligne de resultat ne pouvant pas calculer cette methode");
 
 				return '';
@@ -356,7 +356,7 @@ 
 			return null;
 		}
 
-		return '(' . implode(") {$operateur} (", $filtres_string) . ')';
+		return '('.implode(") {$operateur} (", $filtres_string).')';
 	}
 
 	/**
@@ -439,7 +439,7 @@ 
 			return $this->composer_filtre($v[1], $v[0], $v[2]);
 		}
 
-		return null;  // sera ignore
+		return null; // sera ignore
 	}
 
 	/**
@@ -468,28 +468,28 @@ 
 		// if (!in_array($cle, array('cle', 'valeur')))
 		//	return;
 
-		$a = '$this->get_select(\'' . $cle . '\')';
+		$a = '$this->get_select(\''.$cle.'\')';
 
 		$filtre = '';
 
 		if ('REGEXP' == $op) {
-			$filtre = 'filtrer("match", ' . $a . ', ' . str_replace('\"', '"', (string) $valeur) . ')';
+			$filtre = 'filtrer("match", '.$a.', '.str_replace('\"', '"', (string) $valeur).')';
 			$op = '';
 		} else {
 			if ('LIKE' == $op) {
 				$valeur = str_replace(['\"', '_', '%'], ['"', '.', '.*'], preg_quote((string) $valeur));
-				$filtre = 'filtrer("match", ' . $a . ', ' . $valeur . ')';
+				$filtre = 'filtrer("match", '.$a.', '.$valeur.')';
 				$op = '';
 			} else {
 				if ('=' == $op) {
 					$op = '==';
 				} else {
 					if ('IN' == $op) {
-						$filtre = 'in_array(' . $a . ', array' . $valeur . ')';
+						$filtre = 'in_array('.$a.', array'.$valeur.')';
 						$op = '';
 					} else {
 						if (!in_array($op, ['<', '<=', '>', '>='])) {
-							spip_log('operateur non reconnu ' . $op); // [todo] mettre une erreur de squelette
+							spip_log('operateur non reconnu '.$op); // [todo] mettre une erreur de squelette
 							$op = '';
 						}
 					}
@@ -498,7 +498,7 @@ 
 		}
 
 		if ($op) {
-			$filtre = $a . $op . str_replace('\"', '"', (string) $valeur);
+			$filtre = $a.$op.str_replace('\"', '"', (string) $valeur);
 		}
 
 		if ($not) {
@@ -547,7 +547,7 @@ 
 		// Creer la fonction de filtrage sur $this
 		if ($this->filtre) {
 			if ($filtres = $this->assembler_filtres($this->filtre)) {
-				$filtres = 'return ' . $filtres . ';';
+				$filtres = 'return '.$filtres.';';
 				$this->func_filtre = fn () => eval($filtres);
 			} else {
 				$this->func_filtre = null;

Indentation +554 added lines, -554 removed lines

@@ -8,566 +8,566 @@
 
 class Decorator extends FilterIterator
 {
-	/**
-	 * Conditions de filtrage
-	 * ie criteres de selection.
-	 *
-	 * @var array
-	 */
-	protected $filtre = [];
-
-	/**
-	 * Fonction de filtrage compilee a partir des criteres de filtre.
-	 *
-	 * @var string
-	 */
-	protected $func_filtre;
-
-	/**
-	 * Critere {offset, limit}.
-	 *
-	 * @var int
-	 * @var int
-	 */
-	protected $offset;
-	protected $limit;
-
-	/**
-	 * nombre d'elements recuperes depuis la position 0,
-	 * en tenant compte des filtres.
-	 *
-	 * @var int
-	 */
-	protected $fetched = 0;
-
-	/**
-	 * Y a t'il une erreur ?
-	 *
-	 * @var bool
-	 */
-	protected $err = false;
-
-	// Extension SPIP des iterateurs PHP
-	/**
-	 * type de l'iterateur.
-	 *
-	 * @var string
-	 */
-	protected $type;
-
-	/**
-	 * position courante de l'iterateur.
-	 *
-	 * @var int
-	 */
-	protected $pos = 0;
-
-	/**
-	 * nombre total resultats dans l'iterateur.
-	 *
-	 * @var int
-	 */
-	protected $total;
-
-	/**
-	 * nombre maximal de recherche pour $total
-	 * si l'iterateur n'implemente pas de fonction specifique.
-	 */
-	protected $max = 100000;
-
-	/**
-	 * Liste des champs a inserer dans les $row
-	 * retournes par ->fetch().
-	 */
-	protected $select = [];
-	private readonly Iterator $iter;
-
-	public function __construct(
-		Iterator $iter,
-		/** Parametres de l'iterateur */
-		protected array $command,
-		/** Infos du compilateur */
-		protected array $info
-	) {
-		parent::__construct($iter);
-		parent::rewind(); // remettre a la premiere position (bug? connu de FilterIterator)
-
-		// recuperer l'iterateur transmis
-		$this->iter = $this->getInnerIterator();
-
-		// chercher la liste des champs a retourner par
-		// fetch si l'objet ne les calcule pas tout seul
-		if (!method_exists($this->iter, 'fetch')) {
-			$this->calculer_select();
-			$this->calculer_filtres();
-		}
-
-		// emptyIterator critere {si} faux n'a pas d'erreur !
-		if (property_exists($this->iter, 'err') && $this->iter->err !== null) {
-			$this->err = $this->iter->err;
-		}
-
-		// 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();
-	}
-
-	/**
-	 * Drapeau a activer en cas d'echec
-	 * (select SQL errone, non chargement des DATA, etc).
-	 */
-	public function err() {
-		if (method_exists($this->iter, 'err')) {
-			return $this->iter->err();
-		}
-		if (property_exists($this->iter, 'err')) {
-			return $this->iter->err;
-		}
-
-		return false;
-	}
-
-	// recuperer la valeur d'une balise #X
-	// en fonction des methodes
-	// et proprietes disponibles
-	public function get_select($nom) {
-		if (is_object($this->iter) && method_exists($this->iter, $nom)) {
-			try {
-				return $this->iter->{$nom}();
-			} catch (Exception) {
-				// #GETCHILDREN sur un fichier de DirectoryIterator ...
-				spip_log("Methode {$nom} en echec sur " . $this->iter::class);
-				spip_log("Cela peut être normal : retour d'une ligne de resultat ne pouvant pas calculer cette methode");
-
-				return '';
-			}
-		}
-		/*
+    /**
+     * Conditions de filtrage
+     * ie criteres de selection.
+     *
+     * @var array
+     */
+    protected $filtre = [];
+
+    /**
+     * Fonction de filtrage compilee a partir des criteres de filtre.
+     *
+     * @var string
+     */
+    protected $func_filtre;
+
+    /**
+     * Critere {offset, limit}.
+     *
+     * @var int
+     * @var int
+     */
+    protected $offset;
+    protected $limit;
+
+    /**
+     * nombre d'elements recuperes depuis la position 0,
+     * en tenant compte des filtres.
+     *
+     * @var int
+     */
+    protected $fetched = 0;
+
+    /**
+     * Y a t'il une erreur ?
+     *
+     * @var bool
+     */
+    protected $err = false;
+
+    // Extension SPIP des iterateurs PHP
+    /**
+     * type de l'iterateur.
+     *
+     * @var string
+     */
+    protected $type;
+
+    /**
+     * position courante de l'iterateur.
+     *
+     * @var int
+     */
+    protected $pos = 0;
+
+    /**
+     * nombre total resultats dans l'iterateur.
+     *
+     * @var int
+     */
+    protected $total;
+
+    /**
+     * nombre maximal de recherche pour $total
+     * si l'iterateur n'implemente pas de fonction specifique.
+     */
+    protected $max = 100000;
+
+    /**
+     * Liste des champs a inserer dans les $row
+     * retournes par ->fetch().
+     */
+    protected $select = [];
+    private readonly Iterator $iter;
+
+    public function __construct(
+        Iterator $iter,
+        /** Parametres de l'iterateur */
+        protected array $command,
+        /** Infos du compilateur */
+        protected array $info
+    ) {
+        parent::__construct($iter);
+        parent::rewind(); // remettre a la premiere position (bug? connu de FilterIterator)
+
+        // recuperer l'iterateur transmis
+        $this->iter = $this->getInnerIterator();
+
+        // chercher la liste des champs a retourner par
+        // fetch si l'objet ne les calcule pas tout seul
+        if (!method_exists($this->iter, 'fetch')) {
+            $this->calculer_select();
+            $this->calculer_filtres();
+        }
+
+        // emptyIterator critere {si} faux n'a pas d'erreur !
+        if (property_exists($this->iter, 'err') && $this->iter->err !== null) {
+            $this->err = $this->iter->err;
+        }
+
+        // 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();
+    }
+
+    /**
+     * Drapeau a activer en cas d'echec
+     * (select SQL errone, non chargement des DATA, etc).
+     */
+    public function err() {
+        if (method_exists($this->iter, 'err')) {
+            return $this->iter->err();
+        }
+        if (property_exists($this->iter, 'err')) {
+            return $this->iter->err;
+        }
+
+        return false;
+    }
+
+    // recuperer la valeur d'une balise #X
+    // en fonction des methodes
+    // et proprietes disponibles
+    public function get_select($nom) {
+        if (is_object($this->iter) && method_exists($this->iter, $nom)) {
+            try {
+                return $this->iter->{$nom}();
+            } catch (Exception) {
+                // #GETCHILDREN sur un fichier de DirectoryIterator ...
+                spip_log("Methode {$nom} en echec sur " . $this->iter::class);
+                spip_log("Cela peut être normal : retour d'une ligne de resultat ne pouvant pas calculer cette methode");
+
+                return '';
+            }
+        }
+        /*
 		if (property_exists($this->iter, $nom)) {
 			return $this->iter->$nom;
 		}*/
-		// cle et valeur par defaut
-		// ICI PLANTAGE SI ON NE CONTROLE PAS $nom
-		if (
-			in_array($nom, ['cle', 'valeur'])
-			&& method_exists($this, $nom)
-		) {
-			return $this->{$nom}();
-		}
-
-		// Par defaut chercher en xpath dans la valeur()
-		return table_valeur($this->valeur(), $nom, null);
-	}
-
-	public function next(): void {
-		++$this->pos;
-		parent::next();
-	}
-
-	/**
-	 * revient au depart.
-	 */
-	public function rewind(): void {
-		$this->pos = 0;
-		$this->fetched = 0;
-		parent::rewind();
-	}
-
-	/**
-	 * aller a la position absolue n,
-	 * comptee depuis le debut.
-	 *
-	 * @param int    $n
-	 *                         absolute pos
-	 * @param string $continue
-	 *                         param for sql_ api
-	 *
-	 * @return bool
-	 *              success or fail if not implemented
-	 */
-	public function seek($n = 0, $continue = null) {
-		if ($this->func_filtre || !method_exists($this->iter, 'seek') || !$this->iter->seek($n)) {
-			$this->seek_loop($n);
-		}
-		$this->pos = $n;
-		$this->fetched = $n;
-
-		return true;
-	}
-
-	/**
-	 * Avancer de $saut pas.
-	 *
-	 * @param int $saut
-	 * @param int|null $max
-	 *
-	 * @return int
-	 */
-	public function skip($saut, $max = null) {
-		// pas de saut en arriere autorise pour cette fonction
-		if (($saut = (int) $saut) <= 0) {
-			return $this->pos;
-		}
-		$seek = $this->pos + $saut;
-		// si le saut fait depasser le maxi, on libere la resource
-		// et on sort
-		if (is_null($max)) {
-			$max = $this->count();
-		}
-
-		if ($seek >= $max || $seek >= $this->count()) {
-			// sortie plus rapide que de faire next() jusqu'a la fin !
-			$this->free();
-
-			return $max;
-		}
-
-		$this->seek($seek);
-
-		return $this->pos;
-	}
-
-	/**
-	 * Renvoyer un tableau des donnees correspondantes
-	 * a la position courante de l'iterateur
-	 * en controlant si on respecte le filtre
-	 * Appliquer aussi le critere {offset,limit}.
-	 *
-	 * @return array|bool
-	 */
-	public function fetch() {
-		if (method_exists($this->iter, 'fetch')) {
-			return $this->iter->fetch();
-		}
-		while (
-				$this->valid()
-				&& (!$this->accept() || $this->offset !== null && $this->fetched++ < $this->offset)
-		) {
-			$this->next();
-		}
-
-		if (!$this->valid()) {
-			return false;
-		}
-
-		if (
-				$this->limit !== null
-				&& $this->fetched > $this->offset + $this->limit
-		) {
-			return false;
-		}
-
-		$r = [];
-		foreach ($this->select as $nom) {
-			$r[$nom] = $this->get_select($nom);
-		}
-		$this->next();
-
-		return $r;
-	}
-
-	// retourner la cle pour #CLE
-	public function cle() {
-		return $this->key();
-	}
-
-	// retourner la valeur pour #VALEUR
-	public function valeur() {
-		return $this->current();
-	}
-
-	/**
-	 * Accepte-t-on l'entree courante lue ?
-	 * On execute les filtres pour le savoir.
-	 */
-	public function accept(): bool {
-		if ($f = $this->func_filtre) {
-			return $f();
-		}
-
-		return true;
-	}
-
-	/**
-	 * liberer la ressource.
-	 *
-	 * @return bool
-	 */
-	public function free() {
-		if (method_exists($this->iter, 'free')) {
-			$this->iter->free();
-		}
-		$this->pos = $this->total = 0;
-
-		return true;
-	}
-
-	/**
-	 * Compter le nombre total de resultats
-	 * pour #TOTAL_BOUCLE.
-	 *
-	 * @return int
-	 */
-	public function count() {
-		if (is_null($this->total)) {
-			if (
-				method_exists($this->iter, 'count')
-				&& !$this->func_filtre
-			) {
-				return $this->total = $this->iter->count();
-			}
-			// compter les lignes et rembobiner
-			$total = 0;
-			$pos = $this->pos; // sauver la position
-			$this->rewind();
-			while ($this->fetch() && $total < $this->max) {
-				++$total;
-			}
-			$this->seek($pos);
-			$this->total = $total;
-		}
-
-		return $this->total;
-	}
-
-	/**
-	 * Assembler le tableau de filtres traduits depuis les conditions SQL
-	 * les filtres vides ou null sont ignores.
-	 *
-	 * @param array $filtres
-	 * @param string $operateur
-	 *
-	 * @return null|string
-	 */
-	protected function assembler_filtres($filtres, $operateur = 'AND') {
-		$filtres_string = [];
-		foreach ($filtres as $k => $v) {
-			// si c'est un tableau de OR/AND + 2 sous-filtres, on recurse pour transformer en chaine
-			if (is_array($v) && in_array(reset($v), ['OR', 'AND'])) {
-				$op = array_shift($v);
-				$v = $this->assembler_filtres($v, $op);
-			}
-			if (is_null($v) || !is_string($v) || empty($v)) {
-				continue;
-			}
-			$filtres_string[] = $v;
-		}
-
-		if ($filtres_string === []) {
-			return null;
-		}
-
-		return '(' . implode(") {$operateur} (", $filtres_string) . ')';
-	}
-
-	/**
-	 * Traduire un element du tableau where SQL en un filtre.
-	 *
-	 * @param array|string $v
-	 *
-	 * @return null|array|string
-	 */
-	protected function traduire_condition_sql_en_filtre($v) {
-		if (is_array($v)) {
-			if ((count($v) >= 2) && ('REGEXP' == $v[0]) && ("'.*'" == $v[2])) {
-				return 'true';
-			}
-			if ((count($v) >= 2) && ('LIKE' == $v[0]) && ("'%'" == $v[2])) {
-				return 'true';
-			}
-			$op = $v[0] ?: $v;
-		} else {
-			$op = $v;
-		}
-		if (!$op || 1 == $op || '0=0' == $op) {
-			return 'true';
-		}
-		if ('0=1' === $op) {
-			return 'false';
-		}
-		// traiter {cle IN a,b} ou {valeur !IN a,b}
-		if (preg_match(',^\(([\w/]+)(\s+NOT)?\s+IN\s+(\(.*\))\)$,', (string) $op, $regs)) {
-			return $this->composer_filtre($regs[1], 'IN', $regs[3], $regs[2]);
-		}
-
-		// 3 possibilites : count($v) =
-		// * 1 : {x y} ; on recoit $v[0] = y
-		// * 2 : {x !op y} ; on recoit $v[0] = 'NOT', $v[1] = array() // array du type {x op y}
-		// * 3 : {x op y} ; on recoit $v[0] = 'op', $v[1] = x, $v[2] = y
-
-		// 1 : forcement traite par un critere, on passe
-		if (!$v || !is_array($v) || 1 == count($v)) {
-			return null; // sera ignore
-		}
-		if (2 == count($v) && is_array($v[1])) {
-			return $this->composer_filtre($v[1][1], $v[1][0], $v[1][2], 'NOT');
-		}
-		if (3 == count($v)) {
-			// traiter le OR/AND suivi de 2 valeurs
-			if (in_array($op, ['OR', 'AND'])) {
-				array_shift($v);
-				foreach (array_keys($v) as $k) {
-					$v[$k] = $this->traduire_condition_sql_en_filtre($v[$k]);
-					if (null === $v[$k]) {
-						unset($v[$k]);
-					} elseif ('true' === $v[$k]) {
-						if ('OR' === $op) {
-							return 'true';
-						}
-						if ('AND' === $op) {
-							unset($v[$k]);
-						}
-					} elseif ('false' === $v[$k]) {
-						if ('OR' === $op) {
-							unset($v[$k]);
-						}
-						if ('AND' === $op) {
-							return 'false';
-						}
-					}
-				}
-				if ($v === []) {
-					return null;
-				}
-				if (1 === count($v)) {
-					return reset($v);
-				}
-				array_unshift($v, $op);
-
-				return $v;
-			}
-
-			return $this->composer_filtre($v[1], $v[0], $v[2]);
-		}
-
-		return null;  // sera ignore
-	}
-
-	/**
-	 * Calculer un filtre sur un champ du tableau.
-	 *
-	 * @param string $cle
-	 * @param string $op
-	 * @param string $valeur
-	 * @param bool $not
-	 *
-	 * @return null|string
-	 */
-	protected function composer_filtre($cle, $op, $valeur, $not = false) {
-		if (
-			method_exists($this->iter, 'exception_des_criteres')
-			&& in_array($cle, $this->iter->exception_des_criteres())
-		) {
-			return null;
-		}
-		// TODO: analyser le filtre pour refuser ce qu'on ne sait pas traiter ?
-		// mais c'est normalement deja opere par calculer_critere_infixe()
-		// qui regarde la description 'desc' (en casse reelle d'ailleurs : {isDir=1}
-		// ne sera pas vu si l'on a defini desc['field']['isdir'] pour que #ISDIR soit present.
-		// il faudrait peut etre definir les 2 champs isDir et isdir... a reflechir...
-
-		// if (!in_array($cle, array('cle', 'valeur')))
-		//	return;
-
-		$a = '$this->get_select(\'' . $cle . '\')';
-
-		$filtre = '';
-
-		if ('REGEXP' == $op) {
-			$filtre = 'filtrer("match", ' . $a . ', ' . str_replace('\"', '"', (string) $valeur) . ')';
-			$op = '';
-		} else {
-			if ('LIKE' == $op) {
-				$valeur = str_replace(['\"', '_', '%'], ['"', '.', '.*'], preg_quote((string) $valeur));
-				$filtre = 'filtrer("match", ' . $a . ', ' . $valeur . ')';
-				$op = '';
-			} else {
-				if ('=' == $op) {
-					$op = '==';
-				} else {
-					if ('IN' == $op) {
-						$filtre = 'in_array(' . $a . ', array' . $valeur . ')';
-						$op = '';
-					} else {
-						if (!in_array($op, ['<', '<=', '>', '>='])) {
-							spip_log('operateur non reconnu ' . $op); // [todo] mettre une erreur de squelette
-							$op = '';
-						}
-					}
-				}
-			}
-		}
-
-		if ($op) {
-			$filtre = $a . $op . str_replace('\"', '"', (string) $valeur);
-		}
-
-		if ($not) {
-			$filtre = "!({$filtre})";
-		}
-
-		return $filtre;
-	}
-
-	// calcule les elements a retournes par fetch()
-	// enleve les elements inutiles du select()
-	//
-	private function calculer_select() {
-		if ($select = &$this->command['select']) {
-			foreach ($select as $s) {
-				// /!\ $s = '.nom'
-				if ('.' == $s[0]) {
-					$s = substr((string) $s, 1);
-				}
-				$this->select[] = $s;
-			}
-		}
-	}
-
-	private function calculer_filtres() {
-		// Issu de calculer_select() de public/composer L.519
-		// TODO: externaliser...
-		//
-		// retirer les criteres vides:
-		// {X ?} avec X absent de l'URL
-		// {par #ENV{X}} avec X absent de l'URL
-		// IN sur collection vide (ce dernier devrait pouvoir etre fait a la compil)
-		if ($where = &$this->command['where']) {
-			foreach ($where as $k => $v) {
-				$this->filtre[] = $this->traduire_condition_sql_en_filtre($v);
-			}
-		}
-
-		// critere {2,7}
-		if (isset($this->command['limit']) && $this->command['limit']) {
-			$limit = explode(',', (string) $this->command['limit']);
-			$this->offset = $limit[0];
-			$this->limit = $limit[1];
-		}
-
-		// Creer la fonction de filtrage sur $this
-		if ($this->filtre) {
-			if ($filtres = $this->assembler_filtres($this->filtre)) {
-				$filtres = 'return ' . $filtres . ';';
-				$this->func_filtre = fn () => eval($filtres);
-			} else {
-				$this->func_filtre = null;
-			}
-		}
-	}
-
-	/*
+        // cle et valeur par defaut
+        // ICI PLANTAGE SI ON NE CONTROLE PAS $nom
+        if (
+            in_array($nom, ['cle', 'valeur'])
+            && method_exists($this, $nom)
+        ) {
+            return $this->{$nom}();
+        }
+
+        // Par defaut chercher en xpath dans la valeur()
+        return table_valeur($this->valeur(), $nom, null);
+    }
+
+    public function next(): void {
+        ++$this->pos;
+        parent::next();
+    }
+
+    /**
+     * revient au depart.
+     */
+    public function rewind(): void {
+        $this->pos = 0;
+        $this->fetched = 0;
+        parent::rewind();
+    }
+
+    /**
+     * aller a la position absolue n,
+     * comptee depuis le debut.
+     *
+     * @param int    $n
+     *                         absolute pos
+     * @param string $continue
+     *                         param for sql_ api
+     *
+     * @return bool
+     *              success or fail if not implemented
+     */
+    public function seek($n = 0, $continue = null) {
+        if ($this->func_filtre || !method_exists($this->iter, 'seek') || !$this->iter->seek($n)) {
+            $this->seek_loop($n);
+        }
+        $this->pos = $n;
+        $this->fetched = $n;
+
+        return true;
+    }
+
+    /**
+     * Avancer de $saut pas.
+     *
+     * @param int $saut
+     * @param int|null $max
+     *
+     * @return int
+     */
+    public function skip($saut, $max = null) {
+        // pas de saut en arriere autorise pour cette fonction
+        if (($saut = (int) $saut) <= 0) {
+            return $this->pos;
+        }
+        $seek = $this->pos + $saut;
+        // si le saut fait depasser le maxi, on libere la resource
+        // et on sort
+        if (is_null($max)) {
+            $max = $this->count();
+        }
+
+        if ($seek >= $max || $seek >= $this->count()) {
+            // sortie plus rapide que de faire next() jusqu'a la fin !
+            $this->free();
+
+            return $max;
+        }
+
+        $this->seek($seek);
+
+        return $this->pos;
+    }
+
+    /**
+     * Renvoyer un tableau des donnees correspondantes
+     * a la position courante de l'iterateur
+     * en controlant si on respecte le filtre
+     * Appliquer aussi le critere {offset,limit}.
+     *
+     * @return array|bool
+     */
+    public function fetch() {
+        if (method_exists($this->iter, 'fetch')) {
+            return $this->iter->fetch();
+        }
+        while (
+                $this->valid()
+                && (!$this->accept() || $this->offset !== null && $this->fetched++ < $this->offset)
+        ) {
+            $this->next();
+        }
+
+        if (!$this->valid()) {
+            return false;
+        }
+
+        if (
+                $this->limit !== null
+                && $this->fetched > $this->offset + $this->limit
+        ) {
+            return false;
+        }
+
+        $r = [];
+        foreach ($this->select as $nom) {
+            $r[$nom] = $this->get_select($nom);
+        }
+        $this->next();
+
+        return $r;
+    }
+
+    // retourner la cle pour #CLE
+    public function cle() {
+        return $this->key();
+    }
+
+    // retourner la valeur pour #VALEUR
+    public function valeur() {
+        return $this->current();
+    }
+
+    /**
+     * Accepte-t-on l'entree courante lue ?
+     * On execute les filtres pour le savoir.
+     */
+    public function accept(): bool {
+        if ($f = $this->func_filtre) {
+            return $f();
+        }
+
+        return true;
+    }
+
+    /**
+     * liberer la ressource.
+     *
+     * @return bool
+     */
+    public function free() {
+        if (method_exists($this->iter, 'free')) {
+            $this->iter->free();
+        }
+        $this->pos = $this->total = 0;
+
+        return true;
+    }
+
+    /**
+     * Compter le nombre total de resultats
+     * pour #TOTAL_BOUCLE.
+     *
+     * @return int
+     */
+    public function count() {
+        if (is_null($this->total)) {
+            if (
+                method_exists($this->iter, 'count')
+                && !$this->func_filtre
+            ) {
+                return $this->total = $this->iter->count();
+            }
+            // compter les lignes et rembobiner
+            $total = 0;
+            $pos = $this->pos; // sauver la position
+            $this->rewind();
+            while ($this->fetch() && $total < $this->max) {
+                ++$total;
+            }
+            $this->seek($pos);
+            $this->total = $total;
+        }
+
+        return $this->total;
+    }
+
+    /**
+     * Assembler le tableau de filtres traduits depuis les conditions SQL
+     * les filtres vides ou null sont ignores.
+     *
+     * @param array $filtres
+     * @param string $operateur
+     *
+     * @return null|string
+     */
+    protected function assembler_filtres($filtres, $operateur = 'AND') {
+        $filtres_string = [];
+        foreach ($filtres as $k => $v) {
+            // si c'est un tableau de OR/AND + 2 sous-filtres, on recurse pour transformer en chaine
+            if (is_array($v) && in_array(reset($v), ['OR', 'AND'])) {
+                $op = array_shift($v);
+                $v = $this->assembler_filtres($v, $op);
+            }
+            if (is_null($v) || !is_string($v) || empty($v)) {
+                continue;
+            }
+            $filtres_string[] = $v;
+        }
+
+        if ($filtres_string === []) {
+            return null;
+        }
+
+        return '(' . implode(") {$operateur} (", $filtres_string) . ')';
+    }
+
+    /**
+     * Traduire un element du tableau where SQL en un filtre.
+     *
+     * @param array|string $v
+     *
+     * @return null|array|string
+     */
+    protected function traduire_condition_sql_en_filtre($v) {
+        if (is_array($v)) {
+            if ((count($v) >= 2) && ('REGEXP' == $v[0]) && ("'.*'" == $v[2])) {
+                return 'true';
+            }
+            if ((count($v) >= 2) && ('LIKE' == $v[0]) && ("'%'" == $v[2])) {
+                return 'true';
+            }
+            $op = $v[0] ?: $v;
+        } else {
+            $op = $v;
+        }
+        if (!$op || 1 == $op || '0=0' == $op) {
+            return 'true';
+        }
+        if ('0=1' === $op) {
+            return 'false';
+        }
+        // traiter {cle IN a,b} ou {valeur !IN a,b}
+        if (preg_match(',^\(([\w/]+)(\s+NOT)?\s+IN\s+(\(.*\))\)$,', (string) $op, $regs)) {
+            return $this->composer_filtre($regs[1], 'IN', $regs[3], $regs[2]);
+        }
+
+        // 3 possibilites : count($v) =
+        // * 1 : {x y} ; on recoit $v[0] = y
+        // * 2 : {x !op y} ; on recoit $v[0] = 'NOT', $v[1] = array() // array du type {x op y}
+        // * 3 : {x op y} ; on recoit $v[0] = 'op', $v[1] = x, $v[2] = y
+
+        // 1 : forcement traite par un critere, on passe
+        if (!$v || !is_array($v) || 1 == count($v)) {
+            return null; // sera ignore
+        }
+        if (2 == count($v) && is_array($v[1])) {
+            return $this->composer_filtre($v[1][1], $v[1][0], $v[1][2], 'NOT');
+        }
+        if (3 == count($v)) {
+            // traiter le OR/AND suivi de 2 valeurs
+            if (in_array($op, ['OR', 'AND'])) {
+                array_shift($v);
+                foreach (array_keys($v) as $k) {
+                    $v[$k] = $this->traduire_condition_sql_en_filtre($v[$k]);
+                    if (null === $v[$k]) {
+                        unset($v[$k]);
+                    } elseif ('true' === $v[$k]) {
+                        if ('OR' === $op) {
+                            return 'true';
+                        }
+                        if ('AND' === $op) {
+                            unset($v[$k]);
+                        }
+                    } elseif ('false' === $v[$k]) {
+                        if ('OR' === $op) {
+                            unset($v[$k]);
+                        }
+                        if ('AND' === $op) {
+                            return 'false';
+                        }
+                    }
+                }
+                if ($v === []) {
+                    return null;
+                }
+                if (1 === count($v)) {
+                    return reset($v);
+                }
+                array_unshift($v, $op);
+
+                return $v;
+            }
+
+            return $this->composer_filtre($v[1], $v[0], $v[2]);
+        }
+
+        return null;  // sera ignore
+    }
+
+    /**
+     * Calculer un filtre sur un champ du tableau.
+     *
+     * @param string $cle
+     * @param string $op
+     * @param string $valeur
+     * @param bool $not
+     *
+     * @return null|string
+     */
+    protected function composer_filtre($cle, $op, $valeur, $not = false) {
+        if (
+            method_exists($this->iter, 'exception_des_criteres')
+            && in_array($cle, $this->iter->exception_des_criteres())
+        ) {
+            return null;
+        }
+        // TODO: analyser le filtre pour refuser ce qu'on ne sait pas traiter ?
+        // mais c'est normalement deja opere par calculer_critere_infixe()
+        // qui regarde la description 'desc' (en casse reelle d'ailleurs : {isDir=1}
+        // ne sera pas vu si l'on a defini desc['field']['isdir'] pour que #ISDIR soit present.
+        // il faudrait peut etre definir les 2 champs isDir et isdir... a reflechir...
+
+        // if (!in_array($cle, array('cle', 'valeur')))
+        //	return;
+
+        $a = '$this->get_select(\'' . $cle . '\')';
+
+        $filtre = '';
+
+        if ('REGEXP' == $op) {
+            $filtre = 'filtrer("match", ' . $a . ', ' . str_replace('\"', '"', (string) $valeur) . ')';
+            $op = '';
+        } else {
+            if ('LIKE' == $op) {
+                $valeur = str_replace(['\"', '_', '%'], ['"', '.', '.*'], preg_quote((string) $valeur));
+                $filtre = 'filtrer("match", ' . $a . ', ' . $valeur . ')';
+                $op = '';
+            } else {
+                if ('=' == $op) {
+                    $op = '==';
+                } else {
+                    if ('IN' == $op) {
+                        $filtre = 'in_array(' . $a . ', array' . $valeur . ')';
+                        $op = '';
+                    } else {
+                        if (!in_array($op, ['<', '<=', '>', '>='])) {
+                            spip_log('operateur non reconnu ' . $op); // [todo] mettre une erreur de squelette
+                            $op = '';
+                        }
+                    }
+                }
+            }
+        }
+
+        if ($op) {
+            $filtre = $a . $op . str_replace('\"', '"', (string) $valeur);
+        }
+
+        if ($not) {
+            $filtre = "!({$filtre})";
+        }
+
+        return $filtre;
+    }
+
+    // calcule les elements a retournes par fetch()
+    // enleve les elements inutiles du select()
+    //
+    private function calculer_select() {
+        if ($select = &$this->command['select']) {
+            foreach ($select as $s) {
+                // /!\ $s = '.nom'
+                if ('.' == $s[0]) {
+                    $s = substr((string) $s, 1);
+                }
+                $this->select[] = $s;
+            }
+        }
+    }
+
+    private function calculer_filtres() {
+        // Issu de calculer_select() de public/composer L.519
+        // TODO: externaliser...
+        //
+        // retirer les criteres vides:
+        // {X ?} avec X absent de l'URL
+        // {par #ENV{X}} avec X absent de l'URL
+        // IN sur collection vide (ce dernier devrait pouvoir etre fait a la compil)
+        if ($where = &$this->command['where']) {
+            foreach ($where as $k => $v) {
+                $this->filtre[] = $this->traduire_condition_sql_en_filtre($v);
+            }
+        }
+
+        // critere {2,7}
+        if (isset($this->command['limit']) && $this->command['limit']) {
+            $limit = explode(',', (string) $this->command['limit']);
+            $this->offset = $limit[0];
+            $this->limit = $limit[1];
+        }
+
+        // Creer la fonction de filtrage sur $this
+        if ($this->filtre) {
+            if ($filtres = $this->assembler_filtres($this->filtre)) {
+                $filtres = 'return ' . $filtres . ';';
+                $this->func_filtre = fn () => eval($filtres);
+            } else {
+                $this->func_filtre = null;
+            }
+        }
+    }
+
+    /*
 	 * aller a la position $n en parcourant
 	 * un par un tous les elements
 	 */
-	private function seek_loop($n) {
-		if ($this->pos > $n) {
-			$this->rewind();
-		}
+    private function seek_loop($n) {
+        if ($this->pos > $n) {
+            $this->rewind();
+        }
 
-		while ($this->pos < $n && $this->valid()) {
-			$this->next();
-		}
+        while ($this->pos < $n && $this->valid()) {
+            $this->next();
+        }
 
-		return true;
-	}
+        return true;
+    }
 }

ecrire/auth/sha256.inc.php

Indentation +2 added lines, -2 removed lines

@@ -12,6 +12,6 @@
  * @return string Le SHA de la chaîne
  */
 function spip_sha256($str) {
-	trigger_deprecation('spip', '5.0', 'Using "%s" is deprecated, use "%s" instead.', 'spip_sha256($str)', 'hash(\'sha256\', $str)');
-	return hash('sha256', $str);
+    trigger_deprecation('spip', '5.0', 'Using "%s" is deprecated, use "%s" instead.', 'spip_sha256($str)', 'hash(\'sha256\', $str)');
+    return hash('sha256', $str);
 }