MOBILE-4690 quiz: Wait the questions to load before validating

Author: crazyserver

src/addons/qtype/ddimageortext/component/ddimageortext.ts

@@ -45,18 +45,23 @@ export class AddonQtypeDdImageOrTextComponent
      */
     init(): void {
         if (!this.question) {
+            this.onReadyPromise.resolve();
+
             return;
         }
 
         const questionElement = this.initComponent();
         if (!questionElement) {
+            this.onReadyPromise.resolve();
+
             return;
         }
 
         // Get D&D area and question text.
         const ddArea = questionElement.querySelector('.ddarea');
         if (!ddArea) {
             this.logger.warn('Aborting because of an error parsing question.', this.question.slot);
+            this.onReadyPromise.resolve();
 
             return CoreQuestionHelper.showComponentError(this.onAbort);
         }
@@ -84,6 +89,7 @@ export class AddonQtypeDdImageOrTextComponent
         }
 
         this.question.loaded = false;
+        this.onReadyPromise.resolve();
     }
 
     /**

src/addons/qtype/ddmarker/component/ddmarker.ts

@@ -51,11 +51,15 @@ export class AddonQtypeDdMarkerComponent
      */
     init(): void {
         if (!this.question) {
+            this.onReadyPromise.resolve();
+
             return;
         }
 
         const questionElement = this.initComponent();
         if (!questionElement) {
+            this.onReadyPromise.resolve();
+
             return;
         }
 
@@ -65,6 +69,7 @@ export class AddonQtypeDdMarkerComponent
 
         if (!ddArea || !ddForm) {
             this.logger.warn('Aborting because of an error parsing question.', this.question.slot);
+            this.onReadyPromise.resolve();
 
             return CoreQuestionHelper.showComponentError(this.onAbort);
         }
@@ -107,6 +112,7 @@ export class AddonQtypeDdMarkerComponent
         }
 
         this.question.loaded = false;
+        this.onReadyPromise.resolve();
     }
 
     /**

src/addons/qtype/ddwtos/component/ddwtos.ts

@@ -47,11 +47,15 @@ export class AddonQtypeDdwtosComponent extends CoreQuestionBaseComponent<AddonMo
      */
     init(): void {
         if (!this.question) {
+            this.onReadyPromise.resolve();
+
             return;
         }
 
         const questionElement = this.initComponent();
         if (!questionElement) {
+            this.onReadyPromise.resolve();
+
             return;
         }
 
@@ -65,6 +69,7 @@ export class AddonQtypeDdwtosComponent extends CoreQuestionBaseComponent<AddonMo
         const answerContainer = questionElement.querySelector('.answercontainer');
         if (!answerContainer) {
             this.logger.warn('Aborting because of an error parsing question.', this.question.slot);
+            this.onReadyPromise.resolve();
 
             return CoreQuestionHelper.showComponentError(this.onAbort);
         }
@@ -98,6 +103,7 @@ export class AddonQtypeDdwtosComponent extends CoreQuestionBaseComponent<AddonMo
         this.question.text = questionText;
 
         this.question.loaded = false;
+        this.onReadyPromise.resolve();
     }
 
     /**

src/addons/qtype/description/component/description.ts

@@ -36,12 +36,16 @@ export class AddonQtypeDescriptionComponent extends CoreQuestionBaseComponent {
     init(): void {
         const questionEl = this.initComponent();
         if (!questionEl) {
+            this.onReadyPromise.resolve();
+
             return;
         }
 
         // Get the "seen" hidden input.
         const input = questionEl.querySelector<HTMLInputElement>('input[type="hidden"][name*=seen]');
         if (!input) {
+            this.onReadyPromise.resolve();
+
             return;
         }
 

src/addons/qtype/essay/component/essay.ts

@@ -45,6 +45,8 @@ export class AddonQtypeEssayComponent extends CoreQuestionBaseComponent<AddonMod
      */
     init(): void {
         if (!this.question) {
+            this.onReadyPromise.resolve();
+
             return;
         }
 
@@ -57,6 +59,7 @@ export class AddonQtypeEssayComponent extends CoreQuestionBaseComponent<AddonMod
         if (this.question?.allowsAttachments && this.uploadFilesSupported && !this.review) {
             this.loadAttachments();
         }
+        this.onReadyPromise.resolve();
     }
 
     /**

src/addons/qtype/gapselect/component/gapselect.ts

@@ -36,6 +36,7 @@ export class AddonQtypeGapSelectComponent extends CoreQuestionBaseComponent {
      */
     init(): void {
         this.initOriginalTextComponent('.qtext');
+        this.onReadyPromise.resolve();
     }
 
     /**

src/addons/qtype/match/component/match.ts

@@ -35,6 +35,7 @@ export class AddonQtypeMatchComponent extends CoreQuestionBaseComponent<AddonMod
      */
     init(): void {
         this.initMatchComponent();
+        this.onReadyPromise.resolve();
     }
 
 }

src/addons/qtype/multianswer/component/multianswer.ts

@@ -35,6 +35,7 @@ export class AddonQtypeMultiAnswerComponent extends CoreQuestionBaseComponent {
      */
     init(): void {
         this.initOriginalTextComponent('.formulation');
+        this.onReadyPromise.resolve();
     }
 
     /**

src/addons/qtype/multichoice/component/multichoice.ts

@@ -35,6 +35,7 @@ export class AddonQtypeMultichoiceComponent extends CoreQuestionBaseComponent<Ad
      */
     init(): void {
         this.initMultichoiceComponent();
+        this.onReadyPromise.resolve();
     }
 
     /**

src/addons/qtype/numerical/component/numerical.ts

@@ -35,6 +35,7 @@ export class AddonQtypeNumericalComponent extends CoreQuestionBaseComponent<Addo
      */
     init(): void {
         this.initCalculatedComponent();
+        this.onReadyPromise.resolve();
     }
 
 }