Merge pull request #15677 from Automattic/vkarpov15/gh-15672-2

perf: avoid getting all modified paths in update when checking if versionKey needs to be set

Author: vkarpov15

lib/helpers/update/decorateUpdateWithVersionKey.js

@@ -1,7 +1,5 @@
 'use strict';
 
-const modifiedPaths = require('./modifiedPaths');
-
 /**
  * Decorate the update with a version key, if necessary
  * @api private
@@ -12,15 +10,26 @@ module.exports = function decorateUpdateWithVersionKey(update, options, versionK
     return;
   }
 
-  const updatedPaths = modifiedPaths(update);
-  if (!updatedPaths[versionKey]) {
-    if (options.overwrite) {
+  if (options.overwrite) {
+    if (!hasKey(update, versionKey)) {
       update[versionKey] = 0;
-    } else {
-      if (!update.$setOnInsert) {
-        update.$setOnInsert = {};
-      }
-      update.$setOnInsert[versionKey] = 0;
     }
+  } else if (
+    !hasKey(update, versionKey) &&
+    !hasKey(update?.$set, versionKey) &&
+    !hasKey(update?.$inc, versionKey) &&
+    !hasKey(update?.$setOnInsert, versionKey)
+  ) {
+    if (!update.$setOnInsert) {
+      update.$setOnInsert = {};
+    }
+    update.$setOnInsert[versionKey] = 0;
   }
 };
+
+function hasKey(obj, key) {
+  if (obj == null || typeof obj !== 'object') {
+    return false;
+  }
+  return Object.prototype.hasOwnProperty.call(obj, key);
+}