From 55c940970013b69b0cdffd866f05e66fda8eeced Mon Sep 17 00:00:00 2001
From: Rudie Ekkelenkamp <ekkelenkamp@gmail.com>
Date: Mon, 6 May 2024 13:35:52 +0200
Subject: [PATCH] Period from actions response should be used as fallback for
 download. (#854)

* Period from actions response should be used as fallback for download.

* review: simplify parsePiDateTime.

---------

Co-authored-by: ekkelenk <rudie.ekkelenkamp@deltares.nl>
---
 .../TimeSeriesFileDownloadComponent.vue       | 21 +++++++++++++++++--
 .../timeseries/TimeSeriesComponent.vue        |  2 ++
 src/lib/display/DisplayConfig.ts              |  2 ++
 src/services/useDisplayConfig/index.ts        |  1 +
 src/services/useSsdPi/index.ts                |  1 +
 5 files changed, 25 insertions(+), 2 deletions(-)

diff --git a/src/components/download/TimeSeriesFileDownloadComponent.vue b/src/components/download/TimeSeriesFileDownloadComponent.vue
index 855104d6b..0b4475fde 100644
--- a/src/components/download/TimeSeriesFileDownloadComponent.vue
+++ b/src/components/download/TimeSeriesFileDownloadComponent.vue
@@ -46,6 +46,7 @@
 
 <script lang="ts" setup>
 import {
+  ActionsPeriodDate,
   DocumentFormat,
   filterActionsFilter,
   TimeSeriesFilter,
@@ -133,15 +134,30 @@ function isFilterActionsFilter(
 
 const baseUrl = configManager.get('VITE_FEWS_WEBSERVICES_URL')
 
+function parsePiDateTime(dateTime: ActionsPeriodDate | undefined) {
+  return dateTime ? `${dateTime.date}T${dateTime.time}Z` : dateTime
+}
+
+// use startTime and endTime if set, otherwise use the options from the store, otherwise use the period form the config
 function determineViewPeriod(): string {
   const _options = toValue(viewPeriodFromStore)
   let viewPeriod: string = ''
-  const startDate: Date | null | undefined = props.startTime
+  let startDate: Date | null | undefined = props.startTime
     ? props.startTime
     : _options?.startTime
-  const endDate: Date | null | undefined = props.endTime
+  let endDate: Date | null | undefined = props.endTime
     ? props.endTime
     : _options?.endTime
+
+  if (!startDate) {
+    const parsedStartDate = parsePiDateTime(props.config?.period?.startDate)
+    if (parsedStartDate) startDate = new Date(parsedStartDate)
+  }
+  if (!endDate) {
+    const parsedEndDate = parsePiDateTime(props.config?.period?.endDate)
+    if (parsedEndDate) endDate = new Date(parsedEndDate)
+  }
+
   if (startDate || endDate) {
     // if either startTime or endTime is set, use it.
     if (startDate) {
@@ -198,6 +214,7 @@ const downloadFile = (downloadFormat: string) => {
       return
     }
   }
+
   const timeSeriesFilter: TimeSeriesTopologyActionsFilter = {
     documentFormat: downloadFormat,
     nodeId: props.config?.nodeId ?? '',
diff --git a/src/components/timeseries/TimeSeriesComponent.vue b/src/components/timeseries/TimeSeriesComponent.vue
index f80e1ffe4..69dcd7b12 100644
--- a/src/components/timeseries/TimeSeriesComponent.vue
+++ b/src/components/timeseries/TimeSeriesComponent.vue
@@ -86,6 +86,7 @@ const props = withDefaults(defineProps<Props>(), {
       class: '',
       requests: [],
       subplots: [],
+      period: undefined,
     }
   },
   elevationChartConfig: () => {
@@ -98,6 +99,7 @@ const props = withDefaults(defineProps<Props>(), {
       class: '',
       requests: [],
       subplots: [],
+      period: undefined,
     }
   },
   displayType: DisplayType.TimeSeriesChart,
diff --git a/src/lib/display/DisplayConfig.ts b/src/lib/display/DisplayConfig.ts
index cc65c3504..7c3b5ef61 100644
--- a/src/lib/display/DisplayConfig.ts
+++ b/src/lib/display/DisplayConfig.ts
@@ -1,4 +1,5 @@
 import { ChartConfig } from '../charts/types/ChartConfig.js'
+import { ActionPeriod } from '@deltares/fews-pi-requests'
 
 export enum DisplayType {
   TimeSeriesChart = 'TimeSeriesChart',
@@ -13,5 +14,6 @@ export interface DisplayConfig {
   title: string
   class: string
   requests: any[]
+  period: ActionPeriod | undefined
   subplots: ChartConfig[]
 }
diff --git a/src/services/useDisplayConfig/index.ts b/src/services/useDisplayConfig/index.ts
index e4b8cb23e..b65369f58 100644
--- a/src/services/useDisplayConfig/index.ts
+++ b/src/services/useDisplayConfig/index.ts
@@ -62,6 +62,7 @@ function actionsResponseToDisplayConfig(
       class: 'singles',
       index: timeSeriesDisplayIndex,
       requests: result.requests,
+      period: result.config.timeSeriesDisplay.period,
       subplots,
     }
     displays.push(display)
diff --git a/src/services/useSsdPi/index.ts b/src/services/useSsdPi/index.ts
index 5b08b2a35..871a1d9a4 100644
--- a/src/services/useSsdPi/index.ts
+++ b/src/services/useSsdPi/index.ts
@@ -74,6 +74,7 @@ export function useSsdPi(
         title,
         class: 'singles',
         requests: result.requests,
+        period: undefined,
         subplots,
       }
       _displays.push(display)