From e42ee155644ef9374aad9c2cf5a3ee51ae17e1e7 Mon Sep 17 00:00:00 2001
From: coderaiser <mnemonic.enemy@gmail.com>
Date: Fri, 7 Nov 2014 05:59:39 -0500
Subject: [PATCH] refactor(config) onLSChange

---
 lib/client/config.js | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/lib/client/config.js b/lib/client/config.js
index 847d8169..a7e9051a 100644
--- a/lib/client/config.js
+++ b/lib/client/config.js
@@ -175,13 +175,13 @@ var CloudCmd, Util, DOM, io;
             
             if (type === 'checkbox')
                 if (el.id === 'localStorage')
-                    data = onLocalStorageChange(data);
+                    onLocalStorageChange();
                 else if (el.id === 'diff')
-                    data = onLSChange(data);
+                    onLSChange(el);
                 else if (el.id === 'buffer')
-                    data = onLSChange(data);
+                    onLSChange(el);
                 else if (el.id === 'dirStorage')
-                    data = onLSChange(data);
+                    onLSChange(el);
                 else if (el.id === 'auth')
                     onAuthChange(data);
             
@@ -249,10 +249,11 @@ var CloudCmd, Util, DOM, io;
             }
         }
         
-        function onLocalStorageChange(checked) {
+        function onLocalStorageChange() {
             var isChecked,
                 names           = ['diff', 'buffer', 'dirStorage', 'localStorage'],
                 elements        = names.map(DOM.getById),
+                showed,
                 el              = {},
                 msg             = 'Diff, Buffer and Directory Storage do not work without localStorage';
             
@@ -274,21 +275,20 @@ var CloudCmd, Util, DOM, io;
                         
                         onChange(element);
                     }
+                    
+                    return element;
                 });
-                
-                checked = false;
             }
-            
-            return checked;
         }
         
-        function onLSChange(checked) {
-            var element = DOM.getById('localStorage', Element);
+        function onLSChange(el) {
+            var elLocalStorage  = DOM.getById('localStorage', Element),
+                msg             = el.id + ' depends on localStorage';
             
-            if (checked && !element.checked)
-                checked = onLocalStorageChange(checked);
-            
-            return checked;
+            if (el.checked && !elLocalStorage.checked) {
+                alert(msg);
+                elLocalStorage.checked = true;
+            }
         }
         
         function onAuthChange(checked) {