Repository navigation
#509 [CI] fix: sortir les stubs de test de saturne, et corriger ce qu'ils masquaient - #510
Merged
nicolas-eoxia merged 1 commit intoSep 25, 2026
Conversation
…r ce qu'ils masquaient phpstan.neon prenait ../saturne/ en entier : les bouchons DoliDB, Conf, Translate, User et HookManager du bootstrap PHPUnit gagnaient sur les vraies classes du coeur en CI, et l'analyse etait plus verte qu'elle ne devrait. Les cinq erreurs ainsi revelees sont corrigees plutot que gelees, la baseline n'a pas bouge : - admin/setup.php testait $action == 'update' a l'interieur du bloc qui teste deja cette egalite ; - l'API deferencait fetch_object() sans garde ; - trois GETPOST non types partaient dans dol_mktime(). Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #509
phpstan.neonprenait../saturne/en entier dansscanDirectories,tests/compris. Le bootstrap PHPUnit du socle y déclare des bouchonsDoliDB,Conf,Translate,UseretHookManagerqui gagnent sur les vraies classes du coeur dans le sparse-checkout de la CI : les appels qu'ils ne connaissent pas ne sont plus résolus, et des erreurs réelles disparaissent de l'analyse. Le piège est qu'un tel trou ne fait jamais rougir la CI.Les cinq erreurs démasquées sont corrigées, pas gelées — la baseline n'a pas bougé :
admin/setup.php:106Loose comparison using == between 'update' and 'update' will always evaluate to trueclass/api_dolicar.class.php:176view/registrationcertificatefr/quickcreation.php:324dol_mktime expects int, string givenGETPOSTINTAnalyse rejouée en local avec le phar PHPStan et PHP 8.1, sur le périmètre complet : No errors.
Dernier module du parc à recevoir cette exclusion, après Digirisk (#5266), DigiQuali (#2628), DoliMeet (#929), DoliSIRH (#696), DigiBoard (#53) et EasyURL (#152).
🤖 Generated with Claude Code