PMD Results

The following document contains the results of PMD 4.2.2.

Files

net/sourceforge/mystique/application/AddAttachmentForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 59
Found non-transient, non-static member. Please mark as transient or provide accessors. 62
Avoid excessively long variable names like attachmentRepository 62
Found non-transient, non-static member. Please mark as transient or provide accessors. 65
Parameter 'binder' is not assigned and could be declared final 68
Parameter 'model' is not assigned and could be declared final 75
Parameter 'ticketId' is not assigned and could be declared final 75
Parameter 'request' is not assigned and could be declared final 75
Local variable 'attachment' could be declared final 78
Local variable 'owner' could be declared final 80
Local variable 'ticket' could be declared final 83
Parameter 'multipartFile' is not assigned and could be declared final 94
Parameter 'attachment' is not assigned and could be declared final 94
Parameter 'result' is not assigned and could be declared final 95
Parameter 'status' is not assigned and could be declared final 95
A method should have only one exit point, and that should be the last statement in the method 99
Local variable 'ticketId' could be declared final 111

net/sourceforge/mystique/application/AddCommentForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 55
Found non-transient, non-static member. Please mark as transient or provide accessors. 58
Parameter 'binder' is not assigned and could be declared final 61
Parameter 'model' is not assigned and could be declared final 68
Parameter 'ticketId' is not assigned and could be declared final 68
Parameter 'request' is not assigned and could be declared final 68
Local variable 'comment' could be declared final 71
Local variable 'author' could be declared final 73
Local variable 'ticket' could be declared final 76
Parameter 'result' is not assigned and could be declared final 87
Parameter 'status' is not assigned and could be declared final 87
Parameter 'comment' is not assigned and could be declared final 87
A method should have only one exit point, and that should be the last statement in the method 92
Local variable 'ticketId' could be declared final 98

net/sourceforge/mystique/application/AddComponentForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 54
Found non-transient, non-static member. Please mark as transient or provide accessors. 57
Avoid excessively long variable names like componentRepository 57
Parameter 'binder' is not assigned and could be declared final 60
Parameter 'model' is not assigned and could be declared final 67
Parameter 'projectId' is not assigned and could be declared final 67
Local variable 'component' could be declared final 69
Local variable 'project' could be declared final 71
Parameter 'result' is not assigned and could be declared final 82
Parameter 'component' is not assigned and could be declared final 82
Parameter 'status' is not assigned and could be declared final 83
A method should have only one exit point, and that should be the last statement in the method 87
A method should have only one exit point, and that should be the last statement in the method 94
Local variable 'projectId' could be declared final 98

net/sourceforge/mystique/application/AddConfirmationForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 55
Found non-transient, non-static member. Please mark as transient or provide accessors. 58
Avoid excessively long variable names like confirmationRepository 58
Parameter 'binder' is not assigned and could be declared final 61
Parameter 'model' is not assigned and could be declared final 68
Parameter 'ticketId' is not assigned and could be declared final 68
Parameter 'request' is not assigned and could be declared final 68
Local variable 'confirmation' could be declared final 71
Local variable 'responsible' could be declared final 73
Local variable 'ticket' could be declared final 76
Parameter 'result' is not assigned and could be declared final 87
Parameter 'confirmation' is not assigned and could be declared final 87
Parameter 'status' is not assigned and could be declared final 88
A method should have only one exit point, and that should be the last statement in the method 92
Local variable 'ticketId' could be declared final 98

net/sourceforge/mystique/application/AddMilestoneForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 54
Found non-transient, non-static member. Please mark as transient or provide accessors. 57
Avoid excessively long variable names like milestoneRepository 57
Parameter 'binder' is not assigned and could be declared final 60
Parameter 'model' is not assigned and could be declared final 67
Parameter 'projectId' is not assigned and could be declared final 67
Local variable 'milestone' could be declared final 69
Local variable 'project' could be declared final 71
Parameter 'milestone' is not assigned and could be declared final 82
Parameter 'result' is not assigned and could be declared final 82
Parameter 'status' is not assigned and could be declared final 83
A method should have only one exit point, and that should be the last statement in the method 87
A method should have only one exit point, and that should be the last statement in the method 94
Local variable 'projectId' could be declared final 98

net/sourceforge/mystique/application/AddProjectForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 51
Parameter 'binder' is not assigned and could be declared final 54
Parameter 'model' is not assigned and could be declared final 61
Parameter 'project' is not assigned and could be declared final 71
Parameter 'result' is not assigned and could be declared final 71
Parameter 'status' is not assigned and could be declared final 71
A method should have only one exit point, and that should be the last statement in the method 76
A method should have only one exit point, and that should be the last statement in the method 83

net/sourceforge/mystique/application/AddTicketForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 52
Parameter 'binder' is not assigned and could be declared final 55
Parameter 'model' is not assigned and could be declared final 62
Parameter 'request' is not assigned and could be declared final 62
Local variable 'ticket' could be declared final 64
Local variable 'reporter' could be declared final 66
Parameter 'result' is not assigned and could be declared final 77
Parameter 'ticket' is not assigned and could be declared final 77
Parameter 'status' is not assigned and could be declared final 77
A method should have only one exit point, and that should be the last statement in the method 82

