diff --git a/src/views/users.js b/src/views/users.js index c9b3f2077..73c5c67c0 100644 --- a/src/views/users.js +++ b/src/views/users.js @@ -326,20 +326,23 @@ angular.module('Application').controller('UsersController', ['$scope', '$locatio if ($scope.groupEdit.roles.canManageDomains) roles.push('manage_domains'); Client.updateGroup($scope.groupEdit.groupInfo.id, $scope.groupEdit.name, roles, function (error) { - $scope.groupEdit.busy = !!error; + if (error) { + $scope.groupEdit.busy = false; - if (error && error.statusCode === 409) { - $scope.groupEdit.error.name = 'Name already taken'; - $scope.groupEditForm.name.$setPristine(); - $('#groupEditName').focus(); - return; - } else if (error && error.statusCode === 400) { - $scope.groupEdit.error.name = error.message; - $scope.groupEditForm.name.$setPristine(); - $('#groupEditName').focus(); - return; + if (error.statusCode === 409) { + $scope.groupEdit.error.name = 'Name already taken'; + $scope.groupEditForm.name.$setPristine(); + $('#groupEditName').focus(); + return; + } else if (error.statusCode === 400) { + $scope.groupEdit.error.name = error.message; + $scope.groupEditForm.name.$setPristine(); + $('#groupEditName').focus(); + return; + } else { + return console.error('Unable to edit group.', error.statusCode, error.message); + } } - if (error) return console.error('Unable to edit group.', error.statusCode, error.message); var userIds = $scope.groupEdit.selectedUsers.map(function (u) { return u.id; });