fix: use angular.copy before storing values

It should prevent the copy of angular properties such as $$hashKey Fixes scotttrinh#29

Author: ocombe

dist/angular-localForage.js

@@ -81,29 +81,19 @@
 // Directly adds a value to storage
 var setItem = function(key, value) {
 var deferred = $q.defer(),
-args = arguments;
-
-//avoid $promises attributes from value objects, if is there.
-if (angular.isObject(value) && angular.isDefined(value.$promise)) {
-delete value.$promise; //delete attribut from object structure.
-}
+args = arguments,
+localCopy = angular.copy(value);
 
-//avoid $$hashKey attributes from value arrays, if is there.
-if (angular.isArray(value)) {
-angular.forEach(value, function(subvalue, key){
-if (angular.isDefined(subvalue.$$hashKey)) {
-if (key === 0) {
-value = angular.fromJson(angular.toJson(value));
-}
-}
-});
+//avoid $promises attributes from value objects, if present.
+if (angular.isObject(localCopy) && angular.isDefined(localCopy.$promise)) {
+delete localCopy.$promise; //delete attribut from object structure.
 }
 
-localforage.setItem(prefix() + key, value).then(function success() {
+localforage.setItem(prefix() + key, localCopy).then(function success() {
 if(notify.setItem) {
-$rootScope.$broadcast('LocalForageModule.setItem', {key: key, newvalue: value, driver: localforage.driver()});
+$rootScope.$broadcast('LocalForageModule.setItem', {key: key, newvalue: localCopy, driver: localforage.driver()});
 }
-deferred.resolve(value);
+deferred.resolve(localCopy);
 }, function error(data) {
 onError(data, args, setItem, deferred);
 });

dist/angular-localForage.min.js

@@ -74,29 +74,19 @@
 // Directly adds a value to storage
 var setItem = function(key, value) {
 var deferred = $q.defer(),
-args = arguments;
-
-//avoid $promises attributes from value objects, if is there.
-if (angular.isObject(value) && angular.isDefined(value.$promise)) {
-delete value.$promise; //delete attribut from object structure.
-}
+args = arguments,
+localCopy = angular.copy(value);
 
-//avoid $$hashKey attributes from value arrays, if is there.
-if (angular.isArray(value)) {
-angular.forEach(value, function(subvalue, key){
-if (angular.isDefined(subvalue.$$hashKey)) {
-if (key === 0) {
-value = angular.fromJson(angular.toJson(value));
-}
-}
-});
+//avoid $promises attributes from value objects, if present.
+if (angular.isObject(localCopy) && angular.isDefined(localCopy.$promise)) {
+delete localCopy.$promise; //delete attribut from object structure.
 }
 
-localforage.setItem(prefix() + key, value).then(function success() {
+localforage.setItem(prefix() + key, localCopy).then(function success() {
 if(notify.setItem) {
-$rootScope.$broadcast('LocalForageModule.setItem', {key: key, newvalue: value, driver: localforage.driver()});
+$rootScope.$broadcast('LocalForageModule.setItem', {key: key, newvalue: localCopy, driver: localforage.driver()});
 }
-deferred.resolve(value);
+deferred.resolve(localCopy);
 }, function error(data) {
 onError(data, args, setItem, deferred);
 });