Merge pull request #5 from Thundernerd/develop

Develop

Author: Thundernerd

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## [1.2.0] - 2020-09-02
+
+### Updated
+- When invoking a copy of the subscriptions is made to allow for modifying the original subscriptions without exceptions
+
 ## [1.1.2] - 2020-08-23
 
 ### Added

Runtime/SafeEvent.cs

@@ -19,8 +19,10 @@ public void Invoke()
         {
             if (subscriptions == null)
                 return;
-            
-            foreach (Action subscription in subscriptions)
+
+            HashSet<Action> subscriptionsCopy = new HashSet<Action>(subscriptions);
+
+            foreach (Action subscription in subscriptionsCopy)
             {
 #if UNITY_EDITOR || DEBUG
                 ThrowIfInvalidSubscription(subscription);

Runtime/SafeEventOneParameter.cs

@@ -20,7 +20,9 @@ public void Invoke(T obj)
             if (subscriptions == null)
                 return;
 
-            foreach (Action<T> subscription in subscriptions)
+            HashSet<Action<T>> subscriptionsCopy = new HashSet<Action<T>>(subscriptions);
+
+            foreach (Action<T> subscription in subscriptionsCopy)
             {
 #if UNITY_EDITOR || DEBUG
                 ThrowIfInvalidSubscription(subscription);

Runtime/SafeEventTwoParameters.cs

@@ -20,7 +20,9 @@ public void Invoke(T arg1, T2 arg2)
             if (subscriptions == null)
                 return;
 
-            foreach (Action<T, T2> subscription in subscriptions)
+            HashSet<Action<T, T2>> subscriptionsCopy = new HashSet<Action<T, T2>>(subscriptions);
+
+            foreach (Action<T, T2> subscription in subscriptionsCopy)
             {
 #if UNITY_EDITOR || DEBUG
                 ThrowIfInvalidSubscription(subscription);

package.json

@@ -1,7 +1,7 @@
 {
   "name": "net.tnrd.safeevent",
   "displayName": "Safe Event",
-  "version": "1.1.2",
+  "version": "1.2.0",
   "unity": "2019.1",
   "description": "An event class that has extra checks to help prevent mistakes",
   "keywords": [