react-native-onyx 1.0.30 → 1.0.31

package/lib/Onyx.js

@@ -778,6 +778,19 @@ function notifySubscribersOnNextTick(key, value, canUpdateSubscriber) {
     Promise.resolve().then(() => keyChanged(key, value, canUpdateSubscriber));
 }
 
+/**
+ * This method is similar to notifySubscribersOnNextTick but it is built for working specifically with collections
+ * so that keysChanged() is triggered for the collection and not keyChanged(). If this was not done, then the
+ * subscriber callbacks receive the data in a different format than they normally expect and it breaks code.
+ *
+ * @param {String} key
+ * @param {*} value
+ */
+// eslint-disable-next-line rulesdir/no-negated-variables
+function notifyCollectionSubscribersOnNextTick(key, value) {
+    Promise.resolve().then(() => keysChanged(key, value));
+}
+
 /**
  * Remove a key from Onyx and update the subscribers
  *
@@ -1038,24 +1051,55 @@ function clear(keysToPreserve = []) {
             const defaultKeys = _.keys(defaultKeyStates);
 
             // The only keys that should not be cleared are:
-            // 1. Anything specifically passed in keysToPreserve (because some keys like language preferences, offline status, or activeClients need to remain in Onyx even when signed out)
-            // 2. Any keys with a default state (because they need to remain in Onyx as their default, and setting them to null would cause unknown behavior)
+            // 1. Anything specifically passed in keysToPreserve (because some keys like language preferences, offline
+            //      status, or activeClients need to remain in Onyx even when signed out)
+            // 2. Any keys with a default state (because they need to remain in Onyx as their default, and setting them
+            //      to null would cause unknown behavior)
             const keysToClear = _.difference(keys, keysToPreserve, defaultKeys);
             keyValuesToReset.push(..._.map(keysToClear, key => [key, null]));
 
-            // Remove any keysToPreserve from the defaultKeyStates because if they are passed in it has been explicitly called out to preserve those values instead of resetting them back
+            // Remove any keysToPreserve from the defaultKeyStates because if they are passed in it has been explicitly
+            // called out to preserve those values instead of resetting them back
             // to the default.
             const defaultKeyValuePairs = _.pairs(_.omit(defaultKeyStates, ...keysToPreserve));
 
-            // Add the default key value pairs to the keyValuesToReset so that they get set back to their default values when we clear Onyx
+            // Add the default key value pairs to the keyValuesToReset so that they get set back to their default values
+            // when we clear Onyx
             keyValuesToReset.push(...defaultKeyValuePairs);
 
+            // We now have all the key/values that need to be reset, but we're not done yet!
+            // There will be two groups of key/values and they each need to be updated a little bit differently.
+            // Collection keys need to be notified differently than non collection keys
+            const keyValuesToResetAsCollection = {};
+            const keyValuesToResetIndividually = {};
+
             // Make sure that we also reset the cache values before clearing the values from storage.
-            // We do this before clearing Storage so that any call to clear() followed by merge() on a key with a default state results in the merged value getting saved, since the update
-            // from the merge() call would happen on the tick after the update from this clear()
+            // We do this before clearing Storage so that any call to clear() followed by merge() on a key with a
+            // default state results in the merged value getting saved, since the update from the merge() call would
+            // happen on the tick after the update from this clear()
             _.each(keyValuesToReset, (keyValue) => {
-                cache.set(keyValue[0], keyValue[1]);
-                notifySubscribersOnNextTick(keyValue[0], keyValue[1]);
+                const key = keyValue[0];
+                const value = keyValue[1];
+                cache.set(key, value);
+
+                const collectionKey = key.substring(0, key.indexOf('_') + 1);
+                if (collectionKey) {
+                    if (!keyValuesToResetAsCollection[collectionKey]) {
+                        keyValuesToResetAsCollection[collectionKey] = {};
+                    }
+                    keyValuesToResetAsCollection[collectionKey][key] = value;
+                    return;
+                }
+
+                keyValuesToResetIndividually[key] = value;
+            });
+
+            // Notify the subscribers for each key/value group so they can receive the new values
+            _.each(keyValuesToResetIndividually, (value, key) => {
+                notifySubscribersOnNextTick(key, value);
+            });
+            _.each(keyValuesToResetAsCollection, (value, key) => {
+                notifyCollectionSubscribersOnNextTick(key, value);
             });
 
             return Storage.multiSet(keyValuesToReset);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-onyx",
-  "version": "1.0.30",
+  "version": "1.0.31",
   "author": "Expensify, Inc.",
   "homepage": "https://expensify.com",
   "description": "State management for React Native",