Avoid duplicate lesson slugs

apbetioli · Jun 3, 2024 · 1bbf9cb · 1bbf9cb
1 parent ea656f5
commit 1bbf9cb
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 3 deletions.
diff --git a/app/admin/courses/[id]/course-form-lesson.tsx b/app/admin/courses/[id]/course-form-lesson.tsx
@@ -44,10 +44,17 @@ export function CourseFormLesson({
   }, [lesson])
 
   const handleInputChange = (name: string, value: any) => {
-    if (isNew && name === 'name') {
-      setCurrent({ ...current, [name]: value, slug: kebabCase(value) })
+    if (name === 'name') {
+      setCurrent({
+        ...current,
+        [name]: value,
+        slug: kebabCase(group.name + value),
+      })
     } else {
-      setCurrent({ ...current, [name]: value })
+      setCurrent({
+        ...current,
+        [name]: value,
+      })
     }
   }
 

diff --git a/lib/features/admin-slice.ts b/lib/features/admin-slice.ts
@@ -154,6 +154,15 @@ const adminSlice = createSlice({
       const groupToUpdate = findGroup(state, action.payload.group)
       groupToUpdate.changed = true
 
+      const exists = groupToUpdate.lessons.find(
+        (lesson) => lesson.slug === action.payload.lesson.slug,
+      )
+      if (exists) {
+        throw new Error(
+          'Lesson slug already exists! Please choose another name.',
+        )
+      }
+
       groupToUpdate.lessons.push(
         createLesson({
           ...action.payload.lesson,
@@ -170,6 +179,17 @@ const adminSlice = createSlice({
       const index = findLessonIndex(groupToUpdate, action.payload.lesson)
       groupToUpdate.changed = true
 
+      const exists = groupToUpdate.lessons.find(
+        (lesson) =>
+          lesson.slug === action.payload.lesson.slug &&
+          lesson.uiId !== action.payload.lesson.uiId,
+      )
+      if (exists) {
+        throw new Error(
+          'Lesson slug already exists! Please choose another name.',
+        )
+      }
+
       groupToUpdate.lessons.splice(index, 1, {
         ...action.payload.lesson,
         changed: true,