When the Current Koha/EDI.pm was written the api to Acquisitions::ModReceiveOrder was unstable as a result the process was duplicated in Koha::EDI as the interface has stabilised EDI should revert to using the Acquisitions interface and remove the duplicate code. This should guard against bugs from incompatibilities and allow it to take advantage of enhancements in C4::acquisitions (e.g. new handling of Standing Orders)
Created attachment 57071 [details] [review]
This patch removes the code which duplicated functionality from C4::Acquisitions and instead calls ModReceiveOrder. Removes potential divergencies from the two paths. Koha::EDI now only required to load data provided by the GIR segment in the invoice.
This removes some anomalies which caused partial receipts to sometimes skip adding a barcode to some items (if supplier applies barcodes)
Looking at a small change to this patch to eliminate an unnecssary warning