<table><tr><td style="">leinir created this revision.<br />leinir added reviewers: ronaldv, KDE Store.<br />leinir added projects: KNewStuff, KDE Store.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />leinir requested review of this revision.
</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/D18038">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>After much discussion on the topic, it came to light that the specific grammar used for the default tag in KNSCore was incorrect, as it was an action verb (rather than an adjective the way the metadata tags are supposed to be). While this does mean that previous versions of Frameworks will not have the filtering functionality by default, it is otherwise non-invasive and causes no side effects apart from the server-suggested filtering working.</p>

<p>CCBUG: <a href="https://bugs.kde.org/show_bug.cgi?id=402888" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=402888</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Run the test tool without this patch: Items which are supposed to be excluded are not excluded<br />
Run the test tool with this patch: Items which are supposed to be excluded from listings are excluded.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18038">https://phabricator.kde.org/D18038</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/knewstuffentrytest.cpp<br />
src/core/engine.cpp<br />
src/core/engine.h<br />
src/core/tagsfilterchecker.cpp<br />
src/core/tagsfilterchecker.h<br />
tests/testdata/entry.xml</div></div></div><br /><div><strong>To: </strong>leinir, ronaldv, KDE Store<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ZrenBot, ngraham, bruns, akiraohgaki, alexanderschmidt, siyuandong, ronaldv, mikesomov, starbuck<br /></div>