Comment 4 for bug 1770202

Revision history for this message
Tiffany Little (tslittle) wrote :

It *looks* like mapping the EDI codes to cancel reasons is done in EDI.pm around line 782. That was the first place I looked, so it may be somewhere else too/instead.

My first thought was just to add in more codes to that portion of the Perl module, then realized that could easily get out of control. So I'm inclined to lean toward Chris's suggestion in comment #3 of bug #1627373 of storing the EDI availability codes in DB tables. I'd also say they should go ahead and be mapped to the cancel reasons in those tables as well. Then EDI.pm can just talk to that table.

And looking at the stock cancel reasons, in conjunction with this, we should probably expand the available stock cancel reasons. We have some in PINES that I thought were stock but appears they aren't, like Canceled: EBL which would map nicely to the 221 availability code.