https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15895
--- Comment #8 from Jonathan Druart <jonathan.dru...@bugs.koha-community.org> --- (In reply to Kyle M Hall from comment #3) > (In reply to Jonathan Druart from comment #2) > > Comment on attachment 48331 [details] [review] [review] > > Bug 15895 - Add Koha::Account module, use Koha::Account::pay internally for > > recordpayment > > > > Review of attachment 48331 [details] [review] [review]: > > ----------------------------------------------------------------- > > > > ::: t/db_dependent/Accounts.t > > @@ +158,5 @@ > > > > > > + my $line1 = Koha::Account::Line->new({ borrowernumber => > > > $borrower->borrowernumber, amountoutstanding => 100 })->store(); > > > + my $line2 = Koha::Account::Line->new({ borrowernumber => > > > $borrower->borrowernumber, amountoutstanding => 200 })->store(); > > > + $line1->_result->discard_changes; > > > + $line2->_result->discard_changes; > > > > Why do you need these 2 calls? > > The discard_changes call updates the objects from the database. Without > them, any default values inserted by MySQL are not available in the initial > object. Actually the 2 objects $line1 and $line2 are not reused later, I don't think they are useful. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/