LupusNobilis
/
sogo
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

(js) Simplify promises handling in User.$filter

pull/91/head
Francis Lachapelle 2015-07-07 16:34:58 -04:00
parent fe20cc220f
commit dd5b325864
1 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
UI/WebServerResources/js/Common/User.service.js
View File

@ -41,14 +41,12 @@
* @return a promise of an array of matching User objects
*/
User.$filter = function(search, excludedUsers) {
var deferred = User.$q.defer(),
param = {search: search};
var param = {search: search};
if (!search) {
// No query specified
User.$users = [];
deferred.resolve(User.$users);
return deferred.promise;
return User.$q.when(User.$users);
}
if (angular.isUndefined(User.$users)) {
// First session query
@ -56,12 +54,11 @@
}
else if (User.$query == search) {
// Query hasn't changed
deferred.resolve(User.$users);
return deferred.promise;
return User.$q.when(User.$users);
}
User.$query = search;
User.$$resource.fetch(null, 'usersSearch', param).then(function(response) {
return User.$$resource.fetch(null, 'usersSearch', param).then(function(response) {
var results, index, user;
if (excludedUsers) {
// Remove excluded users from response
@ -93,10 +90,9 @@
User.$users.splice(index, 1);
}
}
deferred.resolve(User.$users);
}, deferred.reject);
return deferred.promise;
User.$log.debug(User.$users);
return User.$users;
});
};
/**
@ -113,6 +109,10 @@
this.$$image = this.image || User.$gravatar(this.c_email);
};
User.prototype.toString = function() {
return '[User ' + this.c_email + ']';
};
/**
* @function $shortFormat
* @memberof User.prototype