PMD Results

The following document contains the results of PMD 7.29.0.

Violations By Priority

Priority 2

team/projectpulse/notification/ReminderService.java

Rule Violation Line
GuardLogStatement Logger calls should be surrounded by log level guards. 72

team/projectpulse/notification/WeeklyReminderScheduler.java

Rule Violation Line
GuardLogStatement Logger calls should be surrounded by log level guards. 81

team/projectpulse/ram/document/template/DocumentTemplateRegistry.java

Rule Violation Line
GuardLogStatement Logger calls should be surrounded by log level guards. 48

team/projectpulse/security/AuthController.java

Rule Violation Line
GuardLogStatement Logger calls should be surrounded by log level guards. 38

Priority 3

team/projectpulse/activity/Activity.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 39

team/projectpulse/activity/ActivityController.java

Rule Violation Line
UnusedLocalVariable Avoid unused local variables such as 'updatedActivityDto'. 70

team/projectpulse/activity/ActivitySecurityService.java

Rule Violation Line
LambdaCanBeMethodReference Lambda expression could be written as a method reference: `Section::getSectionId` 80

team/projectpulse/activity/ActivitySpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 5

team/projectpulse/course/Course.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 35

team/projectpulse/course/CourseSpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 6
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 10
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 15

team/projectpulse/evaluation/EvaluationService.java

Rule Violation Line
LiteralsFirstInComparisons Position literals first in String comparisons 257

team/projectpulse/evaluation/PeerEvaluation.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 38

team/projectpulse/evaluation/PeerEvaluationAverage.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 19
UncommentedEmptyConstructor Document empty constructor 19

team/projectpulse/evaluation/Rating.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 19

team/projectpulse/evaluation/RatingAverage.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 9

team/projectpulse/evaluation/WeeklyPeerEvaluationReport.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 13
UncommentedEmptyConstructor Document empty constructor 13

team/projectpulse/instructor/Instructor.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 32

team/projectpulse/instructor/InstructorSecurityService.java

Rule Violation Line
SimplifyBooleanReturns This if statement can be replaced by `return !{condition} && {elseBranch};` 43–45

team/projectpulse/instructor/InstructorService.java

Rule Violation Line
LiteralsFirstInComparisons Position literals first in String comparisons 116

team/projectpulse/instructor/InstructorSpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 6
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 10
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 15
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 20

team/projectpulse/notification/WeeklyReminderScheduler.java

Rule Violation Line
ControlStatementBraces This statement should have braces 55
ControlStatementBraces This statement should have braces 77

team/projectpulse/ram/collaboration/Comment.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 38
UncommentedEmptyConstructor Document empty constructor 38

team/projectpulse/ram/collaboration/CommentThread.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 62
UncommentedEmptyConstructor Document empty constructor 62

team/projectpulse/ram/collaboration/converter/CommentDtoToCommentConverter.java

Rule Violation Line
ControlStatementBraces This statement should have braces 12

team/projectpulse/ram/collaboration/converter/CommentThreadToCommentThreadDtoConverter.java

Rule Violation Line
ControlStatementBraces This statement should have braces 22

team/projectpulse/ram/collaboration/converter/CommentToCommentDtoConverter.java

Rule Violation Line
ControlStatementBraces This statement should have braces 20

team/projectpulse/ram/document/DocumentSection.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 72
UncommentedEmptyConstructor Document empty constructor 72
ControlStatementBraces This statement should have braces 101
ControlStatementBraces This statement should have braces 107
CompareObjectsWithEquals Use equals() to compare object references. 109

team/projectpulse/ram/document/DocumentSectionLock.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 35
UncommentedEmptyConstructor Document empty constructor 35
ControlStatementBraces This statement should have braces 39
SimplifyBooleanReturns This if statement can be replaced by `return {condition} || {elseBranch};` 40
ControlStatementBraces This statement should have braces 40

team/projectpulse/ram/document/RequirementDocument.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 55
UncommentedEmptyConstructor Document empty constructor 55
ControlStatementBraces This statement should have braces 59
ControlStatementBraces This statement should have braces 66
CompareObjectsWithEquals Use equals() to compare object references. 69
ControlStatementBraces This statement should have braces 75
ControlStatementBraces This statement should have braces 82
CompareObjectsWithEquals Use equals() to compare object references. 85

team/projectpulse/ram/document/RequirementDocumentSpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 19

team/projectpulse/ram/requirement/ArtifactKeySequence.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 30

team/projectpulse/ram/requirement/ArtifactLink.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 53
UncommentedEmptyConstructor Document empty constructor 53

team/projectpulse/ram/requirement/ArtifactLinkSpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 5

team/projectpulse/ram/requirement/RequirementArtifact.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 80
ControlStatementBraces This statement should have braces 113
ControlStatementBraces This statement should have braces 119
CompareObjectsWithEquals Use equals() to compare object references. 121

team/projectpulse/ram/requirement/RequirementArtifactSpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 5

team/projectpulse/ram/usecase/UseCase.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 76
UncommentedEmptyConstructor Document empty constructor 76

team/projectpulse/ram/usecase/UseCaseController.java

Rule Violation Line
DanglingJavadoc Javadoc comment does not belong to any class, method or field 33–36

team/projectpulse/ram/usecase/UseCaseExtension.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 58

team/projectpulse/ram/usecase/UseCaseExtensionStep.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 21

team/projectpulse/ram/usecase/UseCaseLock.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 35
UncommentedEmptyConstructor Document empty constructor 35
ControlStatementBraces This statement should have braces 39
SimplifyBooleanReturns This if statement can be replaced by `return {condition} || {elseBranch};` 40
ControlStatementBraces This statement should have braces 40

team/projectpulse/ram/usecase/UseCaseMainStep.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 31

team/projectpulse/ram/usecase/UseCaseService.java

Rule Violation Line
UnusedPrivateMethod Avoid unused private methods such as 'updateCollection(Set<T>, Set<T>)'. 251

team/projectpulse/rubric/Criterion.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 24

team/projectpulse/rubric/CriterionSpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 9
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 14

team/projectpulse/rubric/Rubric.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 27

team/projectpulse/rubric/RubricSpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 9

team/projectpulse/section/Section.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 54
ControlStatementBraces This statement should have braces 128
ControlStatementBraces This statement should have braces 129
ControlStatementBraces This statement should have braces 142
ControlStatementBraces This statement should have braces 143

team/projectpulse/section/SectionSecurityService.java

Rule Violation Line
SimplifyBooleanReturns This if statement can be replaced by `return !{condition} && {elseBranch};` 78–80

team/projectpulse/section/SectionService.java

Rule Violation Line
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 57

team/projectpulse/section/SectionSpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 9

team/projectpulse/security/AuthService.java

Rule Violation Line
LiteralsFirstInComparisons Position literals first in String comparisons 39

team/projectpulse/security/JwtProvider.java

Rule Violation Line
LambdaCanBeMethodReference Lambda expression could be written as a method reference: `GrantedAuthority::getAuthority` 30

team/projectpulse/security/SecurityConfiguration.java

Rule Violation Line
LambdaCanBeMethodReference Lambda expression could be written as a method reference: `AbstractHttpConfigurer::disable` 317

team/projectpulse/student/Student.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 17

team/projectpulse/student/StudentSecurityService.java

Rule Violation Line
SimplifyBooleanReturns This if statement can be replaced by `return !{condition} && {elseBranch};` 43–45

team/projectpulse/student/StudentService.java

Rule Violation Line
LiteralsFirstInComparisons Position literals first in String comparisons 120

team/projectpulse/student/StudentSpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 9
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 14
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 19
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 24
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 34

team/projectpulse/system/PeerEvaluationUserAuditorAware.java

Rule Violation Line
ControlStatementBraces This statement should have braces 25

team/projectpulse/system/Result.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 18

team/projectpulse/system/StatusCode.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3

team/projectpulse/system/UserUtils.java

Rule Violation Line
ControlStatementBraces This statement should have braces 27
ControlStatementBraces This statement should have braces 30
ControlStatementBraces This statement should have braces 33
ControlStatementBraces This statement should have braces 35
ControlStatementBraces This statement should have braces 36
ControlStatementBraces This statement should have braces 37
LiteralsFirstInComparisons Position literals first in String comparisons 46
LiteralsFirstInComparisons Position literals first in String comparisons 61

team/projectpulse/team/Team.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 28
ControlStatementBraces This statement should have braces 96
ControlStatementBraces This statement should have braces 102

team/projectpulse/team/TeamService.java

Rule Violation Line
CompareObjectsWithEquals Use equals() to compare object references. 178
CompareObjectsWithEquals Use equals() to compare object references. 191

team/projectpulse/team/TeamSpecs.java

Rule Violation Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 9
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 14

team/projectpulse/user/PeerEvaluationUser.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 36

team/projectpulse/user/resetpassword/ResetPasswordToken.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 18

team/projectpulse/user/userinvitation/UserInvitation.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 17

Priority 4

team/projectpulse/section/dto/SectionDto.java

Rule Violation Line
UnnecessaryImport Unused import 'java.time.LocalTime' 8

team/projectpulse/security/JwtProvider.java

Rule Violation Line
UselessParentheses Useless parentheses around `authentication.getPrincipal()`. 38

team/projectpulse/user/UserController.java

Rule Violation Line
UnnecessaryImport Unused import 'org.springframework.security.core.userdetails.UserDetails' 3

Files

team/projectpulse/activity/Activity.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 39

team/projectpulse/activity/ActivityController.java

Rule Violation Priority Line
UnusedLocalVariable Avoid unused local variables such as 'updatedActivityDto'. 3 70

team/projectpulse/activity/ActivitySecurityService.java

Rule Violation Priority Line
LambdaCanBeMethodReference Lambda expression could be written as a method reference: `Section::getSectionId` 3 80

team/projectpulse/activity/ActivitySpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 5

team/projectpulse/course/Course.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 35

team/projectpulse/course/CourseSpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 6
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 10
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 15

team/projectpulse/evaluation/EvaluationService.java

Rule Violation Priority Line
LiteralsFirstInComparisons Position literals first in String comparisons 3 257

team/projectpulse/evaluation/PeerEvaluation.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 38

team/projectpulse/evaluation/PeerEvaluationAverage.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 19
UncommentedEmptyConstructor Document empty constructor 3 19

team/projectpulse/evaluation/Rating.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 19

team/projectpulse/evaluation/RatingAverage.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 9

team/projectpulse/evaluation/WeeklyPeerEvaluationReport.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 13
UncommentedEmptyConstructor Document empty constructor 3 13

team/projectpulse/instructor/Instructor.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 32

team/projectpulse/instructor/InstructorSecurityService.java

Rule Violation Priority Line
SimplifyBooleanReturns This if statement can be replaced by `return !{condition} && {elseBranch};` 3 43–45

team/projectpulse/instructor/InstructorService.java

Rule Violation Priority Line
LiteralsFirstInComparisons Position literals first in String comparisons 3 116

team/projectpulse/instructor/InstructorSpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 6
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 10
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 15
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 20

team/projectpulse/notification/ReminderService.java

Rule Violation Priority Line
GuardLogStatement Logger calls should be surrounded by log level guards. 2 72

team/projectpulse/notification/WeeklyReminderScheduler.java

Rule Violation Priority Line
ControlStatementBraces This statement should have braces 3 55
ControlStatementBraces This statement should have braces 3 77
GuardLogStatement Logger calls should be surrounded by log level guards. 2 81