net/sourceforge/mystique/application/AddVersionForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 54
Found non-transient, non-static member. Please mark as transient or provide accessors. 57
Parameter 'binder' is not assigned and could be declared final 60
Parameter 'model' is not assigned and could be declared final 67
Parameter 'projectId' is not assigned and could be declared final 67
Local variable 'version' could be declared final 69
Local variable 'project' could be declared final 71
Parameter 'result' is not assigned and could be declared final 82
Parameter 'status' is not assigned and could be declared final 82
Parameter 'version' is not assigned and could be declared final 82
A method should have only one exit point, and that should be the last statement in the method 87
A method should have only one exit point, and that should be the last statement in the method 94
Local variable 'projectId' could be declared final 98

net/sourceforge/mystique/application/AttachmentController.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 45
Found non-transient, non-static member. Please mark as transient or provide accessors. 48
Avoid excessively long variable names like attachmentRepository 48
Parameter 'model' is not assigned and could be declared final 51
Parameter 'ticketId' is not assigned and could be declared final 51
Local variable 'ticket' could be declared final 53
Local variable 'list' could be declared final 54
Parameter 'attachmentId' is not assigned and could be declared final 65
Local variable 'attachment' could be declared final 67
Local variable 'ticketId' could be declared final 68

net/sourceforge/mystique/application/CommentController.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 45
Found non-transient, non-static member. Please mark as transient or provide accessors. 48
Parameter 'model' is not assigned and could be declared final 51
Parameter 'ticketId' is not assigned and could be declared final 51
Local variable 'ticket' could be declared final 53
Local variable 'list' could be declared final 54
Parameter 'commentId' is not assigned and could be declared final 65
Local variable 'comment' could be declared final 67
Local variable 'ticketId' could be declared final 68

net/sourceforge/mystique/application/ComponentController.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 45
Found non-transient, non-static member. Please mark as transient or provide accessors. 48
Avoid excessively long variable names like componentRepository 48
Parameter 'model' is not assigned and could be declared final 51
Parameter 'projectId' is not assigned and could be declared final 51
Local variable 'project' could be declared final 53
Local variable 'list' could be declared final 54
Parameter 'componentId' is not assigned and could be declared final 65
Local variable 'component' could be declared final 67
Local variable 'projectId' could be declared final 68

net/sourceforge/mystique/application/EditComponentForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 52
Avoid excessively long variable names like componentRepository 52
Parameter 'binder' is not assigned and could be declared final 55
Parameter 'model' is not assigned and could be declared final 62
Parameter 'componentId' is not assigned and could be declared final 62
Local variable 'component' could be declared final 64
Parameter 'result' is not assigned and could be declared final 73
Parameter 'component' is not assigned and could be declared final 73
Parameter 'status' is not assigned and could be declared final 74
A method should have only one exit point, and that should be the last statement in the method 78
A method should have only one exit point, and that should be the last statement in the method 85
Local variable 'projectId' could be declared final 89

net/sourceforge/mystique/application/EditMilestoneForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 52
Avoid excessively long variable names like milestoneRepository 52
Parameter 'binder' is not assigned and could be declared final 55
Parameter 'model' is not assigned and could be declared final 62
Parameter 'milestoneId' is not assigned and could be declared final 62
Local variable 'milestone' could be declared final 64
Parameter 'milestone' is not assigned and could be declared final 73
Parameter 'result' is not assigned and could be declared final 73
Parameter 'status' is not assigned and could be declared final 74
A method should have only one exit point, and that should be the last statement in the method 78
A method should have only one exit point, and that should be the last statement in the method 85
Local variable 'projectId' could be declared final 89

net/sourceforge/mystique/application/EditProjectForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 52
Parameter 'binder' is not assigned and could be declared final 55
Parameter 'model' is not assigned and could be declared final 62
Parameter 'projectId' is not assigned and could be declared final 62
Local variable 'project' could be declared final 64
Parameter 'project' is not assigned and could be declared final 73
Parameter 'result' is not assigned and could be declared final 73
Parameter 'status' is not assigned and could be declared final 73
A method should have only one exit point, and that should be the last statement in the method 78
A method should have only one exit point, and that should be the last statement in the method 85

net/sourceforge/mystique/application/EditTicketForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 51
Parameter 'binder' is not assigned and could be declared final 54
Parameter 'model' is not assigned and could be declared final 61
Parameter 'ticketId' is not assigned and could be declared final 61
Local variable 'ticket' could be declared final 63
Parameter 'result' is not assigned and could be declared final 72
Parameter 'ticket' is not assigned and could be declared final 72
Parameter 'status' is not assigned and could be declared final 72
A method should have only one exit point, and that should be the last statement in the method 77

