D17272: Fix searches for rating 10 (5 stars)
Stefan BrĂ¼ns
noreply at phabricator.kde.org
Fri Nov 30 23:19:18 GMT 2018
bruns created this revision.
bruns added reviewers: Baloo, Frameworks, ngraham, poboiko, mgallien.
Herald added projects: Frameworks, Baloo.
Herald added a subscriber: kde-frameworks-devel.
bruns requested review of this revision.
REVISION SUMMARY
The comparator callback/lambda used a string comparision, which returned
the wrong documents, as "2" > "10", and "10" < "9", i.e. selecting the
"Highest rating" in dolphin returned all documents with a rating of 2
or higher.
Convert the numeric suffix of the rating term ("R2 <https://phabricator.kde.org/source/gcompris/>", "R10 <https://phabricator.kde.org/diffusion/10/>", ...) to
an integer value and use it for comparision.
BUG: 357960
REPOSITORY
R293 Baloo
BRANCH
submit
REVISION DETAIL
https://phabricator.kde.org/D17272
AFFECTED FILES
autotests/unit/engine/postingdbtest.cpp
src/engine/postingdb.cpp
src/engine/postingdb.h
src/engine/transaction.cpp
src/engine/transaction.h
src/lib/searchstore.cpp
To: bruns, #baloo, #frameworks, ngraham, poboiko, mgallien
Cc: kde-frameworks-devel, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181130/cccfe0e3/attachment.html>
More information about the Kde-frameworks-devel
mailing list