From: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Date: Fri, 1 Jun 2012 19:43:54 +0000 (-0400)
Subject: Fix broken menu entries under "Other -> Booking" in patron interface
X-Git-Url: http://git.equinoxoli.org/?p=transitory.git;a=commitdiff_plain;h=9d7807b356d75453a5892e75b5538dcc2c908c40

Fix broken menu entries under "Other -> Booking" in patron interface

This issue was reported to me by Steve Callender.  The menu entries
under "Other -> Booking" had all stopped working.

There's a catalog of URL constants under window.urls sometimes, and
formerly (or other times?) at xulG.urls for XUL-based staff client
interfaces.  There's a new-tab-opening method that needed to be more
flexible when looking for that catalog.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
---

diff --git a/Open-ILS/web/js/dojo/openils/XUL.js b/Open-ILS/web/js/dojo/openils/XUL.js
index 6bf2c3b..a7f4bfc 100644
--- a/Open-ILS/web/js/dojo/openils/XUL.js
+++ b/Open-ILS/web/js/dojo/openils/XUL.js
@@ -57,7 +57,8 @@ if(!dojo._hasResource["openils.XUL"]) {
         var loc = xulG.url_prefix(url);
 
         if (wrap_in_browser) {
-            loc = xulG.urls.XUL_BROWSER + "?url=" + window.escape(loc);
+            var urls = xulG.urls || window.urls;
+            loc = urls.XUL_BROWSER + "?url=" + window.escape(loc);
             content_params = dojo.mixin(
                 {
                     "no_xulG": false, "show_print_button": true,