net/sourceforge/mystique/application/EditVersionForm.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 52
Parameter 'binder' is not assigned and could be declared final 55
Parameter 'model' is not assigned and could be declared final 62
Parameter 'versionId' is not assigned and could be declared final 62
Local variable 'version' could be declared final 64
Parameter 'result' is not assigned and could be declared final 73
Parameter 'status' is not assigned and could be declared final 73
Parameter 'version' is not assigned and could be declared final 73
A method should have only one exit point, and that should be the last statement in the method 78
A method should have only one exit point, and that should be the last statement in the method 85
Local variable 'projectId' could be declared final 89

net/sourceforge/mystique/application/MilestoneController.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 45
Found non-transient, non-static member. Please mark as transient or provide accessors. 48
Avoid excessively long variable names like milestoneRepository 48
Parameter 'model' is not assigned and could be declared final 51
Parameter 'projectId' is not assigned and could be declared final 51
Local variable 'project' could be declared final 53
Local variable 'list' could be declared final 54
Parameter 'milestoneId' is not assigned and could be declared final 65
Local variable 'milestone' could be declared final 67
Local variable 'projectId' could be declared final 68

net/sourceforge/mystique/application/ProjectController.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 41
Parameter 'model' is not assigned and could be declared final 44
Parameter 'projectId' is not assigned and could be declared final 54
Local variable 'project' could be declared final 56

net/sourceforge/mystique/application/TicketController.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 41
Parameter 'model' is not assigned and could be declared final 44
Parameter 'ticketId' is not assigned and could be declared final 54
Local variable 'ticket' could be declared final 56

net/sourceforge/mystique/application/VersionController.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 45
Found non-transient, non-static member. Please mark as transient or provide accessors. 48
Parameter 'model' is not assigned and could be declared final 51
Parameter 'projectId' is not assigned and could be declared final 51
Local variable 'project' could be declared final 53
Local variable 'list' could be declared final 54
Parameter 'versionId' is not assigned and could be declared final 65
Local variable 'version' could be declared final 67
Local variable 'projectId' could be declared final 68

net/sourceforge/mystique/domain/MystiqueException.java

Violation Line
Parameter 'message' is not assigned and could be declared final 34
Parameter 'cause' is not assigned and could be declared final 38
Parameter 'message' is not assigned and could be declared final 42
Parameter 'cause' is not assigned and could be declared final 42

net/sourceforge/mystique/domain/entity/Attachment.java

Violation Line
Avoid variables with short names like id 59
Overridable method 'setCreated' called during object construction 86
Parameter 'inputStream' is not assigned and could be declared final 94
Parameter 'path' is not assigned and could be declared final 94
Avoid variables with short names like fs 96
Local variable 'fs' could be declared final 96
Local variable 'fileName' could be declared final 97
Local variable 'outputStream' could be declared final 99
Parameter 'outputStream' is not assigned and could be declared final 111
Parameter 'path' is not assigned and could be declared final 111
Avoid variables with short names like fs 113
Local variable 'fs' could be declared final 113
Local variable 'fileName' could be declared final 114
Local variable 'inputStream' could be declared final 116
Avoid variables with short names like id 132
Parameter 'id' is not assigned and could be declared final 132
Parameter 'name' is not assigned and could be declared final 140
Parameter 'description' is not assigned and could be declared final 151
Parameter 'mimeType' is not assigned and could be declared final 162
Parameter 'length' is not assigned and could be declared final 173
Parameter 'created' is not assigned and could be declared final 181
Parameter 'owner' is not assigned and could be declared final 189
Parameter 'ticket' is not assigned and could be declared final 197
Avoid using final local variables, turn them into fields 203
Found 'DD'-anomaly for variable 'result' (lines '204'-'205'). 204 - 205
Parameter 'obj' is not assigned and could be declared final 210
Avoid using if statements without curly braces 211 - 212
A method should have only one exit point, and that should be the last statement in the method 212
Avoid using if statements without curly braces 213 - 214
A method should have only one exit point, and that should be the last statement in the method 214
Avoid using if statements without curly braces 215 - 216
A method should have only one exit point, and that should be the last statement in the method 216
Local variable 'other' could be declared final 217
Avoid using if statements without curly braces 219 - 220
A method should have only one exit point, and that should be the last statement in the method 220
Avoid using if statements without curly braces 221 - 222
A method should have only one exit point, and that should be the last statement in the method 222

net/sourceforge/mystique/domain/entity/Change.java

Violation Line
Avoid variables with short names like id 48
Overridable method 'setCreated' called during object construction 63
Avoid variables with short names like id 70
Parameter 'id' is not assigned and could be declared final 70
Parameter 'created' is not assigned and could be declared final 78
Parameter 'responsible' is not assigned and could be declared final 86
Parameter 'ticket' is not assigned and could be declared final 94
Avoid using final local variables, turn them into fields 100
Found 'DD'-anomaly for variable 'result' (lines '101'-'102'). 101 - 102
Parameter 'obj' is not assigned and could be declared final 107
Avoid using if statements without curly braces 108 - 109
A method should have only one exit point, and that should be the last statement in the method 109
Avoid using if statements without curly braces 110 - 111
A method should have only one exit point, and that should be the last statement in the method 111
Avoid using if statements without curly braces 112 - 113
A method should have only one exit point, and that should be the last statement in the method 113
Local variable 'other' could be declared final 114
Avoid using if statements without curly braces 116 - 117
A method should have only one exit point, and that should be the last statement in the method 117
Avoid using if statements without curly braces 118 - 119
A method should have only one exit point, and that should be the last statement in the method 119

