PMD Results
The following document contains the results of PMD 3.9.
Files
org/apache/myfaces/component/html/ext/AbstractHtmlDataTable.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_SORTABLE'. | 78 |
| Avoid unused private fields such as 'DEFAULT_EMBEDDED'. | 79 |
| Avoid unused private fields such as 'DEFAULT_DETAILSTAMP_EXPANDED'. | 80 |
| Avoid unused private fields such as 'DEFAULT_NEWSPAPER_COLUMNS'. | 83 |
| Avoid unused private fields such as 'DEFAULT_NEWSPAPER_ORIENTATION'. | 84 |
| Avoid unused private fields such as 'DEFAULT_DETAILSTAMP_LOCATION'. | 85 |
| These nested if statements could be combined | 1535 - 1538 |
| Avoid unused private fields such as 'DEFAULT_RENDEREDIFEMPTY'. | 1549 |
org/apache/myfaces/component/html/ext/AbstractHtmlSelectManyCheckbox.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'booleanFromObject(Object,boolean)'. | 97 |
org/apache/myfaces/component/html/ext/HtmlDataTableHack.java
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 593 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 614 |
| Avoid unused private methods such as 'booleanFromObject(Object,boolean)'. | 639 |
org/apache/myfaces/component/html/ext/_SerializableResultSetDataModel.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'resultSet'. | 33 - 36 |
| Avoid unused constructor parameters such as 'first'. | 33 - 36 |
| Avoid unused constructor parameters such as 'rows'. | 33 - 36 |
org/apache/myfaces/component/html/util/HtmlComponentUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 83 - 97 |
org/apache/myfaces/component/html/util/StreamingAddResource.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'position'. | 478 - 496 |
| These nested if statements could be combined | 1015 - 1022 |
org/apache/myfaces/custom/buffer/AbstractBuffer.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 75 |
org/apache/myfaces/custom/calendar/AbstractHtmlInputCalendar.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 88 |
org/apache/myfaces/custom/calendar/HtmlCalendarRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'value'. | 262 - 326 |
| Avoid empty if statements | 381 - 383 |
| Avoid empty if statements | 400 - 402 |
org/apache/myfaces/custom/checkbox/AbstractHtmlCheckbox.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 53 |
org/apache/myfaces/custom/collapsiblepanel/AbstractHtmlCollapsiblePanel.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 57 |
| Overriding method merely calls super | 154 - 157 |
| Overriding method merely calls super | 214 - 217 |
| Overriding method merely calls super | 261 - 263 |
| Overriding method merely calls super | 268 - 271 |
| Overriding method merely calls super | 276 - 279 |
| Overriding method merely calls super | 284 - 287 |
org/apache/myfaces/custom/datalist/AbstractHtmlDataList.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 52 |
org/apache/myfaces/custom/datascroller/AbstractHtmlDataScroller.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 74 |
| Avoid unused private fields such as 'DEFAULT_IMMEDIATE'. | 75 |
org/apache/myfaces/custom/date/AbstractHtmlInputDate.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 56 |
| Avoid unused method parameters such as 'type'. | 187 - 192 |
org/apache/myfaces/custom/div/AbstractDiv.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 41 |
org/apache/myfaces/custom/dojo/DojoUtils.java
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 185 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 186 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 187 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 419 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 420 |
| These nested if statements could be combined | 548 - 550 |
org/apache/myfaces/custom/fileupload/HtmlFileUploadRenderer.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'log'. | 55 |
| These nested if statements could be combined | 71 - 74 |
org/apache/myfaces/custom/htmlTag/AbstractHtmlTag.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 59 |
org/apache/myfaces/custom/jslistener/AbstractJsValueChangeListener.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 45 |
org/apache/myfaces/custom/jslistener/JsValueChangeListenerRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'context'. | 124 - 170 |
| Avoid empty if statements | 149 - 154 |
org/apache/myfaces/custom/jsvalueset/AbstractHtmlJsValueSet.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 48 |
| Overriding method merely calls super | 70 - 73 |
org/apache/myfaces/custom/layout/AbstractHtmlPanelLayout.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 53 |
| Avoid unused private fields such as 'DEFAULT_BORDER'. | 55 |
| Avoid unused private fields such as 'DEFAULT_LAYOUT'. | 56 |
org/apache/myfaces/custom/navigation/AbstractHtmlCommandNavigation.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 61 |
| These nested if statements could be combined | 97 - 100 |
org/apache/myfaces/custom/navigation/AbstractHtmlPanelNavigation.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 63 |
| Avoid unused private fields such as 'DEFAULT_BORDER'. | 65 |
| Avoid empty if statements | 90 - 93 |
org/apache/myfaces/custom/navmenu/AbstractUINavigationMenuItem.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_IMMEDIATE'. | 57 |
org/apache/myfaces/custom/navmenu/htmlnavmenu/AbstractHtmlCommandNavigationItem.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 62 |
| These nested if statements could be combined | 154 - 156 |
org/apache/myfaces/custom/navmenu/htmlnavmenu/AbstractHtmlPanelNavigationMenu.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 66 |
| Avoid unused private fields such as 'DEFAULT_BORDER'. | 69 |
| Avoid empty if statements | 96 - 99 |
org/apache/myfaces/custom/navmenu/htmlnavmenu/HtmlNavigationMenuRenderer.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 559 |
org/apache/myfaces/custom/navmenu/jscookmenu/AbstractHtmlCommandJSCookMenu.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 53 |
org/apache/myfaces/custom/navmenu/jscookmenu/HtmlJSCookMenuRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 224 - 226 |
org/apache/myfaces/custom/newspaper/AbstractHtmlNewspaperTable.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_NEWSPAPER_COLUMNS'. | 53 |
org/apache/myfaces/custom/panelstack/AbstractHtmlPanelStack.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 46 |
org/apache/myfaces/custom/panelstack/HtmlPanelStackRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 79 - 81 |
org/apache/myfaces/custom/picklist/HtmlPicklistRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'uiComponent'. | 210 - 217 |
| Avoid unused method parameters such as 'facesContext'. | 331 - 338 |
| Avoid unused method parameters such as 'converter'. | 331 - 338 |
| Avoid unused method parameters such as 'uiComponent'. | 331 - 338 |
org/apache/myfaces/custom/popup/AbstractHtmlPopup.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 49 |
org/apache/myfaces/custom/radio/AbstractHtmlRadio.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 54 |
org/apache/myfaces/custom/schedule/AbstractHtmlSchedule.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 52 |
org/apache/myfaces/custom/schedule/ScheduleTagHandler.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 36 - 41 |
org/apache/myfaces/custom/selectOneLanguage/AbstractSelectOneLanguage.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_maxLength'. | 59 |
| Avoid unused private fields such as '_emptySelection'. | 61 |
org/apache/myfaces/custom/stylesheet/AbstractStylesheet.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 43 |
org/apache/myfaces/custom/subform/AbstractSubForm.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 281 - 284 |
org/apache/myfaces/custom/subform/SubFormRenderer.java
| Violation | Line |
|---|---|
| Invoke equals() on the object you've already ensured is not null | 78 |
org/apache/myfaces/custom/swapimage/AbstractHtmlSwapImage.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 44 |
| Avoid unused private fields such as 'DEFAULT_ISMAP'. | 45 |
org/apache/myfaces/custom/tabbedpane/AbstractHtmlPanelTab.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 50 |
| Avoid unused private fields such as 'DEFAULT_DISABLED'. | 51 |
org/apache/myfaces/custom/tabbedpane/AbstractHtmlPanelTabbedPane.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 65 |
| Avoid unused private fields such as 'DEFAULT_SELECTEDINDEX'. | 66 |
| Avoid unused private fields such as 'DEFAULT_SERVER_SIDE_TAB_SWITCH'. | 67 |
| Avoid unused private fields such as 'DEFAULT_BORDER'. | 70 |
| Overriding method merely calls super | 72 - 75 |
org/apache/myfaces/custom/tabbedpane/HtmlPanelTabbedPaneTagHandler.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 39 - 42 |
org/apache/myfaces/custom/tree/AbstractHtmlTreeCheckbox.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 56 |
org/apache/myfaces/custom/tree/AbstractHtmlTreeImageCommandLink.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 39 |
org/apache/myfaces/custom/tree/HtmlTree.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 75 |
| Avoid empty if statements | 764 - 767 |
| Overriding method merely calls super | 774 - 777 |
org/apache/myfaces/custom/tree/renderkit/html/HtmlTreeCheckboxRenderer.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 123 - 125 |
org/apache/myfaces/custom/tree2/AbstractHtmlTree.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_RENDERER_TYPE'. | 57 |
org/apache/myfaces/custom/tree2/HtmlTreeRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 115 - 118 |
org/apache/myfaces/custom/tree2/TreeWalkerBase.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 105 - 111 |
org/apache/myfaces/custom/updateactionlistener/UpdateActionListener.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 181 |
| An empty statement (semicolon) not part of a loop | 182 |
org/apache/myfaces/dateformat/SimpleDateFormatter.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 1072 - 1074 |
org/apache/myfaces/renderkit/html/ext/HtmlCheckboxRenderer.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 443 - 445 |
org/apache/myfaces/renderkit/html/ext/HtmlMessageRenderer.java
| Violation | Line |
|---|---|
| Avoid empty while statements | 170 |
org/apache/myfaces/renderkit/html/ext/HtmlRadioRenderer.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 174 - 176 |
org/apache/myfaces/renderkit/html/ext/HtmlTableRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'facesContext'. | 312 - 464 |
| These nested if statements could be combined | 691 - 693 |
| Avoid unnecessary temporaries when converting primitives to Strings | 704 |
| These nested if statements could be combined | 722 - 724 |
org/apache/myfaces/renderkit/html/util/AddResourceFactory.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 347 - 350 |
org/apache/myfaces/renderkit/html/util/DefaultAddResource.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 650 - 657 |
org/apache/myfaces/renderkit/html/util/ExtensionsPhaseListener.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 132 - 142 |
org/apache/myfaces/renderkit/html/util/MyFacesResourceLoader.java
| Violation | Line |
|---|---|
| Avoid empty finally blocks | 221 - 224 |
org/apache/myfaces/renderkit/html/util/NonBufferingAddResource.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 556 - 560 |
org/apache/myfaces/tomahawk/util/ExternalContextUtils.java
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 59 - 78 |
| Unnecessary final modifier in final class | 87 - 107 |
| Unnecessary final modifier in final class | 116 - 155 |
| These nested if statements could be combined | 124 - 151 |
org/apache/myfaces/webapp/filter/ExtensionsFilter.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_uploadRepositoryPath'. | 137 |
org/apache/myfaces/webapp/filter/ExtensionsFilterConfig.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'filterName'. | 199 |
| Avoid empty if statements | 217 - 219 |
org/apache/myfaces/webapp/filter/TomahawkFacesContextWrapper.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 122 |
org/apache/myfaces/webapp/filter/WelcomeFileFilter.java
| Violation | Line |
|---|---|
| Avoid unnecessary return statements | 169 |