From 6842d8dfc29b76ba16ab912a47c065bed813f9a5 Mon Sep 17 00:00:00 2001
From: Maxim Kadushkin <maxim.kadushkin@onlyoffice.com>
Date: Thu, 20 Apr 2017 12:27:27 +0300
Subject: [PATCH] [DE] keep bool value in local storage

---
 apps/common/main/lib/util/LocalStorage.js          | 5 +++++
 apps/documenteditor/main/app/controller/Main.js    | 2 +-
 apps/documenteditor/main/app/controller/Toolbar.js | 6 +++---
 3 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/apps/common/main/lib/util/LocalStorage.js b/apps/common/main/lib/util/LocalStorage.js
index 63dd7819f..6449a3e88 100644
--- a/apps/common/main/lib/util/LocalStorage.js
+++ b/apps/common/main/lib/util/LocalStorage.js
@@ -78,6 +78,10 @@ define(['gateway'], function () {
             }
         };
 
+        var _setItemAsBool = function(name, value, just) {
+            _setItem(name, value ? 1 : 0, just);
+        };
+
         var _getItem = function(name) {
             if (_lsAllowed)
                 return localStorage.getItem(name);
@@ -110,6 +114,7 @@ define(['gateway'], function () {
             },
             getItem: _getItem,
             getBool: _getItemAsBool,
+            setBool: _setItemAsBool,
             setItem: _setItem,
             setKeysFilter: function(value) {
                 _filter = value;
diff --git a/apps/documenteditor/main/app/controller/Main.js b/apps/documenteditor/main/app/controller/Main.js
index 017417c59..25978fd1f 100644
--- a/apps/documenteditor/main/app/controller/Main.js
+++ b/apps/documenteditor/main/app/controller/Main.js
@@ -799,7 +799,7 @@ define([
                 value = Common.localStorage.getItem("de-settings-spellcheck");
                 me.api.asc_setSpellCheck(value===null || parseInt(value) == 1);
 
-                Common.localStorage.setItem("de-settings-showsnaplines", me.api.get_ShowSnapLines() ? 1 : 0);
+                Common.localStorage.setBool("de-settings-showsnaplines", me.api.get_ShowSnapLines());
 
                 function checkWarns() {
                     if (!window['AscDesktopEditor']) {
diff --git a/apps/documenteditor/main/app/controller/Toolbar.js b/apps/documenteditor/main/app/controller/Toolbar.js
index 90be43926..13bee3982 100644
--- a/apps/documenteditor/main/app/controller/Toolbar.js
+++ b/apps/documenteditor/main/app/controller/Toolbar.js
@@ -331,7 +331,7 @@ define([
         onChangeCompactView: function(view, compact) {
             this.toolbar.setFolded(compact);
 
-            Common.localStorage.setItem('de-compact-toolbar', compact ? 1 : 0);
+            Common.localStorage.setBool('de-compact-toolbar', compact);
             Common.NotificationCenter.trigger('layout:changed', 'toolbar');
             Common.NotificationCenter.trigger('edit:complete', this.toolbar);
         },
@@ -1936,7 +1936,7 @@ define([
             var headerView  = this.getApplication().getController('Statusbar').getView('Statusbar');
             headerView  && headerView.setVisible(!checked);
 
-            Common.localStorage.setItem('de-hidden-status', checked ? 1 : 0);
+            Common.localStorage.setBool('de-hidden-status', checked);
 
             Common.NotificationCenter.trigger('layout:changed', 'status');
             Common.NotificationCenter.trigger('edit:complete', this.toolbar);
@@ -1947,7 +1947,7 @@ define([
                 this.api.asc_SetViewRulers(!checked);
             }
 
-            Common.localStorage.setItem('de-hidden-rulers', checked ? 1 : 0);
+            Common.localStorage.setBool('de-hidden-rulers', checked);
 
             Common.NotificationCenter.trigger('layout:changed', 'rulers');
             Common.NotificationCenter.trigger('edit:complete', this.toolbar);