Skip to content

[Mod] chore: version 23.1.0 - #507

Merged
nicolas-eoxia merged 1 commit into
Evarisk:developfrom
nicolas-eoxia:release/23.1.0
Sep 25, 2026
Merged

nicolas-eoxia merged 1 commit into
Evarisk:developfrom
nicolas-eoxia:release/23.1.0

Conversation

@nicolas-eoxia

Copy link
Copy Markdown
Contributor

Préparation de la 23.1.0 : bump des trois fichiers de version, notes réécrites, et deux garde-fous de la chaîne qualité.

  • core/modules/modDoliCar.class.php et core/triggers/interface_99_modDoliCar_DoliCarTriggers.class.php → 23.1.0
  • README.md → version, date, bornes alignées sur le descripteur (Dolibarr 23.0.0 - 24.0.0 au lieu de 21.0.0 - 23.0.2) et Saturne Framework 23.2.1
  • RELEASE_NOTES.md réécrit

Minor : la version apporte sept fonctionnalités (carnet de bord, historique véhicule, création rapide, contrôle public).

Les deux garde-fous, tous deux vérifiés ailleurs aujourd'hui :

  • le trigger redéclare $version avec son type. Sans cela, PHPStan la refuse — le PHPDoc du coeur la type comme une énumération — et la baseline fige le numéro, ce qui casse la CI au bump suivant (arrivé sur reedcrm et digiquali) ;
  • phpstan.neon exclut ../saturne/tests/, dont le bootstrap PHPUnit déclare des bouchons DoliDB, Conf, Translate, User et HookManager qui gagnent sur les vraies classes du coeur en CI.

Smoke test : 14 parcours sur 14 au vert sur un bac à sable monté sur cette branche, avec le zip Saturne 23.2.1 publié déposé par-dessus. Ces parcours n'existaient pas, ils viennent d'être écrits à partir des entrées de menu du descripteur.

🤖 Generated with Claude Code

Le trigger redeclare $version avec son type : sans cette redeclaration PHPStan
la refuse, la baseline fige le numero, et la release suivante casse la CI.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@nicolas-eoxia

Copy link
Copy Markdown
Contributor Author

J'ai retiré de cette PR l'exclusion ../saturne/tests/ que j'y avais mise : elle démasque cinq erreurs réelles, qui n'ont rien à faire dans une PR de release.

Elles sont toutes dans le périmètre du module, et méritent d'être corrigées plutôt que gelées :

  • Loose comparison using == between 'update' and 'update' will always evaluate to true — une comparaison qui ne teste rien ;
  • Cannot access property $rowid on object|false — un fetch_object() déréférencé sans garde ;
  • trois Parameter #4/#5/#6 of function dol_mktime expects int, string given — des GETPOST non typés.

Elles sont invisibles aujourd'hui parce que le bootstrap PHPUnit de Saturne déclare des bouchons DoliDB, Conf, Translate, User et HookManager qui gagnent sur les vraies classes du coeur en CI. Suivi dans une issue dédiée.

@nicolas-eoxia
nicolas-eoxia merged commit 86013ae into Evarisk:develop Sep 25, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant