Repository navigation
#982 [Registre] fix: chemins du registre publiés tels qu'installés, lisibles par Saturne < 23.3.0 - #983
Merged
nicolas-eoxia merged 1 commit intoNov 2, 2026
Conversation
…lles, lisibles par Saturne < 23.3.0 Depuis le retrait de custom/, saturneExtendGetObjectsMetadata() publiait 'dolimeet/...' : seuls les lecteurs passant par dol_buildpath() le resolvent. Saturne < 23.3.0 et les anciens lecteurs prefixent DOL_DOCUMENT_ROOT : fatale sur toute page qui appelle le registre, y compris celles des autres modules. Le chemin est maintenant calcule depuis l'emplacement reel du module, relatif a DOL_DOCUMENT_ROOT : custom/dolimeet sur une installation standard, lu par tous les lecteurs. Le nom nu ne sert qu'a un module installe hors de la racine principale, seul cas que visait le retrait de custom/. 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.
Constat
saturneExtendGetObjectsMetadata()publiait les chemins du registre sans préfixe ('dolimeet/class/…'), depuis le retrait decustom/. Seuls les lecteurs qui passent pardol_buildpath()savent les résoudre. Les anciens lecteurs ajoutentDOL_DOCUMENT_ROOTdevant le chemin : c'est le cas de Saturne avant 23.3.0, et de digiquali ou dolicar d'avant la phase A. Toute page qui appelle le registre tombe alors en fatale, y compris les pages des autres modules. Exemple, la liste des risques de Digirisk :Aucun module ne déclare de version minimale de Saturne : rien n'empêchait cette combinaison chez un client.
Correctif
Plutôt que d'imposer une version, on supprime la dépendance : le chemin est calculé depuis l'emplacement réel du module, relatif à
DOL_DOCUMENT_ROOT.dol_buildpath)htdocs/custom/dolimeet)custom/dolimeet/…htdocs/dolimeetdolimeet/…htdocsdolimeet/…(comme aujourd'hui)On garde l'objectif du retrait de
custom/(le cas de la racine alternative) sans perdre la compatibilité. Le même correctif est appliqué aux cinq producteurs du registre : Evarisk/digiquali#2715, Evarisk/Digirisk#5658, Evarisk/dolicar#556, Eoxia/reedcrm#1198, #982.Vérification
Bac à sable Dolibarr 23.0.3, avec les six modules actifs et la page de diagnostic qui appelle
saturne_get_objects_metadata():develophtdocs/digiquali/class/question.class.php), liste des risques Digirisk casséeDOL_DOCUMENT_ROOTet pardol_buildpath(), liste des risques OK (18 lignes), journal videcreate_url/list_urlrépondent en 200 (302 pour une liste qui redirige)DoliMeet : son producteur ne tourne que si un contexte DoliMeet est déjà chargé sur la page (
globalcard, présent sur la plupart des fiches). Mesures prises avec ce contexte, page de diagnostic :develop(les autres corrigés)htdocs/dolimeet/class/trainingsession.class.phpdolimeet_trainsess; aucune fatale, journal videsession_card.php?action=create&object_type=trainingsessionrépond en 200Closes #982
🤖 Generated with Claude Code