<table><tr><td style="">nowicki created this revision.<br />nowicki added a reviewer: KDE PIM.<br />nowicki set the repository for this revision to rMAILCOMMON PIM: Common Mail Support.<br />Restricted Application added a project: KDE PIM.<br />Restricted Application added a subscriber: kde-pim.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D3181" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The goal of this patch is to fix mail filtering for non-IMAP resources. Currently the Mail Filter Agent will looks for inbox collections and only subscribe to items on these. The logic to find the inbox folders is to look for the default inbox collection or collections with remote identifier set to "INBOX". This will work fine for IMAP resources as their remote IDs follow the original IMAP folder name. For other resources, like Akonadi EWS the inbox folder is not guaranteed to have a meaningful name (in case of EWS this will be a long random-like base64 string).</p>

<p>The resource has the ability to indicate special collections in its tree, including the inbox collection. Unfortunately the folderIsInbox() method will not take this into account.</p>

<p>The patch changes the behaviour of the above method to not only use the default inbox collection but any collection that is marked as an inbox collection.</p>

<p>The only side-effect I have found is that the collection properties dialog in KMail will no longer allow changing the inbox collection icon as the code blocks this setting from all local or inbox collections. I guess this should be acceptable.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rMAILCOMMON PIM: Common Mail Support</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3181" rel="noreferrer">https://phabricator.kde.org/D3181</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/kernel/mailkernel.cpp</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>nowicki, KDE PIM<br /><strong>Cc: </strong>kde-pim, spencerb, dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>