<table><tr><td style="">knauss added a subscriber: ikloecker.<br />knauss added inline comments.
</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/D21444">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D21444#inline-123758">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dfaure</span> wrote in <span style="color: #4b4d51; font-weight: bold;">antispamconfig.cpp:65</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">qputenv in the test and qgetenv here.<br />
Or make the test set an extern bool to true. See e.g. kio_resolve_local_urls in kio.</p>

<p style="padding: 0; margin: 8px;">Or make the test closer to the regular case ;-)<br />
In fact I'm wondering why this says setReadDefaults(true), i.e. ignore user settings. That's usually for "Revert to defaults" buttons.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p style="padding: 0; margin: 8px;">qputenv in the test and qgetenv here.<br />
 Or make the test set an extern bool to true. See e.g. kio_resolve_local_urls in kio.</p></blockquote>

<p style="padding: 0; margin: 8px;">thx for the tips.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p style="padding: 0; margin: 8px;">Or make the test closer to the regular case ;-)</p></blockquote>

<p style="padding: 0; margin: 8px;">Well I want the tests to be working without installation and as non root I can't write to /usr/share. So I would need to control the default location and I don't see a way to achieve this.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p style="padding: 0; margin: 8px;">In fact I'm wondering why this says setReadDefaults(true), i.e. ignore user settings. That's usually for "Revert to defaults" buttons.</p></blockquote>

<p style="padding: 0; margin: 8px;">This I'm wondering, too. I tracked this line of code exists from the beginning of the git history and this file was added 2004 and that line was never touched since then. The commit message is:<br />
"Add optional graphical spam status to fancy headers. Based on patch by Patrick Audley." commited by <a href="https://phabricator.kde.org/p/ikloecker/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ikloecker</a>. But I expect Ingo also don't know why this line is needed.</p>

<p style="padding: 0; margin: 8px;">I would suggest to remove this line as it seems to me, that it breaks stuff (e.g. our tests) and I can't imagine what we gain if we keep this line intact. With removing the line we gain the possibility for users to change the "kmail.antispamrc" for their needs.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R94 PIM: Message Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21444">https://phabricator.kde.org/D21444</a></div></div><br /><div><strong>To: </strong>knauss, KDE PIM, dfaure<br /><strong>Cc: </strong>ikloecker, dvratil, kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss<br /></div>