Skip to content

Commit 68d1aeb

Browse files
fil512Ken Stevensclaude
authored
ValueSet code case sensitivity (#7395)
* fix with test * Add changelog entry for ValueSet case-sensitive code fix 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]> * Fix ValueSet expansion test to properly validate case-sensitive code matching Updated testExpandValueSet_CaseSensitiveCodeSystem_CodesWithDifferentCase to use FRAGMENT content mode and include entire system instead of enumerating specific concepts. This ensures the test exercises the Hibernate Search code path where the case-sensitivity bug manifests, providing proper TDD validation. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]> * fix with test * fix with test * Fix ValueSet expansion for case-sensitive CodeSystems with FRAGMENT content When a user persists a CodeSystem that overrides a built-in HL7 CodeSystem, the JPA-persisted version should take precedence over the built-in default. This is critical for case-sensitive CodeSystems where codes differ only by case (e.g., "drug" vs "Drug"). The fix: - Add removeCodeSystem/removeValueSet methods to DefaultProfileValidationSupport to selectively remove URLs from the in-memory cache - At startup, scan JPA for persisted CodeSystem/ValueSet URLs and remove them from the default profile cache so JPA versions take precedence - When a CodeSystem or ValueSet is created/updated, remove its URL from the default profile cache and invalidate validation caches - Fix lazy initialization issue where removal was silently ignored if the terminology maps hadn't been initialized yet - Add null checks for delete operations where the resource may be null 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]> * fix test * back out first change * back out first change * add fix discussed with James * Add changelog entry for database validation override setting Adds changelog documenting the new JpaStorageSettings.setAllowDatabaseValidationOverride() setting which allows database-stored CodeSystem and ValueSet resources to take precedence over built-in default profile validation resources. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]> * fix version * Add documentation for allowDatabaseValidationOverride setting Documents the new JpaStorageSettings.setAllowDatabaseValidationOverride(boolean) setting in the validation support modules documentation, explaining how it can be used to give database-stored terminology resources precedence over built-in HL7 definitions during validation. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]> * doc * spotless and review feedback * version bump --------- Co-authored-by: Ken Stevens <[email protected]> Co-authored-by: Claude <[email protected]>
1 parent 68a150c commit 68d1aeb

File tree

85 files changed

+294
-83
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

85 files changed

+294
-83
lines changed

‎hapi-deployable-pom/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<parent>
66
<groupId>ca.uhn.hapi.fhir</groupId>
77
<artifactId>hapi-fhir</artifactId>
8-
<version>8.7.0-SNAPSHOT</version>
8+
<version>8.7.1-SNAPSHOT</version>
99

1010
<relativePath>../pom.xml</relativePath>
1111
</parent>

‎hapi-fhir-android/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<parent>
66
<groupId>ca.uhn.hapi.fhir</groupId>
77
<artifactId>hapi-deployable-pom</artifactId>
8-
<version>8.7.0-SNAPSHOT</version>
8+
<version>8.7.1-SNAPSHOT</version>
99

1010
<relativePath>../hapi-deployable-pom/pom.xml</relativePath>
1111
</parent>

‎hapi-fhir-base/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<parent>
66
<groupId>ca.uhn.hapi.fhir</groupId>
77
<artifactId>hapi-deployable-pom</artifactId>
8-
<version>8.7.0-SNAPSHOT</version>
8+
<version>8.7.1-SNAPSHOT</version>
99

1010
<relativePath>../hapi-deployable-pom/pom.xml</relativePath>
1111
</parent>

‎hapi-fhir-bom/pom.xml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,15 @@
44
<modelVersion>4.0.0</modelVersion>
55
<groupId>ca.uhn.hapi.fhir</groupId>
66
<artifactId>hapi-fhir-bom</artifactId>
7-
<version>8.7.0-SNAPSHOT</version>
7+
<version>8.7.1-SNAPSHOT</version>
88

99
<packaging>pom</packaging>
1010
<name>HAPI FHIR BOM</name>
1111

1212
<parent>
1313
<groupId>ca.uhn.hapi.fhir</groupId>
1414
<artifactId>hapi-deployable-pom</artifactId>
15-
<version>8.7.0-SNAPSHOT</version>
15+
<version>8.7.1-SNAPSHOT</version>
1616

1717
<relativePath>../hapi-deployable-pom/pom.xml</relativePath>
1818
</parent>

‎hapi-fhir-checkstyle/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<parent>
66
<groupId>ca.uhn.hapi.fhir</groupId>
77
<artifactId>hapi-fhir</artifactId>
8-
<version>8.7.0-SNAPSHOT</version>
8+
<version>8.7.1-SNAPSHOT</version>
99

1010
<relativePath>../pom.xml</relativePath>
1111
</parent>

‎hapi-fhir-cli/hapi-fhir-cli-api/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<parent>
55
<groupId>ca.uhn.hapi.fhir</groupId>
66
<artifactId>hapi-deployable-pom</artifactId>
7-
<version>8.7.0-SNAPSHOT</version>
7+
<version>8.7.1-SNAPSHOT</version>
88

99
<relativePath>../../hapi-deployable-pom/pom.xml</relativePath>
1010
</parent>

‎hapi-fhir-cli/hapi-fhir-cli-app/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<parent>
77
<groupId>ca.uhn.hapi.fhir</groupId>
88
<artifactId>hapi-fhir-cli</artifactId>
9-
<version>8.7.0-SNAPSHOT</version>
9+
<version>8.7.1-SNAPSHOT</version>
1010

1111
<relativePath>../pom.xml</relativePath>
1212
</parent>

‎hapi-fhir-cli/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<parent>
66
<groupId>ca.uhn.hapi.fhir</groupId>
77
<artifactId>hapi-fhir</artifactId>
8-
<version>8.7.0-SNAPSHOT</version>
8+
<version>8.7.1-SNAPSHOT</version>
99

1010
<relativePath>../pom.xml</relativePath>
1111
</parent>

‎hapi-fhir-client-apache-http5/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<parent>
55
<groupId>ca.uhn.hapi.fhir</groupId>
66
<artifactId>hapi-deployable-pom</artifactId>
7-
<version>8.7.0-SNAPSHOT</version>
7+
<version>8.7.1-SNAPSHOT</version>
88

99
<relativePath>../hapi-deployable-pom/pom.xml</relativePath>
1010
</parent>

‎hapi-fhir-client-okhttp/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<parent>
55
<groupId>ca.uhn.hapi.fhir</groupId>
66
<artifactId>hapi-deployable-pom</artifactId>
7-
<version>8.7.0-SNAPSHOT</version>
7+
<version>8.7.1-SNAPSHOT</version>
88

99
<relativePath>../hapi-deployable-pom/pom.xml</relativePath>
1010
</parent>

0 commit comments

Comments
 (0)