client/settings: decouple settings from controller

1 files changed, 3 insertions, 31 deletions

diff --git a/client/js/controllers/settings_controller.js b/client/js/controllers/settings_controller.js
index 51fe8bb..777bf3a 100644
--- a/client/js/controllers/settings_controller.js
+++ b/client/js/controllers/settings_controller.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const page = require('page');
-const events = require('../events.js');
+const settings = require('../settings.js');
 const topNavController = require('../controllers/top_nav_controller.js');
 const SettingsView = require('../views/settings_view.js');
 
@@ -17,38 +17,10 @@ class SettingsController {
     settingsRoute() {
         topNavController.activate('settings');
         this.settingsView.render({
-            getSettings: () => this.getSettings(),
-            saveSettings: newSettings => this.saveSettings(newSettings),
+            getSettings: () => settings.getSettings(),
+            saveSettings: newSettings => settings.saveSettings(newSettings),
         });
     }
-
-    saveSettings(browsingSettings) {
-        localStorage.setItem('settings', JSON.stringify(browsingSettings));
-        events.notify(events.Success, 'Settings saved');
-        events.notify(events.SettingsChange);
-    }
-
-    getSettings(settings) {
-        const defaultSettings = {
-            endlessScroll: false,
-        };
-        let ret = {};
-        let userSettings = localStorage.getItem('settings');
-        if (userSettings) {
-            userSettings = JSON.parse(userSettings);
-        }
-        if (!userSettings) {
-            userSettings = {};
-        }
-        for (let key of Object.keys(defaultSettings)) {
-            if (key in userSettings) {
-                ret[key] = userSettings[key];
-            } else {
-                ret[key] = defaultSettings[key];
-            }
-        }
-        return ret;
-    }
 };
 
 module.exports = new SettingsController();