https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946

--- Comment #21 from Tomás Cohen Arazi <[email protected]> ---
(In reply to Marcel de Rooy from comment #20)
> Martin: Are you happy with the way this patch deals with the cancellation as
> to the accounts handling? Testing amountoutstanding and choosing
> line->cancel or account->add_credit? Balance wise, it works fine.

I think ->cancel is correct but cancelling has some caveats: some exceptions
will be thrown if (say) the line is already offsetted (partially/fully payed,
writen off, etc).

Those cases lead me to think ->void would suit better, but we need to (maybe)
think about possible refunds.

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