PMD Results
The following document contains the results of PMD 7.17.0.
Violations By Priority
Priority 4
org/tailormap/api/controller/admin/SolrAdminController.java
Rule |
Violation |
Line |
UselessParentheses |
Useless parentheses around `null == searchIndex.getLastIndexed() || searchI...`. |
192 |
UselessParentheses |
Useless parentheses around `null != searchIndex.getSchedule() && null != se...`. |
194–195 |
UselessParentheses |
Useless parentheses around `createNewIndex ? "creation of a new" : "update of"`. |
219 |
org/tailormap/api/controller/admin/TaskAdminController.java
org/tailormap/api/drawing/DrawingService.java
Rule |
Violation |
Line |
UselessParentheses |
Useless parentheses around `authentication instanceof AnonymousAuthenticati...`. |
422 |
org/tailormap/api/persistence/GeoService.java
Rule |
Violation |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'org.tailormap.api.viewer.model': 'Service' is already in scope because it is imported in this file |
310 |
org/tailormap/api/persistence/User.java
org/tailormap/api/persistence/helper/ApplicationHelper.java
Rule |
Violation |
Line |
UselessParentheses |
Useless parentheses around `n.getChildrenIds() != null && n.getChildrenIds(...`. |
265–266 |
org/tailormap/api/repository/events/SearchIndexEventHandler.java
Rule |
Violation |
Line |
UselessParentheses |
Useless parentheses around `succes ? "succeeded" : "failed"`. |
71 |
org/tailormap/api/scheduling/IndexTask.java
Rule |
Violation |
Line |
UselessParentheses |
Useless parentheses around `1 + (int) context.getMergedJobDataMap().getOrDe...`. |
123 |
org/tailormap/api/scheduling/TaskType.java
org/tailormap/api/security/AuthorisationService.java
Rule |
Violation |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AuthorisationService': 'ACCESS_TYPE_VIEW' is already in scope because it is declared in an enclosing type |
80 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AuthorisationService': 'ACCESS_TYPE_VIEW' is already in scope because it is declared in an enclosing type |
85 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AuthorisationService': 'ACCESS_TYPE_VIEW' is already in scope because it is declared in an enclosing type |
95 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AuthorisationService': 'ACCESS_TYPE_VIEW' is already in scope because it is declared in an enclosing type |
104 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AuthorisationService': 'ACCESS_TYPE_VIEW' is already in scope because it is declared in an enclosing type |
109 |
UselessParentheses |
Useless parentheses around `allowed ? "allowed" : "denied"`. |
208 |
UselessParentheses |
Useless parentheses around `allowed ? "allowed" : "denied"`. |
216 |
org/tailormap/api/solr/SolrHelper.java
Files
org/tailormap/api/controller/admin/SolrAdminController.java
Rule |
Violation |
Priority |
Line |
UselessParentheses |
Useless parentheses around `null == searchIndex.getLastIndexed() || searchI...`. |
4 |
192 |
UselessParentheses |
Useless parentheses around `null != searchIndex.getSchedule() && null != se...`. |
4 |
194–195 |
UselessParentheses |
Useless parentheses around `createNewIndex ? "creation of a new" : "update of"`. |
4 |
219 |
org/tailormap/api/controller/admin/TaskAdminController.java
org/tailormap/api/drawing/DrawingService.java
Rule |
Violation |
Priority |
Line |
UselessParentheses |
Useless parentheses around `authentication instanceof AnonymousAuthenticati...`. |
4 |
422 |
org/tailormap/api/persistence/GeoService.java
Rule |
Violation |
Priority |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'org.tailormap.api.viewer.model': 'Service' is already in scope because it is imported in this file |
4 |
310 |
org/tailormap/api/persistence/User.java
org/tailormap/api/persistence/helper/ApplicationHelper.java
Rule |
Violation |
Priority |
Line |
UselessParentheses |
Useless parentheses around `n.getChildrenIds() != null && n.getChildrenIds(...`. |
4 |
265–266 |
org/tailormap/api/repository/events/SearchIndexEventHandler.java
Rule |
Violation |
Priority |
Line |
UselessParentheses |
Useless parentheses around `succes ? "succeeded" : "failed"`. |
4 |
71 |
org/tailormap/api/scheduling/IndexTask.java
Rule |
Violation |
Priority |
Line |
UselessParentheses |
Useless parentheses around `1 + (int) context.getMergedJobDataMap().getOrDe...`. |
4 |
123 |
org/tailormap/api/scheduling/TaskType.java
org/tailormap/api/security/AuthorisationService.java
Rule |
Violation |
Priority |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AuthorisationService': 'ACCESS_TYPE_VIEW' is already in scope because it is declared in an enclosing type |
4 |
80 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AuthorisationService': 'ACCESS_TYPE_VIEW' is already in scope because it is declared in an enclosing type |
4 |
85 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AuthorisationService': 'ACCESS_TYPE_VIEW' is already in scope because it is declared in an enclosing type |
4 |
95 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AuthorisationService': 'ACCESS_TYPE_VIEW' is already in scope because it is declared in an enclosing type |
4 |
104 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AuthorisationService': 'ACCESS_TYPE_VIEW' is already in scope because it is declared in an enclosing type |
4 |
109 |
UselessParentheses |
Useless parentheses around `allowed ? "allowed" : "denied"`. |
4 |
208 |
UselessParentheses |
Useless parentheses around `allowed ? "allowed" : "denied"`. |
4 |
216 |
org/tailormap/api/solr/SolrHelper.java
Suppressed Violations
Filename |
Rule message |
Suppression type |
Reason |
org/tailormap/api/configuration/dev/PopulateTestData.java |
Do not hard code the IP address ${variableName} |
@suppresswarnings |
|
org/tailormap/api/configuration/dev/PopulateTestData.java |
Do not hard code the IP address ${variableName} |
@suppresswarnings |
|
org/tailormap/api/configuration/dev/PopulateTestData.java |
Do not hard code the IP address solrUrl |
@suppresswarnings |
|
org/tailormap/api/configuration/dev/PopulateTestData.java |
Do not hard code the IP address ${variableName} |
@suppresswarnings |
|
org/tailormap/api/configuration/dev/PopulateTestData.java |
Do not hard code the IP address ${variableName} |
@suppresswarnings |
|
org/tailormap/api/controller/GeoServiceProxyController.java |
Ensure that resources like this HttpClient object are closed after use |
@suppresswarnings |
|
org/tailormap/api/geotools/wfs/WFSProxy.java |
Ensure that resources like this HttpClient object are closed after use |
@suppresswarnings |
|
org/tailormap/api/persistence/Group.java |
Avoid unused private methods such as 'removeMembers()'. |
@suppresswarnings |
|