net/sourceforge/mystique/domain/entity/Comment.java

Violation Line
Avoid variables with short names like id 52
Overridable method 'setCreated' called during object construction 70
Avoid variables with short names like id 82
Parameter 'id' is not assigned and could be declared final 82
Parameter 'content' is not assigned and could be declared final 90
Parameter 'created' is not assigned and could be declared final 101
Parameter 'author' is not assigned and could be declared final 109
Parameter 'ticket' is not assigned and could be declared final 117
Avoid using final local variables, turn them into fields 123
Found 'DD'-anomaly for variable 'result' (lines '124'-'125'). 124 - 125
Parameter 'obj' is not assigned and could be declared final 130
Avoid using if statements without curly braces 131 - 132
A method should have only one exit point, and that should be the last statement in the method 132
Avoid using if statements without curly braces 133 - 134
A method should have only one exit point, and that should be the last statement in the method 134
Avoid using if statements without curly braces 135 - 136
A method should have only one exit point, and that should be the last statement in the method 136
Local variable 'other' could be declared final 137
Avoid using if statements without curly braces 139 - 140
A method should have only one exit point, and that should be the last statement in the method 140
Avoid using if statements without curly braces 141 - 142
A method should have only one exit point, and that should be the last statement in the method 142

net/sourceforge/mystique/domain/entity/Component.java

Violation Line
Avoid variables with short names like id 55
Overridable method 'setTicketsReported' called during object construction 68
Avoid variables with short names like id 80
Parameter 'id' is not assigned and could be declared final 80
Parameter 'name' is not assigned and could be declared final 88
Parameter 'ticketsReported' is not assigned and could be declared final 99
Parameter 'project' is not assigned and could be declared final 107
Avoid using final local variables, turn them into fields 113
Found 'DD'-anomaly for variable 'result' (lines '114'-'115'). 114 - 115
Parameter 'obj' is not assigned and could be declared final 120
Avoid using if statements without curly braces 121 - 122
A method should have only one exit point, and that should be the last statement in the method 122
Avoid using if statements without curly braces 123 - 124
A method should have only one exit point, and that should be the last statement in the method 124
Avoid using if statements without curly braces 125 - 126
A method should have only one exit point, and that should be the last statement in the method 126
Local variable 'other' could be declared final 127
Avoid using if statements without curly braces 129 - 130
A method should have only one exit point, and that should be the last statement in the method 130
Avoid using if statements without curly braces 131 - 132
A method should have only one exit point, and that should be the last statement in the method 132

net/sourceforge/mystique/domain/entity/Confirmation.java

Violation Line
Avoid variables with short names like id 54
Overridable method 'setCreated' called during object construction 69
Avoid variables with short names like id 81
Parameter 'id' is not assigned and could be declared final 81
Parameter 'created' is not assigned and could be declared final 89
Parameter 'responsible' is not assigned and could be declared final 97
Parameter 'ticket' is not assigned and could be declared final 105
Avoid using final local variables, turn them into fields 111
Found 'DD'-anomaly for variable 'result' (lines '112'-'113'). 112 - 113
Parameter 'obj' is not assigned and could be declared final 118
Avoid using if statements without curly braces 119 - 120
A method should have only one exit point, and that should be the last statement in the method 120
Avoid using if statements without curly braces 121 - 122
A method should have only one exit point, and that should be the last statement in the method 122
Avoid using if statements without curly braces 123 - 124
A method should have only one exit point, and that should be the last statement in the method 124
Local variable 'other' could be declared final 125
Avoid using if statements without curly braces 127 - 128
A method should have only one exit point, and that should be the last statement in the method 128
Avoid using if statements without curly braces 129 - 130
A method should have only one exit point, and that should be the last statement in the method 130

net/sourceforge/mystique/domain/entity/Milestone.java

Violation Line
Avoid variables with short names like id 58
Overridable method 'setTicketsReported' called during object construction 79
Avoid variables with short names like id 91
Parameter 'id' is not assigned and could be declared final 91
Parameter 'name' is not assigned and could be declared final 99
Parameter 'due' is not assigned and could be declared final 107
Parameter 'completed' is not assigned and could be declared final 115
Parameter 'ticketsReported' is not assigned and could be declared final 123
Parameter 'project' is not assigned and could be declared final 131
Avoid using final local variables, turn them into fields 137
Found 'DD'-anomaly for variable 'result' (lines '138'-'139'). 138 - 139
Parameter 'obj' is not assigned and could be declared final 144
Avoid using if statements without curly braces 145 - 146
A method should have only one exit point, and that should be the last statement in the method 146
Avoid using if statements without curly braces 147 - 148
A method should have only one exit point, and that should be the last statement in the method 148
Avoid using if statements without curly braces 149 - 150
A method should have only one exit point, and that should be the last statement in the method 150
Local variable 'other' could be declared final 151
Avoid using if statements without curly braces 153 - 154
A method should have only one exit point, and that should be the last statement in the method 154
Avoid using if statements without curly braces 155 - 156
A method should have only one exit point, and that should be the last statement in the method 156

