[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdenetwork/kopete/libkopete
From:       Martin Blumenstingl <darklight.xdarklight () googlemail ! com>
Date:       2011-04-23 13:19:29
Message-ID: 20110423131929.EEC89AC85D () svn ! kde ! org
[Download RAW message or body]

SVN commit 1228913 by mblumenstingl:

Fix "last seen" property when quitting kopete.

The contact list was saved too early so the "last seen" property was written to the \
contact list in the main memory. But this change was not written to the disk.

BUG: 266697
REVIEW: 6526
REVIEWBOARD: https://svn.reviewboard.kde.org/r/6526/



 M  +7 -9      kopetepluginmanager.cpp  


--- trunk/KDE/kdenetwork/kopete/libkopete/kopetepluginmanager.cpp #1228912:1228913
@@ -173,15 +173,6 @@
 
 	_kpmp->shutdownMode = PluginManagerPrivate::ShuttingDown;
 
-
-	/* save the contact list now, just in case a change was made very recently
-	   and it hasn't autosaved yet
-	   from a OO point of view, theses lines should not be there, but i don't
-	   see better place -Olivier
-	*/
-	Kopete::ContactList::self()->shutdown(); // Save and shutdown contact list
-	Kopete::AccountManager::self()->save();
-
 	// Remove any pending plugins to load, we're shutting down now :)
 	_kpmp->pluginsToLoad.clear();
 
@@ -200,6 +191,13 @@
 		current.value()->aboutToUnload();
 	}
 
+	// save the contact list now, just in case a change was made very recently
+	// and it hasn't autosaved yet
+	// from a OO point of view, theses lines should not be there, but i don't
+	// see better place -Olivier
+	Kopete::ContactList::self()->shutdown(); // Save and shutdown contact list
+	Kopete::AccountManager::self()->save();
+
 	// When running under valgrind, don't enable the timer because it will almost
 	// certainly fire due to valgrind's much slower processing
 #if defined(HAVE_VALGRIND_H) && !defined(NDEBUG) && defined(__i386__)


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic