Skip to content

Commit 9c9c7c5

Browse files
authored
fix: use hasDraftsEnabled utility instead of direct check (#14843)
Replaces direct checks of `collectionConfig.versions?.drafts` with the `hasDraftsEnabled` utility function for consistency across document permissions, update operations, and duplicate document functionality.
1 parent ccfaee0 commit 9c9c7c5

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

packages/next/src/views/Document/getDocumentPermissions.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ export const getDocumentPermissions = async (args: {
4646
req,
4747
})
4848

49-
if (collectionConfig.versions?.drafts) {
49+
if (hasDraftsEnabled(collectionConfig)) {
5050
hasPublishPermission = (
5151
await docAccessOperation({
5252
id,

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ export const updateOperation = async <
171171

172172
let docs
173173

174-
if (collectionConfig.versions?.drafts && shouldSaveDraft) {
174+
if (hasDraftsEnabled(collectionConfig) && shouldSaveDraft) {
175175
const versionsWhere = appendVersionToQueryKey(fullWhere)
176176

177177
await validateQueryPaths({

packages/ui/src/elements/DuplicateDocument/index.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import type { SanitizedCollectionConfig } from 'payload'
55
import { useModal } from '@faceless-ui/modal'
66
import { getTranslation } from '@payloadcms/translations'
77
import { useRouter } from 'next/navigation.js'
8-
import { formatAdminURL } from 'payload/shared'
8+
import { formatAdminURL, hasDraftsEnabled } from 'payload/shared'
99
import * as qs from 'qs-esm'
1010
import React, { useCallback, useMemo } from 'react'
1111
import { toast } from 'sonner'
@@ -95,7 +95,7 @@ export const DuplicateDocument: React.FC<Props> = ({
9595
addQueryPrefix: true,
9696
})}`,
9797
{
98-
body: JSON.stringify(collectionConfig.versions?.drafts ? { _status: 'draft' } : {}),
98+
body: JSON.stringify(hasDraftsEnabled(collectionConfig) ? { _status: 'draft' } : {}),
9999
headers,
100100
},
101101
)

0 commit comments

Comments
 (0)