net/sourceforge/mystique/domain/entity/Project.java

Violation Line
Avoid variables with short names like id 55
The String literal "project" appears 4 times in this file; the first occurrence is on line 60 60
Overridable method 'setTicketsReported' called during object construction 73
Overridable method 'setComponents' called during object construction 74
Overridable method 'setMilestones' called during object construction 75
Overridable method 'setVersions' called during object construction 76
Parameter 'component' is not assigned and could be declared final 84
Parameter 'milestone' is not assigned and could be declared final 94
Parameter 'version' is not assigned and could be declared final 104
Avoid variables with short names like id 118
Parameter 'id' is not assigned and could be declared final 118
Parameter 'name' is not assigned and could be declared final 126
Parameter 'ticketsReported' is not assigned and could be declared final 137
Parameter 'components' is not assigned and could be declared final 145
Parameter 'milestones' is not assigned and could be declared final 153
Parameter 'versions' is not assigned and could be declared final 161
Avoid using final local variables, turn them into fields 167
Found 'DD'-anomaly for variable 'result' (lines '168'-'169'). 168 - 169
Parameter 'obj' is not assigned and could be declared final 174
Avoid using if statements without curly braces 175 - 176
A method should have only one exit point, and that should be the last statement in the method 176
Avoid using if statements without curly braces 177 - 178
A method should have only one exit point, and that should be the last statement in the method 178
Avoid using if statements without curly braces 179 - 180
A method should have only one exit point, and that should be the last statement in the method 180
Local variable 'other' could be declared final 181
Avoid using if statements without curly braces 183 - 184
A method should have only one exit point, and that should be the last statement in the method 184
Avoid using if statements without curly braces 185 - 186
A method should have only one exit point, and that should be the last statement in the method 186

net/sourceforge/mystique/domain/entity/Ticket.java

Violation Line
This class has a bunch of public methods and attributes 21 - 421
Too many fields 69 - 421
Avoid variables with short names like id 74
Avoid excessively long variable names like qualityAssuranceUsers 142
Overridable method 'setReported' called during object construction 161
Overridable method 'setModified' called during object construction 162
Overridable method 'setDuplicates' called during object construction 163
Overridable method 'setUsersAssignedTo' called during object construction 164
Overridable method 'setQualityAssuranceUsers' called during object construction 165
Overridable method 'setAttachments' called during object construction 166
Overridable method 'setItDependsOn' called during object construction 167
Overridable method 'setDependsOnIt' called during object construction 168
Overridable method 'setComments' called during object construction 169
Overridable method 'setConfirmations' called during object construction 170
Parameter 'attachment' is not assigned and could be declared final 178
Parameter 'comment' is not assigned and could be declared final 183
Parameter 'confirmation' is not assigned and could be declared final 188
Avoid variables with short names like id 202
Parameter 'id' is not assigned and could be declared final 202
Parameter 'summary' is not assigned and could be declared final 210
Parameter 'description' is not assigned and could be declared final 221
Parameter 'reporter' is not assigned and could be declared final 232
Parameter 'reported' is not assigned and could be declared final 240
Parameter 'modified' is not assigned and could be declared final 248
Parameter 'project' is not assigned and could be declared final 256
Parameter 'component' is not assigned and could be declared final 264
Parameter 'milestone' is not assigned and could be declared final 272
Parameter 'version' is not assigned and could be declared final 280
Parameter 'severity' is not assigned and could be declared final 288
Parameter 'priority' is not assigned and could be declared final 296
Parameter 'duplicateOf' is not assigned and could be declared final 304
Parameter 'duplicates' is not assigned and could be declared final 312
Parameter 'type' is not assigned and could be declared final 320
Parameter 'status' is not assigned and could be declared final 328
Parameter 'resolution' is not assigned and could be declared final 336
Parameter 'usersAssignedTo' is not assigned and could be declared final 344
Avoid excessively long variable names like qualityAssuranceUsers 352
Parameter 'qualityAssuranceUsers' is not assigned and could be declared final 352
Parameter 'itDependsOn' is not assigned and could be declared final 360
Parameter 'dependsOnIt' is not assigned and could be declared final 368
Parameter 'attachments' is not assigned and could be declared final 376
Parameter 'comments' is not assigned and could be declared final 384
Parameter 'confirmations' is not assigned and could be declared final 392
Avoid using final local variables, turn them into fields 398
Found 'DD'-anomaly for variable 'result' (lines '399'-'400'). 399 - 400
Parameter 'obj' is not assigned and could be declared final 405
Avoid using if statements without curly braces 406 - 407
A method should have only one exit point, and that should be the last statement in the method 407
Avoid using if statements without curly braces 408 - 409
A method should have only one exit point, and that should be the last statement in the method 409
Avoid using if statements without curly braces 410 - 411
A method should have only one exit point, and that should be the last statement in the method 411
Local variable 'other' could be declared final 412
Avoid using if statements without curly braces 414 - 415
A method should have only one exit point, and that should be the last statement in the method 415
Avoid using if statements without curly braces 416 - 417
A method should have only one exit point, and that should be the last statement in the method 417

