From 2cc9df22ef56b6814712132150f0cd9b47929bf5 Mon Sep 17 00:00:00 2001
From: edoceo <code@edoceo.com>
Date: Wed, 1 Aug 2012 19:33:20 -0700
Subject: [PATCH] lp949322 Copy Patron barcode to clipboard when clicking their
 name CTRL+Click will open the patron record in a new tab

Signed-off-by: edoceo <code@edoceo.com>

This affects the patron label in Item Status -> Alt View -> Holds/Transits.  Looks good to me.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
---
 .../xul/staff_client/server/circ/alternate_copy_summary.js | 7 +++++++
 .../staff_client/server/circ/alternate_copy_summary.xul    | 2 +-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/Open-ILS/xul/staff_client/server/circ/alternate_copy_summary.js b/Open-ILS/xul/staff_client/server/circ/alternate_copy_summary.js
index 128fd22be4..29a7dd80cb 100644
--- a/Open-ILS/xul/staff_client/server/circ/alternate_copy_summary.js
+++ b/Open-ILS/xul/staff_client/server/circ/alternate_copy_summary.js
@@ -626,6 +626,13 @@ function load_item() {
             JSAN.use('patron.util'); 
             var au_obj = patron.util.retrieve_fleshed_au_via_id( ses(), details.hold.usr() );
             $('hold_patron_name').setAttribute('value', $('circStrings').getFormattedString('staff.circ.copy_details.user_details', [au_obj.family_name(), au_obj.first_given_name(), au_obj.card().barcode()]) );
+            $('hold_patron_name').onclick = function(e) {
+                if (e.ctrlKey) {
+                   window.xulG.new_patron_tab( {}, { 'id' : au_obj.id() } );
+                   return;
+                }
+                copy_to_clipboard(au_obj.card().barcode());
+            };
 
             set("hold_status", details.hold.status()); 
             set("transit", details.hold.transit()); 
diff --git a/Open-ILS/xul/staff_client/server/circ/alternate_copy_summary.xul b/Open-ILS/xul/staff_client/server/circ/alternate_copy_summary.xul
index bcdf212112..7d39fd9f82 100644
--- a/Open-ILS/xul/staff_client/server/circ/alternate_copy_summary.xul
+++ b/Open-ILS/xul/staff_client/server/circ/alternate_copy_summary.xul
@@ -287,7 +287,7 @@
                 <tabpanel orient="vertical"><!-- Hold/Transit -->
                     <groupbox flex="1" id="holds" style="overflow: none; min-height: 80;" oils_persist="height">
                         <caption id="hold_caption" label="&staff.circ.copy_details.hold_caption;"/>
-                        <label id="hold_patron_name" class="patronNameLarge"/>
+                        <label id="hold_patron_name" class="patronNameLarge click_link"/>
                         <tree id="hold" flex="1" enableColumnDrag="true"/>
                         <spacer FIXME="label and tree get swapped without this"/>
                     </groupbox>
-- 
2.43.2