From: Peter Penz <peter.penz19@gmail.com>
Date: Fri, 5 Nov 2010 17:46:40 +0000 (+0000)
Subject: Take care to not reset the preview-settings if they have not been changed.
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/22b050a1d04e22097bc9d149e8e73b2d7fdfcdcc?ds=sidebyside

Take care to not reset the preview-settings if they have not been changed.

CCBUG: 252254


svn path=/trunk/KDE/kdebase/apps/; revision=1193355
---

diff --git a/src/settings/general/previewssettingspage.cpp b/src/settings/general/previewssettingspage.cpp
index 265dc209f..77608d3a3 100644
--- a/src/settings/general/previewssettingspage.cpp
+++ b/src/settings/general/previewssettingspage.cpp
@@ -111,13 +111,15 @@ PreviewsSettingsPage::~PreviewsSettingsPage()
 
 void PreviewsSettingsPage::applySettings()
 {
-    m_enabledPreviewPlugins.clear();
     const int count = m_previewPluginsList->count();
-    for (int i = 0; i < count; ++i) {
-        const QListWidgetItem* item = m_previewPluginsList->item(i);
-        if (item->checkState() == Qt::Checked) {
-            const QString enabledPlugin = item->data(Qt::UserRole).toString();
-            m_enabledPreviewPlugins.append(enabledPlugin);
+    if (count > 0) {
+        m_enabledPreviewPlugins.clear();
+        for (int i = 0; i < count; ++i) {
+            const QListWidgetItem* item = m_previewPluginsList->item(i);
+            if (item->checkState() == Qt::Checked) {
+                const QString enabledPlugin = item->data(Qt::UserRole).toString();
+                m_enabledPreviewPlugins.append(enabledPlugin);
+            }
         }
     }