Merge pull request #4713 from rtibbles/exam_update

Don't give misleading message to learners on exam submission.
learningequality · Jan 11, 2019 · f33f32b · f33f32b
2 parents 3419aa3 + 11bc500
commit f33f32b
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/kolibri/plugins/learn/assets/src/views/ExamPage/index.vue b/kolibri/plugins/learn/assets/src/views/ExamPage/index.vue
@@ -218,7 +218,8 @@
         return Promise.resolve();
       },
       goToQuestion(questionNumber) {
-        this.saveAnswer(true).then(() => {
+        const promise = this.debouncedSetAndSaveCurrentExamAttemptLog.flush() || Promise.resolve();
+        promise.then(() => {
           this.$router.push({
             name: ClassesPageNames.EXAM_VIEWER,
             params: {
@@ -230,6 +231,15 @@
         });
       },
       toggleModal() {
+        // Flush any existing save event to ensure
+        // that the subit modal contains the latest state
+        if (!this.submitModalOpen) {
+          const promise =
+            this.debouncedSetAndSaveCurrentExamAttemptLog.flush() || Promise.resolve();
+          return promise.then(() => {
+            this.submitModalOpen = !this.submitModalOpen;
+          });
+        }
         this.submitModalOpen = !this.submitModalOpen;
       },
       finishExam() {