From 737de91bbc73ccf149681929af30ec486a203473 Mon Sep 17 00:00:00 2001
From: Patrick Toal <ptoal@takeflight.ca>
Date: Mon, 20 May 2024 21:53:09 -0400
Subject: [PATCH] Update naming

---
 .env.production                               |   2 +
 src/boot/appwrite.ts                          |  51 +++--
 .../scheduling/IntervalTemplateComponent.vue  |   7 +-
 .../boat/BoatScheduleTableComponent.vue       |   3 +-
 src/pages/schedule/ManageCalendar.vue         |  28 ++-
 src/stores/auth.ts                            |   2 +-
 src/stores/reservation.ts                     |   4 +-
 src/stores/schedule.ts                        | 178 +++++++-----------
 src/stores/schedule.types.ts                  |   5 +
 src/utils/misc.ts                             |   2 +
 src/utils/schedule.ts                         |  71 +++++++
 11 files changed, 201 insertions(+), 152 deletions(-)
 create mode 100644 .env.production
 create mode 100644 src/utils/schedule.ts

diff --git a/.env.production b/.env.production
new file mode 100644
index 0000000..52bb7a7
--- /dev/null
+++ b/.env.production
@@ -0,0 +1,2 @@
+APPWRITE_API_ENDPOINT='https://appwrite.oys.undock.ca/v1'
+APPWRITE_API_PROJECT='bab'
diff --git a/src/boot/appwrite.ts b/src/boot/appwrite.ts
index ac8feae..ea03ab5 100644
--- a/src/boot/appwrite.ts
+++ b/src/boot/appwrite.ts
@@ -17,20 +17,45 @@ if (process.env.APPWRITE_API_ENDPOINT && process.env.APPWRITE_API_PROJECT)
   client
     .setEndpoint(process.env.APPWRITE_API_ENDPOINT)
     .setProject(process.env.APPWRITE_API_PROJECT);
-
+else if (process.env.DEV) {
+  client
+    .setEndpoint('http://localhost:4000/api/v1')
+    .setProject('65ede55a213134f2b688');
+} else {
+  client.setEndpoint('https://appwrite.oys.undock.ca/v1').setProject('bab');
+}
 //TODO move this to config file
-const AppwriteIds = {
-  databaseId: '65ee1cbf9c2493faf15f',
-  collection: {
-    boat: '66341910003e287cd71c',
-    reservation: '663f8847000b8f5e29bb',
-    skillTags: '66072582a74d94a4bd01',
-    task: '65ee1cd5b550023fae4f',
-    taskTags: '65ee21d72d5c8007c34c',
-    timeBlock: '66361869002883fb4c4b',
-    timeBlockTemplate: '66361f480007fdd639af',
-  },
-};
+const AppwriteIds = process.env.DEV
+  ? {
+      databaseId: '65ee1cbf9c2493faf15f',
+      collection: {
+        boat: '66341910003e287cd71c',
+        reservation: '663f8847000b8f5e29bb',
+        skillTags: '66072582a74d94a4bd01',
+        task: '65ee1cd5b550023fae4f',
+        taskTags: '65ee21d72d5c8007c34c',
+        interval: '66361869002883fb4c4b',
+        intervalTemplate: '66361f480007fdd639af',
+      },
+      function: {
+        userinfo: 'userinfo',
+      },
+    }
+  : {
+      databaseId: 'bab_prod',
+      collection: {
+        boat: 'boat',
+        reservation: 'reservation',
+        skillTags: 'skillTags',
+        task: 'task',
+        taskTags: 'taskTags',
+        interval: 'interval',
+        intervalTemplate: 'intervalTemplate',
+      },
+      function: {
+        userinfo: '664038294b5473ef0c8d',
+      },
+    };
 
 const account = new Account(client);
 const databases = new Databases(client);
diff --git a/src/components/scheduling/IntervalTemplateComponent.vue b/src/components/scheduling/IntervalTemplateComponent.vue
index 48889ff..4ceab50 100644
--- a/src/components/scheduling/IntervalTemplateComponent.vue
+++ b/src/components/scheduling/IntervalTemplateComponent.vue
@@ -113,12 +113,9 @@
 </template>
 
 <script setup lang="ts">
