From d93a8af77ac671a593e48224ac5682a0e20d62c2 Mon Sep 17 00:00:00 2001
From: Kathy Lussier <klussier@masslnc.org>
Date: Fri, 20 Feb 2015 09:03:13 -0500
Subject: [PATCH] LP1423922: Quick staff option to place another hold

When a hold is successful in the staff client, provide a link to place another
hold on the same title. This is a time savings for staff when placing holds
for multiple patrons on a newly-added title or when doing book club requests.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
---
 Open-ILS/src/templates/opac/css/style.css.tt2     |  2 ++
 Open-ILS/src/templates/opac/parts/place_hold.tt2  |  1 +
 .../templates/opac/parts/place_hold_result.tt2    | 15 ++++++++++++++-
 .../OPAC/place-another-hold-link.txt              |  7 +++++++
 4 files changed, 24 insertions(+), 1 deletion(-)
 create mode 100644 docs/RELEASE_NOTES_NEXT/OPAC/place-another-hold-link.txt

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index 577ecba61b..f93bf9e635 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -1206,6 +1206,8 @@ a.dash-link:hover { text-decoration: underline !important; }
 .hold-items-list-title { font-size: [% css_fonts.size_bigger %]; }
 .hold-items-list-problem { color: [% css_colors.text_alert %]; }
 
+.hold_success_links > span { margin: 0 2em; }
+
 .mr_holds_no_formats { 
   padding: 5px;
   margin-left: 25px;
diff --git a/Open-ILS/src/templates/opac/parts/place_hold.tt2 b/Open-ILS/src/templates/opac/parts/place_hold.tt2
index 8c70ca5ac7..192b65ba8c 100644
--- a/Open-ILS/src/templates/opac/parts/place_hold.tt2
+++ b/Open-ILS/src/templates/opac/parts/place_hold.tt2
@@ -16,6 +16,7 @@
             redirect = redirect.replace('^http:', 'https:') 
         %]
         <input type="hidden" name="redirect_to" value="[% redirect | html %]" />
+        <input type="hidden" name="hold_source_page" value="[% CGI.param('hold_source_page') | html %]" />
 
         [% IF ctx.is_staff %]
         <p class="staff-hold">
diff --git a/Open-ILS/src/templates/opac/parts/place_hold_result.tt2 b/Open-ILS/src/templates/opac/parts/place_hold_result.tt2
index 017a163f12..5ec0afc691 100644
--- a/Open-ILS/src/templates/opac/parts/place_hold_result.tt2
+++ b/Open-ILS/src/templates/opac/parts/place_hold_result.tt2
@@ -128,7 +128,20 @@
         [% IF any_failures OR ctx.general_hold_error %]
         <a href="[% CGI.param('redirect_to') || CGI.referer | html %]">[% l('Cancel') %]</a>
         [% ELSE %]
-        <a href="[% CGI.param('redirect_to') || CGI.referer | html %]">[% l('Continue') %]</a>
+        <div class='hold_success_links'>
+          <span><a href="[% CGI.param('redirect_to') || CGI.referer | html %]">[% l('Continue') %]</a></span>
+           [% IF ctx.is_staff %]
+             [% IF CGI.param('hold_type') == 'C';
+                  hold_type_label = 'copy';
+                ELSIF CGI.param('hold_type') == 'V';
+                  hold_type_label = 'volume';
+                ELSE;
+                  hold_type_label = 'title';
+              END %]
+               <span><a href="[% mkurl(ctx.opac_root _ '/place_hold', {hold_source_page => CGI.param('hold_source_page')}) %]">
+               [% l('Place another hold for this ') %] [% hold_type_label %]</a>
+               </span>
+          [% END %]
         [% END %]
         </span>
     </form>
diff --git a/docs/RELEASE_NOTES_NEXT/OPAC/place-another-hold-link.txt b/docs/RELEASE_NOTES_NEXT/OPAC/place-another-hold-link.txt
new file mode 100644
index 0000000000..521b59952a
--- /dev/null
+++ b/docs/RELEASE_NOTES_NEXT/OPAC/place-another-hold-link.txt
@@ -0,0 +1,7 @@
+Staff option to place another hold on same title
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+When a hold is successful in the client, staff will now see a link
+to place another hold on the same title. This link provides some workflow 
+improvement for times when staff are placing holds for multiple patrons on a
+newly-added title or when they are placing holds for book clubs.
+
-- 
2.43.2