Vue2&Vue3 - The None item is duplicated when the separateSpecialChoic…

…es option is enabled fix #7126 (#7128)
surveyjs · Oct 12, 2023 · a5080a3 · a5080a3
1 parent 71232b7
commit a5080a3
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 3 deletions.
diff --git a/packages/survey-vue3-ui/src/Radiogroup.vue b/packages/survey-vue3-ui/src/Radiogroup.vue
@@ -11,7 +11,7 @@
   >
     <survey-radiogroup-item
       v-if="!question.hasColumns && !question.blockedRow"
-      v-for="(item, index) in question.visibleChoices"
+      v-for="(item, index) in question.bodyItems"
       :key="item.value"
       :class="question.getItemClass(item)"
       :question="question"

diff --git a/src/vue/radiogroup.vue b/src/vue/radiogroup.vue
@@ -9,7 +9,7 @@
   >
     <survey-radiogroup-item
       v-if="!question.hasColumns && !question.blockedRow"
-      v-for="(item, index) in question.visibleChoices"
+      v-for="(item, index) in question.bodyItems"
       :key="item.value"
       :class="getItemClass(item)"
       :question="question"
@@ -91,4 +91,4 @@ export class Radiogroup extends QuestionVue<QuestionRadiogroupModel> {
 }
 Vue.component("survey-radiogroup", Radiogroup);
 export default Radiogroup;
-</script>
+</script> 
diff --git a/testCafe/questions/radiogroup.js b/testCafe/questions/radiogroup.js
@@ -335,4 +335,24 @@ frameworks.forEach(framework => {
       .expect(Selector(".sv-string-viewer").withText(oldOtherText).count).eql(0)
       .expect(Selector(".sv-string-viewer").withText(newOtherText).count).eql(2);
   });
+  test("showNoneItem&separateSpecialChoices", async t => {
+    const currentJson = {
+      elements: [
+        {
+          type: "radiogroup",
+          separateSpecialChoices: true,
+          showNoneItem: true,
+          name: "car",
+          choices: [
+            "item1",
+            "item2",
+            "item3"
+          ]
+        }
+      ]
+    };
+    await initSurvey(framework, currentJson);
+    await t
+      .expect(Selector(".sv-string-viewer").withText("None").count).eql(1);
+  });
 });