From 36ffcdb778ee26d55e70081ca9ba61b6a76f504d Mon Sep 17 00:00:00 2001
From: Mike Rylander <mrylander@gmail.com>
Date: Wed, 24 May 2017 21:08:38 -0400
Subject: [PATCH] webstaff: Receiving data sanity, and form validation

Signed-off-by: Mike Rylander <mrylander@gmail.com>
---
 Open-ILS/src/templates/staff/serials/t_batch_receive.tt2          | 6 +++---
 .../web/js/ui/default/staff/serials/directives/view-items-grid.js | 8 +++++---
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/Open-ILS/src/templates/staff/serials/t_batch_receive.tt2 b/Open-ILS/src/templates/staff/serials/t_batch_receive.tt2
index 60c55bd5c4..409f62b868 100644
--- a/Open-ILS/src/templates/staff/serials/t_batch_receive.tt2
+++ b/Open-ILS/src/templates/staff/serials/t_batch_receive.tt2
@@ -1,4 +1,4 @@
-<form ng-submit="ok(items)" role="form">
+<form name="batch_receive_form" ng-submit="ok(items)" role="form">
 <div class="modal-header">
     <button type="button" class="close" ng-click="cancel()" 
         aria-hidden="true">&times;</button>
@@ -122,7 +122,7 @@
     <div class="col-md-2">
       <input ng-disabled="bind_or_none($index)" class="form-control" focus-me="$first"
              ng-model="item._barcode" type="text" id="item_barcode_{{$index}}"
-             eg-enter="focus_next_barcode($index)"/>
+             ng-required="!bind_or_none($index)" eg-enter="focus_next_barcode($index)"/>
     </div>
     <div class="col-md-1">
       <input type="checkbox" ng-disabled="cannot_print($index)" ng-model="item._print_routing_list"/>
@@ -135,7 +135,7 @@
   <div class="row">
     <div class="col-md-8"></div>
     <div class="col-md-4">
-      <input type="submit" class="btn btn-primary" value='{{ save_label || "[% l('Save') %]" }}'></input>
+      <input type="submit" class="btn btn-primary" ng-disabled="batch_receive_form.$error.required.length" value='{{ save_label || "[% l('Save') %]" }}'></input>
       <button class="btn btn-warning" ng-click="cancel()">[% l('Cancel') %]</button>
     </div>
   </div>
diff --git a/Open-ILS/web/js/ui/default/staff/serials/directives/view-items-grid.js b/Open-ILS/web/js/ui/default/staff/serials/directives/view-items-grid.js
index 26e76c8e90..c884a79e61 100644
--- a/Open-ILS/web/js/ui/default/staff/serials/directives/view-items-grid.js
+++ b/Open-ILS/web/js/ui/default/staff/serials/directives/view-items-grid.js
@@ -350,10 +350,12 @@ function($scope , $q , egSerialsCoreSvc , egCore , egGridDataProvider , orderByF
                             pile_o_promises.push(
                                 egCore.pcrud.retrieve(
                                     'acn', i.unit().call_number()
-                                ).then(function(cn){ i._call_number = cn.label() })
+                                ).then(function(cn){ if (cn.deleted() == 'f') i._call_number = cn.label() })
                             );
                         } else {
-                            if (i.stream().distribution()[mode + '_call_number']()) {
+                            if (i.stream().distribution()[mode + '_call_number']() && 
+                                i.stream().distribution()[mode + '_call_number']().deleted() == 'f'
+                            ) {
                                 i._call_number = i.stream().distribution()[mode + '_call_number']().label();
                             } else {
                                 pile_o_promises.push(
@@ -384,7 +386,7 @@ function($scope , $q , egSerialsCoreSvc , egCore , egGridDataProvider , orderByF
 
                     pile_o_promises.push(egCore.pcrud.search(
                         'acn',
-                        {record : bibId, owning_lib : egCore.org.fullPath(egCore.auth.user().ws_ou(), true)},
+                        {deleted : 'f', record : bibId, owning_lib : egCore.org.fullPath(egCore.auth.user().ws_ou(), true)},
                         {},{ atomic : true }
                     ).then(function (list) {
                         $scope.acn_list = list.map(function(i){return egCore.idl.toHash(i)});
-- 
2.11.0