From 5c6624e1d194695c24fbb1d5fa2284d0d56835ad Mon Sep 17 00:00:00 2001 From: Matt-Yorkley Date: Thu, 15 Mar 2018 12:56:40 +0000 Subject: [PATCH] Tidy up setDirty() calls in enterprise_controller.js --- .../enterprises/controllers/enterprise_controller.js.coffee | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/app/assets/javascripts/admin/enterprises/controllers/enterprise_controller.js.coffee b/app/assets/javascripts/admin/enterprises/controllers/enterprise_controller.js.coffee index d8ef5e1b23..7d1935010e 100644 --- a/app/assets/javascripts/admin/enterprises/controllers/enterprise_controller.js.coffee +++ b/app/assets/javascripts/admin/enterprises/controllers/enterprise_controller.js.coffee @@ -38,8 +38,7 @@ angular.module("admin.enterprises") return for i, user of $scope.Enterprise.users when user.id == manager.id $scope.Enterprise.users.splice i, 1 - if $scope.enterprise_form? - $scope.enterprise_form.$setDirty() + $scope.enterprise_form?.$setDirty() $scope.addManager = (manager) -> if manager.id? and angular.isNumber(manager.id) and manager.email? @@ -49,6 +48,6 @@ angular.module("admin.enterprises") confirmed: manager.confirmed if (user for user in $scope.Enterprise.users when user.id == manager.id).length == 0 $scope.Enterprise.users.push manager - $scope.enterprise_form.$setDirty() + $scope.enterprise_form?.$setDirty() else alert ("#{manager.email}" + " " + t("is_already_manager"))