From b232bc074d1e5dc2b623a3c379170193949f4756 Mon Sep 17 00:00:00 2001 From: Jason Boyer Date: Thu, 15 Nov 2018 13:45:44 -0500 Subject: [PATCH] LP1803584: Fix Clear button in template editor There are multiple controllers in staff/cat/volcopy/ and 2 of them have a clearWorking() function. The template editor controller is referencing $scope.MultiMap[] which will never exist in the template editor, while the actual item editor controller clearWorking() does not reference $scope.MultiMap[], meaning that clearing multiple items at once would not work as expected. This branch clears up this confusion. Signed-off-by: Jason Boyer --- Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js b/Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js index f23adb38fd..11547dc4c6 100644 --- a/Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js +++ b/Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js @@ -1369,6 +1369,7 @@ function($scope , $q , $window , $routeParams , $location , $timeout , egCore , $scope.clearWorking = function () { angular.forEach($scope.working, function (v,k,o) { + $scope.working.MultiMap[k] = []; if (!angular.isObject(v)) { if (typeof v != 'undefined') $scope.working[k] = undefined; @@ -2515,7 +2516,6 @@ function($scope , $q , $window , $routeParams , $location , $timeout , egCore , $scope.clearWorking = function () { angular.forEach($scope.working, function (v,k,o) { - $scope.working.MultiMap[k] = []; if (!angular.isObject(v)) { if (typeof v != 'undefined') $scope.working[k] = undefined; -- 2.11.0