"None" choice in checkbox question is not deselected on clicking on i…

…t when it is selected fix #8438 (#8439)
surveyjs · Jun 19, 2024 · 9648712 · 9648712
1 parent cbb0ccf
commit 9648712
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/src/question_checkbox.ts b/src/question_checkbox.ts
@@ -167,7 +167,7 @@ export class QuestionCheckboxModel extends QuestionCheckboxBase {
       }
     } else {
       if(this.isNoneItem(item)) {
-        this.renderedValue = [item.value];
+        this.renderedValue = checked ? [item.value] : [];
       } else {
         const newValue: Array<any> = [].concat(this.renderedValue || []);
         const index = newValue.indexOf(item.value);

diff --git a/tests/question_baseselecttests.ts b/tests/question_baseselecttests.ts
@@ -2160,3 +2160,22 @@ QUnit.test("valuePropertyName & complete trigger, bug#8434", (assert) => {
   q.renderedValue = ["none"];
   assert.equal(survey.calcIsCompleteButtonVisible(), true, "#4");
 });
+QUnit.test("Unselect none item, bug#8438", (assert) => {
+  const survey = new SurveyModel({
+    "elements": [
+      {
+        "type": "checkbox",
+        "name": "q1",
+        "choices": [1, 2, 3],
+        "showNoneItem": true
+      }
+    ]
+  });
+  const q = <QuestionCheckboxModel>survey.getQuestionByName("q1");
+  q.clickItemHandler(q.noneItem, true);
+  assert.deepEqual(q.value, ["none"], "#1");
+  q.clickItemHandler(q.noneItem, false);
+  assert.equal(q.isEmpty(), true, "#2");
+  q.clickItemHandler(q.noneItem, true);
+  assert.deepEqual(q.value, ["none"], "#3");
+});