Summary: | Remove use of recordpayment in ProcessOfflinePayment | ||
---|---|---|---|
Product: | Koha | Reporter: | Kyle M Hall <kyle> |
Component: | Architecture, internals, and plumbing | Assignee: | Kyle M Hall <kyle> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | bgkriegel, jonathan.druart, veron |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Bug Depends on: | 15895 | ||
Bug Blocks: | 15899 | ||
Attachments: |
Bug 15900 - Remove use of recordpayment in ProcessOfflinePayment
Bug 15900 - Remove use of recordpayment in ProcessOfflinePayment Bug 15900 - Remove use of recordpayment in ProcessOfflinePayment Bug 15900 [QA Followup] - Use Koha::Patrons instead of Koha::Borrowers Bug 15900 - Remove use of recordpayment in ProcessOfflinePayment Bug 15900 [QA Followup] - Use Koha::Patrons instead of Koha::Borrowers |
Description
Kyle M Hall
2016-02-24 15:27:36 UTC
Created attachment 48337 [details] [review] Bug 15900 - Remove use of recordpayment in ProcessOfflinePayment Test plan: 1) Apply this patch 2) prove t/db_dependent/Circulation.t On top of 15895, test fails prove t/db_dependent/Circulation.t t/db_dependent/Circulation.t .. 1/83 Can't locate object method "find" via package "Koha::Borrowers" (perhaps you forgot to load "Koha::Borrowers"?) at /home/bgkriegel/kohaclone/C4/Circulation.pm line 3912. Am I missing something? Created attachment 52142 [details] [review] Bug 15900 - Remove use of recordpayment in ProcessOfflinePayment Test plan: 1) Apply this patch 2) prove t/db_dependent/Circulation.t Created attachment 54742 [details] [review] Bug 15900 - Remove use of recordpayment in ProcessOfflinePayment Test plan: 1) Apply this patch 2) prove t/db_dependent/Circulation.t Signed-off-by: Josef Moravec <josef.moravec@gmail.com> (In reply to Bernardo Gonzalez Kriegel from comment #2) > On top of 15895, test fails > > prove t/db_dependent/Circulation.t > t/db_dependent/Circulation.t .. 1/83 Can't locate object method "find" via > package "Koha::Borrowers" (perhaps you forgot to load "Koha::Borrowers"?) at > /home/bgkriegel/kohaclone/C4/Circulation.pm line 3912. > > Am I missing something? Same here. Created attachment 55261 [details] [review] Bug 15900 [QA Followup] - Use Koha::Patrons instead of Koha::Borrowers Since the initial submission of this patch, the module Koha::Borrowers has been moved to Koha::Patrons. This patch changes the call from Koha::Borrowers to Koha::Patrons. Created attachment 55872 [details] [review] Bug 15900 - Remove use of recordpayment in ProcessOfflinePayment Test plan: 1) Apply this patch 2) prove t/db_dependent/Circulation.t Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 55873 [details] [review] Bug 15900 [QA Followup] - Use Koha::Patrons instead of Koha::Borrowers Since the initial submission of this patch, the module Koha::Borrowers has been moved to Koha::Patrons. This patch changes the call from Koha::Borrowers to Koha::Patrons. Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Pushed to master for 16.11! |