Improve SN site deletion and UI consistency

Adds @transactional to SN site deletion API and ensures related facet entities are deleted via cascade. Refactors navigation menu initialization in the SN site page to handle new and existing sites correctly. Updates dialog and sidebar components for improved styling and import consistency.

Author: alegauss

turing-app/src/main/java/com/viglet/turing/api/sn/console/TurSNSiteAPI.java

@@ -28,6 +28,7 @@
 
 import org.springframework.context.annotation.ComponentScan;
 import org.springframework.http.MediaType;
+import org.springframework.transaction.annotation.Transactional;
 import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
@@ -47,6 +48,7 @@
 import com.viglet.turing.persistence.model.sn.genai.TurSNSiteGenAi;
 import com.viglet.turing.persistence.model.sn.locale.TurSNSiteLocale;
 import com.viglet.turing.persistence.repository.sn.TurSNSiteRepository;
+import com.viglet.turing.persistence.repository.sn.field.TurSNSiteFieldExtFacetRepository;
 import com.viglet.turing.persistence.repository.sn.genai.TurSNSiteGenAiRepository;
 import com.viglet.turing.persistence.repository.sn.locale.TurSNSiteLocaleRepository;
 import com.viglet.turing.properties.TurConfigProperties;
@@ -71,6 +73,7 @@
 public class TurSNSiteAPI {
     private final TurSNSiteRepository turSNSiteRepository;
     private final TurSNSiteLocaleRepository turSNSiteLocaleRepository;
+    private final TurSNSiteFieldExtFacetRepository turSNSiteFieldExtFacetRepository;
     private final TurSNSiteGenAiRepository turSNSiteGenAiRepository;
     private final TurSNSiteExport turSNSiteExport;
     private final TurSNTemplate turSNTemplate;
@@ -81,6 +84,7 @@ public class TurSNSiteAPI {
 
     public TurSNSiteAPI(TurSNSiteRepository turSNSiteRepository,
             TurSNSiteLocaleRepository turSNSiteLocaleRepository,
+            TurSNSiteFieldExtFacetRepository turSNSiteFieldExtFacetRepository,
             TurSNSiteGenAiRepository turSNSiteGenAiRepository,
             TurSNSiteExport turSNSiteExport,
             TurSNTemplate turSNTemplate,
@@ -90,6 +94,7 @@ public TurSNSiteAPI(TurSNSiteRepository turSNSiteRepository,
             TurConfigProperties turConfigProperties) {
         this.turSNSiteRepository = turSNSiteRepository;
         this.turSNSiteLocaleRepository = turSNSiteLocaleRepository;
+        this.turSNSiteFieldExtFacetRepository = turSNSiteFieldExtFacetRepository;
         this.turSNSiteGenAiRepository = turSNSiteGenAiRepository;
         this.turSNSiteExport = turSNSiteExport;
         this.turSNTemplate = turSNTemplate;
@@ -172,8 +177,9 @@ public TurSNSite turSNSiteUpdate(@PathVariable String id, @RequestBody TurSNSite
 
     }
 
+    @Transactional
+    @Operation(summary = "Delete a Semantic Navigation Site")
     @DeleteMapping("/{id}")
-
     public boolean turSNSiteDelete(@PathVariable String id) {
         Optional<TurSNSite> turSNSite = turSNSiteRepository.findById(id);
         turSNSite.ifPresent(

turing-app/src/main/java/com/viglet/turing/persistence/model/sn/field/TurSNSiteFieldExt.java

@@ -26,6 +26,8 @@
 import java.util.HashSet;
 import java.util.Set;
 
+import org.hibernate.annotations.OnDelete;
+import org.hibernate.annotations.OnDeleteAction;
 import org.hibernate.annotations.UuidGenerator;
 
 import com.fasterxml.jackson.annotation.JsonBackReference;
@@ -76,8 +78,10 @@ public class TurSNSiteFieldExt implements Serializable {
     private String description;
     @Column(length = 50)
     private String facetName;
+
     @Builder.Default
     @OneToMany(mappedBy = "turSNSiteFieldExt", orphanRemoval = true, fetch = FetchType.LAZY, cascade = CascadeType.ALL)
+    @OnDelete(action = OnDeleteAction.CASCADE)
     private Set<TurSNSiteFieldExtFacet> facetLocales = new HashSet<>();
     @Builder.Default
     @Column

turing-react/src/app/console/sn/sn.site.page.tsx

@@ -18,62 +18,16 @@ import {
 import { useEffect, useState } from "react";
 import { useNavigate, useParams } from "react-router-dom";
 import { toast } from "sonner";
-
-const turSNSiteService = new TurSNSiteService();
-const data = {
+let data = {
     navMain: [
         {
             title: "Settings",
             url: "/detail",
             icon: IconSettings,
-        },
-        {
-            title: "Multi Languages",
-            url: "/locale",
-            icon: IconLanguage,
-        },
-        {
-            title: "Fields",
-            url: "/field",
-            icon: IconAlignBoxCenterStretch,
-        },
-        {
-            title: "Behavior",
-            url: "/behavior",
-            icon: IconScale,
-        },
-        {
-            title: "Facet Ordering",
-            url: "/facet-ordering",
-            icon: IconReorder,
-        },
-        {
-            title: "Generative AI",
-            url: "/ai",
-            icon: IconCpu2,
-        },
-        {
-            title: "Result Ranking",
-            url: "/result-ranking",
-            icon: IconNumber123,
-        },
-        {
-            title: "Merge Providers",
-            url: "/merge-providers",
-            icon: IconGitMerge,
-        },
-        {
-            title: "Spotlight",
-            url: "/spotlight",
-            icon: IconSpeakerphone,
-        },
-        {
-            title: "Top Search Terms",
-            url: "/top-terms",
-            icon: IconDashboard,
-        },
-    ],
-}
+        }]
+};
+const turSNSiteService = new TurSNSiteService();
+
 export default function SNSitePage() {
     const { id } = useParams() as { id: string };
     const [snSite, setSnSite] = useState<TurSNSite>({} as TurSNSite);
@@ -83,17 +37,83 @@ export default function SNSitePage() {
     const urlBase = `${ROUTES.SN_INSTANCE}/${id}`;
     useEffect(() => {
         if (id !== "new") {
+            data = {
+                navMain: [
+                    {
+                        title: "Settings",
+                        url: "/detail",
+                        icon: IconSettings,
+                    },
+                    {
+                        title: "Multi Languages",
+                        url: "/locale",
+                        icon: IconLanguage,
+                    },
+                    {
+                        title: "Fields",
+                        url: "/field",
+                        icon: IconAlignBoxCenterStretch,
+                    },
+                    {
+                        title: "Behavior",
+                        url: "/behavior",
+                        icon: IconScale,
+                    },
+                    {
+                        title: "Facet Ordering",
+                        url: "/facet-ordering",
+                        icon: IconReorder,
+                    },
+                    {
+                        title: "Generative AI",
+                        url: "/ai",
+                        icon: IconCpu2,
+                    },
+                    {
+                        title: "Result Ranking",
+                        url: "/result-ranking",
+                        icon: IconNumber123,
+                    },
+                    {
+                        title: "Merge Providers",
+                        url: "/merge-providers",
+                        icon: IconGitMerge,
+                    },
+                    {
+                        title: "Spotlight",
+                        url: "/spotlight",
+                        icon: IconSpeakerphone,
+                    },
+                    {
+                        title: "Top Search Terms",
+                        url: "/top-terms",
+                        icon: IconDashboard,
+                    },
+                ],
+            }
             turSNSiteService.get(id).then(setSnSite);
             setIsNew(false);
         }
+        else {
+            setSnSite({} as TurSNSite);
+            setIsNew(true);
+            data = {
+                navMain: [
+                    {
+                        title: "Settings",
+                        url: "/detail",
+                        icon: IconSettings,
+                    }]
+            };
+        }
+
     }, [id])
 
     async function onDelete() {
-        console.log("delete");
         try {
             if (await turSNSiteService.delete(snSite)) {
                 toast.success(`The ${snSite.name} Search Engine was deleted`);
-                navigate(urlBase);
+                navigate(`${ROUTES.SN_INSTANCE}`);
             } else {
                 toast.error(`The ${snSite.name} Search Engine was not deleted`);
             }

turing-react/src/components/dialog.delete.tsx

@@ -1,6 +1,6 @@
-import { Button } from "./ui/button";
-import React, { type Dispatch, type SetStateAction } from "react";
 import { IconTrash } from "@tabler/icons-react";
+import React, { type Dispatch, type SetStateAction } from "react";
+import { Button } from "./ui/button";
 import { Dialog, DialogContent, DialogDescription, DialogFooter, DialogHeader, DialogTitle, DialogTrigger } from "./ui/dialog";
 
 interface Props {
@@ -17,7 +17,7 @@ export const DialogDelete: React.FC<Props> = ({ feature, name, onDelete, open, s
         <DialogTrigger asChild>
           <Button variant={"outline"} className="mr-5"><IconTrash /></Button>
         </DialogTrigger>
-        <DialogContent className="sm:max-w-[450px]">
+        <DialogContent className="sm:max-w-150">
           <DialogHeader>
             <DialogTitle>Are you absolutely sure?</DialogTitle>
             <DialogDescription>
@@ -28,7 +28,7 @@ export const DialogDelete: React.FC<Props> = ({ feature, name, onDelete, open, s
             This action cannot be undone. This will permanently delete
             the {name} {feature}.
           </p>
-          <DialogFooter>
+          <DialogFooter className="sm:justify-center">
             <Button onClick={onDelete} variant="destructive">I understand
               the consequences, delete this {feature}</Button>
           </DialogFooter>

turing-react/src/components/sub.page.tsx

@@ -1,8 +1,8 @@
-import { NavLink, Outlet, useLocation } from "react-router-dom";
 import React, { type Dispatch, type SetStateAction } from "react";
+import { NavLink, Outlet, useLocation } from "react-router-dom";
+import { DialogDelete } from "./dialog.delete";
 import { Card } from "./ui/card";
 import { Sidebar, SidebarContent, SidebarGroup, SidebarGroupContent, SidebarHeader, SidebarInset, SidebarMenu, SidebarMenuAction, SidebarMenuButton, SidebarMenuItem, SidebarProvider } from "./ui/sidebar";
-import { DialogDelete } from "./dialog.delete";
 
 interface NavMainItem {
   title: string;
@@ -46,9 +46,9 @@ export const SubPage: React.FC<Props> = ({ icon: Icon, feature, name, urlBase, i
                 <SidebarMenuItem>
                   <SidebarMenuButton
                     asChild
-                    className="data-[slot=sidebar-menu-button]:!p-1.5">
+                    className="data-[slot=sidebar-menu-button]:p-1.5!">
                     <NavLink to="/admin">
-                      <Icon className="!size-5" />
+                      <Icon className="size-5!" />
                       {isNew ? (<span
                         className="text-base font-semibold">New {feature}</span>) : (
                         <span className="text-base font-semibold">{name}</span>)}