From 62acdcbc8817cd14630309c50cf0b9a87038420e Mon Sep 17 00:00:00 2001
From: survellow <59056368+survellow@users.noreply.github.com.>
Date: Sat, 24 Feb 2024 04:14:11 +0100
Subject: [PATCH] feat:#30 reactive URL parameters RoomOccupation

---
 docker-compose.yml                         |  2 +-
 frontend/src/components/RoomOccupation.vue | 69 +++++++++--------
 frontend/src/helpers/dates.ts              |  8 ++
 frontend/src/view/FreeRooms.vue            |  3 +-
 frontend/src/view/RoomFinder.vue           | 90 +++++++++++++---------
 reverseproxy.conf                          |  7 +-
 6 files changed, 105 insertions(+), 74 deletions(-)
 create mode 100644 frontend/src/helpers/dates.ts

diff --git a/docker-compose.yml b/docker-compose.yml
index 6e3e589..c3ec523 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -11,7 +11,7 @@ services:
       - "8090:8090"
     volumes:
       - pb_data:/htwkalender/data # for production with volume
-      # - ./backend/pb_data:/pb_data # for development with bind mount from project directory
+      # - ./backend:/htwkalender/data # for development with bind mount from project directory
 
   htwkalender-frontend:
     build:
diff --git a/frontend/src/components/RoomOccupation.vue b/frontend/src/components/RoomOccupation.vue
index 6a01952..5acb041 100644
--- a/frontend/src/components/RoomOccupation.vue
+++ b/frontend/src/components/RoomOccupation.vue
@@ -7,6 +7,9 @@ import { computed, ComputedRef, inject, ref, Ref, watch } from "vue";
 import { CalendarOptions, DatesSetArg, EventInput } from "@fullcalendar/core";
 import { fetchEventsByRoomAndDuration } from "../api/fetchRoom.ts";
 import { useI18n } from "vue-i18n";
+import allLocales from "@fullcalendar/core/locales-all";
+import router from "@/router";
+import { formatYearMonthDay } from "@/helpers/dates";
 const { t } = useI18n({ useScope: "global" });
 
 const props = defineProps({
@@ -14,11 +17,6 @@ const props = defineProps({
     type: String,
     required: true,
   },
-  date: {
-    type: Date,
-    required: false,
-    default: null,
-  },
 });
 
 type CalenderEvent = {
@@ -28,7 +26,28 @@ type CalenderEvent = {
   showFree: boolean;
 };
 
-const propDate: Ref<Date | null> = ref(props.date);
+const date: Ref<Date> = ref(new Date());
+
+// Watch for changes in URL parameter
+router.afterEach(setDateFromQuery);
+
+// Set the selected date from the URL
+function setDateFromQuery() {
+  const queryDate = router.currentRoute.value.query.date;
+  if (typeof queryDate === "string") {
+    if (queryDate === formatYearMonthDay(date.value)) {
+      return;
+    }
+    // date is in format like YYYYMMDD
+    // TODO check if date is valid
+    const year = queryDate.substring(0, 4);
+    const month = queryDate.substring(4, 6);
+    const day = queryDate.substring(6, 8);
+    date.value = new Date(`${year}-${month}-${day}`);
+  }
+}
+
+setDateFromQuery();
 
 const currentDateFrom: Ref<string> = ref("");
 const currentDateTo: Ref<string> = ref("");
@@ -71,15 +90,13 @@ async function getOccupation() {
     });
 }
 
