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 |
44 |
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 |
53 |
Avoid modifiers which are implied by the context |
55 |
Avoid modifiers which are implied by the context |
56 |
Avoid modifiers which are implied by the context |
59 |
Avoid modifiers which are implied by the context |
60 |
Avoid modifiers which are implied by the context |
62 |
Avoid modifiers which are implied by the context |
63 |
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. |
163 |
These nested if statements could be combined |
163–165 |
Useless parentheses. |
179 |
Useless parentheses. |
185 |
info/magnolia/ui/vaadin/gwt/client/editor/dom/MgnlElement.java
Violation |
Line |
Useless parentheses. |
118 |
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/AbstractFrameEventHandler.java
Violation |
Line |
Avoid unused private methods such as 'onPageEditorReady()'. |
66 |
info/magnolia/ui/vaadin/gwt/client/editor/jsni/JavascriptUtils.java
Violation |
Line |
Avoid unused private fields such as 'EDITOR_COOKIE_PATH'. |
65 |
Avoid unused private fields such as 'editorPositionCookieName'. |
67 |
Avoid unused private fields such as 'editorContentIdCookieName'. |
68 |
Empty initializer was found |
70–74 |
Empty static initializer was found |
70–74 |
info/magnolia/ui/vaadin/gwt/client/editor/jsni/ReadyStateWatch.java
Violation |
Line |
Avoid unused private methods such as 'fireReadyStateChange()'. |
96 |
info/magnolia/ui/vaadin/gwt/client/editor/jsni/WebkitFrameEventHandler.java
Violation |
Line |
Avoid unused private fields such as 'touchStartY'. |
57 |
info/magnolia/ui/vaadin/gwt/client/editor/model/focus/FocusModelImpl.java
Violation |
Line |
Useless parentheses. |
158 |
Useless parentheses. |
168 |
info/magnolia/ui/vaadin/gwt/client/widget/PageEditorViewImpl.java
Violation |
Line |
Avoid unused private fields such as 'url'. |
59 |
info/magnolia/ui/vaadin/gwt/client/widget/controlbar/AbstractBar.java
Violation |
Line |
Overriding method merely calls super |
105–107 |
Useless parentheses. |
133 |
info/magnolia/ui/vaadin/gwt/client/widget/controlbar/AreaEndBar.java
Violation |
Line |
Overriding method merely calls super |
64–66 |
info/magnolia/ui/vaadin/gwt/client/widget/controlbar/ComponentPlaceHolder.java
Violation |
Line |
Overriding method merely calls super |
89–91 |
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 |