Repository navigation
[FIX] #48689 Failed test: Einleitungsseite bearbeiten - Learning Map einfügen - #12386
Merged
alex40724 merged 1 commit intoNov 5, 2026
Conversation
Adaptive Learning Paths replaced the Curriculum page element by LearningMap in ilias_pg_9.dtd. ilias_pg_12.dtd was created from the version before that change and became the active DTD, so pages with a learning map failed validation on insert and save. The DTD now declares LearningMap instead of Curriculum, as ilias_pg_9.dtd already does.
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.
Mantis: https://mantis.ilias.de/view.php?id=48689
Cause
Adaptive Learning Paths (#12210) replaced the
Curriculumpage element byLearningMapinilias_pg_9.dtd. One day later,ilias_pg_12.dtdwas added in "Questions: Build General Structure" (e3e0208), based on the version before that change, and became the active DTD inilPageObject. Since then, inserting a learning map into a page fails DTD validation.Change
ilias_pg_12.dtddeclaresLearningMapinstead ofCurriculum, asilias_pg_9.dtdalready does. There is noCurriculumpage component left, and the ALP migration converts existing pages toLearningMap.Tested
Validated a page containing
<LearningMap/>withDOMDocument::validate()against the old and the new DTD: invalid before, valid after.