PMD Results

The following document contains the results of PMD 4.2.5.

Files

org/apache/myfaces/tobago/facelets/AbstractTobagoTagLibrary.java

ViolationLine
Parameter 'namespace' is not assigned and could be declared final27
Parameter 'rendererType' is not assigned and could be declared final42
Parameter 'name' is not assigned and could be declared final42
Parameter 'handlerType' is not assigned and could be declared final42
Parameter 'componentType' is not assigned and could be declared final42

org/apache/myfaces/tobago/facelets/AttributeHandler.java

ViolationLine
The class 'AttributeHandler' has a Cyclomatic Complexity of 8 (Highest = 60).50 - 345
The class 'AttributeHandler' has a Cyclomatic Complexity of 8 (Highest = 60).50 - 345
Parameter 'config' is not assigned and could be declared final60
Parameter 'parent' is not assigned and could be declared final67
Parameter 'faceletContext' is not assigned and could be declared final67
The method 'apply' has a Cyclomatic Complexity of 60.67 - 279
Avoid really long methods.67 - 279
The method 'apply' has a Cyclomatic Complexity of 60.67 - 279
The method apply() has an NCSS line count of 16367 - 279
Avoid if (x != y) ..; else ..;75 - 277
Avoid if (x != y) ..; else ..;79 - 97
Local variable 'expression' could be declared final82 - 83
Avoid if (x != y) ..; else ..;102 - 120
Local variable 'expression' could be declared final105 - 106
Local variable 'expression' could be declared final126 - 127
Assigning an Object to null is a code smell. Consider refactoring.133
Local variable 'expressionFactory' could be declared final143
Local variable 'action' could be declared final144 - 145
Local variable 'expression' could be declared final153 - 154
Assigning an Object to null is a code smell. Consider refactoring.160
Assigning an Object to null is a code smell. Consider refactoring.167
Local variable 'expressionFactory' could be declared final172
Local variable 'actionListener' could be declared final173 - 174
Local variable 'result' could be declared final180
Local variable 'expression' could be declared final186 - 187
Assigning an Object to null is a code smell. Consider refactoring.193
Local variable 'expression' could be declared final201
Local variable 'nameValue' could be declared final212
Local variable 'components' could be declared final223
Local variable 'expression' could be declared final236
Local variable 'methodExpression' could be declared final243
Local variable 'methodExpression' could be declared final250 - 251
Local variable 'action' could be declared final259
Local variable 'action' could be declared final265
Parameter 'string' is not assigned and could be declared final281
Parameter 'string' is not assigned and could be declared final285
Parameter 'string' is not assigned and could be declared final290
Parameter 'string' is not assigned and could be declared final294
Parameter 'faceletContext' is not assigned and could be declared final298
Local variable 'myValue' could be declared final299
Parameter 'args' is not assigned and could be declared final303
Parameter 'faceletContext' is not assigned and could be declared final303
Parameter 'returnType' is not assigned and could be declared final303
Local variable 'expression' could be declared final308
Avoid if (x != y) ..; else ..;309 - 315
Local variable 'expressionFactory' could be declared final310
Parameter 'nameValue' is not assigned and could be declared final321
Parameter 'parent' is not assigned and could be declared final321
Parameter 'faceletContext' is not assigned and could be declared final321
Local variable 'expression' could be declared final326
Parameter 'expression' is not assigned and could be declared final336
Parameter 'nameValue' is not assigned and could be declared final336
Parameter 'parent' is not assigned and could be declared final336
Parameter 'faceletContext' is not assigned and could be declared final336
Local variable 'converter' could be declared final338 - 339

org/apache/myfaces/tobago/facelets/ContentTypeRule.java

ViolationLine
Parameter 'metadataTarget' is not assigned and could be declared final32
Parameter 'name' is not assigned and could be declared final32
Parameter 'attribute' is not assigned and could be declared final32
Parameter 'attribute' is not assigned and could be declared final43
Parameter 'ctx' is not assigned and could be declared final47
Parameter 'instance' is not assigned and could be declared final47
Local variable 'components' could be declared final48

org/apache/myfaces/tobago/facelets/ConverterHandler.java