net/sourceforge/mystique/domain/entity/User.java

Violation Line
Avoid variables with short names like id 57
Overridable method 'setCreated' called during object construction 83
Overridable method 'setTicketsReported' called during object construction 84
Overridable method 'setTicketsAssigned' called during object construction 85
Overridable method 'setTicketsToApprove' called during object construction 86
Avoid variables with short names like id 98
Parameter 'id' is not assigned and could be declared final 98
Parameter 'name' is not assigned and could be declared final 106
Parameter 'email' is not assigned and could be declared final 117
Parameter 'password' is not assigned and could be declared final 128
Parameter 'created' is not assigned and could be declared final 139
Parameter 'ticketsReported' is not assigned and could be declared final 147
Parameter 'ticketsAssigned' is not assigned and could be declared final 155
Parameter 'ticketsToApprove' is not assigned and could be declared final 163
Avoid using final local variables, turn them into fields 169
Found 'DD'-anomaly for variable 'result' (lines '170'-'171'). 170 - 171
Parameter 'obj' is not assigned and could be declared final 176
Avoid using if statements without curly braces 177 - 178
A method should have only one exit point, and that should be the last statement in the method 178
Avoid using if statements without curly braces 179 - 180
A method should have only one exit point, and that should be the last statement in the method 180
Avoid using if statements without curly braces 181 - 182
A method should have only one exit point, and that should be the last statement in the method 182
Local variable 'other' could be declared final 183
Avoid using if statements without curly braces 185 - 186
A method should have only one exit point, and that should be the last statement in the method 186
Avoid using if statements without curly braces 187 - 188
A method should have only one exit point, and that should be the last statement in the method 188

net/sourceforge/mystique/domain/entity/Version.java

Violation Line
Avoid variables with short names like id 54
Overridable method 'setTicketsReported' called during object construction 67
Avoid variables with short names like id 79
Parameter 'id' is not assigned and could be declared final 79
Parameter 'number' is not assigned and could be declared final 87
Parameter 'ticketsReported' is not assigned and could be declared final 98
Parameter 'project' is not assigned and could be declared final 106
Avoid using final local variables, turn them into fields 112
Found 'DD'-anomaly for variable 'result' (lines '113'-'114'). 113 - 114
Parameter 'obj' is not assigned and could be declared final 119
Avoid using if statements without curly braces 120 - 121
A method should have only one exit point, and that should be the last statement in the method 121
Avoid using if statements without curly braces 122 - 123
A method should have only one exit point, and that should be the last statement in the method 123
Avoid using if statements without curly braces 124 - 125
A method should have only one exit point, and that should be the last statement in the method 125
Local variable 'other' could be declared final 126
Avoid using if statements without curly braces 128 - 129
A method should have only one exit point, and that should be the last statement in the method 129
Avoid using if statements without curly braces 130 - 131
A method should have only one exit point, and that should be the last statement in the method 131

net/sourceforge/mystique/domain/repository/Repository.java

Violation Line
Do not use the short type 30
Avoid modifiers which are implied by the context 30
Do not use the short type 32
Avoid modifiers which are implied by the context 32
Do not use the short type 34
Avoid modifiers which are implied by the context 34

net/sourceforge/mystique/domain/repository/RepositoryException.java

Violation Line
Parameter 'message' is not assigned and could be declared final 36
Parameter 'cause' is not assigned and could be declared final 40
Parameter 'message' is not assigned and could be declared final 44
Parameter 'cause' is not assigned and could be declared final 44

net/sourceforge/mystique/domain/repository/SimpleRepository.java

Violation Line
Avoid variables with short names like id 28

net/sourceforge/mystique/domain/valueobject/Configuration.java

Violation Line
Avoid excessively long variable names like attachmentsMaxLength 30
Parameter 'attachmentsPath' is not assigned and could be declared final 36
Avoid excessively long variable names like attachmentsMaxLength 44
Parameter 'attachmentsMaxLength' is not assigned and could be declared final 44

net/sourceforge/mystique/infrastructure/persistence/AttachmentJpaRepository.java

Violation Line
Avoid variables with short names like id 44
Parameter 'id' is not assigned and could be declared final 44
Parameter 'limit' is not assigned and could be declared final 53
Parameter 'ticket' is not assigned and could be declared final 53
Parameter 'offset' is not assigned and could be declared final 53
Local variable 'query' could be declared final 55
Parameter 'attachment' is not assigned and could be declared final 67

net/sourceforge/mystique/infrastructure/persistence/CommentJpaRepository.java

