[open-ils-commits] [GIT] Evergreen ILS branch master updated. 0a762feebd9ddb45584182ea438bd6693e524cdf

[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  0a762feebd9ddb45584182ea438bd6693e524cdf (commit)
      from  149ffb312303594a4667e09abca5c670dd2de954 (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 0a762feebd9ddb45584182ea438bd6693e524cdf
Author: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
Date:   Thu Feb 20 12:21:57 2014 -0500

    LP#1282704: Serials: in routing lists, avoid extra page breaks wasting paper at print time
    
    If you're using batch receive, and you have two copies (and therefore
    two streams) that you're receiving with the Routing List? checkbox
    checked, but only one of those streams actually has a routing list,
    you'll note before this patch is applied that you get an extra page of
    blank paper if you print the routing list that pops up.
    
    This fixes that.
    
    Signed-off-by: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

diff --git a/Open-ILS/web/js/ui/default/serial/print_routing_list_users.js b/Open-ILS/web/js/ui/default/serial/print_routing_list_users.js
index 9d5f852..917ff75 100644
--- a/Open-ILS/web/js/ui/default/serial/print_routing_list_users.js
+++ b/Open-ILS/web/js/ui/default/serial/print_routing_list_users.js
@@ -9,6 +9,8 @@ function ListRenderer() {
     var self = this;
 
     this.render = function() {
+        var rendered_something = false;
+
         for (var i = 0; i < this.streams.length; i++) {
             var stream = this.streams[i];
 
@@ -35,7 +37,7 @@ function ListRenderer() {
 
             this.render_users(stream, list);
 
-            if (i) {
+            if (rendered_something) {
                 dojo.create(
                     "hr",
                     {"style": "page-break-after: always"}, this.target, "last"
@@ -43,6 +45,7 @@ function ListRenderer() {
             }
 
             dojo.place(list, this.target, "last");
+            rendered_something = true;
         }
 
         return this; /* for chaining */

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

Summary of changes:
 .../ui/default/serial/print_routing_list_users.js  |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS