<table><tr><td style="">leinir updated this revision to Diff 37047.<br />leinir edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-7mpmu3ynknnzori/">(Show Details)</a><br />leinir edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-w635zjgai6kr7oa/">(Show Details)</a><br />leinir added a comment.<br />Restricted Application edited subscribers, added: kde-frameworks-devel; removed: Frameworks.
</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/D6513">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">add test tool</li>
<li class="remarkup-list-item">add support for filtering by tags on both base entry and download items contained within that entry</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">This is to add support for the new tags support in Attica found in D6512, which is based on the proposal in T6133 to add tags support in the next version of OCS.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">This introduces filtering which is applied onto any entry before handing it to the framework user. Filters can be set either via knsrc files, or through the Engine API. A default filter is set to ensure that if the server suggests certain items should not be shown, those entries will be hidden from the user (this is already supported by the KDE Store, and this ensures we support that in our client side).<br />
<br />
It further adds a test tool to more easily perform testing on KNewStuff's functionality (originally based on the old knewstuff2 tester, but now with more functionality, and better visual feedback).<br />
<br />
</span>As it depends on D6512, this patch should consequently not be merged before that one is.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Run the test tool and ensure filters are applied correctly (the test entry data requests two of the entries be filtered out).</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D6513?vs=16201&id=37047">https://phabricator.kde.org/D6513?vs=16201&id=37047</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6513">https://phabricator.kde.org/D6513</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/knewstuffentrytest.cpp<br />
src/attica/atticaprovider.cpp<br />
src/core/CMakeLists.txt<br />
src/core/engine.cpp<br />
src/core/engine.h<br />
src/core/entryinternal.cpp<br />
src/core/entryinternal.h<br />
src/core/provider.h<br />
src/core/tagsfilterchecker.cpp<br />
src/core/tagsfilterchecker.h<br />
src/staticxml/staticxmlprovider.cpp<br />
tests/CMakeLists.txt<br />
tests/khotnewstuff_test-ui/main.qml<br />
tests/khotnewstuff_test.cpp<br />
tests/khotnewstuff_test.h<br />
tests/khotnewstuff_test.knsrc.in<br />
tests/knewstuff2_test.cpp<br />
tests/knewstuff2_test.h<br />
tests/knewstuff2_test.knsrc<br />
tests/testdata/entry.xml<br />
tests/testdata/provider.xml</div></div></div><br /><div><strong>To: </strong>leinir, KNewStuff, apol, KDE Store, whiting<br /><strong>Cc: </strong>kde-frameworks-devel, KNewStuff, michaelh, ZrenBot, ngraham, bruns, Frameworks<br /></div>