EDI Invoices: try harder to find SAN, billed amounts, surcharges in the correct places

Bug #1052647 reported by Lebbeous Fogle-Weekley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
New
Undecided
Unassigned

Bug Description

Bugfix for Evergreen 2.3+:

    EDI invoices: Sometimes NAD/C082/3039 isn't buyer SAN, but EDI acct #

    Address an issue reported by Bill Erickson where a number that seems to
    be buyer SAN in some cases is actually the number from
    acq.edi_account.vendacct in others.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/senator/fix-edi-invoice-ou-lookup

Changed in evergreen:
milestone: 2.3.0 → 2.3.1
Revision history for this message
Bill Erickson (berick) wrote :

I've pushed a sign-off to Lebbeous' original commit, plus one additional related commit to

working => user/berick/acq-invoice-extract-san

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

Found one more issue w/ invoice data extraction. patch coming...

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

Another commit pushed to working => user/berick/acq-invoice-extract-san to make the EDI invoicing more discerning in LI billed amount extraction.

Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Interpretations of standard vary widely. Here's another small commit along the lines of "look harder for other places data can hide in an EDI invoice," in this case re surcharges, taxes.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/senator/acq-invoice-other-charges

So now this ticket refers to two branches,

user/berick/acq-invoice-extract-san (with 2 commits)
collab/senator/acq-invoice-other-charges (with 1)

summary: - EDI Invoices: try another axis for matching invoice to ordering agency
+ EDI Invoices: try harder to find SAN, billed amounts, surcharges in the
+ correct places
Revision history for this message
Bill Erickson (berick) wrote :

Works well. I've pushed a signed-off commit back to my original branch. We're back down to one branch with one un-signed commit.

user/berick/acq-invoice-extract-san

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

Changes in these branches have been included and in some cases superseded by code in bug #1065270. Removing pullrequest and marking as duplicate.

tags: removed: pullrequest
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.