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 | 128 - 130 |
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 | 75 - 77 |
| Overriding method merely calls super | 116 - 118 |
info/magnolia/ui/vaadin/gwt/client/applauncher/connector/AppLauncherConnector.java
| Violation | Line |
|---|
| Overriding method merely calls super | 143 - 145 |
info/magnolia/ui/vaadin/gwt/client/dialog/widget/DialogHeaderWidget.java
| Violation | Line |
|---|
| These nested if statements could be combined | 124 - 126 |
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 | 173 - 175 |
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/MagnoliaShellConnector.java
| Violation | Line |
|---|
| Avoid empty if statements | 260 - 262 |
info/magnolia/ui/vaadin/gwt/client/magnoliashell/shell/MagnoliaShellViewImpl.java
| Violation | Line |
|---|
| These nested if statements could be combined | 215 - 217 |
info/magnolia/ui/vaadin/gwt/client/magnoliashell/viewport/widget/AppsViewportWidget.java
| Violation | Line |
|---|
| Avoid unused local variables such as 'style'. | 195 |
info/magnolia/ui/vaadin/gwt/client/richtext/VMagnoliaRichTextField.java
| Violation | Line |
|---|
| Overriding method merely calls super | 160 - 162 |
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 |
|---|
| Avoid unused private fields such as 'eventBus'. | 96 |
| An empty statement (semicolon) not part of a loop | 252 |
org/vaadin/openesignforms/ckeditor/widgetset/client/ui/VCKEditorTextField.java
| Violation | Line |
|---|
| Avoid unused imports such as 'com.vaadin.client.VConsole' | 26 |
| These nested if statements could be combined | 305 - 307 |
| Avoid unnecessary return statements | 448 |
org/vaadin/peter/contextmenu/client/ContextMenuItemWidgetHandler.java
| Violation | Line |
|---|
| These nested if statements could be combined | 112 - 114 |
