refactor(Paths): Remove portable state from Settings

It now duplicates the state in Paths where it is needed, creating the
chance for desync

src/persistence/settings.cpp

@@ -65,7 +65,6 @@ QThread* Settings::settingsThread{nullptr};
 Settings::Settings()
     : loaded(false)
     , useCustomDhtList{false}
-    , makeToxPortable{false}
     , currentProfileId(0)
 {
     settingsThread = new QThread();
@@ -159,7 +158,7 @@ void Settings::loadGlobal()
 
     s.beginGroup("Advanced");
     {
-        makeToxPortable = s.value("makeToxPortable", false).toBool();
+        paths.setPortable(s.value("makeToxPortable", false).toBool());
         enableIPv6 = s.value("enableIPv6", true).toBool();
         forceTCP = s.value("forceTCP", false).toBool();
         enableLanDiscovery = s.value("enableLanDiscovery", true).toBool();
@@ -623,7 +622,7 @@ void Settings::saveGlobal()
 
     s.beginGroup("Advanced");
     {
-        s.setValue("makeToxPortable", makeToxPortable);
+        s.setValue("makeToxPortable", paths.isPortable());
         s.setValue("enableIPv6", enableIPv6);
         s.setValue("forceTCP", forceTCP);
         s.setValue("enableLanDiscovery", enableLanDiscovery);
@@ -864,7 +863,7 @@ void Settings::setEnableIPv6(bool enabled)
 bool Settings::getMakeToxPortable() const
 {
     QMutexLocker locker{&bigLock};
-    return makeToxPortable;
+    return paths.isPortable();
 }
 
 void Settings::setMakeToxPortable(bool newValue)
@@ -872,16 +871,14 @@ void Settings::setMakeToxPortable(bool newValue)
     bool changed = false;
     {
         QMutexLocker locker{&bigLock};
-        if (newValue != makeToxPortable) {
+        auto const oldSettingsPath = paths.getSettingsDirPath() + globalSettingsFile;
-            QFile(paths.getSettingsDirPath() + globalSettingsFile).remove();
+        changed = paths.setPortable(newValue);
-            makeToxPortable = newValue;
+        if (changed) {
-            changed = paths.setPortable(newValue);
+            QFile(oldSettingsPath).remove();
             saveGlobal();
+            emit makeToxPortableChanged(newValue);
         }
     }
-    if (changed) {
-        emit makeToxPortableChanged(newValue);
-    }
 }
 
 bool Settings::getAutorun() const

src/persistence/settings.h

@@ -595,7 +595,6 @@ private:
     bool showIdenticons;
     bool enableIPv6;
     QString translation;
-    bool makeToxPortable;
     bool autostartInTray;
     bool closeToTray;
     bool minimizeToTray;