From 7e7b4c71be1d3a4b2339cda9ef3bbb7618a8964c Mon Sep 17 00:00:00 2001 From: David Cook Date: Mon, 6 Jan 2014 15:19:21 +1100 Subject: [PATCH] Bug 11477 - Add names for librarian and borrowers in the logs This patch adds the first name and surname of the librarian and the user involved in a transaction log, if available. If neither the firstname or the surname is available, then the 'object' number will be prefaced with the 'Member' qualifier. _TEST PLAN_ 0) This patch depends on Bug 11473, so you must apply it first Before applying this patch (11477): 1) View the logs. 2) Note that the Librarian column just has a number 3) Note that the Object column will just say Member X (where X is a number), in the circulation, fines, and patron/member modules. Apply the patch. 4) Reload the logs (you don't need to make new ones) 5) Note that the Librarian and Object columns mentioned above now have firstname and surname appearing (where available) Signed-off-by: Chris Cormack Signed-off-by: Katrin Fischer Works as described, passes all tests and QA script. Signed-off-by: Galen Charlton --- .../prog/en/modules/tools/viewlog.tt | 4 ++-- tools/viewlog.pl | 22 +++++++++++++++++++ 2 files changed, 24 insertions(+), 2 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/viewlog.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/viewlog.tt index 9809383316..38b2fcfac5 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/viewlog.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/viewlog.tt @@ -127,13 +127,13 @@ [% loopro.timestamp %] - [% loopro.user %] + [% IF ( loopro.userfirstname ) || ( loopro.usersurname ) %][% loopro.userfirstname %] [% loopro.usersurname %] [% END %][% loopro.user %] [% loopro.module %] [% loopro.action %] [% IF ( loopro.module == 'MEMBERS' ) || ( loopro.module == 'CIRCULATION' ) || ( loopro.module == 'FINES' ) %] - [% IF ( loopro.object ) %]Member [% loopro.object %][% END %] + [% IF ( loopro.object ) %][% IF ( loopro.borrowerfirstname ) || ( loopro.borrowersurname ) %][% loopro.borrowerfirstname %] [% loopro.borrowersurname %] [% ELSE %]Member [% END %][% loopro.object %][% END %] [% ELSE %] [% IF ( loopro.module == 'CATALOGUING' ) %] [% IF ( loopro.info.substr(0, 4) == 'item' ) %] diff --git a/tools/viewlog.pl b/tools/viewlog.pl index 949bec7201..1fbd6be100 100755 --- a/tools/viewlog.pl +++ b/tools/viewlog.pl @@ -124,6 +124,28 @@ if ($do_it) { $result->{'biblioitemnumber'}=$item->{'biblionumber'}; $result->{'barcode'}=$item->{'barcode'}; } + #always add firstname and surname for librarian/user + if ($result->{'user'}){ + my $userdetails = C4::Members::GetMemberDetails($result->{'user'}); + if ($userdetails->{'firstname'}){ + $result->{'userfirstname'} = $userdetails->{'firstname'}; + } + if ($userdetails->{'surname'}){ + $result->{'usersurname'} = $userdetails->{'surname'}; + } + } + #add firstname and surname for borrower, when using the CIRCULATION, MEMBERS, FINES + if ($result->{module} eq "CIRCULATION" || $result->{module} eq "MEMBERS" || $result->{module} eq "FINES"){ + if($result->{'object'}){ + my $borrowerdetails = C4::Members::GetMemberDetails($result->{'object'}); + if ($borrowerdetails->{'firstname'}){ + $result->{'borrowerfirstname'} = $borrowerdetails->{'firstname'}; + } + if ($borrowerdetails->{'surname'}){ + $result->{'borrowersurname'} = $borrowerdetails->{'surname'}; + } + } + } } if ( $output eq "screen" ) { -- 2.39.5