updating page name and quiz name

nextjs/src/app/api/courses/[courseName]/modules/[moduleName]/pages/[pageName]/route.ts

@@ -16,27 +16,48 @@ export const GET = async (
     return Response.json(settings);
   });
 
-  export const PUT = async (
+export const PUT = async (
   request: Request,
   {
     params: { courseName, moduleName, pageName },
   }: { params: { courseName: string; moduleName: string; pageName: string } }
-  ) =>
+) =>
   await withErrorHandling(async () => {
-      const page = await request.json();
+    const { page, previousModuleName, previousPageName } = await request.json();
-      await fileStorageService.pages.updatePage(courseName, moduleName, pageName, page);
+    await fileStorageService.pages.updatePage(
+      courseName,
+      moduleName,
+      pageName,
+      page
+    );
+
+    if (
+      previousModuleName &&
+      previousPageName &&
+      (page.name !== previousPageName || moduleName !== previousModuleName)
+    ) {
+      fileStorageService.pages.delete({
+        courseName,
+        moduleName: previousModuleName,
+        pageName: previousPageName,
+      });
+    }
     return Response.json({});
   });
 
-    export const POST = async (
+export const POST = async (
   request: Request,
   {
     params: { courseName, moduleName, pageName },
   }: { params: { courseName: string; moduleName: string; pageName: string } }
-    ) =>
+) =>
   await withErrorHandling(async () => {
     const page = await request.json();
-        await fileStorageService.pages.updatePage(courseName, moduleName, pageName, page);
+    await fileStorageService.pages.updatePage(
+      courseName,
+      moduleName,
+      pageName,
+      page
+    );
     return Response.json({});
   });
-      

nextjs/src/app/course/[courseName]/modules/ExpandableModule.tsx

