feat: consolidates create and duplicate operations (#9866)

Author: JarrodMFlesch

docs/local-api/overview.mdx

@@ -131,6 +131,9 @@ const post = await payload.create({
   // Alternatively, you can directly pass a File,
   // if file is provided, filePath will be omitted
   file: uploadedFile,
+
+  // If you want to create a document that is a duplicate of another document
+  duplicateFromID: 'document-id-to-duplicate',
 })
 ```
 

packages/graphql/src/resolvers/collections/create.ts

@@ -30,15 +30,13 @@ export function createResolver<TSlug extends CollectionSlug>(
       context.req.locale = args.locale
     }
 
-    const options = {
+    const result = await createOperation({
       collection,
       data: args.data,
       depth: 0,
       draft: args.draft,
       req: isolateObjectProperty(context.req, 'transactionID'),
-    }
-
-    const result = await createOperation(options)
+    })
 
     return result
   }

packages/graphql/src/resolvers/collections/duplicate.ts

@@ -7,6 +7,7 @@ import type { Context } from '../types.js'
 export type Resolver<TData> = (
   _: unknown,
   args: {
+    data: TData
     draft: boolean
     fallbackLocale?: string
     id: string
@@ -28,15 +29,14 @@ export function duplicateResolver<TSlug extends CollectionSlug>(
     req.fallbackLocale = args.fallbackLocale || fallbackLocale
     context.req = req
 
-    const options = {
+    const result = await duplicateOperation({
       id: args.id,
       collection,
+      data: args.data,
       depth: 0,
       draft: args.draft,
       req: isolateObjectProperty(req, 'transactionID'),
-    }
-
-    const result = await duplicateOperation(options)
+    })
 
     return result
   }

packages/graphql/src/schema/initCollections.ts

@@ -280,6 +280,9 @@ export function initCollections({ config, graphqlResult }: InitCollectionsGraphQ
         type: collection.graphQL.type,
         args: {
           id: { type: new GraphQLNonNull(idType) },
+          ...(createMutationInputType
+            ? { data: { type: collection.graphQL.mutationInputType } }
+            : {}),
         },
         resolve: duplicateResolver(collection),
       }

packages/next/src/routes/rest/collections/duplicate.ts

@@ -27,6 +27,7 @@ export const duplicate: CollectionRouteHandlerWithID = async ({
   const doc = await duplicateOperation({
     id,
     collection,
+    data: req.data,
     depth: isNumber(depth) ? Number(depth) : undefined,
     draft,
     populate: sanitizePopulateParam(req.query.populate),

packages/payload/src/collections/operations/create.ts

@@ -13,6 +13,7 @@ import type {
   BeforeOperationHook,
   BeforeValidateHook,
   Collection,
+  DataFromCollectionSlug,
   RequiredDataFromCollectionSlug,
   SelectFromCollectionSlug,
 } from '../config/types.js'
@@ -21,6 +22,7 @@ import { ensureUsernameOrEmail } from '../../auth/ensureUsernameOrEmail.js'
 import executeAccess from '../../auth/executeAccess.js'
 import { sendVerificationEmail } from '../../auth/sendVerificationEmail.js'
 import { registerLocalStrategy } from '../../auth/strategies/local/register.js'
+import { getDuplicateDocumentData } from '../../duplicateDocument/index.js'
 import { afterChange } from '../../fields/hooks/afterChange/index.js'
 import { afterRead } from '../../fields/hooks/afterRead/index.js'
 import { beforeChange } from '../../fields/hooks/beforeChange/index.js'
@@ -43,6 +45,7 @@ export type Arguments<TSlug extends CollectionSlug> = {
   disableTransaction?: boolean
   disableVerificationEmail?: boolean
   draft?: boolean
+  duplicateFromID?: DataFromCollectionSlug<TSlug>['id']
   overrideAccess?: boolean
   overwriteExistingFiles?: boolean
   populate?: PopulateType
@@ -97,6 +100,7 @@ export const createOperation = async <
       depth,
       disableVerificationEmail,
       draft = false,
+      duplicateFromID,
       overrideAccess,
       overwriteExistingFiles = false,
       populate,
@@ -115,6 +119,23 @@ export const createOperation = async <
 
     const shouldSaveDraft = Boolean(draft && collectionConfig.versions.drafts)
 
+    let duplicatedFromDocWithLocales: JsonObject = {}
+    let duplicatedFromDoc: JsonObject = {}
+
+    if (duplicateFromID) {
+      const duplicateResult = await getDuplicateDocumentData({
+        id: duplicateFromID,
+        collectionConfig,
+        draftArg: shouldSaveDraft,
+        overrideAccess,
+        req,
+        shouldSaveDraft,
+      })
+
+      duplicatedFromDoc = duplicateResult.duplicatedFromDoc
+      duplicatedFromDocWithLocales = duplicateResult.duplicatedFromDocWithLocales
+    }
+
     // /////////////////////////////////////
     // Access
     // /////////////////////////////////////
@@ -131,7 +152,9 @@ export const createOperation = async <
       collection,
       config,
       data,
+      isDuplicating: Boolean(duplicateFromID),
       operation: 'create',
+      originalDoc: duplicatedFromDoc,
       overwriteExistingFiles,
       req,
       throwOnMissingFile:
@@ -148,7 +171,7 @@ export const createOperation = async <
       collection: collectionConfig,
       context: req.context,
       data,
-      doc: {},
+      doc: duplicatedFromDoc,
       global: null,
       operation: 'create',
       overrideAccess,
@@ -169,6 +192,7 @@ export const createOperation = async <
             context: req.context,
             data,
             operation: 'create',
+            originalDoc: duplicatedFromDoc,
             req,
           })) || data
       },
@@ -188,6 +212,7 @@ export const createOperation = async <
           context: req.context,
           data,
           operation: 'create',
+          originalDoc: duplicatedFromDoc,
           req,
         })) || data
     }, Promise.resolve())
@@ -200,8 +225,8 @@ export const createOperation = async <
       collection: collectionConfig,
       context: req.context,
       data,
-      doc: {},
-      docWithLocales: {},
+      doc: duplicatedFromDoc,
+      docWithLocales: duplicatedFromDocWithLocales,
       global: null,
       operation: 'create',
       req,