feat: Enhance station label generation with detailed date ranges and resolution, and refine date range defaults and validation based on data resolution.

server.R

@@ -1,23 +1,73 @@
 # server.R for DWD App
 
 # Helper function for label generation
-generate_station_label <- function(name, id, state, start_date, end_date, detailed_summary) {
+generate_station_label <- function(name, id, state, start_date, end_date, detailed_summary,
+                                   overall_start = NULL, overall_end = NULL, resolution = NULL) {
     tryCatch(
         {
-            s <- as.Date(as.character(start_date), format = "%Y%m%d")
-            e <- as.Date(as.character(end_date), format = "%Y%m%d")
+            format_range <- function(start_val, end_val) {
+                format_val <- function(value, is_end = FALSE) {
+                    if (is.null(value) || is.na(value) || value == "") {
+                        return("Unknown")
+                    }
+
+                    if (inherits(value, "Date")) {
+                        return(format(value, "%Y-%m-%d"))
+                    }
+
+                    value <- as.character(value)
+                    if (value == "99999999") {
+                        return(if (is_end) "Present" else "Unknown")
+                    }
+                    if (value == "00000000") {
+                        return("Unknown")
+                    }
+
+                    if (grepl("^\\d{4}-\\d{2}-\\d{2}$", value)) {
+                        d <- as.Date(value)
+                    } else {
+                        d <- as.Date(value, format = "%Y%m%d")
+                    }
+                    if (is.na(d)) {
+                        return("Unknown")
+                    }
+                    format(d, "%Y-%m-%d")
+                }
+
+                paste0(format_val(start_val, FALSE), " to ", format_val(end_val, TRUE))
+            }
+
+            res_label <- if (is.null(resolution) || is.na(resolution) || resolution == "") {
+                "Data"
+            } else {
+                paste0(toupper(substr(resolution, 1, 1)), substr(resolution, 2, nchar(resolution)))
+            }
+
+            station_range <- format_range(start_date, end_date)
+            coverage_start <- if (is.null(overall_start)) start_date else overall_start
+            coverage_end <- if (is.null(overall_end)) end_date else overall_end
+            coverage_range <- format_range(coverage_start, coverage_end)
+            summary_text <- if (is.null(detailed_summary) || is.na(detailed_summary) || detailed_summary == "") {
+                "No Data"
+            } else {
+                detailed_summary
+            }
 
             paste0(
                 "<div style='font-size:18px; max-width: 400px;'>",
                 "<b>", htmltools::htmlEscape(name), "</b> (", id, ")<br>",
                 "<span style='font-size:80%;'>State: ", htmltools::htmlEscape(state), "</span><br>",
                 "<span style='font-size:80%; color:#555;'>",
-                "Station Active: ", format(s, "%Y-%m-%d"), " &ndash; ", format(e, "%Y-%m-%d"),
+                "Station Active: ", station_range,
+                "</span><br>",
+                "<span style='font-size:80%; color:#555;'>",
+                res_label, " Coverage: ", coverage_range,
                 "</span><br>",
-                "<div style='font-size:80%; color:#333; font-weight:bold; margin-top:5px;'>Data Availability:</div>",
+                "<div style='font-size:80%; color:#333; font-weight:bold; margin-top:5px;'>",
+                "Data Availability (", res_label, "):</div>",
                 "<div style='font-size:75%; color:#333; line-height: 1.1;'>• ",
                 paste0(
-                    gsub(", ", "<br>• ", htmltools::htmlEscape(detailed_summary)),
+                    gsub(", ", "<br>• ", htmltools::htmlEscape(summary_text)),
                     "<br><small><i>Note: 'Present' indicates ongoing monitoring;<br>",
                     "some parameters may have<br>",
                     "upload delays.</i></small>"
@@ -46,20 +96,25 @@ server <- function(input, output, session) {
         {
             session$sendCustomMessage("freezeUI", list(text = "Switching resolution..."))
 
-            # Update date range defaults for Monthly resolution
-            if (tolower(input$data_resolution) == "monthly") {
-                tryCatch(
-                    {
-                        # For Monthly: default to last 6 years
-                        new_end <- Sys.Date()
+            # Update date range based on resolution
+            res <- tolower(input$data_resolution)
+
+            tryCatch(
+                {
+                    new_end <- Sys.Date()
+                    if (res %in% c("monthly", "daily")) {
+                        # For Monthly/Daily: default to last 6 years
                         new_start <- new_end - (365 * 6)
-                        updateDateRangeInput(session, "date_range", start = new_start, end = new_end)
-                    },
-                    error = function(e) {
-                        message("Note: Could not update date range for monthly: ", e$message)
+                    } else {
+                        # For Hourly: default to last 1 year (366 days)
+                        new_start <- new_end - 366
                     }
-                )
-            }
+                    updateDateRangeInput(session, "date_range", start = new_start, end = new_end)
+                },
+                error = function(e) {
+                    message("Note: Could not update date range: ", e$message)
+                }
+            )
         },
         priority = 1000,
         ignoreInit = TRUE
@@ -191,9 +246,13 @@ server <- function(input, output, session) {
 
         # Highlight & Zoom
         # Generate Label
+        res <- if (is.null(input$data_resolution)) NULL else tolower(input$data_resolution)
+        overall_start <- if ("station_overall_start" %in% names(s_meta)) s_meta$station_overall_start[1] else s_meta$start_date[1]
+        overall_end <- if ("station_overall_end" %in% names(s_meta)) s_meta$station_overall_end[1] else s_meta$end_date[1]
         lbl <- generate_station_label(
             s_meta$name[1], s_meta$id[1], s_meta$state[1],
-            s_meta$start_date[1], s_meta$end_date[1], s_meta$detailed_summary[1]
+            s_meta$start_date[1], s_meta$end_date[1], s_meta$detailed_summary[1],
+            overall_start, overall_end, res
         )
 
         highlight_selected_station(leafletProxy("map"), lng_val, lat_val, lbl)
@@ -268,15 +327,24 @@ server <- function(input, output, session) {
     # Update Markers
     observe({
         df <- filtered_stations()
-        req(df)
+        req(df, input$data_resolution, input$main_nav)
+
+        if (input$main_nav != "Map View") {
+            return()
+        }
 
+        res <- tolower(input$data_resolution)
+        n_rows <- nrow(df)
+        coverage_start <- if ("station_overall_start" %in% names(df)) df$station_overall_start else df$start_date
+        coverage_end <- if ("station_overall_end" %in% names(df)) df$station_overall_end else df$end_date
 
         # Pre-process label with HTML using Helper
-        df <- df %>% mutate(
-            label_html = purrr::pmap_chr(
-                list(name, id, state, start_date, end_date, detailed_summary),
-                generate_station_label
-            )
+        df$label_html <- purrr::pmap_chr(
+            list(
+                df$name, df$id, df$state, df$start_date, df$end_date,
+                df$detailed_summary, coverage_start, coverage_end, rep(res, n_rows)
+            ),
+            generate_station_label
         )
 
         proxy <- leafletProxy("map", data = df) %>%
@@ -323,9 +391,13 @@ server <- function(input, output, session) {
         s_meta <- all_stations() %>% filter(id == id_val)
 
         lbl <- if (nrow(s_meta) > 0) {
+            res <- if (is.null(input$data_resolution)) NULL else tolower(input$data_resolution)
+            overall_start <- if ("station_overall_start" %in% names(s_meta)) s_meta$station_overall_start[1] else s_meta$start_date[1]
+            overall_end <- if ("station_overall_end" %in% names(s_meta)) s_meta$station_overall_end[1] else s_meta$end_date[1]
             generate_station_label(
                 s_meta$name[1], s_meta$id[1], s_meta$state[1],
-                s_meta$start_date[1], s_meta$end_date[1], s_meta$detailed_summary[1]
+                s_meta$start_date[1], s_meta$end_date[1], s_meta$detailed_summary[1],
+                overall_start, overall_end, res
             )
         } else {
             paste0("<b>", id_val, "</b>")
@@ -362,9 +434,12 @@ server <- function(input, output, session) {
         # Calculate current diff
         diff <- as.numeric(difftime(d_end, d_start, units = "days"))
 
-        # Max window depends on resolution: 10 years (3652 days) for Monthly, 1 year (366 days) for others
-        is_monthly <- tolower(input$data_resolution) == "monthly"
-        max_days <- if (is_monthly) 3652 else 366
+        # Max window depends on resolution:
+        # Monthly/Daily: ~10 years (3652 days) to allow 6-year views
+        # Hourly: 1 year (366 days)
+        res <- tolower(input$data_resolution)
+        is_extended <- res %in% c("monthly", "daily")
+        max_days <- if (is_extended) 3652 else 366
         min_days <- 28
 
         if (diff > max_days || diff < min_days) {
@@ -1042,9 +1117,13 @@ server <- function(input, output, session) {
         lng_val <- s$longitude[1]
         # Highlight & Zoom
         # Generate Label
+        res <- if (is.null(input$data_resolution)) NULL else tolower(input$data_resolution)
+        overall_start <- if ("station_overall_start" %in% names(s)) s$station_overall_start[1] else s$start_date[1]
+        overall_end <- if ("station_overall_end" %in% names(s)) s$station_overall_end[1] else s$end_date[1]
         lbl <- generate_station_label(
             s$name[1], s$id[1], s$state[1],
-            s$start_date[1], s$end_date[1], s$detailed_summary[1]
+            s$start_date[1], s$end_date[1], s$detailed_summary[1],
+            overall_start, overall_end, res
         )
 
         leafletProxy("map") %>%