PMD Results
The following document contains the results of PMD 4.2.5.
Files
info/magnolia/module/admininterface/DialogMVCHandler.java
| Violation | Line |
|---|
| These nested if statements could be combined | 255 - 259 |
info/magnolia/module/admininterface/MultiValueSaveHandler.java
| Violation | Line |
|---|
| Avoid unused local variables such as 'data'. | 77 |
info/magnolia/module/admininterface/UUIDSaveHandler.java
| Violation | Line |
|---|
| These nested if statements could be combined | 78 - 80 |
| These nested if statements could be combined | 94 - 100 |
info/magnolia/module/admininterface/commands/ActivationCommand.java
| Violation | Line |
|---|
| Avoid unused method parameters such as 'ctx'. | 105 - 126 |
| These nested if statements could be combined | 282 |
info/magnolia/module/admininterface/commands/DeleteCommand.java
| Violation | Line |
|---|
| Avoid unused private fields such as 'log'. | 52 |
info/magnolia/module/admininterface/commands/MarkNodeAsDeletedCommand.java
| Violation | Line |
|---|
| No need to import a type that lives in the same package | 52 |
info/magnolia/module/admininterface/config/PermissionConfiguration.java
| Violation | Line |
|---|
| Avoid unused private fields such as 'PERMISSION_ALL'. | 39 |
| Avoid unused private fields such as 'PERMISSION_READ'. | 41 |
| Avoid unused private fields such as 'PERMISSION_NO'. | 43 |
info/magnolia/module/admininterface/config/SecurityConfiguration.java
| Violation | Line |
|---|
| Avoid unused private fields such as 'log'. | 58 |
| These nested if statements could be combined | 106 - 110 |
info/magnolia/module/admininterface/dialogs/ParagraphEditDialog.java
| Violation | Line |
|---|
| Avoid unused imports such as 'info.magnolia.cms.util.RequestFormUtil' | 39 |
info/magnolia/module/admininterface/dialogs/RolesEditDialog.java
| Violation | Line |
|---|
| Avoid empty catch blocks | 141 - 143 |
info/magnolia/module/admininterface/dialogs/UserEditDialog.java
| Violation | Line |
|---|
| Avoid empty catch blocks | 153 - 155 |
| Avoid empty catch blocks | 219 - 221 |
info/magnolia/module/admininterface/pages/AdminCentralPage.java
| Violation | Line |
|---|
| Avoid unused imports such as 'info.magnolia.cms.security.ExternalUser' | 40 |
info/magnolia/module/admininterface/pages/DevelopmentUtilsPage.java
| Violation | Line |
|---|
| Avoid unused imports such as 'info.magnolia.cms.core.SystemProperty' | 41 |
| Avoid unused imports such as 'java.net.URLEncoder' | 59 |
| Avoid empty catch blocks | 463 - 465 |
info/magnolia/module/admininterface/pages/ExportPage.java
| Violation | Line |
|---|
| Avoid unused private fields such as 'log'. | 88 |
| These nested if statements could be combined | 301 - 303 |
info/magnolia/module/admininterface/pages/LogViewerPage.java
| Violation | Line |
|---|
| Avoid empty catch blocks | 283 - 285 |
info/magnolia/module/admininterface/pages/QuickstartPage.java
| Violation | Line |
|---|
| Avoid unused imports such as 'info.magnolia.context.MgnlContext' | 40 |
info/magnolia/module/admininterface/setup/for4_4/RegisterMgnlDeletedType.java
| Violation | Line |
|---|
| Avoid empty catch blocks | 66 - 69 |
info/magnolia/module/admininterface/templates/MgnlDeletedTemplateModel.java
| Violation | Line |
|---|
| Avoid unused local variables such as 'iterator'. | 65 |
info/magnolia/module/admininterface/trees/UsersTreeHandler.java
| Violation | Line |
|---|
| Avoid unused imports such as 'info.magnolia.context.MgnlContext' | 52 |
