PMD Results
The following document contains the results of PMD 5.1.2.
Files
info/magnolia/pages/app/action/CopyComponentAction.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'contentClipboard'. |
77 |
info/magnolia/pages/app/action/CreateComponentAction.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'componentProvider'. |
115 |
| Avoid unused constructor parameters such as 'componentProvider'. |
125 |
| Avoid unused constructor parameters such as 'i18n'. |
125 |
info/magnolia/pages/app/action/DeletePageItemAction.java
| Violation |
Line |
| Avoid empty catch blocks |
83–85 |
info/magnolia/pages/app/action/EditElementAction.java
| Violation |
Line |
| Avoid unused private methods such as 'resolveItemForDeprecations(AbstractElement,SubAppContext,EventBus)'. |
99 |
info/magnolia/pages/app/action/PasteComponentAction.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'contentClipboard'. |
73 |
info/magnolia/pages/app/action/PreviewPreviousVersionAction.java
| Violation |
Line |
| Avoid unused private fields such as 'log'. |
57 |
info/magnolia/pages/app/availability/PageHasSubpagesAndIsNotDeletedRule.java
| Violation |
Line |
| Useless parentheses. |
51 |
info/magnolia/pages/app/availability/PageHasSubpagesRule.java
| Violation |
Line |
| Avoid empty catch blocks |
73–75 |
info/magnolia/pages/app/editor/PagesEditorSubApp.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'versionManager'. |
155 |
| Avoid unused constructor parameters such as 'pageBarView'. |
165 |
| Avoid unused constructor parameters such as 'versionManager'. |
166 |
| Avoid unused constructor parameters such as 'i18NAuthoringSupport'. |
166 |
| Avoid unused constructor parameters such as 'i18nContentSupport'. |
166 |
| Avoid unused constructor parameters such as 'statusBarView'. |
167 |
| These nested if statements could be combined |
268–270 |
info/magnolia/pages/app/editor/PagesEditorSubAppView.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
50–53 |
info/magnolia/pages/app/editor/availability/AcceptsComponentsRule.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'contentClipboard'. |
60 |
info/magnolia/pages/app/editor/availability/IsAreaEditableRule.java
| Violation |
Line |
| Useless parentheses. |
58 |
info/magnolia/pages/app/editor/availability/IsElementCopyableRule.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'contentClipboard'. |
62 |
info/magnolia/pages/app/editor/availability/IsPageEditableRule.java
| Violation |
Line |
| Useless parentheses. |
53 |
info/magnolia/pages/app/editor/event/ComponentMoveEvent.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
83–85 |
info/magnolia/pages/app/editor/event/NodeSelectedEvent.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
66–68 |
info/magnolia/pages/app/editor/pagebar/languageselector/LanguageSelectorView.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
58–60 |
info/magnolia/pages/app/editor/pagebar/platformselector/PlatformSelectorView.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
53–55 |
info/magnolia/pages/app/editor/parameters/DefaultPageEditorStatus.java
| Violation |
Line |
| Useless parentheses. |
90 |
| Useless parentheses. |
91 |
info/magnolia/pages/app/editor/parameters/PageEditorStatus.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
49 |
| Avoid modifiers which are implied by the context |
50 |
| Avoid modifiers which are implied by the context |
51 |
info/magnolia/pages/app/editor/statusbar/activationstatus/ActivationStatus.java
| Violation |
Line |
| Avoid empty catch blocks |
105–107 |
| Avoid empty catch blocks |
124–126 |
