Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MSH Version ID set to return type string when it can be an array. #110

Closed
securit opened this issue Nov 14, 2023 · 4 comments
Closed

MSH Version ID set to return type string when it can be an array. #110

securit opened this issue Nov 14, 2023 · 4 comments
Assignees

Comments

@securit
Copy link

securit commented Nov 14, 2023

At

public function getVersionId(int $position = 12): string
on #92 you have changed the getVersionId to have a return type of string, but in HL7v2.7 and later this field is an array.

https://hl7-definition.caristix.com/v2/HL7v2.7/DataTypes/VID
https://hl7-definition.caristix.com/v2/HL7v2.7.1/DataTypes/VID
https://hl7-definition.caristix.com/v2/HL7v2.8/DataTypes/VID

For maximum compatibility I suggest that you change this to:

public function getVersionId(int $position = 12): string|array
@senaranya senaranya self-assigned this Nov 15, 2023
@senaranya
Copy link
Owner

Thanks for raising this issue. Can you please share a sample MSH for me to use in tests?

@securit
Copy link
Author

securit commented Nov 15, 2023

This should do:

MSH|^~\&|Sending App|sendEDI|Receiving App|receiveEDI|20221213152007|PKI|REF^I12^REF_I12|GenericNZ2.4REF|P|2.4^NZL^1.0

@senaranya
Copy link
Owner

Do take a look - #111

@senaranya
Copy link
Owner

Resolved in #111

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants