Skip to content

Commit 87be7af

Browse files
committed
fix: remove should use an angular promise
1 parent 2d6c0c7 commit 87be7af

File tree

1 file changed

+11
-3
lines changed

1 file changed

+11
-3
lines changed

src/angular-localForage.js

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -108,13 +108,21 @@
108108

109109
// Remove an item from storage
110110
var removeItem = function(key) {
111-
var promise = localforage.removeItem(prefix() + key);
111+
var deferred = $q.defer(),
112+
args = arguments;
113+
114+
localforage.removeItem(prefix() + key).then(function success() {
115+
deferred.resolve();
116+
}, function error(data) {
117+
onError(data, args, removeItem, deferred);
118+
});
119+
112120
if(notify.removeItem) {
113-
return promise.then(function(value) {
121+
return deferred.promise.then(function(value) {
114122
$rootScope.$broadcast('LocalForageModule.removeItem', {key: key, driver: localforage.driver()});
115123
});
116124
} else {
117-
return promise;
125+
return deferred.promise;
118126
}
119127
};
120128

0 commit comments

Comments
 (0)