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
Fixed fetching messages for campaign replies #6376
Conversation
…implementation of IMAP_SEARCH
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.
That's perfect!
@alanhartless Tested and it Works Perfectly with Gmail on 2.14.0 |
I've overwritten the two files of the fix (https://github.com/mautic/mautic/pull/6376/files/c124868b322b2f9bbc875d59376e8823d313c3ac#diff-917b16827cc7ddce112292c90eac2bd9) on my v2.14.0 instance. It didn't help. Still doesn't work. CronJob log with v2.14.0: With overwritten files: Did I do it right to test it? |
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.
Tested. Works for me.
4 approved reviews. We should merge it.
Any native speaker, please review our docs mautic/documentation#291 |
I am using mailjet (i have tried amazon ses as well) as send service and gmail for inbox and it never worked. |
Is it solved yet?? |
Please be sure you are submitting this against the staging branch.
Description:
The first time messages were fetched for campaign replies, the imap search "UNSEEN" was used. But subsequent calls used "UID n:*" where n is the UID of the last message processed. Turns out, PHP's implementation of imap_search does not support that criteria. This PR fixes this by using imap_fetch_overview for the UID criteria.
Steps to reproduce the bug:
php app/console mautic:emails:fetch
. If testing in the cloud, jobs are ran every 2 minutesSteps to test this PR: