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)'. |
331 |
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. |
167 |
These nested if statements could be combined |
167–169 |
Useless parentheses. |
183 |
Useless parentheses. |
189 |
info/magnolia/ui/vaadin/gwt/client/editor/dom/MgnlElement.java
Violation |
Line |
Useless parentheses. |
120 |
info/magnolia/ui/vaadin/gwt/client/editor/dom/processor/AreaProcessor.java
Violation |
Line |
Useless parentheses. |
134 |
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. |
212 |
Useless parentheses. |
222 |
info/magnolia/ui/vaadin/gwt/client/widget/PageEditorViewImpl.java
Violation |
Line |
Avoid unused private fields such as 'url'. |
61 |
info/magnolia/ui/vaadin/gwt/client/widget/controlbar/AbstractBar.java
Violation |
Line |
Overriding method merely calls super |
140–142 |
Useless parentheses. |
168 |
info/magnolia/ui/vaadin/gwt/client/widget/controlbar/AreaEndBar.java
Violation |
Line |
Overriding method merely calls super |
75–77 |
info/magnolia/ui/vaadin/gwt/client/widget/controlbar/ComponentPlaceHolder.java
Violation |
Line |
Overriding method merely calls super |
90–92 |
info/magnolia/ui/vaadin/gwt/client/widget/dnd/MoveWidget.java
Violation |
Line |
Useless parentheses. |
93 |
Useless parentheses. |
94 |
Overriding method merely calls super |
126–128 |
Overriding method merely calls super |
131–133 |