From: Kai Uwe Broulik <kde@privat.broulik.de>
Date: Fri, 25 Aug 2017 14:48:48 +0000 (+0200)
Subject: [AdditionalInfoDialog] Prevent hiding "text" role
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/dfbaf21218c7a96dd7b7d1c4275820cd94c920f5

[AdditionalInfoDialog] Prevent hiding "text" role

You cannot do this from the menu but the option was there (but ignored) in the "View Properties" dialog.

BUG: 302622

Differential Revision: https://phabricator.kde.org/D7386
---

diff --git a/src/settings/additionalinfodialog.cpp b/src/settings/additionalinfodialog.cpp
index 23a65b5ac..4589b5d44 100644
--- a/src/settings/additionalinfodialog.cpp
+++ b/src/settings/additionalinfodialog.cpp
@@ -69,9 +69,9 @@ AdditionalInfoDialog::AdditionalInfoDialog(QWidget* parent,
         QListWidgetItem* item = new QListWidgetItem(info.translation, m_listWidget);
         item->setCheckState(visibleRoles.contains(info.role) ? Qt::Checked : Qt::Unchecked);
 
-        const bool enable = (!info.requiresBaloo && !info.requiresIndexer) ||
+        const bool enable = ((!info.requiresBaloo && !info.requiresIndexer) ||
                             (info.requiresBaloo) ||
-                            (info.requiresIndexer && indexingEnabled);
+                            (info.requiresIndexer && indexingEnabled)) && info.role != "text";
 
         if (!enable) {
             item->setFlags(item->flags() & ~Qt::ItemIsEnabled);