spip / SPIP

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, $lien['raw']);
-				$lien['texte'] = $t;
-			}
-			$href = $sanitize_callback($lien['href']);
-			if ($href !== $lien['href']) {
-				$lien['raw'] = str_replace($lien['href'], $href, $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($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, $lien['raw']);
-						// et prenons le href comme la vraie url a linker
-						$lien['href'] = $href;
-					}
-
-					if ($desechappe_crochets && str_contains($lien['raw'], "\x1")) {
-						$lien['raw'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], $lien['raw']);
-						$lien['texte'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], $lien['texte']);
-						$lien['href'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], $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, $lien['raw']);
+                $lien['texte'] = $t;
+            }
+            $href = $sanitize_callback($lien['href']);
+            if ($href !== $lien['href']) {
+                $lien['raw'] = str_replace($lien['href'], $href, $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($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, $lien['raw']);
+                        // et prenons le href comme la vraie url a linker
+                        $lien['href'] = $href;
+                    }
+
+                    if ($desechappe_crochets && str_contains($lien['raw'], "\x1")) {
+                        $lien['raw'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], $lien['raw']);
+                        $lien['texte'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], $lien['texte']);
+                        $lien['href'] = str_replace(["\x1\x5", "\x1\x6"], ['[', ']'], $lien['href']);
+                    }
+                }
+            }
+        }
+
+        return $liens;
+    }
 }

ecrire/src/Texte/Collecteur/AbstractCollecteur.php

Indentation +140 added lines, -140 removed lines

