PMD Results

The following document contains the results of PMD 4.2.5.

Files

com/vaadin/client/ui/VScrollTablePatched.java

ViolationLine
Avoid unused private fields such as 'initialContentReceived'.488
Avoid unused private fields such as 'touchScrollDelegate'.524
Avoid empty if statements2100 - 2102
These nested if statements could be combined2169 - 2182
These nested if statements could be combined2171 - 2181
These nested if statements could be combined3352 - 3355
These nested if statements could be combined3534 - 3540
These nested if statements could be combined4223 - 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 combined6823 - 6832
Ensure you override both equals() and hashCode()6982

com/vaadin/client/ui/table/TableConnectorPatched.java

ViolationLine
These nested if statements could be combined266 - 286

info/magnolia/ui/vaadin/dialog/ConfirmationDialog.java

ViolationLine
Overriding method merely calls super126 - 128

info/magnolia/ui/vaadin/editor/CroppableImage.java

ViolationLine
Overriding method merely calls super62 - 64

info/magnolia/ui/vaadin/editor/JCrop.java

ViolationLine
Avoid unused constructor parameters such as 'handler'.62 - 86

info/magnolia/ui/vaadin/form/Form.java

ViolationLine
Avoid unused private fields such as 'isValidationVisible'.63

info/magnolia/ui/vaadin/gwt/client/actionbar/connector/ActionbarConnector.java

ViolationLine
Overriding method merely calls super154 - 156

info/magnolia/ui/vaadin/gwt/client/applauncher/connector/AppLauncherConnector.java

ViolationLine
Overriding method merely calls super111 - 113

info/magnolia/ui/vaadin/gwt/client/dialog/widget/DialogHeaderWidget.java

ViolationLine
These nested if statements could be combined95 - 97

info/magnolia/ui/vaadin/gwt/client/form/formsection/connector/FormSectionConnector.java

ViolationLine
Overriding method merely calls super160 - 162

info/magnolia/ui/vaadin/gwt/client/form/tab/connector/FormTabConnector.java

ViolationLine
Overriding method merely calls super48 - 50

info/magnolia/ui/vaadin/gwt/client/grid/VMagnoliaTable.java

ViolationLine
Overriding method merely calls super174 - 176

info/magnolia/ui/vaadin/gwt/client/jquerywrapper/JQueryFunction.java

ViolationLine
Unnecessary final modifier in final class46 - 50

info/magnolia/ui/vaadin/gwt/client/layout/CssRule.java

ViolationLine
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

ViolationLine
Avoid unused local variables such as 'viewportWidget'.232

info/magnolia/ui/vaadin/gwt/client/magnoliashell/shell/MagnoliaShellViewImpl.java

ViolationLine
These nested if statements could be combined206 - 208

info/magnolia/ui/vaadin/gwt/client/magnoliashell/viewport/connector/AppsViewportConnector.java

ViolationLine
These nested if statements could be combined94 - 96

info/magnolia/ui/vaadin/gwt/client/magnoliashell/viewport/widget/AppsViewportWidget.java

ViolationLine
Avoid unused local variables such as 'style'.186

info/magnolia/ui/vaadin/gwt/client/richtext/VMagnoliaRichTextField.java

ViolationLine
Avoid unused private methods such as 'getPaintableId()'.204

info/magnolia/ui/vaadin/gwt/client/tabsheet/tab/widget/MagnoliaTabWidget.java

ViolationLine
Avoid unused constructor parameters such as 'connector'.48 - 52

info/magnolia/ui/vaadin/gwt/client/tabsheet/widget/MagnoliaTabSheetViewImpl.java

ViolationLine
An empty statement (semicolon) not part of a loop241

org/vaadin/peter/contextmenu/client/ContextMenuItemWidgetHandler.java

ViolationLine
These nested if statements could be combined112 - 114