react-native-onyx 1.0.26 → 1.0.28

package/lib/Onyx.js

@@ -10,6 +10,14 @@ import createDeferredTask from './createDeferredTask';
 import fastMerge from './fastMerge';
 import * as PerformanceUtils from './metrics/PerformanceUtils';
 
+// Method constants
+const METHOD = {
+    SET: 'set',
+    MERGE: 'merge',
+    MERGE_COLLECTION: 'mergecollection',
+    CLEAR: 'clear',
+};
+
 // Keeps track of the last connectionID that was used so we can keep incrementing it
 let lastConnectionID = 0;
 
@@ -515,7 +523,7 @@ function keyChanged(key, data, canUpdateSubscriber) {
             // returned by the selector and only if the selected data has changed.
             if (subscriber.selector) {
                 subscriber.withOnyxInstance.setState((prevState) => {
-                    const previousValue = getSubsetOfData(prevState, subscriber.selector);
+                    const previousValue = getSubsetOfData(prevState[subscriber.statePropertyName], subscriber.selector);
                     const newValue = getSubsetOfData(data, subscriber.selector);
                     if (!deepEqual(previousValue, newValue)) {
                         return {
@@ -1101,36 +1109,41 @@ function mergeCollection(collectionKey, collection) {
  * Insert API responses and lifecycle data into Onyx
  *
  * @param {Array} data An array of objects with shape {onyxMethod: oneOf('set', 'merge', 'mergeCollection'), key: string, value: *}
+ * @returns {Promise} resolves when all operations are complete
  */
 function update(data) {
     // First, validate the Onyx object is in the format we expect
     _.each(data, ({onyxMethod, key}) => {
-        if (!_.contains(['clear', 'set', 'merge', 'mergecollection'], onyxMethod)) {
+        if (!_.contains([METHOD.CLEAR, METHOD.SET, METHOD.MERGE, METHOD.MERGE_COLLECTION], onyxMethod)) {
             throw new Error(`Invalid onyxMethod ${onyxMethod} in Onyx update.`);
         }
-        if (onyxMethod !== 'clear' && !_.isString(key)) {
+        if (onyxMethod !== METHOD.CLEAR && !_.isString(key)) {
             throw new Error(`Invalid ${typeof key} key provided in Onyx update. Onyx key must be of type string.`);
         }
     });
 
+    const promises = [];
+
     _.each(data, ({onyxMethod, key, value}) => {
         switch (onyxMethod) {
-            case 'set':
-                set(key, value);
+            case METHOD.SET:
+                promises.push(set(key, value));
                 break;
-            case 'merge':
-                merge(key, value);
+            case METHOD.MERGE:
+                promises.push(merge(key, value));
                 break;
-            case 'mergecollection':
-                mergeCollection(key, value);
+            case METHOD.MERGE_COLLECTION:
+                promises.push(mergeCollection(key, value));
                 break;
-            case 'clear':
-                clear();
+            case METHOD.CLEAR:
+                promises.push(clear());
                 break;
             default:
                 break;
         }
     });
+
+    return Promise.all(promises);
 }
 
 /**
@@ -1219,6 +1232,7 @@ const Onyx = {
     addToEvictionBlockList,
     removeFromEvictionBlockList,
     isSafeEvictionKey,
+    METHOD,
 };
 
 /**

package/package.json

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