@@ -12,144 +12,144 @@
 namespace Spip\Texte\Collecteur;
 
 abstract class AbstractCollecteur {
-	protected static string $markPrefix = 'COLLECT';
-	protected string $markId;
-
-	/**
-	 * Collecteur générique des occurences d'une preg dans un texte avec leurs positions et longueur
-	 * @param string $texte
-	 *   texte à analyser pour la collecte
-	 * @param string $if_chars
-	 *   caractere(s) à tester avant de tenter la preg
-	 * @param string $start_with
-	 *   caractere(s) par lesquels commencent l'expression recherchée (permet de démarrer la preg à la prochaine occurence de cette chaine)
-	 * @param string $preg
-	 *   preg utilisée pour la collecte
-	 * @param int $max_items
-	 *   pour limiter le nombre de preg collectée (pour la detection simple de présence par exemple)
-	 * @return array
-	 */
-	protected static function collecteur(string $texte, string $if_chars, string $start_with, string $preg, int $max_items = 0): array {
-
-		$collection = [];
-		$pos = 0;
-		while (
-			(!$if_chars || str_contains($texte, $if_chars))
-			&& ($next = ($start_with ? strpos($texte, $start_with, $pos) : $pos)) !== false
-			&& preg_match($preg, $texte, $r, PREG_OFFSET_CAPTURE, $next)
-		) {
-			$found_pos = $r[0][1];
-			$found_length = strlen($r[0][0]);
-			$match = [
-				'raw' => $r[0][0],
-				'match' => array_column($r, 0),
-				'pos' => $found_pos,
-				'length' => $found_length
-			];
-
-			$collection[] = $match;
-
-			if ($max_items && count($collection) === $max_items) {
-				break;
-			}
-
-			$pos = $match['pos'] + $match['length'];
-		}
-
-		return $collection;
-	}
-
-	/**
-	 * Sanitizer une collection d'occurences
-	 */
-	protected function sanitizer_collection(array $collection, string $sanitize_callback): array {
-		foreach ($collection as &$c) {
-			$c['raw'] = $sanitize_callback($c['raw']);
-		}
-
-		return $collection;
-	}
-
-	/**
-	 * @return array
-	 */
-	public function collecter(string $texte, array $options = []): array {
-		return [];
-	}
-
-	public function detecter($texte): bool {
-		if (!empty($this->markId) && str_contains($texte, $this->markId)) {
-			return true;
-		}
-		return !empty($this->collecter($texte, ['detecter_presence' => true]));
-	}
-
-	/**
-	 * Echapper les occurences de la collecte par un texte neutre du point de vue HTML
-	 *
-	 * @see retablir()
-	 * @param string $texte
-	 * @param array $options
-	 *   string $sanitize_callback
-	 * @return array
-	 *   texte, marqueur utilise pour echapper les modeles
-	 */
-	public function echapper(string $texte, array $options = []): string {
-		if (!function_exists('creer_uniqid')) {
-			include_spip('inc/acces');
-		}
-
-		$collection = $this->collecter($texte, $options);
-		if (!empty($options['sanitize_callback']) && is_callable($options['sanitize_callback'])) {
-			$collection = $this->sanitizer_collection($collection, $options['sanitize_callback']);
-		}
-
-		if ($collection !== []) {
-			if (empty($this->markId)) {
-				// generer un marqueur qui n'existe pas dans le texte
-				do {
-					$this->markId = substr(md5(uniqid(static::class, 1)), 0, 7);
-					$this->markId = '@|' . static::$markPrefix . $this->markId . '|';
-				} while (str_contains($texte, $this->markId));
-			}
-
-			$offset_pos = 0;
-			foreach ($collection as $c) {
-				$rempl = $this->markId . base64_encode($c['raw']) . '|@';
-				$texte = substr_replace($texte, $rempl, $c['pos'] + $offset_pos, $c['length']);
-				$offset_pos += strlen($rempl) - $c['length'];
-			}
-		}
-
-		return $texte;
-	}
-
-
-	/**
-	 * Retablir les occurences échappées précédemment
-	 *
-	 * @see echapper()
-	 */
-	function retablir(string $texte): string {
-
-		if (!empty($this->markId)) {
-			$lm = strlen($this->markId);
-			$pos = 0;
-			while (
-				($p = strpos($texte, $this->markId, $pos)) !== false
-				&& ($end = strpos($texte, '|@', $p + $lm))
-			) {
-				$base64 = substr($texte, $p + $lm, $end - ($p + $lm));
-				if ($c = base64_decode($base64, true)) {
-					$texte = substr_replace($texte, $c, $p, $end + 2 - $p);
-					$pos = $p + strlen($c);
-				}
-				else {
-					$pos = $end;
-				}
-			}
-		}
-
-		return $texte;
-	}
+    protected static string $markPrefix = 'COLLECT';
+    protected string $markId;
+
+    /**
+     * Collecteur générique des occurences d'une preg dans un texte avec leurs positions et longueur
+     * @param string $texte
+     *   texte à analyser pour la collecte
+     * @param string $if_chars
+     *   caractere(s) à tester avant de tenter la preg
+     * @param string $start_with
+     *   caractere(s) par lesquels commencent l'expression recherchée (permet de démarrer la preg à la prochaine occurence de cette chaine)
+     * @param string $preg
+     *   preg utilisée pour la collecte
+     * @param int $max_items
+     *   pour limiter le nombre de preg collectée (pour la detection simple de présence par exemple)
+     * @return array
+     */
+    protected static function collecteur(string $texte, string $if_chars, string $start_with, string $preg, int $max_items = 0): array {
+
+        $collection = [];
+        $pos = 0;
+        while (
+            (!$if_chars || str_contains($texte, $if_chars))
+            && ($next = ($start_with ? strpos($texte, $start_with, $pos) : $pos)) !== false
+            && preg_match($preg, $texte, $r, PREG_OFFSET_CAPTURE, $next)
+        ) {
+            $found_pos = $r[0][1];
+            $found_length = strlen($r[0][0]);
+            $match = [
+                'raw' => $r[0][0],
+                'match' => array_column($r, 0),
+                'pos' => $found_pos,
+                'length' => $found_length
+            ];
+
+            $collection[] = $match;
+
+            if ($max_items && count($collection) === $max_items) {
+                break;
+            }
+
+            $pos = $match['pos'] + $match['length'];
+        }
+
+        return $collection;
+    }
+
+    /**
+     * Sanitizer une collection d'occurences
+     */
+    protected function sanitizer_collection(array $collection, string $sanitize_callback): array {
+        foreach ($collection as &$c) {
+            $c['raw'] = $sanitize_callback($c['raw']);
+        }
+
+        return $collection;
+    }
+
+    /**
+     * @return array
+     */
+    public function collecter(string $texte, array $options = []): array {
+        return [];
+    }
+
+    public function detecter($texte): bool {
+        if (!empty($this->markId) && str_contains($texte, $this->markId)) {
+            return true;
+        }
+        return !empty($this->collecter($texte, ['detecter_presence' => true]));
+    }
+
+    /**
+     * Echapper les occurences de la collecte par un texte neutre du point de vue HTML
+     *
+     * @see retablir()
+     * @param string $texte
+     * @param array $options
+     *   string $sanitize_callback
+     * @return array
+     *   texte, marqueur utilise pour echapper les modeles
+     */
+    public function echapper(string $texte, array $options = []): string {
+        if (!function_exists('creer_uniqid')) {
+            include_spip('inc/acces');
+        }
+
+        $collection = $this->collecter($texte, $options);
+        if (!empty($options['sanitize_callback']) && is_callable($options['sanitize_callback'])) {
+            $collection = $this->sanitizer_collection($collection, $options['sanitize_callback']);
+        }
+
+        if ($collection !== []) {
+            if (empty($this->markId)) {
+                // generer un marqueur qui n'existe pas dans le texte
+                do {
+                    $this->markId = substr(md5(uniqid(static::class, 1)), 0, 7);
+                    $this->markId = '@|' . static::$markPrefix . $this->markId . '|';
+                } while (str_contains($texte, $this->markId));
+            }
+
+            $offset_pos = 0;
+            foreach ($collection as $c) {
+                $rempl = $this->markId . base64_encode($c['raw']) . '|@';
+                $texte = substr_replace($texte, $rempl, $c['pos'] + $offset_pos, $c['length']);
+                $offset_pos += strlen($rempl) - $c['length'];
+            }
+        }
+
+        return $texte;
+    }
+
+
+    /**
+     * Retablir les occurences échappées précédemment
+     *
+     * @see echapper()
+     */
+    function retablir(string $texte): string {
+
+        if (!empty($this->markId)) {
+            $lm = strlen($this->markId);
+            $pos = 0;
+            while (
+                ($p = strpos($texte, $this->markId, $pos)) !== false
+                && ($end = strpos($texte, '|@', $p + $lm))
+            ) {
+                $base64 = substr($texte, $p + $lm, $end - ($p + $lm));
+                if ($c = base64_decode($base64, true)) {
+                    $texte = substr_replace($texte, $c, $p, $end + 2 - $p);
+                    $pos = $p + strlen($c);
+                }
+                else {
+                    $pos = $end;
+                }
+            }
+        }
+
+        return $texte;
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -109,13 +109,13 @@
 				// generer un marqueur qui n'existe pas dans le texte
 				do {
 					$this->markId = substr(md5(uniqid(static::class, 1)), 0, 7);
-					$this->markId = '@|' . static::$markPrefix . $this->markId . '|';
+					$this->markId = '@|'.static::$markPrefix.$this->markId.'|';
 				} while (str_contains($texte, $this->markId));
 			}
 
 			$offset_pos = 0;
 			foreach ($collection as $c) {
-				$rempl = $this->markId . base64_encode($c['raw']) . '|@';
+				$rempl = $this->markId.base64_encode($c['raw']).'|@';
 				$texte = substr_replace($texte, $rempl, $c['pos'] + $offset_pos, $c['length']);
 				$offset_pos += strlen($rempl) - $c['length'];
 			}

ecrire/src/Sql/Sqlite/Sqlite.php

Indentation +95 added lines, -95 removed lines

@@ -8,112 +8,112 @@
  **/
 class Sqlite
 {
-	/** @var Requeteur[] Liste des instances de requêteurs créés */
-	public static $requeteurs = [];
-	/** @var bool[] Pour chaque connexion, flag pour savoir si une transaction est en cours */
-	public static $transaction_en_cours = [];
+    /** @var Requeteur[] Liste des instances de requêteurs créés */
+    public static $requeteurs = [];
+    /** @var bool[] Pour chaque connexion, flag pour savoir si une transaction est en cours */
+    public static $transaction_en_cours = [];
 
 
-	/**
-	 * Retourne une unique instance du requêteur
-	 *
-	 * Retourne une instance unique du requêteur pour une connexion SQLite
-	 * donnée
-	 *
-	 * @param string $serveur
-	 *    Nom du connecteur
-	 * @return Requeteur
-	 *    Instance unique du requêteur
-	 **/
-	public static function requeteur($serveur) {
-		if (!isset(static::$requeteurs[$serveur])) {
-			static::$requeteurs[$serveur] = new Requeteur($serveur);
-		}
+    /**
+     * Retourne une unique instance du requêteur
+     *
+     * Retourne une instance unique du requêteur pour une connexion SQLite
+     * donnée
+     *
+     * @param string $serveur
+     *    Nom du connecteur
+     * @return Requeteur
+     *    Instance unique du requêteur
+     **/
+    public static function requeteur($serveur) {
+        if (!isset(static::$requeteurs[$serveur])) {
+            static::$requeteurs[$serveur] = new Requeteur($serveur);
+        }
 
-		return static::$requeteurs[$serveur];
-	}
+        return static::$requeteurs[$serveur];
+    }
 
-	/**
-	 * Prépare le texte d'une requête avant son exécution
-	 *
-	 * Adapte la requête au format plus ou moins MySQL par un format
-	 * compris de SQLite.
-	 *
-	 * Change les préfixes de tables SPIP par ceux véritables
-	 *
-	 * @param string $query Requête à préparer
-	 * @param string $serveur Nom de la connexion
-	 * @return string           Requête préparée
-	 */
-	public static function traduire_requete($query, $serveur) {
-		$requeteur = static::requeteur($serveur);
-		$traducteur = new Traducteur($query, $requeteur->prefixe, $requeteur->sqlite_version);
+    /**
+     * Prépare le texte d'une requête avant son exécution
+     *
+     * Adapte la requête au format plus ou moins MySQL par un format
+     * compris de SQLite.
+     *
+     * Change les préfixes de tables SPIP par ceux véritables
+     *
+     * @param string $query Requête à préparer
+     * @param string $serveur Nom de la connexion
+     * @return string           Requête préparée
+     */
+    public static function traduire_requete($query, $serveur) {
+        $requeteur = static::requeteur($serveur);
+        $traducteur = new Traducteur($query, $requeteur->prefixe, $requeteur->sqlite_version);
 
-		return $traducteur->traduire_requete();
-	}
+        return $traducteur->traduire_requete();
+    }
 
-	/**
-	 * Démarre une transaction
-	 *
-	 * @param string $serveur Nom de la connexion
-	 **/
-	public static function demarrer_transaction($serveur) {
-		Sqlite::executer_requete('BEGIN TRANSACTION', $serveur);
-		Sqlite::$transaction_en_cours[$serveur] = true;
-	}
+    /**
+     * Démarre une transaction
+     *
+     * @param string $serveur Nom de la connexion
+     **/
+    public static function demarrer_transaction($serveur) {
+        Sqlite::executer_requete('BEGIN TRANSACTION', $serveur);
+        Sqlite::$transaction_en_cours[$serveur] = true;
+    }
 
-	/**
-	 * Exécute la requête donnée
-	 *
-	 * @param string $query Requête
-	 * @param string $serveur Nom de la connexion
-	 * @param null|bool $tracer Demander des statistiques (temps) ?
-	 **/
-	public static function executer_requete($query, $serveur, $tracer = null) {
-		$requeteur = Sqlite::requeteur($serveur);
+    /**
+     * Exécute la requête donnée
+     *
+     * @param string $query Requête
+     * @param string $serveur Nom de la connexion
+     * @param null|bool $tracer Demander des statistiques (temps) ?
+     **/
+    public static function executer_requete($query, $serveur, $tracer = null) {
+        $requeteur = Sqlite::requeteur($serveur);
 
-		return $requeteur->executer_requete($query, $tracer);
-	}
+        return $requeteur->executer_requete($query, $tracer);
+    }
 
-	/**
-	 * Obtient l'identifiant de la dernière ligne insérée ou modifiée
-	 *
-	 * @param string $serveur Nom de la connexion
-	 * return int                Identifiant
-	 **/
-	public static function last_insert_id($serveur) {
-		$requeteur = Sqlite::requeteur($serveur);
+    /**
+     * Obtient l'identifiant de la dernière ligne insérée ou modifiée
+     *
+     * @param string $serveur Nom de la connexion
+     * return int                Identifiant
+     **/
+    public static function last_insert_id($serveur) {
+        $requeteur = Sqlite::requeteur($serveur);
 
-		return $requeteur->last_insert_id();
-	}
+        return $requeteur->last_insert_id();
+    }
 
-	/**
-	 * Annule une transaction
-	 *
-	 * @param string $serveur Nom de la connexion
-	 **/
-	public static function annuler_transaction($serveur) {
-		Sqlite::executer_requete('ROLLBACK', $serveur);
-		Sqlite::$transaction_en_cours[$serveur] = false;
-	}
+    /**
+     * Annule une transaction
+     *
+     * @param string $serveur Nom de la connexion
+     **/
+    public static function annuler_transaction($serveur) {
+        Sqlite::executer_requete('ROLLBACK', $serveur);
+        Sqlite::$transaction_en_cours[$serveur] = false;
+    }
 
-	/**
-	 * Termine une transaction
-	 *
-	 * @param string $serveur Nom de la connexion
-	 **/
-	public static function finir_transaction($serveur) {
-		// si pas de transaction en cours, ne rien faire et le dire
-		if (
-			!isset(Sqlite::$transaction_en_cours[$serveur])
-			|| Sqlite::$transaction_en_cours[$serveur] == false
-		) {
-			return false;
-		}
-		// sinon fermer la transaction et retourner true
-		Sqlite::executer_requete('COMMIT', $serveur);
-		Sqlite::$transaction_en_cours[$serveur] = false;
+    /**
+     * Termine une transaction
+     *
+     * @param string $serveur Nom de la connexion
+     **/
+    public static function finir_transaction($serveur) {
+        // si pas de transaction en cours, ne rien faire et le dire
+        if (
+            !isset(Sqlite::$transaction_en_cours[$serveur])
+            || Sqlite::$transaction_en_cours[$serveur] == false
+        ) {
+            return false;
+        }
+        // sinon fermer la transaction et retourner true
+        Sqlite::executer_requete('COMMIT', $serveur);
+        Sqlite::$transaction_en_cours[$serveur] = false;
 
-		return true;
-	}
+        return true;
+    }
 }

ecrire/src/Sql/Sqlite/PDOStatement.php

Indentation +5 added lines, -5 removed lines

@@ -6,10 +6,10 @@
  * Pouvoir retrouver le PDO utilisé pour générer un résultat de requête.
  */
 final class PDOStatement extends \PDOStatement {
-	private function __construct(private \PDO &$PDO) {
-	}
+    private function __construct(private \PDO &$PDO) {
+    }
 
-	public function getPDO(): \PDO {
-		return $this->PDO;
-	}
+    public function getPDO(): \PDO {
+        return $this->PDO;
+    }
 }

ecrire/src/Sql/Sqlite/Requeteur.php

Indentation +105 added lines, -105 removed lines

@@ -13,109 +13,109 @@
 
 class Requeteur
 {
-	/** @var string texte de la requête */
-	public $query = ''; // la requete
-	/** @var string Nom de la connexion */
-	public $serveur = '';
-	/** @var \PDO|null Identifiant de la connexion SQLite */
-	public $link = null;
-	/** @var string Prefixe des tables SPIP */
-	public $prefixe = '';
-	/** @var string Nom de la base de donnée */
-	public $db = '';
-	/** @var bool Doit-on tracer les requetes (var_profile) ? */
-	public $tracer = false; // doit-on tracer les requetes (var_profile)
-
-	/** @var string Version de SQLite (2 ou 3) */
-	public $sqlite_version = '';
-
-	/**
-	 * Constructeur
-	 *
-	 * @param string $serveur
-	 */
-	public function __construct($serveur = '') {
-		_sqlite_init();
-		$this->serveur = strtolower($serveur);
-
-		if (!($this->link = _sqlite_link($this->serveur)) && (!defined('_ECRIRE_INSTALL') || !_ECRIRE_INSTALL)) {
-			spip_log('Aucune connexion sqlite (link)', 'sqlite.' . _LOG_ERREUR);
-
-			return;
-		}
-
-		$this->sqlite_version = _sqlite_is_version('', $this->link);
-
-		$this->prefixe = $GLOBALS['connexions'][$this->serveur ?: 0]['prefixe'];
-		$this->db = $GLOBALS['connexions'][$this->serveur ?: 0]['db'];
-
-		// tracage des requetes ?
-		$this->tracer = (isset($_GET['var_profile']) && $_GET['var_profile']);
-	}
-
-	/**
-	 * Lancer la requête transmise et faire le tracage si demandé
-	 *
-	 * @param string $query
-	 *     Requête à exécuter
-	 * @param bool|null $tracer
-	 *     true pour tracer la requête
-	 * @return bool|\PDOStatement|array
-	 */
-	public function executer_requete($query, $tracer = null) {
-		if (is_null($tracer)) {
-			$tracer = $this->tracer;
-		}
-		$err = '';
-		$t = 0;
-		if ($tracer || defined('_DEBUG_TRACE_QUERIES') && _DEBUG_TRACE_QUERIES) {
-			include_spip('public/tracer');
-			$t = trace_query_start();
-		}
-
-		# spip_log("requete: $this->serveur >> $query",'sqlite.'._LOG_DEBUG); // boum ? pourquoi ?
-		if ($this->link) {
-			// memoriser la derniere erreur PHP vue
-			$last_error = (function_exists('error_get_last') ? error_get_last() : '');
-			$e = null;
-			// sauver la derniere requete
-			$GLOBALS['connexions'][$this->serveur ?: 0]['last'] = $query;
-			$GLOBALS['connexions'][$this->serveur ?: 0]['total_requetes']++;
-
-			try {
-				$r = $this->link->query($query);
-			} catch (\PDOException $e) {
-				spip_log('PDOException: ' . $e->getMessage(), 'sqlite.' . _LOG_DEBUG);
-				$r = false;
-			}
-
-			// loger les warnings/erreurs eventuels de sqlite remontant dans PHP
-			if ($e && $e instanceof \PDOException) {
-				$err = strip_tags($e->getMessage()) . ' in ' . $e->getFile() . ' line ' . $e->getLine();
-				spip_log("$err - " . $query, 'sqlite.' . _LOG_ERREUR);
-			} elseif (($err = (function_exists('error_get_last') ? error_get_last() : '')) && $err != $last_error) {
-				$err = strip_tags($err['message']) . ' in ' . $err['file'] . ' line ' . $err['line'];
-				spip_log("$err - " . $query, 'sqlite.' . _LOG_ERREUR);
-			} else {
-				$err = '';
-			}
-		} else {
-			$r = false;
-		}
-
-		if (spip_sqlite_errno($this->serveur)) {
-			$err .= spip_sqlite_error($query, $this->serveur);
-		}
-
-		return $t ? trace_query_end($query, $t, $r, $err, $this->serveur) : $r;
-	}
-
-	/**
-	 * Obtient l'identifiant de la dernière ligne insérée ou modifiée
-	 *
-	 * @return string|false
-	 **/
-	public function last_insert_id() {
-		return $this->link->lastInsertId();
-	}
+    /** @var string texte de la requête */
+    public $query = ''; // la requete
+    /** @var string Nom de la connexion */
+    public $serveur = '';
+    /** @var \PDO|null Identifiant de la connexion SQLite */
+    public $link = null;
+    /** @var string Prefixe des tables SPIP */
+    public $prefixe = '';
+    /** @var string Nom de la base de donnée */
+    public $db = '';
+    /** @var bool Doit-on tracer les requetes (var_profile) ? */
+    public $tracer = false; // doit-on tracer les requetes (var_profile)
+
+    /** @var string Version de SQLite (2 ou 3) */
+    public $sqlite_version = '';
+
+    /**
+     * Constructeur
+     *
+     * @param string $serveur
+     */
+    public function __construct($serveur = '') {
+        _sqlite_init();
+        $this->serveur = strtolower($serveur);
+
+        if (!($this->link = _sqlite_link($this->serveur)) && (!defined('_ECRIRE_INSTALL') || !_ECRIRE_INSTALL)) {
+            spip_log('Aucune connexion sqlite (link)', 'sqlite.' . _LOG_ERREUR);
+
+            return;
+        }
+
+        $this->sqlite_version = _sqlite_is_version('', $this->link);
+
+        $this->prefixe = $GLOBALS['connexions'][$this->serveur ?: 0]['prefixe'];
+        $this->db = $GLOBALS['connexions'][$this->serveur ?: 0]['db'];
+
+        // tracage des requetes ?
+        $this->tracer = (isset($_GET['var_profile']) && $_GET['var_profile']);
+    }
+
+    /**
+     * Lancer la requête transmise et faire le tracage si demandé
+     *
+     * @param string $query
+     *     Requête à exécuter
+     * @param bool|null $tracer
+     *     true pour tracer la requête
+     * @return bool|\PDOStatement|array
+     */
+    public function executer_requete($query, $tracer = null) {
+        if (is_null($tracer)) {
+            $tracer = $this->tracer;
+        }
+        $err = '';
+        $t = 0;
+        if ($tracer || defined('_DEBUG_TRACE_QUERIES') && _DEBUG_TRACE_QUERIES) {
+            include_spip('public/tracer');
+            $t = trace_query_start();
+        }
+
+        # spip_log("requete: $this->serveur >> $query",'sqlite.'._LOG_DEBUG); // boum ? pourquoi ?
+        if ($this->link) {
+            // memoriser la derniere erreur PHP vue
+            $last_error = (function_exists('error_get_last') ? error_get_last() : '');
+            $e = null;
+            // sauver la derniere requete
+            $GLOBALS['connexions'][$this->serveur ?: 0]['last'] = $query;
+            $GLOBALS['connexions'][$this->serveur ?: 0]['total_requetes']++;
+
+            try {
+                $r = $this->link->query($query);
+            } catch (\PDOException $e) {
+                spip_log('PDOException: ' . $e->getMessage(), 'sqlite.' . _LOG_DEBUG);
+                $r = false;
+            }
+
+            // loger les warnings/erreurs eventuels de sqlite remontant dans PHP
+            if ($e && $e instanceof \PDOException) {
+                $err = strip_tags($e->getMessage()) . ' in ' . $e->getFile() . ' line ' . $e->getLine();
+                spip_log("$err - " . $query, 'sqlite.' . _LOG_ERREUR);
+            } elseif (($err = (function_exists('error_get_last') ? error_get_last() : '')) && $err != $last_error) {
+                $err = strip_tags($err['message']) . ' in ' . $err['file'] . ' line ' . $err['line'];
+                spip_log("$err - " . $query, 'sqlite.' . _LOG_ERREUR);
+            } else {
+                $err = '';
+            }
+        } else {
+            $r = false;
+        }
+
+        if (spip_sqlite_errno($this->serveur)) {
+            $err .= spip_sqlite_error($query, $this->serveur);
+        }
+
+        return $t ? trace_query_end($query, $t, $r, $err, $this->serveur) : $r;
+    }
+
+    /**
+     * Obtient l'identifiant de la dernière ligne insérée ou modifiée
+     *
+     * @return string|false
+     **/
+    public function last_insert_id() {
+        return $this->link->lastInsertId();
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -39,7 +39,7 @@ 
 		$this->serveur = strtolower($serveur);
 
 		if (!($this->link = _sqlite_link($this->serveur)) && (!defined('_ECRIRE_INSTALL') || !_ECRIRE_INSTALL)) {
-			spip_log('Aucune connexion sqlite (link)', 'sqlite.' . _LOG_ERREUR);
+			spip_log('Aucune connexion sqlite (link)', 'sqlite.'._LOG_ERREUR);
 
 			return;
 		}
@@ -85,17 +85,17 @@ 
 			try {
 				$r = $this->link->query($query);
 			} catch (\PDOException $e) {
-				spip_log('PDOException: ' . $e->getMessage(), 'sqlite.' . _LOG_DEBUG);
+				spip_log('PDOException: '.$e->getMessage(), 'sqlite.'._LOG_DEBUG);
 				$r = false;
 			}
 
 			// loger les warnings/erreurs eventuels de sqlite remontant dans PHP
 			if ($e && $e instanceof \PDOException) {
-				$err = strip_tags($e->getMessage()) . ' in ' . $e->getFile() . ' line ' . $e->getLine();
-				spip_log("$err - " . $query, 'sqlite.' . _LOG_ERREUR);
+				$err = strip_tags($e->getMessage()).' in '.$e->getFile().' line '.$e->getLine();
+				spip_log("$err - ".$query, 'sqlite.'._LOG_ERREUR);
 			} elseif (($err = (function_exists('error_get_last') ? error_get_last() : '')) && $err != $last_error) {
-				$err = strip_tags($err['message']) . ' in ' . $err['file'] . ' line ' . $err['line'];
-				spip_log("$err - " . $query, 'sqlite.' . _LOG_ERREUR);
+				$err = strip_tags($err['message']).' in '.$err['file'].' line '.$err['line'];
+				spip_log("$err - ".$query, 'sqlite.'._LOG_ERREUR);
 			} else {
 				$err = '';
 			}

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($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($this->query, 'INSERT IGNORE')) {
-			spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.' . _LOG_DEBUG);
-			$this->query = 'INSERT ' . substr($this->query, '13');
-		}
-
-		// Correction des dates avec INTERVAL
-		// utiliser sql_date_proche() de preference
-		if (str_contains($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),
-				$this->query
-			);
-		}
-
-		if (str_contains($this->query, 'LEFT(')) {
-			$this->query = str_replace('LEFT(', '_LEFT(', $this->query);
-		}
-
-		if (str_contains($this->query, 'TIMESTAMPDIFF(')) {
-			$this->query = preg_replace('/TIMESTAMPDIFF\(\s*([^,]*)\s*,/Uims', "TIMESTAMPDIFF('\\1',", $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($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*\([^\)]*\)/', '', $this->query);
-		}
-
-		// Correction Field
-		// remplace FIELD(table,i,j,k...) par CASE WHEN table=i THEN n ... ELSE 0 END
-		if (str_contains($this->query, 'FIELD')) {
-			$this->query = preg_replace_callback(
-				'/FIELD\s*\(([^\)]*)\)/',
-				fn(array $matches): string => $this->_remplacerFieldParCase($matches),
-				$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', $this->query, $regs)) {
-			$suite = strstr($this->query, $regs[0]);
-			$this->query = substr($this->query, 0, -strlen($suite));
-		} else {
-			$suite = '';
-		}
-		$pref = ($this->prefixe) ? $this->prefixe . '_' : '';
-		$this->query = preg_replace('/([,\s])spip_/S', '\1' . $pref, $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($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($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($this->query, 'INSERT IGNORE')) {
+            spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.' . _LOG_DEBUG);
+            $this->query = 'INSERT ' . substr($this->query, '13');
+        }
+
+        // Correction des dates avec INTERVAL
+        // utiliser sql_date_proche() de preference
+        if (str_contains($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),
+                $this->query
+            );
+        }
+
+        if (str_contains($this->query, 'LEFT(')) {
+            $this->query = str_replace('LEFT(', '_LEFT(', $this->query);
+        }
+
+        if (str_contains($this->query, 'TIMESTAMPDIFF(')) {
+            $this->query = preg_replace('/TIMESTAMPDIFF\(\s*([^,]*)\s*,/Uims', "TIMESTAMPDIFF('\\1',", $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($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*\([^\)]*\)/', '', $this->query);
+        }
+
+        // Correction Field
+        // remplace FIELD(table,i,j,k...) par CASE WHEN table=i THEN n ... ELSE 0 END
+        if (str_contains($this->query, 'FIELD')) {
+            $this->query = preg_replace_callback(
+                '/FIELD\s*\(([^\)]*)\)/',
+                fn(array $matches): string => $this->_remplacerFieldParCase($matches),
+                $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', $this->query, $regs)) {
+            $suite = strstr($this->query, $regs[0]);
+            $this->query = substr($this->query, 0, -strlen($suite));
+        } else {
+            $suite = '';
+        }
+        $pref = ($this->prefixe) ? $this->prefixe . '_' : '';
+        $this->query = preg_replace('/([,\s])spip_/S', '\1' . $pref, $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($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($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($this->query, 'INSERT IGNORE')) {
-			spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.' . _LOG_DEBUG);
-			$this->query = 'INSERT ' . substr($this->query, '13');
+			spip_log("Sqlite : requete transformee -> $this->query", 'sqlite.'._LOG_DEBUG);
+			$this->query = 'INSERT '.substr($this->query, '13');
 		}
 
 		// Correction des dates avec INTERVAL
@@ -84,7 +84,7 @@ 
 		if (($this->sqlite_version == 2) && (str_contains($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*\([^\)]*\)/', '', $this->query);
 		}
@@ -107,8 +107,8 @@ 
 		} else {
 			$suite = '';
 		}
-		$pref = ($this->prefixe) ? $this->prefixe . '_' : '';
-		$this->query = preg_replace('/([,\s])spip_/S', '\1' . $pref, $this->query) . $suite;
+		$pref = ($this->prefixe) ? $this->prefixe.'_' : '';
+		$this->query = preg_replace('/([,\s])spip_/S', '\1'.$pref, $this->query).$suite;
 
 		// Correction zero AS x
 		// pg n'aime pas 0+x AS alias, sqlite, dans le meme style,

ecrire/src/Compilateur/Noeud/Idiome.php

Indentation +81 added lines, -81 removed lines

@@ -7,85 +7,85 @@
  **/
 class Idiome
 {
-	/** Type de noeud */
-	public string $type = 'idiome';
-
-	/** Clé de traduction demandée. Exemple 'item_oui' */
-	public string $nom_champ = '';
-
-	/** Module de langue où chercher la clé de traduction. Exemple 'medias' */
-	public string $module = '';
-
-	/** Arguments à passer à la chaîne */
-	public array $arg = [];
-
-	/**
-	 * Filtres à appliquer au résultat
-	 *
-	 *
-	 * * FIXME: type unique.
-	 * @var false|array
-	 *     - false: erreur de syntaxe
-	 */
-	public $param = [];
-
-	/** Source des filtres (compatibilité) (?) */
-	public array $fonctions = [];
-
-	/**
-	 * Inutilisé, propriété générique de l'AST
-	 *
-	 * @var string|array
-	 */
-	public $avant = '';
-
-	/**
-	 * Inutilisé, propriété générique de l'AST
-	 *
-	 * @var string|array
-	 */
-	public $apres = '';
-
-	/** Identifiant de la boucle */
-	public string $id_boucle = '';
-
-	/**
-	 * AST du squelette, liste de toutes les boucles
-	 *
-	 * @var Boucle[]
-	 */
-	public array $boucles;
-
-	/** Alias de table d'application de la requête ou nom complet de la table SQL */
-	public ?string $type_requete = null;
-
-	/** Résultat de la compilation: toujours une expression PHP */
-	public string $code = '';
-
-	/**
-	 * Interdire les scripts
-	 *
-	 * @see interdire_scripts()
-	 */
-	public bool $interdire_scripts = false;
-
-	/**
-	 * Description du squelette
-	 *
-	 * Sert pour la gestion d'erreur et la production de code dependant du contexte
-	 *
-	 * Peut contenir les index :
-	 * - nom : Nom du fichier de cache
-	 * - gram : Nom de la grammaire du squelette (détermine le phraseur à utiliser)
-	 * - sourcefile : Chemin du squelette
-	 * - squelette : Code du squelette
-	 * - id_mere : Identifiant de la boucle parente
-	 * - documents : Pour embed et img dans les textes
-	 * - session : Pour un cache sessionné par auteur
-	 * - niv : Niveau de tabulation
-	 */
-	public array $descr = [];
-
-	/** Numéro de ligne dans le code source du squelette */
-	public int $ligne = 0;
+    /** Type de noeud */
+    public string $type = 'idiome';
+
+    /** Clé de traduction demandée. Exemple 'item_oui' */
+    public string $nom_champ = '';
+
+    /** Module de langue où chercher la clé de traduction. Exemple 'medias' */
+    public string $module = '';
+
+    /** Arguments à passer à la chaîne */
+    public array $arg = [];
+
+    /**
+     * Filtres à appliquer au résultat
+     *
+     *
+     * * FIXME: type unique.
+     * @var false|array
+     *     - false: erreur de syntaxe
+     */
+    public $param = [];
+
+    /** Source des filtres (compatibilité) (?) */
+    public array $fonctions = [];
+
+    /**
+     * Inutilisé, propriété générique de l'AST
+     *
+     * @var string|array
+     */
+    public $avant = '';
+
+    /**
+     * Inutilisé, propriété générique de l'AST
+     *
+     * @var string|array
+     */
+    public $apres = '';
+
+    /** Identifiant de la boucle */
+    public string $id_boucle = '';
+
+    /**
+     * AST du squelette, liste de toutes les boucles
+     *
+     * @var Boucle[]
+     */
+    public array $boucles;
+
+    /** Alias de table d'application de la requête ou nom complet de la table SQL */
+    public ?string $type_requete = null;
+
+    /** Résultat de la compilation: toujours une expression PHP */
+    public string $code = '';
+
+    /**
+     * Interdire les scripts
+     *
+     * @see interdire_scripts()
+     */
+    public bool $interdire_scripts = false;
+
+    /**
+     * Description du squelette
+     *
+     * Sert pour la gestion d'erreur et la production de code dependant du contexte
+     *
+     * Peut contenir les index :
+     * - nom : Nom du fichier de cache
+     * - gram : Nom de la grammaire du squelette (détermine le phraseur à utiliser)
+     * - sourcefile : Chemin du squelette
+     * - squelette : Code du squelette
+     * - id_mere : Identifiant de la boucle parente
+     * - documents : Pour embed et img dans les textes
+     * - session : Pour un cache sessionné par auteur
+     * - niv : Niveau de tabulation
+     */
+    public array $descr = [];
+
+    /** Numéro de ligne dans le code source du squelette */
+    public int $ligne = 0;
 }

ecrire/src/Compilateur/Noeud/Champ.php

Indentation +100 added lines, -100 removed lines

@@ -7,104 +7,104 @@
  */
 class Champ
 {
-	/** Type de noeud */
-	public string $type = 'champ';
-
-	/** Nom du champ demandé. Exemple 'ID_ARTICLE' */
-	public ?string $nom_champ = null;
-
-	/** Identifiant de la boucle parente si explicité */
-	public ?string $nom_boucle = '';
-
-	/**
-	 * Partie optionnelle avant
-	 *
-	 * @var null|string|array
-	 */
-	public $avant;
-
-	/**
-	 * Partie optionnelle après
-	 *
-	 * @var null|string|array
-	 */
-	public $apres;
-
-	/**
-	 * Étoiles : annuler des automatismes
-	 *
-	 * - '*' annule les filtres automatiques
-	 * - '**' annule en plus les protections de scripts
-	 *
-	 * FIXME: type unique.
-	 */
-	public ?string $etoile = '';
-
-	/**
-	 * Arguments et filtres explicites sur la balise
-	 *
-	 * - $param[0] contient les arguments de la balise
-	 * - $param[1..n] contient les filtres à appliquer à la balise
-	 *
-	 * FIXME: type unique.
-	 * @var false|array
-	 *     - false: erreur de syntaxe
-	 */
-	public $param = [];
-
-	/** Source des filtres (compatibilité) (?) */
-	public array $fonctions = [];
-
-	/**
-	 * Identifiant de la boucle
-	 *
-	 * @var string
-	 */
-	public $id_boucle = '';
-
-	/**
-	 * AST du squelette, liste de toutes les boucles
-	 *
-	 * @var Boucle[]
-	 */
-	public array $boucles;
-
-	/** Alias de table d'application de la requête ou nom complet de la table SQL */
-	public ?string $type_requete = null;
-
-	/** Résultat de la compilation: toujours une expression PHP */
-	public string $code = '';
-
-	/**
-	 * Interdire les scripts
-	 *
-	 * false si on est sûr de cette balise
-	 *
-	 * @see interdire_scripts()
-	 */
-	public bool $interdire_scripts = true;
-
-	/**
-	 * Description du squelette
-	 *
-	 * Sert pour la gestion d'erreur et la production de code dependant du contexte
-	 *
-	 * Peut contenir les index :
-	 *
-	 * - nom : Nom du fichier de cache
-	 * - gram : Nom de la grammaire du squelette (détermine le phraseur à utiliser)
-	 * - sourcefile : Chemin du squelette
-	 * - squelette : Code du squelette
-	 * - id_mere : Identifiant de la boucle parente
-	 * - documents : Pour embed et img dans les textes
-	 * - session : Pour un cache sessionné par auteur
-	 * - niv : Niveau de tabulation
-	 */
-	public array $descr = [];
-
-	/** Numéro de ligne dans le code source du squelette*/
-	public int $ligne = 0;
-
-	/** Drapeau pour reperer les balises calculées par une fonction explicite */
-	public bool $balise_calculee = false;
+    /** Type de noeud */
+    public string $type = 'champ';
+
+    /** Nom du champ demandé. Exemple 'ID_ARTICLE' */
+    public ?string $nom_champ = null;
+
+    /** Identifiant de la boucle parente si explicité */
+    public ?string $nom_boucle = '';
+
+    /**
+     * Partie optionnelle avant
+     *
+     * @var null|string|array
+     */
+    public $avant;
+
+    /**
+     * Partie optionnelle après
+     *
+     * @var null|string|array
+     */
+    public $apres;
+
+    /**
+     * Étoiles : annuler des automatismes
+     *
+     * - '*' annule les filtres automatiques
+     * - '**' annule en plus les protections de scripts
+     *
+     * FIXME: type unique.
+     */
+    public ?string $etoile = '';
+
+    /**
+     * Arguments et filtres explicites sur la balise
+     *
+     * - $param[0] contient les arguments de la balise
+     * - $param[1..n] contient les filtres à appliquer à la balise
+     *
+     * FIXME: type unique.
+     * @var false|array
+     *     - false: erreur de syntaxe
+     */
+    public $param = [];
+
+    /** Source des filtres (compatibilité) (?) */
+    public array $fonctions = [];
+
+    /**
+     * Identifiant de la boucle
+     *
+     * @var string
+     */
+    public $id_boucle = '';
+
+    /**
+     * AST du squelette, liste de toutes les boucles
+     *
+     * @var Boucle[]
+     */
+    public array $boucles;
+
+    /** Alias de table d'application de la requête ou nom complet de la table SQL */
+    public ?string $type_requete = null;
+
+    /** Résultat de la compilation: toujours une expression PHP */
+    public string $code = '';
+
+    /**
+     * Interdire les scripts
+     *
+     * false si on est sûr de cette balise
+     *
+     * @see interdire_scripts()
+     */
+    public bool $interdire_scripts = true;
+
+    /**
+     * Description du squelette
+     *
+     * Sert pour la gestion d'erreur et la production de code dependant du contexte
+     *
+     * Peut contenir les index :
+     *
+     * - nom : Nom du fichier de cache
+     * - gram : Nom de la grammaire du squelette (détermine le phraseur à utiliser)
+     * - sourcefile : Chemin du squelette
+     * - squelette : Code du squelette
+     * - id_mere : Identifiant de la boucle parente
+     * - documents : Pour embed et img dans les textes
+     * - session : Pour un cache sessionné par auteur
+     * - niv : Niveau de tabulation
+     */
+    public array $descr = [];
+
+    /** Numéro de ligne dans le code source du squelette*/
+    public int $ligne = 0;
+
+    /** Drapeau pour reperer les balises calculées par une fonction explicite */
+    public bool $balise_calculee = false;
 }

ecrire/src/Compilateur/Noeud/Critere.php

Indentation +28 added lines, -28 removed lines

@@ -8,32 +8,32 @@
  * Sous-noeud de Boucle
  **/
 class Critere {
-	/** Type de noeud */
-	public string $type = 'critere';
-
-	/** Opérateur (>, <, >=, IN, ...) */
-	public ?string $op = null;
-
-	/** Présence d'une négation (truc !op valeur) */
-	public bool $not = false;
-
-	/** Présence d'une exclusion (!truc op valeur) */
-	public string $exclus = '';
-
-	/** Présence d'une condition dans le critère (truc ?) */
-	public bool $cond = false;
-
-	/**
-	 * Paramètres du critère
-	 * - $param[0] : élément avant l'opérateur
-	 * - $param[1..n] : éléments après l'opérateur
-	 *
-	 * FIXME: type unique.
-	 * @var false|array
-	 *     - false: erreur de syntaxe
-	 */
-	public $param = [];
-
-	/** Numéro de ligne dans le code source du squelette */
-	public int $ligne = 0;
+    /** Type de noeud */
+    public string $type = 'critere';
+
+    /** Opérateur (>, <, >=, IN, ...) */
+    public ?string $op = null;
+
+    /** Présence d'une négation (truc !op valeur) */
+    public bool $not = false;
+
+    /** Présence d'une exclusion (!truc op valeur) */
+    public string $exclus = '';
+
+    /** Présence d'une condition dans le critère (truc ?) */
+    public bool $cond = false;
+
+    /**
+     * Paramètres du critère
+     * - $param[0] : élément avant l'opérateur
+     * - $param[1..n] : éléments après l'opérateur
+     *
+     * FIXME: type unique.
+     * @var false|array
+     *     - false: erreur de syntaxe
+     */
+    public $param = [];
+
+    /** Numéro de ligne dans le code source du squelette */
+    public int $ligne = 0;
 }