Skip to content

Replaced conditional instanceof chain with polymorphism in OpenmrsLo…

e940cc5
Select commit
Loading
Failed to load commit list.
Open

TRUNK-5748: Refactor code smells in openmrs-core #5937

Replaced conditional instanceof chain with polymorphism in OpenmrsLo…
e940cc5
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 17, 2026 in 49s

Annotations

Check warning on line 1007 in api/src/main/java/org/openmrs/Concept.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Format specifiers should be used instead of string concatenation.

See more on https://sonarcloud.io/project/issues?id=openmrs_openmrs-core&issues=AZz0jgk1xyShGgqbSEDW&open=AZz0jgk1xyShGgqbSEDW&pullRequest=5937

Check warning on line 24 in api/src/main/java/org/openmrs/logging/OpenmrsLoggingUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unnecessary import: same package classes are always implicitly imported.

See more on https://sonarcloud.io/project/issues?id=openmrs_openmrs-core&issues=AZz6T5iSoxUOsGaROSig&open=AZz6T5iSoxUOsGaROSig&pullRequest=5937

Check warning on line 1007 in api/src/main/java/org/openmrs/Concept.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the built-in formatting to construct this argument.

See more on https://sonarcloud.io/project/issues?id=openmrs_openmrs-core&issues=AZz0jgk1xyShGgqbSEDV&open=AZz0jgk1xyShGgqbSEDV&pullRequest=5937

Check failure on line 24 in api/src/main/java/org/openmrs/logic/op/OperatorConstants.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename field "EQUALS" to prevent any misunderstanding/clash with method "equals" defined in superclass "java.lang.Object".

See more on https://sonarcloud.io/project/issues?id=openmrs_openmrs-core&issues=AZz0jgcMxyShGgqbSEDU&open=AZz0jgcMxyShGgqbSEDU&pullRequest=5937