spip / SPIP

ecrire/src/Compilateur/Noeud/Boucle.php

Indentation +364 added lines, -364 removed lines

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

ecrire/tests/bootstrap.php

Indentation +15 added lines, -15 removed lines

@@ -9,22 +9,22 @@ 
 
 // pas en cli ? passe ton chemin ! (ce script est un vilain trou de securite)
 if (PHP_SAPI !== 'cli') {
-	throw new RuntimeException('Operation not allowed.');
+    throw new RuntimeException('Operation not allowed.');
 }
 
 // let's go spip
 if (!defined('_SPIP_TEST_INC')) {
-	define('_SPIP_TEST_INC', dirname(__FILE__, 2));
+    define('_SPIP_TEST_INC', dirname(__FILE__, 2));
 }
 
 // si rien defini on va dans le public
 
 if (!defined('_SPIP_TEST_CHDIR')) {
-	define('_SPIP_TEST_CHDIR', dirname(_SPIP_TEST_INC));
+    define('_SPIP_TEST_CHDIR', dirname(_SPIP_TEST_INC));
 }
 
 if (!defined('_DIR_TESTS')) {
-	define('_DIR_TESTS', substr(_SPIP_TEST_INC, strlen(_SPIP_TEST_CHDIR) + 1) . '/');
+    define('_DIR_TESTS', substr(_SPIP_TEST_INC, strlen(_SPIP_TEST_CHDIR) + 1) . '/');
 }
 
 // chdir pour charger SPIP
@@ -34,8 +34,8 @@ 
 
 // pour notice sur recuperer_fond()
 if (!isset($GLOBALS['spip_lang'])) {
-	include_spip('inc/lang');
-	utiliser_langue_visiteur();
+    include_spip('inc/lang');
+    utiliser_langue_visiteur();
 }
 
 $GLOBALS['taille_des_logs'] = 1024;
@@ -46,16 +46,16 @@ 
 @error_reporting(E_ALL);
 
 function spip_tests_loger_webmestre() {
-	// il faut charger une session webmestre
-	include_spip('base/abstract_sql');
-	$webmestre = sql_fetsel('*', 'spip_auteurs', "statut='0minirezo' AND webmestre='oui'", '', 'id_auteur', '0,1');
-	include_spip('inc/auth');
-	auth_loger($webmestre);
+    // il faut charger une session webmestre
+    include_spip('base/abstract_sql');
+    $webmestre = sql_fetsel('*', 'spip_auteurs', "statut='0minirezo' AND webmestre='oui'", '', 'id_auteur', '0,1');
+    include_spip('inc/auth');
+    auth_loger($webmestre);
 }
 
 function spip_tests_deloger_webmestre() {
-	if (!empty($GLOBALS['visiteur_session']['id_auteur'])) {
-		supprimer_sessions($GLOBALS['visiteur_session']['id_auteur'], false);
-	}
-	$GLOBALS['visiteur_session'] = [];
+    if (!empty($GLOBALS['visiteur_session']['id_auteur'])) {
+        supprimer_sessions($GLOBALS['visiteur_session']['id_auteur'], false);
+    }
+    $GLOBALS['visiteur_session'] = [];
 }

Spacing +2 added lines, -2 removed lines

@@ -24,13 +24,13 @@
 }
 
 if (!defined('_DIR_TESTS')) {
-	define('_DIR_TESTS', substr(_SPIP_TEST_INC, strlen(_SPIP_TEST_CHDIR) + 1) . '/');
+	define('_DIR_TESTS', substr(_SPIP_TEST_INC, strlen(_SPIP_TEST_CHDIR) + 1).'/');
 }
 
 // chdir pour charger SPIP
 chdir(_SPIP_TEST_CHDIR);
 