Violation Line
Avoid variables with short names like id 44
Parameter 'id' is not assigned and could be declared final 44
Parameter 'limit' is not assigned and could be declared final 53
Parameter 'ticket' is not assigned and could be declared final 53
Parameter 'offset' is not assigned and could be declared final 53
Local variable 'query' could be declared final 55
Parameter 'comment' is not assigned and could be declared final 67

net/sourceforge/mystique/infrastructure/persistence/ComponentJpaRepository.java

Violation Line
Avoid variables with short names like id 45
Parameter 'id' is not assigned and could be declared final 45
Parameter 'limit' is not assigned and could be declared final 54
Parameter 'project' is not assigned and could be declared final 54
Parameter 'offset' is not assigned and could be declared final 54
Local variable 'query' could be declared final 56
Parameter 'project' is not assigned and could be declared final 68
Parameter 'name' is not assigned and could be declared final 68
Local variable 'query' could be declared final 70
Parameter 'component' is not assigned and could be declared final 80
Local variable 'storedComponent' could be declared final 84

net/sourceforge/mystique/infrastructure/persistence/ConfirmationJpaRepository.java

Violation Line
Avoid variables with short names like id 46
Parameter 'id' is not assigned and could be declared final 46
Parameter 'limit' is not assigned and could be declared final 55
Parameter 'ticket' is not assigned and could be declared final 55
Parameter 'offset' is not assigned and could be declared final 55
Local variable 'query' could be declared final 57
Parameter 'responsible' is not assigned and could be declared final 69
Parameter 'ticket' is not assigned and could be declared final 69
Local variable 'query' could be declared final 71
Parameter 'confirmation' is not assigned and could be declared final 81
Avoid excessively long variable names like storedConfirmation 89
Local variable 'storedConfirmation' could be declared final 89 - 90

net/sourceforge/mystique/infrastructure/persistence/JpaRepository.java

Violation Line
Abstract classes should be named AbstractXXX 34 - 48
Found non-transient, non-static member. Please mark as transient or provide accessors. 37
Parameter 'entity' is not assigned and could be declared final 41

net/sourceforge/mystique/infrastructure/persistence/MilestoneJpaRepository.java

Violation Line
Avoid variables with short names like id 45
Parameter 'id' is not assigned and could be declared final 45
Parameter 'limit' is not assigned and could be declared final 54
Parameter 'project' is not assigned and could be declared final 54
Parameter 'offset' is not assigned and could be declared final 54
Local variable 'query' could be declared final 56
Parameter 'project' is not assigned and could be declared final 68
Parameter 'name' is not assigned and could be declared final 68
Local variable 'query' could be declared final 70
Parameter 'milestone' is not assigned and could be declared final 80
Local variable 'storedMilestone' could be declared final 84

net/sourceforge/mystique/infrastructure/persistence/ProjectJpaRepository.java

Violation Line
Avoid variables with short names like id 44
Parameter 'id' is not assigned and could be declared final 44
Parameter 'limit' is not assigned and could be declared final 53
Parameter 'offset' is not assigned and could be declared final 53
Local variable 'query' could be declared final 55
Parameter 'name' is not assigned and could be declared final 65
Local variable 'query' could be declared final 67
Parameter 'project' is not assigned and could be declared final 76
Local variable 'storedProject' could be declared final 80

net/sourceforge/mystique/infrastructure/persistence/TicketJpaRepository.java

Violation Line
Avoid variables with short names like id 42
Parameter 'id' is not assigned and could be declared final 42
Parameter 'limit' is not assigned and could be declared final 51
Parameter 'offset' is not assigned and could be declared final 51
Local variable 'query' could be declared final 53
Parameter 'ticket' is not assigned and could be declared final 63

net/sourceforge/mystique/infrastructure/persistence/UserJpaRepository.java

Violation Line
Avoid variables with short names like id 44
Parameter 'id' is not assigned and could be declared final 44
Parameter 'limit' is not assigned and could be declared final 53
Parameter 'offset' is not assigned and could be declared final 53
Local variable 'query' could be declared final 55
Parameter 'email' is not assigned and could be declared final 65
Local variable 'query' could be declared final 67
Parameter 'user' is not assigned and could be declared final 76
Local variable 'storedUser' could be declared final 80

net/sourceforge/mystique/infrastructure/persistence/VersionJpaRepository.java

Violation Line
Avoid variables with short names like id 46
Parameter 'id' is not assigned and could be declared final 46
Parameter 'limit' is not assigned and could be declared final 55
Parameter 'project' is not assigned and could be declared final 55
Parameter 'offset' is not assigned and could be declared final 55
Local variable 'query' could be declared final 57
Parameter 'project' is not assigned and could be declared final 69
Parameter 'number' is not assigned and could be declared final 69
Local variable 'query' could be declared final 71
Parameter 'version' is not assigned and could be declared final 81
Local variable 'storedVersion' could be declared final 85

net/sourceforge/mystique/infrastructure/validation/CommentValidator.java

