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

[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  b14b8d123df40abd9c543efa62fc4f59479b3f62 (commit)
      from  173eb0fb2cfa3be5d703f44c6a0a49162b21668e (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 b14b8d123df40abd9c543efa62fc4f59479b3f62
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 b8a5e49..b38953d 100644
--- a/Open-ILS/web/opac/skin/default/js/holds.js
+++ b/Open-ILS/web/opac/skin/default/js/holds.js
@@ -561,7 +561,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