-require_once _SPIP_TEST_CHDIR . '/ecrire/inc_version.php';
+require_once _SPIP_TEST_CHDIR.'/ecrire/inc_version.php';
 
 // pour notice sur recuperer_fond()
 if (!isset($GLOBALS['spip_lang'])) {

ecrire/tests/Templating.php

Indentation +31 added lines, -31 removed lines

@@ -10,35 +10,35 @@
 
 class Templating
 {
-	public function __construct(
-		private readonly LoaderInterface $loader
-	) {
-	}
-
-	public static function fromString(array $options = []): self {
-		return new static(new StringLoader($options));
-	}
-
-	public static function fromFile(): self {
-		return new static(new FileLoader());
-	}
-
-	public function getLoader(): LoaderInterface {
-		return $this->loader;
-	}
-
-	public function load(string $name): Template {
-		$source = $this->loader->getSourceFile($name);
-		return new Template($source);
-	}
-
-	public function render(string $name, array $contexte = [], string $connect = ''): string {
-		return $this->load($name)
-			->render($contexte, $connect);
-	}
-
-	public function rawRender(string $name, array $contexte = [], string $connect = ''): array {
-		return $this->load($name)
-			->rawRender($contexte, $connect);
-	}
+    public function __construct(
+        private readonly LoaderInterface $loader
+    ) {
+    }
+
+    public static function fromString(array $options = []): self {
+        return new static(new StringLoader($options));
+    }
+
+    public static function fromFile(): self {
+        return new static(new FileLoader());
+    }
+
+    public function getLoader(): LoaderInterface {
+        return $this->loader;
+    }
+
+    public function load(string $name): Template {
+        $source = $this->loader->getSourceFile($name);
+        return new Template($source);
+    }
+
+    public function render(string $name, array $contexte = [], string $connect = ''): string {
+        return $this->load($name)
+            ->render($contexte, $connect);
+    }
+
+    public function rawRender(string $name, array $contexte = [], string $connect = ''): array {
+        return $this->load($name)
+            ->rawRender($contexte, $connect);
+    }
 }

ecrire/tests/Formulaire/ProtegeChampTest.php

Indentation +142 added lines, -142 removed lines

@@ -13,160 +13,160 @@
 
 class ProtegeChampTest extends TestCase
 {
-	public static function setUpBeforeClass(): void {
-		find_in_path('./balise/formulaire_.php', '', true);
-	}
+    public static function setUpBeforeClass(): void {
+        find_in_path('./balise/formulaire_.php', '', true);
+    }
 
-	#[DataProvider('providerFormulaireProtegeChamp')]
-	public function testFormulaireProtegeChamp($expected, ...$args): void {
-		$actual = protege_champ(...$args);
-		$this->assertSame($expected, $actual);
-	}
+    #[DataProvider('providerFormulaireProtegeChamp')]
+    public function testFormulaireProtegeChamp($expected, ...$args): void {
+        $actual = protege_champ(...$args);
+        $this->assertSame($expected, $actual);
+    }
 
-	public static function providerFormulaireProtegeChamp(): array {
-		return [
-			0 => [
-				0 => 'i:1;',
-				1 => 'i:1;',
-			],
-			1 => [
-				0 => 's:4:"toto";',
-				1 => 's:4:"toto";',
-			],
-			2 => [
-				0 => 'b:1;',
-				1 => 'b:1;',
-			],
-			3 => [
-				0 => 'b:0;',
-				1 => 'b:0;',
-			],
-			4 => [
-				0 => 'a:1:{i:0;s:4:"toto";}',
-				1 => 'a:1:{i:0;s:4:"toto";}',
-			],
-			5 => [
-				0 => '',
-				1 => '',
-			],
-			6 => [
-				0 => '0',
-				1 => '0',
-			],
-			7 => [
-				0 => 'Un texte avec des <a href="http://spip.net">liens</a> [Article 1->art1] [spip->http://www.spip.net] http://www.spip.net',
-				1 => 'Un texte avec des <a href="http://spip.net">liens</a> [Article 1->art1] [spip->http://www.spip.net] http://www.spip.net',
-			],
-			8 => [
-				0 => 'Un texte avec des entit&amp;eacute;s &amp;amp;&amp;lt;&amp;gt;&amp;quot;',
-				1 => 'Un texte avec des entit&eacute;s &amp;&lt;&gt;&quot;',
-			],
-			9 => [
-				0 => 'Un texte sans entites &amp;&lt;&gt;&quot;&#039;',
-				1 => 'Un texte sans entites &<>"\'',
-			],
-			10 => [
-				0 => '{{{Des raccourcis}}} {italique} {{gras}} &lt;code&gt;du code&lt;/code&gt;',
-				1 => '{{{Des raccourcis}}} {italique} {{gras}} <code>du code</code>',
-			],
-			11 => [
-				0 => 'Un modele &lt;modeleinexistant|lien=[-&gt;http://www.spip.net]&gt;',
-				1 => 'Un modele <modeleinexistant|lien=[->http://www.spip.net]>',
-			],
-			12 => [
-				0 => 'Un texte avec des retour
+    public static function providerFormulaireProtegeChamp(): array {
+        return [
+            0 => [
+                0 => 'i:1;',
+                1 => 'i:1;',
+            ],
+            1 => [
+                0 => 's:4:&quot;toto&quot;;',
+                1 => 's:4:"toto";',
+            ],
+            2 => [
+                0 => 'b:1;',
+                1 => 'b:1;',
+            ],
+            3 => [
+                0 => 'b:0;',
+                1 => 'b:0;',
+            ],
+            4 => [
+                0 => 'a:1:{i:0;s:4:&quot;toto&quot;;}',
+                1 => 'a:1:{i:0;s:4:"toto";}',
+            ],
+            5 => [
+                0 => '',
+                1 => '',
+            ],
+            6 => [
+                0 => '0',
+                1 => '0',
+            ],
+            7 => [
+                0 => 'Un texte avec des &lt;a href=&quot;http://spip.net&quot;&gt;liens&lt;/a&gt; [Article 1-&gt;art1] [spip-&gt;http://www.spip.net] http://www.spip.net',
+                1 => 'Un texte avec des <a href="http://spip.net">liens</a> [Article 1->art1] [spip->http://www.spip.net] http://www.spip.net',
+            ],
+            8 => [
+                0 => 'Un texte avec des entit&amp;eacute;s &amp;amp;&amp;lt;&amp;gt;&amp;quot;',
+                1 => 'Un texte avec des entit&eacute;s &amp;&lt;&gt;&quot;',
+            ],
+            9 => [
+                0 => 'Un texte sans entites &amp;&lt;&gt;&quot;&#039;',
+                1 => 'Un texte sans entites &<>"\'',
+            ],
+            10 => [
+                0 => '{{{Des raccourcis}}} {italique} {{gras}} &lt;code&gt;du code&lt;/code&gt;',
+                1 => '{{{Des raccourcis}}} {italique} {{gras}} <code>du code</code>',
+            ],
+            11 => [
+                0 => 'Un modele &lt;modeleinexistant|lien=[-&gt;http://www.spip.net]&gt;',
+                1 => 'Un modele <modeleinexistant|lien=[->http://www.spip.net]>',
+            ],
+            12 => [
+                0 => 'Un texte avec des retour
 a la ligne et meme des
 
 paragraphes',
-				1 => 'Un texte avec des retour
+                1 => 'Un texte avec des retour
 a la ligne et meme des
 
 paragraphes',
-			],
-			13 => [
-				0 => [],
-				1 => [],
-			],
-			14 => [
-				0 => [
-					0 => '',
-					1 => '0',
-					2 => 'Un texte avec des &lt;a href=&quot;http://spip.net&quot;&gt;liens&lt;/a&gt; [Article 1-&gt;art1] [spip-&gt;http://www.spip.net] http://www.spip.net',
-					3 => 'Un texte avec des entit&amp;eacute;s &amp;amp;&amp;lt;&amp;gt;&amp;quot;',
-					4 => 'Un texte sans entites &amp;&lt;&gt;&quot;&#039;',
-					5 => '{{{Des raccourcis}}} {italique} {{gras}} &lt;code&gt;du code&lt;/code&gt;',
-					6 => 'Un modele &lt;modeleinexistant|lien=[-&gt;http://www.spip.net]&gt;',
-					7 => 'Un texte avec des retour
+            ],
+            13 => [
+                0 => [],
+                1 => [],
+            ],
+            14 => [
+                0 => [
+                    0 => '',
+                    1 => '0',
+                    2 => 'Un texte avec des &lt;a href=&quot;http://spip.net&quot;&gt;liens&lt;/a&gt; [Article 1-&gt;art1] [spip-&gt;http://www.spip.net] http://www.spip.net',
+                    3 => 'Un texte avec des entit&amp;eacute;s &amp;amp;&amp;lt;&amp;gt;&amp;quot;',
+                    4 => 'Un texte sans entites &amp;&lt;&gt;&quot;&#039;',
+                    5 => '{{{Des raccourcis}}} {italique} {{gras}} &lt;code&gt;du code&lt;/code&gt;',
+                    6 => 'Un modele &lt;modeleinexistant|lien=[-&gt;http://www.spip.net]&gt;',
+                    7 => 'Un texte avec des retour
 a la ligne et meme des
 
 paragraphes',
-				],
-				1 => [
-					0 => '',
-					1 => '0',
-					2 => 'Un texte avec des <a href="http://spip.net">liens</a> [Article 1->art1] [spip->http://www.spip.net] http://www.spip.net',
-					3 => 'Un texte avec des entit&eacute;s &amp;&lt;&gt;&quot;',
-					4 => 'Un texte sans entites &<>"\'',
-					5 => '{{{Des raccourcis}}} {italique} {{gras}} <code>du code</code>',
-					6 => 'Un modele <modeleinexistant|lien=[->http://www.spip.net]>',
-					7 => 'Un texte avec des retour
+                ],
+                1 => [
+                    0 => '',
+                    1 => '0',
+                    2 => 'Un texte avec des <a href="http://spip.net">liens</a> [Article 1->art1] [spip->http://www.spip.net] http://www.spip.net',
+                    3 => 'Un texte avec des entit&eacute;s &amp;&lt;&gt;&quot;',
+                    4 => 'Un texte sans entites &<>"\'',
+                    5 => '{{{Des raccourcis}}} {italique} {{gras}} <code>du code</code>',
+                    6 => 'Un modele <modeleinexistant|lien=[->http://www.spip.net]>',
+                    7 => 'Un texte avec des retour
 a la ligne et meme des
 
 paragraphes',
-				],
-			],
-			15 => [
-				0 => [
-					0 => 0,
-					1 => -1,
-					2 => 1,
-					3 => 2,
-					4 => 3,
-					5 => 4,
-					6 => 5,
-					7 => 6,
-					8 => 7,
-					9 => 10,
-					10 => 20,
-					11 => 30,
-					12 => 50,
-					13 => 100,
-					14 => 1000,
-					15 => 10000,
-				],
-				1 => [
-					0 => 0,
-					1 => -1,
-					2 => 1,
-					3 => 2,
-					4 => 3,
-					5 => 4,
-					6 => 5,
-					7 => 6,
-					8 => 7,
-					9 => 10,
-					10 => 20,
-					11 => 30,
-					12 => 50,
-					13 => 100,
-					14 => 1000,
-					15 => 10000,
-				],
-			],
-			16 => [
-				0 => [
-					0 => '1',
-					1 => '',
-				],
-				1 => [
-					0 => true,
-					1 => false,
-				],
-			],
-			17 => [
-				0 => null,
-				1 => null,
-			],
-		];
-	}
+                ],
+            ],
+            15 => [
+                0 => [
+                    0 => 0,
+                    1 => -1,
+                    2 => 1,
+                    3 => 2,
+                    4 => 3,
+                    5 => 4,
+                    6 => 5,
+                    7 => 6,
+                    8 => 7,
+                    9 => 10,
+                    10 => 20,
+                    11 => 30,
+                    12 => 50,
+                    13 => 100,
+                    14 => 1000,
+                    15 => 10000,
+                ],
+                1 => [
+                    0 => 0,
+                    1 => -1,
+                    2 => 1,
+                    3 => 2,
+                    4 => 3,
+                    5 => 4,
+                    6 => 5,
+                    7 => 6,
+                    8 => 7,
+                    9 => 10,
+                    10 => 20,
+                    11 => 30,
+                    12 => 50,
+                    13 => 100,
+                    14 => 1000,
+                    15 => 10000,
+                ],
+            ],
+            16 => [
+                0 => [
+                    0 => '1',
+                    1 => '',
+                ],
+                1 => [
+                    0 => true,
+                    1 => false,
+                ],
+            ],
+            17 => [
+                0 => null,
+                1 => null,
+            ],
+        ];
+    }
 }

ecrire/tests/Action/EditerLiensTest.php

Indentation +263 added lines, -263 removed lines

@@ -19,276 +19,276 @@
 
 class EditerLiensTest extends TestCase
 {
-	public static function setUpBeforeClass(): void {
-		include_spip('action/editer_liens');
-	}
+    public static function setUpBeforeClass(): void {
+        include_spip('action/editer_liens');
+    }
 
-	public static function tearDownAfterClass(): void {
-		include_spip('base/abstract_sql');
-		sql_delete('spip_auteurs_liens', "objet='spirou'");
-		sql_delete('spip_auteurs_liens', "objet='zorglub'");
-	}
+    public static function tearDownAfterClass(): void {
+        include_spip('base/abstract_sql');
+        sql_delete('spip_auteurs_liens', "objet='spirou'");
+        sql_delete('spip_auteurs_liens', "objet='zorglub'");
+    }
 
-	public function testObjetAssociable() {
-		$essais = [
-			[
-				0 => false,
-				1 => 'article',
-			],
-			[
-				0 => ['id_auteur', 'spip_auteurs_liens'],
-				1 => 'auteur',
-			],
-			[
-				0 => ['id_mot', 'spip_mots_liens'],
-				1 => 'mot',
-			],
-			[
-				0 => ['id_document', 'spip_documents_liens'],
-				1 => 'document',
-			],
-			[
-				0 => false,
-				1 => "mot' OR 1=1'",
-			],
-		];
+    public function testObjetAssociable() {
+        $essais = [
+            [
+                0 => false,
+                1 => 'article',
+            ],
+            [
+                0 => ['id_auteur', 'spip_auteurs_liens'],
+                1 => 'auteur',
+            ],
+            [
+                0 => ['id_mot', 'spip_mots_liens'],
+                1 => 'mot',
+            ],
+            [
+                0 => ['id_document', 'spip_documents_liens'],
+                1 => 'document',
+            ],
+            [
+                0 => false,
+                1 => "mot' OR 1=1'",
+            ],
+        ];
 
-		foreach ($essais as $k => $essai) {
-			$expected = array_shift($essai);
-			$this->assertEquals($expected, objet_associable(...$essai), "Echec {$k} : objet_associable " . end($essai));
-		}
-	}
+        foreach ($essais as $k => $essai) {
+            $expected = array_shift($essai);
+            $this->assertEquals($expected, objet_associable(...$essai), "Echec {$k} : objet_associable " . end($essai));
+        }
+    }
 
-	#[Depends('testObjetAssociable')]
-	public function testObjetAssocier() {
-		$essais = [
-			[
-				0 => false,
-				1 => [
-					'article' => 1,
-				],
-				2 => [
-					'spirou' => 1,
-				],
-			],
-			[
-				0 => 1,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'spirou' => 1,
-				],
-			],
-			[
-				0 => 0,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'spirou' => 1,
-				],
-			],
-			[
-				0 => 2,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'spirou' => [2, 3],
-				],
-			],
-			[
-				0 => 1,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'spirou' => [2, 3, 4],
-				],
-			],
-			[
-				10,
-				[
-					'auteur' => 1,
-				],
-				[
-					'zorglub' => [1, 2, 3, 4, 5, 6, 7, 8, 9, 10],
-				],
-			],
-			[
-				6,
-				[
-					'auteur' => 1,
-				],
-				[
-					'spirou' => [1, 2, 3, 4, 5, 6, 7, 8, 9, 10],
-				],
-			],
-		];
+    #[Depends('testObjetAssociable')]
+    public function testObjetAssocier() {
+        $essais = [
+            [
+                0 => false,
+                1 => [
+                    'article' => 1,
+                ],
+                2 => [
+                    'spirou' => 1,
+                ],
+            ],
+            [
+                0 => 1,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'spirou' => 1,
+                ],
+            ],
+            [
+                0 => 0,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'spirou' => 1,
+                ],
+            ],
+            [
+                0 => 2,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'spirou' => [2, 3],
+                ],
+            ],
+            [
+                0 => 1,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'spirou' => [2, 3, 4],
+                ],
+            ],
+            [
+                10,
+                [
+                    'auteur' => 1,
+                ],
+                [
+                    'zorglub' => [1, 2, 3, 4, 5, 6, 7, 8, 9, 10],
+                ],
+            ],
+            [
+                6,
+                [
+                    'auteur' => 1,
+                ],
+                [
+                    'spirou' => [1, 2, 3, 4, 5, 6, 7, 8, 9, 10],
+                ],
+            ],
+        ];
 
