some cleanup #840

winfriedgerlach · 2025-07-07T15:21:09Z

fixed lots of small but minor issues, hope you find this helpful!

winfriedgerlach · 2025-07-07T15:22:14Z

.gitignore

@@ -124,7 +124,6 @@ local.properties
 .factorypath
 .project
 .settings
-.springBeans


duplicate of line 121

robogary · 2025-07-10T11:12:24Z

This Pull Request has failed the formatting check

Please run mvn spotless:apply or mvn clean install -DskipTests to fix the formatting issues.

You can automate this auto-formatting process to execute on the git pre-push hook, by installing pre-commit and then calling pre-commit install --hook-type pre-push. This will cause formatting to run automatically whenever you push.

jkiddo · 2025-07-10T11:14:37Z

@winfriedgerlach - please run mvn spotless:apply to fix the formatting violations.

jkiddo · 2025-07-10T11:15:22Z

Besides that, LGTM

winfriedgerlach · 2025-07-11T13:22:56Z

hey @jkiddo , sorry, fixed now! I ran mvn spotless:apply and pushed the changes

robogary · 2025-07-11T13:36:10Z

Formatting check succeeded!

jkiddo · 2025-08-01T06:52:53Z

Looks good to me - I'd like a second set of 👀 on it as well

Copilot

Pull Request Overview

This PR focuses on cleaning up code quality issues across the codebase by standardizing syntax, fixing spelling errors, and applying modern Java patterns. The changes improve maintainability and consistency without altering functionality.

Updated variable declarations from var to let in JavaScript and deprecated Java patterns to modern alternatives
Corrected numerous spelling and grammatical errors in comments, documentation, and configuration files
Applied modern Java language features like pattern matching with instanceof and text blocks

Reviewed Changes

Copilot reviewed 32 out of 33 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
src/test/smoketest/plain_server.http	Updated variable declarations from `var` to `let`
src/test/smoketest/SMOKE_TEST.md	Fixed spelling errors in documentation
src/test/resources/application.yaml	Corrected comment syntax and spelling errors
src/test/java/some/custom/pkg1/CustomInterceptorPojo.java	Applied pattern matching with instanceof
src/test/java/some/custom/pkg1/CustomInterceptorBean.java	Applied pattern matching with instanceof
src/test/java/ca/uhn/fhir/jpa/starter/SocketImplementation.java	Removed printStackTrace call and improved logging
src/test/java/ca/uhn/fhir/jpa/starter/ParallelUpdatesVersionConflictTest.java	Modernized lambda expressions and removed unused imports
src/test/java/ca/uhn/fhir/jpa/starter/MeasureEvaluationConfig.java	Removed unused imports
src/test/java/ca/uhn/fhir/jpa/starter/IServerSupport.java	Replaced deprecated Charsets with StandardCharsets
src/test/java/ca/uhn/fhir/jpa/starter/ExampleServerR4IT.java	Replaced string concatenation with text blocks and improved assertions
src/test/java/ca/uhn/fhir/jpa/starter/ExampleServerR4BIT.java	Replaced string concatenation with text blocks
src/test/java/ca/uhn/fhir/jpa/starter/ExampleServerDstu3IT.java	Removed unnecessary blank lines
src/test/java/ca/uhn/fhir/jpa/starter/ExampleServerDbpmR5IT.java	Changed class visibility modifier
src/test/java/ca/uhn/fhir/jpa/starter/ElasticsearchLastNR4IT.java	Removed unused imports and changed class visibility
src/test/java/ca/uhn/fhir/jpa/starter/CdsHooksServletIT.java	Replaced string concatenation with text blocks and fixed spelling
src/main/webapp/WEB-INF/templates/about.html	Updated variable declarations and fixed punctuation
src/main/resources/cds.application.yaml	Fixed spelling errors and comment syntax
src/main/resources/application.yaml	Fixed spelling errors and comment syntax
src/main/java/ca/uhn/fhir/jpa/starter/util/EnvironmentHelper.java	Applied pattern matching with instanceof
src/main/java/ca/uhn/fhir/jpa/starter/cr/PostInitProviderRegisterer.java	Made inner class static
src/main/java/ca/uhn/fhir/jpa/starter/common/validation/RepositoryValidationInterceptorFactoryR5.java	Fixed variable naming consistency
src/main/java/ca/uhn/fhir/jpa/starter/common/validation/RepositoryValidationInterceptorFactoryR4B.java	Fixed variable naming consistency
src/main/java/ca/uhn/fhir/jpa/starter/common/validation/RepositoryValidationInterceptorFactoryR4.java	Fixed variable naming consistency
src/main/java/ca/uhn/fhir/jpa/starter/common/StarterJpaConfig.java	Improved logging and simplified Optional usage
src/main/java/ca/uhn/fhir/jpa/starter/common/FhirServerConfigCommon.java	Modernized logging with parameterized messages
src/main/java/ca/uhn/fhir/jpa/starter/cdshooks/ModuleConfigurationPrefetchSvc.java	Simplified collection size check
src/main/java/ca/uhn/fhir/jpa/starter/cdshooks/CdsHooksServlet.java	Added @serial annotation
src/main/java/ca/uhn/fhir/jpa/starter/Application.java	Fixed spelling in comment
pom.xml	Fixed comment punctuation
charts/hapi-fhir-jpaserver/README.md.gotmpl	Fixed spelling error
charts/hapi-fhir-jpaserver/README.md	Fixed table formatting and spelling error
README.md	Fixed spelling errors throughout documentation

