PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 3

org/itracker/model/Configuration.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static 35–106
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Type(Integer, Integer)': enum constructors are implicitly private 47–50

org/itracker/model/CustomField.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static 262–290
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Type(Integer)': enum constructors are implicitly private 268–270
UnnecessaryModifier Unnecessary modifier 'static' on enum 'DateFormat': nested enums are implicitly static 297–307

org/itracker/model/ImportDataModel.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'log'. 30
SimplifiedTernary Ternary operators that can be simplified with || or && 82
SimplifiedTernary Ternary operators that can be simplified with || or && 94
SimplifiedTernary Ternary operators that can be simplified with || or && 103
SimplifiedTernary Ternary operators that can be simplified with || or && 111

org/itracker/model/IssueActivityType.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'IssueActivityType(Integer)': enum constructors are implicitly private 40–42

org/itracker/model/IssueHistory.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Status': nested enums are implicitly static 127–140
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Status(int)': enum constructors are implicitly private 136–138

org/itracker/model/IssueRelation.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static 111–198
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Type(Integer)': enum constructors are implicitly private 167–169

org/itracker/model/Notification.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Role': nested enums are implicitly static 205–237
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Role(int)': enum constructors are implicitly private 229–231
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static 239–263
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Type(int)': enum constructors are implicitly private 255–257

org/itracker/model/Status.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Status(Integer)': enum constructors are implicitly private 40–42

org/itracker/model/WorkflowScript.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'numUses'. 68

org/itracker/model/util/IssueUtilities.java

Rule Violation Line
CollapsibleIfStatements These nested if statements could be combined 138–140
CollapsibleIfStatements These nested if statements could be combined 620–627
CollapsibleIfStatements These nested if statements could be combined 697–703

org/itracker/model/util/SystemConfigurationUtilities.java

Rule Violation Line
EmptyCatchBlock Avoid empty catch blocks 109–111
EmptyCatchBlock Avoid empty catch blocks 111–113

org/itracker/model/util/UserUtilities.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'getAllPermissionsSet': private methods cannot be overridden 123–125

org/itracker/model/util/WorkflowUtilities.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 60

org/itracker/persistence/dao/BaseDAO.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'save': the method is declared in an interface type 17
UnnecessaryModifier Unnecessary modifier 'public' on method 'saveOrUpdate': the method is declared in an interface type 26
UnnecessaryModifier Unnecessary modifier 'public' on method 'delete': the method is declared in an interface type 33
UnnecessaryModifier Unnecessary modifier 'public' on method 'detach': the method is declared in an interface type 40
UnnecessaryModifier Unnecessary modifier 'public' on method 'refresh': the method is declared in an interface type 47
UnnecessaryModifier Unnecessary modifier 'public' on method 'merge': the method is declared in an interface type 56

org/itracker/persistence/dao/ConfigurationDAO.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByPrimaryKey': the method is declared in an interface type 12
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByType': the method is declared in an interface type 14
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByType': the method is declared in an interface type 22
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByTypeAndValue': the method is declared in an interface type 25
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByTypeValueKey': the method is declared in an interface type 33
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByTypeAndValue': the method is declared in an interface type 41

org/itracker/persistence/dao/CustomFieldDAO.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByPrimaryKey': the method is declared in an interface type 10
UnnecessaryModifier Unnecessary modifier 'public' on method 'findAll': the method is declared in an interface type 12

org/itracker/persistence/dao/CustomFieldValueDAO.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByPrimaryKey': the method is declared in an interface type 12
UnnecessaryModifier Unnecessary modifier 'public' on method 'findAll': the method is declared in an interface type 14

org/itracker/persistence/dao/ProjectDAO.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getLastIssueUpdateDate': the method is declared in an interface type 46

org/itracker/persistence/dao/UserPreferencesDAO.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByUserId': the method is declared in an interface type 10

org/itracker/persistence/dao/VersionDAO.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByPrimaryKey': the method is declared in an interface type 12
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByProjectId': the method is declared in an interface type 14

org/itracker/persistence/dao/WorkflowScriptDAO.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByPrimaryKey': the method is declared in an interface type 19
UnnecessaryModifier Unnecessary modifier 'public' on method 'findAll': the method is declared in an interface type 26

Priority 4

org/itracker/model/AbstractEntity.java

Rule Violation Line
UselessParentheses Useless parentheses. 148

org/itracker/model/CustomFieldValue.java

Rule Violation Line
UselessParentheses Useless parentheses. 76

org/itracker/model/ImportDataModel.java

