From caca5cc00bc39828421b2c779c03255bb09cae20 Mon Sep 17 00:00:00 2001
From: David Baker <dave@matrix.org>
Date: Tue, 23 May 2017 17:06:02 +0100
Subject: [PATCH] Remove c+ped UserSettingsStore functions

as they don't seem to be necessary
---
 src/vector/index.js | 26 ++++----------------------
 1 file changed, 4 insertions(+), 22 deletions(-)

diff --git a/src/vector/index.js b/src/vector/index.js
index 62595c227..d1bf914b6 100644
--- a/src/vector/index.js
+++ b/src/vector/index.js
@@ -57,7 +57,6 @@ var RunModernizrTests = require("./modernizr"); // this side-effects a global
 var ReactDOM = require("react-dom");
 // Workaround for broken export
 import * as counterpart from 'counterpart-riot';
-var languageHandler = require("matrix-react-sdk/lib/languageHandler");
 var sdk = require("matrix-react-sdk");
 var PlatformPeg = require("matrix-react-sdk/lib/PlatformPeg");
 sdk.loadSkin(require('../component-index'));
@@ -66,6 +65,8 @@ var UpdateChecker = require("./updater");
 var q = require('q');
 var request = require('browser-request');
 import Modal from 'matrix-react-sdk/lib/Modal';
+import * as UserSettingsStore from 'matrix-react-sdk/lib/UserSettingsStore';
+import languageHandler from 'matrix-react-sdk/lib/languageHandler';
 
 import url from 'url';
 
@@ -218,20 +219,6 @@ function getConfig() {
     return deferred.promise;
 }
 
-
-// This is needed to not load the UserSettingsStore before languages are laoded
-function getLocalSettings() {
-    const localSettingsString = localStorage.getItem('mx_local_settings') || '{}';
-    return JSON.parse(localSettingsString);
-}
-// This is needed to not load the UserSettingsStore before languages are laoded
-function setLocalSetting(type, value) {
-    const settings = getLocalSettings();
-    settings[type] = value;
-    // FIXME: handle errors
-    localStorage.setItem('mx_local_settings', JSON.stringify(settings));
-}
-
 function onLoadCompleted() {
     // if we did a token login, we're now left with the token, hs and is
     // url as query params in the url; a little nasty but let's redirect to
@@ -340,13 +327,8 @@ async function loadApp() {
 }
 
 async function loadLanguage() {
-    const _localSettings = getLocalSettings();
-    let languages = [];
-    if (!_localSettings.hasOwnProperty('language')) {
-        languages = languageHandler.getNormalizedLanguageKeys(languageHandler.getLanguageFromBrowser());
-    } else {
-        languages = languageHandler.getNormalizedLanguageKeys(_localSettings.language);
-    }
+    const lang = UserSettingsStore.getLocalSetting('language', languageHandler.getLanguageFromBrowser());
+    const languages = languageHandler.getNormalizedLanguageKeys(lang);
     languageHandler.setLanguage(languages, counterpart);
     setLocalSetting('language', languages[0]);
 }