[open-ils-commits] [GIT] Evergreen ILS branch rel_2_1 updated. d5aaefd25fc41cacdd3462ddcb5ebdfc205f2d73

[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_1 has been updated
       via  d5aaefd25fc41cacdd3462ddcb5ebdfc205f2d73 (commit)
      from  4757e6f3771d5e42995053c380e98759f169974d (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 d5aaefd25fc41cacdd3462ddcb5ebdfc205f2d73
Author: Jason Etheridge <jason at esilibrary.com>
Date:   Tue Sep 27 15:42:49 2011 -0400

    LP#860845 hold placement UI can break
    
    hold placement UI can break and give misleading error on Submit
    
    Adds some defensive coding, but since I don't quite grok everything going on here, may not be the best fix.
    
    Signed-off-by: Jason Etheridge <jason at esilibrary.com>
    Signed-off-by: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>

diff --git a/Open-ILS/web/opac/skin/default/js/holds.js b/Open-ILS/web/opac/skin/default/js/holds.js
index 4d1329f..61dd8a4 100644
--- a/Open-ILS/web/opac/skin/default/js/holds.js
+++ b/Open-ILS/web/opac/skin/default/js/holds.js
@@ -544,7 +544,9 @@ function __holdsDrawWindow() {
 				return (i.record() == holdArgs.record);	
 			}
 		);
-		formats = desc[0].item_type();
+		if (desc) {
+			formats = desc[0].item_type();
+		}
 	}
 
 	if( holdArgs.type == 'M' ) {

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

Summary of changes:
 Open-ILS/web/opac/skin/default/js/holds.js |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS