PMD Results
The following document contains the results of PMD 6.4.0.
Files
com/googlecode/mgwt/collection/client/JsLightMap.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
56–58 |
com/googlecode/mgwt/collection/shared/LightArray.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
33 |
| Avoid modifiers which are implied by the context |
3 |
41 |
| Avoid modifiers which are implied by the context |
3 |
48 |
| Avoid modifiers which are implied by the context |
3 |
62 |
| Avoid modifiers which are implied by the context |
3 |
69 |
com/googlecode/mgwt/collection/shared/LightArrayBoolean.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
30 |
| Avoid modifiers which are implied by the context |
3 |
38 |
| Avoid modifiers which are implied by the context |
3 |
46 |
| Avoid modifiers which are implied by the context |
3 |
53 |
| Avoid modifiers which are implied by the context |
3 |
60 |
| Avoid modifiers which are implied by the context |
3 |
67 |
com/googlecode/mgwt/collection/shared/LightArrayInt.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
30 |
| Avoid modifiers which are implied by the context |
3 |
38 |
| Avoid modifiers which are implied by the context |
3 |
46 |
| Avoid modifiers which are implied by the context |
3 |
53 |
| Avoid modifiers which are implied by the context |
3 |
60 |
| Avoid modifiers which are implied by the context |
3 |
67 |
com/googlecode/mgwt/collection/shared/LightMap.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
31 |
| Avoid modifiers which are implied by the context |
3 |
39 |
| Avoid modifiers which are implied by the context |
3 |
46 |
| Avoid modifiers which are implied by the context |
3 |
53 |
| Avoid modifiers which are implied by the context |
3 |
61 |
| Avoid modifiers which are implied by the context |
3 |
69 |
com/googlecode/mgwt/dom/client/event/EventTypes.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
4 |
| Avoid modifiers which are implied by the context |
3 |
6 |
com/googlecode/mgwt/dom/client/event/mouse/SimulatedTouchEndEvent.java
| Violation |
Priority |
Line |
| Avoid unused private fields such as 'x'. |
3 |
34 |
| Avoid unused private fields such as 'y'. |
3 |
35 |
| Avoid unused private fields such as 'x_start'. |
3 |
36 |
| Avoid unused private fields such as 'y_start'. |
3 |
37 |
| Avoid unused private fields such as 'multiTouch'. |
3 |
38 |
com/googlecode/mgwt/dom/client/event/mouse/SimulatedTouchMoveEvent.java
| Violation |
Priority |
Line |
| Avoid unused private fields such as 'x'. |
3 |
34 |
| Avoid unused private fields such as 'y'. |
3 |
35 |
| Avoid unused private fields such as 'x_start'. |
3 |
37 |
| Avoid unused private fields such as 'y_start'. |
3 |
38 |
| Avoid unused private fields such as 'multiTouch'. |
3 |
39 |
com/googlecode/mgwt/dom/client/event/tap/HasTapHandlers.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
34 |
com/googlecode/mgwt/dom/client/event/tap/TapEvent.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' |
4 |
56 |
com/googlecode/mgwt/dom/client/event/tap/TapHandler.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
27 |
com/googlecode/mgwt/dom/client/event/touch/HasTouchHandlers.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
39 |
| Avoid modifiers which are implied by the context |
3 |
51 |
| Avoid modifiers which are implied by the context |
3 |
63 |
| Avoid modifiers which are implied by the context |
3 |
75 |
| Avoid modifiers which are implied by the context |
3 |
87 |
com/googlecode/mgwt/dom/client/event/touch/JsTouch.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
34–36 |
| Avoid modifiers which are implied by the context |
3 |
43–45 |
com/googlecode/mgwt/dom/client/event/touch/Touch.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
31 |
| Avoid modifiers which are implied by the context |
3 |
38 |
| Avoid modifiers which are implied by the context |
3 |
40 |
com/googlecode/mgwt/dom/client/event/touch/TouchCancelHandler.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
32 |
com/googlecode/mgwt/dom/client/event/touch/TouchEndHandler.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
32 |
com/googlecode/mgwt/dom/client/event/touch/TouchMoveHandler.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
32 |
com/googlecode/mgwt/dom/client/event/touch/TouchStartHandler.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
32 |
com/googlecode/mgwt/dom/client/recognizer/EventPropagator.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
34 |
com/googlecode/mgwt/dom/client/recognizer/TimeProvider.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
26 |
com/googlecode/mgwt/dom/client/recognizer/TimerExecutor.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
26 |
| Avoid modifiers which are implied by the context |
3 |
28–30 |
| Avoid modifiers which are implied by the context |
3 |
29 |
com/googlecode/mgwt/dom/client/recognizer/longtap/HasLongTapHandlers.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
34 |
com/googlecode/mgwt/dom/client/recognizer/longtap/LongTapEvent.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' |
4 |
67 |
com/googlecode/mgwt/dom/client/recognizer/longtap/LongTapHandler.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
32 |
com/googlecode/mgwt/dom/client/recognizer/longtap/LongTapRecognizer.java
| Violation |
Priority |
Line |
| These nested if statements could be combined |
3 |
202–205 |
com/googlecode/mgwt/dom/client/recognizer/pinch/HasPinchHandlers.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
33 |
com/googlecode/mgwt/dom/client/recognizer/pinch/OffsetProvider.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
32 |
| Avoid modifiers which are implied by the context |
3 |
39 |
com/googlecode/mgwt/dom/client/recognizer/pinch/PinchEvent.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' |
4 |
64 |
com/googlecode/mgwt/dom/client/recognizer/pinch/PinchHandler.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
33 |
com/googlecode/mgwt/dom/client/recognizer/swipe/HasSwipeHandlers.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
34 |
| Avoid modifiers which are implied by the context |
3 |
42 |
| Avoid modifiers which are implied by the context |
3 |
50 |
com/googlecode/mgwt/dom/client/recognizer/swipe/SwipeEndEvent.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' |
4 |
54 |
com/googlecode/mgwt/dom/client/recognizer/swipe/SwipeEndHandler.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
32 |
com/googlecode/mgwt/dom/client/recognizer/swipe/SwipeMoveEvent.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' |
4 |
60 |
com/googlecode/mgwt/dom/client/recognizer/swipe/SwipeMoveHandler.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
32 |
com/googlecode/mgwt/dom/client/recognizer/swipe/SwipeStartEvent.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' |
4 |
58 |
com/googlecode/mgwt/dom/client/recognizer/tap/MultiTapEvent.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' |
4 |
85 |
com/googlecode/mgwt/dom/client/recognizer/tap/MultiTapHandler.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
32 |
com/googlecode/mgwt/dom/client/recognizer/tap/MultiTapRecognizer.java
| Violation |
Priority |
Line |
| These nested if statements could be combined |
3 |
191–194 |
| These nested if statements could be combined |
3 |
265–269 |
com/googlecode/mgwt/ui/client/widget/touch/TouchWidgetImpl.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
40 |
| Avoid modifiers which are implied by the context |
3 |
49 |
| Avoid modifiers which are implied by the context |
3 |
58 |
| Avoid modifiers which are implied by the context |
3 |
67 |
com/vaadin/client/ui/VSliderPatched.java
| Violation |
Priority |
Line |
| Avoid empty if statements |
3 |
255–256 |
| Avoid empty if statements |
3 |
257–258 |
| Avoid empty if statements |
3 |
258–259 |
| Useless qualified this usage in the same class. |
3 |
415 |
com/vaadin/v7/client/ui/VScrollTablePatched.java
| Violation |
Priority |
Line |
| Avoid unused imports such as 'com.google.gwt.event.dom.client.ContextMenuEvent' |
4 |
51 |
| Avoid unused imports such as 'com.google.gwt.event.dom.client.ContextMenuHandler' |
4 |
52 |
| Avoid unused imports such as 'com.google.gwt.user.client.ui.RootPanel' |
4 |
80 |
| Avoid modifiers which are implied by the context |
3 |
174 |
| Avoid unused private fields such as 'detachedScrollPosition'. |
3 |
451 |
| Avoid unused private fields such as 'initialContentReceived'. |
3 |
704 |
| Avoid unused private fields such as 'touchScrollDelegate'. |
3 |
745 |
| Ternary operators that can be simplified with || or && |
3 |
1534–1536 |
| Overriding method merely calls super |
3 |
2226–2229 |
| Useless parentheses. |
4 |
2388–2390 |
| Useless parentheses. |
4 |
2416–2417 |
| Avoid empty if statements |
3 |
2464–2466 |
| Avoid unused local variables such as 'hCell'. |
3 |
2472 |
| Useless parentheses. |
4 |
2517 |
| These nested if statements could be combined |
3 |
2533–2546 |
| These nested if statements could be combined |
3 |
2535–2545 |
| Useless parentheses. |
4 |
2597 |
| Useless parentheses. |
4 |
2653–2654 |
| Useless parentheses. |
4 |
2655–2656 |
| Useless parentheses. |
4 |
2708 |
| Useless parentheses. |
4 |
2758–2759 |
| Useless parentheses. |
4 |
2779–2781 |
| These nested if statements could be combined |
3 |
3329–3374 |
| Useless parentheses. |
4 |
3515 |
| These nested if statements could be combined |
3 |
3798–3801 |
| These nested if statements could be combined |
3 |
3980–3986 |
| Useless parentheses. |
4 |
4466 |
| These nested if statements could be combined |
3 |
4684–4687 |
| Ternary operators that can be simplified with || or && |
3 |
5484–5486 |
| Unnecessary use of fully qualified name 'com.google.gwt.dom.client.Element' due to existing import 'com.google.gwt.dom.client.Element' |
4 |
5490 |
| Unnecessary use of fully qualified name 'com.google.gwt.dom.client.Element' due to existing import 'com.google.gwt.dom.client.Element' |
4 |
5974 |
| These nested if statements could be combined |
3 |
6173–6286 |
| Useless parentheses. |
4 |
6214–6215 |
| Useless parentheses. |
4 |
6312–6313 |
| Useless parentheses. |
4 |
6409–6410 |
| Avoid using a branching statement as the last in a loop. |
2 |
6789 |
| Avoid unused method parameters such as 'rowUidl'. |
3 |
6797 |
| Avoid unused method parameters such as 'rowUidl'. |
3 |
6804 |
| Useless parentheses. |
4 |
6836 |
| Useless parentheses. |
4 |
6909 |
| Useless parentheses. |
4 |
6909 |
| Useless parentheses. |
4 |
7143–7144 |
| Useless parentheses. |
4 |
7201 |
| These nested if statements could be combined |
3 |
7366–7375 |
| Useless parentheses. |
4 |
7505 |
| Useless parentheses. |
4 |
7530–7531 |
| Useless parentheses. |
4 |
7540–7542 |
| Useless parentheses. |
4 |
7565 |
| Useless parentheses. |
4 |
7566 |
com/vaadin/v7/client/ui/VTreeTablePatched.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'com.google.gwt.dom.client.Element' due to existing import 'com.google.gwt.dom.client.Element' |
4 |
153 |
| Avoid using a branching statement as the last in a loop. |
2 |
357 |
| Useless parentheses. |
4 |
410 |
| Useless parentheses. |
4 |
800–801 |
| Useless parentheses. |
4 |
802 |
com/vaadin/v7/client/ui/table/TableConnectorPatched.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
184 |
| Avoid unused local variables such as 'totalRowsHaveChanged'. |
3 |
184 |
| These nested if statements could be combined |
3 |
325–343 |
com/vaadin/v7/client/ui/treetable/TreeTableConnectorPatched.java
| Violation |
Priority |
Line |
| Avoid unused imports such as 'com.vaadin.shared.ui.Connect' |
4 |
24 |
| Avoid unused imports such as 'com.vaadin.v7.ui.TreeTable' |
4 |
32 |
| These nested if statements could be combined |
3 |
61–69 |
| Useless parentheses. |
4 |
84–85 |
info/magnolia/ui/vaadin/actionbar/Actionbar.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
216 |
| Avoid modifiers which are implied by the context |
3 |
217 |
| Avoid modifiers which are implied by the context |
3 |
219 |
info/magnolia/ui/vaadin/ckeditor/MagnoliaCKEditorConfig.java
| Violation |
Priority |
Line |
| Overriding method merely calls super |
3 |
104–106 |
| Overriding method merely calls super |
3 |
109–111 |
info/magnolia/ui/vaadin/dialog/BaseDialog.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'com.vaadin.ui.Component.Event' due to existing import 'com.vaadin.ui.Component' |
4 |
242 |
| Unnecessary use of fully qualified name 'com.vaadin.ui.Component.Event' due to existing import 'com.vaadin.ui.Component' |
4 |
276 |
| Unnecessary use of fully qualified name 'com.vaadin.ui.Component.Event' due to existing import 'com.vaadin.ui.Component' |
4 |
309 |
info/magnolia/ui/vaadin/dialog/ConfirmationDialog.java
| Violation |
Priority |
Line |
| Useless qualified this usage in the same class. |
3 |
112 |
| Useless qualified this usage in the same class. |
3 |
116 |
| Overriding method merely calls super |
3 |
131–133 |
info/magnolia/ui/vaadin/editor/CroppableImage.java
| Violation |
Priority |
Line |
| Overriding method merely calls super |
3 |
62–64 |
| Avoid modifiers which are implied by the context |
3 |
151 |
| Avoid modifiers which are implied by the context |
3 |
152–153 |
| Avoid modifiers which are implied by the context |
3 |
161 |
| Avoid modifiers which are implied by the context |
3 |
162–163 |
info/magnolia/ui/vaadin/editor/JCrop.java
| Violation |
Priority |
Line |
| Avoid unused constructor parameters such as 'handler'. |
3 |
65 |
info/magnolia/ui/vaadin/editor/JCropField.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
158 |
| Useless parentheses. |
4 |
158 |
info/magnolia/ui/vaadin/form/field/FieldLayout.java
| Violation |
Priority |
Line |
| Overriding method merely calls super |
3 |
113–115 |
info/magnolia/ui/vaadin/grid/MagnoliaTreeTable.java
| Violation |
Priority |
Line |
| Avoid unused private fields such as 'log'. |
3 |
56 |
| Overriding method merely calls super |
3 |
74–76 |
| Overriding method merely calls super |
3 |
115–117 |
info/magnolia/ui/vaadin/gwt/client/actionbar/connector/ActionbarConnector.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
74 |
info/magnolia/ui/vaadin/gwt/client/actionbar/widget/ActionbarWidgetViewImpl.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
354 |
| Useless parentheses. |
4 |
357 |
info/magnolia/ui/vaadin/gwt/client/applauncher/connector/AppLauncherConnector.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
125 |
| Overriding method merely calls super |
3 |
145–147 |
info/magnolia/ui/vaadin/gwt/client/applauncher/widget/AppLauncherView.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
52–57 |
info/magnolia/ui/vaadin/gwt/client/applauncher/widget/VPermanentAppTileGroup.java
| Violation |
Priority |
Line |
| Overriding method merely calls super |
3 |
87–89 |
info/magnolia/ui/vaadin/gwt/client/ckeditor/VMagnoliaCKEditorTextField.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'VMagnoliaCKEditorTextEvents.Listener' due to existing static import 'info.magnolia.ui.vaadin.gwt.client.ckeditor.VMagnoliaCKEditorTextEvents.*' |
4 |
55 |
| Overriding method merely calls super |
3 |
158–160 |
info/magnolia/ui/vaadin/gwt/client/dialog/connector/DialogContainingFormConnector.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
117 |
info/magnolia/ui/vaadin/gwt/client/dialog/widget/DialogHeaderWidget.java
| Violation |
Priority |
Line |
| These nested if statements could be combined |
3 |
124–126 |
info/magnolia/ui/vaadin/gwt/client/form/connector/FormConnector.java
| Violation |
Priority |
Line |
| Useless qualified this usage in the same class. |
3 |
121 |
info/magnolia/ui/vaadin/gwt/client/form/formsection/widget/FormSectionWidget.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
78 |
info/magnolia/ui/vaadin/gwt/client/form/tab/connector/FormTabConnector.java
| Violation |
Priority |
Line |
| Overriding method merely calls super |
3 |
48–50 |
info/magnolia/ui/vaadin/gwt/client/form/widget/FormViewImpl.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
181 |
info/magnolia/ui/vaadin/gwt/client/grid/VMagnoliaTable.java
| Violation |
Priority |
Line |
| Overriding method merely calls super |
3 |
176–178 |
| Unnecessary use of fully qualified name 'com.google.gwt.user.client.Element' due to existing import 'com.google.gwt.user.client.Element' |
4 |
222 |
info/magnolia/ui/vaadin/gwt/client/grid/VMagnoliaTreeTable.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
60 |
info/magnolia/ui/vaadin/gwt/client/jquerywrapper/Callbacks.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
66–68 |
| Avoid modifiers which are implied by the context |
3 |
70–72 |
info/magnolia/ui/vaadin/gwt/client/jquerywrapper/JQueryCallback.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
40 |
info/magnolia/ui/vaadin/gwt/client/jquerywrapper/JQueryFunction.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
46–50 |
| An empty statement (semicolon) not part of a loop |
3 |
51 |
info/magnolia/ui/vaadin/gwt/client/layout/lazylayout/widget/EscalatorPanel.java
| Violation |
Priority |
Line |
| Useless qualified this usage in the same class. |
3 |
230 |
| Useless qualified this usage in the same class. |
3 |
240 |
| Useless qualified this usage in the same class. |
3 |
252 |
info/magnolia/ui/vaadin/gwt/client/layout/lazylayout/widget/LazyThumbnailLayoutImageBundle.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
45 |
| Avoid modifiers which are implied by the context |
3 |
47 |
info/magnolia/ui/vaadin/gwt/client/magnoliashell/shell/MagnoliaShellConnector.java
| Violation |
Priority |
Line |
| These nested if statements could be combined |
3 |
114–116 |
info/magnolia/ui/vaadin/gwt/client/magnoliashell/shell/MagnoliaShellViewImpl.java
| Violation |
Priority |
Line |
| These nested if statements could be combined |
3 |
231–233 |
info/magnolia/ui/vaadin/gwt/client/magnoliashell/shell/VShellImageBundle.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
45 |
| Avoid modifiers which are implied by the context |
3 |
47 |
| Avoid modifiers which are implied by the context |
3 |
50 |
info/magnolia/ui/vaadin/gwt/client/magnoliashell/viewport/animation/ZoomAnimation.java
| Violation |
Priority |
Line |
| Useless qualified this usage in the same class. |
3 |
73 |
info/magnolia/ui/vaadin/gwt/client/magnoliashell/viewport/connector/AppsViewportConnector.java
| Violation |
Priority |
Line |
| Useless qualified this usage in the same class. |
3 |
162 |
info/magnolia/ui/vaadin/gwt/client/magnoliashell/viewport/widget/AppsViewportWidget.java
| Violation |
Priority |
Line |
| Avoid unused local variables such as 'style'. |
3 |
196 |
info/magnolia/ui/vaadin/gwt/client/richtext/TextAreaStretcherConnector.java
| Violation |
Priority |
Line |
| These nested if statements could be combined |
3 |
115–118 |
info/magnolia/ui/vaadin/gwt/client/richtext/VMagnoliaRichTextField.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'VMagnoliaCKEditorTextEvents.Listener' due to existing static import 'info.magnolia.ui.vaadin.gwt.client.ckeditor.VMagnoliaCKEditorTextEvents.*' |
4 |
56 |
| Overriding method merely calls super |
3 |
165–167 |
info/magnolia/ui/vaadin/gwt/client/shared/icon/IconParameters.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
41 |
info/magnolia/ui/vaadin/gwt/client/tabsheet/tab/widget/MagnoliaTabWidget.java
| Violation |
Priority |
Line |
| Avoid unused constructor parameters such as 'connector'. |
3 |
51 |
info/magnolia/ui/vaadin/gwt/client/tabsheet/widget/TabBarWidget.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
146 |
| Useless parentheses. |
4 |
149 |
info/magnolia/ui/vaadin/layout/LazyCardLayout.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
83 |
org/vaadin/alump/ckeditor/client/VCKEditorTextField.java
| Violation |
Priority |
Line |
| These nested if statements could be combined |
3 |
334–336 |
| These nested if statements could be combined |
3 |
433–443 |
| Avoid unnecessary return statements |
3 |
508 |
| Useless qualified this usage in the same class. |
3 |
531 |
| Useless qualified this usage in the same class. |
3 |
533 |
| Useless qualified this usage in the same class. |
3 |
534 |
