]> cloud.milkyroute.net Git - dolphin.git/commitdiff
fix(search): Allow to set empty type
authorIsmael Asensio <isma.af@gmail.com>
Wed, 13 Nov 2019 21:48:45 +0000 (22:48 +0100)
committerIsmael Asensio <isma.af@gmail.com>
Wed, 13 Nov 2019 21:53:18 +0000 (22:53 +0100)
Summary:
This small patch allows to set and empty `type` to the extended options, which will set the `Any Type` option (index 0).
It doesn't make any change in current code, since this case is not called, but it helps simplifying follow-up patches.

Test Plan:
No behavior changes with current code.
Calling `setFacetsType(QString())` actually sets the first option (`Any Type`)

Reviewers: elvisangelaccio, #dolphin

Reviewed By: elvisangelaccio, #dolphin

Subscribers: kfm-devel

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D25255

src/search/dolphinfacetswidget.cpp

index 1827192259dcd4e6afa1fbe6aff2db1eae594e94..ae05509e77fc6ffe8894ce4fd6a6d453b0163a17 100644 (file)
@@ -153,7 +153,7 @@ void DolphinFacetsWidget::setRatingTerm(const QString& term)
 
 void DolphinFacetsWidget::setFacetType(const QString& type)
 {
-    for (int index = 1; index <= m_typeSelector->count(); index++) {
+    for (int index = 0; index <= m_typeSelector->count(); index++) {
         if (type == m_typeSelector->itemData(index).toString()) {
             m_typeSelector->setCurrentIndex(index);
             break;