Discuter de SPIP

[formidable_tablesorter ↪ refacto_amelioration_diverses] 20 commits

Notifications Commit Zone

SPIP-contrib (SPIP-contrib) Juillet 17, 2025, 9:13 1

spip-contrib-extensions/formidable_tablesorter | 20 commits

Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

change: HeaderBuilderInterface renvoie une liste d’Headers (HeadersList).

Le but à terme est d’avoir des saisies qui renvoient plusieurs colonnes.

Ajouté
Formidable_ts/HeadersList.php
Formidable_ts/TraitList.php
Modifié
Formidable_ts/CellsList.php
Formidable_ts/HeaderBuilder/Champ.php
Formidable_ts/HeaderBuilder/Extra.php
Formidable_ts/HeaderBuilder/HeaderBuilderInterface.php
Formidable_ts/HeaderBuilder/Natif.php
Formidable_ts/HeaderBuilder/Natif/DateEnvoi.php
Formidable_ts/HeaderBuilder/Natif/IdAuteur.php
Formidable_ts/HeaderBuilder/Natif/IdFormulairesReponse.php
Formidable_ts/HeaderBuilder/Natif/Ip.php
Formidable_ts/HeaderBuilder/Natif/Statut.php
Formidable_ts/Table.php

Détails : change: `HeaderBuilderInterface` renvoie une liste d'Headers (`HeadersList`). (f37922b4) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

change: l’interface CellBuilder renvoi plutôt une liste de cellul, permettra ainsi d’avoir certaines saisies sur plusieurs colonnes

Modifié
Formidable_ts/CellBuilder/CellBuilderInterface.php
Formidable_ts/CellBuilder/Champ.php
Formidable_ts/CellBuilder/Extra.php
Formidable_ts/CellBuilder/Natif.php
Formidable_ts/CellBuilder/Natif/DateEnvoi.php
Formidable_ts/CellBuilder/Natif/IdAuteur.php
Formidable_ts/CellBuilder/Natif/IdFormulairesReponse.php
Formidable_ts/CellBuilder/Natif/Ip.php
Formidable_ts/CellBuilder/Natif/Statut.php
Formidable_ts/Table.php

Détails : change: l'interface CellBuilder renvoi plutôt une liste de cellul, permettra... (2fdf2384) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

refactor: une seule fonction pour déterminer le nom de la colonne en fonction de la saisie, on anticipe un peu sur le fait qu’on va avoir des saisies en plusieurs colonnes

Modifié
Formidable_ts/Table.php

Détails : refactor: une seule fonction pour déterminer le nom de la colonne en fonction... (24e92823) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

feat: une class CellsList pour stocker une liste de cellules

Ajouté
Formidable_ts/CellsList.php

Détails : feat: une class `CellsList` pour stocker une liste de cellules (aee9da88) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

refactor: tri des cellules

On calcule la valeur de tri uniquement si on tri sur la colonne
(perf)
La méthode de calcul est internalisée à la classe \Formidable_ts\Cell
On peut déléguer à une méthode statique \Formidable_ts\Sorter\<type_de_saisie>::getSortValue, par exemple pour la saisie evenements qui doit récupérer la date de début de l’évènement
En contrepartie, on passe aux objets de la class Cell une valeur
brute (raw_value) à l’instanciation.

Ajouté
Formidable_ts/Sorter/Evenements.php
Modifié
Formidable_ts/Cell.php
Formidable_ts/CellBuilder/Champ.php
Formidable_ts/CellBuilder/Extra.php
Formidable_ts/CellBuilder/Natif/DateEnvoi.php
Formidable_ts/CellBuilder/Natif/IdAuteur.php
Formidable_ts/CellBuilder/Natif/IdFormulairesReponse.php
Formidable_ts/CellBuilder/Natif/Ip.php
Formidable_ts/CellBuilder/Natif/Statut.php
Formidable_ts/Table.php

Détails : refactor: tri des cellules (22192431) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

remove: getter generique sur la classe Cell. Si on veut accéder à une propriété, on la déclare commme accessible plus sûr

Modifié
Formidable_ts/Cell.php
Formidable_ts/Header.php
Formidable_ts/Table.php

Détails : remove: getter generique sur la classe `Cell`. Si on veut accéder à une... (5f74f75d) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

remove: le getter de la classe Table; si on veut pouvoir obtenir publiquement une propriété, on la déclare publique

Modifié
Formidable_ts/Table.php

Détails : remove: le getter de la classe `Table`; si on veut pouvoir obtenir... (80e060ff) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

remove: code inutile, vu que par défaut $data=$valeur

Modifié
Formidable_ts/Table.php

Détails : remove: code inutile, vu que par défaut `$data=$valeur` (65a970b4) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

refactor: simplifier, un sql_in fait très bien l’affaire même sur une valeur unitaire

Modifié
Formidable_ts/Table.php

Détails : refactor: simplifier, un sql_in fait très bien l'affaire même sur une valeur unitaire (ebc23147) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

remove: code mort

Modifié
formidable_ts_pipelines.php

Détails : remove: code mort (bc97f2c8) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

perf: SPIP 4.2 inclut de base un polyfill pour mbstring

Modifié
Formidable_ts/Table.php

