[open-ils-commits] [GIT] Evergreen ILS branch rel_3_0 updated. 1fb068b214bdc029c73d22e2046d86ace918d9de

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_3_0 has been updated
       via  1fb068b214bdc029c73d22e2046d86ace918d9de (commit)
      from  2fc3252063450c10032e0e7e987874c2774d400c (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 1fb068b214bdc029c73d22e2046d86ace918d9de
Author: Jason Boyer <jboyer at library.in.gov>
Date:   Thu Jan 4 13:38:00 2018 -0500

    LP1712646: Require a Billing Type to Bill User
    
    Previously billing a user without choosing a billing type
    would either fail silently or default to misc. This patch
    requires staff to choose a billing type before the new
    bill window can be dismissed.
    
    Signed-off-by: Jason Boyer <jboyer at library.in.gov>
    Signed-off-by: Bill Erickson <berickxx at gmail.com>

diff --git a/Open-ILS/src/templates/staff/circ/share/t_bill_patron_dialog.tt2 b/Open-ILS/src/templates/staff/circ/share/t_bill_patron_dialog.tt2
index f6d55aa..dd10c8b 100644
--- a/Open-ILS/src/templates/staff/circ/share/t_bill_patron_dialog.tt2
+++ b/Open-ILS/src/templates/staff/circ/share/t_bill_patron_dialog.tt2
@@ -59,11 +59,10 @@
           [% l('Billing Type:') %]
         </label>
         <div class="col-md-8">
-          <select ng-model="billArgs.billingType" class="form-control"
-            ng-change="updateDefaultPrice()">
-            <option ng-repeat="type in billingTypes | orderBy:'name()'" value="{{type.id()}}">
-              {{type.name()}}
-            </option>
+          <select str-to-int ng-required="true" ng-model="billArgs.billingType" class="form-control"
+            ng-change="updateDefaultPrice()"
+            ng-options="type.id() as type.name() for type in billingTypes | orderBy: 'name()'" >
+           <option value=""></option>
           </select>
         </div>
       </div>
@@ -71,7 +70,7 @@
         <label for="bill-dialog-amount" class="control-label col-md-4">[% l('Amount:') %]</label>
         <div class="col-md-8">
           <input type="number" min="0" step="any" class="form-control" 
-            focus-me='focus' required id="bill-dialog-amount" 
+            focus-me='focus' ng-required="true" id="bill-dialog-amount"
             ng-model="billArgs.amount"/>
         </div>
       </div>
diff --git a/Open-ILS/web/js/ui/default/staff/circ/services/billing.js b/Open-ILS/web/js/ui/default/staff/circ/services/billing.js
index b01f9bf..352cd74 100644
--- a/Open-ILS/web/js/ui/default/staff/circ/services/billing.js
+++ b/Open-ILS/web/js/ui/default/staff/circ/services/billing.js
@@ -64,8 +64,9 @@ function($uibModal , $q , egCore) {
     // fetch the org-focused billing types
     // Cache on egEnv
     service.fetchBillingTypes = function() {
-        if (egCore.env.cbt) 
+        if (egCore.env.cbt) {
             return $q.when(egCore.env.cbt.list);
+        }
 
         return egCore.pcrud.search('cbt', 
             {   // first 100 are reserved for system-generated bills
@@ -170,6 +171,3 @@ function($uibModal , $q , egCore) {
     return service;
 }]);
 
-
-
-

-----------------------------------------------------------------------

Summary of changes:
 .../staff/circ/share/t_bill_patron_dialog.tt2      |   11 +++++------
 .../js/ui/default/staff/circ/services/billing.js   |    6 ++----
 2 files changed, 7 insertions(+), 10 deletions(-)


hooks/post-receive
-- 
Evergreen ILS