Calling the "Show" group "hiddenGroup" is a bit surprising to the casual reader ;)

svn path=/trunk/KDE/kdebase/apps/; revision=932508

diff --git a/src/settings/servicessettingspage.cpp b/src/settings/servicessettingspage.cpp
index a6d0905290e298d7cf8c0cde068a0b78cf89f234..57e91e5dd55288a88f1a08b6cf80c46bb86884e2 100644 (file)
--- a/src/settings/servicessettingspage.cpp
+++ b/src/settings/servicessettingspage.cpp
@@ -61,14 +61,14 @@ ServicesSettingsPage::~ServicesSettingsPage()
 void ServicesSettingsPage::applySettings()
 {
     KConfig config("kservicemenurc", KConfig::NoGlobals);
 void ServicesSettingsPage::applySettings()
 {
     KConfig config("kservicemenurc", KConfig::NoGlobals);

-    KConfigGroup hiddenGroup = config.group("Show");
+    KConfigGroup showGroup = config.group("Show");

 
     const int count = m_servicesList->count();
     for (int i = 0; i < count; ++i) {
         QListWidgetItem* item = m_servicesList->item(i);
         const bool show = (item->checkState() == Qt::Checked);
         const QString service = item->data(Qt::UserRole).toString();
 
     const int count = m_servicesList->count();
     for (int i = 0; i < count; ++i) {
         QListWidgetItem* item = m_servicesList->item(i);
         const bool show = (item->checkState() == Qt::Checked);
         const QString service = item->data(Qt::UserRole).toString();

-        hiddenGroup.writeEntry(service, show);
+        showGroup.writeEntry(service, show);

     }
 }
 
     }
 }
 


@@ -93,7 +93,7 @@ bool ServicesSettingsPage::event(QEvent* event)
 void ServicesSettingsPage::loadServices()
 {
     const KConfig config("kservicemenurc", KConfig::NoGlobals);
 void ServicesSettingsPage::loadServices()
 {
     const KConfig config("kservicemenurc", KConfig::NoGlobals);

-    const KConfigGroup hiddenGroup = config.group("Show");
+    const KConfigGroup showGroup = config.group("Show");

 
     const KService::List entries = KServiceTypeTrader::self()->query("KonqPopupMenu/Plugin");
     foreach (const KSharedPtr<KService>& service, entries) {
 
     const KService::List entries = KServiceTypeTrader::self()->query("KonqPopupMenu/Plugin");
     foreach (const KSharedPtr<KService>& service, entries) {


@@ -108,7 +108,7 @@ void ServicesSettingsPage::loadServices()
                                                             m_servicesList);
                 const QString service = action.name();
                 item->setData(Qt::UserRole, service);
                                                             m_servicesList);
                 const QString service = action.name();
                 item->setData(Qt::UserRole, service);

-                const bool show = hiddenGroup.readEntry(service, true);
+                const bool show = showGroup.readEntry(service, true);

                 item->setCheckState(show ? Qt::Checked : Qt::Unchecked);
             }
         }
                 item->setCheckState(show ? Qt::Checked : Qt::Unchecked);
             }
         }