-		foreach ($essais as $k => $essai) {
-			$expected = array_shift($essai);
-			$this->assertEquals(
-				$expected,
-				objet_associer(...$essai),
-				"Echec {$k} : objet_associer " . json_encode($essai, JSON_THROW_ON_ERROR)
-			);
-		}
-	}
+        foreach ($essais as $k => $essai) {
+            $expected = array_shift($essai);
+            $this->assertEquals(
+                $expected,
+                objet_associer(...$essai),
+                "Echec {$k} : objet_associer " . json_encode($essai, JSON_THROW_ON_ERROR)
+            );
+        }
+    }
 
-	#[Depends('testObjetAssocier')]
-	public function testObjetQualifierLiens() {
-		$essais = [
-			[
-				0 => false,
-				1 => [
-					'article' => 1,
-				],
-				2 => [
-					'zorglub' => 1,
-				],
-				3 => [
-					'vu' => 'oui',
-				],
-			],
-			[
-				0 => 1,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'zorglub' => 1,
-				],
-				3 => [
-					'vu' => 'oui',
-				],
-			],
-			[
-				0 => 1,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'zorglub' => 1,
-				],
-				3 => [
-					'vu' => 'oui',
-				],
-			],
-			[
-				0 => false,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'zorglub' => 1,
-				],
-				3 => [
-					'veraer' => 'oui',
-				],
-			],
-			[
-				0 => 2,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'zorglub' => [2, 3],
-				],
-				3 => [
-					'vu' => 'oui',
-				],
-			],
-		];
+    #[Depends('testObjetAssocier')]
+    public function testObjetQualifierLiens() {
+        $essais = [
+            [
+                0 => false,
+                1 => [
+                    'article' => 1,
+                ],
+                2 => [
+                    'zorglub' => 1,
+                ],
+                3 => [
+                    'vu' => 'oui',
+                ],
+            ],
+            [
+                0 => 1,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'zorglub' => 1,
+                ],
+                3 => [
+                    'vu' => 'oui',
+                ],
+            ],
+            [
+                0 => 1,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'zorglub' => 1,
+                ],
+                3 => [
+                    'vu' => 'oui',
+                ],
+            ],
+            [
+                0 => false,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'zorglub' => 1,
+                ],
+                3 => [
+                    'veraer' => 'oui',
+                ],
+            ],
+            [
+                0 => 2,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'zorglub' => [2, 3],
+                ],
+                3 => [
+                    'vu' => 'oui',
+                ],
+            ],
+        ];
 
