Bugzilla – Attachment 84716 Details for
Bug 20912
Rental fees based on time period
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 20912: (QA follow-up) Move Fees to Charges::
Bug-20912-QA-follow-up-Move-Fees-to-Charges.patch (text/plain), 3.37 KB, created by
Tomás Cohen Arazi (tcohen)
on 2019-02-04 14:38:14 UTC
(
hide
)
Description:
Bug 20912: (QA follow-up) Move Fees to Charges::
Filename:
MIME Type:
Creator:
Tomás Cohen Arazi (tcohen)
Created:
2019-02-04 14:38:14 UTC
Size:
3.37 KB
patch
obsolete
>From 2eec61cc718821b62752b82873eec8b68d15d22b Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Tue, 29 Jan 2019 11:28:28 +0000 >Subject: [PATCH] Bug 20912: (QA follow-up) Move Fees to Charges:: > >In preparation for the introduction of Koha::Charges::Fines I have moved >this ::Fees class into the Koha::Charges:: namespace > >Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> >--- > C4/Circulation.pm | 8 ++++---- > Koha/{ => Charges}/Fees.pm | 6 +++--- > t/db_dependent/Koha/{ => Charges}/Fees.t | 4 ++-- > 3 files changed, 9 insertions(+), 9 deletions(-) > rename Koha/{ => Charges}/Fees.pm (97%) > rename t/db_dependent/Koha/{ => Charges}/Fees.t (97%) > >diff --git a/C4/Circulation.pm b/C4/Circulation.pm >index d1865411eb..6c7ae420ee 100644 >--- a/C4/Circulation.pm >+++ b/C4/Circulation.pm >@@ -59,7 +59,7 @@ use Koha::RefundLostItemFeeRules; > use Koha::Account::Lines; > use Koha::Account::Offsets; > use Koha::Config::SysPrefs; >-use Koha::Fees; >+use Koha::Charges::Fees; > use Carp; > use List::MoreUtils qw( uniq any ); > use Scalar::Util qw( looks_like_number ); >@@ -707,7 +707,7 @@ sub CanBookBeIssued { > # So issuingimpossible should be ok. > } > >- my $fees = Koha::Fees->new( >+ my $fees = Koha::Charges::Fees->new( > { > patron => $patron, > library => $library, >@@ -1337,7 +1337,7 @@ sub AddIssue { > > my $patron = Koha::Patrons->find( $borrower ); > my $library = Koha::Libraries->find( $branch ); >- my $fees = Koha::Fees->new( >+ my $fees = Koha::Charges::Fees->new( > { > patron => $patron, > library => $library, >@@ -2906,7 +2906,7 @@ sub AddRenewal { > } > > >- my $fees = Koha::Fees->new( >+ my $fees = Koha::Charges::Fees->new( > { > patron => $patron, > library => $library, >diff --git a/Koha/Fees.pm b/Koha/Charges/Fees.pm >similarity index 97% >rename from Koha/Fees.pm >rename to Koha/Charges/Fees.pm >index 33bf3aa64e..62f534147c 100644 >--- a/Koha/Fees.pm >+++ b/Koha/Charges/Fees.pm >@@ -1,4 +1,4 @@ >-package Koha::Fees; >+package Koha::Charges::Fees; > > # Copyright 2018 ByWater Solutions > # >@@ -27,11 +27,11 @@ use Koha::Exceptions; > > =head1 NAME > >-Koha::Feess - Module calculating fees in Koha >+Koha::Charges::Fees - Module calculating fees in Koha > > =head3 new > >-Koha::Fees->new( >+Koha::Charges::Fees->new( > { > patron => $patron, > library => $library, >diff --git a/t/db_dependent/Koha/Fees.t b/t/db_dependent/Koha/Charges/Fees.t >similarity index 97% >rename from t/db_dependent/Koha/Fees.t >rename to t/db_dependent/Koha/Charges/Fees.t >index db16a8236c..55c324216b 100644 >--- a/t/db_dependent/Koha/Fees.t >+++ b/t/db_dependent/Koha/Charges/Fees.t >@@ -30,7 +30,7 @@ use C4::Calendar; > use Koha::DateUtils qw(dt_from_string); > > BEGIN { >- use_ok('Koha::Fees'); >+ use_ok('Koha::Charges::Fees'); > } > > my $builder = t::lib::TestBuilder->new(); >@@ -89,7 +89,7 @@ my $patron = $builder->build_object( > my $dt_from = dt_from_string(); > my $dt_to = dt_from_string()->add( days => 6 ); > >-my $fees = Koha::Fees->new( >+my $fees = Koha::Charges::Fees->new( > { > patron => $patron, > library => $library, >-- >2.20.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 20912
:
76065
|
76066
|
76067
|
76068
|
76069
|
77302
|
77303
|
77304
|
81141
|
81142
|
81143
|
81450
|
81771
|
81772
|
81773
|
81774
|
81775
|
81778
|
82195
|
82196
|
84412
|
84413
|
84414
|
84415
|
84416
|
84417
|
84418
|
84419
|
84420
|
84570
|
84571
|
84572
|
84573
|
84574
|
84575
|
84576
|
84577
|
84578
|
84579
|
84580
|
84581
|
84582
|
84583
|
84584
|
84585
|
84610
|
84611
|
84612
|
84613
|
84614
|
84615
|
84616
|
84617
|
84618
|
84619
|
84620
|
84621
|
84622
|
84623
|
84624
|
84625
|
84707
|
84708
|
84709
|
84710
|
84711
|
84712
|
84713
|
84714
|
84715
|
84716
|
84717
|
84718
|
84719
|
84720
|
84721
|
84722
|
84723
|
84724
|
85291
|
85292
|
85293
|
85294
|
85295
|
85296
|
85297
|
85298
|
85299
|
85300
|
85301
|
85302
|
85303
|
85304
|
85305
|
85306
|
85307
|
85308
|
85316
|
85543
|
85544
|
85545
|
85546
|
85547
|
85548
|
85549
|
85550
|
85551
|
85552
|
85553
|
85554
|
85555
|
85556
|
85557
|
85558
|
85559
|
85560
|
85561
|
85628
|
85631
|
86140
|
86141
|
86142
|
86143
|
86144
|
86145
|
86146
|
86147
|
86148
|
86149
|
86151
|
86152
|
86153
|
86154
|
86155
|
86156
|
86157
|
86158
|
86159
|
86160
|
86244
|
86245
|
86246
|
86247
|
86248
|
86249
|
86250
|
86251
|
86252
|
86253
|
86254
|
86255
|
86256
|
86257
|
86258
|
86259
|
86260
|
86261
|
86262
|
86263
|
86264
|
86282
|
86283
|
86284
|
86285
|
86286
|
86287
|
86288
|
86289
|
86290
|
86291
|
86292
|
86293
|
86294
|
86295
|
86296
|
86297
|
86298
|
86299
|
86300
|
86301
|
86302
|
86306
|
86307
|
86308
|
86309
|
86310
|
86311
|
86312
|
86313
|
86314
|
86315
|
86316
|
86317
|
86318
|
86319
|
86320
|
86321
|
86322
|
86323
|
86324
|
86325
|
86326
|
86327
|
87363
|
87413
|
87703