[open-ils-commits] [GIT] Evergreen ILS branch rel_2_2 updated. 64658bb975b6fd9ca5df403e0e810219df97dbfd

Evergreen Git git at git.evergreen-ils.org
Mon Feb 4 10:18:51 EST 2013


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_2 has been updated
       via  64658bb975b6fd9ca5df403e0e810219df97dbfd (commit)
      from  42980607363da546727ab0e29ae921d907c3cade (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 64658bb975b6fd9ca5df403e0e810219df97dbfd
Author: Jason Etheridge <jason at esilibrary.com>
Date:   Thu Sep 20 13:10:15 2012 -0400

    LP1053526 - Don't inadvertently modify the in-memory org tree when printing
    
    To see an example of this causing a problem, load Holdings Maintenance for
    an arbitrary bib record with items. Make sure the Hide Empty Libs checkbox
    is checked. Hit the Print button at the bottom of the UI (it's okay to then
    Cancel the print). Then hit Refresh. It should result in an org.children()
    is null error.
    
    Signed-off-by: Jason Etheridge <jason at esilibrary.com>
    Signed-off-by: Ben Shum <bshum at biblio.org>

diff --git a/Open-ILS/xul/staff_client/chrome/content/util/list.js b/Open-ILS/xul/staff_client/chrome/content/util/list.js
index d6057e3..4adb879 100644
--- a/Open-ILS/xul/staff_client/chrome/content/util/list.js
+++ b/Open-ILS/xul/staff_client/chrome/content/util/list.js
@@ -1576,8 +1576,8 @@ util.list.prototype = {
                 params.staff = data.list.au[0];
             }
             if (!params.lib && data.list.au && data.list.au[0] && data.list.au[0].ws_ou() && data.hash.aou && data.hash.aou[ data.list.au[0].ws_ou() ]) {
-                params.lib = data.hash.aou[ data.list.au[0].ws_ou() ];
-                params.lib.children(null);
+                params.lib = JSON2js( js2JSON( data.hash.aou[ data.list.au[0].ws_ou() ] ) ); // clone this sucker
+                params.lib.children(null); // since we're modifying it
             }
             if (params.template && data.print_list_templates[ params.template ]) {
                 var template = data.print_list_templates[ params.template ];

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

Summary of changes:
 .../xul/staff_client/chrome/content/util/list.js   |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
Evergreen ILS


More information about the open-ils-commits mailing list