react-native-onyx 1.0.21 → 1.0.22

package/lib/Onyx.js

@@ -367,11 +367,15 @@ function keysChanged(collectionKey, partialCollection) {
 /**
  * When a key change happens, search for any callbacks matching the key or collection key and trigger those callbacks
  *
+ * @example
+ * keyChanged(key, value, subscriber => subscriber.initWithStoredValues === false)
+ *
  * @private
  * @param {String} key
  * @param {*} data
+ * @param {Function} [canUpdateSubscriber] only subscribers that pass this truth test will be updated
  */
-function keyChanged(key, data) {
+function keyChanged(key, data, canUpdateSubscriber) {
     // Add or remove this key from the recentlyAccessedKeys lists
     if (!_.isNull(data)) {
         addLastAccessedKey(key);
@@ -385,7 +389,7 @@ function keyChanged(key, data) {
     const stateMappingKeys = _.keys(callbackToStateMapping);
     for (let i = 0; i < stateMappingKeys.length; i++) {
         const subscriber = callbackToStateMapping[stateMappingKeys[i]];
-        if (!subscriber || !isKeyMatch(subscriber.key, key)) {
+        if (!subscriber || !isKeyMatch(subscriber.key, key) || (_.isFunction(canUpdateSubscriber) && !canUpdateSubscriber(subscriber))) {
             continue;
         }
 
@@ -631,12 +635,16 @@ function disconnect(connectionID, keyToRemoveFromEvictionBlocklist) {
  * available async. Since we have code in our main applications that might expect things to work this way it's not safe to change this
  * behavior just yet.
  *
+ * @example
+ * notifySubscribersOnNextTick(key, value, subscriber => subscriber.initWithStoredValues === false)
+ *
  * @param {String} key
  * @param {*} value
+ * @param {Function} [canUpdateSubscriber] only subscribers that pass this truth test will be updated
  */
 // eslint-disable-next-line rulesdir/no-negated-variables
-function notifySubscribersOnNextTick(key, value) {
-    Promise.resolve().then(() => keyChanged(key, value));
+function notifySubscribersOnNextTick(key, value, canUpdateSubscriber) {
+    Promise.resolve().then(() => keyChanged(key, value, canUpdateSubscriber));
 }
 
 /**
@@ -703,6 +711,14 @@ function set(key, value) {
         Logger.logAlert(`Onyx.set() called after Onyx.merge() for key: ${key}. It is recommended to use set() or merge() not both.`);
     }
 
+    // If the value in the cache is the same as what we have then do not update subscribers unless they
+    // have initWithStoredValues: false then they MUST get all updates even if nothing has changed.
+    if (!cache.hasValueChanged(key, value)) {
+        cache.addToAccessedKeys(key);
+        notifySubscribersOnNextTick(key, value, subscriber => subscriber.initWithStoredValues === false);
+        return Promise.resolve();
+    }
+
     // Adds the key to cache when it's not available
     cache.set(key, value);
     notifySubscribersOnNextTick(key, value);

package/lib/OnyxCache.js

@@ -1,4 +1,5 @@
 import _ from 'underscore';
+import {deepEqual} from 'fast-equals';
 import fastMerge from './fastMerge';
 
 const isDefined = _.negate(_.isUndefined);
@@ -191,6 +192,15 @@ class OnyxCache {
     setRecentKeysLimit(limit) {
         this.maxRecentKeysSize = limit;
     }
+
+    /**
+     * @param {String} key
+     * @param {*} value
+     * @returns {Boolean}
+     */
+    hasValueChanged(key, value) {
+        return !deepEqual(this.storageMap[key], value);
+    }
 }
 
 const instance = new OnyxCache();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-onyx",
-  "version": "1.0.21",
+  "version": "1.0.22",
   "author": "Expensify, Inc.",
   "homepage": "https://expensify.com",
   "description": "State management for React Native",
@@ -40,6 +40,7 @@
   },
   "dependencies": {
     "ascii-table": "0.0.9",
+    "fast-equals": "^4.0.3",
     "lodash": "^4.17.21",
     "underscore": "^1.13.1"
   },