Skip to content

Resolve OpenKM download failure and ECG controller access issue #145

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.

Sign up for GitHub

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

Merged
vanitha1822 merged 4 commits into from
Apr 7, 2026

Remove authorization check from getKMFile method

00f7c29
Select commit
Loading
Failed to load commit list.
Merged

Resolve OpenKM download failure and ECG controller access issue #145

Remove authorization check from getKMFile method
00f7c29
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Apr 7, 2026 in 40s

Quality Gate passed

Issues
9 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
2.8% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 567 in src/main/java/com/iemr/tm/service/common/transaction/CommonServiceImpl.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=PSMRI_TM-API&issues=AZ1nNTtvbHT1rcuWw2Ni&open=AZ1nNTtvbHT1rcuWw2Ni&pullRequest=145

Check warning on line 566 in src/main/java/com/iemr/tm/service/common/transaction/CommonServiceImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Invoke method(s) only conditionally. Use the built-in formatting to construct this argument. 

See more on https://sonarcloud.io/project/issues?id=PSMRI_TM-API&issues=AZ1nNTtvbHT1rcuWw2Nd&open=AZ1nNTtvbHT1rcuWw2Nd&pullRequest=145

Check warning on line 567 in src/main/java/com/iemr/tm/service/common/transaction/CommonServiceImpl.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=PSMRI_TM-API&issues=AZ1nNTtvbHT1rcuWw2Ne&open=AZ1nNTtvbHT1rcuWw2Ne&pullRequest=145

Check warning on line 576 in src/main/java/com/iemr/tm/service/common/transaction/CommonServiceImpl.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=PSMRI_TM-API&issues=AZ1nNTtvbHT1rcuWw2Nj&open=AZ1nNTtvbHT1rcuWw2Nj&pullRequest=145

Check failure on line 563 in src/main/java/com/iemr/tm/service/common/transaction/CommonServiceImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "fileID" 3 times. 

See more on https://sonarcloud.io/project/issues?id=PSMRI_TM-API&issues=AZ1nNTtvbHT1rcuWw2Nf&open=AZ1nNTtvbHT1rcuWw2Nf&pullRequest=145

Check warning on line 566 in src/main/java/com/iemr/tm/service/common/transaction/CommonServiceImpl.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=PSMRI_TM-API&issues=AZ1nNTtvbHT1rcuWw2Nh&open=AZ1nNTtvbHT1rcuWw2Nh&pullRequest=145

Check failure on line 557 in src/main/java/com/iemr/tm/service/common/transaction/CommonServiceImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 23 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=PSMRI_TM-API&issues=AZ1nNTtvbHT1rcuWw2Ng&open=AZ1nNTtvbHT1rcuWw2Ng&pullRequest=145

Check warning on line 71 in pom.xml

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this commented out code. 

See more on https://sonarcloud.io/project/issues?id=PSMRI_TM-API&issues=AZ1nNTxybHT1rcuWw2Nl&open=AZ1nNTxybHT1rcuWw2Nl&pullRequest=145

Check warning on line 583 in src/main/java/com/iemr/tm/service/common/transaction/CommonServiceImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof JSONObject dataObj' 

See more on https://sonarcloud.io/project/issues?id=PSMRI_TM-API&issues=AZ1nNTtvbHT1rcuWw2Nk&open=AZ1nNTtvbHT1rcuWw2Nk&pullRequest=145
View more details on SonarQubeCloud