Merge branch 'local-storage-version' of https://github.com/mencio/ninja-internal into integration-candidate

14 files changed, 169 insertions, 182 deletions

diff --git a/js/controllers/local-storage-controller.js b/js/controllers/local-storage-controller.js
index 6963b245..1388f3e5 100755
--- a/js/controllers/local-storage-controller.js
+++ b/js/controllers/local-storage-controller.js
@@ -7,44 +7,51 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
 var Montage = require("montage/core/core").Montage,
     Component       = require("montage/ui/component").Component;
 
-exports.LocalStorage = Montage.create( Montage, {
+exports.LocalStorage = Montage.create( Component, {
 
-    getItem: {
-        value: function(item) {
-            var item;
-
-            if (window.localStorage) {
-                item  = window.localStorage.getItem(item);
-                if(item !== null) return JSON.parse(item)
-                return null;
-            } else {
-                alert("Local Storage is not supported on your browser");
-                return null;
-            }
+    canStore: {
+        value: null
+    },
 
-            /*
-            if (window.localStorage) {
-                this.getItem = function(item) {
-                    return window.localStorage.getItem(item);
-                }(item);
-            } else {
-                alert("Local Storage is not supported on your browser");
+    deserializedFromTemplate: {
+        value: function() {
+            this.canStore = window.localStorage;
+            this.application.localStorage = this;
+
+            // Redefine setItem and getItem if local storage is not available.
+            if(!this.canStore) {
+                this.getItem = function() {
+                    console.log("Local Storage is not supported on your browser");
+                    return "";
+                };
+
+                this.setItem = function() {
+                    console.log("Local Storage is not supported on your browser");
+                    return false;
+                }
+            }
 
+            // Temporary clear the local storage if we find the version key
+            if(window.localStorage.version) {
+                window.localStorage.clear();
             }
-            */
+        }
+    },
+
+    getItem: {
+        value: function(key) {
+            var value = window.localStorage.getItem("ninja-" + key);
+            if(value !== null) value = JSON.parse(value);
+
+            return value;
         }
     },
 
     setItem: {
-        value: function(item, value) {
-            if (window.localStorage) {
-                window.localStorage.setItem(item, JSON.stringify(value));
-                return true;
-            } else {
-                alert("Local Storage is not supported on your browser");
-                return false;
-            }
+        value: function(key, value) {
+            window.localStorage.setItem("ninja-" + key, JSON.stringify(value));
+
+            return value;
         }
     }
-
 });
\ No newline at end of file
diff --git a/js/data/settings.js b/js/data/settings.js
deleted file mode 100755
index ffea2075..00000000
--- a/js/data/settings.js
+++ /dev/null
@@ -1,71 +0,0 @@
-/* <copyright>
-This file contains proprietary software owned by Motorola Mobility, Inc.<br/>
-No rights, expressed or implied, whatsoever to this software are provided by Motorola Mobility, Inc. hereunder.<br/>
-(c) Copyright 2011 Motorola Mobility, Inc.  All Rights Reserved.
-</copyright> */
-
-var Montage = require("montage/core/core").Montage,
-    Component       = require("montage/ui/component").Component,
-    LocalStorage    = require("js/controllers/local-storage-controller").LocalStorage;
-
-exports.Settings = Montage.create( Component, {
-
-    version: {
-        value: "11.1213"
-    },
-
-    _settings: {
-        value: null
-    },
-
-    settings: {
-        get: function() { return this._settings; },
-        set: function(value) { this._settings = value; }
-    },
-
-    getSetting: {
-        value: function(objName, fieldName, namespace) {
-            try {
-                objName = objName.replace(/-/gi, "_").replace(/\//gi, "zzSlash");
-                return this.settings[objName][fieldName];
-            } catch(e) {
-                return null;
-            }
-        }
-    },
-
-    setSetting: {
-        value: function(objName, fieldName, value, namespace) {
-            try {
-                objName = objName.replace(/-/gi, "_").replace(/\//gi, "zzSlash");
-
-                if(this.settings === null) {
-                    this.settings = {};
-                }
-
-                if (this.settings[objName] == null) {
-                    this.settings[objName] = {};
-                }
-
-                this.settings[objName][fieldName] = value;
-
-                LocalStorage.setItem("settings", this.settings);
-            } catch(e) {
-                return null;
-            }
-        }
-    },
-    
-    deserializedFromSerialization: {
-        value: function() {
-
-            if (LocalStorage.getItem("version") != this.version) {
-                this.settings = {}
-                LocalStorage.setItem("version",this.version);
-            } else {
-                this.settings = LocalStorage.getItem("settings");
-            }
-
-        }
-    }
-});
\ No newline at end of file
diff --git a/js/io/system/coreioapi.js b/js/io/system/coreioapi.js
index 2051da43..f428a229 100755
--- a/js/io/system/coreioapi.js
+++ b/js/io/system/coreioapi.js
@@ -28,9 +28,9 @@ exports.CoreIoApi = Montage.create(Component, {
                 value: function () {
                         ////////////////////////////////////////////////////////////
                         //Checking for local storage of URL for IO
-                        if (window.localStorage['ioRootUrl']) {
+            if (this.application.localStorage.getItem("ioRootUrl")) {
                                 //Getting URL from local storage
-                                this.rootUrl = window.localStorage['ioRootUrl'];
+                this.rootUrl = this.application.localStorage.getItem("ioRootUrl");
                                 //Checks for IO API to be active
                                 this.ioServiceDetected = this.cloudAvailable();
                         } else {
@@ -156,7 +156,7 @@ exports.CoreIoApi = Montage.create(Component, {
             return this._rootUrl;
         },
         set: function(value) {
-                this._rootUrl = window.localStorage["ioRootUrl"] = value;
+                this._rootUrl = this.application.localStorage.setItem("ioRootUrl", value);
         }
     },
         ////////////////////////////////////////////////////////////////////
diff --git a/js/io/ui/cloudpopup.reel/cloudpopup.js b/js/io/ui/cloudpopup.reel/cloudpopup.js
index d2e82662..a4656bb5 100755
--- a/js/io/ui/cloudpopup.reel/cloudpopup.js
+++ b/js/io/ui/cloudpopup.reel/cloudpopup.js
@@ -63,8 +63,8 @@ exports.CloudPopup = Montage.create(Component, {
         enumerable: false,
         value: function() {
                 //
-                if (window.localStorage['ioRootUrl']) {
-                        this.components.url.value = window.localStorage['ioRootUrl'];
+            if (this.application.localStorage.getItem("ioRootUrl")) {
+                this.components.url.value = this.application.localStorage.getItem("ioRootUrl");
                 }
                 //
                 this.testConnection();
diff --git a/js/ninja.reel/ninja.html b/js/ninja.reel/ninja.html
index 1393b4b9..b1fb6352 100755
--- a/js/ninja.reel/ninja.html
+++ b/js/ninja.reel/ninja.html
@@ -22,9 +22,9 @@
                 "name": "Preloader"
             },
 
-            "settings1": {
+            "localStorag