fix restore connfig for raiseOnDesktop/ActivityChange

app/dockview.cpp

@@ -132,9 +132,8 @@ DockView::~DockView()
         m_configView->deleteLater();
     }*/
 
-    if (m_visibility) {
-        m_visibility->deleteLater();
-    }
+    if (m_visibility)
+        delete m_visibility;
 }
 
 void DockView::init()

app/visibilitymanager.cpp

@@ -66,6 +66,7 @@ VisibilityManagerPrivate::~VisibilityManagerPrivate()
 {
     wm->removeDockStruts(view->winId());
     wm->removeDock(view->winId());
+    saveConfig();
 }
 
 inline void VisibilityManagerPrivate::setMode(Dock::Visibility mode)
@@ -229,14 +230,12 @@ void VisibilityManagerPrivate::setBlockHiding(bool blockHiding)
 inline void VisibilityManagerPrivate::setTimerShow(int msec)
 {
     timerShow.setInterval(msec);
-    saveConfig();
     emit q->timerShowChanged();
 }
 
 inline void VisibilityManagerPrivate::setTimerHide(int msec)
 {
     timerHide.setInterval(msec);
-    saveConfig();
     emit q->timerHideChanged();
 }
 
@@ -415,8 +414,8 @@ inline void VisibilityManagerPrivate::saveConfig()
     config.writeEntry("visibility", static_cast<int>(mode));
     config.writeEntry("timerShow", timerShow.interval());
     config.writeEntry("timerHide", timerHide.interval());
-    config.writeEntry("raiseOnDesktopChange", false);
-    config.writeEntry("raiseOnActivityChange", false);
+    config.writeEntry("raiseOnDesktopChange", raiseOnDesktopChange);
+    config.writeEntry("raiseOnActivityChange", raiseOnActivityChange);
     view->containment()->configNeedsSaving();
 }
 
@@ -504,6 +503,8 @@ VisibilityManager::VisibilityManager(PlasmaQuick::ContainmentView *view)
 
 VisibilityManager::~VisibilityManager()
 {
+    qDebug() << "VisibilityManager deleting...";
+    delete d;
 }
 
 Dock::Visibility VisibilityManager::mode() const

containment/contents/config/main.xml

@@ -114,12 +114,5 @@
     <entry name="showOnlyCurrentActivity" type="Bool">
       <default>true</default>
     </entry>
-    <entry name="raiseOnDesktopChange" type="Bool">
-      <default>false</default>
-    </entry>
-    <entry name="raiseOnActivityChange" type="Bool">
-      <default>false</default>
-    </entry>
-
   </group>
 </kcfg>