PMD Results
The following document contains the results of PMD 6.21.0.
Violations By Priority
Priority 3
info/magnolia/ui/contentapp/action/MoveAction.java
info/magnolia/ui/contentapp/detail/action/SaveDetailSubAppAction.java
info/magnolia/ui/contenttype/AppWithContentType.java
info/magnolia/ui/dev/NoScssCacheVaadinServlet.java
info/magnolia/ui/editor/MultiFormView.java
info/magnolia/ui/field/factory/LinkFieldFactory.java
info/magnolia/ui/field/factory/RichTextFieldFactory.java
info/magnolia/ui/field/factory/TokenFieldWrapper.java
info/magnolia/ui/framework/UiComponentContextProxy.java
info/magnolia/ui/framework/app/DefaultLocationHistoryMapper.java
info/magnolia/ui/framework/app/embedded/EmbeddedPageSubApp.java
info/magnolia/ui/framework/app/embedded/EmbeddedPageViewImpl.java
info/magnolia/ui/framework/ioc/DeflateUiContextDependentBindings.java
info/magnolia/ui/framework/ioc/UiContextBoundComponentProvider.java
Rule |
Violation |
Line |
UnusedFormalParameter |
Avoid unused constructor parameters such as 'currentUiContextReference'. |
67 |
info/magnolia/ui/framework/layout/DeclarativeLayoutProducer.java
Rule |
Violation |
Line |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'templateReader': resource specifications are implicitly final |
90 |
info/magnolia/ui/framework/message/LocalMessageDispatcher.java
info/magnolia/ui/framework/message/MessagesManager.java
Rule |
Violation |
Line |
UnnecessaryModifier |
Unnecessary modifier 'public' on interface 'MessageListener': members of interface types are implicitly public |
50–57 |
info/magnolia/ui/framework/task/LocalTaskDispatcher.java
info/magnolia/util/Util.java
Priority 4
info/magnolia/ui/contentapp/action/CommitAction.java
info/magnolia/ui/contentapp/browser/GridWithShortcuts.java
info/magnolia/ui/contentapp/browser/ListView.java
Rule |
Violation |
Line |
UnusedImports |
Avoid unused imports such as 'java.util.Iterator' |
39 |
info/magnolia/ui/contentapp/detail/ContentDetailSubApp.java
info/magnolia/ui/datasource/WithDatasource.java
info/magnolia/ui/editor/FormPresenter.java
info/magnolia/ui/field/ConfiguredComplexPropertyDefinition.java
Rule |
Violation |
Line |
UnusedImports |
Avoid unused imports such as 'java.util.Optional' |
44 |
info/magnolia/ui/field/LinkFieldBinder.java
Rule |
Violation |
Line |
UnusedImports |
Avoid unused imports such as 'info.magnolia.ui.contentapp.FilteringMode' |
38 |
info/magnolia/ui/field/MultiFieldDefinition.java
info/magnolia/ui/framework/app/AppControllerImpl.java
info/magnolia/ui/framework/app/AppInstanceControllerImpl.java
info/magnolia/ui/framework/app/embedded/EmbeddedPageSubApp.java
info/magnolia/ui/framework/ioc/UiComponentContextParameterResolver.java
info/magnolia/ui/framework/layout/DeclarativeLayoutProducer.java
Rule |
Violation |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
89 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.read' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
95 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
110 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
112 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
115 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.ComponentFactory' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
130 |
info/magnolia/util/Util.java
Files
info/magnolia/ui/contentapp/action/CommitAction.java
info/magnolia/ui/contentapp/action/MoveAction.java
Rule |
Violation |
Priority |
Line |
UnusedFormalParameter |
Avoid unused constructor parameters such as 'appDescriptor'. |
3 |
83 |
info/magnolia/ui/contentapp/browser/GridWithShortcuts.java
info/magnolia/ui/contentapp/browser/ListView.java
Rule |
Violation |
Priority |
Line |
UnusedImports |
Avoid unused imports such as 'java.util.Iterator' |
4 |
39 |
info/magnolia/ui/contentapp/detail/ContentDetailSubApp.java
info/magnolia/ui/contentapp/detail/action/SaveDetailSubAppAction.java
info/magnolia/ui/contenttype/AppWithContentType.java
info/magnolia/ui/datasource/WithDatasource.java
info/magnolia/ui/dev/NoScssCacheVaadinServlet.java
info/magnolia/ui/editor/FormPresenter.java
info/magnolia/ui/editor/MultiFormView.java
info/magnolia/ui/field/ConfiguredComplexPropertyDefinition.java
Rule |
Violation |
Priority |
Line |
UnusedImports |
Avoid unused imports such as 'java.util.Optional' |
4 |
44 |
info/magnolia/ui/field/LinkFieldBinder.java
Rule |
Violation |
Priority |
Line |
UnusedImports |
Avoid unused imports such as 'info.magnolia.ui.contentapp.FilteringMode' |
4 |
38 |
info/magnolia/ui/field/MultiFieldDefinition.java
info/magnolia/ui/field/factory/LinkFieldFactory.java
Rule |
Violation |
Priority |
Line |
UnusedFormalParameter |
Avoid unused constructor parameters such as 'itemResolver'. |
3 |
95 |
info/magnolia/ui/field/factory/RichTextFieldFactory.java
Rule |
Violation |
Priority |
Line |
UnusedFormalParameter |
Avoid unused constructor parameters such as 'appDescriptorRegistry'. |
3 |
102 |
info/magnolia/ui/field/factory/TokenFieldWrapper.java
info/magnolia/ui/framework/UiComponentContextProxy.java
info/magnolia/ui/framework/app/AppControllerImpl.java
info/magnolia/ui/framework/app/AppInstanceControllerImpl.java
info/magnolia/ui/framework/app/DefaultLocationHistoryMapper.java
info/magnolia/ui/framework/app/embedded/EmbeddedPageSubApp.java
info/magnolia/ui/framework/app/embedded/EmbeddedPageViewImpl.java
info/magnolia/ui/framework/ioc/DeflateUiContextDependentBindings.java
info/magnolia/ui/framework/ioc/UiComponentContextParameterResolver.java
info/magnolia/ui/framework/ioc/UiContextBoundComponentProvider.java
Rule |
Violation |
Priority |
Line |
UnusedFormalParameter |
Avoid unused constructor parameters such as 'currentUiContextReference'. |
3 |
67 |
info/magnolia/ui/framework/layout/DeclarativeLayoutProducer.java
Rule |
Violation |
Priority |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
89 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'templateReader': resource specifications are implicitly final |
3 |
90 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.read' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
95 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
110 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
112 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.ComponentMapper' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
115 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'Design.ComponentFactory' due to existing static import 'com.vaadin.ui.declarative.Design.*' |
4 |
130 |
info/magnolia/ui/framework/message/LocalMessageDispatcher.java
Rule |
Violation |
Priority |
Line |
UnusedFormalParameter |
Avoid unused constructor parameters such as 'vaadinSession'. |
3 |
65 |
info/magnolia/ui/framework/message/MessagesManager.java
Rule |
Violation |
Priority |
Line |
UnnecessaryModifier |
Unnecessary modifier 'public' on interface 'MessageListener': members of interface types are implicitly public |
3 |
50–57 |
info/magnolia/ui/framework/task/LocalTaskDispatcher.java
info/magnolia/util/Util.java