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.

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

\imap_header dosen't work with UID #44

Merged
merged 1 commit into from
Aug 11, 2014
Merged

Conversation

ysramirez
Copy link
Contributor

http://us1.php.net/manual/en/function.imap-header.php
http://us1.php.net/manual/en/function.imap-headerinfo.php

msg_number
The message number

http://us1.php.net/manual/en/function.imap-msgno.php
imap_msgno — Gets the message sequence number for the given UID

I don't know why PHP dosen't has a way to read the header with the UID but this is the solution.

http://us1.php.net/manual/en/function.imap-header.php
http://us1.php.net/manual/en/function.imap-headerinfo.php

 msg_number
    The message number

http://us1.php.net/manual/en/function.imap-msgno.php
imap_msgno — Gets the message sequence number for the given UID

I don't know why PHP dosen't has a way to read the header with the UID but this is the solution.
@cordoval
Copy link
Contributor

let's expedite this one please 😊 @ddeboer thanks! Of course if it is correct, though it seems it breaks tests.

@ddeboer
Copy link
Owner

ddeboer commented Aug 11, 2014

I'm going to add a test case and then merge this.

@ddeboer
Copy link
Owner

ddeboer commented Aug 11, 2014

@cordoval The tests on PRs always fail because the only way to test this library is functionally, against a live IMAP server. I use a GMail test account, but the encrypted password for that is not available on Travis for forks (this is a normal Travis security measure). If anyone has an idea how to improve this situation, please let me know. :)

@ddeboer ddeboer merged commit 6dd4cb2 into ddeboer:master Aug 11, 2014
@ysramirez
Copy link
Contributor Author

Thanks

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.

3 participants