Revert "fix: update result.updatedAt after saveVersion in the create operation"

This reverts commit f698ea9a98.

packages/payload/src/versions/saveVersion.ts

@@ -39,6 +39,8 @@ export const saveVersion = async ({
   let createNewVersion = true
   const now = new Date().toISOString()
   const versionData = deepCopyObjectSimple(doc)
+  versionData.updatedAt = now
+
   if (draft) {
     versionData._status = 'draft'
   }

test/plugin-redirects/int.spec.ts

@@ -18,12 +18,14 @@ describe('@payloadcms/plugin-redirects', () => {
   beforeAll(async () => {
     ;({ payload } = await initPayloadInt(dirname))
 
-    page = await payload.create({
+    const { id } = await payload.create({
       collection: 'pages',
       data: {
         title: 'Test',
       },
     })
+
+    page = await payload.findByID({ collection: 'pages', id })
   })
 
   afterAll(async () => {

test/select/int.spec.ts

@@ -2063,8 +2063,8 @@ function createDeepPost() {
   })
 }
 
-function createVersionedPost() {
-  return payload.create({
+async function createVersionedPost() {
+  const { id } = await payload.create({
     collection: 'versioned-posts',
     data: {
       number: 2,
@@ -2073,6 +2073,8 @@ function createVersionedPost() {
       blocks: [{ blockType: 'test', text: 'hela' }],
     },
   })
+
+  return payload.findByID({ collection: 'versioned-posts', id, draft: true })
 }
 
 function createPoint() {

test/versions/int.spec.ts

@@ -355,7 +355,7 @@ describe('Versions', () => {
         // createdAt from non-versions should be the same as version_createdAt in versions
         expect(fromNonVersionsTable.createdAt).toBe(latestVersionData.version.createdAt)
         // When creating new version - updatedAt should match version.updatedAt
-        expect(fromNonVersionsTable.updatedAt).toBe(latestVersionData.version.updatedAt)
+        expect(upd.updatedAt).toBe(latestVersionData.version.updatedAt)
       })
     })
 
@@ -576,6 +576,33 @@ describe('Versions', () => {
           { id: doc.id, message: 'The following field is invalid: title' },
         ])
       })
+
+      it('should have correct updatedAt timestamps when saving drafts', async () => {
+        const created = await payload.create({
+          collection: draftCollectionSlug,
+          data: {
+            description: 'desc',
+            title: 'title',
+          },
+          draft: true,
+        })
+
+        await wait(10)
+
+        const updated = await payload.update({
+          id: created.id,
+          collection: draftCollectionSlug,
+          data: {
+            title: 'updated title',
+          },
+          draft: true,
+        })
+
+        const createdUpdatedAt = new Date(created.updatedAt)
+        const updatedUpdatedAt = new Date(updated.updatedAt)
+
+        expect(Number(updatedUpdatedAt)).toBeGreaterThan(Number(createdUpdatedAt))
+      })
     })
 
     describe('Update Many', () => {
@@ -1340,7 +1367,7 @@ describe('Versions', () => {
         // createdAt from non-versions should be the same as version_createdAt in versions
         expect(fromNonVersionsTable.createdAt).toBe(latestVersionData.version.createdAt)
         // When creating a new version - updatedAt should match
-        expect(fromNonVersionsTable.updatedAt).toBe(latestVersionData.version.updatedAt)
+        expect(upd.updatedAt).toBe(latestVersionData.version.updatedAt)
       })
     })