PMD Results
The following document contains the results of PMD 6.4.0.
Files
info/magnolia/ui/LightBoxBuilder.java
| Violation |
Priority |
Line |
| Avoid unused imports such as 'com.vaadin.server.Sizeable' |
4 |
36 |
info/magnolia/ui/contentapp/action/CommitActionDefinition.java
| Violation |
Priority |
Line |
| Avoid unused imports such as 'info.magnolia.ui.api.action.ConfiguredActionDefinition' |
4 |
36 |
info/magnolia/ui/contentapp/browser/Workbench.java
| Violation |
Priority |
Line |
| Avoid unused imports such as 'com.vaadin.ui.Label' |
4 |
48 |
info/magnolia/ui/contentapp/detail/ContentDetailSubApp.java
| Violation |
Priority |
Line |
| Avoid unused imports such as 'com.vaadin.ui.CssLayout' |
4 |
64 |
| Useless parentheses. |
4 |
218 |
info/magnolia/ui/datasource/WithDatasource.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
55 |
info/magnolia/ui/dev/NoScssCacheVaadinServlet.java
| Violation |
Priority |
Line |
| Avoid unused private fields such as 'log'. |
3 |
58 |
info/magnolia/ui/dialog/EditorActionLayout.java
| Violation |
Priority |
Line |
| Avoid unused imports such as 'com.vaadin.ui.Label' |
4 |
42 |
info/magnolia/ui/field/SelectFieldBinder.java
| Violation |
Priority |
Line |
| Avoid unused local variables such as 'type'. |
3 |
69 |
info/magnolia/ui/field/factory/TokenFieldWrapper.java
| Violation |
Priority |
Line |
| Overriding method merely calls super |
3 |
129–131 |
info/magnolia/ui/framework/ViewContextProxy.java
| Violation |
Priority |
Line |
| Too many static imports may lead to messy code |
3 |
34–124 |
info/magnolia/ui/framework/app/AppControllerImpl.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
487–490 |
info/magnolia/ui/framework/app/AppInstanceControllerImpl.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
197 |
| Useless parentheses. |
4 |
197 |
| Useless parentheses. |
4 |
493 |
info/magnolia/ui/framework/app/DefaultLocationHistoryMapper.java
| Violation |
Priority |
Line |
| Avoid unused method parameters such as 'parameter'. |
3 |
82 |
| Avoid unused method parameters such as 'subAppId'. |
3 |
82 |
info/magnolia/ui/framework/app/embedded/EmbeddedPageSubApp.java
| Violation |
Priority |
Line |
| Avoid unused private fields such as 'log'. |
3 |
53 |
| Useless parentheses. |
4 |
66 |
info/magnolia/ui/framework/app/embedded/EmbeddedPageViewImpl.java
| Violation |
Priority |
Line |
| Avoid unused private fields such as 'log'. |
3 |
53 |
| Avoid unused constructor parameters such as 'appContext'. |
3 |
64 |
info/magnolia/ui/framework/chooser/SingleItemWorkbenchChooser.java
| Violation |
Priority |
Line |
| Avoid unused imports such as 'info.magnolia.ui.framework.datasource.definition.DatasourceDefinition' |
4 |
37 |
info/magnolia/ui/framework/databinding/SwitchableFormView.java
| Violation |
Priority |
Line |
| Avoid unused private fields such as 'formFieldFactory'. |
3 |
80 |
info/magnolia/ui/framework/ioc/DeflateUiContextDependentBindings.java
| Violation |
Priority |
Line |
| Avoid unused private fields such as 'log'. |
3 |
111 |
info/magnolia/ui/framework/ioc/ViewContextParameterResolver.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
119 |
info/magnolia/ui/framework/layout/DeclarativeLayoutProducer.java
| Violation |
Priority |
Line |
| Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
87 |
| Avoid modifiers which are implied by the context |
3 |
88 |
| Unnecessary use of fully qualified name 'Design.read' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
93 |
| Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
108 |
| Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
110 |
| Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
113 |
| Unnecessary use of fully qualified name 'Design.ComponentFactory' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
128 |
info/magnolia/ui/framework/message/LocalMessageDispatcher.java
| Violation |
Priority |
Line |
| Avoid unused constructor parameters such as 'vaadinSession'. |
3 |
65 |
info/magnolia/ui/framework/message/MessagesManager.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
50–57 |
info/magnolia/ui/framework/task/LocalTaskDispatcher.java
| Violation |
Priority |
Line |
| Avoid unused constructor parameters such as 'vaadinSession'. |
3 |
78 |
| Avoid empty catch blocks |
3 |
105–106 |
info/magnolia/util/Util.java
| Violation |
Priority |
Line |
| These nested if statements could be combined |
3 |
54–56 |
| Useless parentheses. |
4 |
70 |
