Problem/Motivation

When you resume a learning path, but there's no next step, you get a fatal error. This is because the type of OpignoGroupManagedContent::getFirstStep() isn't being checked.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 3261546.patch887 bytescatch

Comments

catch created an issue. See original summary.

catch’s picture

catch
he/him
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new 3261546.patch887 bytes

Initial patch.

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Needs work

The error is a symptom of a data issue which can happen when building learning paths via the learning path manager.

When modules are being added/removed/re-ordered (unfortunately we don't have steps to reproduce yet), you can end up with a 'phantom' link in the database, which isn't shown in the manager UI at all. This link gets loaded when building the steps and you then get errors. The workaround to remove it is to remove a module from the learning path and add it back again, which clears it up, but would obviously be good to fix the root cause.