Skip to content

Learning path: Support SCORM 2004 progress measure - #8682

Closed
christianbeeznest wants to merge 4 commits into
chamilo:masterfrom
christianbeeznest:GH-8670
Closed

Learning path: Support SCORM 2004 progress measure#8682
christianbeeznest wants to merge 4 commits into
chamilo:masterfrom
christianbeeznest:GH-8670

Conversation

@christianbeeznest

Copy link
Copy Markdown
Contributor

This PR adds partial SCORM 2004 support for cmi.progress_measure.

Changes

  • Adds c_lp_item_view.progress.
  • Maps progress in CLpItemView.
  • Adds GetValue/SetValue support for cmi.progress_measure.
  • Stores the SCO-reported progress as a 0..1 value.
  • Does not force progress to 1.0 when completion_status is completed.
  • Updates LP progress calculation to use progress_measure for SCORM 2004 items.
  • Supports single-SCO and multi-SCO progress calculation.
  • Exposes API_1484_11 for SCORM 2004 SCOs.

Tests

  • Tested with a minimal SCORM 2004 single-SCO package.
  • Verified SetValue("cmi.progress_measure", 0.8) stores progress = 0.8.
  • Verified completed status does not overwrite progress to 1.0.
  • Verified LP progress uses the stored progress_measure value.

@christianbeeznest

Copy link
Copy Markdown
Contributor Author

Closing this PR without merging.

The SCORM 2004 progress_measure support has already been covered in the current master through the migrated Learning Path Vue/Symfony runtime, including the V210 migration.

This older PR was created before the LP migration and now mainly affects the legacy runtime, while also adding a duplicated V200 migration. If legacy runtime support is still needed, it would be cleaner to open a new PR from the current master with only the remaining legacy-specific changes.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant