PMD Results

The following document contains the results of PMD 5.2.3.

Files

com/CORSFilter.java

Violation Line
These nested if statements could be combined 38–44

com/entity/BaseEntity.java

Violation Line
Avoid unused private fields such as 'version'. 59

com/repository/IAuthorityRepository.java

Violation Line
Avoid unused imports such as 'com.entity.TranslationEntity' 25

com/repository/IGroupRepository.java

Violation Line
Avoid unused imports such as 'com.entity.TranslationEntity' 25

com/repository/IInvitationGroupRepository.java

Violation Line
Avoid unused imports such as 'com.entity.InvitationEntity' 24
Avoid unused imports such as 'com.entity.TranslationEntity' 26

com/repository/IInvitationRepository.java

Violation Line
Avoid unused imports such as 'com.entity.TranslationEntity' 25
Avoid unused imports such as 'org.springframework.context.annotation.Bean' 26

com/repository/IProjectGroupRepository.java

Violation Line
Avoid unused imports such as 'com.entity.TranslationEntity' 25

com/repository/IProjectUserRepository.java

Violation Line
Avoid unused imports such as 'com.entity.TranslationEntity' 25

com/repository/ITranslationRepository.java

Violation Line
Avoid unused imports such as 'com.entity.ProjectEntity' 25

com/rest/ConfigRest.java

Violation Line
Avoid unused imports such as 'javax.inject.Inject' 32

com/rest/ProjectRest.java

Violation Line
Avoid unused private fields such as 'modelMapper'. 51

com/rest/UserRest.java

Violation Line
Avoid unused private methods such as 'convertToResponseDto(UserEntity)'. 97

com/utils/SecurityContextReader.java

Violation Line
Avoid unused imports such as 'org.springframework.context.annotation.Bean' 24
Avoid unused imports such as 'org.springframework.context.annotation.Configuration' 25
Avoid unused imports such as 'org.springframework.context.annotation.Profile' 26
Avoid unused imports such as 'org.springframework.stereotype.Component' 29

com/rest/it/ProjectRestIntTest.java

Violation Line
Avoid unused imports such as 'org.junit.After' 32

com/rest/unit/InvitationRestUnitTest.java

Violation Line
Unnecessary use of fully qualified name 'Mockito.mock' due to existing import 'org.mockito.Mockito' 40
Unnecessary use of fully qualified name 'Mockito.mock' due to existing import 'org.mockito.Mockito' 41
Avoid unused local variables such as 'invitationResponseDtos'. 54

com/rest/unit/ProjectRestUnitTest.java

Violation Line
Too many static imports may lead to messy code 1–113
Avoid unused imports such as 'com.dto.InvitationResponseDto' 26
Avoid unused imports such as 'com.rest.InvitationRest' 30
Avoid unused imports such as 'java.util.List' 39

com/rest/unit/UserRestUnitTest.java

Violation Line
Unnecessary use of fully qualified name 'Mockito.mock' due to existing import 'org.mockito.Mockito' 51
Unnecessary use of fully qualified name 'Mockito.mock' due to existing import 'org.mockito.Mockito' 52
Unnecessary use of fully qualified name 'Mockito.mock' due to existing import 'org.mockito.Mockito' 54
Unnecessary use of fully qualified name 'Mockito.mock' due to existing import 'org.mockito.Mockito' 55
Unnecessary use of fully qualified name 'Mockito.doNothing' due to existing import 'org.mockito.Mockito' 77
Avoid unused local variables such as 'responseEntity'. 79
Unnecessary use of fully qualified name 'Mockito.doNothing' due to existing import 'org.mockito.Mockito' 91
Avoid unused local variables such as 'responseEntity'. 93
Unnecessary use of fully qualified name 'Mockito.doNothing' due to existing import 'org.mockito.Mockito' 147