Merge pull request #8715 from glassez/max-ratio

Fix setting preferences via WebAPI. Closes #7335

src/webui/api/appcontroller.cpp

@@ -409,14 +409,18 @@ void AppController::setPreferencesAction()
     if (m.contains("dont_count_slow_torrents"))
         session->setIgnoreSlowTorrentsForQueueing(m["dont_count_slow_torrents"].toBool());
     // Share Ratio Limiting
-    if (m.contains("max_ratio_enabled"))
+    if (m.contains("max_ratio_enabled")) {
-        session->setGlobalMaxRatio(m["max_ratio"].toReal());
+        if (m["max_ratio_enabled"].toBool())
-    else
+            session->setGlobalMaxRatio(m["max_ratio"].toReal());
-        session->setGlobalMaxRatio(-1);
+        else
-    if (m.contains("max_seeding_time_enabled"))
+            session->setGlobalMaxRatio(-1);
-        session->setGlobalMaxSeedingMinutes(m["max_seeding_time"].toInt());
+    }
-    else
+    if (m.contains("max_seeding_time_enabled")) {
-        session->setGlobalMaxSeedingMinutes(-1);
+        if (m["max_seeding_time_enabled"].toBool())
+            session->setGlobalMaxSeedingMinutes(m["max_seeding_time"].toInt());
+        else
+            session->setGlobalMaxSeedingMinutes(-1);
+    }
     if (m.contains("max_ratio_act"))
         session->setMaxRatioAction(static_cast<MaxRatioAction>(m["max_ratio_act"].toInt()));
     // Add trackers