From e8f7862a55b82e1a0c36e33c88b1330fd49f725d Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Fri, 17 Dec 2021 10:18:53 -0300 Subject: [PATCH] Bug 27946: Article request fee methods in Koha::Patron This patch adds article_request_fee() and add_article_request_fee_if_needed() methods to Koha::Patron. To test: 1. Apply this patch 2. Run: $ kshell k$ prove t/db_dependent/Koha/Patron.t => SUCCESS: Tests pass! 3. Sign off :-D --- Koha/Patron.pm | 84 +++++++++++++++++++++++++ t/db_dependent/Koha/Patron.t | 115 ++++++++++++++++++++++++++++++++++- 2 files changed, 198 insertions(+), 1 deletion(-) diff --git a/Koha/Patron.pm b/Koha/Patron.pm index 85d9c01bf1..0c3c2396c1 100644 --- a/Koha/Patron.pm +++ b/Koha/Patron.pm @@ -1003,6 +1003,90 @@ sub can_request_article { return $count < $limit ? 1 : 0; } +=head3 article_request_fee + + my $fee = $patron->article_request_fee( + { + [ library_id => $library->id, ] + } + ); + +Returns the fee to be charged to the patron when it places an article request. + +A I can be passed as parameter, falling back to userenv if absent. + +=cut + +sub article_request_fee { + my ($self, $params) = @_; + + my $library_id = $params->{library_id}; + + $library_id //= C4::Context->userenv ? C4::Context->userenv->{'branch'} : undef; + + my $rule = Koha::CirculationRules->get_effective_rule( + { + branchcode => $library_id, + categorycode => $self->categorycode, + rule_name => 'article_request_fee' + } + ); + + my $fee = ($rule) ? $rule->rule_value + 0 : 0; + + return $fee; +} + +=head3 add_article_request_fee_if_needed + + my $fee = $patron->add_article_request_fee_if_needed( + { + [ item_id => $item->id, + library_id => $library->id, ] + } + ); + +If an article request fee needs to be charged, it adds a debit to the patron's +account. + +Returns the fee line. + +A I can be passed as parameter, falling back to userenv if absent. + +=cut + +sub add_article_request_fee_if_needed { + my ($self, $params) = @_; + + my $library_id = $params->{library_id}; + my $item_id = $params->{item_id}; + + $library_id //= C4::Context->userenv ? C4::Context->userenv->{'branch'} : undef; + + my $amount = $self->article_request_fee( + { + library_id => $library_id, + } + ); + + my $debit_line; + + if ( $amount > 0 ) { + $debit_line = $self->account->add_debit( + { + amount => $amount, + user_id => C4::Context->userenv ? C4::Context->userenv->{'number'} : undef, + interface => C4::Context->interface, + library_id => $library_id, + type => 'ARTICLE_REQUEST_FEE', + item_id => $item_id, + } + ); + } + + return $debit_line; +} + =head3 article_requests my $article_requests = $patron->article_requests; diff --git a/t/db_dependent/Koha/Patron.t b/t/db_dependent/Koha/Patron.t index 2929349661..524dc6e4ed 100755 --- a/t/db_dependent/Koha/Patron.t +++ b/t/db_dependent/Koha/Patron.t @@ -19,10 +19,11 @@ use Modern::Perl; -use Test::More tests => 10; +use Test::More tests => 12; use Test::Exception; use Test::Warn; +use Koha::CirculationRules; use Koha::Database; use Koha::DateUtils qw(dt_from_string); use Koha::ArticleRequests; @@ -820,6 +821,9 @@ subtest 'article_requests() tests' => sub { $schema->storage->txn_begin; + my $library = $builder->build_object({ class => 'Koha::Libraries' }); + t::lib::Mocks::mock_userenv( { branchcode => $library->id } ); + my $patron = $builder->build_object( { class => 'Koha::Patrons' } ); my $article_requests = $patron->article_requests; @@ -846,3 +850,112 @@ subtest 'article_requests() tests' => sub { $schema->storage->txn_rollback; }; + +subtest 'article_request_fee() tests' => sub { + + plan tests => 3; + + $schema->storage->txn_begin; + + # Cleanup, to avoid interference + Koha::CirculationRules->search( { rule_name => 'article_request_fee' } )->delete; + + t::lib::Mocks::mock_preference( 'ArticleRequests', 1 ); + + my $item = $builder->build_sample_item; + + my $library_1 = $builder->build_object( { class => 'Koha::Libraries' } ); + my $library_2 = $builder->build_object( { class => 'Koha::Libraries' } ); + my $patron = $builder->build_object( { class => 'Koha::Patrons' } ); + + # Rule that should never be picked, because the patron's categor is always picked + Koha::CirculationRules->set_rule( + { categorycode => undef, + branchcode => undef, + rule_name => 'article_request_fee', + rule_value => 1, + } + ); + + is( $patron->article_request_fee( { library_id => $library_2->id } ), 1, 'library_id used correctly' ); + + Koha::CirculationRules->set_rule( + { categorycode => $patron->categorycode, + branchcode => undef, + rule_name => 'article_request_fee', + rule_value => 2, + } + ); + + Koha::CirculationRules->set_rule( + { categorycode => $patron->categorycode, + branchcode => $library_1->id, + rule_name => 'article_request_fee', + rule_value => 3, + } + ); + + is( $patron->article_request_fee( { library_id => $library_2->id } ), 2, 'library_id used correctly' ); + + t::lib::Mocks::mock_userenv( { branchcode => $library_1->id } ); + + is( $patron->article_request_fee(), 3, 'env used correctly' ); + + $schema->storage->txn_rollback; +}; + +subtest 'add_article_request_fee_if_needed() tests' => sub { + + plan tests => 12; + + $schema->storage->txn_begin; + + my $amount = 0; + + my $patron_mock = Test::MockModule->new('Koha::Patron'); + $patron_mock->mock( 'article_request_fee', sub { return $amount; } ); + + my $patron = $builder->build_object( { class => 'Koha::Patrons' } ); + + is( $patron->article_request_fee, $amount, 'article_request_fee mocked' ); + + my $library_1 = $builder->build_object( { class => 'Koha::Libraries' } ); + my $library_2 = $builder->build_object( { class => 'Koha::Libraries' } ); + my $staff = $builder->build_object( { class => 'Koha::Patrons' } ); + my $item = $builder->build_sample_item; + + t::lib::Mocks::mock_userenv( + { branchcode => $library_1->id, patron => $staff } ); + + my $debit = $patron->add_article_request_fee_if_needed(); + is( $debit, undef, 'No fee, no debit line' ); + + # positive value + $amount = 1; + + $debit = $patron->add_article_request_fee_if_needed({ item_id => $item->id }); + is( ref($debit), 'Koha::Account::Line', 'Debit object type correct' ); + is( $debit->amount, $amount, + 'amount set to $patron->article_request_fee value' ); + is( $debit->manager_id, $staff->id, + 'manager_id set to userenv session user' ); + is( $debit->branchcode, $library_1->id, + 'branchcode set to userenv session library' ); + is( $debit->debit_type_code, 'ARTICLE_REQUEST_FEE', + 'debit_type_code set correctly' ); + is( $debit->itemnumber, $item->id, + 'itemnumber set correctly' ); + + $amount = 100; + + $debit = $patron->add_article_request_fee_if_needed({ library_id => $library_2->id }); + is( ref($debit), 'Koha::Account::Line', 'Debit object type correct' ); + is( $debit->amount, $amount, + 'amount set to $patron->article_request_fee value' ); + is( $debit->branchcode, $library_2->id, + 'branchcode set to userenv session library' ); + is( $debit->itemnumber, undef, + 'itemnumber set correctly to undef' ); + + $schema->storage->txn_rollback; +}; -- 2.32.0