Fix encoding decoding

src/Database/Database.php

@@ -6292,12 +6292,11 @@ public function encode(Document $collection, Document $document): Document
     {
         $attributes = $collection->getAttribute('attributes', []);
 
-        $internalAttributes = \array_filter(Database::INTERNAL_ATTRIBUTES, function ($attribute) {
-            // We don't want to encode permissions into a JSON string
-            return $attribute['$id'] !== '$permissions';
-        });
-
-        $attributes = \array_merge($attributes, $internalAttributes);
+        foreach ($this->getInternalAttributes() as $attribute) {
+            if ($attribute['$id'] !== '$permissions') { // Don't encode permissions into a JSON string
+                $attributes[] = new Document($attribute);
+            }
+        }
 
         foreach ($attributes as $attribute) {
             $key = $attribute['$id'] ?? '';
@@ -6362,18 +6361,19 @@ public function decode(Document $collection, Document $document, array $selectio
         foreach ($relationships as $relationship) {
             $key = $relationship['$id'] ?? '';
 
-            if (
-                \array_key_exists($key, (array)$document)
-                || \array_key_exists($this->adapter->filter($key), (array)$document)
-            ) {
+            if ($document->offsetExists($key) || $document->offsetExists($this->adapter->filter($key))) {
                 $value = $document->getAttribute($key);
                 $value ??= $document->getAttribute($this->adapter->filter($key));
                 $document->removeAttribute($this->adapter->filter($key));
                 $document->setAttribute($key, $value);
             }
         }
 
-        $attributes = \array_merge($attributes, $this->getInternalAttributes());
+        foreach ($this->getInternalAttributes() as $attribute) {
+            if ($attribute['$id'] !== '$permissions') {
+                $attributes[] = new Document($attribute);
+            }
+        }
 
         foreach ($attributes as $attribute) {
             $key = $attribute['$id'] ?? '';
@@ -6392,10 +6392,11 @@ public function decode(Document $collection, Document $document, array $selectio
             $value = ($array) ? $value : [$value];
             $value = (is_null($value)) ? [] : $value;
 
-            foreach ($value as &$node) {
-                foreach (\array_reverse($filters) as $filter) {
+            foreach ($value as $k => $node) {
+                foreach (array_reverse($filters) as $filter) {
                     $node = $this->decodeAttribute($filter, $node, $document, $key);
                 }
+                $value[$k] = $node;
             }
 
             if (
@@ -6426,7 +6427,11 @@ public function casting(Document $collection, Document $document): Document
 
         $attributes = $collection->getAttribute('attributes', []);
 
-        $attributes = \array_merge($attributes, $this->getInternalAttributes());
+        foreach ($this->getInternalAttributes() as $attribute) {
+            if ($attribute['$id'] !== '$permissions') {
+                $attributes[] = new Document($attribute);
+            }
+        }
 
         foreach ($attributes as $attribute) {
             $key = $attribute['$id'] ?? '';

src/Database/Document.php

@@ -27,11 +27,11 @@ class Document extends ArrayObject
      */
     public function __construct(array $input = [])
     {
-        if (isset($input['$id']) && !\is_string($input['$id'])) {
+        if (array_key_exists('$id', $input) && !\is_string($input['$id'])) {
             throw new StructureException('$id must be of type string');
         }
 
-        if (isset($input['$permissions']) && !is_array($input['$permissions'])) {
+        if (array_key_exists('$permissions', $input) && !is_array($input['$permissions'])) {
             throw new StructureException('$permissions must be of type array');
         }
 

tests/e2e/Adapter/Scopes/PermissionTests.php

@@ -14,6 +14,40 @@
 
 trait PermissionTests
 {
+    public function testCreateDocumentsEmptyPermission(): void
+    {
+        /** @var Database $database */
+        $database = static::getDatabase();
+
+        $database->createCollection(__FUNCTION__);
+
+        /**
+         * Validate the decode function does not add $permissions null entry when no permissions are provided
+         */
+
+        $document = $database->createDocument(__FUNCTION__, new Document());
+
+        $this->assertArrayNotHasKey('$permissions', $document);
+        $this->assertEquals(null, $document->getAttribute('$permissions'));
+
+        $documents = [];
+
+        for ($i = 0; $i < 2; $i++) {
+            $documents[] = new Document();
+        }
+
+        $results = [];
+        $count = $database->createDocuments(__FUNCTION__, $documents, onNext: function ($doc) use (&$results) {
+            $results[] = $doc;
+        });
+
+        $this->assertEquals(2, $count);
+        foreach ($results as $result) {
+            $this->assertArrayNotHasKey('$permissions', $result);
+            $this->assertEquals(null, $result->getAttribute('$permissions'));
+        }
+    }
+
     public function testReadPermissionsFailure(): Document
     {
         Authorization::cleanRoles();