From 72c1a499b730f3794e6309d6ea1659355865c571 Mon Sep 17 00:00:00 2001
From: Bill Ott <bott@grpl.org>
Date: Mon, 20 Sep 2021 17:34:00 -0500
Subject: [PATCH] LP#1705332 - Add creation date and creator(staff user
 account) to patron barcodes.

Code from Bill Ott for local change that they have been using for a number of years.

Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
---
 Open-ILS/examples/fm_IDL.xml                                      | 3 +++
 Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm            | 1 +
 .../src/perlmods/lib/OpenILS/Application/Storage/CDBI/actor.pm    | 2 +-
 Open-ILS/src/templates/actor/user/register.tt2                    | 4 ++++
 .../src/templates/staff/circ/patron/t_patron_cards_dialog.tt2     | 8 ++++++++
 Open-ILS/src/templates/staff/css/circ.css.tt2                     | 2 ++
 6 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/Open-ILS/examples/fm_IDL.xml b/Open-ILS/examples/fm_IDL.xml
index a4fb45b078..41b82278d9 100644
--- a/Open-ILS/examples/fm_IDL.xml
+++ b/Open-ILS/examples/fm_IDL.xml
@@ -7416,9 +7416,12 @@ SELECT  usr,
 			<field reporter:label="Barcode" name="barcode" reporter:datatype="text"/>
 			<field reporter:label="Card ID" name="id" reporter:datatype="id" />
 			<field reporter:label="User" name="usr" reporter:datatype="link"/>
+                        <field reporter:label="Creator" name="creator" reporter:datatype="link" />
+                        <field reporter:label="Create Date" name="create_date" reporter:datatype="timestamp"/>
 		</fields>
 		<links>
 			<link field="usr" reltype="has_a" key="id" map="" class="au"/>
+                        <link field="creator" reltype="has_a" key="id" map="" class="au"/>
 		</links>
 		<permacrud xmlns="http://open-ils.org/spec/opensrf/IDL/permacrud/v1">
 			<actions>
diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm
index 56df630e79..a3f85b4407 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm
@@ -1039,6 +1039,7 @@ sub _add_update_cards {
 sub _add_card {
     my( $e, $card ) = @_;
     $card->clear_id();
+    $card->creator($e->requestor->id);
 
     $logger->info("Adding new patron card ".$card->barcode);
 
diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/CDBI/actor.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/CDBI/actor.pm
index 6bf0481662..8772a0ad35 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/CDBI/actor.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/CDBI/actor.pm
@@ -150,7 +150,7 @@ use base qw/actor/;
 
 __PACKAGE__->table( 'actor_card' );
 __PACKAGE__->columns( Primary => qw/id/ );
-__PACKAGE__->columns( Essential => qw/usr barcode active/ );
+__PACKAGE__->columns( Essential => qw/usr barcode active creator create_date/ );
 
 #-------------------------------------------------------------------------------
 package actor::user_access_entry;
diff --git a/Open-ILS/src/templates/actor/user/register.tt2 b/Open-ILS/src/templates/actor/user/register.tt2
index 7a02e9f8ad..7fcc04a62e 100644
--- a/Open-ILS/src/templates/actor/user/register.tt2
+++ b/Open-ILS/src/templates/actor/user/register.tt2
@@ -62,6 +62,8 @@
                     <th id='uedit-all-cards-barcode'>[% l('Barcode') %]</th>
                     <th id='uedit-all-cards-active'>[% l('Active') %]</th>
                     <th id='uedit-all-cards-primary'>[% l('Primary') %]</th>
+                    <th id='uedit-all-cards-create_date'>[% l('Created') %]</th>
+                    <th id='uedit-all-cards-creator'>[% l('Creator') %]</th>
                 </tr>
             </thead>
             <tbody id='uedit-all-cards-tbody'>
@@ -69,6 +71,8 @@
                     <td><div name='barcode'></div></td>
                     <td><div name='active'></div></td>
                     <td><div name='primary'></div></td>
+                    <td><div name='create_date'></div></td>
+                    <td><div name='creator'></div></td>
                 </tr>
             </tbody>
             <tbody>
diff --git a/Open-ILS/src/templates/staff/circ/patron/t_patron_cards_dialog.tt2 b/Open-ILS/src/templates/staff/circ/patron/t_patron_cards_dialog.tt2
index b171d71c77..10c59701f6 100644
--- a/Open-ILS/src/templates/staff/circ/patron/t_patron_cards_dialog.tt2
+++ b/Open-ILS/src/templates/staff/circ/patron/t_patron_cards_dialog.tt2
@@ -15,6 +15,12 @@
         <div class="col-md-4">
           <label>[% l('Primary') %]</label>
         </div>
+        <div class="col-md-4">
+          <label>[% l('Created') %]</label>
+        </div>
+        <div class="col-md-4">
+          <label>[% l('Creator') %]</label>
+        </div>
       </div>
       <div class="row" ng-repeat="card in args.cards">
         <div class="col-md-4">{{card.barcode}}</div>
@@ -28,6 +34,8 @@
             ng-value='card.id'
             ng-disabled="!perms.UPDATE_PATRON_PRIMARY_CARD"/>
         </div>
+        <div class="col-md-4">{{card.create_date | limitTo:10}}</div>
+        <div class="col-md-4">{{card.creator}}</div>
       </div>
     </div>
     <div class="modal-footer">
diff --git a/Open-ILS/src/templates/staff/css/circ.css.tt2 b/Open-ILS/src/templates/staff/css/circ.css.tt2
index 5a921017bc..b7e92bcba1 100644
--- a/Open-ILS/src/templates/staff/css/circ.css.tt2
+++ b/Open-ILS/src/templates/staff/css/circ.css.tt2
@@ -154,6 +154,8 @@ but the ones I'm finding aren't quite cutting it..*/
   font-weight: bold;
 }
 
+.patron-reg-barcodes div .col-md-4 { width: 115px; }
+
 .patron-reg-barcodes > .header {
   font-weight: bold;
 }
-- 
2.11.0