Copilot · 2025-08-01T06:54:02Z

src/test/resources/application.yaml

@@ -31,7 +31,7 @@ spring:
    allow-bean-definition-overriding: true
  flyway:
    enabled: false
-    check-location: false
+    fail-on-missing-locations: false


The property name change from check-location to fail-on-missing-locations appears to be changing the Spring Flyway configuration property. Verify that this property name is correct for the version of Spring Boot being used, as this could cause configuration to be ignored.

Suggested change

fail-on-missing-locations: false

check-location: false

AFAIU this is the non-deprecated replacement of "check-location"

Copilot · 2025-08-01T06:54:03Z

src/main/resources/cds.application.yaml

+    # or will be instantiated via reflection using a no-arg contructor; then registered with the server
    #custom-interceptor-classes:  

    # comma-separated list of fully qualified provider classes. 
    # classes listed here will be fetched from the Spring context when combined with 'custom-bean-packages', 
-    # or will be instantiated via reflection using an no-arg contructor; then registered with the server  
+    # or will be instantiated via reflection using a no-arg contructor; then registered with the server


The word 'contructor' should be spelled 'constructor'.

Copilot · 2025-08-01T06:54:03Z

src/main/resources/cds.application.yaml

    #custom-interceptor-classes:  

    # comma-separated list of fully qualified provider classes. 
    # classes listed here will be fetched from the Spring context when combined with 'custom-bean-packages', 
-    # or will be instantiated via reflection using an no-arg contructor; then registered with the server  
+    # or will be instantiated via reflection using a no-arg contructor; then registered with the server


The word 'contructor' should be spelled 'constructor'.

Copilot · 2025-08-01T06:54:04Z

src/main/resources/application.yaml

@@ -269,12 +269,12 @@ hapi:

    # comma-separated list of fully qualified interceptor classes. 
    # classes listed here will be fetched from the Spring context when combined with 'custom-bean-packages', 
-    # or will be instantiated via reflection using an no-arg contructor; then registered with the server  
+    # or will be instantiated via reflection using a no-arg contructor; then registered with the server


The word 'contructor' should be spelled 'constructor'.

Suggested change

# or will be instantiated via reflection using a no-arg contructor; then registered with the server

# or will be instantiated via reflection using a no-arg constructor; then registered with the server

Copilot · 2025-08-01T06:54:04Z

src/main/resources/application.yaml

    #custom-interceptor-classes:  

    # comma-separated list of fully qualified provider classes. 
    # classes listed here will be fetched from the Spring context when combined with 'custom-bean-packages', 
-    # or will be instantiated via reflection using an no-arg contructor; then registered with the server  
+    # or will be instantiated via reflection using a no-arg contructor; then registered with the server


The word 'contructor' should be spelled 'constructor'.

winfriedgerlach · 2025-08-08T08:44:55Z

@jkiddo @patrick-werner fixed spelling mistakes found by CoPilot

robogary · 2025-08-10T16:54:41Z

Formatting check succeeded!

some cleanup

2d833fc

winfriedgerlach commented Jul 7, 2025

View reviewed changes

.gitignore

@@ -124,7 +124,6 @@ local.properties

.factorypath

.project

.settings

.springBeans

Copy link

Author

winfriedgerlach Jul 7, 2025

Choose a reason for hiding this comment

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

duplicate of line 121

results from mvn spotless:apply

db4d427

jkiddo requested review from patrick-werner and Copilot August 1, 2025 06:52

Copilot AI reviewed Aug 1, 2025

View reviewed changes

fix contructor --> constructor

81d0512

	# or will be instantiated via reflection using a no-arg contructor; then registered with the server
	# or will be instantiated via reflection using a no-arg constructor; then registered with the server

some cleanup #840

Are you sure you want to change the base?

some cleanup #840

Uh oh!

Conversation

winfriedgerlach commented Jul 7, 2025

Uh oh!

winfriedgerlach Jul 7, 2025

Choose a reason for hiding this comment

Uh oh!

robogary commented Jul 10, 2025

Uh oh!

jkiddo commented Jul 10, 2025

Uh oh!

jkiddo commented Jul 10, 2025

Uh oh!

winfriedgerlach commented Jul 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

robogary commented Jul 11, 2025

Uh oh!

jkiddo commented Aug 1, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Aug 1, 2025

Choose a reason for hiding this comment

Uh oh!

winfriedgerlach Aug 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 1, 2025

Choose a reason for hiding this comment

Uh oh!

winfriedgerlach Aug 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 1, 2025

Choose a reason for hiding this comment

Uh oh!

winfriedgerlach Aug 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 1, 2025

Choose a reason for hiding this comment

Uh oh!

winfriedgerlach Aug 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 1, 2025

Choose a reason for hiding this comment

Uh oh!

winfriedgerlach Aug 8, 2025

Choose a reason for hiding this comment

Uh oh!

winfriedgerlach commented Aug 8, 2025

Uh oh!

robogary commented Aug 10, 2025

Uh oh!

Uh oh!

winfriedgerlach commented Jul 11, 2025 •

edited

Loading