LP1803584 Avoid JS errors when clearing copy templates user/berick/lp1803584-copy-edit-template-clear
authorBill Erickson <berickxx@gmail.com>
Fri, 8 Mar 2019 17:09:07 +0000 (12:09 -0500)
committerBill Erickson <berickxx@gmail.com>
Fri, 8 Mar 2019 17:09:10 +0000 (12:09 -0500)
Minor fix to sanity check multi-copy data in the copy template editor
after clearing the template contents.  This cleans up some ugly console
errors.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js

index f39f445..5295be3 100644 (file)
@@ -1179,7 +1179,8 @@ function($scope , $q , $window , $routeParams , $location , $timeout , egCore ,
         var keys = Object.keys($scope.working.MultiMap);
         // for-loop for shortcut exit
         for (var i = 0; i < keys.length; i++) {
-            if ($scope.working.MultiMap[keys[i]].length > 1) {
+            if ($scope.working.MultiMap[keys[i]] &&
+                $scope.working.MultiMap[keys[i]].length > 1) {
                 return true;
             }
         }