fix(Shuttle): Fix Error Handling When Sending Timetable

Author: kcyoow

src/app/components/routemap/DotAnimation.tsx

@@ -70,7 +70,7 @@ export const getDotAnimationConfig = (
   tab: string,
   currTimetable: SingleShuttleSchedule,
 ) => {
-  if (currTimetable !== undefined) {
+  if (currTimetable.time !== '') {
     switch (tab) {
       case 'shuttlecoke_o':
         return [getDotAniTimetableConfig(currTimetable.type, 1)]

src/app/components/shuttle/Shuttle.tsx

@@ -446,6 +446,8 @@ export const Shuttle = ({ location }: ShuttleStop) => {
     const { t } = useTranslation()
 
     if (timetable.data === undefined) {
+      if (currTimetable[0].time !== '')
+        setCurrTimetable([{ type: 'NA', time: '' }])
       return <></>
     }
 
@@ -454,6 +456,8 @@ export const Shuttle = ({ location }: ShuttleStop) => {
     }
 
     if (timetable.status === 'error') {
+      if (currTimetable[0].time !== '')
+        setCurrTimetable([{ type: 'NA', time: '' }])
       // Timetable API error
       return (
         <>
@@ -471,6 +475,8 @@ export const Shuttle = ({ location }: ShuttleStop) => {
     }
 
     if (timetable.data.length === 0) {
+      if (currTimetable[0].time !== '')
+        setCurrTimetable([{ type: 'NA', time: '' }])
       // Timetable doesn't exist
       return (
         <>
@@ -482,17 +488,11 @@ export const Shuttle = ({ location }: ShuttleStop) => {
     }
 
     const filtered = timetable.data.filter((val) => isAfterCurrentTime(val))
-
-    // Send filtered[0](or also include filtered[1] when bus arrive simultaneously) Array to RouteVisual
-    // when filtered has been updated.
-    if (filtered[0] !== currTimetable[0]) {
-      if (filtered.length >= 2 && filtered[0].time === filtered[1].time)
-        setCurrTimetable([filtered[0], filtered[1]])
-      else setCurrTimetable([filtered[0]])
-    }
-
     const reverted = filtered.map((val) => convertUnixToTime(val))
+
     if (filtered.length === 0) {
+      if (currTimetable[0].time !== '')
+        setCurrTimetable([{ type: 'NA', time: '' }])
       // Buses are done for today. User should refresh after midnight.
       return (
         <>
@@ -503,6 +503,14 @@ export const Shuttle = ({ location }: ShuttleStop) => {
       )
     }
 
+    // Send filtered[0](or also include filtered[1] when bus arrive simultaneously) Array to RouteVisual
+    // when filtered has been updated.
+    if (filtered[0] !== currTimetable[0]) {
+      if (filtered.length >= 2 && filtered[0].time === filtered[1].time)
+        setCurrTimetable([filtered[0], filtered[1]])
+      else setCurrTimetable([filtered[0]])
+    }
+
     // Otherwise - normal case
     return (
       <>

src/app/context/TimeTableContext.tsx

@@ -22,7 +22,7 @@ export const TimeTableContextProvider = ({
 }: React.PropsWithChildren) => {
   const [currTimetable, setCurrTimetable] = useState<
     Array<SingleShuttleSchedule>
-  >([])
+  >([{ type: 'NA', time: '' }])
 
   const value = React.useMemo(
     () => ({ currTimetable, setCurrTimetable }),