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=eab0ef37637139fcb56f2546c0cc7a3b23aa97ff;p=working%2FEvergreen.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 6dee94d0cf..55d9c2b584 100644
--- a/Open-ILS/web/opac/skin/default/js/holds.js
+++ b/Open-ILS/web/opac/skin/default/js/holds.js
@@ -460,7 +460,9 @@ function __holdsDrawWindow() {
 				return (i.record() == holdArgs.record);	
 			}
 		);
-		formats = desc[0].item_type();
+		if (desc) {
+			formats = desc[0].item_type();
+		}
 	}
 
 	if( holdArgs.type == 'M' ) {