From 65bafffb001b4d6d0e050a01b0d2b0c5895d6425 Mon Sep 17 00:00:00 2001 From: Pawel Kozlowski Date: Tue, 27 Nov 2012 21:24:28 +0100 Subject: [PATCH] fix(admin-users-edit): Display proper confirmation messages on save and remove --- client/src/app/admin/users/admin-users-edit.js | 4 ++-- client/test/unit/app/admin/users/admin-usersSpec.js | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/client/src/app/admin/users/admin-users-edit.js b/client/src/app/admin/users/admin-users-edit.js index 40f06d2e..7145e078 100644 --- 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 index afa45200..76a9abf3 100644 --- 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(); });