ViolationLine
The class 'ConverterHandler' has a Cyclomatic Complexity of 5 (Highest = 8).41 - 84
Parameter 'config' is not assigned and could be declared final47
Parameter 'parent' is not assigned and could be declared final53
Parameter 'faceletContext' is not assigned and could be declared final53
The method 'apply' has a Cyclomatic Complexity of 8.53 - 83
Local variable 'valueHolder' could be declared final57
New exception is thrown in catch block, original stack trace may be lost69
Deeply nested if..then statements are hard to read71 - 73

org/apache/myfaces/tobago/facelets/DeprecatedDimensionRule.java

ViolationLine
Parameter 'metadataTarget' is not assigned and could be declared final37
Parameter 'name' is not assigned and could be declared final37
Parameter 'attribute' is not assigned and could be declared final37
Parameter 'attribute' is not assigned and could be declared final50
Parameter 'ctx' is not assigned and could be declared final54
Parameter 'instance' is not assigned and could be declared final54
Local variable 'expression' could be declared final58
Parameter 'attribute' is not assigned and could be declared final67
Parameter 'ctx' is not assigned and could be declared final71
Parameter 'instance' is not assigned and could be declared final71
Local variable 'expression' could be declared final75

org/apache/myfaces/tobago/facelets/FixCharacterEncodingFilter.java

ViolationLine
Document empty method34 - 36
Parameter 'filterConfig' is not assigned and could be declared final34
Parameter 'servletRequest' is not assigned and could be declared final38
Parameter 'filterChain' is not assigned and could be declared final39
Parameter 'servletResponse' is not assigned and could be declared final39
Document empty method47 - 48

org/apache/myfaces/tobago/facelets/FlowLayoutRule.java

ViolationLine
The class 'FlowLayoutRule' has a Cyclomatic Complexity of 10 (Highest = 9).30 - 137
The class 'FlowLayoutRule' has a Cyclomatic Complexity of 10 (Highest = 9).30 - 137
Parameter 'metadataTarget' is not assigned and could be declared final34
Parameter 'name' is not assigned and could be declared final34
Parameter 'attribute' is not assigned and could be declared final34
The method 'applyRule' has a Cyclomatic Complexity of 9.34 - 58
These nested if statements could be combined36 - 55
These nested if statements could be combined36 - 55
Deeply nested if..then statements are hard to read37 - 39
Deeply nested if..then statements are hard to read40 - 42
Deeply nested if..then statements are hard to read43 - 45
Deeply nested if..then statements are hard to read46 - 48
Deeply nested if..then statements are hard to read49 - 51
Deeply nested if..then statements are hard to read52 - 54
Parameter 'attribute' is not assigned and could be declared final63
Parameter 'ctx' is not assigned and could be declared final67
Parameter 'instance' is not assigned and could be declared final67
Local variable 'gridLayout' could be declared final68
Parameter 'attribute' is not assigned and could be declared final76
Parameter 'ctx' is not assigned and could be declared final80
Parameter 'instance' is not assigned and could be declared final80
Local variable 'gridLayout' could be declared final81
Parameter 'attribute' is not assigned and could be declared final89
Parameter 'ctx' is not assigned and could be declared final93
Parameter 'instance' is not assigned and could be declared final93
Local variable 'gridLayout' could be declared final94
Parameter 'attribute' is not assigned and could be declared final102
Parameter 'ctx' is not assigned and could be declared final106
Parameter 'instance' is not assigned and could be declared final106
Local variable 'gridLayout' could be declared final107
Parameter 'attribute' is not assigned and could be declared final115
Parameter 'ctx' is not assigned and could be declared final119
Parameter 'instance' is not assigned and could be declared final119
Local variable 'gridLayout' could be declared final120
Parameter 'attribute' is not assigned and could be declared final128
Parameter 'ctx' is not assigned and could be declared final132
Parameter 'instance' is not assigned and could be declared final132
Local variable 'gridLayout' could be declared final133

org/apache/myfaces/tobago/facelets/GridLayoutConstraintHandler.java

