[open-ils-commits] r16401 - trunk/Open-ILS/xul/staff_client/server/circ (senator)

[svn at svn.open-ils.org]

Author: senator
Date: 2010-05-06 15:52:20 -0400 (Thu, 06 May 2010)
New Revision: 16401

Modified:
   trunk/Open-ILS/xul/staff_client/server/circ/util.js
Log:
Wrap interface results from "find originating purchase order" in browser


Modified: trunk/Open-ILS/xul/staff_client/server/circ/util.js
===================================================================
--- trunk/Open-ILS/xul/staff_client/server/circ/util.js	2010-05-06 19:49:25 UTC (rev 16400)
+++ trunk/Open-ILS/xul/staff_client/server/circ/util.js	2010-05-06 19:52:20 UTC (rev 16401)
@@ -3158,14 +3158,17 @@
             "onresponse": function(r) {
                 if (r = openils.Util.readResponse(r)) {
                     if (r.purchase_order()) {
-                        /* XXX would prefer to use browser.xul to wrap this so
-                         * that we get back/forward/reload buttons, but that
-                         * doesn't work in this context. need to find out why.
-                         */
-                        xulG.new_tab(
-                            urls.EG_ACQ_PO_VIEW +
-                                "/" + r.purchase_order() + "/" + r.id(),
-                            {}, {}
+                        var url = urls.XUL_BROWSER + "?url=" +
+                            xulG.url_prefix(
+                                escape(urls.EG_ACQ_PO_VIEW +
+                                    "/" + r.purchase_order() + "/" + r.id())
+                            );
+                        window.xulG.new_tab(
+                            url, {"browser": true}, {
+                                "no_xulG": false,
+                                "show_print_button": false,
+                                "show_nav_buttons": true
+                            }
                         );
                     } else {
                         /* unlikely: got an LI with no PO */