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

Kyle M Hall <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
             Status|Needs Signoff               |Failed QA

--- Comment #7 from Kyle M Hall <[email protected]> ---
a) _insert_accontlines() has a typo, it should be _insert_accountlines()

b) Can you please either rewrite chargelostitem() to use manualinvoice() for
creating the accountline, or change manualinvoice() to use
_insert_accontlines() as well?

c) You have chargelostitem() returning an accountline as a value, which can be
either the processing fee or the lost fee, but not both even though both may be
charged. This return value is not even used by any code. The simplest
resolution would be to delete the line.

Thanks!

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
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/

Reply via email to