ViolationLine
The class 'GridLayoutConstraintHandler' has a Cyclomatic Complexity of 32 (Highest = 62).38 - 343
The class 'GridLayoutConstraintHandler' has a Cyclomatic Complexity of 32 (Highest = 62).38 - 343
Too many fields38 - 343
Parameter 'config' is not assigned and could be declared final72
Avoid reassigning parameters such as 'parent'98
Avoid reassigning parameters such as 'parent'98
Parameter 'faceletContext' is not assigned and could be declared final98
The method 'apply' has a Cyclomatic Complexity of 62.98 - 342
The method apply() has an NPath complexity of 210640429898 - 342
Avoid really long methods.98 - 342
The method 'apply' has a Cyclomatic Complexity of 62.98 - 342
The method apply() has an NCSS line count of 14098 - 342
No need to check for null before an instanceof100
No need to check for null before an instanceof102 - 103
Local variable 'component' could be declared final107
Local variable 'container' could be declared final240

org/apache/myfaces/tobago/facelets/GridLayoutRule.java

ViolationLine
The class 'GridLayoutRule' has a Cyclomatic Complexity of 12 (Highest = 11).29 - 168
The class 'GridLayoutRule' has a Cyclomatic Complexity of 12 (Highest = 11).29 - 168
Parameter 'metadataTarget' is not assigned and could be declared final33
Parameter 'name' is not assigned and could be declared final33
Parameter 'attribute' is not assigned and could be declared final33
The method 'applyRule' has a Cyclomatic Complexity of 11.33 - 63
The method applyRule() has an NPath complexity of 25833 - 63
The method 'applyRule' has a Cyclomatic Complexity of 11.33 - 63
These nested if statements could be combined35 - 60
These nested if statements could be combined35 - 60
Deeply nested if..then statements are hard to read36 - 38
Deeply nested if..then statements are hard to read39 - 41
Deeply nested if..then statements are hard to read42 - 44
Deeply nested if..then statements are hard to read45 - 47
Deeply nested if..then statements are hard to read48 - 50
Deeply nested if..then statements are hard to read51 - 53
Deeply nested if..then statements are hard to read54 - 56
Deeply nested if..then statements are hard to read57 - 59
Parameter 'attribute' is not assigned and could be declared final68
Parameter 'ctx' is not assigned and could be declared final72
Parameter 'instance' is not assigned and could be declared final72
Local variable 'gridLayout' could be declared final73
Parameter 'attribute' is not assigned and could be declared final81
Parameter 'ctx' is not assigned and could be declared final85
Parameter 'instance' is not assigned and could be declared final85
Local variable 'gridLayout' could be declared final86
Parameter 'attribute' is not assigned and could be declared final94
Parameter 'ctx' is not assigned and could be declared final98
Parameter 'instance' is not assigned and could be declared final98
Local variable 'gridLayout' could be declared final99
Parameter 'attribute' is not assigned and could be declared final107
Parameter 'ctx' is not assigned and could be declared final111
Parameter 'instance' is not assigned and could be declared final111
Local variable 'gridLayout' could be declared final112
Parameter 'attribute' is not assigned and could be declared final120
Parameter 'ctx' is not assigned and could be declared final124
Parameter 'instance' is not assigned and could be declared final124
Local variable 'gridLayout' could be declared final125
Parameter 'attribute' is not assigned and could be declared final133
Parameter 'ctx' is not assigned and could be declared final137
Parameter 'instance' is not assigned and could be declared final137
Local variable 'gridLayout' could be declared final138
Parameter 'attribute' is not assigned and could be declared final146
Parameter 'ctx' is not assigned and could be declared final150
Parameter 'instance' is not assigned and could be declared final150
Local variable 'gridLayout' could be declared final151
Parameter 'attribute' is not assigned and could be declared final159
Parameter 'ctx' is not assigned and could be declared final163
Parameter 'instance' is not assigned and could be declared final163
Local variable 'gridLayout' could be declared final164

org/apache/myfaces/tobago/facelets/LoadBundleHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final44
Parameter 'parent' is not assigned and could be declared final50
Parameter 'faceletContext' is not assigned and could be declared final50
Local variable 'name' could be declared final52
Local variable 'map' could be declared final53
Local variable 'facesContext' could be declared final54

org/apache/myfaces/tobago/facelets/MetaInfResourcesClasspathResourceResolver.java

ViolationLine
The class 'MetaInfResourcesClasspathResourceResolver' has a Cyclomatic Complexity of 4 (Highest = 5).43 - 71
Avoid reassigning parameters such as 'path'53
Avoid reassigning parameters such as 'path'53
The method 'resolveUrl' has a Cyclomatic Complexity of 5.53 - 70
This call to String.startsWith can be rewritten using String.charAt(0)59

org/apache/myfaces/tobago/facelets/PopupReferenceHandler.java

ViolationLine
The class 'PopupReferenceHandler' has a Cyclomatic Complexity of 4 (Highest = 5).44 - 74
Parameter 'config' is not assigned and could be declared final48
Parameter 'parent' is not assigned and could be declared final53
Parameter 'faceletContext' is not assigned and could be declared final53
The method 'apply' has a Cyclomatic Complexity of 5.53 - 73
Local variable 'actionSource' could be declared final57
Local variable 'forValueExpression' could be declared final61

org/apache/myfaces/tobago/facelets/PositionRule.java

ViolationLine
Parameter 'metadataTarget' is not assigned and could be declared final37
Parameter 'name' is not assigned and could be declared final37
Parameter 'attribute' is not assigned and could be declared final37
Parameter 'attribute' is not assigned and could be declared final50
Parameter 'ctx' is not assigned and could be declared final54
Parameter 'instance' is not assigned and could be declared final54
Local variable 'expression' could be declared final58
Parameter 'attribute' is not assigned and could be declared final67
Parameter 'ctx' is not assigned and could be declared final71
Parameter 'instance' is not assigned and could be declared final71
Local variable 'expression' could be declared final75

org/apache/myfaces/tobago/facelets/ResetInputActionListenerHandler.java

ViolationLine
The class 'ResetInputActionListenerHandler' has a Cyclomatic Complexity of 4 (Highest = 6).41 - 73
Parameter 'config' is not assigned and could be declared final45
Parameter 'parent' is not assigned and could be declared final50
Parameter 'faceletContext' is not assigned and could be declared final50
The method 'apply' has a Cyclomatic Complexity of 6.50 - 72
Local variable 'actionSource' could be declared final54
Local variable 'forValueExpression' could be declared final60

org/apache/myfaces/tobago/facelets/ScriptHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final36
Parameter 'component' is not assigned and could be declared final40
Parameter 'context' is not assigned and could be declared final40
Parameter 'parent' is not assigned and could be declared final40
Local variable 'content' could be declared final41
Local variable 'iter' could be declared final42
Local variable 'text' could be declared final44
Document empty method53 - 54
Parameter 'c' is not assigned and could be declared final53
Parameter 'ctx' is not assigned and could be declared final53

org/apache/myfaces/tobago/facelets/SheetStateChangeSourceRule.java

ViolationLine
Parameter 'name' is not assigned and could be declared final38
Parameter 'attribute' is not assigned and could be declared final38
Parameter 'metadataTarget' is not assigned and could be declared final39
These nested if statements could be combined41 - 43
These nested if statements could be combined41 - 43
Parameter 'attribute' is not assigned and could be declared final52
Parameter 'ctx' is not assigned and could be declared final56
Parameter 'instance' is not assigned and could be declared final56

org/apache/myfaces/tobago/facelets/SortActionSourceRule.java

ViolationLine
Parameter 'name' is not assigned and could be declared final39
Parameter 'attribute' is not assigned and could be declared final39
Parameter 'metadataTarget' is not assigned and could be declared final40
These nested if statements could be combined42 - 44
These nested if statements could be combined42 - 44
Parameter 'attribute' is not assigned and could be declared final53
Parameter 'ctx' is not assigned and could be declared final57
Parameter 'instance' is not assigned and could be declared final57

org/apache/myfaces/tobago/facelets/StyleHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final35
Parameter 'component' is not assigned and could be declared final39
Parameter 'context' is not assigned and could be declared final39
Parameter 'parent' is not assigned and could be declared final39
Local variable 'content' could be declared final40
Local variable 'iter' could be declared final41
Local variable 'text' could be declared final43
Document empty method51 - 52
Parameter 'c' is not assigned and could be declared final51
Parameter 'ctx' is not assigned and could be declared final51

