Fix edge case for element missing in document

aaronhuggins / node-x12

ASC X12 parser, generator, query engine, and mapper; now with support for streams.

https://aaronhuggins.github.io/node-x12/

MIT License

49 stars 14 forks source link

Fix edge case for element missing in document #26

Closed ahuggins-nhs closed 2 years ago

ahuggins-nhs commented 3 years ago

Adds mode of either strict or loose to the query engine
Default mode is strict
Mode loose will return a result for a missing element, so long as the segment which should have been it's parent exists
Transaction map engine uses strict mode by default, but loose can be set

ahuggins-nhs commented 3 years ago

@Mohmedvaid may I include the EDI doc you provided as part of the test suite under the license for this library?

ahuggins-nhs commented 3 years ago

@Mohmedvaid Tests pass and cover your exact use case in #23. If you approve of including the EDI file, then I will merge this and release a new version.

Keep in mind that you'll need to pass a new flag of mode='loose' when calling either X12Transaction.toObject or constructing X12TransactionMap directly.