|
Description
Kyle M Hall (khall)
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! |