Skip to content

Fix stale event data on iOS and Android#81

Open
jfversluis wants to merge 5 commits intomainfrom
fix/stale-event-data-v2
Open

Fix stale event data on iOS and Android#81
jfversluis wants to merge 5 commits intomainfrom
fix/stale-event-data-v2

Conversation

@jfversluis
Copy link
Copy Markdown
Owner

Reapplies the stale event data fix that was in #77 (reverted from v4.1.0 release).

Fixes:

  • iOS: Call EventStore.Reset() before all read operations to clear cached data
  • Android: Switch GetEvents from Events table to Instances table for accurate date-range queries; fix timezone double-counting in date filters

Tests:

  • New xUnit test project with 12 tests (facade delegation + date filter TDD tests)

Fixes #51

jfversluis and others added 5 commits March 9, 2026 14:57
- Restore Deleted != 1 filter on Android Instances query to prevent
  soft-deleted events from reappearing
- Remove aggressive EventStore.Reset() from iOS single-item lookups
  (GetPlatformCalendar, GetPlatformEvent) while keeping it for listing
  methods (GetCalendars, GetEvents)
- Document recurring event expansion behavior in ICalendarStore XML docs
- Remove unused referenceOffset parameter from ToFilterTimestampMillis

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Use CalendarStore.ToFilterTimestampMillis() in Android GetEvents so
  the tested helper is actually called from production code
- Cache attendees and reminders by EventId in ToEventsFromInstances to
  avoid redundant N+1 queries for recurring event occurrences

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Extract BuildInstancesSelection() for testable SQL selection logic
- Extract generic GetOrAdd() cache helper, replacing dedicated methods
- Add 6 new tests: selection string construction (4) and cache behavior (2)
- Total test count: 18

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CalendarStore not seeing event changes.

1 participant