Rule Violation Line
UselessParentheses Useless parentheses. 46
UselessParentheses Useless parentheses. 51
UselessParentheses Useless parentheses. 56–57
UselessParentheses Useless parentheses. 82
UselessParentheses Useless parentheses. 94
UselessParentheses Useless parentheses. 103
UselessParentheses Useless parentheses. 111

org/itracker/model/IssueAttachment.java

Rule Violation Line
UselessParentheses Useless parentheses. 134

org/itracker/model/IssueField.java

Rule Violation Line
UselessParentheses Useless parentheses. 221

org/itracker/model/IssueSearchQuery.java

Rule Violation Line
UselessParentheses Useless parentheses. 83
UselessParentheses Useless parentheses. 92

org/itracker/model/SystemConfiguration.java

Rule Violation Line
UselessParentheses Useless parentheses. 52
UselessParentheses Useless parentheses. 70–71
UselessParentheses Useless parentheses. 81–82
UselessParentheses Useless parentheses. 92

org/itracker/model/User.java

Rule Violation Line
UselessParentheses Useless parentheses. 226–228

org/itracker/model/Version.java

Rule Violation Line
UselessParentheses Useless parentheses. 186–187
UselessParentheses Useless parentheses. 195–196
UselessParentheses Useless parentheses. 197–199
UselessParentheses Useless parentheses. 247

org/itracker/model/util/IssueUtilities.java

Rule Violation Line
UselessParentheses Useless parentheses. 132–133
UselessParentheses Useless parentheses. 352
UselessParentheses Useless parentheses. 392
UselessParentheses Useless parentheses. 481
UselessParentheses Useless parentheses. 499–500
UselessParentheses Useless parentheses. 508
UselessParentheses Useless parentheses. 794

org/itracker/model/util/ProjectUtilities.java

Rule Violation Line
UselessParentheses Useless parentheses. 115

org/itracker/model/util/PropertiesFileHandler.java

Rule Violation Line
UselessParentheses Useless parentheses. 74

org/itracker/model/util/UserUtilities.java

Rule Violation Line
UselessParentheses Useless parentheses. 215
UselessParentheses Useless parentheses. 262
UselessParentheses Useless parentheses. 348
UselessParentheses Useless parentheses. 384
UselessParentheses Useless parentheses. 443

org/itracker/persistence/dao/AbstractEnumUserType.java

Rule Violation Line
UselessParentheses Useless parentheses. 75

org/itracker/persistence/dao/IntBooleanType.java

Rule Violation Line
UselessParentheses Useless parentheses. 50

org/itracker/persistence/dao/TimestampInterceptor.java

Rule Violation Line
UselessParentheses Useless parentheses. 80

Files

org/itracker/model/AbstractEntity.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 148

org/itracker/model/Configuration.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static 3 35–106
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Type(Integer, Integer)': enum constructors are implicitly private 3 47–50

org/itracker/model/CustomField.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static 3 262–290
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Type(Integer)': enum constructors are implicitly private 3 268–270
UnnecessaryModifier Unnecessary modifier 'static' on enum 'DateFormat': nested enums are implicitly static 3 297–307

org/itracker/model/CustomFieldValue.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 76

org/itracker/model/ImportDataModel.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'log'. 3 30
UselessParentheses Useless parentheses. 4 46
UselessParentheses Useless parentheses. 4 51
UselessParentheses Useless parentheses. 4 56–57
UselessParentheses Useless parentheses. 4 82
SimplifiedTernary Ternary operators that can be simplified with || or && 3 82
SimplifiedTernary Ternary operators that can be simplified with || or && 3 94
UselessParentheses Useless parentheses. 4 94
SimplifiedTernary Ternary operators that can be simplified with || or && 3 103
UselessParentheses Useless parentheses. 4 103
UselessParentheses Useless parentheses. 4 111
SimplifiedTernary Ternary operators that can be simplified with || or && 3 111

org/itracker/model/IssueActivityType.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'IssueActivityType(Integer)': enum constructors are implicitly private 3 40–42

org/itracker/model/IssueAttachment.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 134

org/itracker/model/IssueField.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 221

org/itracker/model/IssueHistory.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Status': nested enums are implicitly static 3 127–140
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Status(int)': enum constructors are implicitly private 3 136–138

org/itracker/model/IssueRelation.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static 3 111–198
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Type(Integer)': enum constructors are implicitly private 3 167–169

org/itracker/model/IssueSearchQuery.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 83
UselessParentheses Useless parentheses. 4 92

org/itracker/model/Notification.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Role': nested enums are implicitly static 3 205–237
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Role(int)': enum constructors are implicitly private 3 229–231
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Type': nested enums are implicitly static 3 239–263
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Type(int)': enum constructors are implicitly private 3 255–257

