mailbox-component
mailbox-component copied to clipboard
Console options for days instead of timestamp
Because RefreshMailboxCommand::refreshMailbox() uses imap_search() with 'SINCE "' . $timestamp->format('d F Y') . '"' which is 'SINCE "28 December 2021"', for example, so as not to mislead users that only message from one hour ago are fetched when using -t 60, instead of ~~timestamp~~ option we could use a days option for uvdesk:refresh-mailbox command.
So instead of running something like
php bin/console uvdesk:refresh-mailbox -t 2 [email protected]
they could run something like
php bin/console uvdesk:refresh-mailbox -d 1 [email protected]
to fetch emails since yesterday until now
or
php bin/console uvdesk:refresh-mailbox -d 0 [email protected]
to fetch only emails of curent day.
If you agree with this enhachment, i could post a pull request with this implementation.
For better approach for performance reasons on large mailboxes will be to use imap_search($stream, 'UID ' . $latest_uid . ':*', SE_UID);
.
@see https://stackoverflow.com/questions/32698415/php-imap-search-unseen-since-date-with-time