From: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Date: Fri, 25 Jan 2013 19:52:28 +0000 (-0500)
Subject: Revert "EDI: accept all misc. invoice charge values"
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=3842d34beaefd51c77d75314bb7dfba4e3472aae;p=working%2FEvergreen.git

Revert "EDI: accept all misc. invoice charge values"

Bummer, this change was *too* broad, and was picking up extra MOA lines
that refer to altogether different amounts when trying to parse misc
charges.

This reverts commit 9753d02ed61b4e794b8499e6a9eb1f1ea763e37a.

But also in reverting, I'm removing the comment that suggested (wrongly)
such a move.
---

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Utils/EDIReader.pm b/Open-ILS/src/perlmods/lib/OpenILS/Utils/EDIReader.pm
index 25f59eeebf..5a28b8fc25 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Utils/EDIReader.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Utils/EDIReader.pm
@@ -55,7 +55,7 @@ my %edi_li_quant_fields = (
 
 my %edi_charge_fields = (
     charge_type   => qr/^ALC\+C\++([^\+]+)/,
-    charge_amount => qr/^MOA\+\d+:([^:]+)/
+    charge_amount => qr/^MOA\+(?:8|131|304):([^:]+)/
 );
 
 sub new {