org/apache/myfaces/tobago/facelets/SuggestMethodRule.java

ViolationLine
Parameter 'name' is not assigned and could be declared final37
Parameter 'attribute' is not assigned and could be declared final37
Parameter 'metadataTarget' is not assigned and could be declared final38
These nested if statements could be combined40 - 42
These nested if statements could be combined40 - 42
Parameter 'attribute' is not assigned and could be declared final50
Parameter 'ctx' is not assigned and could be declared final54
Parameter 'instance' is not assigned and could be declared final54

org/apache/myfaces/tobago/facelets/SupportsMarkupRule.java

ViolationLine
Parameter 'metadataTarget' is not assigned and could be declared final37
Parameter 'name' is not assigned and could be declared final37
Parameter 'attribute' is not assigned and could be declared final37
These nested if statements could be combined39 - 41
These nested if statements could be combined39 - 41
Parameter 'attribute' is not assigned and could be declared final50
Parameter 'ctx' is not assigned and could be declared final54
Parameter 'instance' is not assigned and could be declared final54
Local variable 'expression' could be declared final58

org/apache/myfaces/tobago/facelets/SupportsRenderedPartiallyRule.java

ViolationLine
Parameter 'metadataTarget' is not assigned and could be declared final37
Parameter 'name' is not assigned and could be declared final37
Parameter 'attribute' is not assigned and could be declared final37
These nested if statements could be combined39 - 41
These nested if statements could be combined39 - 41
Parameter 'attribute' is not assigned and could be declared final50
Parameter 'ctx' is not assigned and could be declared final54
Parameter 'instance' is not assigned and could be declared final54
Local variable 'components' could be declared final56
Local variable 'expression' could be declared final59

org/apache/myfaces/tobago/facelets/TabChangeListenerHandler.java

ViolationLine
The class 'TabChangeListenerHandler' has a Cyclomatic Complexity of 7 (Highest = 9).42 - 104
Parameter 'config' is not assigned and could be declared final51
Parameter 'parent' is not assigned and could be declared final67
Parameter 'faceletContext' is not assigned and could be declared final67
The method 'apply' has a Cyclomatic Complexity of 9.67 - 103
Local variable 'changeSource' could be declared final72
New exception is thrown in catch block, original stack trace may be lost83
Deeply nested if..then statements are hard to read85 - 87
Avoid if (x != y) ..; else ..;89 - 98

org/apache/myfaces/tobago/facelets/TabChangeSourceRule.java

ViolationLine
Parameter 'name' is not assigned and could be declared final38
Parameter 'attribute' is not assigned and could be declared final38
Parameter 'metadataTarget' is not assigned and could be declared final39
These nested if statements could be combined41 - 43
These nested if statements could be combined41 - 43
Parameter 'attribute' is not assigned and could be declared final52
Parameter 'ctx' is not assigned and could be declared final56
Parameter 'instance' is not assigned and could be declared final56

org/apache/myfaces/tobago/facelets/TobagoComponentHandler.java

ViolationLine
The class 'TobagoComponentHandler' has a Cyclomatic Complexity of 5 (Highest = 11).48 - 136
The class 'TobagoComponentHandler' has a Cyclomatic Complexity of 5 (Highest = 11).48 - 136
Parameter 'componentConfig' is not assigned and could be declared final50
Parameter 'aClass' is not assigned and could be declared final54
The method 'createMetaRuleset' has a Cyclomatic Complexity of 11.54 - 87
The method createMetaRuleset() has an NPath complexity of 102454 - 87
The method 'createMetaRuleset' has a Cyclomatic Complexity of 11.54 - 87
Local variable 'metaRuleset' could be declared final55
Parameter 'component' is not assigned and could be declared final89
Parameter 'context' is not assigned and could be declared final89
Parameter 'parent' is not assigned and could be declared final89
Parameter 'component' is not assigned and could be declared final97
Parameter 'context' is not assigned and could be declared final97
Parameter 'parent' is not assigned and could be declared final97
Parameter 'component' is not assigned and could be declared final109
Parameter 'context' is not assigned and could be declared final109
The method 'addDefaultValidators' has a Cyclomatic Complexity of 7.109 - 135
Local variable 'tobagoConfig' could be declared final110
Local variable 'validatorInfoMap' could be declared final111
Local variable 'validators' could be declared final115
Local variable 'classNames' could be declared final117
Local variable 'it' could be declared final122
Local variable 'entry' could be declared final124
Local variable 'it' could be declared final130

