The SurveyModel.progressValue property returns 0 when a user selects …

…a matrix row value fix #9530
surveyjs · Feb 27, 2025 · 7ec8ebe · 7ec8ebe
1 parent c2dec5c
commit 7ec8ebe
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/packages/survey-core/src/survey.ts b/packages/survey-core/src/survey.ts
@@ -6183,6 +6183,7 @@ export class SurveyModel extends SurveyElementCore
         question.onSurveyValueChanged(newValue);
       }
     }
+    this.updateProgressText(true);
     this.fireOnValueChanged(valueName, newValue, !!questionName ? this.getQuestionByName(questionName) : undefined);
     if (this.isDisposed) return;
     this.checkElementsBindings(valueName, newValue);

diff --git a/packages/survey-core/tests/surveytests.ts b/packages/survey-core/tests/surveytests.ts
@@ -1123,6 +1123,21 @@ QUnit.test("survey.progressBarType = 'pages', Bug #6563",
     assert.equal(survey.progressText, "Page 4 of 4", "page4, #3");
   }
 );
+QUnit.test("survey.progressBarType = 'value', Bug #9532", function (assert) {
+  const survey = new SurveyModel({
+    progressBarType: "questions",
+    elements: [{ type: "text", name: "q1" }, { type: "text", name: "q2" }],
+  });
+  let progressValue = 0;
+  assert.equal(survey.progressValue, 0, "progressValue #1");
+  survey.onValueChanged.add((sender, options) => {
+    progressValue = sender.progressValue;
+  });
+  survey.getQuestionByName("q1").value = "1";
+  assert.equal(progressValue, 50, "progressValue #2");
+  survey.getQuestionByName("q2").value = "2";
+  assert.equal(progressValue, 100, "progressValue #3");
+});
 QUnit.test("Next, Prev, Next", function (assert) {
   var survey = new SurveyModel();
   survey.addPage(createPageWithQuestion("Page 1"));