PMD Results

The following document contains the results of PMD 4.1.

Files

org/paneris/melati/site/Display.java

Violation Line
Found 'DD'-anomaly for variable 'id' (lines '79'-'81'). 79 - 81
Found 'DD'-anomaly for variable 'id' (lines '81'-'89'). 81 - 89
Found 'DD'-anomaly for variable 'value' (lines '85'-'86'). 85 - 86
Found 'DD'-anomaly for variable 'url' (lines '103'-'104'). 103 - 104
A method should have only one exit point, and that should be the last statement in the method 107
A method should have only one exit point, and that should be the last statement in the method 115
A method should have only one exit point, and that should be the last statement in the method 119
A method should have only one exit point, and that should be the last statement in the method 123
A method should have only one exit point, and that should be the last statement in the method 128
Found 'DD'-anomaly for variable 'url' (lines '132'-'133'). 132 - 133
A method should have only one exit point, and that should be the last statement in the method 134
A method should have only one exit point, and that should be the last statement in the method 135
A method should have only one exit point, and that should be the last statement in the method 139
A method should have only one exit point, and that should be the last statement in the method 140

org/paneris/melati/site/Populate.java

Violation Line
Avoid unnecessary return statements 293

org/paneris/melati/site/SiteServlet.java

Violation Line
A method should have only one exit point, and that should be the last statement in the method 84
A method should have only one exit point, and that should be the last statement in the method 86
Found 'DD'-anomaly for variable 'value' (lines '176'-'177'). 176 - 177
Found 'DU'-anomaly for variable 'value' (lines '177'-'203'). 177 - 203
A method should have only one exit point, and that should be the last statement in the method 340

org/paneris/melati/site/UploadedFileFormDataAdaptorFactory.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 66 - 67
It is a good practice to call super() in a constructor 66 - 67
A method should have only one exit point, and that should be the last statement in the method 83 - 84

org/paneris/melati/site/model/Div.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 46
It is a good practice to call super() in a constructor 46

org/paneris/melati/site/model/DivTable.java

Violation Line
Found 'DD'-anomaly for variable 'c' (lines '91'-'94'). 91 - 94
Found 'DD'-anomaly for variable 'c' (lines '94'-'94'). 94

org/paneris/melati/site/model/Link.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 44
It is a good practice to call super() in a constructor 44

org/paneris/melati/site/model/LinkType.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 39
It is a good practice to call super() in a constructor 39

org/paneris/melati/site/model/Page.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 67 - 68
It is a good practice to call super() in a constructor 67 - 68
A method should have only one exit point, and that should be the last statement in the method 145
Found 'DD'-anomaly for variable 'children' (lines '159'-'168'). 159 - 168
Found 'DD'-anomaly for variable 'j' (lines '162'-'165'). 162 - 165
Found 'DD'-anomaly for variable 'j' (lines '165'-'165'). 165
Found 'DD'-anomaly for variable 'children' (lines '168'-'172'). 168 - 172
Found 'DU'-anomaly for variable 'k' (lines '169'-'177'). 169 - 177
Found 'DD'-anomaly for variable 'children' (lines '172'-'172'). 172
Found 'DU'-anomaly for variable 'k' (lines '173'-'177'). 173 - 177

org/paneris/melati/site/model/PageTable.java

Violation Line
Use explicit scoping instead of the default package private level 64
A method should have only one exit point, and that should be the last statement in the method 159

org/paneris/melati/site/model/Setting.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 27
It is a good practice to call super() in a constructor 27

org/paneris/melati/site/model/Style.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 37
It is a good practice to call super() in a constructor 37

org/paneris/melati/site/model/Template.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 38
It is a good practice to call super() in a constructor 38

org/paneris/melati/site/model/UploadedDocument.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 38
It is a good practice to call super() in a constructor 38

org/paneris/melati/site/model/UploadedFile.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 40
It is a good practice to call super() in a constructor 40

org/paneris/melati/site/model/UploadedFileTable.java

Violation Line
Found 'DD'-anomaly for variable 'userTroid' (lines '70'-'72'). 70 - 72

org/paneris/melati/site/model/UploadedFileType.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 37
It is a good practice to call super() in a constructor 37

org/paneris/melati/site/model/UploadedImage.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 42
It is a good practice to call super() in a constructor 42