Simon-Initiative / course-digest

Tool to produce a summary or digest of OLI course package contents
MIT License
2 stars 0 forks source link

[MER-2183] [MER-2147] Statistics input ingestion fixes #171

Closed marc-hughes closed 1 year ago

marc-hughes commented 1 year ago

This fixes two related issues.

MER-2147

Sometimes, the input xml would have an extra, redundant input_ref in the stem of multiple choice questions referencing the multiple choice question itself.

When this happened:

  1. There would be an extra text-input field in preview / delivery modes that didn't do anything
  2. The stem of that question would not be editable in the authoring environment since an error would appear instead of the rich text editor.

To keep this change as targeted as possible, it only strips any input_ref in the stem of an MCQ that matches the MCQ id.

Fixes: https://github.com/Simon-Initiative/oli-torus/issues/3601

MER-2183

If we try to ingest a short answer question that does not have a part id assigned, it’s not handled correctly and the resulting question will have no part ID.

In delivery and preview, this causes the question to error out on validation and there is no user-feedback.

This can be seen in the last question of the gsu stats page “Behavior of Sample Mean (3 of 3)”