-import {
-  copyIntervalTemplate,
-  timeTuplesOverlapped,
-  useScheduleStore,
-} from 'src/stores/schedule';
+import { useScheduleStore } from 'src/stores/schedule';
 import { IntervalTemplate } from 'src/stores/schedule.types';
+import { copyIntervalTemplate, timeTuplesOverlapped } from 'src/utils/schedule';
 import { ref } from 'vue';
 const alert = ref(false);
 const overlapped = ref();
diff --git a/src/components/scheduling/boat/BoatScheduleTableComponent.vue b/src/components/scheduling/boat/BoatScheduleTableComponent.vue
index a325575..a218f1d 100644
--- a/src/components/scheduling/boat/BoatScheduleTableComponent.vue
+++ b/src/components/scheduling/boat/BoatScheduleTableComponent.vue
@@ -91,7 +91,6 @@ const calendar = ref<QCalendarDay | null>(null);
 
 onMounted(async () => {
   await useBoatStore().fetchBoats();
-  await scheduleStore.fetchIntervals();
   await scheduleStore.fetchIntervalTemplates();
 });
 
@@ -177,7 +176,7 @@ function selectBlock(event: MouseEvent, scope: DayBodyScope, block: Interval) {
 
 const boatBlocks = computed((): Record<string, Interval[]> => {
   return scheduleStore
-    .getIntervalsForDate(selectedDate.value)
+    .getIntervals(selectedDate.value)
     .reduce((result, interval) => {
       if (!result[interval.boatId]) result[interval.boatId] = [];
       if (
diff --git a/src/pages/schedule/ManageCalendar.vue b/src/pages/schedule/ManageCalendar.vue
index 40d3a8c..4eddfd7 100644
--- a/src/pages/schedule/ManageCalendar.vue
+++ b/src/pages/schedule/ManageCalendar.vue
@@ -113,7 +113,7 @@
         /></q-item>
         <q-separator spaced />
         <IntervalTemplateComponent
-          v-for="template in timeblockTemplates"
+          v-for="template in intervalTemplates"
           :key="template.$id"
           :model-value="template"
         />
@@ -146,11 +146,7 @@ import {
   today,
 } from '@quasar/quasar-ui-qcalendar';
 import { Boat, useBoatStore } from 'src/stores/boat';
-import {
-  blocksOverlapped,
-  buildInterval,
-  useScheduleStore,
-} from 'src/stores/schedule';
+import { useScheduleStore } from 'src/stores/schedule';
 import { onMounted, ref } from 'vue';
 import type {
   Interval,
@@ -161,12 +157,13 @@ import { date } from 'quasar';
 import IntervalTemplateComponent from 'src/components/scheduling/IntervalTemplateComponent.vue';
 import NavigationBar from 'src/components/scheduling/NavigationBar.vue';
 import { storeToRefs } from 'pinia';
+import { buildInterval, intervalsOverlapped } from 'src/utils/schedule';
 
 const selectedDate = ref(today());
 const { fetchBoats } = useBoatStore();
 const scheduleStore = useScheduleStore();
 const { boats } = storeToRefs(useBoatStore());
-const { timeblockTemplates } = storeToRefs(useScheduleStore());
+const intervalTemplates = scheduleStore.getIntervalTemplates();
 const calendar = ref();
 const overlapped = ref();
 const alert = ref(false);
@@ -185,12 +182,11 @@ const newTemplate = ref<IntervalTemplate>({
 
 onMounted(async () => {
   await fetchBoats();
-  await scheduleStore.fetchIntervals();
   await scheduleStore.fetchIntervalTemplates();
 });
 
 const filteredIntervals = (date: Timestamp, boat: Boat) => {
-  return scheduleStore.getIntervals(date, boat).value;
+  return scheduleStore.getIntervals(date, boat);
 };
 
 const sortedIntervals = (date: Timestamp, boat: Boat) => {
@@ -210,7 +206,7 @@ function createTemplate() {
   newTemplate.value.$id = 'unsaved';
 }
 function createIntervals(boat: Boat, templateId: string, date: string) {
-  const intervals = timeBlocksFromTemplate(boat, templateId, date);
+  const intervals = intervalsFromTemplate(boat, templateId, date);
   intervals.forEach((interval) => scheduleStore.createInterval(interval));
 }
 
@@ -218,12 +214,14 @@ function getIntervals(date: Timestamp, boat: Boat) {
   return scheduleStore.getIntervals(date, boat);
 }
 
-function timeBlocksFromTemplate(
+function intervalsFromTemplate(
   boat: Boat,
   templateId: string,
   date: string
 ): Interval[] {
-  const template = timeblockTemplates.value.find((t) => t.$id === templateId);
+  const template = scheduleStore
+    .getIntervalTemplates()
+    .value.find((t) => t.$id === templateId);
   return template
     ? template.timeTuples.map((timeTuple: TimeTuple) =>
         buildInterval(boat, timeTuple, date)
@@ -280,13 +278,13 @@ function onDrop(
     const templateId = e.dataTransfer.getData('ID');
     const date = scope.timestamp.date;
     const resource = scope.resource;
-    const existingIntervals = getIntervals(scope.timestamp, resource).value;
+    const existingIntervals = getIntervals(scope.timestamp, resource);
     const boatsToApply = type === 'head-day' ? boats.value : [resource];
     overlapped.value = boatsToApply
       .map((boat) =>
-        blocksOverlapped(
+        intervalsOverlapped(
           existingIntervals.concat(
-            timeBlocksFromTemplate(boat, templateId, date)
+            intervalsFromTemplate(boat, templateId, date)
           )
         )
       )
diff --git a/src/stores/auth.ts b/src/stores/auth.ts
index 9207b29..a14448d 100644
--- a/src/stores/auth.ts
+++ b/src/stores/auth.ts
@@ -38,7 +38,7 @@ export const useAuthStore = defineStore('auth', () => {
         userNames.value[id] = '';
         functions
           .createExecution(
-            '664038294b5473ef0c8d',
+            'userinfo',
             '',
             false,
             '/userinfo/' + id,
diff --git a/src/stores/reservation.ts b/src/stores/reservation.ts
index f8fa5ea..6243f5c 100644
--- a/src/stores/reservation.ts
+++ b/src/stores/reservation.ts
@@ -5,9 +5,9 @@ import { AppwriteIds, databases } from 'src/boot/appwrite';
 import { ID, Query } from 'appwrite';
 import { date } from 'quasar';
 import { Timestamp, parseDate, today } from '@quasar/quasar-ui-qcalendar';
+import { LoadingTypes } from 'src/utils/misc';
 
 export const useReservationStore = defineStore('reservation', () => {
-  type LoadingTypes = 'loaded' | 'pending' | undefined;
   const reservations = ref<Map<string, Reservation>>(new Map());
   const datesLoaded = ref<Record<string, LoadingTypes>>({});
 
@@ -39,7 +39,7 @@ export const useReservationStore = defineStore('reservation', () => {
       setDateLoaded(startDate, endDate, 'loaded');
     } catch (error) {
       console.error('Failed to fetch reservations', error);
-      setDateLoaded(startDate, endDate, undefined);
+      setDateLoaded(startDate, endDate, 'error');
     }
   };
   const createReservation = async (reservation: Reservation) => {
diff --git a/src/stores/schedule.ts b/src/stores/schedule.ts
index 69c8830..d3fd9dd 100644
--- a/src/stores/schedule.ts
+++ b/src/stores/schedule.ts
@@ -1,127 +1,78 @@
 import { defineStore } from 'pinia';
-import { ComputedRef, computed, ref } from 'vue';
+import { Ref, computed, ref } from 'vue';
 import { Boat } from './boat';
-import {
-  Timestamp,
-  parseDate,
-  parsed,
-  compareDate,
-} from '@quasar/quasar-ui-qcalendar';
+import { Timestamp } from '@quasar/quasar-ui-qcalendar';
 
-import { IntervalTemplate, TimeTuple, Interval } from './schedule.types';
+import { IntervalTemplate, Interval, IntervalRecord } from './schedule.types';
 import { AppwriteIds, databases } from 'src/boot/appwrite';
-import { ID, Models } from 'appwrite';
-
-export function arrayToTimeTuples(arr: string[]) {
-  const timeTuples: TimeTuple[] = [];
-  for (let i = 0; i < arr.length; i += 2) {
-    timeTuples.push([arr[i], arr[i + 1]]);
-  }
-  return timeTuples;
-}
-
-export function timeTuplesOverlapped(tuples: TimeTuple[]): Interval[] {
-  return blocksOverlapped(
-    tuples.map((tuples) => {
-      return {
-        boatId: '',
-        start: '01/01/2001 ' + tuples[0],
-        end: '01/01/2001 ' + tuples[1],
-      };
-    })
-  ).map((t) => {
-    return { ...t, start: t.start.split(' ')[1], end: t.end.split(' ')[1] };
-  });
-}
-
-export function blocksOverlapped(blocks: Interval[] | Interval[]): Interval[] {
-  return Array.from(
-    new Set(
-      blocks
-        .sort((a, b) => Date.parse(a.start) - Date.parse(b.start))
-        .reduce((acc: Interval[], block, i, arr) => {
-          if (i > 0 && block.start < arr[i - 1].end)
-            acc.push(arr[i - 1], block);
-          return acc;
-        }, [])
-    )
-  );
-}
-
-export function copyTimeTuples(tuples: TimeTuple[]): TimeTuple[] {
-  return tuples.map((t) => Object.assign([], t));
-}
-export function copyIntervalTemplate(
-  template: IntervalTemplate
-): IntervalTemplate {
-  return {
-    ...template,
-    timeTuples: copyTimeTuples(template.timeTuples || []),
-  };
-}
-
-export function buildInterval(
-  resource: Boat,
-  time: TimeTuple,
-  blockDate: string
-): Interval {
-  /* When the time zone offset is absent, date-only forms are interpreted
-     as a UTC time and date-time forms are interpreted as local time. */
-  const result = {
-    boatId: resource.$id,
-    start: new Date(blockDate + 'T' + time[0]).toISOString(),
-    end: new Date(blockDate + 'T' + time[1]).toISOString(),
-  };
-  return result;
-}
+import { ID, Models, Query } from 'appwrite';
+import { arrayToTimeTuples } from 'src/utils/schedule';
 
 export const useScheduleStore = defineStore('schedule', () => {
   // TODO: Implement functions to dynamically pull this data.
-  const intervals = ref<Interval[]>([]);
+  const intervals = ref<Map<string, Interval>>(new Map());
+  const intervalDates = ref<IntervalRecord>({});
   const intervalTemplates = ref<IntervalTemplate[]>([]);
 
-  const getIntervals = (
-    date: Timestamp,
-    boat: Boat
-  ): ComputedRef<Interval[]> => {
-    return computed(() =>
-      intervals.value.filter((block) => {
-        return (
-          compareDate(parseDate(new Date(block.start)) as Timestamp, date) &&
-          block.boatId === boat.$id
-        );
-      })
-    );
+  const getIntervals = (date: Timestamp | string, boat?: Boat): Interval[] => {
+    const searchDate = typeof date === 'string' ? date : date.date;
+    const dayStart = new Date(searchDate + 'T00:00');
+    const dayEnd = new Date(searchDate + 'T23:59');
+    if (!intervalDates.value[searchDate]) {
+      intervalDates.value[searchDate] = 'pending';
+      fetchIntervals(searchDate);
+    }
+    return computed(() => {
+      return Array.from(intervals.value.values()).filter((interval) => {
+        const intervalStart = new Date(interval.start);
+        const intervalEnd = new Date(interval.end);
+
+        const isWithinDay = intervalStart < dayEnd && intervalEnd > dayStart;
+        const matchesBoat = boat ? boat.$id === interval.boatId : true;
+        return isWithinDay && matchesBoat;
+      });
+    }).value;
   };
 
-  const getIntervalsForDate = (date: string): Interval[] => {
-    // TODO: This needs to actually make sure we have the dates we need, stay in sync, etc.
-
-    return intervals.value.filter((b) => {
-      return compareDate(
-        parseDate(new Date(b.start)) as Timestamp,
-        parsed(date) as Timestamp
-      );
-    });
-  };
-
-  async function fetchIntervals() {
+  async function fetchIntervals(dateString: string) {
     try {
       const response = await databases.listDocuments(
         AppwriteIds.databaseId,
-        AppwriteIds.collection.timeBlock
+        AppwriteIds.collection.interval,
+        [
+          Query.greaterThanEqual(
+            'end',
+            new Date(dateString + 'T00:00').toISOString()
+          ),
+          Query.lessThanEqual(
+            'start',
+            new Date(dateString + 'T23:59').toISOString()
+          ),
+          Query.limit(50), // We are asuming that we won't have more than 50 intervals per day.
+        ]
       );
-      intervals.value = response.documents as Interval[];
+      response.documents.forEach((d) =>
+        intervals.value.set(d.$id, d as Interval)
+      );
+      intervalDates.value[dateString] = 'loaded';
+      console.info(`Loaded ${response.documents.length} intervals from server`);
     } catch (error) {
-      console.error('Failed to fetch timeblocks', error);
+      console.error('Failed to fetch intervals', error);
+      intervalDates.value[dateString] = 'error';
     }
   }
 
+  const getIntervalTemplates = (): Ref<IntervalTemplate[]> => {
+    // Should subscribe to get new intervaltemplates when they are created
+    if (!intervalTemplates.value) fetchIntervalTemplates();
+    return intervalTemplates;
+  };
+
   async function fetchIntervalTemplates() {
     try {
       const response = await databases.listDocuments(
         AppwriteIds.databaseId,
-        AppwriteIds.collection.timeBlockTemplate
+        AppwriteIds.collection.intervalTemplate
       );
       intervalTemplates.value = response.documents.map(
         (d: Models.Document): IntervalTemplate => {
@@ -140,11 +91,11 @@ export const useScheduleStore = defineStore('schedule', () => {
     try {
       const response = await databases.createDocument(
         AppwriteIds.databaseId,
-        AppwriteIds.collection.timeBlock,
+        AppwriteIds.collection.interval,
         ID.unique(),
         interval
       );
-      intervals.value.push(response as Interval);
+      intervals.value.set(response.$id, response as Interval);
     } catch (e) {
       console.error('Error creating Interval: ' + e);
     }
@@ -154,12 +105,12 @@ export const useScheduleStore = defineStore('schedule', () => {
       if (interval.$id) {
         const response = await databases.updateDocument(
           AppwriteIds.databaseId,
-          AppwriteIds.collection.timeBlock,
+          AppwriteIds.collection.interval,
           interval.$id,
           { ...interval, $id: undefined }
         );
-        intervals.value.push(response as Interval);
-        console.log(`Saved Interval: ${interval.$id}`);
+        intervals.value.set(response.$id, response as Interval);
+        console.info(`Saved Interval: ${interval.$id}`);
       } else {
         console.error('Update interval called without an ID');
       }
@@ -171,10 +122,11 @@ export const useScheduleStore = defineStore('schedule', () => {
     try {
       await databases.deleteDocument(
         AppwriteIds.databaseId,
-        AppwriteIds.collection.timeBlock,
+        AppwriteIds.collection.interval,
         id
       );
-      intervals.value = intervals.value.filter((block) => block.$id !== id);
+      intervals.value.delete(id);
+      console.info(`Deleted interval: ${id}`);
     } catch (e) {
       console.error('Error deleting Interval: ' + e);
     }
@@ -183,7 +135,7 @@ export const useScheduleStore = defineStore('schedule', () => {
     try {
       const response = await databases.createDocument(
         AppwriteIds.databaseId,
-        AppwriteIds.collection.timeBlockTemplate,
+        AppwriteIds.collection.intervalTemplate,
         ID.unique(),
         { name: template.name, timeTuple: template.timeTuples.flat(2) }
       );
@@ -196,7 +148,7 @@ export const useScheduleStore = defineStore('schedule', () => {
     try {
       await databases.deleteDocument(
         AppwriteIds.databaseId,
-        AppwriteIds.collection.timeBlockTemplate,
+        AppwriteIds.collection.intervalTemplate,
         id
       );
       intervalTemplates.value = intervalTemplates.value.filter(
@@ -213,7 +165,7 @@ export const useScheduleStore = defineStore('schedule', () => {
     try {
       const response = await databases.updateDocument(
         AppwriteIds.databaseId,
-        AppwriteIds.collection.timeBlockTemplate,
+        AppwriteIds.collection.intervalTemplate,
         id,
         {
           name: template.name,
@@ -234,10 +186,8 @@ export const useScheduleStore = defineStore('schedule', () => {
   };
 
   return {
-    timeblocks: intervals,
-    timeblockTemplates: intervalTemplates,
-    getIntervalsForDate,
     getIntervals,
+    getIntervalTemplates,
     fetchIntervals,
     fetchIntervalTemplates,
     createInterval,
diff --git a/src/stores/schedule.types.ts b/src/stores/schedule.types.ts
index 90d27a3..c3a60aa 100644
--- a/src/stores/schedule.types.ts
+++ b/src/stores/schedule.types.ts
@@ -1,4 +1,5 @@
 import { Models } from 'appwrite';
+import { LoadingTypes } from 'src/utils/misc';
 
 export type StatusTypes = 'tentative' | 'confirmed' | 'pending' | undefined;
 export type Reservation = Partial<Models.Document> & {
@@ -27,3 +28,7 @@ export type IntervalTemplate = Partial<Models.Document> & {
   name: string;
   timeTuples: TimeTuple[];
 };
+
+export interface IntervalRecord {
+  [key: string]: LoadingTypes;
+}
diff --git a/src/utils/misc.ts b/src/utils/misc.ts
index 18a12b8..cf7d7df 100644
--- a/src/utils/misc.ts
+++ b/src/utils/misc.ts
@@ -5,3 +5,5 @@ export function getNewId(): string {
   // Trivial placeholder
   //return Math.max(...reservations.value.map((item) => item.id)) + 1;
 }
+
+export type LoadingTypes = 'loaded' | 'pending' | 'error' | undefined;
diff --git a/src/utils/schedule.ts b/src/utils/schedule.ts
new file mode 100644
index 0000000..efbd733
--- /dev/null
+++ b/src/utils/schedule.ts
@@ -0,0 +1,71 @@
+import { Boat } from 'src/stores/boat';
+import {
+  Interval,
+  IntervalTemplate,
+  TimeTuple,
+} from 'src/stores/schedule.types';
+
+export function arrayToTimeTuples(arr: string[]) {
+  const timeTuples: TimeTuple[] = [];
+  for (let i = 0; i < arr.length; i += 2) {
+    timeTuples.push([arr[i], arr[i + 1]]);
+  }
+  return timeTuples;
+}
+
+export function timeTuplesOverlapped(tuples: TimeTuple[]): Interval[] {
+  return intervalsOverlapped(
+    tuples.map((tuples) => {
+      return {
+        boatId: '',
+        start: '01/01/2001 ' + tuples[0],
+        end: '01/01/2001 ' + tuples[1],
+      };
+    })
+  ).map((t) => {
+    return { ...t, start: t.start.split(' ')[1], end: t.end.split(' ')[1] };
+  });
+}
+
+export function intervalsOverlapped(
+  blocks: Interval[] | Interval[]
+): Interval[] {
+  return Array.from(
+    new Set(
+      blocks
+        .sort((a, b) => Date.parse(a.start) - Date.parse(b.start))
+        .reduce((acc: Interval[], block, i, arr) => {
+          if (i > 0 && block.start < arr[i - 1].end)
+            acc.push(arr[i - 1], block);
+          return acc;
+        }, [])
+    )
+  );
+}
+
+export function copyTimeTuples(tuples: TimeTuple[]): TimeTuple[] {
+  return tuples.map((t) => Object.assign([], t));
+}
+export function copyIntervalTemplate(
+  template: IntervalTemplate
+): IntervalTemplate {
+  return {
+    ...template,
+    timeTuples: copyTimeTuples(template.timeTuples || []),
+  };
+}
+
+export function buildInterval(
+  resource: Boat,
+  time: TimeTuple,
+  blockDate: string
+): Interval {
+  /* When the time zone offset is absent, date-only forms are interpreted
+     as a UTC time and date-time forms are interpreted as local time. */
+  const result = {
+    boatId: resource.$id,
+    start: new Date(blockDate + 'T' + time[0]).toISOString(),
+    end: new Date(blockDate + 'T' + time[1]).toISOString(),
+  };
+  return result;
+}