[Koha-bugs] [Bug 15548] New patron related code should have been put to Patron instead of Borrower

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Jan 12 04:11:10 CET 2016


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15548

Héctor Eduardo Castro Avalos <hector.hecaxmmx at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hector.hecaxmmx at gmail.com

--- Comment #3 from Héctor Eduardo Castro Avalos <hector.hecaxmmx at gmail.com> ---
Hi Marc and Jonathan

For me all involved tests works successfully

t/Patron.t
t/db_dependent/Accounts.t
t/db_dependent/BiblioObject.t
t/db_dependent/Circulation/AnonymiseIssueHistory.t
t/db_dependent/Circulation/IssuingRules/maxsuspensiondays.t
t/db_dependent/Creators/Lib.t
t/db_dependent/Hold.t
t/db_dependent/Koha_borrower_modifications.t
t/db_dependent/Patron.t
t/db_dependent/Patron/Borrower_Debarments.t
t/db_dependent/Patron/Borrower_Discharge.t
t/db_dependent/Patron/Borrower_Files.t
t/db_dependent/Patrons.t
t/db_dependent/Review.t
t/db_dependent/api/v1/patrons.t

The next was renamed/removed
Renamed t/Borrower.t
Renamed t/db_dependent/Borrower.t
Renamed t/db_dependent/Borrowers.t

git grep Koha::Borrower showed entries in release notes for 3.18 only.

I will start to test the modules implied. It seems to be that you only replaced
"borrower" to "patron" but IMHO I think this needs a rigorous test plan since
you changed a lot in this bug.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list