Messages marked as deleted are not considered vanished during sync
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 1k
- Forks
- 348
- Avg merge
- 12h 28m
- Merged PRs (30d)
- 91
Description
Steps to reproduce
- Have a mailbox with messages
- Configure the account in Mail but also with Evolution or a similar client
- Open Mail to show the mailbox
- Use Evolution to move a message that is visible in Mail to another mailbox
- Trigger a sync in Mail
Expected behavior
Mail syncs the mailbox against IMAP. That message does not disappear but is marked as deleted (until EXPUNGE is run). Mail will still put this message into the vanished IDs array so the front-end deletes it.
Actual behavior
The flag change is sync'ed to the DB, but the message ID is not added to vanished IDs because we don't filter for not deleted messages when we build the changed/vanished IDs.
Mail app
Mail app version: 1.11.0-dev / master
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in the mailbox sync path that updates the database and builds the changed/vanished IDs arrays; the issue does not name a file or test. Reproduce the IMAP move scenario, then verify that a message marked deleted is included in vanished IDs and removed by the front end after sync.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend, databases
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100