Omit sunrises and sunsets that are at epoch second 0

Fixes #31
davidtakac · Apr 19, 2024 · 24dad8d · 24dad8d
1 parent cb98d6b
commit 24dad8d
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/app/src/main/java/com/davidtakac/bura/forecast/ForecastDataDownloader.kt b/app/src/main/java/com/davidtakac/bura/forecast/ForecastDataDownloader.kt
@@ -70,10 +70,10 @@ class ForecastDataDownloader(private val userAgentProvider: UserAgentProvider) {
             val json = JSONObject(jsonString)
 
             val daily = json.getJSONObject("daily")
-            // When a day has no sunrise or sunset, Open-Meteo returns 1970-01-01, but the app
+            // When a day has no sunrise or sunset, Open-Meteo returns epoch second 0, but the app
             // expects an omitted timestamp. These filters drop such placeholders.
-            val sunrises = daily.getJSONArray("sunrise").mapToList(LocalDateTime::parse).filter { it != LocalDateTime.MIN }
-            val sunsets = daily.getJSONArray("sunset").mapToList(LocalDateTime::parse).filter { it != LocalDateTime.MIN }
+            val sunrises = daily.getJSONArray("sunrise").mapToList(LocalDateTime::parse).filter { it.year != 1970 }
+            val sunsets = daily.getJSONArray("sunset").mapToList(LocalDateTime::parse).filter { it.year != 1970 }
 
             val hourly = json.getJSONObject("hourly")