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 4d1329f..61dd8a4 100644 (file)
--- 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' ) {