From 347f2d360f072614306c8ac63b47d32a0e1e01ed 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 Signed-off-by: Beth Willis Signed-off-by: Bill Erickson Signed-off-by: Galen Charlton --- Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js | 2 -- 1 file changed, 2 deletions(-) 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 79dbf81e61..5f8770330a 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 @@ -2567,8 +2567,6 @@ function($scope , $q , $window , $routeParams , $location , $timeout , egCore , $scope.clearWorking = function () { angular.forEach($scope.working, function (v,k,o) { - if (k != 'MultiMap') $scope.working.MultiMap[k] = []; - $scope.working.MultiMap[k] = []; if (!angular.isObject(v)) { if (typeof v != 'undefined') $scope.working[k] = undefined; -- 2.11.0