Bug 19993: use Modern::Perl in Acquisition perl scripts

[koha-ffzg.git] / acqui / acqui-home.pl

diff --git a/acqui/acqui-home.pl b/acqui/acqui-home.pl
index b58c8f4..cbb8896 100755 (executable)
--- a/acqui/acqui-home.pl
+++ b/acqui/acqui-home.pl
@@ -26,8 +26,7 @@ this script is the main page for acqui
 
 =cut
 
-use strict;
-use warnings;
+use Modern::Perl;
 
 use CGI qw ( -utf8 );
 use C4::Auth;
@@ -35,10 +34,10 @@ use C4::Output;
 use C4::Acquisition;
 use C4::Budgets;
 use C4::Members;
-use C4::Branch;
 use C4::Debug;
 use C4::Suggestions;
 use Koha::Acquisition::Currencies;
+use Koha::Patrons;
 
 my $query = CGI->new;
 my ( $template, $loggedinuser, $cookie, $userflags ) = get_template_and_user(
@@ -71,11 +70,12 @@ my @budget_loop;
 foreach my $budget ( @{$budget_arr} ) {
     next unless (CanUserUseBudget($loggedinuser, $budget, $userflags));
 
-    my $member = GetMember( borrowernumber => $budget->{budget_owner_id} );
-    if ($member) {
-        $budget->{budget_owner_firstname} = $member->{'firstname'};
-        $budget->{budget_owner_surname} = $member->{'surname'};
-        $budget->{budget_owner_borrowernumber} = $member->{'borrowernumber'};
+    my $patron = Koha::Patrons->find( $budget->{budget_owner_id} );
+    if ( $patron ) {
+        # FIXME should pass the entire object into budget_owner
+        $budget->{budget_owner_firstname} = $patron->firstname;
+        $budget->{budget_owner_surname} = $patron->surname;
+        $budget->{budget_owner_borrowernumber} = $patron->borrowernumber;
     }
 
     if ( !defined $budget->{budget_amount} ) {