plugins: fix saveConfig logic

Change-Id: Iebb5804098699ca206a506cf8e1362faf1ef1aff

src/client/plugin_manager_interface.cpp

@@ -28,13 +28,21 @@ namespace DRing {
 bool
 loadPlugin(const std::string& path)
 {
-    return jami::Manager::instance().getJamiPluginManager().loadPlugin(path);
+    bool status = jami::Manager::instance().getJamiPluginManager().loadPlugin(path);
+
+    jami::Manager::instance().pluginPreferences.saveStateLoadedPlugins(path, status);
+    jami::Manager::instance().saveConfig();
+    return status;
 }
 
 bool
 unloadPlugin(const std::string& path)
 {
-    return jami::Manager::instance().getJamiPluginManager().unloadPlugin(path);
+    bool status = jami::Manager::instance().getJamiPluginManager().unloadPlugin(path);
+
+    jami::Manager::instance().pluginPreferences.saveStateLoadedPlugins(path, false);
+    jami::Manager::instance().saveConfig();
+    return status;
 }
 
 void

src/plugin/jamipluginmanager.cpp

@@ -302,8 +302,6 @@ JamiPluginManager::loadPlugin(const std::string& rootPath)
         bool status = pm_.load(getPluginDetails(rootPath).at("soPath"));
         JAMI_INFO() << "PLUGIN: load status - " << status;
 
-        jami::Manager::instance().pluginPreferences.saveStateLoadedPlugins(rootPath, status);
-        jami::Manager::instance().saveConfig();
         return status;
 
     } catch (const std::exception& e) {
@@ -322,9 +320,6 @@ JamiPluginManager::unloadPlugin(const std::string& rootPath)
         bool status = pm_.unload(getPluginDetails(rootPath).at("soPath"));
         JAMI_INFO() << "PLUGIN: unload status - " << status;
 
-        jami::Manager::instance().pluginPreferences.saveStateLoadedPlugins(rootPath, false);
-        jami::Manager::instance().saveConfig();
-
         return status;
     } catch (const std::exception& e) {
         JAMI_ERR() << e.what();