-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rewrite expunge #2701
Rewrite expunge #2701
Conversation
1dcbaa7
to
16aef39
Compare
16aef39
to
536b1a8
Compare
|
||
uidExpungeCommand.execute(connection, this); | ||
} else { | ||
UidSearchCommand searchCommand = new UidSearchCommand.Builder() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this warrants its own method. also, a reference to the rfc would be nice, since this type of fallback method from a spec is not very common
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...I don't think so, actually?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have moved the UidSearchCommand and it's execution into a new method getUnknownDeletedUids()
and have added a comment regarding the RFC at the top of the try block
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, I see. I meant the whole thing in the else block, maybe something like expungeWithoutUidPlus
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah ok, done
536b1a8
to
5ac3367
Compare
5ac3367
to
9654cf3
Compare
39156ee
to
524d8ba
Compare
Rebased in #2927 |
This PR rewrites the
expunge()
method inImapFolder
to ensure that only messages that were marked with the\Deleted
flag by K-9 Mail are expunged. This process is described in section 4.2.4 of RFC 4549