# along with Koha; if not, see <http://www.gnu.org/licenses>.
-use strict;
-use warnings;
+use Modern::Perl;
use CGI qw ( -utf8 );
use C4::Circulation;
use C4::Context;
use C4::Items;
use C4::Members;
+use Koha::Items;
+use Koha::Patrons;
use Date::Calc qw( Today Date_to_Days );
my $query = new CGI;
my $errorstring = q{};
my $renewed = q{};
-my $member_details = GetMemberDetails($borrowernumber);
+my $patron = Koha::Patrons->find( $borrowernumber );
-if ( $member_details->{'BlockExpiredPatronOpacActions'}
- && $member_details->{'is_expired'} )
+if ( $patron->category->effective_BlockExpiredPatronOpacActions
+ && $patron->is_expired )
{
$errorstring = 'card_expired';
}
my $renewalbranch = C4::Context->preference('OpacRenewalBranch');
my $branchcode;
if ( $renewalbranch eq 'itemhomebranch' ) {
- my $item = GetItem($itemnumber);
- $branchcode = $item->{'homebranch'};
+ my $item = Koha::Items->find($itemnumber);
+ $branchcode = $item->homebranch;
}
elsif ( $renewalbranch eq 'patronhomebranch' ) {
- my $borrower = GetMemberDetails($borrowernumber);
- $branchcode = $borrower->{'branchcode'};
+ $branchcode = Koha::Patrons->find( $borrowernumber )->branchcode;
}
elsif ( $renewalbranch eq 'checkoutbranch' ) {
- my $issue = GetOpenIssue($itemnumber);
+ my $issue = GetOpenIssue($itemnumber); # FIXME Should not be $item->checkout?
$branchcode = $issue->{'branchcode'};
}
elsif ( $renewalbranch eq 'NULL' ) {
else {
$branchcode = 'OPACRenew';
}
- AddRenewal( $borrowernumber, $itemnumber, $branchcode );
+ AddRenewal( $borrowernumber, $itemnumber, $branchcode, undef, undef );
push( @renewed, $itemnumber );
}
else {