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'. | 486 |
| Avoid unused private fields such as 'touchScrollDelegate'. | 522 |
| Avoid empty if statements | 2087 - 2089 |
| These nested if statements could be combined | 2165 - 2178 |
| These nested if statements could be combined | 2167 - 2177 |
| These nested if statements could be combined | 3344 - 3347 |
| These nested if statements could be combined | 3526 - 3532 |
| These nested if statements could be combined | 4215 - 4218 |
| Avoid unused local variables such as 'thisTrElement'. | 6008 |
| Avoid unused method parameters such as 'rowUidl'. | 6266 - 6271 |
| Avoid unused method parameters such as 'rowUidl'. | 6273 - 6281 |
| These nested if statements could be combined | 6809 - 6818 |
| Ensure you override both equals() and hashCode() | 6968 |
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 | 118 - 120 |
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/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 | 123 - 125 |
info/magnolia/ui/vaadin/gwt/client/form/formsection/connector/FormSectionConnector.java
| Violation | Line |
|---|
| Overriding method merely calls super | 153 - 155 |
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()'. | 183 |
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 | 228 |
org/vaadin/peter/contextmenu/client/ContextMenuItemWidgetHandler.java
| Violation | Line |
|---|
| These nested if statements could be combined | 112 - 114 |
