From: Jason Etheridge <jason@esilibrary.com>
Date: Tue, 27 Sep 2011 19:42:49 +0000 (-0400)
Subject: LP#860845 hold placement UI can break
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=b14b8d123df40abd9c543efa62fc4f59479b3f62;p=evergreen%2Fequinox.git

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@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@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 b8a5e4902a..b38953d0b4 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' ) {