Update app.R

app.R

@@ -61,8 +61,20 @@ pitch_colors <- c(
 
 # Function to convert date formats
 convert_date_format <- function(date_string, output_format = "yyyy") {
-  if (is.na(date_string) || date_string == "") {
-    return(NA)
+  # Handle NULL, zero-length, or NA inputs safely
+  if (is.null(date_string) || length(date_string) == 0) return(NA_character_)
+  if (inherits(date_string, "Date") || inherits(date_string, "POSIXct")) {
+    # Already a proper date object (common from parquet) — format directly
+    if (is.na(date_string)) return(NA_character_)
+    parsed_date <- as.Date(date_string)
+    if (output_format == "mdyy") {
+      return(gsub("/0", "/", gsub("^0", "", format(parsed_date, "%m/%d/%y"))))
+    } else {
+      return(format(parsed_date, "%Y-%m-%d"))
+    }
+  }
+  if (is.na(date_string) || identical(as.character(date_string), "")) {
+    return(NA_character_)
   }
   
   date_string <- as.character(date_string)
@@ -105,7 +117,22 @@ convert_date_columns <- function(df, output_format = "yyyy") {
   
   for (col in date_columns) {
     if (col %in% names(df)) {
-      df[[col]] <- sapply(df[[col]], function(x) convert_date_format(x, output_format), USE.NAMES = FALSE)
+      col_data <- df[[col]]
+      
+      # If the column is already a Date or POSIXct (common from parquet),
+      # format it directly instead of running through the regex-based parser
+      if (inherits(col_data, "Date") || inherits(col_data, "POSIXct")) {
+        if (output_format == "mdyy") {
+          df[[col]] <- ifelse(is.na(col_data), NA_character_,
+                              gsub("/0", "/", gsub("^0", "", format(as.Date(col_data), "%m/%d/%y"))))
+        } else {
+          df[[col]] <- ifelse(is.na(col_data), NA_character_,
+                              format(as.Date(col_data), "%Y-%m-%d"))
+        }
+      } else {
+        # Character column — use the string-parsing path
+        df[[col]] <- sapply(df[[col]], function(x) convert_date_format(x, output_format), USE.NAMES = FALSE)
+      }
     }
   }