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 | 145 - 146 |
org/apache/myfaces/trinidad/bean/FacesBeanImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 576 - 579 |
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 | 66 - 67 |
org/apache/myfaces/trinidad/change/ChangeUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 188 - 196 |
org/apache/myfaces/trinidad/change/RemoveChildComponentChange.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 119 - 123 |
org/apache/myfaces/trinidad/component/StampState.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_EMPTY_ARRAY'. | 566 |
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 | 121 - 125 |
| These nested if statements could be combined | 1287 - 1302 |
org/apache/myfaces/trinidad/component/UIXComponentBase.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 638 |
| An empty statement (semicolon) not part of a loop | 662 |
| These nested if statements could be combined | 1178 - 1179 |
| These nested if statements could be combined | 1212 - 1213 |
| These nested if statements could be combined | 1247 - 1248 |
| An empty statement (semicolon) not part of a loop | 1968 |
org/apache/myfaces/trinidad/component/UIXHierarchy.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 296 - 320 |
org/apache/myfaces/trinidad/convert/DateTimeConverter.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 578 |
| An empty statement (semicolon) not part of a loop | 1194 |
| Avoid unused method parameters such as 'pattern'. | 1806 - 1814 |
| Avoid unused method parameters such as 'locale'. | 1806 - 1814 |
| Avoid unused method parameters such as 'tZone'. | 1806 - 1814 |
| Avoid unused method parameters such as 'pattern'. | 1817 - 1828 |
| Avoid unused method parameters such as 'locale'. | 1817 - 1828 |
| Avoid unused method parameters such as 'tZone'. | 1817 - 1828 |
| Avoid unused method parameters such as 'format'. | 1817 - 1828 |
org/apache/myfaces/trinidad/menu/ImmutableGroupNode.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'accessKey'. | 185 |
| Avoid unused local variables such as 'label'. | 300 |
org/apache/myfaces/trinidad/menu/ImmutableItemNode.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'accessKey'. | 228 |
| Avoid unused local variables such as 'label'. | 382 |
| Avoid unused local variables such as 'labelAndAcessKeyEval'. | 461 |
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 | 248 - 257 |
org/apache/myfaces/trinidad/model/RowKeySetTreeImpl.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as '_containsDefaultNodes()'. | 556 |
| Avoid unused private methods such as '_dumpFlags()'. | 574 |
| Avoid unused constructor parameters such as 'noop'. | 715 - 717 |
| Avoid unused private fields such as '_LOG'. | 860 |
org/apache/myfaces/trinidad/resource/DirectoryResourceLoader.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_LOG'. | 120 |
org/apache/myfaces/trinidad/util/ClassLoaderUtils.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 126 |
org/apache/myfaces/trinidad/util/CollectionUtils.java
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 1089 |
org/apache/myfaces/trinidad/util/ComponentReference.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'stream'. | 417 - 421 |
org/apache/myfaces/trinidad/util/ExternalContextUtils.java
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 56 - 74 |
| Unnecessary final modifier in final class | 91 - 109 |
| Unnecessary final modifier in final class | 119 - 158 |
| These nested if statements could be combined | 127 - 154 |
| Unnecessary final modifier in final class | 451 - 454 |
| Unnecessary final modifier in final class | 471 - 474 |
org/apache/myfaces/trinidad/util/FastMessageFormat.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 92 - 105 |
| Avoid unused private fields such as '_LOG'. | 116 |
org/apache/myfaces/trinidad/util/RequestStateMap.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 73 - 75 |
org/apache/myfaces/trinidad/validator/DoubleRangeValidator.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 331 - 335 |
| These nested if statements could be combined | 340 - 344 |
| These nested if statements could be combined | 349 - 353 |
org/apache/myfaces/trinidad/validator/LengthValidator.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 381 - 385 |
| These nested if statements could be combined | 390 - 394 |
| These nested if statements could be combined | 399 - 403 |
org/apache/myfaces/trinidad/validator/LongRangeValidator.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 334 - 338 |
| These nested if statements could be combined | 343 - 347 |
| These nested if statements could be combined | 352 - 356 |
org/apache/myfaces/trinidad/webapp/ResourceServlet.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 340 - 344 |
| Avoid empty catch blocks | 477 - 480 |
org/apache/myfaces/trinidad/component/UIXIterator.java
| Violation | Line |
|---|---|
| Avoid returning from a finally block | 457 |
org/apache/myfaces/trinidad/component/UIXPage.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 207 - 211 |
org/apache/myfaces/trinidad/component/UIXSelectMany.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 217 |
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 |
|---|---|
| These nested if statements could be combined | 618 - 635 |
org/apache/myfaces/trinidad/component/UIXTree.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 389 - 406 |
org/apache/myfaces/trinidad/component/UIXTreeTable.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 506 - 509 |
org/apache/myfaces/trinidad/component/core/CoreDocument.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 609 - 612 |
org/apache/myfaces/trinidad/component/core/CoreForm.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 544 - 547 |
org/apache/myfaces/trinidad/component/core/data/CoreColumn.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 765 - 768 |
org/apache/myfaces/trinidad/component/core/data/CoreSelectRangeChoiceBar.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 473 - 476 |
org/apache/myfaces/trinidad/component/core/data/CoreTable.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 914 - 917 |
org/apache/myfaces/trinidad/component/core/data/CoreTree.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 462 - 465 |
org/apache/myfaces/trinidad/component/core/data/CoreTreeTable.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 772 - 775 |
org/apache/myfaces/trinidad/component/core/input/CoreChooseColor.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 533 - 536 |
org/apache/myfaces/trinidad/component/core/input/CoreChooseDate.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 523 - 526 |
org/apache/myfaces/trinidad/component/core/input/CoreInputColor.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 886 - 889 |
org/apache/myfaces/trinidad/component/core/input/CoreInputDate.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 861 - 864 |
org/apache/myfaces/trinidad/component/core/input/CoreInputFile.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 730 - 733 |
org/apache/myfaces/trinidad/component/core/input/CoreInputListOfValues.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 955 - 958 |
org/apache/myfaces/trinidad/component/core/input/CoreInputNumberSpinbox.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 872 - 875 |
org/apache/myfaces/trinidad/component/core/input/CoreInputText.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 946 - 949 |
org/apache/myfaces/trinidad/component/core/input/CoreResetButton.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 537 - 540 |
org/apache/myfaces/trinidad/component/core/input/CoreSelectBooleanCheckbox.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 773 - 776 |
org/apache/myfaces/trinidad/component/core/input/CoreSelectBooleanRadio.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 794 - 797 |
org/apache/myfaces/trinidad/component/core/input/CoreSelectManyCheckbox.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 810 - 813 |
org/apache/myfaces/trinidad/component/core/input/CoreSelectManyListbox.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 805 - 808 |
org/apache/myfaces/trinidad/component/core/input/CoreSelectManyShuttle.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 949 - 952 |
org/apache/myfaces/trinidad/component/core/input/CoreSelectOneChoice.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 805 - 808 |
org/apache/myfaces/trinidad/component/core/input/CoreSelectOneListbox.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 831 - 834 |
org/apache/myfaces/trinidad/component/core/input/CoreSelectOneRadio.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 838 - 841 |
org/apache/myfaces/trinidad/component/core/input/CoreSelectOrderShuttle.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 817 - 820 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelAccordion.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 626 - 629 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelBorderLayout.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 997 - 1000 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelBox.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 563 - 566 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelButtonBar.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 459 - 462 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelCaptionGroup.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 482 - 485 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelChoice.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 559 - 562 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelFormLayout.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 611 - 614 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelGroupLayout.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 481 - 484 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelHeader.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 562 - 565 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelHorizontalLayout.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 510 - 513 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelLabelAndMessage.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 622 - 625 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelList.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 538 - 541 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelPage.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 777 - 780 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelPageHeader.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 619 - 622 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelPopup.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 746 - 749 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelRadio.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 513 - 516 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelSideBar.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 479 - 482 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelTabbed.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 469 - 472 |
org/apache/myfaces/trinidad/component/core/layout/CorePanelTip.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 433 - 436 |
org/apache/myfaces/trinidad/component/core/layout/CoreShowDetail.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 571 - 574 |
org/apache/myfaces/trinidad/component/core/layout/CoreShowDetailHeader.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 560 - 563 |
org/apache/myfaces/trinidad/component/core/layout/CoreShowDetailItem.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 601 - 604 |
org/apache/myfaces/trinidad/component/core/nav/CoreBreadCrumbs.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 482 - 485 |
org/apache/myfaces/trinidad/component/core/nav/CoreCommandButton.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 747 - 750 |
org/apache/myfaces/trinidad/component/core/nav/CoreCommandLink.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 726 - 729 |
org/apache/myfaces/trinidad/component/core/nav/CoreCommandNavigationItem.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 774 - 777 |
org/apache/myfaces/trinidad/component/core/nav/CoreGoButton.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 635 - 638 |
org/apache/myfaces/trinidad/component/core/nav/CoreGoLink.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 616 - 619 |
org/apache/myfaces/trinidad/component/core/nav/CoreNavigationPane.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 534 - 537 |
org/apache/myfaces/trinidad/component/core/nav/CoreNavigationTree.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 448 - 451 |
org/apache/myfaces/trinidad/component/core/nav/CorePage.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 825 - 828 |
org/apache/myfaces/trinidad/component/core/nav/CoreProcessChoiceBar.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 454 - 457 |
org/apache/myfaces/trinidad/component/core/nav/CoreSingleStepButtonBar.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 492 - 495 |
org/apache/myfaces/trinidad/component/core/nav/CoreTrain.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 478 - 481 |
org/apache/myfaces/trinidad/component/core/output/CoreIcon.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 467 - 470 |
org/apache/myfaces/trinidad/component/core/output/CoreImage.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 511 - 514 |
org/apache/myfaces/trinidad/component/core/output/CoreLegend.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 460 - 463 |
org/apache/myfaces/trinidad/component/core/output/CoreMedia.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 1120 - 1123 |
org/apache/myfaces/trinidad/component/core/output/CoreMessage.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 506 - 509 |
org/apache/myfaces/trinidad/component/core/output/CoreMessages.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 504 - 507 |
org/apache/myfaces/trinidad/component/core/output/CoreOutputDocument.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 517 - 520 |
org/apache/myfaces/trinidad/component/core/output/CoreOutputFormatted.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 531 - 534 |
org/apache/myfaces/trinidad/component/core/output/CoreOutputLabel.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 528 - 531 |
org/apache/myfaces/trinidad/component/core/output/CoreOutputText.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 526 - 529 |
org/apache/myfaces/trinidad/component/core/output/CoreProgressIndicator.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 455 - 458 |
org/apache/myfaces/trinidad/component/core/output/CoreStatusIndicator.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 480 - 483 |
org/apache/myfaces/trinidad/component/html/HtmlBody.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 527 - 530 |
org/apache/myfaces/trinidad/component/html/HtmlCellFormat.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 663 - 666 |
org/apache/myfaces/trinidad/component/html/HtmlFrameBorderLayout.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 644 - 647 |