From 77a02a1e257a534007a309297f4b03dfc0a09686 Mon Sep 17 00:00:00 2001
From: Matt Bruce <matt.bruce@verizon.com>
Date: Wed, 14 Sep 2022 11:59:01 -0500
Subject: [PATCH] moved valueChanged to method again this is due to the
 updating internal models via the control, therefore there will be a delay in
 the eventing from the publisher back up to the parent model

Signed-off-by: Matt Bruce <matt.bruce@verizon.com>
---
 VDS/Classes/SelectorGroupHandlerBase.swift        |  5 +++++
 VDS/Components/Checkbox/CheckboxGroup.swift       |  4 +---
 VDS/Components/RadioBox/RadioBoxGroup.swift       |  4 +---
 VDS/Components/RadioButton/RadioButtonGroup.swift |  4 +---
 VDS/Components/RadioSwatch/RadioSwatchGroup.swift | 10 +++++++---
 5 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/VDS/Classes/SelectorGroupHandlerBase.swift b/VDS/Classes/SelectorGroupHandlerBase.swift
index e8679832..37abed69 100644
--- a/VDS/Classes/SelectorGroupHandlerBase.swift
+++ b/VDS/Classes/SelectorGroupHandlerBase.swift
@@ -88,6 +88,11 @@ public class SelectorGroupHandlerBase<GroupModelType: SelectorGroupModelable, Mo
         fatalError("Must override didSelect")
     }
 
+    public func valueChanged() {
+        DispatchQueue.main.asyncAfter(deadline: .now() + Constants.ModelStateDebounce) { [weak self] in
+            self?.sendActions(for: .valueChanged)
+        }
+    }
 }
 
 public class SelectorGroupSelectedHandlerBase<GroupModelType: SelectorGroupSelectedModelable, ModelHandlerType: Control<GroupModelType.SelectorModelType>>: SelectorGroupHandlerBase<GroupModelType, ModelHandlerType> where GroupModelType.SelectorModelType == ModelHandlerType.ModelType {
diff --git a/VDS/Components/Checkbox/CheckboxGroup.swift b/VDS/Components/Checkbox/CheckboxGroup.swift
index 85dd1134..4fb3e5dc 100644
--- a/VDS/Components/Checkbox/CheckboxGroup.swift
+++ b/VDS/Components/Checkbox/CheckboxGroup.swift
@@ -14,9 +14,7 @@ public class CheckboxGroup: CheckboxGroupBase<DefaultCheckboxGroupModel, Checkbo
         if selectedControl.isSelected, hasError{
             hasError.toggle()
         }
-        DispatchQueue.main.asyncAfter(deadline: .now() + Constants.ModelStateDebounce) { [weak self] in
-            self?.sendActions(for: .valueChanged)
-        }
+        valueChanged()
     }
 }
 
diff --git a/VDS/Components/RadioBox/RadioBoxGroup.swift b/VDS/Components/RadioBox/RadioBoxGroup.swift
index 3c6c0456..01e5bc06 100644
--- a/VDS/Components/RadioBox/RadioBoxGroup.swift
+++ b/VDS/Components/RadioBox/RadioBoxGroup.swift
@@ -14,9 +14,7 @@ public class RadioBoxGroup: RadioBoxGroupBase<DefaultRadioBoxGroupModel, RadioBo
         let oldSelectedControl = selectorViews.filter { $0.isSelected == true }.first
         oldSelectedControl?.toggle()
         selectedControl.toggle()
-        DispatchQueue.main.asyncAfter(deadline: .now() + Constants.ModelStateDebounce) { [weak self] in
-            self?.sendActions(for: .valueChanged)
-        }
+        valueChanged()
     }
 }
 
diff --git a/VDS/Components/RadioButton/RadioButtonGroup.swift b/VDS/Components/RadioButton/RadioButtonGroup.swift
index 109aae53..d0c055aa 100644
--- a/VDS/Components/RadioButton/RadioButtonGroup.swift
+++ b/VDS/Components/RadioButton/RadioButtonGroup.swift
@@ -17,9 +17,7 @@ public class RadioButtonGroup: RadioButtonGroupBase<DefaultRadioButtonGroupModel
         if hasError {
             hasError = false
         }
-        DispatchQueue.main.asyncAfter(deadline: .now() + Constants.ModelStateDebounce) { [weak self] in
-            self?.sendActions(for: .valueChanged)
-        }
+        valueChanged()
     }
 }
 
diff --git a/VDS/Components/RadioSwatch/RadioSwatchGroup.swift b/VDS/Components/RadioSwatch/RadioSwatchGroup.swift
index 7fdc5eed..8871d170 100644
--- a/VDS/Components/RadioSwatch/RadioSwatchGroup.swift
+++ b/VDS/Components/RadioSwatch/RadioSwatchGroup.swift
@@ -18,9 +18,7 @@ public class RadioSwatchGroup: RadioSwatchGroupBase<DefaultRadioSwatchGroupModel
         }
         selector.toggle()
         label.text = selector.model.text
-        DispatchQueue.main.asyncAfter(deadline: .now() + Constants.ModelStateDebounce) { [weak self] in
-            self?.sendActions(for: .valueChanged)
-        }
+        valueChanged()
     }
     
 }
@@ -226,6 +224,12 @@ public class RadioSwatchGroupBase<GroupModelType: RadioSwatchGroupModel, ModelHa
         fatalError("Must override didSelect")
     }
     
+    public func valueChanged() {
+        DispatchQueue.main.asyncAfter(deadline: .now() + Constants.ModelStateDebounce) { [weak self] in
+            self?.sendActions(for: .valueChanged)
+        }
+    }
+    
     public var selectedModelHandler: ModelHandlerType? {
         guard let index = model.selectors.firstIndex(where: {$0.selected == true }),
               let cell = collectionView.cellForItem(at: IndexPath(item: index, section: 0)) as? CollectionViewCell<ModelHandlerType> else {