Skip to content

[FIX] LTI: Provide outcome status lists#11681

Merged
Saaweel merged 1 commit into
ILIAS-eLearning:release_10from
surlabs:ilias10_LTI_status_lists
Jun 16, 2026
Merged

[FIX] LTI: Provide outcome status lists#11681
Saaweel merged 1 commit into
ILIAS-eLearning:release_10from
surlabs:ilias10_LTI_status_lists

Conversation

@Saaweel

@Saaweel Saaweel commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Description

This backports the LTI outcome learning progress status list fix to ILIAS 10.

It combines the changes from #11641 and #11667 in one PR:

  • Adds _getInProgress(), _getCompleted(), _getFailed() and _getNotAttempted() for LTI outcome LP status.
  • Derives the status lists from the underlying LTI outcome data instead of ut_lp_marks.
  • Uses the existing determineStatus() logic to classify users.
  • Caches the LTI consumer object per request while calculating the lists.

This keeps parent object / LP collection aggregation working while avoiding a circular dependency during ilLPStatusWrapper::_refreshStatus().

Related

Backport of:

Mantis: https://mantis.ilias.de/view.php?id=41919

@Saaweel Saaweel self-assigned this Jun 16, 2026
@Saaweel Saaweel added bugfix php Pull requests that update Php code labels Jun 16, 2026
@Saaweel Saaweel merged commit 134ed10 into ILIAS-eLearning:release_10 Jun 16, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant