fix: selectAll should not select option twice

Author: adamberecz

src/composables/useOptions.js

@@ -523,7 +523,7 @@ export default function useOptions (props, context, dep)
       return
     }
 
-    select(fo.value.filter(o => !o.disabled))
+    select(fo.value.filter(o => !o.disabled && !isSelected(o)))
   }
 
   // no export

tests/unit/composables/useOptions.spec.js

@@ -1541,6 +1541,25 @@ describe('useOptions', () => {
 
       expect(getValue(select)).toStrictEqual([1,3])
     })
+
+    it('should not select an option twice', async () => {
+      let select = createSelect({
+        mode: 'tags',
+        hideSelected: false,
+        options: [
+          { label: 1, value: 1, },
+          { label: 2, value: 2, disabled: true, },
+          { label: 3, value: 3, },
+        ],
+        value: [1],
+      })
+
+      select.vm.selectAll()
+
+      await nextTick()
+
+      expect(getValue(select)).toStrictEqual([1,3])
+    })
   })
 
   describe('isSelected', () => {