team/projectpulse/ram/collaboration/Comment.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 38
UncommentedEmptyConstructor Document empty constructor 3 38

team/projectpulse/ram/collaboration/CommentThread.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 62
UncommentedEmptyConstructor Document empty constructor 3 62

team/projectpulse/ram/collaboration/converter/CommentDtoToCommentConverter.java

Rule Violation Priority Line
ControlStatementBraces This statement should have braces 3 12

team/projectpulse/ram/collaboration/converter/CommentThreadToCommentThreadDtoConverter.java

Rule Violation Priority Line
ControlStatementBraces This statement should have braces 3 22

team/projectpulse/ram/collaboration/converter/CommentToCommentDtoConverter.java

Rule Violation Priority Line
ControlStatementBraces This statement should have braces 3 20

team/projectpulse/ram/document/DocumentSection.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 72
UncommentedEmptyConstructor Document empty constructor 3 72
ControlStatementBraces This statement should have braces 3 101
ControlStatementBraces This statement should have braces 3 107
CompareObjectsWithEquals Use equals() to compare object references. 3 109

team/projectpulse/ram/document/DocumentSectionLock.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 35
UncommentedEmptyConstructor Document empty constructor 3 35
ControlStatementBraces This statement should have braces 3 39
SimplifyBooleanReturns This if statement can be replaced by `return {condition} || {elseBranch};` 3 40
ControlStatementBraces This statement should have braces 3 40

team/projectpulse/ram/document/RequirementDocument.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 55
UncommentedEmptyConstructor Document empty constructor 3 55
ControlStatementBraces This statement should have braces 3 59
ControlStatementBraces This statement should have braces 3 66
CompareObjectsWithEquals Use equals() to compare object references. 3 69
ControlStatementBraces This statement should have braces 3 75
ControlStatementBraces This statement should have braces 3 82
CompareObjectsWithEquals Use equals() to compare object references. 3 85

team/projectpulse/ram/document/RequirementDocumentSpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 19

team/projectpulse/ram/document/template/DocumentTemplateRegistry.java

Rule Violation Priority Line
GuardLogStatement Logger calls should be surrounded by log level guards. 2 48

team/projectpulse/ram/requirement/ArtifactKeySequence.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 30

team/projectpulse/ram/requirement/ArtifactLink.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 53
UncommentedEmptyConstructor Document empty constructor 3 53

team/projectpulse/ram/requirement/ArtifactLinkSpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 5

team/projectpulse/ram/requirement/RequirementArtifact.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 80
ControlStatementBraces This statement should have braces 3 113
ControlStatementBraces This statement should have braces 3 119
CompareObjectsWithEquals Use equals() to compare object references. 3 121

team/projectpulse/ram/requirement/RequirementArtifactSpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 5

team/projectpulse/ram/usecase/UseCase.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 76
UncommentedEmptyConstructor Document empty constructor 3 76

team/projectpulse/ram/usecase/UseCaseController.java

Rule Violation Priority Line
DanglingJavadoc Javadoc comment does not belong to any class, method or field 3 33–36

team/projectpulse/ram/usecase/UseCaseExtension.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 58

team/projectpulse/ram/usecase/UseCaseExtensionStep.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 21

team/projectpulse/ram/usecase/UseCaseLock.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 35
UncommentedEmptyConstructor Document empty constructor 3 35
ControlStatementBraces This statement should have braces 3 39
SimplifyBooleanReturns This if statement can be replaced by `return {condition} || {elseBranch};` 3 40
ControlStatementBraces This statement should have braces 3 40

team/projectpulse/ram/usecase/UseCaseMainStep.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 31

team/projectpulse/ram/usecase/UseCaseService.java

Rule Violation Priority Line
UnusedPrivateMethod Avoid unused private methods such as 'updateCollection(Set<T>, Set<T>)'. 3 251

team/projectpulse/rubric/Criterion.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 24

