Fixed #7242 - File Question - An empty preview appears when the onDow…

…nloadFiles event returns with the options.callback('error')
surveyjs · Nov 7, 2023 · eedf9c8 · eedf9c8
1 parent 004b949
commit eedf9c8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/src/question_file.ts b/src/question_file.ts
@@ -622,7 +622,9 @@ export class QuestionFileModel extends Question {
         if (this.survey) {
           this.survey.uploadFiles(this, this.name, files, (status, data) => {
             if (status === "error") {
+              this.errors.push(new UploadingFileError(data, this));
               this.stateChanged("error");
+              this.stateChanged("loaded");
             }
             if (status === "success") {
               this.value = (this.value || []).concat(

diff --git a/tests/questionFileTests.ts b/tests/questionFileTests.ts
@@ -481,7 +481,7 @@ QUnit.test(
             })
           );
         } else {
-          options.callback("error");
+          options.callback("error", "custom error text");
         }
       }, 1);
     });
@@ -504,8 +504,10 @@ QUnit.test(
     setTimeout(() => {
       assert.ok(q1.isEmpty());
       assert.equal(q1.value, undefined);
-      assert.equal(stateSec, "->loading->error");
-      assert.equal(state, "error");
+      assert.equal(stateSec, "->loading->error->loaded");
+      assert.equal(q1.errors.length, 1, "Has errors");
+      assert.equal(q1.errors[0].text, "custom error text", "Error text");
+      assert.equal(state, "loaded");
       done();
 
       isSuccess = true;