From dd95f22dfba593c089ca090624b9d0655d62d2f8 Mon Sep 17 00:00:00 2001
From: Anthony Perish <anthonyp@audian.com>
Date: Fri, 27 Oct 2017 15:57:35 -0700
Subject: [PATCH] UI-2945 - Allow user list in smart pbx to be sorted

- Clicking on the appropriate header in the smart pbx user list
  will sort by that column.
---
 submodules/users/users.css |  6 ++++-
 submodules/users/users.js  | 47 +++++++++++++++++++++++++++++++++++---
 2 files changed, 49 insertions(+), 4 deletions(-)

diff --git a/submodules/users/users.css b/submodules/users/users.css
index aafd318..fc34841 100644
--- a/submodules/users/users.css
+++ b/submodules/users/users.css
@@ -95,6 +95,10 @@
 	text-align: left;
 }
 
+#users_container .users-grid .grid-row.title .grid-cell.name , .users-grid .grid-row.title .grid-cell.extension {
+	font-weight: bold;
+}
+
 #users_container .users-grid .grid-row .grid-cell {
 	display: inline-block;
 	overflow: hidden;
@@ -910,4 +914,4 @@
 
 .monster-feature-popup-container[data-feature="call_recording"] .btn-group-wrapper .btn-group {
 	margin-left: 10px;
-}
\ No newline at end of file
+}
diff --git a/submodules/users/users.js b/submodules/users/users.js
index 2be2457..04a0cd4 100644
--- a/submodules/users/users.js
+++ b/submodules/users/users.js
@@ -41,12 +41,13 @@ define(function(require) {
 				parent = args.parent || $('.right-content'),
 				_userId = args.userId,
 				_openedTab = args.openedTab,
+				_sortBy = args.sortBy,
 				callback = args.callback;
 
 			self.usersRemoveOverlay();
 
 			self.usersGetData(function(data) {
-				var dataTemplate = self.usersFormatListData(data),
+				var dataTemplate = self.usersFormatListData(data, _sortBy),
 					template = $(monster.template(self, 'users-layout', dataTemplate)),
 					templateUser;
 
@@ -328,7 +329,7 @@ define(function(require) {
 			return dataUser;
 		},
 
-		usersFormatListData: function(data) {
+		usersFormatListData: function(data, _sortBy) {
 			var self = this,
 				dataTemplate = {
 					existingExtensions: [],
@@ -421,16 +422,48 @@ define(function(require) {
 			});
 
 			var sortedUsers = [];
-
+			//Set blank presence_id for ability to sort by presence_id
 			_.each(mapUsers, function(user) {
+				if(!user.hasOwnProperty('presence_id')){
+					user.presence_id = '';
+				}
 				sortedUsers.push(user);
 			});
+			//Default sort by presence_id
+				if(typeof _sortBy === 'undefined' ){
+					_sortBy = "first_name";
+				}
+				sortedUsers = self.sort(sortedUsers, _sortBy);
 
 			dataTemplate.users = sortedUsers;
 
 			return dataTemplate;
 		},
 
+		/* Automatically sorts an array of objects. secondArg can either be a custom sort to be applied to the dataset, or a fieldName to sort alphabetically on */
+		sort: function(dataSet, secondArg) {
+			var fieldName = 'name',
+				sortFunction = function(a, b) {
+					var aString = a[fieldName].toLowerCase(),
+						bString = b[fieldName].toLowerCase(),
+						result = (aString > bString) ? 1 : (aString < bString) ? -1 : 0;
+
+					return result;
+				},
+				result;
+
+			if(typeof secondArg === 'function') {
+				sortFunction = secondArg;
+			}
+			else if(typeof secondArg === 'string') {
+				fieldName = secondArg;
+			}
+
+			result = dataSet.sort(sortFunction);
+
+			return result;
+		},
+
 		usersDeleteDialog: function(user, callback) {
 			var self = this,
 				dataTemplate = {
@@ -492,6 +525,14 @@ define(function(require) {
 
 			setTimeout(function() { template.find('.search-query').focus(); });
 
+			template.find('.grid-row.title .grid-cell.name').on('click', function() {
+				self.usersRender({ sortBy: 'first_name' });
+			});
+
+			template.find('.grid-row.title .grid-cell.extension').on('click', function() {
+			self.usersRender({ sortBy: 'presence_id' });
+			});
+
 			template.find('.grid-row:not(.title) .grid-cell').on('click', function() {
 				var cell = $(this),
 					type = cell.data('type'),