Repository navigation
#984 [Mod] feat: version minimale de Saturne, activation refusée sur un Saturne trop ancien - #985
Merged
nicolas-eoxia merged 1 commit intoNov 2, 2026
Conversation
…ee sur un Saturne trop ancien Le module depend de Saturne sans version : un Saturne plus ancien que celui avec lequel cette release est testee donne une fatale (fonction saturne_* absente) au lieu d'un refus. Le descripteur declare need_saturne_version, au format de need_dolibarr_version. init() lit la version de Saturne dans son descripteur et refuse, avant toute ecriture, avec le message du coeur activateModuleDependNotSatisfied : la garde est dans le module, pour marcher justement sur un Saturne qui ne la connait pas. Saturne absent est couvert aussi. Cote Saturne : Evarisk/Saturne#1843. Co-Authored-By: Claude Opus 5.5 <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.
Changement
Le module dépendait de Saturne sans version minimale. Or le cœur ne sait pas en vérifier une, et chaque release appelle des fonctions récentes de Saturne. Sur un Saturne trop ancien, on obtenait une fatale au lieu d'un refus.
public $need_saturne_version = [23, 6];, au même format queneed_dolibarr_version. 23.6 est la version de Saturne avec laquelledevelopest testé.init(), avant toute écriture en base, la version de Saturne est lue dans son descripteur (modSaturne->version). Si elle est plus ancienne, l'activation est refusée avec le message du cœuractivateModuleDependNotSatisfied: « … dépend du module « modSaturne >= 23.6 (version installée) » ». La garde est dans le module, pour marcher même sur un Saturne qui ne la connaît pas. Elle couvre aussi le cas où Saturne est absent.Côté Saturne, la page d'accueil saute la mise à jour automatique tant que Saturne est trop ancien : Evarisk/Saturne#1845.
Vérification
Bac à sable Dolibarr 23.0.3 :
init()réel sur Saturne 23.2.1 : refus (-1), message traduit, aucune écriture (CHECKSUMdes tables de module identiques avant et après) ;Closes #984
🤖 Generated with Claude Code