Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Should Message::delete() use the Message UID? #46

Closed
xelan opened this issue Aug 18, 2014 · 1 comment
Closed

Should Message::delete() use the Message UID? #46

xelan opened this issue Aug 18, 2014 · 1 comment

Comments

@xelan
Copy link
Contributor

xelan commented Aug 18, 2014

The method Message::move() has the CP_UID option set, so it uses UIDs for the messages. However, Message::delete() doesn't use the corresponding FT_UID option.
http://php.net/manual/de/function.imap-delete.php

Is this behaviour intentional or should delete also be able to use message UIDs?

Thank you very much!
@ddeboer
Copy link
Owner

ddeboer commented Aug 18, 2014

This is a bug and may well be related to #30.

ddeboer added a commit that referenced this issue Sep 12, 2014
@ddeboer
Merge pull request #54 from ctalbot/ctalbot-use-uid-msg-delete
aa4e208 
Message::delete sets the FT_UID flag.  Fixes #30 Fixes #46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ddeboer @xelan