fix(setDefaultsOnInsert): avoid applying defaults on insert if nested…

… property set Fix #12279
Automattic · Sep 4, 2022 · 2be0e37 · 2be0e37
1 parent 803a786
commit 2be0e37
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 0 deletions.
diff --git a/lib/helpers/setDefaultsOnInsert.js b/lib/helpers/setDefaultsOnInsert.js
@@ -105,6 +105,8 @@ function isModified(modified, path) {
   if (modified[path]) {
     return true;
   }
+
+  // Is any parent path of `path` modified?
   const sp = path.split('.');
   let cur = sp[0];
   for (let i = 1; i < sp.length; ++i) {
@@ -113,5 +115,13 @@ function isModified(modified, path) {
     }
     cur += '.' + sp[i];
   }
+
+  // Is any child of `path` modified?
+  for (const modifiedPath of Object.keys(modified)) {
+    if (modifiedPath.slice(0, path.length + 1) === path + '.') {
+      return true;
+    }
+  }
+
   return false;
 }
diff --git a/test/helpers/setDefaultsOnInsert.test.js b/test/helpers/setDefaultsOnInsert.test.js
@@ -87,4 +87,25 @@ describe('setDefaultsOnInsert', function() {
 
     assert.equal(update.$setOnInsert['time.resolved'], 42);
   });
+
+  it('skips default if parent is $set (gh-12279)', function() {
+    const SubscriptionsConfigSchema = Schema({
+      hasPaidSubscription: { type: Boolean, required: true },
+      hasPastDueInvoice: { type: Boolean, required: true }
+    });
+
+    const CustomerSchema = Schema({
+      subscriptionsConfig: {
+        type: SubscriptionsConfigSchema,
+        required: true,
+        default: { hasPaidSubscription: false, hasPastDueInvoice: false }
+      }
+    });
+
+    const opts = {};
+    let update = { $set: { 'subscriptionsConfig.hasPaidSubscription': 100 } };
+    update = setDefaultsOnInsert({}, CustomerSchema, update, opts);
+
+    assert.ok(!update.$setOnInsert);
+  });
 });