PMD Results
The following document contains the results of PMD 5.1.2.
Files
info/magnolia/ui/form/field/AbstractCustomMultiField.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'i18nAuthoringSupport'. |
96 |
| Avoid unused constructor parameters such as 'i18nContentSupport'. |
107 |
| Unnecessary use of fully qualified name 'com.vaadin.data.Property.ValueChangeEvent' due to existing import 'com.vaadin.data.Property' |
214 |
| Useless parentheses. |
235 |
info/magnolia/ui/form/field/CompositeField.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'i18nContentSupport'. |
67 |
info/magnolia/ui/form/field/LinkField.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'appController'. |
87 |
| Avoid unused constructor parameters such as 'uiContext'. |
87 |
| Unnecessary use of fully qualified name 'com.vaadin.data.Property.ValueChangeEvent' due to existing import 'com.vaadin.data.Property' |
171 |
| Unnecessary use of fully qualified name 'com.vaadin.data.Property.ValueChangeEvent' due to existing import 'com.vaadin.data.Property' |
205 |
info/magnolia/ui/form/field/MultiField.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'i18nContentSupport'. |
104 |
| Avoid unused private methods such as 'removeValueProperty(int)'. |
277 |
| Avoid unused private methods such as 'onDelete(Component,Property)'. |
320 |
| Avoid unused private methods such as 'onMove(Component,Property,boolean)'. |
343 |
| Useless parentheses. |
361 |
info/magnolia/ui/form/field/SwitchableField.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'i18nContentSupport'. |
79 |
info/magnolia/ui/form/field/component/ContentPreviewComponent.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
53 |
info/magnolia/ui/form/field/converter/IdentifierToPathConverter.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
46 |
info/magnolia/ui/form/field/factory/AbstractFieldFactory.java
| Violation |
Line |
| These nested if statements could be combined |
130–133 |
| Useless parentheses. |
190 |
| Useless parentheses. |
190 |
info/magnolia/ui/form/field/factory/CompositeFieldFactory.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'i18nContentSupport'. |
78 |
info/magnolia/ui/form/field/factory/DateFieldFactory.java
| Violation |
Line |
| Avoid unused private methods such as 'setTimeZone(DateField)'. |
97 |
| Avoid empty catch blocks |
104–106 |
info/magnolia/ui/form/field/factory/MultiValueFieldFactory.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'i18nContentSupport'. |
74 |
info/magnolia/ui/form/field/factory/RichTextFieldFactory.java
| Violation |
Line |
| Avoid unused method parameters such as 'path'. |
227 |
info/magnolia/ui/form/field/factory/SwitchableFieldFactory.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'i18nContentSupport'. |
97 |
info/magnolia/ui/form/field/transformer/TransformedProperty.java
| Violation |
Line |
| Useless parentheses. |
78 |
info/magnolia/ui/form/field/transformer/basic/BasicTransformer.java
| Violation |
Line |
| Useless parentheses. |
164 |
| Avoid empty catch blocks |
171–173 |
info/magnolia/ui/form/field/transformer/basic/NotNullInitialStringValueTransformer.java
| Violation |
Line |
| Useless parentheses. |
61 |
info/magnolia/ui/form/field/transformer/composite/SwitchableTransformer.java
| Violation |
Line |
| Overriding method merely calls super |
69–71 |
| Overriding method merely calls super |
74–76 |
info/magnolia/ui/form/field/transformer/multi/DelegatingMultiValueFieldTransformer.java
| Violation |
Line |
| Avoid unused constructor parameters such as 'i18nContentSupport'. |
94 |
| Avoid unused private methods such as 'reorganizeIndex(int)'. |
193 |
info/magnolia/ui/form/field/transformer/multi/MultiTransformer.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
52 |
| Avoid modifiers which are implied by the context |
59 |
info/magnolia/ui/form/field/upload/UploadField.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
45 |
| Avoid modifiers which are implied by the context |
51 |
info/magnolia/ui/form/field/upload/UploadProgressIndicator.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
46 |
| Avoid modifiers which are implied by the context |
51 |
| Avoid modifiers which are implied by the context |
56 |
| Avoid modifiers which are implied by the context |
61 |
info/magnolia/ui/form/field/upload/basic/BasicUploadField.java
| Violation |
Line |
| Useless parentheses. |
325 |
info/magnolia/ui/form/field/upload/basic/BasicUploadProgressIndicator.java
| Violation |
Line |
| Useless parentheses. |
153 |
info/magnolia/ui/form/validator/registry/FieldValidatorFactoryFactory.java
| Violation |
Line |
| Unnecessary use of fully qualified name 'info.magnolia.ui.form.validator.factory.FieldValidatorFactory' due to existing import 'info.magnolia.ui.form.validator.factory.FieldValidatorFactory' |
66 |
