Bugzilla – Attachment 13585 Details for
Bug 9115
[3.6.X] basket window should close automatically when placing a hold
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 9115 - basket window should close automatically when placing a hold
Bug-9115---basket-window-should-close-automaticall.patch (text/plain), 2.83 KB, created by
Kyle M Hall
on 2012-11-21 14:20:18 UTC
(
hide
)
Description:
Bug 9115 - basket window should close automatically when placing a hold
Filename:
MIME Type:
Creator:
Kyle M Hall
Created:
2012-11-21 14:20:18 UTC
Size:
2.83 KB
patch
obsolete
>From d10799eca4035611d67d7e302cb108f0c7962bb3 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Tue, 20 Nov 2012 09:49:19 -0500 >Subject: [PATCH] Bug 9115 - basket window should close automatically when placing a hold >Content-Type: text/plain; charset="utf-8" > >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 <kyle@bywatersolutions.com> >--- > 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 8a0af37..95a59bf 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("<span>"+updated_value+"</span>"); >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 0672027..f1c8a0e 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt >+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt >@@ -124,16 +124,7 @@ function tagAdded() { > }); > [% END %][% END %][% END %] > }); >-[% 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"); >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 9115
:
13555
|
13585
|
13659
|
14003
|
16260
|
17398