-import allLocales from "@fullcalendar/core/locales-all";
-
 const calendarOptions: ComputedRef<CalendarOptions> = computed(() => ({
   locales: allLocales,
   locale: t("languageCode"),
   plugins: [dayGridPlugin, interactionPlugin, timeGridPlugin],
   // local debugging of mobilePage variable in object creation on ternary expression
   initialView: mobilePage.value ? "Day" : "week",
-  initialDate: propDate.value ? propDate.value : new Date(),
+  initialDate: date.value,
   dayHeaderFormat: { weekday: "short", omitCommas: true },
   slotDuration: "00:15:00",
   eventTimeFormat: {
@@ -131,28 +148,18 @@ const calendarOptions: ComputedRef<CalendarOptions> = computed(() => ({
   },
 
   datesSet: function (dateInfo: DatesSetArg) {
-    if (propDate.value !== null) {
-      console.debug("props.date: ", props.date);
-      const date = new Date(props.date);
-
-      // calculate the week start and end date which includes the date
-      const weekStart = new Date(date);
-      weekStart.setDate(date.getDate() - date.getDay() + 1);
-      const weekEnd = new Date(date);
-      weekEnd.setDate(date.getDate() - date.getDay() + 6);
-      currentDateFrom.value = weekStart.toISOString().split("T")[0];
-      currentDateTo.value = weekEnd.toISOString().split("T")[0];
-      propDate.value = null;
-    } else {
-      const view = dateInfo.view;
-      const offset = new Date().getTimezoneOffset();
-      const startDate = new Date(
-        view.activeStart.getTime() - offset * 60 * 1000,
-      );
-      const endDate = new Date(view.activeEnd.getTime() - offset * 60 * 1000);
-      currentDateFrom.value = startDate.toISOString().split("T")[0];
-      currentDateTo.value = endDate.toISOString().split("T")[0];
-    }
+    const view = dateInfo.view;
+    const offset = new Date().getTimezoneOffset();
+    const startDate = new Date(view.activeStart.getTime() - offset * 60 * 1000);
+    const endDate = new Date(view.activeEnd.getTime() - offset * 60 * 1000);
+    currentDateFrom.value = startDate.toISOString().split("T")[0];
+    currentDateTo.value = endDate.toISOString().split("T")[0];
+    router.replace({
+      query: {
+        ...router.currentRoute.value.query,
+        date: formatYearMonthDay(startDate),
+      },
+    });
     getOccupation();
   },
   events: function (
diff --git a/frontend/src/helpers/dates.ts b/frontend/src/helpers/dates.ts
new file mode 100644
index 0000000..48c3527
--- /dev/null
+++ b/frontend/src/helpers/dates.ts
@@ -0,0 +1,8 @@
+/**
+ * Format a date to a string in the format "YYYYMMDD".
+ * @param date - The date to format
+ * @returns string - The formatted date
+ */
+export function formatYearMonthDay(date: Date): string {
+  return date.toISOString().split("T")[0].replace(/-/g, "");
+}
diff --git a/frontend/src/view/FreeRooms.vue b/frontend/src/view/FreeRooms.vue
index 9b65bb1..4740528 100644
--- a/frontend/src/view/FreeRooms.vue
+++ b/frontend/src/view/FreeRooms.vue
@@ -134,6 +134,7 @@ import { requestFreeRooms } from "@/api/requestFreeRooms.ts";
 import { FilterMatchMode } from "primevue/api";
 import { padStart } from "@fullcalendar/core/internal";
 import router from "@/router";
+import { formatYearMonthDay } from "@/helpers/dates";
 
 const mobilePage = inject("mobilePage") as Ref<boolean>;
 const filters = ref({
@@ -146,7 +147,7 @@ function occupationRoute(room: string): void {
     name: "room-schedule",
     query: {
       room: room,
-      date: date.value.toISOString().split("T")[0].replace(/-/g, ""),
+      date: formatYearMonthDay(date.value),
     },
   });
 }
diff --git a/frontend/src/view/RoomFinder.vue b/frontend/src/view/RoomFinder.vue
index 8834715..54cd924 100644
--- a/frontend/src/view/RoomFinder.vue
+++ b/frontend/src/view/RoomFinder.vue
@@ -1,53 +1,67 @@
 <script lang="ts" setup>
-import { Ref, ref } from "vue";
+import { Ref, computed, ref, watch } from "vue";
 import { fetchRoom } from "../api/fetchRoom.ts";
 import DynamicPage from "./DynamicPage.vue";
 import RoomOccupation from "../components/RoomOccupation.vue";
 import { computedAsync } from "@vueuse/core";
+import router from "@/router";
 
-function isRoomAvailable(room: string, rooms: { name: string }[]) {
+type Room = {
+  name: string;
+};
+
+const selectedRoom: Ref<Room> = ref({ name: "" });
+
+// Watch for changes in URL parameter
+router.afterEach(async (to) => {
+  const room = to.query.room;
+  if (room && typeof room === "string") {
+    setRoomFromList(room, rooms.value);
+  }
+});
+
+const rooms = computedAsync<Set<string>>(async () => {
+  let rooms: Set<string> = new Set();
+  return await fetchRoom()
+    .then((data) => {
+      rooms = new Set(data);
+      return rooms;
+    })
+    .finally(() => {
+      const room = router.currentRoute.value.query.room;
+      if (room && typeof room === "string") {
+        // check if room is available in roomsList
+        setRoomFromList(room, rooms);
+      }
+    });
+}, new Set());
+
+const roomsList = computed(() => {
+  return Array.from(rooms.value).map((room) => {
+    return { name: room } as Room;
+  });
+});
+
+/**
+ * Set the room from the list of rooms
+ * @param room Name of the room
+ * @param rooms List of available rooms
+ */
+function setRoomFromList(room: string, rooms: Set<string>) {
   // wait for the roomsList to be available
-  const roomInList = rooms.some((r) => r.name === room);
+  const roomInList: boolean = rooms.has(room);
   if (roomInList) {
     selectedRoom.value.name = room;
   }
 }
 
-const selectedRoom: Ref<{ name: string }> = ref({ name: "" });
-
-// Get Query Parameters if available
-// room, date
-const urlParams = new URLSearchParams(window.location.search);
-const room = urlParams.get("room");
-
-const roomsList = computedAsync(async () => {
-  let rooms: { name: string }[] = [];
-  return await fetchRoom()
-    .then(
-      (data) =>
-        (rooms = data.map((room: string) => {
-          return { name: room };
-        })),
-    )
-    .finally(() => {
-      if (room) {
-        // check if room is available in roomsList
-        isRoomAvailable(room, rooms);
-      }
+watch(selectedRoom, (newRoom: Room) => {
+  if (newRoom.name !== "") {
+    router.push({
+      query: { ...router.currentRoute.value.query, room: newRoom.name },
     });
-}, []);
-
-const selectedDate: Ref<Date | undefined> = ref(undefined);
-
-// Set the selected date from the UR
-const date = urlParams.get("date");
-if (date) {
-  // date is in format like YYYYMMDD
-  const year = date.substring(0, 4);
-  const month = date.substring(4, 6);
-  const day = date.substring(6, 8);
-  selectedDate.value = new Date(`${year}-${month}-${day}`);
-}
+  }
+});
 </script>
 
 <template>
@@ -70,7 +84,7 @@ if (date) {
       />
     </template>
     <template #content>
-      <RoomOccupation :room="selectedRoom.name" :date="selectedDate" />
+      <RoomOccupation :room="selectedRoom.name" />
     </template>
   </DynamicPage>
 </template>
diff --git a/reverseproxy.conf b/reverseproxy.conf
index 9776180..79e4081 100644
--- a/reverseproxy.conf
+++ b/reverseproxy.conf
@@ -73,9 +73,10 @@ http {
     geo $admin {
         default 1;
         10.0.0.0/8 0;     # Private Network
-        192.168.0.0/24 0; # Localhost Network
+        127.0.0.0/8 0;    # Localhost Network
+        192.168.0.0/16 0; # Localhost Network
         141.57.0.0/16 0;  # HTWK Leipzig Network
-        172.18.0.0/24 0;  # Docker Internal Network
+        172.16.0.0/12 0;  # Private Network
     }
 
     map $admin $limit_key {
@@ -154,7 +155,7 @@ http {
             proxy_cache_lock   on;
             proxy_cache_use_stale timeout updating;
             add_header X-Proxy-Cache $upstream_cache_status;
-            limit_req zone=modules burst=5 nodelay;
+            limit_req zone=modules burst=30 nodelay;
         }
 
         location /api/courses {