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

Fixing bug that prevented access all instances of a repeating segment. Added a helper method on Segment to query if it's empty? #12

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

daya
Copy link

@daya daya commented Dec 2, 2021

Fixing bug which prevented accessing all instances of a repeating segment

@daya daya changed the title Fixing bug which prevented accessing all instances of a repeating seg… Fixing bug that prevented access all instances of a repeating segment. Added a helper method on Segment to query if it's empty? Dec 4, 2021
@daya
Copy link
Author

daya commented Dec 4, 2021

the latest commit with Segment#empty? method is very useful since SimpleHl7::Message#method_missing will always cause false positives for a segment that does not actually occur in a HL7 message.

daya added 2 commits December 4, 2021 15:18
…hen one is modifying the original Hl7 message for say anonymization and modify only those segments that exist while containing PII/PHI.
…segment does not exist in raw hl7 message or is empty
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

Successfully merging this pull request may close these issues.

1 participant