Skip to content

fix: ensure unique call log items to prevent crashes#534

Merged
naveensingh merged 5 commits intomainfrom
naveensingh/fix-call-history-dup-crash
Aug 23, 2025
Merged

fix: ensure unique call log items to prevent crashes#534
naveensingh merged 5 commits intomainfrom
naveensingh/fix-call-history-dup-crash

Conversation

@naveensingh
Copy link
Copy Markdown
Member

@naveensingh naveensingh commented Aug 23, 2025

Type of change(s)

  • Bug fix
  • Feature / enhancement
  • Infrastructure / tooling (CI, build, deps, tests)
  • Documentation

What changed and why

  • Recent calls are now always de-duplicated and sorted after fetch in RecentsHelper. This prevents duplicated calls and date headers due to non-contiguous blocks in the list.
  • Search results are now sorted by dayCode ➜ then by startsWith(). This ensures proper sorting and also prevents date header duplication in search results.
  • Replaced RecentCall.getDayCode() with an init time property for slightly better performance.

Closes the following issue(s)

Checklist

  • I read the contribution guidelines.
  • I manually tested my changes on device/emulator (if applicable).
  • I updated the "Unreleased" section in CHANGELOG.md (if applicable).
  • All checks are passing.

Changed the day code format (used internally) to yyyy-MM-dd to help with sorting. Corrected sorting also solves duplicated date headers due to misplaced call entries (caused by `startsWith()` sort).
@naveensingh naveensingh merged commit 47b9a83 into main Aug 23, 2025
5 checks passed
@naveensingh naveensingh deleted the naveensingh/fix-call-history-dup-crash branch August 23, 2025 15:43
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.

App crashes when putting an b or c in the search bar

1 participant