From 464acf31ce98b87a6bc8e20b29b5bda46e0e0cf1 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Tue, 20 Nov 2012 09:49:19 -0500 Subject: [PATCH] Bug 9115 - basket window should close automatically when placing a hold If you choose to place a hold from the Cart pop-up, the entire holds process (possibly including login) takes place in the cart window. Upon completion of the operation you're left with a second window which has lost its context as the Cart. This patch revises the hold process so that when you click the hold link in the cart the operation is moved to the main window and the Cart window closes. Since the holdSel() function doesn't require interaction with template variables I have moved it to basket.js along with the described changes. To test, put items in your Cart and open it. Select items to place on hold and click the "Place hold" link. The cart should close, and the items you selected should appear on the place hold screen in the main window. Signed-off-by: Kyle M Hall Signed-off-by: Katrin Fischer Checked in Firefox and Chromium in Ubuntu, works nicely. All tests pass. Signed-off-by: Jared Camins-Esakov Signed-off-by: Chris Cormack --- koha-tmpl/opac-tmpl/prog/en/js/basket.js | 10 ++++++++++ koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt | 11 +---------- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/koha-tmpl/opac-tmpl/prog/en/js/basket.js b/koha-tmpl/opac-tmpl/prog/en/js/basket.js index 8a0af37c23..95a59bf3ad 100644 --- a/koha-tmpl/opac-tmpl/prog/en/js/basket.js +++ b/koha-tmpl/opac-tmpl/prog/en/js/basket.js @@ -397,6 +397,16 @@ function showLess() { document.location = loc; } +function holdSel() { + var items = document.getElementById('records').value; + if (items) { + parent.opener.document.location = "/cgi-bin/koha/opac-reserve.pl?biblionumbers=" + items; + window.close(); + } else { + alert(MSG_NO_RECORD_SELECTED); + } +} + function updateBasket(updated_value,target) { if(target){ target.$('#basketcount').html(""+updated_value+""); diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt index 35aff0f22b..90a259e5b0 100644 --- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt +++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt @@ -119,16 +119,7 @@ function tagAdded() { }); }); -[% IF ( opacuserlogin ) %][% IF ( RequestOnOpac ) %] - function holdSel() { - var items = document.getElementById('records').value; - if (items) { - document.location = "/cgi-bin/koha/opac-reserve.pl?biblionumbers=" + items; - } else { - alert(MSG_NO_RECORD_SELECTED); - } - } -[% END %][% END %] + function enableCheckboxActions(){ // Enable/disable controls if checkboxes are checked var checkedBoxes = $(".checkboxed input:checkbox:checked"); -- 2.39.5