Skip to content

Enable LOM metadata support for Learning Sequences#11675

Open
kevenClausenKPG wants to merge 4 commits into
ILIAS-eLearning:trunkfrom
kroepelin-projekte:trunk_lso_metadata
Open

Enable LOM metadata support for Learning Sequences#11675
kevenClausenKPG wants to merge 4 commits into
ILIAS-eLearning:trunkfrom
kroepelin-projekte:trunk_lso_metadata

Conversation

@kevenClausenKPG

@kevenClausenKPG kevenClausenKPG commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

This PR completes Learning Object Metadata (LOM) support for the Learning Sequence following components/ILIAS/MetaData/docs/enabling_lom.md.

It includes:

  • Lifecycle wiring: call createMetaData(), updateMetaData(), and deleteMetaData() in the LSO object lifecycle
  • UI: always expose the Meta Data tab via ilObjectMetaDataGUI::getTab() (no CUSTOM_METADATA gating) and show LOM sections on the info screen via addMetaDataSections(..., 0, 'lso')
  • Import/Export: add importer mapping for components/ILIAS/MetaData (md) so LOM sets are correctly remapped on import
  • Setup: add a migration to initialize LOM sets for pre-existing Learning Sequences

https://docu.ilias.de/go/wiki/wpage_7807_1357

@kevenClausenKPG kevenClausenKPG self-assigned this Jun 16, 2026
@kevenClausenKPG kevenClausenKPG added the php Pull requests that update Php code label Jun 16, 2026
@kevenClausenKPG

Copy link
Copy Markdown
Contributor Author

Hey @schmitz-ilias , could you please take a look at my metadata implementation for the LSO and give me some feedback?

@schmitz-ilias schmitz-ilias left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @kevenClausenKPG,

everything related to Metadata looks good all in all, just a few points:

  • The new InitLOMForLearningSequenceMigration needs to be added to ilLearningSequenceSetupAgent::getMigrations, otherwise it will not be shown.
  • It's a bit strange that there is no LuceneObjectDefinition.xml for Learning Sequences at all, I guess Lucene support for the object was just never implemented. Maybe a small project for another day ^^
  • AdvancedMetaData also needs to be added as a tail dependency in ilLearningSequenceExporter (similar to LOM), see for an example ilGroupExporter.
  • Unrelated to Metadata: I haven't looked at this too deeply, but the changes in ilObjLearningSequenceSettingsGUI::update look fishy to me. It seems like the form input related to title, description, and online-status will not be processed anymore?

Note that Taxonomies are currently mostly independent of MetaData, so I can't review that part of the implementation. Let me know if anything is unclear!

Cheers, Tim

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jour fixe php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants