[open-ils-commits] [GIT] Evergreen ILS branch rel_3_1 updated. 14e4e481493ca89bd36cc6576715043059991101

[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_1 has been updated
       via  14e4e481493ca89bd36cc6576715043059991101 (commit)
      from  c40a880f49ca0bab728e9e8f777a87b388103968 (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 14e4e481493ca89bd36cc6576715043059991101
Author: Jason Boyer <jboyer at library.in.gov>
Date:   Thu Jul 19 14:29:32 2018 -0400

    LP1782631: Stop Styling Grocery Bills as Overdues
    
    Only checking for the lack of a value in the
    checkin_time field causes grocery bills to still
    trigger the bill colorizing feature, but because
    they have no stop_fines reason they're colored
    the same as an overdue item.
    
    This patch requires the presence of a due_date
    field before coloring a bill row.
    
    Signed-off-by: Jason Boyer <jboyer at library.in.gov>
    Signed-off-by: Dan Wells <dbw2 at calvin.edu>

diff --git a/Open-ILS/web/js/ui/default/staff/circ/patron/bills.js b/Open-ILS/web/js/ui/default/staff/circ/patron/bills.js
index 63a7121..8f4157f 100644
--- a/Open-ILS/web/js/ui/default/staff/circ/patron/bills.js
+++ b/Open-ILS/web/js/ui/default/staff/circ/patron/bills.js
@@ -223,7 +223,7 @@ function($scope , $q , $routeParams , egCore , egConfirmDialog , $location,
     // Apply coloring to rows based on fines stop reason
     $scope.colorizeBillsList = {
         apply: function(item) {
-            if (!item['circulation.checkin_time']) {
+            if (item['circulation.due_date'] && !item['circulation.checkin_time']) {
                 if (item['circulation.stop_fines'] == 'LOST') {
                     return 'lost-row';
                 } else if (item['circulation.stop_fines'] == 'LONGOVERDUE') {
@@ -240,7 +240,7 @@ function($scope , $q , $routeParams , egCore , egConfirmDialog , $location,
         isEnabled: true,
         template: function(item) {
             var icon = '';
-            if (!item['circulation.checkin_time']) {
+            if (item['circulation.due_date'] && !item['circulation.checkin_time']) {
                 if (item['circulation.stop_fines'] == "LOST") {
                     icon = 'glyphicon-question-sign';
                 } else if (item['circulation.stop_fines'] == "LONGOVERDUE") {

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

Summary of changes:
 .../web/js/ui/default/staff/circ/patron/bills.js   |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
Evergreen ILS