<table><tr><td style="">leinir created this revision.<br />leinir added a project: Discover Software Store.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</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/D6191" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This code removes the custom pagination in KNSBackend in favour of using the internal pagination in KNSCore::Engine. It further removes the explicit call to request data, as this is already done by setting the search term, which caused duplicated results to be returned. Further, remove results already returned when KNS requests the view to be cleared, further reducing duplicate view entries.</p>

<p>nb: The fact setting the search term starts a new search is undocumented, which will need fixing (some thorough documentation work is ongoing in kns as part of a gsoc project)</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Start Discover without patch, notice duplicates for most KNS sources<br />
Start discover with patch, notice no duplicate entries for KNS sources</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R134 Discover Software Store</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6191" rel="noreferrer">https://phabricator.kde.org/D6191</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>libdiscover/backends/KNSBackend/KNSBackend.cpp<br />
libdiscover/backends/KNSBackend/KNSBackend.h</div></div></div><br /><div><strong>To: </strong>leinir, apol, Discover Software Store<br /><strong>Cc: </strong>plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas<br /></div>