magnolia-ui-vaadin-common-widgets Version: 5.0-alpha1-rc1 Last Published: Dec 10, 2012

PMD Results

The following document contains the results of PMD 4.2.5.

Files

com/vaadin/terminal/gwt/client/ui/VScrollTablePatched.java

ViolationLine
Avoid unused private fields such as 'touchScrollDelegate'.465
These nested if statements could be combined1066 - 1069
Avoid empty if statements2049 - 2051
These nested if statements could be combined2120 - 2135
These nested if statements could be combined2122 - 2134
These nested if statements could be combined3088 - 3091
These nested if statements could be combined3269 - 3275
These nested if statements could be combined3913 - 3916
Avoid unused method parameters such as 'rowUidl'.5934 - 5939
Avoid unused method parameters such as 'rowUidl'.5941 - 5949
These nested if statements could be combined6394 - 6401
Ensure you override both equals() and hashCode()6547

info/magnolia/ui/vaadin/actionbar/Actionbar.java

ViolationLine
Avoid unused private fields such as 'opened'.77
Overriding method merely calls super171 - 173

info/magnolia/ui/vaadin/grid/MagnoliaTable.java

ViolationLine
Overriding method merely calls super88 - 90

info/magnolia/ui/vaadin/gwt/client/actionbar/VActionbarItemJSO.java

ViolationLine
Unnecessary final modifier in final class53 - 55
Unnecessary final modifier in final class57 - 59
Unnecessary final modifier in final class61 - 63
Unnecessary final modifier in final class65 - 67
Unnecessary final modifier in final class69 - 71

info/magnolia/ui/vaadin/gwt/client/actionbar/VActionbarSectionJSO.java

ViolationLine
Unnecessary final modifier in final class53 - 55
Unnecessary final modifier in final class57 - 59
Unnecessary final modifier in final class61 - 63

info/magnolia/ui/vaadin/gwt/client/applauncher/VAppGroupJSO.java

ViolationLine
Unnecessary final modifier in final class55 - 57
Unnecessary final modifier in final class59 - 61
Unnecessary final modifier in final class63 - 65
Unnecessary final modifier in final class67 - 69
Unnecessary final modifier in final class71 - 73

info/magnolia/ui/vaadin/gwt/client/editor/GWTSelector.java

ViolationLine
Avoid unused method parameters such as 'selectionContainer'.239 - 259

info/magnolia/ui/vaadin/gwt/client/editor/VImageEditor.java

ViolationLine
Avoid unused local variables such as 'minDimension'.166

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

ViolationLine
Overriding method merely calls super165 - 167

info/magnolia/ui/vaadin/gwt/client/icon/GwtBadgeIcon.java

ViolationLine
Avoid unused private fields such as 'SIZE_DEFAULT'.49

info/magnolia/ui/vaadin/gwt/client/icon/VBadgeIcon.java

ViolationLine
Avoid unused private fields such as 'client'.46
Avoid unused private fields such as 'paintableId'.48

info/magnolia/ui/vaadin/gwt/client/icon/VCompositeIcon.java

ViolationLine
Avoid unused private fields such as 'client'.48
Avoid unused private fields such as 'paintableId'.50

info/magnolia/ui/vaadin/gwt/client/icon/VIcon.java

ViolationLine
Avoid unused private fields such as 'client'.46
Avoid unused private fields such as 'paintableId'.48

info/magnolia/ui/vaadin/gwt/client/icon/VLoadingIcon.java

ViolationLine
Avoid unused private fields such as 'client'.46
Avoid unused private fields such as 'paintableId'.48

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

ViolationLine
Unnecessary final modifier in final class45 - 49

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

ViolationLine
Avoid unused local variables such as 'margins'.182

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

ViolationLine
Avoid unused private fields such as 'client'.57

info/magnolia/ui/vaadin/gwt/client/magnoliashell/VMagnoliaShellViewImpl.java

ViolationLine
These nested if statements could be combined192 - 194

info/magnolia/ui/vaadin/gwt/client/magnoliashell/viewport/VAppsViewport.java

ViolationLine
These nested if statements could be combined173 - 175

info/magnolia/ui/vaadin/gwt/client/previewtoken/VPreviewTokenField.java

ViolationLine
Avoid empty if statements62 - 63

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

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

info/magnolia/ui/vaadin/tabsheet/MagnoliaTab.java

ViolationLine
Overriding method merely calls super131 - 133
Overriding method merely calls super136 - 138

info/magnolia/ui/vaadin/tabsheet/MagnoliaTabSheet.java

ViolationLine
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.135

info/magnolia/ui/vaadin/tabsheet/SimplePanel.java

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