fix(admin-users-edit): Display proper confirmation messages on save a…

…nd remove
xinmeng1 · Nov 27, 2012 · 65bafff · 65bafff
1 parent 342e225
commit 65bafff
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/client/src/app/admin/users/admin-users-edit.js b/client/src/app/admin/users/admin-users-edit.js
@@ -2,7 +2,7 @@ angular.module('admin-users-edit',['services.crud', 'services.i18nNotifications'
 
 .controller('UsersEditCtrl', ['$scope', '$location', 'i18nNotifications', 'user', function ($scope, $location, i18nNotifications, user) {
 
-  $scope.onSave = function () {
+  $scope.onSave = function (user) {
     i18nNotifications.pushForNextRoute('crud.user.save.success', 'success', {id : user.$id()});
     $location.path('/admin/users');
   };
@@ -11,7 +11,7 @@ angular.module('admin-users-edit',['services.crud', 'services.i18nNotifications'
     i18nNotifications.pushForCurrentRoute('crud.user.save.error', 'error');
   };
 
-  $scope.onRemove = function() {
+  $scope.onRemove = function(user) {
     i18nNotifications.pushForNextRoute('crud.user.remove.success', 'success', {id : user.$id()});
   };
 

diff --git a/client/test/unit/app/admin/users/admin-usersSpec.js b/client/test/unit/app/admin/users/admin-usersSpec.js
@@ -41,7 +41,7 @@ describe('admin users', function () {
       });
 
       it('should call $location in onSave', function() {
-        $scope.onSave();
+        $scope.onSave({});
         expect(params.i18nNotifications.pushForNextRoute).toHaveBeenCalled();
         expect(params.$location.path).toHaveBeenCalled();
       });