linting

Author: ArnabChatterjee20k

src/Database/Adapter/Mongo.php

@@ -1589,7 +1589,7 @@ public function upsertDocuments(Document $collection, string $attribute, array $
                 unset($record['_id']); // Don't update _id
 
                 // Get fields to unset for schemaless mode
-                $unsetFields = $this->getUnsetForSchemalessUpsert($oldDocument, $document, $record);
+                $unsetFields = $this->getUpsertAttributeRemovals($oldDocument, $document, $record);
 
                 if (!empty($attribute)) {
                     // Get the attribute value before removing it from $set
@@ -1649,6 +1649,43 @@ public function upsertDocuments(Document $collection, string $attribute, array $
         return \array_map(fn ($change) => $change->getNew(), $changes);
     }
 
+    /**
+     * Get fields to unset for schemaless upsert operations
+     *
+     * @param Document $oldDocument
+     * @param Document $newDocument
+     * @param array<string, mixed> $record
+     * @return array<string, string>
+     */
+    private function getUpsertAttributeRemovals(Document $oldDocument, Document $newDocument, array $record): array
+    {
+        $unsetFields = [];
+
+        if ($this->getSupportForAttributes() || $oldDocument->isEmpty()) {
+            return $unsetFields;
+        }
+
+        $oldUserAttributes = $oldDocument->getAttributes();
+        $newUserAttributes = $newDocument->getAttributes();
+
+        $protectedFields = ['_uid', '_id', '_createdAt', '_updatedAt', '_permissions', '_tenant'];
+
+        foreach ($oldUserAttributes as $originalKey => $originalValue) {
+            if (in_array($originalKey, $protectedFields) || array_key_exists($originalKey, $newUserAttributes)) {
+                continue;
+            }
+
+            $transformed = $this->replaceChars('$', '_', [$originalKey => $originalValue]);
+            $dbKey = array_key_first($transformed);
+
+            if ($dbKey && !array_key_exists($dbKey, $record) && !in_array($dbKey, $protectedFields)) {
+                $unsetFields[$dbKey] = '';
+            }
+        }
+
+        return $unsetFields;
+    }
+
     /**
      * Get sequences for documents that were created
      *
@@ -3410,41 +3447,4 @@ public function getSupportForTrigramIndex(): bool
     {
         return false;
     }
-
-    /**
-     * Get fields to unset for schemaless upsert operations
-     *
-     * @param Document $oldDocument
-     * @param Document $newDocument
-     * @param array<string, mixed> $record
-     * @return array<string, string>
-     */
-    private function getUnsetForSchemalessUpsert(Document $oldDocument, Document $newDocument, array $record): array
-    {
-        $unsetFields = [];
-
-        if ($this->getSupportForAttributes() || $oldDocument->isEmpty()) {
-            return $unsetFields;
-        }
-
-        $oldUserAttributes = $oldDocument->getAttributes();
-        $newUserAttributes = $newDocument->getAttributes();
-
-        $protectedFields = ['_uid', '_id', '_createdAt', '_updatedAt', '_permissions', '_tenant'];
-
-        foreach ($oldUserAttributes as $originalKey => $originalValue) {
-            if (in_array($originalKey, $protectedFields) || array_key_exists($originalKey, $newUserAttributes)) {
-                continue;
-            }
-
-            $transformed = $this->replaceChars('$', '_', [$originalKey => $originalValue]);
-            $dbKey = array_key_first($transformed);
-
-            if ($dbKey && !array_key_exists($dbKey, $record) && !in_array($dbKey, $protectedFields)) {
-                $unsetFields[$dbKey] = '';
-            }
-        }
-
-        return $unsetFields;
-    }
 }