Violation Line
Parameter 'clss' is not assigned and could be declared final 36
Parameter 'errors' is not assigned and could be declared final 43
Parameter 'object' is not assigned and could be declared final 43

net/sourceforge/mystique/infrastructure/validation/ComponentValidator.java

Violation Line
Parameter 'clss' is not assigned and could be declared final 36
Parameter 'errors' is not assigned and could be declared final 43
Parameter 'object' is not assigned and could be declared final 43

net/sourceforge/mystique/infrastructure/validation/ConfirmationValidator.java

Violation Line
Parameter 'clss' is not assigned and could be declared final 35
Parameter 'errors' is not assigned and could be declared final 42
Parameter 'object' is not assigned and could be declared final 42

net/sourceforge/mystique/infrastructure/validation/MilestoneValidator.java

Violation Line
Parameter 'clss' is not assigned and could be declared final 36
Parameter 'errors' is not assigned and could be declared final 43
Parameter 'object' is not assigned and could be declared final 43

net/sourceforge/mystique/infrastructure/validation/MultipartFileValidator.java

Violation Line
Private field 'configuration' could be made final; it is only initialized in the declaration or constructor. 34
Found non-transient, non-static member. Please mark as transient or provide accessors. 34
Parameter 'configuration' is not assigned and could be declared final 36
Parameter 'clss' is not assigned and could be declared final 42
Parameter 'errors' is not assigned and could be declared final 49
Parameter 'object' is not assigned and could be declared final 49
Local variable 'multipartFile' could be declared final 51

net/sourceforge/mystique/infrastructure/validation/ProjectValidator.java

Violation Line
Parameter 'clss' is not assigned and could be declared final 36
Parameter 'errors' is not assigned and could be declared final 43
Parameter 'object' is not assigned and could be declared final 43

net/sourceforge/mystique/infrastructure/validation/TicketValidator.java

Violation Line
Parameter 'clss' is not assigned and could be declared final 36
Parameter 'errors' is not assigned and could be declared final 43
Parameter 'object' is not assigned and could be declared final 43

net/sourceforge/mystique/infrastructure/validation/VersionValidator.java

Violation Line
Parameter 'clss' is not assigned and could be declared final 36
Parameter 'errors' is not assigned and could be declared final 43
Parameter 'object' is not assigned and could be declared final 43

net/sourceforge/mystique/infrastructure/xhtml/BaseTag.java

Violation Line
Local variable 'context' could be declared final 38
Local variable 'request' could be declared final 39
Local variable 'scheme' could be declared final 41
Local variable 'serverName' could be declared final 42
Local variable 'serverPort' could be declared final 43
Local variable 'contextPath' could be declared final 44
Local variable 'builder' could be declared final 46

net/sourceforge/mystique/infrastructure/xhtml/ColumnTag.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 34
Local variable 'tableTag' could be declared final 39
Local variable 'columnOut' could be declared final 42
Local variable 'builder' could be declared final 46
Parameter 'label' is not assigned and could be declared final 52

net/sourceforge/mystique/infrastructure/xhtml/ExceptionTag.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 35
Found non-transient, non-static member. Please mark as transient or provide accessors. 37
Found non-transient, non-static member. Please mark as transient or provide accessors. 39
Local variable 'exceptionOut' could be declared final 44
Local variable 'builder' could be declared final 48
Parameter 'attr' is not assigned and could be declared final 64
Parameter 'cssClass' is not assigned and could be declared final 68
Parameter 'cssStyle' is not assigned and could be declared final 72

net/sourceforge/mystique/infrastructure/xhtml/TableTag.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 38
Found non-transient, non-static member. Please mark as transient or provide accessors. 40
Found non-transient, non-static member. Please mark as transient or provide accessors. 42
Found non-transient, non-static member. Please mark as transient or provide accessors. 44
Found non-transient, non-static member. Please mark as transient or provide accessors. 46
Found non-transient, non-static member. Please mark as transient or provide accessors. 48
Private field 'labels' could be made final; it is only initialized in the declaration or constructor. 50
Found non-transient, non-static member. Please mark as transient or provide accessors. 50
Private field 'rows' could be made final; it is only initialized in the declaration or constructor. 52
Found non-transient, non-static member. Please mark as transient or provide accessors. 52
Local variable 'itemIterator' could be declared final 57
Local variable 'builder' could be declared final 60
Local variable 'rowOut' could be declared final 70
Avoid instantiating new objects inside loops 70
Local variable 'builder' could be declared final 82
Local variable 'labelIterator' could be declared final 94
Local variable 'rowIterator' could be declared final 103
Found 'DU'-anomaly for variable 'i' (lines '104'-'124'). 104 - 124
Avoid variables with short names like i 104
Parameter 'var' is not assigned and could be declared final 126
Parameter 'items' is not assigned and could be declared final 130
Parameter 'noItems' is not assigned and could be declared final 134
Parameter 'cssClass' is not assigned and could be declared final 138
Parameter 'cssStyle' is not assigned and could be declared final 142
Parameter 'label' is not assigned and could be declared final 146