feat(customerCase): check if slug is unique across customer cases, no…

…t all documents (#750)
varianter · Oct 10, 2024 · 973c8cc · 973c8cc
1 parent cdc3fe3
commit 973c8cc
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 11 deletions.
diff --git a/studio/schemas/schemaTypes/slug.ts b/studio/schemas/schemaTypes/slug.ts
@@ -1,3 +1,4 @@
+import { groq } from "next-sanity";
 import {
   SlugValidationContext,
   SlugValue,
@@ -13,26 +14,28 @@ import {
   isPublished,
 } from "studio/utils/documentUtils";
 
-function isSlugUniqueAcrossAllDocuments(
+export function isSlugUniqueAcrossDocuments(
   slug: string,
   { document, getClient }: SlugValidationContext,
+  documentType?: string,
 ) {
   if (document === undefined) {
     return true;
   }
   const language = "language" in document ? document.language : undefined;
-  const isUniqueQuery =
-    language !== undefined
-      ? `
-    !defined(*[!(_id in [$draft, $published]) && slug.current == $slug && language == $language][0]._id)
-  `
-      : `
-    !defined(*[!(_id in [$draft, $published]) && slug.current == $slug][0]._id)
+  const isUniqueQuery = groq`
+    count(*[
+      !(_id in [$draft, $published])
+      && slug.current == $slug
+      ${documentType !== undefined ? `&& _type == $documentType` : ""}
+      ${language !== undefined ? " && language == $language" : ""}
+    ]) == 0
   `;
   return getClient({ apiVersion }).fetch<boolean>(isUniqueQuery, {
     draft: buildDraftId(document._id),
     published: buildPublishedId(document._id),
     slug,
+    ...(documentType !== undefined ? { documentType } : {}),
     ...(language !== undefined ? { language } : {}),
   });
 }
@@ -92,7 +95,7 @@ function createSlugField(source: string) {
       source,
       maxLength: SLUG_MAX_LENGTH,
       slugify,
-      isUnique: isSlugUniqueAcrossAllDocuments,
+      isUnique: isSlugUniqueAcrossDocuments,
     },
     validation: (rule) =>
       rule

diff --git a/studioShared/schemas/documents/customerCase.ts b/studioShared/schemas/documents/customerCase.ts
@@ -2,7 +2,10 @@ import { defineField, defineType } from "sanity";
 
 import languageSchemaField from "i18n/languageSchemaField";
 import { richText, title } from "studio/schemas/fields/text";
-import { titleSlug } from "studio/schemas/schemaTypes/slug";
+import {
+  isSlugUniqueAcrossDocuments,
+  titleSlug,
+} from "studio/schemas/schemaTypes/slug";
 
 export const customerCaseID = "customerCase";
 
@@ -13,7 +16,14 @@ const customerCase = defineType({
   fields: [
     languageSchemaField,
     title,
-    titleSlug,
+    {
+      ...titleSlug,
+      options: {
+        ...titleSlug.options,
+        isUnique: (slug, ctx) =>
+          isSlugUniqueAcrossDocuments(slug, ctx, customerCaseID),
+      },
+    },
     defineField({
       ...richText,
       description: "Enter the body content of the Customer case.",