-		foreach ($essais as $k => $essai) {
-			$expected = array_shift($essai);
-			$this->assertEquals(
-				$expected,
-				objet_qualifier_liens(...$essai),
-				"Echec {$k} : objet_qualifier_liens " . json_encode($essai, JSON_THROW_ON_ERROR)
-			);
-		}
-	}
+        foreach ($essais as $k => $essai) {
+            $expected = array_shift($essai);
+            $this->assertEquals(
+                $expected,
+                objet_qualifier_liens(...$essai),
+                "Echec {$k} : objet_qualifier_liens " . json_encode($essai, JSON_THROW_ON_ERROR)
+            );
+        }
+    }
 
-	#[Depends('testObjetQualifierLiens')]
-	public function testObjetDissocier() {
-		$essais = [
-			[
-				0 => false,
-				1 => [
-					'article' => 1,
-				],
-				2 => [
-					'zorglub' => 1,
-				],
-			],
-			[
-				0 => 1,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'zorglub' => 1,
-				],
-			],
-			[
-				0 => 0,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'zorglub' => 1,
-				],
-			],
-			[
-				0 => 2,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'zorglub' => [2, 3],
-				],
-			],
-			[
-				0 => 1,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'zorglub' => [2, 3, 4],
-				],
-			],
-			[
-				0 => 4,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'zorglub' => [5],
-					'spirou' => [2, 3, 4],
-				],
-			],
-			[
-				0 => 12,
-				1 => [
-					'auteur' => 1,
-				],
-				2 => [
-					'zorglub' => '*',
-					'spirou' => '*',
-				],
-			],
-		];
+    #[Depends('testObjetQualifierLiens')]
+    public function testObjetDissocier() {
+        $essais = [
+            [
+                0 => false,
+                1 => [
+                    'article' => 1,
+                ],
+                2 => [
+                    'zorglub' => 1,
+                ],
+            ],
+            [
+                0 => 1,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'zorglub' => 1,
+                ],
+            ],
+            [
+                0 => 0,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'zorglub' => 1,
+                ],
+            ],
+            [
+                0 => 2,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'zorglub' => [2, 3],
+                ],
+            ],
+            [
+                0 => 1,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'zorglub' => [2, 3, 4],
+                ],
+            ],
+            [
+                0 => 4,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'zorglub' => [5],
+                    'spirou' => [2, 3, 4],
+                ],
+            ],
+            [
+                0 => 12,
+                1 => [
+                    'auteur' => 1,
+                ],
+                2 => [
+                    'zorglub' => '*',
+                    'spirou' => '*',
+                ],
+            ],
+        ];
 
-		foreach ($essais as $k => $essai) {
-			$expected = array_shift($essai);
-			$this->assertEquals(
-				$expected,
-				objet_dissocier(...$essai),
-				"Echec {$k} : objet_dissocier " . json_encode($essai, JSON_THROW_ON_ERROR)
-			);
-		}
-	}
+        foreach ($essais as $k => $essai) {
+            $expected = array_shift($essai);
+            $this->assertEquals(
+                $expected,
+                objet_dissocier(...$essai),
+                "Echec {$k} : objet_dissocier " . json_encode($essai, JSON_THROW_ON_ERROR)
+            );
+        }
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -55,7 +55,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, objet_associable(...$essai), "Echec {$k} : objet_associable " . end($essai));
+			$this->assertEquals($expected, objet_associable(...$essai), "Echec {$k} : objet_associable ".end($essai));
 		}
 	}
 
@@ -132,7 +132,7 @@ 
 			$this->assertEquals(
 				$expected,
 				objet_associer(...$essai),
-				"Echec {$k} : objet_associer " . json_encode($essai, JSON_THROW_ON_ERROR)
+				"Echec {$k} : objet_associer ".json_encode($essai, JSON_THROW_ON_ERROR)
 			);
 		}
 	}
@@ -207,7 +207,7 @@ 
 			$this->assertEquals(
 				$expected,
 				objet_qualifier_liens(...$essai),
-				"Echec {$k} : objet_qualifier_liens " . json_encode($essai, JSON_THROW_ON_ERROR)
+				"Echec {$k} : objet_qualifier_liens ".json_encode($essai, JSON_THROW_ON_ERROR)
 			);
 		}
 	}
@@ -287,7 +287,7 @@ 
 			$this->assertEquals(
 				$expected,
 				objet_dissocier(...$essai),
-				"Echec {$k} : objet_dissocier " . json_encode($essai, JSON_THROW_ON_ERROR)
+				"Echec {$k} : objet_dissocier ".json_encode($essai, JSON_THROW_ON_ERROR)
 			);
 		}
 	}

ecrire/tests/SquelettesTestCase.php

Indentation +258 added lines, -258 removed lines