org/apache/myfaces/tobago/facelets/TobagoValidateHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final27
Parameter 'aClass' is not assigned and could be declared final31
Local variable 'metaRuleset' could be declared final32

org/apache/myfaces/tobago/facelets/WizardComponentHandler.java

ViolationLine
Private field 'outcomeAttribute' could be made final; it is only initialized in the declaration or constructor.30
Parameter 'componentConfig' is not assigned and could be declared final32
Parameter 'parent' is not assigned and could be declared final37
Parameter 'wizard' is not assigned and could be declared final37
Parameter 'faceletContext' is not assigned and could be declared final37
Local variable 'expression' could be declared final43

org/apache/myfaces/tobago/facelets/extension/DateExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final34
Parameter 'panel' is not assigned and could be declared final46
Parameter 'parent' is not assigned and could be declared final46
Parameter 'faceletContext' is not assigned and could be declared final46
Local variable 'application' could be declared final49
Local variable 'picker' could be declared final50
Local variable 'root' could be declared final53
Parameter 'first' is not assigned and could be declared final63

org/apache/myfaces/tobago/facelets/extension/FileExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final30

org/apache/myfaces/tobago/facelets/extension/InExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final26

org/apache/myfaces/tobago/facelets/extension/MenuCheckboxExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final38

org/apache/myfaces/tobago/facelets/extension/MenuRadioExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final42

org/apache/myfaces/tobago/facelets/extension/SelectBooleanCheckboxExtensionHandler.java

ViolationLine
Private field 'itemLabelAttribute' could be made final; it is only initialized in the declaration or constructor.33
Parameter 'config' is not assigned and could be declared final35
Parameter 'input' is not assigned and could be declared final40
Parameter 'faceletContext' is not assigned and could be declared final40
Local variable 'checkbox' could be declared final42
Avoid if (x != y) ..; else ..;43 - 52
Local variable 'expression' could be declared final47

org/apache/myfaces/tobago/facelets/extension/SelectManyCheckboxExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final26

org/apache/myfaces/tobago/facelets/extension/SelectManyListboxExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final30

org/apache/myfaces/tobago/facelets/extension/SelectManyShuttleExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final26

org/apache/myfaces/tobago/facelets/extension/SelectOneChoiceExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final30

org/apache/myfaces/tobago/facelets/extension/SelectOneListboxExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final30

org/apache/myfaces/tobago/facelets/extension/SelectOneRadioExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final26

org/apache/myfaces/tobago/facelets/extension/SeparatorExtensionHandler.java

ViolationLine
Private field 'labelAttribute' could be made final; it is only initialized in the declaration or constructor.42
Parameter 'config' is not assigned and could be declared final44
Parameter 'faceletContext' is not assigned and could be declared final49
Parameter 'separator' is not assigned and could be declared final49
Local variable 'component' could be declared final52
Parameter 'parent' is not assigned and could be declared final60
Parameter 'faceletContext' is not assigned and could be declared final60
Parameter 'separator' is not assigned and could be declared final60
Local variable 'application' could be declared final61
Local variable 'root' could be declared final62
Local variable 'label' could be declared final63
Local variable 'expression' could be declared final72
Parameter 'aClass' is not assigned and could be declared final78
Local variable 'metaRuleset' could be declared final79
Local variable 'attrs' could be declared final84
Local variable 'attr' could be declared final86

org/apache/myfaces/tobago/facelets/extension/TextareaExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final30

org/apache/myfaces/tobago/facelets/extension/TimeExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final26

org/apache/myfaces/tobago/facelets/extension/TobagoExtensionTagLibrary.java

ViolationLine
The String literal "Panel" appears 12 times in this file; the first occurrence is on line 3030

org/apache/myfaces/tobago/facelets/extension/TobagoLabelExtensionHandler.java

