fix(ui): respect formatDocURL returning null in ListDrawer (#16464)

Backport of #16398

Author: AlessioGr

packages/ui/src/providers/TableColumns/RenderDefaultCell/index.tsx

@@ -28,7 +28,7 @@ export const RenderDefaultCell: React.FC<{
     columnIndex,
   }
 
-  if (isLinkedColumn && drawerSlug) {
+  if (isLinkedColumn && drawerSlug && clientProps.link !== false) {
     propsToPass.className = `${baseClass}__first-cell`
     propsToPass.link = false
     propsToPass.onClick = ({ collectionSlug: rowColl, rowData }) => {

test/admin/collections/FormatDocURL/index.ts

@@ -3,6 +3,13 @@ import type { CollectionConfig } from 'payload'
 export const FormatDocURL: CollectionConfig = {
   slug: 'format-doc-url',
   admin: {
+    components: {
+      beforeListTable: [
+        {
+          path: '/components/BeforeList/index.js#SelectFormatDocURLButton',
+        },
+      ],
+    },
     // Custom formatDocURL function to control linking behavior
     formatDocURL: ({ doc, defaultURL, req, collectionSlug, viewType }) => {
       // Disable linking for documents with title 'no-link'

test/admin/components/BeforeList/index.tsx

@@ -21,3 +21,20 @@ export const SelectPostsButton = () => {
     </>
   )
 }
+
+export const SelectFormatDocURLButton = () => {
+  const listDrawerArgs = useMemo<UseListDrawerArgs>(
+    () => ({
+      collectionSlugs: ['format-doc-url'],
+    }),
+    [],
+  )
+  const [ListDrawer, _, { toggleDrawer }] = useListDrawer(listDrawerArgs)
+
+  return (
+    <>
+      <Button onClick={() => toggleDrawer()}>Select format doc</Button>
+      <ListDrawer allowCreate={false} enableRowSelections={false} />
+    </>
+  )
+}

test/admin/e2e/list-view/e2e.spec.ts

@@ -2201,6 +2201,43 @@ describe('List View', () => {
         ),
       )
     })
+
+    test('should disable linking in ListDrawer for documents with formatDocURL returning null', async () => {
+      await payload.create({
+        collection: formatDocURLCollectionSlug,
+        data: { title: 'no-link', description: 'This should not be linkable in drawer' },
+      })
+
+      await payload.create({
+        collection: formatDocURLCollectionSlug,
+        data: { title: 'linkable', description: 'This should be linkable in drawer' },
+      })
+
+      await page.goto(formatDocURLUrl.list)
+
+      const selectButton = page.locator('button:has-text("Select format doc")')
+      await selectButton.waitFor({ state: 'visible' })
+      await selectButton.click()
+
+      const listDrawer = page.locator('.list-drawer.drawer--is-open')
+      await listDrawer.waitFor({ state: 'visible' })
+      await expect(listDrawer).toBeVisible()
+
+      await expect(listDrawer.locator('table tbody tr')).toHaveCount(2)
+
+      // The 'no-link' row should NOT have any clickable link-styled cell —
+      // formatDocURL returned null for it, so the drawer must not attach the
+      // underline className or the selection click handler to any cell.
+      const noLinkRow = listDrawer.locator('table tbody tr').filter({ hasText: 'no-link' })
+      await expect(noLinkRow.locator('button.default-cell__first-cell')).toHaveCount(0)
+      await expect(noLinkRow.locator('a')).toHaveCount(0)
+
+      // The 'linkable' row's linked cell should still be rendered as a
+      // clickable selection button (drawer overrides the link with an
+      // onSelect handler).
+      const linkableRow = listDrawer.locator('table tbody tr').filter({ hasText: 'linkable' })
+      await expect(linkableRow.locator('button.default-cell__first-cell')).toHaveCount(1)
+    })
   })
 })