PMD Results
The following document contains the results of PMD 4.1.
Files
org/apache/myfaces/trinidadinternal/agent/AgentFactoryImpl.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'userAgent'. | 259 - 266 |
org/apache/myfaces/trinidadinternal/agent/CapabilityValue.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 54 - 56 |
org/apache/myfaces/trinidadinternal/agent/parse/CapabilitiesDocumentParser.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 81 |
| An empty statement (semicolon) not part of a loop | 97 |
org/apache/myfaces/trinidadinternal/agent/parse/CapabilityDataDocumentParser.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 92 |
org/apache/myfaces/trinidadinternal/agent/parse/NameVersion.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 365 - 366 |
org/apache/myfaces/trinidadinternal/application/NavigationHandlerImpl.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_LOG'. | 175 |
org/apache/myfaces/trinidadinternal/application/StateManagerImpl.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'context'. | 1042 - 1047 |
org/apache/myfaces/trinidadinternal/application/ViewHandlerImpl.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 156 |
| These nested if statements could be combined | 378 - 382 |
org/apache/myfaces/trinidadinternal/config/CheckSerializationConfigurator.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 975 - 978 |
| These nested if statements could be combined | 993 - 996 |
org/apache/myfaces/trinidadinternal/config/ConfigParser.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 104 |
org/apache/myfaces/trinidadinternal/config/GlobalConfiguratorImpl.java
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 80 - 112 |
| Unnecessary final modifier in final class | 137 - 155 |
org/apache/myfaces/trinidadinternal/config/upload/CompositeUploadedFileProcessorImpl.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'test'. | 113 |
org/apache/myfaces/trinidadinternal/config/upload/FileUploadConfiguratorImpl.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 201 - 205 |
| Avoid unused private methods such as '_copyParamsFromSessionToRequestMap(Map,Map,String)'. | 241 |
org/apache/myfaces/trinidadinternal/context/PageFlowScopeMap.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'cache'. | 72 - 94 |
org/apache/myfaces/trinidadinternal/context/PartialViewContextFactoryImpl.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_factory'. | 44 |
org/apache/myfaces/trinidadinternal/context/RequestContextImpl.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'facesContext'. | 929 - 931 |
org/apache/myfaces/trinidadinternal/context/external/NullEnumeration.java
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 30 - 33 |
org/apache/myfaces/trinidadinternal/context/external/ServletExternalContext.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 195 - 199 |
| Avoid unused private fields such as '_CHAR_ENCODING_CALLED'. | 707 |
org/apache/myfaces/trinidadinternal/convert/ByteConverter.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 122 - 139 |
| Avoid unused method parameters such as 'context'. | 122 - 139 |
org/apache/myfaces/trinidadinternal/convert/ConverterUtils.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 212 - 268 |
org/apache/myfaces/trinidadinternal/convert/DateTimeConverter.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 384 |
| Do not use if statements that are always true or always false | 405 |
| Avoid unused method parameters such as 'context'. | 579 - 597 |
org/apache/myfaces/trinidadinternal/convert/DoubleConverter.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 95 - 100 |
| Avoid unused method parameters such as 'context'. | 95 - 100 |
org/apache/myfaces/trinidadinternal/convert/FloatConverter.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 95 - 100 |
| Avoid unused method parameters such as 'context'. | 95 - 100 |
org/apache/myfaces/trinidadinternal/convert/IntegerConverter.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 129 - 146 |
| Avoid unused method parameters such as 'context'. | 129 - 146 |
org/apache/myfaces/trinidadinternal/convert/LongConverter.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 126 - 143 |
| Avoid unused method parameters such as 'context'. | 126 - 143 |
org/apache/myfaces/trinidadinternal/convert/NumberConverter.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 191 - 215 |
| Avoid unused private fields such as '_LOG'. | 236 |
org/apache/myfaces/trinidadinternal/convert/ShortConverter.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 125 - 142 |
| Avoid unused method parameters such as 'context'. | 125 - 142 |
org/apache/myfaces/trinidadinternal/el/OracleHelpProvider.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 64 - 68 |
org/apache/myfaces/trinidadinternal/facelets/TimezonePropertyTagRule.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_LOG'. | 94 |
org/apache/myfaces/trinidadinternal/image/ImageTypeImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 80 - 85 |
org/apache/myfaces/trinidadinternal/image/cache/BaseColorizedIconKey.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'context'. | 62 - 71 |
| Avoid unused constructor parameters such as 'context'. | 79 - 87 |
org/apache/myfaces/trinidadinternal/image/cache/Cache.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 235 |
org/apache/myfaces/trinidadinternal/image/cache/FileSystemImageCache.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 599 - 600 |
| Avoid unused method parameters such as 'context'. | 997 - 1002 |
| Avoid unused method parameters such as 'context'. | 1189 - 1200 |
| An empty statement (semicolon) not part of a loop | 1676 |
| These nested if statements could be combined | 1690 - 1699 |
org/apache/myfaces/trinidadinternal/image/encode/OctreeQuantizer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 241 - 246 |
org/apache/myfaces/trinidadinternal/image/encode/PNGEncoder.java
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 480 - 492 |
org/apache/myfaces/trinidadinternal/image/laf/browser/BlafImageUtils.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'styleID'. | 355 - 363 |
| Avoid unused method parameters such as 'context'. | 355 - 363 |
| Avoid unused method parameters such as 'isNamed'. | 355 - 363 |
| Avoid unused method parameters such as 'key'. | 355 - 363 |
| Avoid unused private fields such as '_STYLE_ERROR'. | 381 |
| Avoid unused private fields such as '_LOG'. | 386 |
org/apache/myfaces/trinidadinternal/image/laf/browser/ButtonPainter.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 401 |
org/apache/myfaces/trinidadinternal/image/painter/AdjustableBorderPainter.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'context'. | 113 - 116 |
| An empty statement (semicolon) not part of a loop | 144 |
| Avoid unused private fields such as '_styleNamePrefix'. | 161 |
org/apache/myfaces/trinidadinternal/image/painter/ImageUtils.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 275 |
| An empty statement (semicolon) not part of a loop | 311 |
| An empty statement (semicolon) not part of a loop | 322 |
org/apache/myfaces/trinidadinternal/image/xml/encode/AbstractXMLEncoder.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 353 - 354 |
org/apache/myfaces/trinidadinternal/image/xml/parse/FontParser.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 153 |
org/apache/myfaces/trinidadinternal/image/xml/parse/ImageParseUtils.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 86 |
org/apache/myfaces/trinidadinternal/io/DebugHtmlResponseWriter.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 80 - 81 |
org/apache/myfaces/trinidadinternal/io/IndentingResponseWriter.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 71 - 74 |
org/apache/myfaces/trinidadinternal/io/XhtmlResponseWriter.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'contentType'. | 46 - 56 |
org/apache/myfaces/trinidadinternal/menu/ImmutableGroupNode.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'accessKey'. | 188 |
| Avoid unused local variables such as 'label'. | 303 |
org/apache/myfaces/trinidadinternal/menu/ImmutableItemNode.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'accessKey'. | 230 |
| Avoid unused local variables such as 'label'. | 384 |
| Avoid unused local variables such as 'labelAndAcessKeyEval'. | 463 |
org/apache/myfaces/trinidadinternal/menu/MenuContentHandlerImpl.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'npe'. | 336 |
org/apache/myfaces/trinidadinternal/menu/MenuContentHandlerUsingApiImpl.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'npe'. | 339 |
org/apache/myfaces/trinidadinternal/renderkit/RenderKitBase.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 132 - 133 |
org/apache/myfaces/trinidadinternal/renderkit/core/CoreRenderKit.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_BASIC_HTML_RENDERERS'. | 741 |
org/apache/myfaces/trinidadinternal/renderkit/core/CoreRenderingContext.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'context'. | 605 - 647 |
| These nested if statements could be combined | 639 - 640 |
org/apache/myfaces/trinidadinternal/renderkit/core/CoreResponseStateManager.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 243 - 246 |
org/apache/myfaces/trinidadinternal/renderkit/core/desktop/TrainRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'bean'. | 916 - 954 |
| Avoid unused constructor parameters such as 'context'. | 958 - 988 |
org/apache/myfaces/trinidadinternal/renderkit/core/desktop/TreeTableRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 558 - 575 |
org/apache/myfaces/trinidadinternal/renderkit/core/pages/CalendarDialogJSP.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 154 |
org/apache/myfaces/trinidadinternal/renderkit/core/pages/JspUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 71 - 73 |
org/apache/myfaces/trinidadinternal/renderkit/core/ppr/PPRResponseWriter.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 452 - 460 |
| Avoid unused private fields such as '_allowedIds'. | 690 |
org/apache/myfaces/trinidadinternal/renderkit/core/ppr/PartialPageContextImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 256 - 257 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/AutoSubmitUtils.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_START'. | 344 |
| Avoid unused private fields such as '_START_RADIO'. | 345 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/BodyRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'rc'. | 459 - 472 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/ChooseDateRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'isInline'. | 1522 - 1525 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/CoreFormData.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 328 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FormElementRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 229 - 230 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FormRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 146 - 164 |
| These nested if statements could be combined | 1251 - 1272 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 328 - 337 |
| Avoid unused method parameters such as 'component'. | 339 - 348 |
| Avoid unused method parameters such as 'component'. | 350 - 359 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/InputColorRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 76 - 87 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/LabelAndMessageRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 372 - 390 |
| These nested if statements could be combined | 509 - 516 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/MediaRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 615 - 622 |
| These nested if statements could be combined | 618 - 621 |
| These nested if statements could be combined | 1461 - 1465 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/NavigationPaneRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 333 - 345 |
| Avoid unused local variables such as 'styleAppender'. | 594 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/OutputLabelRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 351 - 361 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/OutputUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 230 - 234 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelAccordionRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 523 - 577 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelBorderLayoutRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 876 - 933 |
| Avoid unused method parameters such as 'useLR'. | 876 - 933 |
| Avoid unused method parameters such as 'isIE6'. | 946 - 971 |
| Avoid unused method parameters such as 'bean'. | 946 - 971 |
| Avoid unused method parameters such as 'bean'. | 973 - 1019 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelFormLayoutRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1107 - 1109 |
| Avoid unused method parameters such as 'component'. | 1175 - 1180 |
| Avoid unused method parameters such as 'component'. | 1187 - 1192 |
| Avoid unused method parameters such as 'component'. | 1199 - 1204 |
| Avoid unused method parameters such as 'component'. | 1211 - 1216 |
| Avoid unused method parameters such as 'component'. | 1223 - 1228 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelGroupLayoutRenderer.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 82 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelPartialRootRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 146 - 150 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PollRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'isPartial'. | 345 - 363 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SelectRangeChoiceBarRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 888 - 894 |
| These nested if statements could be combined | 927 - 928 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/ShowDetailRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'bean'. | 516 - 545 |
| Avoid unused local variables such as 'linkId'. | 533 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SimpleInputNumberSpinboxRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 296 - 305 |
| Avoid unused method parameters such as 'component'. | 307 - 316 |
| Avoid unused method parameters such as 'component'. | 318 - 327 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SimpleInputTextRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 460 - 461 |
| An empty statement (semicolon) not part of a loop | 814 |
| An empty statement (semicolon) not part of a loop | 820 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SimpleSelectBooleanRadioRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 188 - 189 |
| These nested if statements could be combined | 217 - 231 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SimpleSelectManyRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 633 - 636 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SimpleSelectOneRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 127 - 129 |
| These nested if statements could be combined | 595 - 596 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SingleStepButtonBarRenderer.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_SINGLE_CONTINUE_TEXT_KEY'. | 330 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/TreeRenderer.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'showLines'. | 335 |
| An empty statement (semicolon) not part of a loop | 894 |
| An empty statement (semicolon) not part of a loop | 895 |
| An empty statement (semicolon) not part of a loop | 896 |
| Avoid unused method parameters such as 'isFirstSibling'. | 973 - 1015 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/XhtmlRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 359 - 363 |
| Do not use if statements that are always true or always false | 891 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/XhtmlUtils.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_CLIENT_BEHAVIORS_KEY'. | 803 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/table/ColumnData.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 418 |
| An empty statement (semicolon) not part of a loop | 418 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/table/TableRenderingContext.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'arc'. | 49 - 106 |
| These nested if statements could be combined | 503 - 507 |
org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/table/TreeNodeColumnRenderer.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 231 - 233 |
| Avoid empty catch blocks | 234 - 236 |
| Avoid empty catch blocks | 237 - 239 |
org/apache/myfaces/trinidadinternal/renderkit/html/CellFormatRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 94 - 97 |
org/apache/myfaces/trinidadinternal/renderkit/html/layout/CorePanelChoiceRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 298 - 315 |
org/apache/myfaces/trinidadinternal/renderkit/html/layout/CorePanelRadioRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 328 - 395 |
org/apache/myfaces/trinidadinternal/renderkit/html/layout/ShowOneListRendererBase.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 592 - 598 |
org/apache/myfaces/trinidadinternal/share/data/ServletRequestParameters.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 109 - 110 |
org/apache/myfaces/trinidadinternal/share/expl/Coercions.java
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 317 |
| An empty statement (semicolon) not part of a loop | 548 |
org/apache/myfaces/trinidadinternal/share/io/CachingNameResolver.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 143 - 147 |
org/apache/myfaces/trinidadinternal/share/io/DefaultNameResolver.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 90 |
org/apache/myfaces/trinidadinternal/share/nls/ImmutableDateFormatContext.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 47 - 50 |
org/apache/myfaces/trinidadinternal/share/nls/ImmutableDecimalFormatContext.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 60 - 63 |
org/apache/myfaces/trinidadinternal/share/nls/MutableLocaleContext.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 103 - 108 |
| Overriding method merely calls super | 116 - 121 |
| Unnecessary final modifier in final class | 129 - 136 |
| Unnecessary final modifier in final class | 145 - 154 |
org/apache/myfaces/trinidadinternal/share/url/EncoderUtils.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 650 |
org/apache/myfaces/trinidadinternal/share/util/CaboHttpUtils.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 139 |
org/apache/myfaces/trinidadinternal/share/util/MultipartFormHandler.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 674 |
org/apache/myfaces/trinidadinternal/share/xml/ParseContextWrapper.java
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 142 - 145 |
org/apache/myfaces/trinidadinternal/share/xml/beans/BeanParser.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 86 - 94 |
org/apache/myfaces/trinidadinternal/skin/AgentAtRuleMatcher.java
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 400 - 403 |
| Unnecessary final modifier in final class | 510 - 513 |
org/apache/myfaces/trinidadinternal/skin/SkinCSSParser.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 77 - 80 |
org/apache/myfaces/trinidadinternal/skin/SkinExtension.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 453 - 463 |
| These nested if statements could be combined | 479 - 487 |
org/apache/myfaces/trinidadinternal/skin/SkinImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 219 - 223 |
| An empty statement (semicolon) not part of a loop | 1023 |
org/apache/myfaces/trinidadinternal/skin/SkinStyleSheetParserUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 246 - 247 |
org/apache/myfaces/trinidadinternal/skin/SkinUtils.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 270 |
| These nested if statements could be combined | 587 - 588 |
org/apache/myfaces/trinidadinternal/skin/StyleSheetEntry.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 216 |
org/apache/myfaces/trinidadinternal/skin/StyleSheetNameResolver.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_LOG'. | 136 |
| Avoid unused private fields such as '_SERVICES_PROVIDER_KEY'. | 137 |
org/apache/myfaces/trinidadinternal/skin/icon/BaseImageIcon.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 217 - 218 |
org/apache/myfaces/trinidadinternal/style/cache/FileSystemStyleCache.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 959 - 960 |
| These nested if statements could be combined | 1123 - 1125 |
| These nested if statements could be combined | 1124 - 1125 |
| Avoid unused private fields such as '_outputDirectory'. | 1516 |
org/apache/myfaces/trinidadinternal/style/util/CSSGenerationUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 450 - 457 |
| Avoid empty if statements | 600 - 602 |
| Avoid unnecessary return statements | 609 |
| These nested if statements could be combined | 722 - 729 |
| Avoid unused method parameters such as 'selector'. | 880 - 913 |
| Avoid unused private fields such as '_SPACE_PATTERN'. | 1423 |
org/apache/myfaces/trinidadinternal/style/util/CSSUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 81 - 86 |
| An empty statement (semicolon) not part of a loop | 507 |
| These nested if statements could be combined | 809 - 814 |
org/apache/myfaces/trinidadinternal/style/xml/parse/ColorNodeParser.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 63 - 64 |
org/apache/myfaces/trinidadinternal/style/xml/parse/ColorValidater.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 47 - 48 |
org/apache/myfaces/trinidadinternal/style/xml/parse/StyleSheetDocument.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 1686 |
| An empty statement (semicolon) not part of a loop | 1711 |
org/apache/myfaces/trinidadinternal/style/xml/parse/StyleSheetDocumentParser.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 156 - 162 |
org/apache/myfaces/trinidadinternal/style/xml/parse/StyleSheetNode.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as '_copyLocaleArrayToSet(Locale)'. | 546 |
org/apache/myfaces/trinidadinternal/taglib/ForEachTag.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 344 - 348 |
org/apache/myfaces/trinidadinternal/taglib/listener/FileDownloadActionListenerTag.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'parentELContext'. | 74 |
| Avoid unused local variables such as 'application'. | 77 |
org/apache/myfaces/trinidadinternal/taglib/listener/ResetActionListenerTag.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 63 - 66 |
org/apache/myfaces/trinidadinternal/taglib/listener/ReturnActionListenerTag.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'parentELContext'. | 66 |
| Avoid unused local variables such as 'application'. | 69 |
org/apache/myfaces/trinidadinternal/taglib/listener/SetActionListenerTag.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'parentELContext'. | 72 |
| Avoid unused local variables such as 'application'. | 75 |
org/apache/myfaces/trinidadinternal/taglib/util/TagUtils.java
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 334 - 345 |
org/apache/myfaces/trinidadinternal/ui/BaseRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 87 - 88 |
org/apache/myfaces/trinidadinternal/ui/RootRenderingContext.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 345 - 357 |
| These nested if statements could be combined | 347 - 356 |
org/apache/myfaces/trinidadinternal/ui/beans/BaseWebBean.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 330 - 336 |
| Overriding method merely calls super | 344 - 348 |
org/apache/myfaces/trinidadinternal/ui/collection/FlaggedAttributeMap.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 75 - 78 |
org/apache/myfaces/trinidadinternal/ui/laf/base/BaseLafRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 571 - 575 |
org/apache/myfaces/trinidadinternal/ui/laf/base/BaseLafUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 144 - 145 |
| Avoid unused method parameters such as 'context'. | 1035 - 1069 |
| Avoid empty if statements | 1046 - 1059 |
| An empty statement (semicolon) not part of a loop | 1084 |
org/apache/myfaces/trinidadinternal/ui/laf/base/desktop/GlobalButtonRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 149 - 155 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/AccentedLafIconProvider.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 129 - 130 |
| Avoid unused method parameters such as 'styleClass'. | 244 - 288 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/BreadCrumbsRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'isLastChild'. | 375 - 392 |
| Avoid unused constructor parameters such as 'breadCrumbs'. | 443 - 455 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/ChoiceRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 84 - 95 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/FormElementRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 125 - 130 |
| These nested if statements could be combined | 154 - 158 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/HiddenLabelUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 105 - 115 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/ImageRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 350 - 353 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/LinkRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 104 - 105 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/NavigationBarRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1338 - 1342 |
| These nested if statements could be combined | 2448 - 2451 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/NavigationPathRenderer.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'isLastChild'. | 311 - 328 |
| Avoid unused constructor parameters such as 'breadCrumbs'. | 379 - 391 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/ProcessChoiceBarRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 696 - 699 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/ProcessUtils.java
| Violation | Line |
|---|---|
| Avoid unused private methods such as '_createKeyValueArray(String,String,String,String,long,String,int,String,String)'. | 351 - 361 |
org/apache/myfaces/trinidadinternal/ui/laf/base/xhtml/XhtmlLafRenderer.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 331 - 344 |
| These nested if statements could be combined | 1437 - 1448 |
| These nested if statements could be combined | 1478 - 1479 |
| These nested if statements could be combined | 1849 - 1853 |
| These nested if statements could be combined | 2163 - 2173 |
| These nested if statements could be combined | 2208 - 2209 |
org/apache/myfaces/trinidadinternal/ui/laf/simple/desktop/SimpleButtonUtils.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 116 - 121 |
| Avoid unused constructor parameters such as 'fContext'. | 191 - 225 |
| Avoid unused constructor parameters such as 'arc'. | 191 - 225 |
org/apache/myfaces/trinidadinternal/uinode/UINodeFacesBean.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 215 - 220 |
| These nested if statements could be combined | 238 - 239 |
org/apache/myfaces/trinidadinternal/util/JavaIntrospector.java
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 433 |
| An empty statement (semicolon) not part of a loop | 447 |
| An empty statement (semicolon) not part of a loop | 467 |
| These nested if statements could be combined | 660 - 674 |
| An empty statement (semicolon) not part of a loop | 1344 |
| An empty statement (semicolon) not part of a loop | 1552 |
| An empty statement (semicolon) not part of a loop | 2076 |
| An empty statement (semicolon) not part of a loop | 2149 |
| An empty statement (semicolon) not part of a loop | 2204 |
| An empty statement (semicolon) not part of a loop | 2243 |
org/apache/myfaces/trinidadinternal/util/MimeUtility.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 72 - 75 |
| An empty statement (semicolon) not part of a loop | 192 |
org/apache/myfaces/trinidadinternal/util/URLUtils.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 79 - 82 |
org/apache/myfaces/trinidadinternal/validator/ByteLengthValidator.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'maxLength'. | 141 - 166 |
| Avoid unused method parameters such as 'context'. | 141 - 166 |
org/apache/myfaces/trinidadinternal/validator/DateRestrictionValidator.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 135 - 166 |
| Avoid unused method parameters such as 'context'. | 135 - 166 |
org/apache/myfaces/trinidadinternal/validator/DateTimeRangeValidator.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 135 - 172 |
| Avoid unused method parameters such as 'context'. | 135 - 172 |
org/apache/myfaces/trinidadinternal/validator/DoubleRangeValidator.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 96 - 127 |
| Avoid unused method parameters such as 'context'. | 96 - 127 |
org/apache/myfaces/trinidadinternal/validator/LengthValidator.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 129 - 163 |
| Avoid unused method parameters such as 'context'. | 129 - 163 |
org/apache/myfaces/trinidadinternal/validator/LongRangeValidator.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'component'. | 97 - 128 |
| Avoid unused method parameters such as 'context'. | 97 - 128 |
org/apache/myfaces/trinidadinternal/validator/RegExpValidator.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'context'. | 82 - 107 |
org/apache/myfaces/trinidadinternal/taglib/UIXFormTag.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 42 - 45 |
org/apache/myfaces/trinidadinternal/taglib/validator/ValidateDateRestrictionTag.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 140 - 143 |