Fix broken menu entries under "Other -> Booking" in patron interface
authorLebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Fri, 1 Jun 2012 19:43:54 +0000 (15:43 -0400)
committerBill Erickson <berick@esilibrary.com>
Fri, 1 Jun 2012 20:29:39 +0000 (16:29 -0400)
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>

Open-ILS/web/js/dojo/openils/XUL.js

index 6bf2c3b..a7f4bfc 100644 (file)
@@ -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,