@@ -12,262 +12,262 @@
 
 abstract class SquelettesTestCase extends TestCase
 {
-	/**
-	 * Determine si une chaine débute par 'NA' (non applicable)
-	 */
-	public static function isNa(string $chaine): bool {
-		return str_starts_with(strtolower(trim($chaine)), 'na');
-	}
-
-	/**
-	 * Retourne le chemin relatif depuis la racine de SPIP
-	 */
-	public static function relativePath(string $fullDirectory): string {
-		if (!defined('_SPIP_TEST_CHDIR')) {
-			throw new RuntimeException('_SPIP_TEST_CHDIR needs to be defined');
-		}
-
-		return substr($fullDirectory, strlen(_SPIP_TEST_CHDIR) + 1);
-	}
-
-	/**
-	 * Determine si une chaine débute par 'OK'
-	 */
-	public static function assertOk($actual, string $message = ''): void {
-		$constraint = new IsOk($actual);
-
-		static::assertThat($actual, $constraint, $message);
-	}
-
-	/**
-	 * Determine si une chaine ne débute pas par 'OK'
-	 */
-	public static function assertNotOk($actual, string $message = ''): void {
-		$constraint = new LogicalNot(new IsOk($actual));
-
-		static::assertThat($actual, $constraint, $message);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un template est 'OK'
-	 *
-	 * @example
-	 * $templating = new Templating(new StringLoader());
-	 * $this->assertOkTemplate($templating, '[(#CONFIG{pasla}|non)ok]');
-	 *
-	 * $templating = new Templating(new FileLoader());
-	 * $this->assertOkTemplate($templating, __DIR__ . '/data/truc.html');
-	 *
-	 * @param string $code Code ou chemin du squelette
-	 * @param array $contexte Contexte de calcul du squelette
-	 * @param string $message Message pour une eventuelle erreur
-	 */
-	public static function assertOkTemplate(
-		Templating $templating,
-		string $code,
-		array $contexte = [],
-		string $message = ''
-	): void {
-		$actual = $templating->render($code, $contexte);
-
-		static::assertOk($actual, $message);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un template est vide
-	 *
-	 * @see assertOkTemplate()
-	 */
-	public static function assertNotOkTemplate(
-		Templating $templating,
-		string $code,
-		array $contexte = [],
-		string $message = ''
-	): void {
-		$actual = $templating->render($code, $contexte);
-
-		static::assertNotOk($actual, $message);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un template est vide
-	 *
-	 * @see assertOkTemplate()
-	 */
-	public static function assertEmptyTemplate(
-		Templating $templating,
-		string $code,
-		array $contexte = [],
-		string $message = ''
-	): void {
-		$actual = $templating->render($code, $contexte);
-
-		static::assertEmpty($actual, $message);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un template n’est pas vide
-	 *
-	 * @see assertOkTemplate()
-	 */
-	public static function assertNotEmptyTemplate(
-		Templating $templating,
-		string $code,
-		array $contexte = [],
-		string $message = ''
-	): void {
-		$actual = $templating->render($code, $contexte);
-
-		static::assertNotEmpty($actual, $message);
-	}
-
-	/**
-	 * Assertion qui vérifie le résultat d’un template
-	 *
-	 * @see assertOkTemplate()
-	 */
-	public static function assertEqualsTemplate(
-		string $expected,
-		Templating $templating,
-		string $code,
-		array $contexte = [],
-		string $message = ''
-	): void {
-		$actual = $templating->render($code, $contexte);
-
-		static::assertEquals($expected, $actual, $message);
-	}
-
-	/**
-	 * Assertion qui vérifie le résultat d’un template
-	 *
-	 * @see assertOkTemplate()
-	 */
-	public static function assertNotEqualsTemplate(
-		string $expected,
-		Templating $templating,
-		string $code,
-		array $contexte = [],
-		string $message = ''
-	): void {
-		$actual = $templating->render($code, $contexte);
-
-		static::assertNotEquals($expected, $actual, $message);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un code de squelette est 'OK'
-	 *
-	 * @example $this->assertOkCode('[(#CONFIG{pasla}|non)ok]');
-	 *
-	 * @param string $code Code ou chemin du squelette
-	 * @param array $contexte Contexte de calcul du squelette
-	 * @param string $message Message pour une eventuelle erreur
-	 */
-	public static function assertOkCode(string $code, array $contexte = [], string $message = ''): void {
-		static::assertOkTemplate(Templating::fromString(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un code de squelette n’est pas 'OK'
-	 *
-	 * @see assertOkCode()
-	 */
-	public function assertNotOkCode(string $code, array $contexte = [], $message = ''): void {
-		static::assertNotOkTemplate(Templating::fromString(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un code de squelette est vide
-	 *
-	 * @see assertOkCode()
-	 */
-	public static function assertEmptyCode(string $code, array $contexte = [], string $message = ''): void {
-		static::assertEmptyTemplate(Templating::fromString(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un code de squelette n’est pas vide
-	 *
-	 * @see assertOkCode()
-	 */
-	public static function assertNotEmptyCode(string $code, array $contexte = [], string $message = ''): void {
-		static::assertNotEmptyTemplate(Templating::fromString(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie le résultat d’un code de squelette
-	 *
-	 * @see assertOkCode()
-	 */
-	public function assertEqualsCode(string $expected, string $code, array $contexte = [], $message = ''): void {
-		static::assertEqualsTemplate($expected, Templating::fromString(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie le résultat d’un code de squelette
-	 *
-	 * @see assertOkCode()
-	 */
-	public function assertNotEqualsCode(string $expected, string $code, array $contexte = [], $message = '') {
-		static::assertNotEqualsTemplate($expected, Templating::fromString(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un fichier de squelette est 'OK'
-	 *
-	 * @example $this->assertOkSquelette(__DIR__ . '/data/squelette.html');
-	 *
-	 * @param string $code Code ou chemin du squelette
-	 * @param array $contexte Contexte de calcul du squelette
-	 * @param string $message Message pour une eventuelle erreur
-	 */
-	public static function assertOkSquelette(string $code, array $contexte = [], string $message = ''): void {
-		static::assertOkTemplate(Templating::fromFile(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un fichier de squelette n’est pas 'OK'
-	 *
-	 * @see assertOkSquelette()
-	 */
-	public function assertNotOkSquelette(string $code, array $contexte = [], $message = ''): void {
-		static::assertNotOkTemplate(Templating::fromFile(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un fichier de squelette est vide
-	 *
-	 * @see assertOkSquelette()
-	 */
-	public static function assertEmptySquelette(string $code, array $contexte = [], string $message = ''): void {
-		static::assertEmptyTemplate(Templating::fromFile(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie que le résultat d’un fichier de squelette n’est pas vide
-	 *
-	 * @see assertOkSquelette()
-	 */
-	public static function assertNotEmptySquelette(string $code, array $contexte = [], string $message = ''): void {
-		static::assertNotEmptyTemplate(Templating::fromFile(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie le résultat d’un fichier de squelette
-	 *
-	 * @see assertOkSquelette()
-	 */
-	public function assertEqualsSquelette(string $expected, string $code, array $contexte = [], $message = ''): void {
-		static::assertEqualsTemplate($expected, Templating::fromFile(), $code, $contexte);
-	}
-
-	/**
-	 * Assertion qui vérifie le résultat d’un fichier de squelette
-	 *
-	 * @see assertOkSquelette()
-	 */
-	public function assertNotEqualsSquelette(string $expected, string $code, array $contexte = [], $message = '') {
-		static::assertNotEqualsTemplate($expected, Templating::fromFile(), $code, $contexte);
-	}
+    /**
+     * Determine si une chaine débute par 'NA' (non applicable)
+     */
+    public static function isNa(string $chaine): bool {
+        return str_starts_with(strtolower(trim($chaine)), 'na');
+    }
+
+    /**
+     * Retourne le chemin relatif depuis la racine de SPIP
+     */
+    public static function relativePath(string $fullDirectory): string {
+        if (!defined('_SPIP_TEST_CHDIR')) {
+            throw new RuntimeException('_SPIP_TEST_CHDIR needs to be defined');
+        }
+
+        return substr($fullDirectory, strlen(_SPIP_TEST_CHDIR) + 1);
+    }
+
+    /**
+     * Determine si une chaine débute par 'OK'
+     */
+    public static function assertOk($actual, string $message = ''): void {
+        $constraint = new IsOk($actual);
+
+        static::assertThat($actual, $constraint, $message);
+    }
+
+    /**
+     * Determine si une chaine ne débute pas par 'OK'
+     */
+    public static function assertNotOk($actual, string $message = ''): void {
+        $constraint = new LogicalNot(new IsOk($actual));
+
+        static::assertThat($actual, $constraint, $message);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un template est 'OK'
+     *
+     * @example
+     * $templating = new Templating(new StringLoader());
+     * $this->assertOkTemplate($templating, '[(#CONFIG{pasla}|non)ok]');
+     *
+     * $templating = new Templating(new FileLoader());
+     * $this->assertOkTemplate($templating, __DIR__ . '/data/truc.html');
+     *
+     * @param string $code Code ou chemin du squelette
+     * @param array $contexte Contexte de calcul du squelette
+     * @param string $message Message pour une eventuelle erreur
+     */
+    public static function assertOkTemplate(
+        Templating $templating,
+        string $code,
+        array $contexte = [],
+        string $message = ''
+    ): void {
+        $actual = $templating->render($code, $contexte);
+
+        static::assertOk($actual, $message);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un template est vide
+     *
+     * @see assertOkTemplate()
+     */
+    public static function assertNotOkTemplate(
+        Templating $templating,
+        string $code,
+        array $contexte = [],
+        string $message = ''
+    ): void {
+        $actual = $templating->render($code, $contexte);
+
+        static::assertNotOk($actual, $message);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un template est vide
+     *
+     * @see assertOkTemplate()
+     */
+    public static function assertEmptyTemplate(
+        Templating $templating,
+        string $code,
+        array $contexte = [],
+        string $message = ''
+    ): void {
+        $actual = $templating->render($code, $contexte);
+
+        static::assertEmpty($actual, $message);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un template n’est pas vide
+     *
+     * @see assertOkTemplate()
+     */
+    public static function assertNotEmptyTemplate(
+        Templating $templating,
+        string $code,
+        array $contexte = [],
+        string $message = ''
+    ): void {
+        $actual = $templating->render($code, $contexte);
+
+        static::assertNotEmpty($actual, $message);
+    }
+
+    /**
+     * Assertion qui vérifie le résultat d’un template
+     *
+     * @see assertOkTemplate()
+     */
+    public static function assertEqualsTemplate(
+        string $expected,
+        Templating $templating,
+        string $code,
+        array $contexte = [],
+        string $message = ''
+    ): void {
+        $actual = $templating->render($code, $contexte);
+
+        static::assertEquals($expected, $actual, $message);
+    }
+
+    /**
+     * Assertion qui vérifie le résultat d’un template
+     *
+     * @see assertOkTemplate()
+     */
+    public static function assertNotEqualsTemplate(
+        string $expected,
+        Templating $templating,
+        string $code,
+        array $contexte = [],
+        string $message = ''
+    ): void {
+        $actual = $templating->render($code, $contexte);
+
+        static::assertNotEquals($expected, $actual, $message);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un code de squelette est 'OK'
+     *
+     * @example $this->assertOkCode('[(#CONFIG{pasla}|non)ok]');
+     *
+     * @param string $code Code ou chemin du squelette
+     * @param array $contexte Contexte de calcul du squelette
+     * @param string $message Message pour une eventuelle erreur
+     */
+    public static function assertOkCode(string $code, array $contexte = [], string $message = ''): void {
+        static::assertOkTemplate(Templating::fromString(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un code de squelette n’est pas 'OK'
+     *
+     * @see assertOkCode()
+     */
+    public function assertNotOkCode(string $code, array $contexte = [], $message = ''): void {
+        static::assertNotOkTemplate(Templating::fromString(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un code de squelette est vide
+     *
+     * @see assertOkCode()
+     */
+    public static function assertEmptyCode(string $code, array $contexte = [], string $message = ''): void {
+        static::assertEmptyTemplate(Templating::fromString(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un code de squelette n’est pas vide
+     *
+     * @see assertOkCode()
+     */
+    public static function assertNotEmptyCode(string $code, array $contexte = [], string $message = ''): void {
+        static::assertNotEmptyTemplate(Templating::fromString(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie le résultat d’un code de squelette
+     *
+     * @see assertOkCode()
+     */
+    public function assertEqualsCode(string $expected, string $code, array $contexte = [], $message = ''): void {
+        static::assertEqualsTemplate($expected, Templating::fromString(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie le résultat d’un code de squelette
+     *
+     * @see assertOkCode()
+     */
+    public function assertNotEqualsCode(string $expected, string $code, array $contexte = [], $message = '') {
+        static::assertNotEqualsTemplate($expected, Templating::fromString(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un fichier de squelette est 'OK'
+     *
+     * @example $this->assertOkSquelette(__DIR__ . '/data/squelette.html');
+     *
+     * @param string $code Code ou chemin du squelette
+     * @param array $contexte Contexte de calcul du squelette
+     * @param string $message Message pour une eventuelle erreur
+     */
+    public static function assertOkSquelette(string $code, array $contexte = [], string $message = ''): void {
+        static::assertOkTemplate(Templating::fromFile(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un fichier de squelette n’est pas 'OK'
+     *
+     * @see assertOkSquelette()
+     */
+    public function assertNotOkSquelette(string $code, array $contexte = [], $message = ''): void {
+        static::assertNotOkTemplate(Templating::fromFile(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un fichier de squelette est vide
+     *
+     * @see assertOkSquelette()
+     */
+    public static function assertEmptySquelette(string $code, array $contexte = [], string $message = ''): void {
+        static::assertEmptyTemplate(Templating::fromFile(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie que le résultat d’un fichier de squelette n’est pas vide
+     *
+     * @see assertOkSquelette()
+     */
+    public static function assertNotEmptySquelette(string $code, array $contexte = [], string $message = ''): void {
+        static::assertNotEmptyTemplate(Templating::fromFile(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie le résultat d’un fichier de squelette
+     *
+     * @see assertOkSquelette()
+     */
+    public function assertEqualsSquelette(string $expected, string $code, array $contexte = [], $message = ''): void {
+        static::assertEqualsTemplate($expected, Templating::fromFile(), $code, $contexte);
+    }
+
+    /**
+     * Assertion qui vérifie le résultat d’un fichier de squelette
+     *
+     * @see assertOkSquelette()
+     */
+    public function assertNotEqualsSquelette(string $expected, string $code, array $contexte = [], $message = '') {
+        static::assertNotEqualsTemplate($expected, Templating::fromFile(), $code, $contexte);
+    }
 }

ecrire/tests/Propre/EchappeHtmlTest.php

Indentation +116 added lines, -116 removed lines

@@ -12,123 +12,123 @@ 
 use PHPUnit\Framework\TestCase;
 
 function callback_test_propre_echappe_html_echappe($regs): string {
-	return 'A';
+    return 'A';
 }
 
 function callback_test_propre_echappe_html_traiter_echap_html($regs): string {
-	return callback_test_propre_echappe_html_echappe($regs);
+    return callback_test_propre_echappe_html_echappe($regs);
 }
 
 function callback_test_propre_echappe_html_traiter_echap_code($regs): string {
-	return callback_test_propre_echappe_html_echappe($regs);
+    return callback_test_propre_echappe_html_echappe($regs);
 }
 
 function callback_test_propre_echappe_html_traiter_echap_cadre($regs): string {
-	return callback_test_propre_echappe_html_echappe($regs);
+    return callback_test_propre_echappe_html_echappe($regs);
 }
 
 function callback_test_propre_echappe_html_traiter_echap_frame($regs): string {
-	return callback_test_propre_echappe_html_echappe($regs);
+    return callback_test_propre_echappe_html_echappe($regs);
 }
 
 function callback_test_propre_echappe_html_traiter_echap_script($regs): string {
-	return callback_test_propre_echappe_html_echappe($regs);
+    return callback_test_propre_echappe_html_echappe($regs);
 }
 
 class EchappeHtmlTest extends TestCase
 {
-	public static function setUpBeforeClass(): void {
-		find_in_path('inc/texte.php', '', true);
-	}
-
-	#[DataProvider('providerSimpleEchappeHtml')]
-	public function testSimpleEchappeHtml($expected, ...$args): void {
-		$actual = echappe_html(
-			$args[0],
-			$args[1] ?? '',
-			$args[2] ?? false,
-			$args[3] ?? '',
-			__NAMESPACE__ . '\\callback_test_propre_echappe_html_'
-		);
-		$this->assertSame($expected, $actual);
-
-		// et on teste le retour
-		$retour = echappe_retour($actual, $args[1] ?? '');
-		$this->assertSame($args[0], $retour);
-	}
-
-	public static function providerSimpleEchappeHtml(): array {
-		find_in_path('inc/texte_mini.php', '', true);
-		$befores = [
-			'',
-			'Un texte avant',
-			"Un texte avant sur\n\nplusieurs lignes et avec un < pour voir",
-			"Un texte avant sur\n\nplusieurs lignes et avec un > pour voir",
-		];
-		$afters = [
-			'',
-			'Un texte après',
-			"Un texte après sur\n\nplusieurs lignes et avec un < pour voir",
-			"Un texte après sur\n\nplusieurs lignes et avec un > pour voir",
-		];
-		$insides = ['', 'Un texte dedans', "\nun texte dedans", "un texte dedans\n", "\nun texte dedans\n"];
-		$balises = ['html', 'pre', 'code', 'cadre', 'frame', 'script', 'style', 'svg'];
-		$attrs = ['', 'class="truc"', 'classe="base64"', 'title="truc"'];
-		$pregs = ['', ',<(svg)(\b[^>]*)?>(.*)</\1>,UimsS'];
-		$essais = [];
-
-		$source = 'SIMPLE';
-		$no_transform = true;
-		foreach ($balises as $balise) {
-			foreach ($attrs as $attr) {
-				$cpt = 1;
-				foreach ($insides as $inside) {
-					$html = "<{$balise}" . ($attr ? " $attr" : '') . ">$inside</$balise>";
-					$code_echappe = \code_echappement($html, $source, $no_transform);
-					foreach ($befores as $before) {
-						foreach ($afters as $after) {
-							$texte = $before . $html . $after;
-							foreach ($pregs as $preg) {
-								// si preg vide et pas balise svg OU si preg ET balise SVG on attend un echappement
-								// sinon on attends le texte d'origine
-								if ($preg ? $balise === 'svg' : $balise !== 'svg') {
-									$expected = $before . $code_echappe . $after;
-								} else {
-									$expected = $texte;
-								}
-								$essais["<{$balise}{$attr}>_$cpt"] = [$expected, $texte, $source, $no_transform, $preg];
-								$cpt++;
-							}
-						}
-					}
-				}
-			}
-		}
-
-		return $essais;
-	}
-
-	#[DataProvider('providerPropreEchappeHtml')]
-	public function testPropreEchappeHtml($expected, ...$args): void {
-		$actual = echappe_html(
-			$args[0],
-			$args[1] ?? '',
-			$args[2] ?? false,
-			$args[3] ?? '',
-			__NAMESPACE__ . '\\callback_test_propre_echappe_html_'
-		);
-		$this->assertSame($expected, $actual);
-	}
-
-	public static function providerPropreEchappeHtml(): array {
-		$essais = [];
-		$marque = '<span class="base64" title="QQ=="></span>';
-		$essais['simple imbriqué'] = [
-			'avant 1' . $marque . 'apres 1</code>apres 2',
-			'avant 1<code class="php"> avant 2<code>le code</code>apres 1</code>apres 2',
-		];
-		$essais['complexe imbriqué'] = [
-			<<<CODE_SAMPLE
+    public static function setUpBeforeClass(): void {
+        find_in_path('inc/texte.php', '', true);
+    }
+
+    #[DataProvider('providerSimpleEchappeHtml')]
+    public function testSimpleEchappeHtml($expected, ...$args): void {
+        $actual = echappe_html(
+            $args[0],
+            $args[1] ?? '',
+            $args[2] ?? false,
+            $args[3] ?? '',
+            __NAMESPACE__ . '\\callback_test_propre_echappe_html_'
+        );
+        $this->assertSame($expected, $actual);
+
+        // et on teste le retour
+        $retour = echappe_retour($actual, $args[1] ?? '');
+        $this->assertSame($args[0], $retour);
+    }
+
+    public static function providerSimpleEchappeHtml(): array {
+        find_in_path('inc/texte_mini.php', '', true);
+        $befores = [
+            '',
+            'Un texte avant',
+            "Un texte avant sur\n\nplusieurs lignes et avec un < pour voir",
+            "Un texte avant sur\n\nplusieurs lignes et avec un > pour voir",
+        ];
+        $afters = [
+            '',
+            'Un texte après',
+            "Un texte après sur\n\nplusieurs lignes et avec un < pour voir",
+            "Un texte après sur\n\nplusieurs lignes et avec un > pour voir",
+        ];
+        $insides = ['', 'Un texte dedans', "\nun texte dedans", "un texte dedans\n", "\nun texte dedans\n"];
+        $balises = ['html', 'pre', 'code', 'cadre', 'frame', 'script', 'style', 'svg'];
+        $attrs = ['', 'class="truc"', 'classe="base64"', 'title="truc"'];
+        $pregs = ['', ',<(svg)(\b[^>]*)?>(.*)</\1>,UimsS'];
+        $essais = [];
+
+        $source = 'SIMPLE';
+        $no_transform = true;
+        foreach ($balises as $balise) {
+            foreach ($attrs as $attr) {
+                $cpt = 1;
+                foreach ($insides as $inside) {
+                    $html = "<{$balise}" . ($attr ? " $attr" : '') . ">$inside</$balise>";
+                    $code_echappe = \code_echappement($html, $source, $no_transform);
+                    foreach ($befores as $before) {
+                        foreach ($afters as $after) {
+                            $texte = $before . $html . $after;
+                            foreach ($pregs as $preg) {
+                                // si preg vide et pas balise svg OU si preg ET balise SVG on attend un echappement
+                                // sinon on attends le texte d'origine
+                                if ($preg ? $balise === 'svg' : $balise !== 'svg') {
+                                    $expected = $before . $code_echappe . $after;
+                                } else {
+                                    $expected = $texte;
+                                }
+                                $essais["<{$balise}{$attr}>_$cpt"] = [$expected, $texte, $source, $no_transform, $preg];
+                                $cpt++;
+                            }
+                        }
+                    }
+                }
+            }
+        }
+
+        return $essais;
+    }
+
+    #[DataProvider('providerPropreEchappeHtml')]
+    public function testPropreEchappeHtml($expected, ...$args): void {
+        $actual = echappe_html(
+            $args[0],
+            $args[1] ?? '',
+            $args[2] ?? false,
+            $args[3] ?? '',
+            __NAMESPACE__ . '\\callback_test_propre_echappe_html_'
+        );
+        $this->assertSame($expected, $actual);
+    }
+
+    public static function providerPropreEchappeHtml(): array {
+        $essais = [];
+        $marque = '<span class="base64" title="QQ=="></span>';
+        $essais['simple imbriqué'] = [
+            'avant 1' . $marque . 'apres 1</code>apres 2',
+            'avant 1<code class="php"> avant 2<code>le code</code>apres 1</code>apres 2',
+        ];
+        $essais['complexe imbriqué'] = [
+            <<<CODE_SAMPLE
 {{{code class="php"}}}
 avant blah
 {$marque}
@@ -172,21 +172,21 @@ 
 
 On peut croire que c'est embétant , faut mettre une div autour pour encadrer , mais cela permet d'orienter geshi en cours de route comme dans [Compte à rebours (revisited)->article6]
 CODE_SAMPLE
-		];
-		$essais['unicode sans rien'] = [
-			'azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-',
-			'azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-',
-		];
-		$essais['sans rien'] = [
-			'astuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolos',
-			'astuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolos',
-		];
-		$essais['code sans imbrication'] = [
-			'avant 1' . $marque . 'apres 2',
-			'avant 1<code class="php"> avant 2 code le code code apres 1</code>apres 2',
-		];
-		$essais['pourriture'] = [
-			<<<CODE_SAMPLE
+        ];
+        $essais['unicode sans rien'] = [
+            'azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-',
+            'azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-azerty小さくてもグローバルなケベックの村-',
+        ];
+        $essais['sans rien'] = [
+            'astuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolos',
+            'astuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolos',
+        ];
+        $essais['code sans imbrication'] = [
+            'avant 1' . $marque . 'apres 2',
+            'avant 1<code class="php"> avant 2 code le code code apres 1</code>apres 2',
+        ];
+        $essais['pourriture'] = [
+            <<<CODE_SAMPLE
 Le code mis en {$marque} ou en {$marque} peut lui même contenir  {$marque} ou {$marque} ...
 {$marque}
 Je voudrais présenter l'usage du plugin coloration_code qui fournit une extension à geshi : la classe "spip".
@@ -283,7 +283,7 @@ 
 
 Donc comme l'ancien coloration_code, le  &lt;/code> est mangé et "et le tour est joué" apparait hors-code.
 CODE_SAMPLE
-		];
-		return $essais;
-	}
+        ];
+        return $essais;
+    }
 }

Spacing +7 added lines, -7 removed lines

@@ -48,7 +48,7 @@ 
 			$args[1] ?? '',
 			$args[2] ?? false,
 			$args[3] ?? '',
-			__NAMESPACE__ . '\\callback_test_propre_echappe_html_'
+			__NAMESPACE__.'\\callback_test_propre_echappe_html_'
 		);
 		$this->assertSame($expected, $actual);
 
@@ -83,16 +83,16 @@ 
 			foreach ($attrs as $attr) {
 				$cpt = 1;
 				foreach ($insides as $inside) {
-					$html = "<{$balise}" . ($attr ? " $attr" : '') . ">$inside</$balise>";
+					$html = "<{$balise}".($attr ? " $attr" : '').">$inside</$balise>";
 					$code_echappe = \code_echappement($html, $source, $no_transform);
 					foreach ($befores as $before) {
 						foreach ($afters as $after) {
-							$texte = $before . $html . $after;
+							$texte = $before.$html.$after;
 							foreach ($pregs as $preg) {
 								// si preg vide et pas balise svg OU si preg ET balise SVG on attend un echappement
 								// sinon on attends le texte d'origine
 								if ($preg ? $balise === 'svg' : $balise !== 'svg') {
-									$expected = $before . $code_echappe . $after;
+									$expected = $before.$code_echappe.$after;
 								} else {
 									$expected = $texte;
 								}
@@ -115,7 +115,7 @@ 
 			$args[1] ?? '',
 			$args[2] ?? false,
 			$args[3] ?? '',
-			__NAMESPACE__ . '\\callback_test_propre_echappe_html_'
+			__NAMESPACE__.'\\callback_test_propre_echappe_html_'
 		);
 		$this->assertSame($expected, $actual);
 	}
@@ -124,7 +124,7 @@ 
 		$essais = [];
 		$marque = '<span class="base64" title="QQ=="></span>';
 		$essais['simple imbriqué'] = [
-			'avant 1' . $marque . 'apres 1</code>apres 2',
+			'avant 1'.$marque.'apres 1</code>apres 2',
 			'avant 1<code class="php"> avant 2<code>le code</code>apres 1</code>apres 2',
 		];
 		$essais['complexe imbriqué'] = [
@@ -182,7 +182,7 @@ 
 			'astuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolosastuce & travaux de mode rigolos',
 		];
 		$essais['code sans imbrication'] = [
-			'avant 1' . $marque . 'apres 2',
+			'avant 1'.$marque.'apres 2',
 			'avant 1<code class="php"> avant 2 code le code code apres 1</code>apres 2',
 		];
 		$essais['pourriture'] = [

ecrire/tests/Propre/CodeImbriqueTest.php

Indentation +11 added lines, -11 removed lines

@@ -15,18 +15,18 @@ 
 
 class CodeImbriqueTest extends TestCase
 {
-	public static function setUpBeforeClass(): void {
-		find_in_path('inc/texte.php', '', true);
-	}
+    public static function setUpBeforeClass(): void {
+        find_in_path('inc/texte.php', '', true);
+    }
 
-	#[DataProvider('providerPropreCodeImbrique')]
-	public function testPropreCodeImbrique($expected, ...$args): void {
-		$actual = strlen(propre(...$args)) > 0;
-		$this->assertSame($expected, $actual);
-	}
+    #[DataProvider('providerPropreCodeImbrique')]
+    public function testPropreCodeImbrique($expected, ...$args): void {
+        $actual = strlen(propre(...$args)) > 0;
+        $this->assertSame($expected, $actual);
+    }
 
-	public static function providerPropreCodeImbrique(): array {
-		return [[true, '({{{Classificar els resultats}}}
+    public static function providerPropreCodeImbrique(): array {
+        return [[true, '({{{Classificar els resultats}}}
 
 <html><tt><b>{par</b> <i>critère_de_classement</i><b>}</b></tt></html> iIndica l’ordre en què es presenten els resultats. Aquest criteri de classificació correspon a una de les etiquetes extretes de la base de dades per cada tipus de bucle. Per exemple, podrem classificar els articles per la data <code>{per date}</code>, per la data en què han estat redactats  <code>{par date_redac}</code> o per títol <code>{par titre}</code>. (Fixeu-vos que, tot i que les etiquetes es presenten en majúscules, els criteris de classificació es presenten en minúscules.)
 
@@ -232,5 +232,5 @@ 
 {{{Divers}}}
 
 {{<code>{logo}</code>}} permet seleccionar només els articles (o seccions, etc) que disposin d\'un logo. Funciona també en el bucle (HIERARCHIE). El criteri invers <code>{!logo}</code> llista els objectes que no tenen logo.']];
-	}
+    }
 }

ecrire/tests/Propre/TraiterModelesTest.php

Indentation +34 added lines, -34 removed lines

@@ -8,38 +8,38 @@
 
 class TraiterModelesTest extends TestCase
 {
-	public static function setUpBeforeClass(): void {
-		find_in_path('inc/modeles.php', '', true);
-	}
-
-	public function testTraiterModelesDefaut(): void {
-		unset($GLOBALS['doublons_documents_inclus']);
-
-		$texte = 'Mon texte <doc1> <img2> <emb3> et paf les doublons';
-		traiter_modeles($texte);
-		$this->assertNull($GLOBALS['doublons_documents_inclus'] ?? null);
-
-		traiter_modeles($texte, [
-			'documents' => ['doc', 'emb', 'img'],
-		]);
-		$this->assertNotEmpty($GLOBALS['doublons_documents_inclus']);
-		$this->assertEquals([1, 2, 3], $GLOBALS['doublons_documents_inclus']);
-	}
-
-	public function testTraiterModelesAlbums(): void {
-		unset($GLOBALS['doublons_documents_inclus']);
-		unset($GLOBALS['doublons_albums_inclus']);
-
-		$texte = 'Mon texte <doc1> <img2> <emb3> et paf les doublons';
-
-		traiter_modeles($texte, ['albums' => ['album']]);
-		$this->assertNull($GLOBALS['doublons_documents_inclus'] ?? null);
-		$this->assertNull($GLOBALS['doublons_albums_inclus'] ?? null);
-
-		$texte = 'Mon texte <doc1> <img2> <emb3> et paf les doublons avec un album <album4> ?';
-		traiter_modeles($texte, ['albums' => ['album']]);
-		$this->assertNull($GLOBALS['doublons_documents_inclus'] ?? null);
-		$this->assertNotEmpty($GLOBALS['doublons_albums_inclus']);
-		$this->assertEquals([4], $GLOBALS['doublons_albums_inclus']);
-	}
+    public static function setUpBeforeClass(): void {
+        find_in_path('inc/modeles.php', '', true);
+    }
+
+    public function testTraiterModelesDefaut(): void {
+        unset($GLOBALS['doublons_documents_inclus']);
+
+        $texte = 'Mon texte <doc1> <img2> <emb3> et paf les doublons';
+        traiter_modeles($texte);
+        $this->assertNull($GLOBALS['doublons_documents_inclus'] ?? null);
+
+        traiter_modeles($texte, [
+            'documents' => ['doc', 'emb', 'img'],
+        ]);
+        $this->assertNotEmpty($GLOBALS['doublons_documents_inclus']);
+        $this->assertEquals([1, 2, 3], $GLOBALS['doublons_documents_inclus']);
+    }
+
+    public function testTraiterModelesAlbums(): void {
+        unset($GLOBALS['doublons_documents_inclus']);
+        unset($GLOBALS['doublons_albums_inclus']);
+
+        $texte = 'Mon texte <doc1> <img2> <emb3> et paf les doublons';
+
+        traiter_modeles($texte, ['albums' => ['album']]);
+        $this->assertNull($GLOBALS['doublons_documents_inclus'] ?? null);
+        $this->assertNull($GLOBALS['doublons_albums_inclus'] ?? null);
+
+        $texte = 'Mon texte <doc1> <img2> <emb3> et paf les doublons avec un album <album4> ?';
+        traiter_modeles($texte, ['albums' => ['album']]);
+        $this->assertNull($GLOBALS['doublons_documents_inclus'] ?? null);
+        $this->assertNotEmpty($GLOBALS['doublons_albums_inclus']);
+        $this->assertEquals([4], $GLOBALS['doublons_albums_inclus']);
+    }
 }