From 34ee29f79fc37d89b66ef8843441693d51b48f0c Mon Sep 17 00:00:00 2001 From: Matthew Raymer Date: Tue, 14 Oct 2025 06:17:20 +0000 Subject: [PATCH] refactor(receiver): remove setFetchTime() call from DailyNotificationReceiver - Remove setFetchTime() call as fetchedAt is now set in constructor - Align with new immutable timestamp model - Ensure consistent timestamp handling across all components This completes the migration to the new timestamp model where fetchedAt is immutable and set at object creation time. --- .../timesafari/dailynotification/DailyNotificationReceiver.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/android/plugin/src/main/java/com/timesafari/dailynotification/DailyNotificationReceiver.java b/android/plugin/src/main/java/com/timesafari/dailynotification/DailyNotificationReceiver.java index 76a8b1f..d2ad93f 100644 --- a/android/plugin/src/main/java/com/timesafari/dailynotification/DailyNotificationReceiver.java +++ b/android/plugin/src/main/java/com/timesafari/dailynotification/DailyNotificationReceiver.java @@ -203,7 +203,7 @@ public class DailyNotificationReceiver extends BroadcastReceiver { nextContent.setSound(content.isSound()); nextContent.setPriority(content.getPriority()); nextContent.setUrl(content.getUrl()); - nextContent.setFetchTime(System.currentTimeMillis()); + // fetchedAt is set in constructor, no need to set it again // Save to storage DailyNotificationStorage storage = new DailyNotificationStorage(context);