Fix afterRerender sometimes throws error (#8449)

surveyjs · Jun 21, 2024 · ffd6b45 · ffd6b45
1 parent c06841c
commit ffd6b45
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/src/base.ts b/src/base.ts
@@ -1172,7 +1172,7 @@ export class Base {
     return this.supportOnElementRenderedEvent && this.onElementRenderedEventEnabled ? this._onElementRerendered : undefined;
   }
   public afterRerender(): void {
-    this.onElementRerendered.fire(this, undefined);
+    this.onElementRerendered?.fire(this, undefined);
   }
 }
 

diff --git a/tests/basetests.ts b/tests/basetests.ts
@@ -816,3 +816,18 @@ QUnit.test("check animationAllowed", function (assert) {
   assert.ok(question.animationAllowed);
   settings.animationEnabled = false;
 });
+
+QUnit.test("check afterRerender function", (assert) => {
+  const survey = new SurveyModel({});
+  assert.notOk(!!survey.onElementRerendered);
+  survey.afterRerender();
+  survey.enableOnElementRenderedEvent();
+  let log = "";
+  survey.onElementRerendered.add(() => log += "->callback");
+  survey.afterRerender();
+  assert.equal(log, "->callback");
+  log = "";
+  survey.disableOnElementRenderedEvent();
+  survey.afterRerender();
+  assert.equal(log, "");
+});