fix: Prevent Selection of both Target and Source when a transfer is made

primefaces · Nov 10, 2024 · 57d9290 · 57d9290
1 parent 9cb5e0a
commit 57d9290
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/components/lib/picklist/PickList.js b/components/lib/picklist/PickList.js
@@ -114,6 +114,7 @@ export const PickList = React.memo(
                         });
                     }
 
+                    onSelectionChange({ originalEvent, value: selectedValue }, 'targetSelection', props.onTargetSelectionChange);
                     break;
 
                 case 'allToTarget':
@@ -127,6 +128,7 @@ export const PickList = React.memo(
                     }
 
                     selectedValue = [];
+                    onSelectionChange({ originalEvent, value: selectedValue }, 'targetSelection', props.onTargetSelectionChange);
 
                     break;
 
@@ -140,6 +142,7 @@ export const PickList = React.memo(
                         });
                     }
 
+                    onSelectionChange({ originalEvent, value: selectedValue }, 'sourceSelection', props.onSourceSelectionChange);
                     break;
 
                 case 'allToSource':
@@ -154,14 +157,13 @@ export const PickList = React.memo(
 
                     selectedValue = [];
 
+                    onSelectionChange({ originalEvent, value: selectedValue }, 'sourceSelection', props.onSourceSelectionChange);
                     break;
 
                 default:
                     break;
             }
 
-            onSelectionChange({ originalEvent, value: selectedValue }, 'sourceSelection', props.onSourceSelectionChange);
-            onSelectionChange({ originalEvent, value: selectedValue }, 'targetSelection', props.onTargetSelectionChange);
             handleChange(event, source, target);
         };