PMD Results
The following document contains the results of PMD 5.1.2.
Files
info/magnolia/ui/workbench/AbstractContentPresenterBase.java
| Violation |
Line |
| Avoid unused local variables such as 'item'. |
188 |
info/magnolia/ui/workbench/ContentView.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
74–85 |
info/magnolia/ui/workbench/WorkbenchPresenter.java
| Violation |
Line |
| Avoid unused private fields such as 'log'. |
72 |
info/magnolia/ui/workbench/WorkbenchView.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
74 |
info/magnolia/ui/workbench/WorkbenchViewImpl.java
| Violation |
Line |
| Avoid unused private fields such as 'previousViewType'. |
87 |
| Avoid unused private fields such as 'i18n'. |
91 |
| Avoid unused local variables such as 'deleteShortcut'. |
130 |
| These nested if statements could be combined |
155–157 |
info/magnolia/ui/workbench/column/definition/BooleanPropertyColumnDefinition.java
| Violation |
Line |
| Avoid empty if statements |
102–104 |
info/magnolia/ui/workbench/column/definition/ColumnAvailabilityRule.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
44 |
info/magnolia/ui/workbench/contenttool/ContentToolDefinition.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
42 |
| Avoid modifiers which are implied by the context |
44 |
info/magnolia/ui/workbench/search/SearchJcrContainer.java
| Violation |
Line |
| Useless parentheses. |
226 |
| Useless parentheses. |
227 |
| Useless parentheses. |
228 |
| Useless parentheses. |
228 |
info/magnolia/ui/workbench/thumbnail/ThumbnailContainer.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
244–246 |
info/magnolia/ui/workbench/tree/HierarchicalJcrContainer.java
| Violation |
Line |
| Useless parentheses. |
181 |
| Useless parentheses. |
195 |
| Avoid unused private methods such as 'getSession()'. |
362 |
info/magnolia/ui/workbench/tree/InplaceEditingFieldFactory.java
| Violation |
Line |
| Unnecessary use of fully qualified name 'com.vaadin.data.Property.ValueChangeEvent' due to existing import 'com.vaadin.data.Property' |
139 |
info/magnolia/ui/workbench/tree/TreeView.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
58–61 |
info/magnolia/ui/workbench/tree/TreeViewImpl.java
| Violation |
Line |
| Avoid unused local variables such as 'field'. |
319 |
info/magnolia/ui/workbench/tree/drop/DropConstraint.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
55 |
| Avoid modifiers which are implied by the context |
66 |
| Avoid modifiers which are implied by the context |
77 |
| Avoid modifiers which are implied by the context |
87 |
| Avoid modifiers which are implied by the context |
93 |
info/magnolia/ui/workbench/tree/drop/TreeViewDropHandler.java
| Violation |
Line |
| These nested if statements could be combined |
191–194 |
| Unnecessary use of fully qualified name 'NodeUtil.isSame' due to existing import 'info.magnolia.jcr.util.NodeUtil' |
235 |
| Unnecessary use of fully qualified name 'NodeUtil.moveProperty' due to existing import 'info.magnolia.jcr.util.NodeUtil' |
280 |
