PMD Results
The following document contains the results of PMD 5.1.2.
Files
info/magnolia/ui/vaadin/editor/PageEditorListener.java
Violation |
Line |
Avoid modifiers which are implied by the context |
45 |
Avoid modifiers which are implied by the context |
46 |
Avoid modifiers which are implied by the context |
47 |
Avoid modifiers which are implied by the context |
48 |
Avoid modifiers which are implied by the context |
49 |
Avoid modifiers which are implied by the context |
50 |
Avoid modifiers which are implied by the context |
51 |
Avoid modifiers which are implied by the context |
54 |
Avoid modifiers which are implied by the context |
55 |
Avoid modifiers which are implied by the context |
56 |
info/magnolia/ui/vaadin/gwt/client/connector/PageEditorConnector.java
Violation |
Line |
Avoid unused private methods such as 'processDocument(Node,MgnlElement)'. |
375 |
info/magnolia/ui/vaadin/gwt/client/editor/dom/CmsNode.java
Violation |
Line |
Useless parentheses. |
100 |
info/magnolia/ui/vaadin/gwt/client/editor/dom/MgnlArea.java
Violation |
Line |
Useless parentheses. |
165 |
These nested if statements could be combined |
165–167 |
Useless parentheses. |
179 |
Useless parentheses. |
184 |
info/magnolia/ui/vaadin/gwt/client/editor/dom/MgnlElement.java
Violation |
Line |
Useless parentheses. |
121 |
info/magnolia/ui/vaadin/gwt/client/editor/dom/processor/AreaProcessor.java
Violation |
Line |
Useless parentheses. |
155 |
info/magnolia/ui/vaadin/gwt/client/editor/dom/processor/ElementProcessor.java
Violation |
Line |
Avoid unused private methods such as 'navigate(String)'. |
127 |
info/magnolia/ui/vaadin/gwt/client/editor/event/ComponentStartMoveEvent.java
Violation |
Line |
Avoid modifiers which are implied by the context |
69–71 |
info/magnolia/ui/vaadin/gwt/client/editor/event/ComponentStopMoveEvent.java
Violation |
Line |
Avoid modifiers which are implied by the context |
100–102 |
info/magnolia/ui/vaadin/gwt/client/editor/event/SelectElementEventHandler.java
Violation |
Line |
Avoid modifiers which are implied by the context |
42 |
info/magnolia/ui/vaadin/gwt/client/editor/jsni/WebkitFrameEventHandler.java
Violation |
Line |
Avoid unused private fields such as 'touchStartY'. |
49 |
info/magnolia/ui/vaadin/gwt/client/editor/model/focus/FocusModelImpl.java
Violation |
Line |
Useless parentheses. |
207 |
info/magnolia/ui/vaadin/gwt/client/widget/controlbar/AbstractBar.java
Violation |
Line |
Overriding method merely calls super |
138–140 |
info/magnolia/ui/vaadin/gwt/client/widget/controlbar/AreaEndBar.java
Violation |
Line |
Overriding method merely calls super |
49–51 |
info/magnolia/ui/vaadin/gwt/client/widget/controlbar/ComponentPlaceHolder.java
Violation |
Line |
Overriding method merely calls super |
93–95 |
info/magnolia/ui/vaadin/gwt/client/widget/controlbar/SimpleBar.java
Violation |
Line |
Useless parentheses. |
62 |
info/magnolia/ui/vaadin/gwt/client/widget/dnd/MoveWidget.java
Violation |
Line |
Useless parentheses. |
97 |
Useless parentheses. |
98 |
Overriding method merely calls super |
130–132 |
Overriding method merely calls super |
135–137 |