PMD Results
The following document contains the results of PMD 4.2.5.
Files
info/magnolia/module/admininterface/AdminTreeMVCHandler.java
Violation | Line |
---|
Avoid unused imports such as 'info.magnolia.context.WebContext' | 56 |
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 | 51 |
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 unused imports such as 'info.magnolia.cms.core.ItemType' | 38 |
Avoid unused imports such as 'info.magnolia.cms.security.Permission' | 44 |
Avoid empty catch blocks | 226 - 228 |
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 imports such as 'info.magnolia.cms.core.SystemProperty' | 38 |
Avoid unused imports such as 'java.net.URLEncoder' | 58 |
Avoid unused private fields such as 'log'. | 88 |
These nested if statements could be combined | 300 - 302 |
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 |