[Erp5-dev] cherry-pick request: EmailReader compatability with various ImapServers is improved

[Boris Kocherov]

https://www.raskon.org/gitweb/?p=erp5.git;a=commit;h=193ffe5df78e401bfe750cd8348085dc58940458

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: Digital signature
URL: <http://mail.tiolive.com/pipermail/erp5-dev/attachments/20130902/22af1bf9/attachment.pgp>

[Arnaud Fontaine]

Hello,

Really sorry for the late reply.

Post by Boris Kocherov
https://www.raskon.org/gitweb/?p=erp5.git;a=commit;h=193ffe5df78e401bfe750cd8348085dc58940458

Thank you very much for this patch. I have applied this patch to my
branch ('arnau'). To be honest, I don't know so much about IMAP nor
erp5_email_reader, so before merging it I have some questions:

* Where is '/' used as a delimiter instead of '.'? (out of curiosity as
I have never seen that yet)

* Why 'ALL' instead of '( ALL )'?

* Another change done in this commit is about handling multiple flags?

Thanks!

Regards,

[Boris Kocherov]

Post by Arnaud Fontaine
Thank you very much for this patch. I have applied this patch to my
branch ('arnau'). To be honest, I don't know so much about IMAP nor
* Where is '/' used as a delimiter instead of '.'? (out of curiosity as
I have never seen that yet)

imap.mail.ru
and "very rarely used" imap.gmail.com

Post by Arnaud Fontaine
* Why 'ALL' instead of '( ALL )'?

gmail.com does not support '( ALL )'.
Dovecot server support both variants.

Post by Arnaud Fontaine
* Another change done in this commit is about handling multiple flags?

Does not. It can be needed for future.

IMAP implementation have another not important issue:
Multilingual charachters are not correctly displayed in folders names.

Best regards,
Boris Kocherov

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: Digital signature
URL: <http://mail.tiolive.com/pipermail/erp5-dev/attachments/20130911/93c883f5/attachment.pgp>

[Arnaud Fontaine]

Hello,

Post by Boris Kocherov

Post by Arnaud Fontaine
Thank you very much for this patch. I have applied this patch to my
branch ('arnau'). To be honest, I don't know so much about IMAP nor
* Where is '/' used as a delimiter instead of '.'? (out of curiosity as
I have never seen that yet)

imap.mail.ru
and "very rarely used" imap.gmail.com

Post by Arnaud Fontaine
* Why 'ALL' instead of '( ALL )'?

gmail.com does not support '( ALL )'.
Dovecot server support both variants.

Post by Arnaud Fontaine
* Another change done in this commit is about handling multiple flags?

Does not. It can be needed for future.
Multilingual charachters are not correctly displayed in folders names.

Thanks for your reply. Would it be ok if I change the commit message
based on your reply (it will be merged anyhow):

EmailReader compatibility with various ImapServers is improved.

* imap.mail.ru and imap.gmail.com (very rarely) use '/' as a delimiter.
* gmail.com does not support '( ALL )', use 'ALL' instead.
* Handle multiple flags.

Regards,

[Boris Kocherov]

Post by Arnaud Fontaine
Thanks for your reply. Would it be ok if I change the commit message
EmailReader compatibility with various ImapServers is improved.
* imap.mail.ru and imap.gmail.com (very rarely) use '/' as a delimiter.
* gmail.com does not support '( ALL )', use 'ALL' instead.
* Handle multiple flags.

and "very rarely used" imap.gmail.com - it was just a joke that gmail.com is
rarely used. NO preposition 'BY' in the previous sentence, so your phrase
"imap.mail.ru and imap.gmail.com (very rarely) use '/' as a delimiter" isn't
correct. Many free famous imap services use '/' as a delimiter, gmail.com
and mail.ru - among this services.

my version message:

EmailReader compatibility with various ImapServers is improved.

* imap.mail.ru and imap.gmail.com use '/' as a delimiter.
* gmail.com does not support '( ALL )', use 'ALL' instead.
* parse multiple flags.

PS. no more ironic jokes

Best regards,
Boris Kocherov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: Digital signature
URL: <http://mail.tiolive.com/pipermail/erp5-dev/attachments/20130911/2e6fb92d/attachment.pgp>

[Arnaud Fontaine]

Post by Arnaud Fontaine
EmailReader compatibility with various ImapServers is improved.
* imap.mail.ru and imap.gmail.com use '/' as a delimiter.
* gmail.com does not support '( ALL )', use 'ALL' instead.
* parse multiple flags.

Thanks, merged and pushed to master.

Cheers,