PMD Results
The following document contains the results of PMD 6.4.0.
Files
info/magnolia/ui/contentapp/ContentBrowserSubApp.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
138 |
info/magnolia/ui/contentapp/browser/Workbench.java
| Violation |
Priority |
Line |
| Avoid unused imports such as 'info.magnolia.ui.framework.ViewDefinition' |
4 |
40 |
| Avoid unused imports such as 'com.vaadin.shared.ui.MarginInfo' |
4 |
48 |
info/magnolia/ui/contentapp/detail/ContentDetailSubApp.java
| Violation |
Priority |
Line |
| Avoid unused private methods such as 'createPrimaryActions()'. |
3 |
131 |
| Avoid unused private methods such as 'createLocateSelector(LocaleContext)'. |
3 |
143 |
| Useless parentheses. |
4 |
251 |
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/WithDatasource.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
56 |
info/magnolia/ui/field/factory/LinkFieldFactory.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
111 |
info/magnolia/ui/field/factory/RichTextFieldFactory.java
| Violation |
Priority |
Line |
| Avoid duplicate imports such as 'info.magnolia.ui.vaadin.ckeditor.MagnoliaCKEditorConfig.ToolbarGroup' |
4 |
54 |
info/magnolia/ui/field/factory/TokenFieldWrapper.java
| Violation |
Priority |
Line |
| Overriding method merely calls super |
3 |
128–130 |
info/magnolia/ui/framework/ViewContextProxy.java
| Violation |
Priority |
Line |
| Too many static imports may lead to messy code |
3 |
34–120 |
info/magnolia/ui/framework/app/AppControllerImpl.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
480–483 |
info/magnolia/ui/framework/app/AppInstanceControllerImpl.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
194 |
| Useless parentheses. |
4 |
194 |
| Useless parentheses. |
4 |
487 |
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/databinding/layout/CustomLayoutProducer.java
| Violation |
Priority |
Line |
| Avoid modifiers which are implied by the context |
3 |
53 |
info/magnolia/ui/framework/ioc/DeflateUiContextDependentBindings.java
| Violation |
Priority |
Line |
| Avoid unused private fields such as 'log'. |
3 |
108 |
info/magnolia/ui/framework/ioc/ViewContextParameterResolver.java
| Violation |
Priority |
Line |
| Useless parentheses. |
4 |
104 |
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 |
