Comment 7 for bug 1115599

Revision history for this message
Bill Erickson (berick) wrote :

Kathy, your patch looks good and solves the problem. There's one small nit that I've pushed a follow up fix for.

Adding "cancelled" to li_post_po_states causes the li_active_states variable to pick up the value (which was intended, for invoicing), but it means that canceled lineitems are now considered valid choices for batch cancellation, which could create minor usability problems (and useless double cancellation attempts).

Sign off and secondary patch pushed to:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1115599-acq-recv-canceled-li