From fbf67ff10a528c9b856c235e4c16568bb0ee2e55 Mon Sep 17 00:00:00 2001
From: volterra79 <boccacci.francesco@gmail.com>
Date: Mon, 26 Feb 2024 08:42:30 +0100
Subject: [PATCH 1/2] Fix check indexItem for select2 directive

---
 src/components/WMSLayersPanel.vue |  2 +-
 src/directives/v-select2.js       | 14 +++++++++-----
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/components/WMSLayersPanel.vue b/src/components/WMSLayersPanel.vue
index 6b182f71c..2bb65f60f 100644
--- a/src/components/WMSLayersPanel.vue
+++ b/src/components/WMSLayersPanel.vue
@@ -23,7 +23,7 @@
     </label>
     <select
       id        = "g3w-wms-layers"
-      multiple  = "multiple"
+      :multiple  = "true"
       clear     = "true"
       v-select2 = "'selectedlayers'"
     >
diff --git a/src/directives/v-select2.js b/src/directives/v-select2.js
index a7aee7542..89b79a881 100644
--- a/src/directives/v-select2.js
+++ b/src/directives/v-select2.js
@@ -13,7 +13,9 @@ export default {
       select2_value,
       indexItem, /** @since 3.9.1 */
     } = vnode.data.attrs || {};
-    const isArray = binding.value && Array.isArray(vnode.context[binding.value]); //check if is an array
+    const isArrayWithIndex = binding.value
+      && Array.isArray(vnode.context[binding.value]) //check if is an array
+      && undefined !== indexItem //check if indexItem is defined /** @since 3.10.0 **/
     $(el)
       .select2({
         width: '100%',
@@ -24,19 +26,21 @@ export default {
       })
       .on('select2:select', (e) => {
         if (binding.value) {
+          //get value
           const value = e.params.data.id;
+          //check is can have multiple value
           if (multiple && (
-            (isArray
+            (isArrayWithIndex
               ? vnode.context[binding.value][indexItem].value
               : vnode.context[binding.value]
             ).filter(d => value === d).length === 0)
           ) {
-            (isArray
+            (isArrayWithIndex
               ? vnode.context[binding.value][indexItem].value
               : vnode.context[binding.value]
             ).push(value);
           } else {
-            if (isArray) {
+            if (isArrayWithIndex) {
               vnode.context[binding.value][indexItem].value = value;
             } else {
               vnode.context[binding.value] = value;
@@ -46,7 +50,7 @@ export default {
       })
       .on('select2:unselect', (e) => {
         if (binding.value && multiple) {
-          if (isArray) {
+          if (isArrayWithIndex) {
             vnode.context[binding.value][indexItem].value = vnode.context[binding.value][indexItem].value.filter(d => e.params.data.id !== d);
           } else {
             vnode.context[binding.value] = vnode.context[binding.value].filter(d => e.params.data.id !== d);

From 34085d1263133afda0f22b271ca00800615ed3bb Mon Sep 17 00:00:00 2001
From: Raruto <Raruto@users.noreply.github.com>
Date: Mon, 26 Feb 2024 09:40:40 +0100
Subject: [PATCH 2/2] reduce delta

---
 src/directives/v-select2.js | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/directives/v-select2.js b/src/directives/v-select2.js
index 89b79a881..c791b675f 100644
--- a/src/directives/v-select2.js
+++ b/src/directives/v-select2.js
@@ -13,9 +13,9 @@ export default {
       select2_value,
       indexItem, /** @since 3.9.1 */
     } = vnode.data.attrs || {};
-    const isArrayWithIndex = binding.value
-      && Array.isArray(vnode.context[binding.value]) //check if is an array
-      && undefined !== indexItem //check if indexItem is defined /** @since 3.10.0 **/
+    const isArray = binding.value
+      && Array.isArray(vnode.context[binding.value]) // check if is an array
+      && undefined !== indexItem                     // check if indexItem is defined
     $(el)
       .select2({
         width: '100%',
@@ -30,17 +30,17 @@ export default {
           const value = e.params.data.id;
           //check is can have multiple value
           if (multiple && (
-            (isArrayWithIndex
+            (isArray
               ? vnode.context[binding.value][indexItem].value
               : vnode.context[binding.value]
             ).filter(d => value === d).length === 0)
           ) {
-            (isArrayWithIndex
+            (isArray
               ? vnode.context[binding.value][indexItem].value
               : vnode.context[binding.value]
             ).push(value);
           } else {
-            if (isArrayWithIndex) {
+            if (isArray) {
               vnode.context[binding.value][indexItem].value = value;
             } else {
               vnode.context[binding.value] = value;
@@ -50,7 +50,7 @@ export default {
       })
       .on('select2:unselect', (e) => {
         if (binding.value && multiple) {
-          if (isArrayWithIndex) {
+          if (isArray) {
             vnode.context[binding.value][indexItem].value = vnode.context[binding.value][indexItem].value.filter(d => e.params.data.id !== d);
           } else {
             vnode.context[binding.value] = vnode.context[binding.value].filter(d => e.params.data.id !== d);