EDIWriter (EDI Attributes) Incorrectly Tests Some Attributes

Bug #1930740 reported by Bill Erickson
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.8
Fix Released
Medium
Unassigned
3.9
Fix Released
Medium
Unassigned

Bug Description

Evergreen 3.7

The EDIWriter.pm code incorrectly references 3 attribute instances with the wrong hash key. The affected attributes are:

    LINEITEM_IDENT_VENDOR_NUMBER
    COPY_SPEC_CODES
    INCLUDE_EMPTY_LI_NOTE

As it stands, setting these values to true for an attribute set has no impact on the final EDI.

Patch en route.

Revision history for this message
Bill Erickson (berick) wrote :
tags: added: pullrequest
Changed in evergreen:
assignee: Bill Erickson (berick) → nobody
Changed in evergreen:
milestone: 3.7.1 → 3.7.2
Revision history for this message
Jessica Woolford (jwoolford) wrote :

We just started using EDI attributes and came across this issue. We tested the patch on our production system and it works as advertised.

I consent to signing off on this code with my name, Jessica Woolford, and email <email address hidden>.

Changed in evergreen:
status: New → Confirmed
tags: added: signedoff
removed: pullrequest
tags: added: acq-edi
tags: removed: edi
Changed in evergreen:
milestone: 3.7.2 → 3.7.3
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Thanks, Bill and Jessica! Pushed to 3.7 and above.

Changed in evergreen:
importance: Undecided → Medium
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
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.