org/itracker/model/Status.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Status(Integer)': enum constructors are implicitly private 3 40–42

org/itracker/model/SystemConfiguration.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 52
UselessParentheses Useless parentheses. 4 70–71
UselessParentheses Useless parentheses. 4 81–82
UselessParentheses Useless parentheses. 4 92

org/itracker/model/User.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 226–228

org/itracker/model/Version.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 186–187
UselessParentheses Useless parentheses. 4 195–196
UselessParentheses Useless parentheses. 4 197–199
UselessParentheses Useless parentheses. 4 247

org/itracker/model/WorkflowScript.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'numUses'. 3 68

org/itracker/model/util/IssueUtilities.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 132–133
CollapsibleIfStatements These nested if statements could be combined 3 138–140
UselessParentheses Useless parentheses. 4 352
UselessParentheses Useless parentheses. 4 392
UselessParentheses Useless parentheses. 4 481
UselessParentheses Useless parentheses. 4 499–500
UselessParentheses Useless parentheses. 4 508
CollapsibleIfStatements These nested if statements could be combined 3 620–627
CollapsibleIfStatements These nested if statements could be combined 3 697–703
UselessParentheses Useless parentheses. 4 794

org/itracker/model/util/ProjectUtilities.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 115

org/itracker/model/util/PropertiesFileHandler.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 74

org/itracker/model/util/SystemConfigurationUtilities.java

Rule Violation Priority Line
EmptyCatchBlock Avoid empty catch blocks 3 109–111
EmptyCatchBlock Avoid empty catch blocks 3 111–113

org/itracker/model/util/UserUtilities.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'final' on method 'getAllPermissionsSet': private methods cannot be overridden 3 123–125
UselessParentheses Useless parentheses. 4 215
UselessParentheses Useless parentheses. 4 262
UselessParentheses Useless parentheses. 4 348
UselessParentheses Useless parentheses. 4 384
UselessParentheses Useless parentheses. 4 443

org/itracker/model/util/WorkflowUtilities.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as 'logger'. 3 60

org/itracker/persistence/dao/AbstractEnumUserType.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 75

org/itracker/persistence/dao/BaseDAO.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'save': the method is declared in an interface type 3 17
UnnecessaryModifier Unnecessary modifier 'public' on method 'saveOrUpdate': the method is declared in an interface type 3 26
UnnecessaryModifier Unnecessary modifier 'public' on method 'delete': the method is declared in an interface type 3 33
UnnecessaryModifier Unnecessary modifier 'public' on method 'detach': the method is declared in an interface type 3 40
UnnecessaryModifier Unnecessary modifier 'public' on method 'refresh': the method is declared in an interface type 3 47
UnnecessaryModifier Unnecessary modifier 'public' on method 'merge': the method is declared in an interface type 3 56

org/itracker/persistence/dao/ConfigurationDAO.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByPrimaryKey': the method is declared in an interface type 3 12
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByType': the method is declared in an interface type 3 14
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByType': the method is declared in an interface type 3 22
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByTypeAndValue': the method is declared in an interface type 3 25
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByTypeValueKey': the method is declared in an interface type 3 33
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByTypeAndValue': the method is declared in an interface type 3 41

org/itracker/persistence/dao/CustomFieldDAO.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByPrimaryKey': the method is declared in an interface type 3 10
UnnecessaryModifier Unnecessary modifier 'public' on method 'findAll': the method is declared in an interface type 3 12

org/itracker/persistence/dao/CustomFieldValueDAO.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByPrimaryKey': the method is declared in an interface type 3 12
UnnecessaryModifier Unnecessary modifier 'public' on method 'findAll': the method is declared in an interface type 3 14

org/itracker/persistence/dao/IntBooleanType.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 50

org/itracker/persistence/dao/ProjectDAO.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'getLastIssueUpdateDate': the method is declared in an interface type 3 46

org/itracker/persistence/dao/TimestampInterceptor.java

Rule Violation Priority Line
UselessParentheses Useless parentheses. 4 80

org/itracker/persistence/dao/UserPreferencesDAO.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByUserId': the method is declared in an interface type 3 10

org/itracker/persistence/dao/VersionDAO.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByPrimaryKey': the method is declared in an interface type 3 12
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByProjectId': the method is declared in an interface type 3 14

org/itracker/persistence/dao/WorkflowScriptDAO.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'public' on method 'findByPrimaryKey': the method is declared in an interface type 3 19
UnnecessaryModifier Unnecessary modifier 'public' on method 'findAll': the method is declared in an interface type 3 26
R.Ø.S.A.