fix: skip validation for unpublish operations with localized required fields (#16120)

## Problem

Unpublishing documents with `required: true` + `localized: true` fields
fails with a validation error when `localization.fallback: false`.

The `UnpublishButton` sends only `{ _status: 'draft' }` with
`unpublishAllLocales=true` but no field data, causing `beforeChange`
validation to fail on missing required fields.

## Solution

Added `unpublishAllLocales` to the `skipValidation` condition in
`beforeChange` for both collections and globals — same pattern already
used for `isSavingDraft`.

## Why

Unpublish is a metadata-only operation (changing `_status`). Field
validation is unnecessary since data already exists in the database.

Fixes #15651

---------

Co-authored-by: Patrik Kozak <35232443+PatrikKozak@users.noreply.github.com>

Author: omar-y-abdi

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

@@ -257,7 +257,9 @@ export const updateDocument = async <
       // only skip validation for drafts when draft validation is false
       (isSavingDraft && !hasDraftValidationEnabled(collectionConfig)) ||
       // Skip validation for trash operations since they're just metadata updates
-      (collectionConfig.trash && (Boolean(data?.deletedAt) || isRestoringDraftFromTrash)),
+      (collectionConfig.trash && (Boolean(data?.deletedAt) || isRestoringDraftFromTrash)) ||
+      // Skip validation for unpublish operations — they only change _status, not document data
+      unpublishAllLocales,
   }
 
   // /////////////////////////////////////

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

@@ -253,7 +253,10 @@ export const updateOperation = async <
       global: globalConfig,
       operation: 'update' as Operation,
       req,
-      skipValidation: isSavingDraft && !hasDraftValidationEnabled(globalConfig),
+      skipValidation:
+        (isSavingDraft && !hasDraftValidationEnabled(globalConfig)) ||
+        // Skip validation for unpublish operations — they only change _status, not document data
+        unpublishAllLocales,
     }
 
     let result: JsonObject = await beforeChange(beforeChangeArgs)

test/versions/int.spec.ts

@@ -1464,6 +1464,50 @@ describe('Versions', () => {
 
         expect(found._status).toBe('draft')
       })
+
+      it('should unpublish a collection document with localized required fields from a non-default locale', async () => {
+        const doc = await payload.create({
+          collection: draftCollectionSlug,
+          data: {
+            _status: 'published',
+            description: 'test',
+            title: 'unpublish localized test',
+          },
+          locale: 'en',
+        })
+
+        const unpublished = await payload.update({
+          id: doc.id,
+          collection: draftCollectionSlug,
+          data: { _status: 'draft' },
+          locale: 'es',
+          unpublishAllLocales: true,
+        })
+
+        expect(unpublished._status).toBe('draft')
+
+        await payload.delete({ collection: draftCollectionSlug, id: doc.id })
+      })
+
+      it('should unpublish a global with localized required fields from a non-default locale', async () => {
+        await payload.updateGlobal({
+          slug: draftGlobalSlug,
+          data: { _status: 'published', title: 'unpublish global localized test' },
+          locale: 'en',
+        })
+
+        const unpublished = await payload.updateGlobal({
+          slug: draftGlobalSlug,
+          data: { _status: 'draft' },
+          fallbackLocale: false,
+          locale: 'es',
+          unpublishAllLocales: true,
+        })
+
+        expect(unpublished._status).toBe('draft')
+
+        await cleanupGlobal({ payload, globalSlug: draftGlobalSlug })
+      })
     })
 
     describe('Draft Types', () => {