fix(localStorage): parsing safety grevory#230

Author: a8m

src/angular-local-storage.js

@@ -163,7 +163,11 @@ angularLocalStorage.provider('localStorageService', function() {
         return null;
       }
 
-      return JSON.parse(item);
+      try {
+        return JSON.parse(item);
+      } catch (e) {
+        return item;
+      }
     };
 
     // Remove an item from local storage

test/spec/localStorageSpec.js

@@ -500,6 +500,19 @@ describe('localStorageService', function() {
     });
   }));
 
+  // Backward compatibility issue-#230
+  it('should return the item as-is if the parsing fail', inject(function($window, localStorageService) {
+    var items = ['{', '[', 'foo'];
+    //set keys
+    items.forEach(function(item, i) {
+      $window.localStorage.setItem('ls.' + i, item);
+    });
+
+    items.forEach(function(item, i) {
+      expect(localStorageService.get(i)).toEqual(item);
+    });
+  }));
+
   //sessionStorage
   describe('SessionStorage', function() {