numbas / numbas-extension-geogebra

An extension for Numbas which integrates GeoGebra materials
Apache License 2.0
1 stars 0 forks source link

Suspend/resume code doesn't handle the ggbapplet data type variables correctly. #5

Closed georgestagg closed 2 years ago

georgestagg commented 4 years ago

Resuming a paused exam gives:

Numbas has failed
Sorry, Numbas has encountered an error which means it can't continue. Below is a description of the error.
Numbas Error: Error loading question 2: Variable geogebra is not defined.

Steps to reproduce.

  1. Make an exam which uses a question that has a ggbapplet variable, for example: Christian's https://numbas.mathcentre.ac.uk/question/68262/mark-a-part-as-correct-if-the-student-moves-a-point-to-the-right-position/
  2. Download exam package as zip.
  3. In Blackboard, create an assignment / open Numbas LTI provider.
  4. In Settings, replace/upload the exam package.
  5. In Blackboard, switch to student view.
  6. As student, click on the assignment and start an attempt.
  7. As student, pause exam and close the 'Paused' window.
  8. As student, in Blackboard, click on the assignment to open Numbas LTI provider.
  9. As student, click on 'Continue' button to continue previous attempt.
  10. Question hangs on loading.

Alternately, the hang also occurs if the instructor tries to review the student's attempt. If the question is rewritten to use the old method of loading the ggbapplet, the question will still hang unless you actually delete the ggbapplet variable.

christianp commented 2 years ago

I think this was fixed by e3865765e7ef3a35a0639c96b4f983a1968e3e24