PMD Results
The following document contains the results of PMD 4.1.
Files
org/apache/myfaces/trinidad/bean/FacesBeanFactory.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 144 - 145 |
org/apache/myfaces/trinidad/bean/FacesBeanImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 476 - 479 |
org/apache/myfaces/trinidad/bean/util/FlaggedPropertyMap.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 65 - 68 |
org/apache/myfaces/trinidad/bean/util/PropertyArrayMap.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 60 - 61 |
org/apache/myfaces/trinidad/bean/util/PropertyHashMap.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 65 - 66 |
org/apache/myfaces/trinidad/change/ChangeUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 142 - 150 |
org/apache/myfaces/trinidad/change/RemoveChildComponentChange.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 119 - 123 |
org/apache/myfaces/trinidad/component/ComponentProcessingContext.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.faces.component.UIComponent' | 21 |
| Avoid unused imports such as 'javax.faces.context.FacesContext' | 22 |
org/apache/myfaces/trinidad/component/StampState.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_EMPTY_ARRAY'. | 567 |
org/apache/myfaces/trinidad/component/TableUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 354 - 355 |
| These nested if statements could be combined | 372 - 373 |
| These nested if statements could be combined | 379 - 380 |
org/apache/myfaces/trinidad/component/TreeState.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 111 - 112 |
| These nested if statements could be combined | 138 - 139 |
org/apache/myfaces/trinidad/component/UIXCollection.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 114 - 118 |
| Avoid unused local variables such as 'childCount'. | 803 |
| Avoid unused private fields such as '_EMPTY_ARRAY'. | 1517 |
org/apache/myfaces/trinidad/component/UIXComponentBase.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 505 |
| An empty statement (semicolon) not part of a loop | 529 |
| These nested if statements could be combined | 951 - 952 |
| These nested if statements could be combined | 986 - 987 |
| These nested if statements could be combined | 1022 - 1023 |
org/apache/myfaces/trinidad/context/RequestContext.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.myfaces.trinidad.util.ComponentUtils' | 38 |
org/apache/myfaces/trinidad/convert/DateTimeConverter.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 556 |
| An empty statement (semicolon) not part of a loop | 1154 |
| Avoid unused method parameters such as 'locale'. | 1766 - 1774 |
| Avoid unused method parameters such as 'pattern'. | 1766 - 1774 |
| Avoid unused method parameters such as 'tZone'. | 1766 - 1774 |
| Avoid unused method parameters such as 'locale'. | 1777 - 1788 |
| Avoid unused method parameters such as 'pattern'. | 1777 - 1788 |
| Avoid unused method parameters such as 'format'. | 1777 - 1788 |
| Avoid unused method parameters such as 'tZone'. | 1777 - 1788 |
org/apache/myfaces/trinidad/model/ModelUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 135 - 136 |
org/apache/myfaces/trinidad/model/RowKeySetImpl.java
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 247 - 252 |
org/apache/myfaces/trinidad/model/RowKeySetTreeImpl.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_LOG'. | 704 |
org/apache/myfaces/trinidad/model/XMLMenuModel.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.faces.application.Application' | 34 |
org/apache/myfaces/trinidad/resource/CachingResourceLoader.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 225 - 227 |
org/apache/myfaces/trinidad/resource/DirectoryResourceLoader.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_LOG'. | 120 |
org/apache/myfaces/trinidad/skin/SkinAddition.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collections' | 21 |
| Avoid unused imports such as 'org.apache.myfaces.trinidad.context.LocaleContext' | 27 |
org/apache/myfaces/trinidad/util/ClassLoaderUtils.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 126 |
org/apache/myfaces/trinidad/util/ExternalContextUtils.java
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 205 - 211 |
| Unnecessary final modifier in final class | 213 - 219 |
| Unnecessary final modifier in final class | 221 - 227 |
| Unnecessary final modifier in final class | 229 - 232 |
org/apache/myfaces/trinidad/util/FastMessageFormat.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 111 |
| An empty statement (semicolon) not part of a loop | 160 |
org/apache/myfaces/trinidad/validator/DoubleRangeValidator.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.faces.validator.Validator' | 27 |
org/apache/myfaces/trinidad/validator/LengthValidator.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.faces.validator.Validator' | 28 |
| Avoid unused imports such as 'org.apache.myfaces.trinidad.util.IntegerUtils' | 34 |
org/apache/myfaces/trinidad/validator/LongRangeValidator.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.faces.validator.Validator' | 29 |
org/apache/myfaces/trinidad/webapp/ResourceServlet.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 258 - 260 |
| Overriding method merely calls super | 352 - 356 |
org/apache/myfaces/trinidad/webapp/UIXComponentELTag.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.myfaces.trinidad.change.AttributeComponentChange' | 45 |
org/apache/myfaces/trinidad/webapp/UIXComponentTag.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.myfaces.trinidad.change.AttributeComponentChange' | 44 |
org/apache/myfaces/trinidad/component/UIXChart.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.faces.component.UIComponent' | 25 |
| Avoid unused imports such as 'javax.faces.context.FacesContext' | 26 |
| Avoid unused imports such as 'javax.faces.el.MethodBinding' | 27 |
| Avoid unused imports such as 'javax.faces.event.PhaseId' | 30 |
org/apache/myfaces/trinidad/component/UIXGo.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.faces.context.FacesContext' | 24 |
org/apache/myfaces/trinidad/component/UIXNavigationTree.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.faces.el.MethodBinding' | 28 |
org/apache/myfaces/trinidad/component/UIXOutput.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.faces.context.FacesContext' | 24 |
org/apache/myfaces/trinidad/component/UIXPage.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.faces.el.MethodBinding' | 30 |
org/apache/myfaces/trinidad/component/UIXSelectMany.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 217 |
org/apache/myfaces/trinidad/component/UIXShowOne.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 25 |
org/apache/myfaces/trinidad/component/UIXSingleStep.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 153 - 156 |
org/apache/myfaces/trinidad/component/UIXTable.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 27 |
org/apache/myfaces/trinidad/component/UIXTreeTable.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.myfaces.trinidad.model.TreeModel' | 42 |