edi_order_pusher.pl blindly pushes ancient orders

[Blake GH]

We ran into an issue turning on edi_order_pusher.pl. It sent hundreds of old PO's. The PO's were closed and finished. Namely, the state was "received" for all of them. But none of them had a row in acq.edi_message because they were either migrated PO's or didn't use EDI at that point in time.

I think this script needs to either:

1. Not collect PO's with state "received"
2. Include some sane value for order_date. Maybe configurable as a bash argument. Like '3 months' or default to '3 months' with an override from a bash argument.
3. Both 1 and 2.

Thoughts?

[Bill Erickson (berick)]

Another option (like #1) would be to only collect PO's whose state is 'on-order'. This has the added effect of avoiding "canceled" PO's.

[Tiffany Little (tslittle)]

+1 to Bill's suggestion of only sending PO's with an 'on-order' state. That seems like a logical argument to be included as part of the script.

[Terran McCanna (tmccanna)]

Bumped to Critical. This caused an old PO of ours to be re-sent that wasn't caught until after items arrived and invoices were received from the vendor.

[Jason Stephenson (jstephenson)]

I put up a quick branch to possibly fix this, based on Bill's comment #1.

user/dyrcona/lp1803734_push_only_on-order_orders in the working repo.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dyrcona/lp1803734_push_only_on-order_orders

[Jason Stephenson (jstephenson)]

I meant to add that I have NOT tested my branch, but it seemed like a no-brainer.

[Chris Sharp (chrissharp123)]

Been running this in PINES production since 1/10 with no errors. Pushed to rel_3_1, rel_3_2, and master.