PMD Results
The following document contains the results of PMD 4.2.5.
Files
com/vaadin/client/ui/VScrollTablePatched.java
| Violation | Line |
|---|
| Avoid unused private fields such as 'initialContentReceived'. | 488 |
| Avoid unused private fields such as 'touchScrollDelegate'. | 524 |
| Avoid empty if statements | 2100 - 2102 |
| These nested if statements could be combined | 2169 - 2182 |
| These nested if statements could be combined | 2171 - 2181 |
| These nested if statements could be combined | 3352 - 3355 |
| These nested if statements could be combined | 3534 - 3540 |
| These nested if statements could be combined | 4223 - 4226 |
| Avoid unused local variables such as 'thisTrElement'. | 6016 |
| Avoid unused method parameters such as 'rowUidl'. | 6280 - 6285 |
| Avoid unused method parameters such as 'rowUidl'. | 6287 - 6295 |
| These nested if statements could be combined | 6823 - 6832 |
| Ensure you override both equals() and hashCode() | 6982 |
com/vaadin/client/ui/table/TableConnectorPatched.java
| Violation | Line |
|---|
| These nested if statements could be combined | 266 - 286 |
info/magnolia/ui/vaadin/dialog/ConfirmationDialog.java
| Violation | Line |
|---|
| Overriding method merely calls super | 124 - 126 |
info/magnolia/ui/vaadin/editor/CroppableImage.java
| Violation | Line |
|---|
| Overriding method merely calls super | 62 - 64 |
info/magnolia/ui/vaadin/editor/JCrop.java
| Violation | Line |
|---|
| Avoid unused constructor parameters such as 'handler'. | 62 - 86 |
info/magnolia/ui/vaadin/form/Form.java
| Violation | Line |
|---|
| Avoid unused private fields such as 'isValidationVisible'. | 63 |
info/magnolia/ui/vaadin/grid/MagnoliaTreeTable.java
| Violation | Line |
|---|
| Overriding method merely calls super | 106 - 108 |
info/magnolia/ui/vaadin/gwt/client/actionbar/connector/ActionbarConnector.java
| Violation | Line |
|---|
| Overriding method merely calls super | 154 - 156 |
info/magnolia/ui/vaadin/gwt/client/applauncher/connector/AppLauncherConnector.java
| Violation | Line |
|---|
| Overriding method merely calls super | 111 - 113 |
info/magnolia/ui/vaadin/gwt/client/form/formsection/connector/FormSectionConnector.java
| Violation | Line |
|---|
| Overriding method merely calls super | 160 - 162 |
info/magnolia/ui/vaadin/gwt/client/form/tab/connector/FormTabConnector.java
| Violation | Line |
|---|
| Overriding method merely calls super | 48 - 50 |
info/magnolia/ui/vaadin/gwt/client/grid/VMagnoliaTable.java
| Violation | Line |
|---|
| Overriding method merely calls super | 174 - 176 |
info/magnolia/ui/vaadin/gwt/client/jquerywrapper/JQueryFunction.java
| Violation | Line |
|---|
| Unnecessary final modifier in final class | 46 - 50 |
info/magnolia/ui/vaadin/gwt/client/layout/CssRule.java
| Violation | Line |
|---|
| Avoid unused private fields such as 'rules'. | 45 |
| Avoid unused private methods such as 'searchForRule(JavaScriptObject,String,boolean)'. | 77 - 79 |
info/magnolia/ui/vaadin/gwt/client/magnoliashell/shell/MagnoliaShellViewImpl.java
| Violation | Line |
|---|
| These nested if statements could be combined | 206 - 208 |
info/magnolia/ui/vaadin/gwt/client/magnoliashell/viewport/connector/AppsViewportConnector.java
| Violation | Line |
|---|
| These nested if statements could be combined | 94 - 96 |
info/magnolia/ui/vaadin/gwt/client/richtext/VMagnoliaRichTextField.java
| Violation | Line |
|---|
| Avoid unused private methods such as 'getPaintableId()'. | 204 |
info/magnolia/ui/vaadin/gwt/client/tabsheet/tab/widget/MagnoliaTabWidget.java
| Violation | Line |
|---|
| Avoid unused constructor parameters such as 'connector'. | 48 - 52 |
info/magnolia/ui/vaadin/gwt/client/tabsheet/widget/MagnoliaTabSheetViewImpl.java
| Violation | Line |
|---|
| An empty statement (semicolon) not part of a loop | 241 |
org/vaadin/peter/contextmenu/client/ContextMenuItemWidgetHandler.java
| Violation | Line |
|---|
| These nested if statements could be combined | 112 - 114 |
