[open-ils-commits] [GIT] Evergreen ILS branch master updated. 70120afae4774cafdd6362c517237808a49d5145

[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, master has been updated
       via  70120afae4774cafdd6362c517237808a49d5145 (commit)
      from  07475722b8b483537b8b2795a800eba5b1af36b2 (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 70120afae4774cafdd6362c517237808a49d5145
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