Détails : perf: SPIP 4.2 inclut de base un polyfill pour mbstring (5a5f45c7) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

refactor: pour tout ce qui est filtre, stocker dans les classes PHP selon le nom, et ne renvoyer selon l’ordre qu’au moment de la génération du json final (plus clair pour s’y retrouver, évite de passer des infos)

Modifié
Formidable_ts/Cell.php
Formidable_ts/CellBuilder/CellBuilderInterface.php
Formidable_ts/CellBuilder/Champ.php
Formidable_ts/CellBuilder/Extra.php
Formidable_ts/CellBuilder/Natif.php
Formidable_ts/CellBuilder/Natif/DateEnvoi.php
Formidable_ts/CellBuilder/Natif/IdAuteur.php
Formidable_ts/CellBuilder/Natif/IdFormulairesReponse.php
Formidable_ts/CellBuilder/Natif/Ip.php
Formidable_ts/CellBuilder/Natif/Statut.php
Formidable_ts/Table.php

Détails : refactor: pour tout ce qui est filtre, stocker dans les classes PHP selon le... (93814389) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

fix: bien nommer la cellule

Modifié
Formidable_ts/CellBuilder/Natif/IdAuteur.php

Détails : fix: bien nommer la cellule (e75e0400) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

refactor: utiliser saisies_saisie_est_champ() pour éliminer les saisies de type explication

Modifié
Formidable_ts/Table.php
paquet.xml

Détails : refactor: utiliser `saisies_saisie_est_champ()` pour éliminer les saisies de type explication (4f112ce5) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

refactor: mettre tout ce qui est construction des header dans des interfaces HeaderBuilder

Ajouté
Formidable_ts/HeaderBuilder/Champ.php
Formidable_ts/HeaderBuilder/Extra.php
Formidable_ts/HeaderBuilder/HeaderBuilderInterface.php
Formidable_ts/HeaderBuilder/Natif.php
Formidable_ts/HeaderBuilder/Natif/DateEnvoi.php
Formidable_ts/HeaderBuilder/Natif/IdAuteur.php
Formidable_ts/HeaderBuilder/Natif/IdFormulairesReponse.php
Formidable_ts/HeaderBuilder/Natif/Ip.php
Formidable_ts/HeaderBuilder/Natif/NatifInterface.php
Formidable_ts/HeaderBuilder/Natif/Statut.php
Formidable_ts/HeaderBuilder/TraitSaisieInfo.php
Modifié
Formidable_ts/Table.php

Détails : refactor: mettre tout ce qui est construction des header dans des interfaces HeaderBuilder (63913454) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

refactor: convention de nommage - ce qui genere quelque choses va s’appeler Builder + varia

Ajouté
Formidable_ts/CellBuilder/CellBuilderInterface.php
Formidable_ts/CellBuilder/Champ.php
Formidable_ts/CellBuilder/Extra.php
Formidable_ts/CellBuilder/Natif.php
Formidable_ts/CellBuilder/Natif/DateEnvoi.php
Formidable_ts/CellBuilder/Natif/IdAuteur.php
Formidable_ts/CellBuilder/Natif/IdFormulairesReponse.php
Formidable_ts/CellBuilder/Natif/Ip.php
Formidable_ts/CellBuilder/Natif/NatifInterface.php
Formidable_ts/CellBuilder/Natif/Statut.php
Supprimé
Formidable_ts/Request/Natif/NatifInterface.php
Modifié
Formidable_ts/Table.php

Détails : refactor: convention de nommage - ce qui genere quelque choses va s'appeler `Builder` + varia (437e9626) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

remove: variable inutilisée

Modifié
Formidable_ts/Table.php

Détails : remove: variable inutilisée (4a47a89e) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

fix: typer retour de méthode interne

Modifié
Formidable_ts/Request/Natif.php
Formidable_ts/Table.php

Détails : fix: typer retour de méthode interne (5a9f7bf0) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

style: utiliser les formes raccourcies des chemins de Classe PHP

Modifié
Formidable_ts/Request/Champ.php
Formidable_ts/Request/Extra.php
Formidable_ts/Request/Natif.php
Formidable_ts/Request/Natif/DateEnvoi.php
Formidable_ts/Request/Natif/IdAuteur.php
Formidable_ts/Request/Natif/IdFormulairesReponse.php
Formidable_ts/Request/Natif/Ip.php
Formidable_ts/Request/Natif/Statut.php
Formidable_ts/Request/RequestInterface.php

Détails : style: utiliser les formes raccourcies des chemins de Classe PHP (453386da) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab

==============================
Par Maïeul Rouquette, le 17 juillet 2025 à 23h12min :

refactor: une classe par type de champ natif

Ajouté
Formidable_ts/Request/Natif/DateEnvoi.php
Formidable_ts/Request/Natif/IdAuteur.php
Formidable_ts/Request/Natif/IdFormulairesReponse.php
Formidable_ts/Request/Natif/Ip.php
Formidable_ts/Request/Natif/NatifInterface.php
Formidable_ts/Request/Natif/Statut.php
Modifié
Formidable_ts/Request/Natif.php

Détails : refactor: une classe par type de champ natif (86803f2f) · Validations · spip-contrib-extensions / formidable_tablesorter · GitLab