ViolationLine
A high number of imports can indicate a high degree of coupling within an object.1 - 267
The class 'TobagoLabelExtensionHandler' has a Cyclomatic Complexity of 3 (Highest = 8).60 - 267
Private field 'labelWidthAttribute' could be made final; it is only initialized in the declaration or constructor.63
Private field 'tipAttribute' could be made final; it is only initialized in the declaration or constructor.64
Private field 'labelAttribute' could be made final; it is only initialized in the declaration or constructor.65
Private field 'markupAttribute' could be made final; it is only initialized in the declaration or constructor.66
Private field 'fieldIdAttribute' could be made final; it is only initialized in the declaration or constructor.67
Parameter 'config' is not assigned and could be declared final71
Parameter 'first' is not assigned and could be declared final88
Parameter 'panel' is not assigned and could be declared final92
Parameter 'ctx' is not assigned and could be declared final92
Local variable 'input' could be declared final96
Local variable 'input' could be declared final112
Parameter 'panel' is not assigned and could be declared final117
Parameter 'parent' is not assigned and could be declared final117
Parameter 'faceletContext' is not assigned and could be declared final117
Local variable 'application' could be declared final119
Local variable 'root' could be declared final120
Avoid if (x != y) ..; else ..;126 - 130
Local variable 'input' could be declared final135
Document empty method145 - 146
An empty method in an abstract class should be abstract instead145 - 146
Parameter 'input' is not assigned and could be declared final145
Parameter 'faceletContext' is not assigned and could be declared final145
Parameter 'panel' is not assigned and could be declared final148
Parameter 'root' is not assigned and could be declared final148
Parameter 'faceletContext' is not assigned and could be declared final148
The method 'addLabel' has a Cyclomatic Complexity of 8.148 - 183
Local variable 'uid' could be declared final149
Local variable 'application' could be declared final153
Local variable 'label' could be declared final154
Local variable 'expression' could be declared final162
Local variable 'expression' could be declared final170
Local variable 'expression' could be declared final178
Parameter 'panel' is not assigned and could be declared final185
Parameter 'uid' is not assigned and could be declared final185
Local variable 'list' could be declared final187
Local variable 'child' could be declared final189
Parameter 'component' is not assigned and could be declared final198
Parameter 'parent' is not assigned and could be declared final198
Parameter 'faceletContext' is not assigned and could be declared final198
Parameter 'panel' is not assigned and could be declared final206
Parameter 'root' is not assigned and could be declared final206
Parameter 'faceletContext' is not assigned and could be declared final206
The method 'addGridLayout' has a Cyclomatic Complexity of 5.206 - 229
Local variable 'application' could be declared final207
Local variable 'gridLayout' could be declared final208
Avoid if (x != y) ..; else ..;210 - 219
Parameter 'ctx' is not assigned and could be declared final231
Parameter 'instance' is not assigned and could be declared final231
Local variable 'type' could be declared final233
Parameter 'aClass' is not assigned and could be declared final242
Local variable 'metaRuleset' could be declared final243
Parameter 'aClass' is not assigned and could be declared final256
Local variable 'metaRuleset' could be declared final257
Local variable 'attrs' could be declared final258
Local variable 'attr' could be declared final260

org/apache/myfaces/tobago/facelets/extension/TobagoMenuExtensionHandler.java

ViolationLine
Parameter 'config' is not assigned and could be declared final45
Parameter 'menuCommand' is not assigned and could be declared final55
Parameter 'faceletContext' is not assigned and could be declared final55
Local variable 'component' could be declared final58
Parameter 'menuCommand' is not assigned and could be declared final66
Parameter 'parent' is not assigned and could be declared final66
Parameter 'faceletContext' is not assigned and could be declared final66
Local variable 'application' could be declared final68
Local variable 'root' could be declared final69
Local variable 'component' could be declared final70
Parameter 'ctx' is not assigned and could be declared final77
Parameter 'instance' is not assigned and could be declared final77
Local variable 'type' could be declared final79
Parameter 'aClass' is not assigned and could be declared final88
Local variable 'metaRuleset' could be declared final89
Local variable 'attrs' could be declared final90
Local variable 'attr' could be declared final92
Parameter 'aClass' is not assigned and could be declared final101
Local variable 'metaRuleset' could be declared final102