react-native-onyx 2.0.49 → 2.0.51

package/dist/Onyx.js

@@ -116,7 +116,13 @@ function connect(connectOptions) {
         // We search all the keys in storage to see if any are a "match" for the subscriber we are connecting so that we
         // can send data back to the subscriber. Note that multiple keys can match as a subscriber could either be
         // subscribed to a "collection key" or a single key.
-        const matchingKeys = Array.from(keys).filter((key) => OnyxUtils_1.default.isKeyMatch(mapping.key, key));
+        const matchingKeys = [];
+        keys.forEach((key) => {
+            if (!OnyxUtils_1.default.isKeyMatch(mapping.key, key)) {
+                return;
+            }
+            matchingKeys.push(key);
+        });
         // If the key being connected to does not exist we initialize the value with null. For subscribers that connected
         // directly via connect() they will simply get a null value sent to them without any information about which key matched
         // since there are none matched. In withOnyx() we wait for all connected keys to return a value before rendering the child
@@ -576,6 +582,10 @@ function updateSnapshots(data) {
             }
             updatedData = Object.assign(Object.assign({}, updatedData), { [key]: (0, pick_1.default)(value, Object.keys(snapshotData[key])) });
         });
+        // Skip the update if there's no data to be merged
+        if (utils_1.default.isEmptyObject(updatedData)) {
+            return;
+        }
         promises.push(() => merge(snapshotKey, { data: updatedData }));
     });
     return Promise.all(promises.map((p) => p()));

package/dist/OnyxUtils.js

@@ -256,16 +256,13 @@ function tryGetCachedValue(key, mapping) {
         if (allCacheKeys.size === 0) {
             return;
         }
-        const matchingKeys = Array.from(allCacheKeys).filter((k) => k.startsWith(key));
-        const values = matchingKeys.reduce((finalObject, matchedKey) => {
-            const cachedValue = OnyxCache_1.default.get(matchedKey);
-            if (cachedValue) {
-                // This is permissible because we're in the process of constructing the final object in a reduce function.
-                // eslint-disable-next-line no-param-reassign
-                finalObject[matchedKey] = cachedValue;
+        const values = {};
+        allCacheKeys.forEach((cacheKey) => {
+            if (!cacheKey.startsWith(key)) {
+                return;
             }
-            return finalObject;
-        }, {});
+            values[cacheKey] = OnyxCache_1.default.get(cacheKey);
+        });
         val = values;
     }
     if (mapping === null || mapping === void 0 ? void 0 : mapping.selector) {

package/package.json

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