Comment 9 for bug 978095

Revision history for this message
Jason Etheridge (phasefx) wrote :

If I wanted to move forward with this, would it be sane to modify sub cancel_lineitem_detail in Acq/Order.pm, right before the final return, so that it examines the cancel_reason for all sibling LIDs, and if they're all cancelled (and cancelled with a reason where keep_debits = false?), then invoke sub cancel_lineitem (ignoring whether it succeeds or not) with the same cancel_reason being fed to cancel_lineitem_detail? And then maybe double-check whether the UI needs a tickle or not to refresh the lineitem state? Then any question on canceling holds could be punted to another bug if needed for fixing sub cancel_lineitem?