Take out getEvents() unintuitive call, and replace by calling directly initRegisterAccounts()

src/dbus/dbusmanagerimpl.cpp

@@ -33,7 +33,8 @@ DBusManagerImpl::exec(){
     _callManager = new CallManager(conn);
     _configurationManager = new ConfigurationManager(conn);
 
-    Manager::instance().getEvents();  // Register accounts
+    // Register accounts
+    Manager::instance().initRegisterAccounts(); //getEvents();
 
     _debug("Starting DBus event loop\n");
     _dispatcher.enter();

src/managerimpl.cpp

@@ -1252,11 +1252,14 @@ ManagerImpl::detachZeroconfEvents(Pattern::Observer& observer)
  *
  * @todo When is this called ? Why this name 'getEvents' ?
  */
+/**
+ * DEPRECATED
 bool
 ManagerImpl::getEvents() {
   initRegisterAccounts();
   return true;
 }
+*/
 
 // TODO: rewrite this
 /**

src/managerimpl.h

@@ -212,8 +212,10 @@ public:
    * This function should definitively be renamed!
    *
    * @todo Receive account name (???)
+   *
+   * DEPRECATED
    */
-  bool getEvents();
+  //bool getEvents();
 
   //
   bool getZeroconf(const std::string& sequenceId);