From e808f232f911b3c856e8185089f126a445c672c1 Mon Sep 17 00:00:00 2001
From: Joris Tirado <joris.tirado@gmail.com>
Date: Wed, 9 Nov 2016 17:20:00 -0800
Subject: [PATCH] UI-2057: Wrap users list request in its own method

---
 submodules/devices/devices.js | 29 +++++++++++++++++++++--------
 1 file changed, 21 insertions(+), 8 deletions(-)

diff --git a/submodules/devices/devices.js b/submodules/devices/devices.js
index 830f0f2..853f0f3 100644
--- a/submodules/devices/devices.js
+++ b/submodules/devices/devices.js
@@ -177,17 +177,13 @@ define(function(require){
 								}
 							}
 
-							self.callApi({
-								resource: 'user.list',
-								data: {
-									accountId: self.accountId
-								},
-								success: function(data, status) {
+							self.devicesListUsers({
+								success: function(users) {
 									var keyTypes = [ 'none', 'presence', 'parking', 'personal_parking', 'speed_dial' ],
 										parkingSpots = [],
 										extra;
 
-									data.data.sort(function(a, b) {
+									users.sort(function(a, b) {
 										return a.last_name.toLowerCase() > b.last_name.toLowerCase() ? 1 : -1;
 									});
 
@@ -204,7 +200,7 @@ define(function(require){
 									});
 
 									extra = {
-										users: data.data,
+										users: users,
 										featureKeys:{
 											parkingSpots: parkingSpots,
 											types: keyTypes
@@ -1227,6 +1223,23 @@ define(function(require){
 			else {
 				callbackError && callbackError();
 			}
+		},
+
+		devicesListUsers: function(args) {
+			var self = this;
+
+			self.callApi({
+				resource: 'user.list',
+				data: {
+					accountId: self.accountId
+				},
+				success: function(data, status) {
+					args.hasOwnProperty('success') && args.success(data.data);
+				},
+				error: function(data, status) {
+					args.hasOwnProperty('error') && args.error();
+				}
+			});
 		}
 	};