payloadcms
diff --git a/‎packages/db-mongodb/src/utilities/transform.spec.ts‎
Lines changed: 60 additions & 0 deletions b/‎packages/db-mongodb/src/utilities/transform.spec.ts‎
Lines changed: 60 additions & 0 deletions
diff --git a/‎packages/db-mongodb/src/utilities/transform.ts‎
Lines changed: 19 additions & 0 deletions b/‎packages/db-mongodb/src/utilities/transform.ts‎
Lines changed: 19 additions & 0 deletions
@@ -406,4 +406,64 @@ describe('transform', () => {
       expect(flattenValuesBefore[i]).toBe(value.toHexString())
     })
   })
+
+  it('should strip null elements from array and blocks fields on read', () => {
+    const data: Record<string, any> = {
+      _id: new Types.ObjectId(),
+      array: [null, { rel_1: new Types.ObjectId() }],
+      arrayLocalized: {
+        en: [null, { rel_1: new Types.ObjectId() }],
+      },
+      blocks: [null, { blockType: 'block', rel_1: new Types.ObjectId() }],
+    }
+
+    const mockAdapter = {
+      allowAdditionalKeys: false,
+      payload: {
+        config,
+      },
+    } as MongooseAdapter
+
+    expect(() =>
+      transform({
+        adapter: mockAdapter,
+        operation: 'read',
+        data,
+        fields: config.collections[0].fields,
+      }),
+    ).not.toThrow()
+
+    expect(data.array).toHaveLength(1)
+    expect(data.array[0]).toMatchObject({})
+    expect(data.arrayLocalized.en).toHaveLength(1)
+    expect(data.blocks).toHaveLength(1)
+    expect(data.blocks[0].blockType).toBe('block')
+  })
+
+  it('should skip null entries in hasMany relationship arrays without coercing them', () => {
+    const validId = new Types.ObjectId()
+    const data: Record<string, any> = {
+      _id: new Types.ObjectId(),
+      rel_2: [null, validId],
+    }
+
+    const mockAdapter = {
+      allowAdditionalKeys: false,
+      payload: {
+        config,
+      },
+    } as MongooseAdapter
+
+    expect(() =>
+      transform({
+        adapter: mockAdapter,
+        operation: 'read',
+        data,
+        fields: config.collections[0].fields,
+      }),
+    ).not.toThrow()
+
+    expect(data.rel_2[0]).toBeNull()
+    expect(data.rel_2[1]).toBe(validId.toHexString())
+  })
 })
@@ -180,6 +180,11 @@ const sanitizeRelationship = ({
 
   if (Array.isArray(value)) {
     result = value.map((val) => {
+      // safety check in the case of base
+      if (val === null || typeof val === 'undefined') {
+        return val
+      }
+
       // Handle has many - polymorphic
       if (isValidRelationObject(val)) {
         const relatedCollectionForSingleValue = config.collections?.find(
@@ -355,6 +360,13 @@ const stripFields = ({
             let hasNull = false
             for (let i = 0; i < localeData.length; i++) {
               const data = localeData[i]
+
+              if (!data || typeof data !== 'object') {
+                localeData[i] = null
+                hasNull = true
+                continue
+              }
+
               let fields: FlattenedField[] | null = null
 
               if (field.type === 'array') {
@@ -429,6 +441,13 @@ const stripFields = ({
 
         for (let i = 0; i < fieldData.length; i++) {
           const data = fieldData[i]
+
+          if (!data || typeof data !== 'object') {
+            fieldData[i] = null
+            hasNull = true
+            continue
+          }
+
           let fields: FlattenedField[] | null = null
 
           if (field.type === 'array') {