PMD Results
The following document contains the results of PMD 5.1.2.
Files
info/magnolia/ui/framework/action/AbstractMultiItemAction.java
Violation |
Line |
Avoid unused private fields such as 'log'. |
65 |
info/magnolia/ui/framework/action/DeleteItemAction.java
Violation |
Line |
Avoid empty catch blocks |
137–139 |
info/magnolia/ui/framework/action/async/DefaultAsyncActionExecutor.java
Violation |
Line |
Useless parentheses. |
114 |
Unnecessary use of fully qualified name 'info.magnolia.module.scheduler.CommandJob' due to existing import 'info.magnolia.module.scheduler.CommandJob' |
128 |
Useless parentheses. |
157 |
info/magnolia/ui/framework/app/AppControllerImpl.java
Violation |
Line |
Avoid unused constructor parameters such as 'moduleRegistry'. |
118 |
info/magnolia/ui/framework/app/AppInstanceControllerImpl.java
Violation |
Line |
Avoid unused constructor parameters such as 'moduleRegistry'. |
151 |
Avoid unused constructor parameters such as 'moduleRegistry'. |
162 |
Useless parentheses. |
239 |
Useless parentheses. |
239 |
Useless parentheses. |
529 |
info/magnolia/ui/framework/app/DefaultLocationHistoryMapper.java
Violation |
Line |
Avoid unused method parameters such as 'subAppId'. |
82 |
Avoid unused method parameters such as 'parameter'. |
82 |
info/magnolia/ui/framework/app/embedded/EmbeddedPageSubApp.java
Violation |
Line |
Avoid unused private fields such as 'log'. |
53 |
info/magnolia/ui/framework/app/embedded/EmbeddedPageViewImpl.java
Violation |
Line |
Avoid unused private fields such as 'log'. |
52 |
Avoid unused constructor parameters such as 'appContext'. |
63 |
info/magnolia/ui/framework/availability/AcceptsClipboardContent.java
Violation |
Line |
Avoid unused private fields such as 'contentConnector'. |
47 |
info/magnolia/ui/framework/availability/IsNotVersionedRule.java
Violation |
Line |
Useless parentheses. |
76 |
info/magnolia/ui/framework/availability/IsVersionRule.java
Violation |
Line |
Useless parentheses. |
61 |
info/magnolia/ui/framework/command/CleanTempFilesCommand.java
Violation |
Line |
Avoid unused constructor parameters such as 'configurationProperties'. |
62 |
info/magnolia/ui/framework/command/ImportZipCommand.java
Violation |
Line |
Useless parentheses. |
216 |
info/magnolia/ui/framework/message/LocalMessageDispatcher.java
Violation |
Line |
Avoid unused constructor parameters such as 'vaadinSession'. |
65 |
info/magnolia/ui/framework/message/MessagesManager.java
Violation |
Line |
Avoid modifiers which are implied by the context |
50–57 |
info/magnolia/ui/framework/setup/AddIsPublishedRuleToAllDeactivateActionsTask.java
Violation |
Line |
These nested if statements could be combined |
67–84 |
info/magnolia/ui/framework/setup/UiFrameworkModuleVersionHandler.java
Violation |
Line |
Useless parentheses. |
135 |
Useless parentheses. |
136 |
Useless parentheses. |
138 |
info/magnolia/ui/framework/task/LocalTaskDispatcher.java
Violation |
Line |
Avoid unused constructor parameters such as 'vaadinSession'. |
78 |
Avoid empty catch blocks |
105–106 |