@@ -122,6 +122,7 @@ export default function ExpandableModule({
             <div className="grid grid-cols-[auto_1fr]">
               {moduleItems.map(({ type, item }) => (
                 <ExpandableModuleItem
+                  key={item.name + type}
                   type={type}
                   item={item}
                   moduleName={moduleName}

nextjs/src/app/course/[courseName]/modules/[moduleName]/page/[pageName]/EditPage.tsx

@@ -11,6 +11,9 @@ import PagePreview from "./PagePreview";
 import { useLocalCourseSettingsQuery } from "@/hooks/localCourse/localCoursesHooks";
 import EditPageButtons from "./EditPageButtons";
 import ClientOnly from "@/components/ClientOnly";
+import { getModuleItemUrl } from "@/services/urlUtils";
+import { useRouter } from "next/navigation";
+import { useCourseContext } from "@/app/course/[courseName]/context/courseContext";
 
 export default function EditPage({
   moduleName,
@@ -19,6 +22,8 @@ export default function EditPage({
   pageName: string;
   moduleName: string;
 }) {
+  const router = useRouter();
+  const { courseName } = useCourseContext();
   const { data: page } = usePageQuery(moduleName, pageName);
   const updatePage = useUpdatePageMutation();
   const [pageText, setPageText] = useState(
@@ -37,10 +42,24 @@ export default function EditPage({
           localPageMarkdownUtils.toMarkdown(updatedPage)
         ) {
           console.log("updating page");
-          updatePage.mutate({
+          updatePage
+            .mutateAsync({
               page: updatedPage,
               moduleName,
-            pageName,
+              pageName: updatedPage.name,
+              previousModuleName: moduleName,
+              previousPageName: pageName,
+            })
+            .then(() => {
+              if (updatedPage.name !== pageName)
+                router.push(
+                  getModuleItemUrl(
+                    courseName,
+                    moduleName,
+                    "page",
+                    updatedPage.name
+                  )
+                );
             });
         }
         setError("");
@@ -70,10 +89,7 @@ export default function EditPage({
       </div>
       {settings.canvasId && (
         <ClientOnly>
-          <EditPageButtons
+          <EditPageButtons pageName={pageName} moduleName={moduleName} />
-            pageName={pageName}
-            moduleName={moduleName}
-          />
         </ClientOnly>
       )}
     </div>

nextjs/src/app/course/[courseName]/modules/[moduleName]/quiz/[quizName]/EditQuiz.tsx

@@ -10,6 +10,9 @@ import QuizPreview from "./QuizPreview";
 import { QuizButtons } from "./QuizButton";
 import ClientOnly from "@/components/ClientOnly";
 import { SuspenseAndErrorHandling } from "@/components/SuspenseAndErrorHandling";
+import { useRouter } from "next/navigation";
+import { getModuleItemUrl } from "@/services/urlUtils";
+import { useCourseContext } from "@/app/course/[courseName]/context/courseContext";
 
 const helpString = `QUESTION REFERENCE
 ---
@@ -59,6 +62,8 @@ export default function EditQuiz({
   quizName: string;
   moduleName: string;
 }) {
+  const router = useRouter();
+  const { courseName } = useCourseContext();
   const { data: quiz } = useQuizQuery(moduleName, quizName);
   const updateQuizMutation = useUpdateQuizMutation();
   const [quizText, setQuizText] = useState(quizMarkdownUtils.toMarkdown(quiz));
@@ -67,7 +72,7 @@ export default function EditQuiz({
 
   useEffect(() => {
     const delay = 1000;
-    const handler = setTimeout(() => {
+    const handler = setTimeout(async () => {
       try {
         console.log("checking if the same...");
         if (
@@ -77,10 +82,24 @@ export default function EditQuiz({
           )
         ) {
           const updatedQuiz = quizMarkdownUtils.parseMarkdown(quizText);
-          updateQuizMutation.mutate({
+          updateQuizMutation
+            .mutateAsync({
               quiz: updatedQuiz,
               moduleName,
-            quizName,
+              quizName: updatedQuiz.name,
+              previousModuleName: moduleName,
+              previousQuizName: quizName,
+            })
+            .then(() => {
+              if (updatedQuiz.name !== quizName)
+                router.push(
+                  getModuleItemUrl(
+                    courseName,
+                    moduleName,
+                    "quiz",
+                    updatedQuiz.name
+                  )
+                );
             });
         }
         setError("");
@@ -92,7 +111,7 @@ export default function EditQuiz({
     return () => {
       clearTimeout(handler);
     };
-  }, [moduleName, quiz, quizName, quizText, updateQuizMutation]);
+  }, [courseName, moduleName, quiz, quizName, quizText, router, updateQuizMutation]);
 
   return (
     <div className="h-full flex flex-col align-middle px-1">

nextjs/src/hooks/localCourse/pageHooks.ts

@@ -123,15 +123,19 @@ export const useUpdatePageMutation = () => {
         encodeURIComponent(moduleName) +
         "/pages/" +
         encodeURIComponent(pageName);
-      await axiosClient.put(url, page);
+      await axiosClient.put(url, {
+        page,
+        previousModuleName,
+        previousPageName,
+      });
     },
     onSuccess: (_, { moduleName, pageName }) => {
       queryClient.invalidateQueries({
         queryKey: localCourseKeys.page(courseName, moduleName, pageName),
       });
-      queryClient.invalidateQueries({
+      // queryClient.invalidateQueries({
-        queryKey: localCourseKeys.pageNames(courseName, moduleName),
+      //   queryKey: localCourseKeys.pageNames(courseName, moduleName),
-      });
+      // });
     },
   });
 };

nextjs/src/services/fileStorage/pageFileStorageService.ts

@@ -65,4 +65,23 @@ export const pageFileStorageService = {
       await fs.unlink(oldFilePath);
     }
   },
+  async delete({
+    courseName,
+    moduleName,
+    pageName,
+  }: {
+    courseName: string;
+    moduleName: string;
+    pageName: string;
+  }) {
+    const filePath = path.join(
+      basePath,
+      courseName,
+      moduleName,
+      "pages",
+      pageName + ".md"
+    );
+    console.log("removing page", filePath);
+    await fs.unlink(filePath)
+  }
 };