[open-ils-commits] [GIT] Evergreen ILS branch rel_2_5 updated. b9be5b319f5a4855b34ce3bb8f508dfa90d22ca0

[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_2_5 has been updated
       via  b9be5b319f5a4855b34ce3bb8f508dfa90d22ca0 (commit)
      from  7c8c96a8c95d0ed4717f5f012cab7986586beca3 (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 b9be5b319f5a4855b34ce3bb8f508dfa90d22ca0
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