Skip to content

fixing NPE#1274

Merged
grahamegrieve merged 1 commit intoHL7:masterfrom
jkiddo:fix/remove-npe
Mar 25, 2026
Merged

fixing NPE#1274
grahamegrieve merged 1 commit intoHL7:masterfrom
jkiddo:fix/remove-npe

Conversation

@jkiddo
Copy link
Copy Markdown
Contributor

@jkiddo jkiddo commented Mar 16, 2026

This pull request updates how R4 and R4B exclusions are handled during the IG loading process. Instead of immediately marking exclusions as they are parsed, the exclusions are now collected and processed together later in the loading sequence. This change improves code clarity and separation of concerns.

Refactoring of exclusion handling:

  • Added r4Exclusions and r4bExclusions lists to the PublisherFields class to collect exclusion values during parsing, instead of directly marking them.
  • Updated the IG loader to add exclusion values to these new lists instead of calling markExempt immediately.

Processing exclusions during loading:

  • Modified the loading process to iterate through the collected exclusions and call markExempt for each, ensuring all exclusions are processed together at the appropriate stage.

jkiddo added a commit to jkiddo/fhir-subscription-backport-ig that referenced this pull request Mar 16, 2026
Copy link
Copy Markdown
Contributor

@grahamegrieve grahamegrieve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@grahamegrieve grahamegrieve merged commit 572d982 into HL7:master Mar 25, 2026
13 of 14 checks passed
@github-project-automation github-project-automation Bot moved this from Inbox to Done in IG Publisher issue board Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants