trent_larson/daily-notification-plugin
4
0
Fork 0
Code Issues Pull Requests 2 Projects Releases Wiki Activity

refactor(receiver): remove setFetchTime() call from DailyNotificationReceiver

Browse Source 
- 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.
This commit is contained in:
Matthew Raymer
2025-10-14 06:17:20 +00:00
parent dc4d342bef
commit 34ee29f79f
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
android/plugin/src/main/java/com/timesafari/dailynotification/DailyNotificationReceiver.java
View File

@@ -203,7 +203,7 @@ public class DailyNotificationReceiver extends BroadcastReceiver {
nextContent.setSound(content.isSound()); nextContent.setSound(content.isSound());
nextContent.setPriority(content.getPriority()); nextContent.setPriority(content.getPriority());
nextContent.setUrl(content.getUrl()); nextContent.setUrl(content.getUrl());
nextContent.setFetchTime(System.currentTimeMillis()); // fetchedAt is set in constructor, no need to set it again
// Save to storage // Save to storage
DailyNotificationStorage storage = new DailyNotificationStorage(context); DailyNotificationStorage storage = new DailyNotificationStorage(context);