From c1fcadaefe26af0445a2a501d576deedaa1176d9 Mon Sep 17 00:00:00 2001
From: Eduard Gert <kontakt@eduardgert.de>
Date: Mon, 7 Oct 2024 17:31:42 +0200
Subject: [PATCH] Fix resetting acl groups on switching active toggle (#417)

---
 src/contexts/RoutesProvider.tsx        | 15 +++++++--------
 src/modules/routes/RouteActiveCell.tsx |  2 +-
 2 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/src/contexts/RoutesProvider.tsx b/src/contexts/RoutesProvider.tsx
index 219bf1f..6bf5f2d 100644
--- a/src/contexts/RoutesProvider.tsx
+++ b/src/contexts/RoutesProvider.tsx
@@ -24,7 +24,7 @@ const RoutesContext = React.createContext(
   },
 );
 
-export default function RoutesProvider({ children }: Props) {
+export default function RoutesProvider({ children }: Readonly<Props>) {
   const routeRequest = useApiCall<Route>("/routes", true);
   const { mutate } = useSWRConfig();
 
@@ -38,9 +38,7 @@ export default function RoutesProvider({ children }: Props) {
 
     notify({
       title: "Network " + route.network_id + "-" + route.network,
-      description: message
-        ? message
-        : "The network route was successfully updated",
+      description: message ?? "The network route was successfully updated",
       promise: routeRequest
         .put(
           {
@@ -56,7 +54,10 @@ export default function RoutesProvider({ children }: Props) {
             metric: toUpdate.metric ?? route.metric ?? 9999,
             masquerade: toUpdate.masquerade ?? route.masquerade ?? true,
             groups: toUpdate.groups ?? route.groups ?? [],
-            access_control_groups: toUpdate.access_control_groups ?? undefined,
+            access_control_groups:
+              toUpdate.access_control_groups ??
+              route.access_control_groups ??
+              undefined,
           },
           `/${route.id}`,
         )
@@ -75,9 +76,7 @@ export default function RoutesProvider({ children }: Props) {
   ) => {
     notify({
       title: "Network " + route.network_id + "-" + route.network,
-      description: message
-        ? message
-        : "The network route was successfully created",
+      description: message ?? "The network route was successfully created",
       promise: routeRequest
         .post({
           network_id: route.network_id,
diff --git a/src/modules/routes/RouteActiveCell.tsx b/src/modules/routes/RouteActiveCell.tsx
index 0044263..387822d 100644
--- a/src/modules/routes/RouteActiveCell.tsx
+++ b/src/modules/routes/RouteActiveCell.tsx
@@ -7,7 +7,7 @@ import { Route } from "@/interfaces/Route";
 type Props = {
   route: Route;
 };
-export default function RouteActiveCell({ route }: Props) {
+export default function RouteActiveCell({ route }: Readonly<Props>) {
   const { updateRoute } = useRoutes();
   const { mutate } = useSWRConfig();