fix: duplicate list preferences stored (#9185)

The collection list columns are stored as user preferences to the
payload-preferences collection. Normally one user should never have
duplicate documents with the same key. This is controlled by using an
upsert normally. The collection list does not have a good way to call
upsert and was creating preferences documents every time. This change
makes it so that existing preferences are updated rather than created
with each column change.

packages/ui/src/utilities/buildTableState.ts

@@ -166,6 +166,7 @@ export const buildTableState = async (
       collection: 'payload-preferences',
       depth: 0,
       limit: 1,
+      pagination: false,
       where: {
         and: [
           {
@@ -186,18 +187,16 @@ export const buildTableState = async (
         ],
       },
     })
-    .then((res) => res.docs[0]?.value as ListPreferences)
+    .then((res) => res.docs[0] ?? { id: null, value: {} })
 
-  let newPrefs = preferencesResult
+  let newPrefs = preferencesResult.value
 
-  if (!preferencesResult || !dequal(columns, preferencesResult?.columns)) {
+  if (!preferencesResult.id || !dequal(columns, preferencesResult?.columns)) {
     const mergedPrefs = {
       ...(preferencesResult || {}),
       columns,
     }
-
-    newPrefs = await payload
-      .create({
+    const preferencesArgs = {
       collection: 'payload-preferences',
       data: {
         key: preferencesKey,
@@ -207,9 +206,20 @@ export const buildTableState = async (
         },
         value: mergedPrefs,
       },
+      depth: 0,
       req,
+    }
+
+    if (preferencesResult.id) {
+      newPrefs = await payload
+        .update({
+          ...preferencesArgs,
+          id: preferencesResult.id,
         })
         ?.then((res) => res.value as ListPreferences)
+    } else {
+      newPrefs = await payload.create(preferencesArgs)?.then((res) => res.value as ListPreferences)
+    }
   }
 
   const fields = collectionConfig.fields