PMD Results
The following document contains the results of PMD 7.17.0.
Violations By Priority
Priority 3
de/dlr/shepard/context/export/ExportService.java
de/dlr/shepard/context/labJournal/entities/LabJournalEntry.java
de/dlr/shepard/data/timeseries/migrations/V1_7_0__int_to_bigint.java
| Rule |
Violation |
Line |
| CheckResultSet |
Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. |
27 |
Priority 4
de/dlr/shepard/auth/apikey/entities/ApiKey.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `HasId.equalsHelper(belongsTo, other.belongsTo) ...`. |
76–82 |
de/dlr/shepard/auth/permission/model/Permissions.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(id, other.id) && Objects.equals(...`. |
109–119 |
de/dlr/shepard/auth/users/endpoints/UserGroupAttributes.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'UserGroupAttributes': 'name' is already in scope because it is declared in an enclosing type |
11 |
de/dlr/shepard/auth/users/entities/User.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `HasId.areEqualSetsByUniqueId(apiKeys, other.api...`. |
82–89 |
de/dlr/shepard/auth/users/io/UserGroupIO.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `HasId.areEqualSets(usernames, other.usernames)`. |
33 |
de/dlr/shepard/common/neo4j/entities/AbstractEntity.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `id.equals(entity.id) && deleted == entity.delet...`. |
71–78 |
de/dlr/shepard/common/neo4j/entities/BasicEntity.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(id, other.id) && deleted == othe...`. |
60–69 |
de/dlr/shepard/common/neo4j/io/AbstractDataObjectIO.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(description, other.description) ...`. |
35 |
de/dlr/shepard/common/neo4j/io/BasicEntityIO.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(id, other.id) && Objects.equals(...`. |
95–102 |
de/dlr/shepard/common/neo4j/migrations/V2__Extract_json.java
de/dlr/shepard/common/search/daos/SearchDAO.java
de/dlr/shepard/common/search/endpoints/BasicCollectionAttributes.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicCollectionAttributes': 'name' is already in scope because it is declared in an enclosing type |
14 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicCollectionAttributes': 'createdBy' is already in scope because it is declared in an enclosing type |
15 |
de/dlr/shepard/common/search/endpoints/BasicContainerAttributes.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicContainerAttributes': 'name' is already in scope because it is declared in an enclosing type |
14 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicContainerAttributes': 'type' is already in scope because it is declared in an enclosing type |
15 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicContainerAttributes': 'createdBy' is already in scope because it is declared in an enclosing type |
16 |
de/dlr/shepard/common/search/query/Neo4jQueryBuilder.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `"((" + neighborhoodIdsIsContainedInPart(node, v...`. |
311–317 |
de/dlr/shepard/common/subscription/entities/Subscription.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(callbackURL, other.callbackURL) ...`. |
72–80 |
de/dlr/shepard/common/util/CypherQueryHelper.java
de/dlr/shepard/context/collection/endpoints/DataObjectAttributes.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'DataObjectAttributes': 'name' is already in scope because it is declared in an enclosing type |
11 |
de/dlr/shepard/context/collection/entities/Collection.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `HasId.areEqualSetsByUniqueId(dataObjects, other...`. |
72–78 |
de/dlr/shepard/context/collection/entities/DataObject.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `HasId.equalsHelper(collection, other.collection...`. |
99–108 |
de/dlr/shepard/context/collection/io/CollectionIO.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `HasId.areEqualSets(dataObjectIds, other.dataObj...`. |
48–52 |
de/dlr/shepard/context/collection/io/DataObjectIO.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `collectionId == other.collectionId && HasId.are...`. |
53–61 |
de/dlr/shepard/context/references/basicreference/endpoints/BasicReferenceAttributes.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicReferenceAttributes': 'name' is already in scope because it is declared in an enclosing type |
13 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicReferenceAttributes': 'type' is already in scope because it is declared in an enclosing type |
14 |
de/dlr/shepard/context/references/basicreference/io/BasicReferenceIO.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `dataObjectId == other.dataObjectId && Objects.e...`. |
34 |
de/dlr/shepard/context/references/dataobject/entities/CollectionReference.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(relationship, other.relationship...`. |
49–52 |
de/dlr/shepard/context/references/dataobject/entities/DataObjectReference.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(relationship, other.relationship...`. |
49–52 |
de/dlr/shepard/context/references/file/io/FileReferenceIO.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `fileContainerId == other.fileContainerId && Has...`. |
39 |
de/dlr/shepard/context/references/structureddata/entities/StructuredDataReference.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `HasId.equalsHelper(structuredDataContainer...) ...`. |
57–60 |
de/dlr/shepard/context/references/structureddata/io/StructuredDataReferenceIO.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `structuredDataContainerId == other.structuredDa...`. |
41–44 |
de/dlr/shepard/context/references/timeseriesreference/model/TimeseriesReference.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `end == other.end && start == other.start && Obj...`. |
60–65 |
de/dlr/shepard/context/references/timeseriesreference/services/TimeseriesReferenceMetricsService.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `metric != AggregateFunction.COUNT && metric != ...`. |
129 |
de/dlr/shepard/context/semantic/endpoints/SemanticRepositoryAttributes.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SemanticRepositoryAttributes': 'name' is already in scope because it is declared in an enclosing type |
11 |
de/dlr/shepard/context/semantic/entities/SemanticAnnotation.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(id, other.id) && Objects.equals(...`. |
68–76 |
de/dlr/shepard/context/version/entities/Version.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(createdAt, other.createdAt) && O...`. |
101–109 |
de/dlr/shepard/data/ContainerAttributes.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'ContainerAttributes': 'name' is already in scope because it is declared in an enclosing type |
11 |
de/dlr/shepard/data/timeseries/migrations/V1_7_0__int_to_bigint.java
| Rule |
Violation |
Line |
| UnnecessaryImport |
Unused import 'org.flywaydb.core.api.MigrationVersion' |
7 |
de/dlr/shepard/data/timeseries/repositories/TimeseriesDataPointRepository.java
de/dlr/shepard/data/timeseries/services/TimeseriesService.java
Files
de/dlr/shepard/auth/apikey/entities/ApiKey.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `HasId.equalsHelper(belongsTo, other.belongsTo) ...`. |
4 |
76–82 |
de/dlr/shepard/auth/permission/model/Permissions.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(id, other.id) && Objects.equals(...`. |
4 |
109–119 |
de/dlr/shepard/auth/users/endpoints/UserGroupAttributes.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'UserGroupAttributes': 'name' is already in scope because it is declared in an enclosing type |
4 |
11 |
de/dlr/shepard/auth/users/entities/User.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `HasId.areEqualSetsByUniqueId(apiKeys, other.api...`. |
4 |
82–89 |
de/dlr/shepard/auth/users/io/UserGroupIO.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `HasId.areEqualSets(usernames, other.usernames)`. |
4 |
33 |
de/dlr/shepard/common/neo4j/entities/AbstractEntity.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `id.equals(entity.id) && deleted == entity.delet...`. |
4 |
71–78 |
de/dlr/shepard/common/neo4j/entities/BasicEntity.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(id, other.id) && deleted == othe...`. |
4 |
60–69 |
de/dlr/shepard/common/neo4j/io/AbstractDataObjectIO.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(description, other.description) ...`. |
4 |
35 |
de/dlr/shepard/common/neo4j/io/BasicEntityIO.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(id, other.id) && Objects.equals(...`. |
4 |
95–102 |
de/dlr/shepard/common/neo4j/migrations/V2__Extract_json.java
de/dlr/shepard/common/search/daos/SearchDAO.java
de/dlr/shepard/common/search/endpoints/BasicCollectionAttributes.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicCollectionAttributes': 'name' is already in scope because it is declared in an enclosing type |
4 |
14 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicCollectionAttributes': 'createdBy' is already in scope because it is declared in an enclosing type |
4 |
15 |
de/dlr/shepard/common/search/endpoints/BasicContainerAttributes.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicContainerAttributes': 'name' is already in scope because it is declared in an enclosing type |
4 |
14 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicContainerAttributes': 'type' is already in scope because it is declared in an enclosing type |
4 |
15 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicContainerAttributes': 'createdBy' is already in scope because it is declared in an enclosing type |
4 |
16 |
de/dlr/shepard/common/search/query/Neo4jQueryBuilder.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `"((" + neighborhoodIdsIsContainedInPart(node, v...`. |
4 |
311–317 |
de/dlr/shepard/common/subscription/entities/Subscription.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(callbackURL, other.callbackURL) ...`. |
4 |
72–80 |
de/dlr/shepard/common/util/CypherQueryHelper.java
de/dlr/shepard/context/collection/endpoints/DataObjectAttributes.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'DataObjectAttributes': 'name' is already in scope because it is declared in an enclosing type |
4 |
11 |
de/dlr/shepard/context/collection/entities/Collection.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `HasId.areEqualSetsByUniqueId(dataObjects, other...`. |
4 |
72–78 |
de/dlr/shepard/context/collection/entities/DataObject.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `HasId.equalsHelper(collection, other.collection...`. |
4 |
99–108 |
de/dlr/shepard/context/collection/io/CollectionIO.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `HasId.areEqualSets(dataObjectIds, other.dataObj...`. |
4 |
48–52 |
de/dlr/shepard/context/collection/io/DataObjectIO.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `collectionId == other.collectionId && HasId.are...`. |
4 |
53–61 |
de/dlr/shepard/context/export/ExportService.java
de/dlr/shepard/context/labJournal/entities/LabJournalEntry.java
de/dlr/shepard/context/references/basicreference/endpoints/BasicReferenceAttributes.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicReferenceAttributes': 'name' is already in scope because it is declared in an enclosing type |
4 |
13 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'BasicReferenceAttributes': 'type' is already in scope because it is declared in an enclosing type |
4 |
14 |
de/dlr/shepard/context/references/basicreference/io/BasicReferenceIO.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `dataObjectId == other.dataObjectId && Objects.e...`. |
4 |
34 |
de/dlr/shepard/context/references/dataobject/entities/CollectionReference.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(relationship, other.relationship...`. |
4 |
49–52 |
de/dlr/shepard/context/references/dataobject/entities/DataObjectReference.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(relationship, other.relationship...`. |
4 |
49–52 |
de/dlr/shepard/context/references/file/io/FileReferenceIO.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `fileContainerId == other.fileContainerId && Has...`. |
4 |
39 |
de/dlr/shepard/context/references/structureddata/entities/StructuredDataReference.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `HasId.equalsHelper(structuredDataContainer...) ...`. |
4 |
57–60 |
de/dlr/shepard/context/references/structureddata/io/StructuredDataReferenceIO.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `structuredDataContainerId == other.structuredDa...`. |
4 |
41–44 |
de/dlr/shepard/context/references/timeseriesreference/model/TimeseriesReference.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `end == other.end && start == other.start && Obj...`. |
4 |
60–65 |
de/dlr/shepard/context/references/timeseriesreference/services/TimeseriesReferenceMetricsService.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `metric != AggregateFunction.COUNT && metric != ...`. |
4 |
129 |
de/dlr/shepard/context/semantic/endpoints/SemanticRepositoryAttributes.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SemanticRepositoryAttributes': 'name' is already in scope because it is declared in an enclosing type |
4 |
11 |
de/dlr/shepard/context/semantic/entities/SemanticAnnotation.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(id, other.id) && Objects.equals(...`. |
4 |
68–76 |
de/dlr/shepard/context/version/entities/Version.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Objects.equals(createdAt, other.createdAt) && O...`. |
4 |
101–109 |
de/dlr/shepard/data/ContainerAttributes.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'ContainerAttributes': 'name' is already in scope because it is declared in an enclosing type |
4 |
11 |
de/dlr/shepard/data/timeseries/migrations/V1_7_0__int_to_bigint.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryImport |
Unused import 'org.flywaydb.core.api.MigrationVersion' |
4 |
7 |
| CheckResultSet |
Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. |
3 |
27 |
de/dlr/shepard/data/timeseries/repositories/TimeseriesDataPointRepository.java
de/dlr/shepard/data/timeseries/services/TimeseriesService.java