team/projectpulse/rubric/CriterionSpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 9
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 14

team/projectpulse/rubric/Rubric.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 27

team/projectpulse/rubric/RubricSpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 9

team/projectpulse/section/Section.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 54
ControlStatementBraces This statement should have braces 3 128
ControlStatementBraces This statement should have braces 3 129
ControlStatementBraces This statement should have braces 3 142
ControlStatementBraces This statement should have braces 3 143

team/projectpulse/section/SectionSecurityService.java

Rule Violation Priority Line
SimplifyBooleanReturns This if statement can be replaced by `return !{condition} && {elseBranch};` 3 78–80

team/projectpulse/section/SectionService.java

Rule Violation Priority Line
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 57

team/projectpulse/section/SectionSpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 9

team/projectpulse/section/dto/SectionDto.java

Rule Violation Priority Line
UnnecessaryImport Unused import 'java.time.LocalTime' 4 8

team/projectpulse/security/AuthController.java

Rule Violation Priority Line
GuardLogStatement Logger calls should be surrounded by log level guards. 2 38

team/projectpulse/security/AuthService.java

Rule Violation Priority Line
LiteralsFirstInComparisons Position literals first in String comparisons 3 39

team/projectpulse/security/JwtProvider.java

Rule Violation Priority Line
LambdaCanBeMethodReference Lambda expression could be written as a method reference: `GrantedAuthority::getAuthority` 3 30
UselessParentheses Useless parentheses around `authentication.getPrincipal()`. 4 38

team/projectpulse/security/SecurityConfiguration.java

Rule Violation Priority Line
LambdaCanBeMethodReference Lambda expression could be written as a method reference: `AbstractHttpConfigurer::disable` 3 317

team/projectpulse/student/Student.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 17

team/projectpulse/student/StudentSecurityService.java

Rule Violation Priority Line
SimplifyBooleanReturns This if statement can be replaced by `return !{condition} && {elseBranch};` 3 43–45

team/projectpulse/student/StudentService.java

Rule Violation Priority Line
LiteralsFirstInComparisons Position literals first in String comparisons 3 120

team/projectpulse/student/StudentSpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 9
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 14
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 19
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 24
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 34

team/projectpulse/system/PeerEvaluationUserAuditorAware.java

Rule Violation Priority Line
ControlStatementBraces This statement should have braces 3 25

team/projectpulse/system/Result.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 18

team/projectpulse/system/StatusCode.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 3

team/projectpulse/system/UserUtils.java

Rule Violation Priority Line
ControlStatementBraces This statement should have braces 3 27
ControlStatementBraces This statement should have braces 3 30
ControlStatementBraces This statement should have braces 3 33
ControlStatementBraces This statement should have braces 3 35
ControlStatementBraces This statement should have braces 3 36
ControlStatementBraces This statement should have braces 3 37
LiteralsFirstInComparisons Position literals first in String comparisons 3 46
LiteralsFirstInComparisons Position literals first in String comparisons 3 61

team/projectpulse/team/Team.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 28
ControlStatementBraces This statement should have braces 3 96
ControlStatementBraces This statement should have braces 3 102

team/projectpulse/team/TeamService.java

Rule Violation Priority Line
CompareObjectsWithEquals Use equals() to compare object references. 3 178
CompareObjectsWithEquals Use equals() to compare object references. 3 191

team/projectpulse/team/TeamSpecs.java

Rule Violation Priority Line
InstantiableUtilityClass All members are static. Consider adding a private no-args constructor to prevent instantiation. 3 5
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 9
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 14

team/projectpulse/user/PeerEvaluationUser.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 36

team/projectpulse/user/UserController.java

Rule Violation Priority Line
UnnecessaryImport Unused import 'org.springframework.security.core.userdetails.UserDetails' 4 3

team/projectpulse/user/resetpassword/ResetPasswordToken.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 18

team/projectpulse/user/userinvitation/UserInvitation.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 17