PMD Results

The following document contains the results of PMD 5.2.3.

Files

Tobago Core - org/apache/myfaces/tobago/compat/FacesUtilsEL.java

Violation Line
This class has too many methods, consider refactoring it. 48268
These nested if statements could be combined 7274
Avoid if (x != y) ..; else ..; 206
Avoid if (x != y) ..; else ..; 216221

Tobago Core - org/apache/myfaces/tobago/component/Attributes.java

Violation Line
Parameter 'explicit' is not assigned and could be declared final 267
Avoid if (x != y) ..; else ..; 272276
Parameter 'name' is not assigned and could be declared final 279

Tobago Core - org/apache/myfaces/tobago/component/LabelLayout.java

Violation Line
Parameter 'facesContext' is not assigned and could be declared final 88

Tobago Core - org/apache/myfaces/tobago/component/MethodBindingToMethodExpression.java

Violation Line
This class has too many methods, consider refactoring it. 37241
Document empty constructor 4748
Consider using varargs for methods or constructors which take an array the last parameter. 96

Tobago Core - org/apache/myfaces/tobago/component/MethodExpressionToMethodBinding.java

Violation Line
Avoid using redundant field initializer for 'isTransient' 38
Unnecessary use of fully qualified name 'javax.faces.el.MethodNotFoundException' due to existing import 'javax.faces.el.MethodNotFoundException' 63
Consider using varargs for methods or constructors which take an array the last parameter. 70
Unnecessary use of fully qualified name 'javax.faces.el.MethodNotFoundException' due to existing import 'javax.faces.el.MethodNotFoundException' 76

Tobago Core - org/apache/myfaces/tobago/component/Sorter.java

Violation Line
Possible God class (WMC=51, ATFD=25, TCC=0.2) 20259
The class 'Sorter' has a Modified Cyclomatic Complexity of 6 (Highest = 28). 49258
The class 'Sorter' has a Standard Cyclomatic Complexity of 6 (Highest = 28). 49258
The class 'Sorter' has a Cyclomatic Complexity of 8 (Highest = 30). 49258
The method perform() has an NPath complexity of 139872 64201
The method 'perform' has a Modified Cyclomatic Complexity of 28. 64201
The method 'perform' has a Cyclomatic Complexity of 30. 64201
Avoid really long methods. 64201
The method 'perform' has a Standard Cyclomatic Complexity of 28. 64201
Avoid if (x != y) ..; else ..; 96137
Avoid if (x != y) ..; else ..; 99132
Avoid if (x != y) ..; else ..; 199
The method 'getFirstSortableChild' has a Cyclomatic Complexity of 13. 224249

Tobago Core - org/apache/myfaces/tobago/component/SupportsRenderedPartially.java

Violation Line
Consider using varargs for methods or constructors which take an array the last parameter. 27

Tobago Core - org/apache/myfaces/tobago/context/ClientProperties.java

Violation Line
Use equals() to compare object references. 133

Tobago Core - org/apache/myfaces/tobago/context/Markup.java

Violation Line
This class has too many methods, consider refactoring it. 57328
Consider using varargs for methods or constructors which take an array the last parameter. 106
Consider using varargs for methods or constructors which take an array the last parameter. 116
Avoid if (x != y) ..; else ..; 176
Avoid if (x != y) ..; else ..; 188
Avoid if (x != y) ..; else ..; 189
Use equals() to compare object references. 212
Avoid if (x != y) ..; else ..; 215226
Avoid if (x != y) ..; else ..; 256265

Tobago Core - org/apache/myfaces/tobago/context/MarkupConfig.java

Violation Line
Private field 'markups' could be made final; it is only initialized in the declaration or constructor. 30

Tobago Core - org/apache/myfaces/tobago/context/ResourceManagerUtils.java

Violation Line
This call to String.startsWith can be rewritten using String.charAt(0) 184
Local variable 'dot' could be declared final 194
Local variable 'slash' could be declared final 198

Tobago Core - org/apache/myfaces/tobago/context/ThemeImpl.java

Violation Line
Possible God class (WMC=60, ATFD=16, TCC=0.07816091954022988) 20314
The class 'ThemeImpl' has a Modified Cyclomatic Complexity of 2 (Highest = 12). 33314
The class 'ThemeImpl' has a Standard Cyclomatic Complexity of 2 (Highest = 12). 33314
Too many fields 33314
The class 'ThemeImpl' has a Cyclomatic Complexity of 2 (Highest = 12). 33314
Private field 'productionResources' could be made final; it is only initialized in the declaration or constructor. 46
Private field 'resources' could be made final; it is only initialized in the declaration or constructor. 47
Avoid using redundant field initializer for 'unmodifiable' 55
Deeply nested if..then statements are hard to read 154159
The method 'toString' has a Cyclomatic Complexity of 12. 273313
The method 'toString' has a Standard Cyclomatic Complexity of 12. 273313
The method 'toString' has a Modified Cyclomatic Complexity of 12. 273313
StringBuffer constructor is initialized with size 16, but has at least 112 characters appended. 274
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 275
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 276
Avoid if (x != y) ..; else ..; 278284
Avoid appending characters as strings in StringBuffer.append. 279
Avoid appending characters as strings in StringBuffer.append. 281
Avoid if (x != y) ..; else ..; 286
Local variable 's' could be declared final 286
Avoid appending characters as strings in StringBuffer.append. 287
Local variable 's' could be declared final 291
Avoid if (x != y) ..; else ..; 291
Avoid appending characters as strings in StringBuffer.append. 292
Local variable 's' could be declared final 296
Avoid if (x != y) ..; else ..; 296
Avoid appending characters as strings in StringBuffer.append. 297
Local variable 's' could be declared final 301
Avoid if (x != y) ..; else ..; 301
Avoid appending characters as strings in StringBuffer.append. 302
Avoid appending characters as strings in StringBuffer.append. 306
Avoid appending characters as strings in StringBuffer.append. 309

Tobago Core - org/apache/myfaces/tobago/context/ThemeResource.java

Violation Line
Avoid if (x != y) ..; else ..; 50
Avoid if (x != y) ..; else ..; 56

Tobago Core - org/apache/myfaces/tobago/context/ThemeResources.java

Violation Line
Parameter 'production' is not assigned and could be declared final 39
Parameter 'exclude' is not assigned and could be declared final 67
Parameter 'exclude' is not assigned and could be declared final 71

Tobago Core - org/apache/myfaces/tobago/context/TobagoBundle.java

Violation Line
Private field 'bundleName' could be made final; it is only initialized in the declaration or constructor. 41

Tobago Core - org/apache/myfaces/tobago/context/TransientStateHolder.java

Violation Line
Document empty constructor 3334

Tobago Core - org/apache/myfaces/tobago/context/UserAgent.java

Violation Line
The class 'UserAgent' has a Cyclomatic Complexity of 3 (Highest = 20). 33390
The class 'UserAgent' has a Standard Cyclomatic Complexity of 3 (Highest = 19). 33390
The class 'UserAgent' has a Modified Cyclomatic Complexity of 3 (Highest = 19). 33390
The String literal "msie" appears 12 times in this file; the first occurrence is on line 43 43
The String literal "opera" appears 4 times in this file; the first occurrence is on line 107 107
The String literal "mozilla" appears 4 times in this file; the first occurrence is on line 131 131
The String literal "gecko" appears 5 times in this file; the first occurrence is on line 154 154
The method 'getInstance' has a Modified Cyclomatic Complexity of 19. 279334
The method 'getInstance' has a Standard Cyclomatic Complexity of 19. 279334
The method 'getInstance' has a Cyclomatic Complexity of 20. 279334
Avoid if (x != y) ..; else ..; 350352
Consider using varargs for methods or constructors which take an array the last parameter. 364
Consider using varargs for methods or constructors which take an array the last parameter. 382

Tobago Core - org/apache/myfaces/tobago/convert/DurationConverter.java

Violation Line
Unnecessary wrapper object creation 65
Avoid declaring a variable if it is unreferenced before a possible exit point. 89
The default label should be the last label in a switch statement 129145

Tobago Core - org/apache/myfaces/tobago/el/ConstantMethodExpression.java

Violation Line
Document empty constructor 3536
Parameter 'outcome' is not assigned and could be declared final 38
Parameter 'context' is not assigned and could be declared final 43
Parameter 'context' is not assigned and could be declared final 49
Parameter 'params' is not assigned and could be declared final 49
Consider using varargs for methods or constructors which take an array the last parameter. 49
Parameter 'o' is not assigned and could be declared final 55
Local variable 'that' could be declared final 63
Avoid if (x != y) ..; else ..; 65
Parameter 'context' is not assigned and could be declared final 85
Parameter 'state' is not assigned and could be declared final 90
Parameter 'context' is not assigned and could be declared final 90

Tobago Core - org/apache/myfaces/tobago/el/UserWrapper.java

Violation Line
Private field 'roles' could be made final; it is only initialized in the declaration or constructor. 35

Tobago Core - org/apache/myfaces/tobago/event/FacesEventWrapper.java

Violation Line
Private field 'wrappedFacesEvent' could be made final; it is only initialized in the declaration or constructor. 31

Tobago Core - org/apache/myfaces/tobago/event/MethodExpressionStateChangeListener.java

Violation Line
Avoid using redundant field initializer for 'isTransient' 33
Document empty constructor 3536

Tobago Core - org/apache/myfaces/tobago/event/MethodExpressionTabChangeListener.java

Violation Line
Avoid using redundant field initializer for 'isTransient' 33
Document empty constructor 3536

Tobago Core - org/apache/myfaces/tobago/event/MethodExpressionTreeExpansionListener.java

Violation Line
Avoid using redundant field initializer for 'isTransient' 34
Document empty constructor 3637
Avoid unused private methods such as 'elContext()'. 48

Tobago Core - org/apache/myfaces/tobago/event/MethodExpressionTreeSelectedListener.java

Violation Line
Avoid using redundant field initializer for 'isTransient' 33
Document empty constructor 3536
Avoid unused private methods such as 'elContext()'. 47

Tobago Core - org/apache/myfaces/tobago/event/PageAction.java

Violation Line
Avoid if (x != y) ..; else ..; 8589

Tobago Core - org/apache/myfaces/tobago/event/PageActionEvent.java

Violation Line
Private field 'action' could be made final; it is only initialized in the declaration or constructor. 31

Tobago Core - org/apache/myfaces/tobago/event/PopupActionListener.java

Violation Line
Document empty constructor 3738

Tobago Core - org/apache/myfaces/tobago/event/PopupFacetActionListener.java

Violation Line
Avoid if (x != y) ..; else ..; 4147

Tobago Core - org/apache/myfaces/tobago/event/ResetInputActionListener.java

Violation Line
Document empty constructor 3435
Consider using varargs for methods or constructors which take an array the last parameter. 37

Tobago Core - org/apache/myfaces/tobago/event/SortActionEvent.java

Violation Line
Private field 'column' could be made final; it is only initialized in the declaration or constructor. 31

Tobago Core - org/apache/myfaces/tobago/event/TreeExpansionEvent.java

Violation Line
Private field 'oldExpanded' could be made final; it is only initialized in the declaration or constructor. 30
Private field 'newExpanded' could be made final; it is only initialized in the declaration or constructor. 31

Tobago Core - org/apache/myfaces/tobago/event/TreeSelectedEvent.java

Violation Line
Private field 'oldSelected' could be made final; it is only initialized in the declaration or constructor. 30
Private field 'newSelected' could be made final; it is only initialized in the declaration or constructor. 31

Tobago Core - org/apache/myfaces/tobago/event/ValueExpressionPopupActionListener.java

Violation Line
Document empty constructor 5657

Tobago Core - org/apache/myfaces/tobago/event/ValueExpressionResetInputActionListener.java

Violation Line
Document empty constructor 4445

Tobago Core - org/apache/myfaces/tobago/event/ValueExpressionSheetStateChangeListener.java

Violation Line
Document empty constructor 3738
Avoid if (x != y) ..; else ..; 4852
No need to check for null before an instanceof 85

Tobago Core - org/apache/myfaces/tobago/event/ValueExpressionTabChangeListener.java

Violation Line
Document empty constructor 3738
Avoid if (x != y) ..; else ..; 4852
No need to check for null before an instanceof 85

Tobago Core - org/apache/myfaces/tobago/facelets/AttributeHandler.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 20361
Possible God class (WMC=78, ATFD=88, TCC=0.027777777777777776) 20361
The class 'AttributeHandler' has a Cyclomatic Complexity of 7 (Highest = 58). 56361
The class 'AttributeHandler' has a Standard Cyclomatic Complexity of 6 (Highest = 51). 56361
The class 'AttributeHandler' has a Modified Cyclomatic Complexity of 6 (Highest = 51). 56361
Avoid really long methods. 74278
The method 'apply' has a Modified Cyclomatic Complexity of 51. 74278
The method 'apply' has a Standard Cyclomatic Complexity of 51. 74278
The method apply() has an NCSS line count of 162 74278
The method 'apply' has a Cyclomatic Complexity of 58. 74278
Avoid if (x != y) ..; else ..; 81276
Avoid if (x != y) ..; else ..; 85103
Avoid if (x != y) ..; else ..; 108126
Avoid if (x != y) ..; else ..; 194201
Consider using varargs for methods or constructors which take an array the last parameter. 302
Avoid if (x != y) ..; else ..; 308314

Tobago Core - org/apache/myfaces/tobago/facelets/ConverterHandler.java

Violation Line
Avoid declaring a variable if it is unreferenced before a possible exit point. 54
New exception is thrown in catch block, original stack trace may be lost 66
Deeply nested if..then statements are hard to read 6870

Tobago Core - org/apache/myfaces/tobago/facelets/FixCharacterEncodingFilter.java

Violation Line
Document empty method 5758

Tobago Core - org/apache/myfaces/tobago/facelets/GridLayoutConstraintHandler.java

Violation Line
Avoid reassigning parameters such as 'parent' 49
Avoid reassigning parameters such as 'parent' 49

Tobago Core - org/apache/myfaces/tobago/facelets/MetaInfResourcesClasspathResourceResolver.java

Violation Line
Avoid reassigning parameters such as 'path' 55
Avoid reassigning parameters such as 'path' 55
This call to String.startsWith can be rewritten using String.charAt(0) 61

Tobago Core - org/apache/myfaces/tobago/facelets/Resource.java

Violation Line
These nested if statements could be combined 7274
Document empty method 114115
Avoid if (x != y) ..; else ..; 136

Tobago Core - org/apache/myfaces/tobago/facelets/ScriptHandler.java

Violation Line
Document empty method 5051

Tobago Core - org/apache/myfaces/tobago/facelets/SegmentLayoutRule.java

Violation Line
These nested if statements could be combined 3953
Local variable 'a' could be declared final 40
Deeply nested if..then statements are hard to read 4143
Deeply nested if..then statements are hard to read 4446
Deeply nested if..then statements are hard to read 4749
Deeply nested if..then statements are hard to read 5052

Tobago Core - org/apache/myfaces/tobago/facelets/SheetStateChangeSourceRule.java

Violation Line
These nested if statements could be combined 4143

Tobago Core - org/apache/myfaces/tobago/facelets/SortActionSourceRule.java

Violation Line
These nested if statements could be combined 4244

Tobago Core - org/apache/myfaces/tobago/facelets/SuggestMethodRule.java

Violation Line
These nested if statements could be combined 4244
These nested if statements could be combined 4751
Deeply nested if..then statements are hard to read 4850
These nested if statements could be combined 4850

Tobago Core - org/apache/myfaces/tobago/facelets/SupportsMarkupRule.java

Violation Line
Local variable 'a' could be declared final 41

Tobago Core - org/apache/myfaces/tobago/facelets/SupportsRenderedPartiallyRule.java

Violation Line
Local variable 'a' could be declared final 41

Tobago Core - org/apache/myfaces/tobago/facelets/TabChangeListenerHandler.java

Violation Line
New exception is thrown in catch block, original stack trace may be lost 77
Deeply nested if..then statements are hard to read 7981
Avoid if (x != y) ..; else ..; 8388

Tobago Core - org/apache/myfaces/tobago/facelets/TabChangeSourceRule.java

Violation Line
These nested if statements could be combined 3840

Tobago Core - org/apache/myfaces/tobago/facelets/TagMethodExpression.java

Violation Line
New exception is thrown in catch block, original stack trace may be lost 64
New exception is thrown in catch block, original stack trace may be lost 66
New exception is thrown in catch block, original stack trace may be lost 68
Consider using varargs for methods or constructors which take an array the last parameter. 73
New exception is thrown in catch block, original stack trace may be lost 77
New exception is thrown in catch block, original stack trace may be lost 79
New exception is thrown in catch block, original stack trace may be lost 81

Tobago Core - org/apache/myfaces/tobago/facelets/extension/SelectBooleanCheckboxExtensionHandler.java

Violation Line
Private field 'itemLabelAttribute' could be made final; it is only initialized in the declaration or constructor. 38
Avoid if (x != y) ..; else ..; 4958

Tobago Core - org/apache/myfaces/tobago/facelets/extension/SeparatorExtensionHandler.java

Violation Line
Private field 'labelAttribute' could be made final; it is only initialized in the declaration or constructor. 44

Tobago Core - org/apache/myfaces/tobago/facelets/extension/TobagoLabelExtensionHandler.java

Violation Line
Avoid unused private fields such as 'LOG'. 52
Private field 'fieldIdAttribute' could be made final; it is only initialized in the declaration or constructor. 54
Avoid declaring a variable if it is unreferenced before a possible exit point. 99
Avoid if (x != y) ..; else ..; 102106

Tobago Core - org/apache/myfaces/tobago/facelets/extension/TobagoMenuExtensionHandler.java

Violation Line
Private field 'fieldIdAttribute' could be made final; it is only initialized in the declaration or constructor. 47
Avoid if (x != y) ..; else ..; 7981

Tobago Core - org/apache/myfaces/tobago/internal/ajax/AjaxInternalUtils.java

Violation Line
This class has too many methods, consider refactoring it. 45266
No need to check for null before an instanceof 83
These nested if statements could be combined 166168
Use equals() to compare object references. 184
These nested if statements could be combined 218220
Parameter 'facesContext' is not assigned and could be declared final 250

Tobago Core - org/apache/myfaces/tobago/internal/ajax/AjaxNavigationListener.java

Violation Line
Parameter 'facesContext' is not assigned and could be declared final 58

Tobago Core - org/apache/myfaces/tobago/internal/ajax/AjaxNavigationState.java

Violation Line
Private field 'viewRoot' could be made final; it is only initialized in the declaration or constructor. 48
Private field 'messages' could be made final; it is only initialized in the declaration or constructor. 50
Avoid if (x != y) ..; else ..; 109113
Avoid if (x != y) ..; else ..; 121128
Use equals() to compare object references. 132
Parameter 'facesContext' is not assigned and could be declared final 180
Local variable 'throwable' could be declared final 184
Parameter 'facesContext' is not assigned and could be declared final 192

Tobago Core - org/apache/myfaces/tobago/internal/ajax/AjaxResponseRenderer.java

Violation Line
Private field 'callback' could be made final; it is only initialized in the declaration or constructor. 62
Avoid reassigning parameters such as 'charset' 119
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 124
Avoid if (x != y) ..; else ..; 139143

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIButton.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2224

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIColumnNode.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2728

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIColumnSelector.java

Violation Line
An empty method in an abstract class should be abstract instead 3234

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUICommand.java

Violation Line
Avoid unused private fields such as 'LOG'. 48
These nested if statements could be combined 6466
Local variable 'child' could be declared final 112
Do not use if statements that are always true or always false 130
Local variable 'bool' could be declared final 142
Parameter 'disabled' is not assigned and could be declared final 149

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIData.java

Violation Line
The String literal "Not a tree model" appears 4 times in this file; the first occurrence is on line 71 71
Avoid if (x != y) ..; else ..; 8084
Avoid if (x != y) ..; else ..; 146150
Avoid if (x != y) ..; else ..; 155
Avoid if (x != y) ..; else ..; 160
String.indexOf(char) is faster than String.indexOf(String). 201

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIFlexLayout.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 3134

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIFlowLayout.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2427

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIForm.java

Violation Line
Avoid if (x != y) ..; else ..; 7687
Avoid if (x != y) ..; else ..; 96107
Parameter 'event' is not assigned and could be declared final 111
These nested if statements could be combined 126128

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIGridLayout.java

Violation Line
Avoid instantiating new objects inside loops 61
Avoid appending characters as strings in StringBuffer.append. 92
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 92
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 93
Avoid appending characters as strings in StringBuffer.append. 94
Avoid appending characters as strings in StringBuffer.append. 104
Avoid appending characters as strings in StringBuffer.append. 114

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIIn.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2224

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUILabel.java

Violation Line
Avoid if (x != y) ..; else ..; 3339
Avoid if (x != y) ..; else ..; 3539

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUILayoutBase.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2729

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIMediator.java

Violation Line
Avoid unused private fields such as 'LOG'. 33

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIMessages.java

Violation Line
Avoid if (x != y) ..; else ..; 4246
Avoid declaring a variable if it is unreferenced before a possible exit point. 64
Avoid instantiating new objects inside loops 74

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIObject.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2627

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIOut.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2728

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIPage.java

Violation Line
Possible God class (WMC=47, ATFD=28, TCC=0.015151515151515152) 20280
The class 'AbstractUIPage' has a Cyclomatic Complexity of 4 (Highest = 11). 46280
Avoid if (x != y) ..; else ..; 8993
Avoid if (x != y) ..; else ..; 130148
The method 'decodeActionComponent' has a Cyclomatic Complexity of 10. 151181
Use equals() to compare object references. 174
Avoid if (x != y) ..; else ..; 191198
Avoid if (x != y) ..; else ..; 207213
The method 'markSubmittedForm' has a Cyclomatic Complexity of 11. 216272
Avoid if (x != y) ..; else ..; 227234
Avoid if (x != y) ..; else ..; 258271

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIPanel.java

Violation Line
Avoid if (x != y) ..; else ..; 4852
Document empty method 6768
An empty method in an abstract class should be abstract instead 6768

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIPopup.java

Violation Line
Avoid if (x != y) ..; else ..; 6772
No need to check for null before an instanceof 88

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIProgress.java

Violation Line
Avoid unused private fields such as 'LOG'. 37
Parameter 'event' is not assigned and could be declared final 51
Local variable 'model' could be declared final 54
Local variable 'm' could be declared final 56
Do not add empty strings 68
Avoid if (x != y) ..; else ..; 7074

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIScript.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2728

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISection.java

Violation Line
Parameter 'context' is not assigned and could be declared final 55
Avoid if (x != y) ..; else ..; 5963
Parameter 'level' is not assigned and could be declared final 73

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISegmentLayout.java

Violation Line
Local variable 'builder' could be declared final 35

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISelectBooleanCheckbox.java

Violation Line
These nested if statements could be combined 6572

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISelectManyCheckbox.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2223

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISelectManyListbox.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2223

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISelectOneBase.java

Violation Line
Avoid if (x != y) ..; else ..; 5865

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISelectOneChoice.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2223

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISelectOneListbox.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2223

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISelectOneRadio.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2223

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISeparator.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2627

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUISheet.java

Violation Line
Possible God class (WMC=125, ATFD=41, TCC=0.013131313131313131) 20585
A high number of imports can indicate a high degree of coupling within an object. 20585
The class 'AbstractUISheet' has a Cyclomatic Complexity of 3 (Highest = 19). 66585
The class 'AbstractUISheet' has a Standard Cyclomatic Complexity of 3 (Highest = 17). 66585
The class 'AbstractUISheet' has a Modified Cyclomatic Complexity of 2 (Highest = 12). 66585
This class has too many methods, consider refactoring it. 68585
Avoid if (x != y) ..; else ..; 8892
Parameter 'event' is not assigned and could be declared final 129
Local variable 'layoutToken' could be declared final 139
Avoid if (x != y) ..; else ..; 265269
Avoid if (x != y) ..; else ..; 275283
Avoid if (x != y) ..; else ..; 296304
Avoid if (x != y) ..; else ..; 303
Avoid if (x != y) ..; else ..; 314
Local variable 'result' could be declared final 336
Local variable 'result' could be declared final 344
Parameter 'facesContext' is not assigned and could be declared final 408
Avoid unused local variables such as 'ignored'. 421
Avoid instantiating new objects inside loops 431
Parameter 'facesContext' is not assigned and could be declared final 437
Avoid reassigning parameters such as 'event' 437
Avoid if (x != y) ..; else ..; 441453
Avoid reassigning parameters such as 'searchId' 478
The method 'performPaging' has a Standard Cyclomatic Complexity of 17. 494555
The method 'performPaging' has a Cyclomatic Complexity of 19. 494555
The method 'performPaging' has a Modified Cyclomatic Complexity of 12. 494555
A switch statement does not contain a break 502545
Avoid if (x != y) ..; else ..; 548552
Document empty method 558559
An empty method in an abstract class should be abstract instead 558559

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUITabGroup.java

Violation Line
Possible God class (WMC=58, ATFD=22, TCC=0.0) 20296
The class 'AbstractUITabGroup' has a Cyclomatic Complexity of 2 (Highest = 11). 50296
The class 'AbstractUITabGroup' has a Modified Cyclomatic Complexity of 2 (Highest = 11). 50296
The class 'AbstractUITabGroup' has a Standard Cyclomatic Complexity of 2 (Highest = 11). 50296
This class has too many methods, consider refactoring it. 51296
An empty method in an abstract class should be abstract instead 5657
Document empty method 5657
The method 'processDecodes' has a Modified Cyclomatic Complexity of 11. 104144
The method 'processDecodes' has a Standard Cyclomatic Complexity of 11. 104144
The method 'processDecodes' has a Cyclomatic Complexity of 11. 104144
Use opposite operator instead of the logic complement operator. 105
Avoid if (x != y) ..; else ..; 105143
Local variable 'facet' could be declared final 121
Use opposite operator instead of the logic complement operator. 148
Avoid if (x != y) ..; else ..; 148162
Use opposite operator instead of the logic complement operator. 167
Avoid if (x != y) ..; else ..; 167182
Use opposite operator instead of the logic complement operator. 203
Avoid if (x != y) ..; else ..; 211215

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUITextarea.java

Violation Line
An empty method in an abstract class should be abstract instead 2527

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIToolBar.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2437

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIToolBarCommand.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2224

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIToolBarSelectOne.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2425

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUITreeCommand.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2223

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUITreeIndent.java

Violation Line
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 2627

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUITreeNode.java

Violation Line
An empty method in an abstract class should be abstract instead 2527

Tobago Core - org/apache/myfaces/tobago/internal/config/ContentSecurityPolicy.java

Violation Line
Avoid using redundant field initializer for 'unmodifiable' 31

Tobago Core - org/apache/myfaces/tobago/internal/config/RenderersConfigImpl.java

Violation Line
Avoid using redundant field initializer for 'merged' 40
Avoid using redundant field initializer for 'unmodifiable' 42
Avoid if (x != y) ..; else ..; 8792

Tobago Core - org/apache/myfaces/tobago/internal/config/TobagoConfigBuilder.java

Violation Line
Private field 'list' could be made final; it is only initialized in the declaration or constructor. 45
Avoid unused local variables such as 'builder'. 72
Avoid instantiating new objects inside loops 103

Tobago Core - org/apache/myfaces/tobago/internal/config/TobagoConfigEntityResolver.java

Violation Line
Position literals first in String comparisons 52
Position literals first in String comparisons 54
Position literals first in String comparisons 56
Position literals first in String comparisons 58
Avoid if (x != y) ..; else ..; 6369

Tobago Core - org/apache/myfaces/tobago/internal/config/TobagoConfigFragment.java

Violation Line
Too many fields 33219
Private field 'before' could be made final; it is only initialized in the declaration or constructor. 38
Private field 'after' could be made final; it is only initialized in the declaration or constructor. 39
Private field 'supportedThemeNames' could be made final; it is only initialized in the declaration or constructor. 41
Private field 'resourceDirs' could be made final; it is only initialized in the declaration or constructor. 43
Private field 'themeDefinitions' could be made final; it is only initialized in the declaration or constructor. 50
Private field 'mimeTypes' could be made final; it is only initialized in the declaration or constructor. 55
Parameter 'sanitizerClass' is not assigned and could be declared final 187
Parameter 'sanitizerProperties' is not assigned and could be declared final 195
Parameter 'autoAccessKeyFromLabel' is not assigned and could be declared final 203
Parameter 'extension' is not assigned and could be declared final 207
Parameter 'type' is not assigned and could be declared final 207
Avoid if (x != y) ..; else ..; 217

Tobago Core - org/apache/myfaces/tobago/internal/config/TobagoConfigImpl.java

Violation Line
Possible God class (WMC=71, ATFD=26, TCC=0.07899159663865546) 20445
The class 'TobagoConfigImpl' has a Standard Cyclomatic Complexity of 2 (Highest = 11). 53445
The class 'TobagoConfigImpl' has a Modified Cyclomatic Complexity of 2 (Highest = 11). 53445
Too many fields 53445
The class 'TobagoConfigImpl' has a Cyclomatic Complexity of 2 (Highest = 11). 53445
Private field 'contentSecurityPolicy' could be made final; it is only initialized in the declaration or constructor. 68
Avoid using redundant field initializer for 'unmodifiable' 75
Local variable 'theme' could be declared final 97
The method 'resolveThemes' has a Standard Cyclomatic Complexity of 11. 124170
The method 'resolveThemes' has a Cyclomatic Complexity of 11. 124170
The method 'resolveThemes' has a Modified Cyclomatic Complexity of 11. 124170
Avoid if (x != y) ..; else ..; 131158
Parameter 'availableTheme' is not assigned and could be declared final 229
The method 'initProjectState' has a Modified Cyclomatic Complexity of 10. 253292
The method 'initProjectState' has a Standard Cyclomatic Complexity of 10. 253292
The method 'initProjectState' has a Cyclomatic Complexity of 10. 253292
Use block level rather than method level synchronization 294313
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 303
Parameter 'sanitizer' is not assigned and could be declared final 374
Parameter 'autoAccessKeyFromLabel' is not assigned and could be declared final 384
StringBuffer constructor is initialized with size 16, but has at least 320 characters appended. 405
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 406
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 406
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 412
Avoid if (x != y) ..; else ..; 413
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 413
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 414
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 415
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 416
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 417
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 418
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 419
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 420
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 421
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 422
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 423
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 424
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 425
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 426
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 427
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 428
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 429
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 430
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 431
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 432
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 433
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 434
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 435
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 439
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 440
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 441

Tobago Core - org/apache/myfaces/tobago/internal/config/TobagoConfigParser.java

Violation Line
Possible God class (WMC=108, ATFD=38, TCC=0.05454545454545454) 20470
This class has too many methods, consider refactoring it. 48470
The class 'TobagoConfigParser' has a Standard Cyclomatic Complexity of 6 (Highest = 33). 48470
The class 'TobagoConfigParser' has a Cyclomatic Complexity of 6 (Highest = 33). 48470
StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 96
Perhaps 'extension' could be replaced by a local variable. 99
Perhaps 'type' could be replaced by a local variable. 100
Document empty constructor 104105
The method 'startElement' has a Cyclomatic Complexity of 16. 156261
The method 'startElement' has a Standard Cyclomatic Complexity of 16. 156261
Avoid really long methods. 156261
A switch statement does not contain a break 165260
Avoid if (x != y) ..; else ..; 177181
The method 'endElement' has a Standard Cyclomatic Complexity of 33. 269428
The method endElement() has an NCSS line count of 119 269428
The method 'endElement' has a Cyclomatic Complexity of 33. 269428
Avoid really long methods. 269428
A switch statement does not contain a break 279308
Avoid if (x != y) ..; else ..; 295299

Tobago Core - org/apache/myfaces/tobago/internal/config/TobagoConfigSorter.java

Violation Line
Possible God class (WMC=66, ATFD=65, TCC=0.20952380952380953) 20341
This class has too many methods, consider refactoring it. 36341
The class 'TobagoConfigSorter' has a Modified Cyclomatic Complexity of 5 (Highest = 19). 36341
The class 'TobagoConfigSorter' has a Standard Cyclomatic Complexity of 5 (Highest = 19). 36341
The class 'TobagoConfigSorter' has a Cyclomatic Complexity of 5 (Highest = 19). 36341
Private field 'list' could be made final; it is only initialized in the declaration or constructor. 40
Prefer StringBuffer over += for concatenating strings 66
The method merge() has an NPath complexity of 49155 73168
The method 'merge' has a Modified Cyclomatic Complexity of 19. 73168
The method 'merge' has a Standard Cyclomatic Complexity of 19. 73168
The method 'merge' has a Cyclomatic Complexity of 19. 73168
Local variable 'theme' could be declared final 139
Avoid instantiating new objects inside loops 179
Avoid instantiation through private constructors from outside of the constructors class. 179
Avoid instantiating new objects inside loops 190
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 191
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 192
The String literal "' < '" appears 4 times in this file; the first occurrence is on line 193 193
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 193
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 194
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 195
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 195
Avoid appending characters as strings in StringBuffer.append. 198
Avoid instantiating new objects inside loops 214
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 215
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 216
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 217
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 218
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 219
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 220
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 221
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 222
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 223
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 223
Avoid appending characters as strings in StringBuffer.append. 226
Avoid instantiating new objects inside loops 259
Avoid instantiation through private constructors from outside of the constructors class. 259
Avoid instantiation through private constructors from outside of the constructors class. 265
Avoid instantiating new objects inside loops 265
Parameter 'tobagoConfig' is not assigned and could be declared final 293
Parameter 'map' is not assigned and could be declared final 293

Tobago Core - org/apache/myfaces/tobago/internal/context/ClientPropertiesKey.java

Violation Line
The class 'ClientPropertiesKey' has a Cyclomatic Complexity of 2 (Highest = 10). 32135
The method 'equals' has a Cyclomatic Complexity of 10. 6898

Tobago Core - org/apache/myfaces/tobago/internal/context/DateTimeI18n.java

Violation Line
A class which only has private constructors should be final 31106
Parameter 'locale' is not assigned and could be declared final 44
Unnecessary use of fully qualified name 'java.util.Calendar.MONTH' due to existing import 'java.util.Calendar' 56
Parameter 'locale' is not assigned and could be declared final 73
Use block level rather than method level synchronization 7381

Tobago Core - org/apache/myfaces/tobago/internal/context/ImageCacheKey.java

Violation Line
Parameter 'o' is not assigned and could be declared final 36
Local variable 'that' could be declared final 44
Avoid if (x != y) ..; else ..; 49

Tobago Core - org/apache/myfaces/tobago/internal/context/MeasureValue.java

Violation Line
Private field 'value' could be made final; it is only initialized in the declaration or constructor. 31
Avoid if (x != y) ..; else ..; 52
Avoid if (x != y) ..; else ..; 61
Do not add empty strings 66

Tobago Core - org/apache/myfaces/tobago/internal/context/ResourceLocator.java

Violation Line
Possible God class (WMC=71, ATFD=41, TCC=0.07575757575757576) 20450
The class 'ResourceLocator' has a Standard Cyclomatic Complexity of 5 (Highest = 12). 61450
The class 'ResourceLocator' has a Cyclomatic Complexity of 6 (Highest = 14). 61450
This class has too many methods, consider refactoring it. 61450
The class 'ResourceLocator' has a Modified Cyclomatic Complexity of 5 (Highest = 12). 61450
Private field 'servletContext' could be made final; it is only initialized in the declaration or constructor. 69
Private field 'resourceManager' could be made final; it is only initialized in the declaration or constructor. 70
The method locateResourcesInWar() has an NPath complexity of 240 85136
The method 'locateResourcesInWar' has a Standard Cyclomatic Complexity of 12. 85136
The method 'locateResourcesInWar' has a Modified Cyclomatic Complexity of 12. 85136
The method 'locateResourcesInWar' has a Cyclomatic Complexity of 14. 85136
Avoid reassigning parameters such as 'path' 86
Avoid instantiating new objects inside loops 150
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 163
Avoid if (x != y) ..; else ..; 236240
The method 'addResourcesFromZip' has a Cyclomatic Complexity of 12. 275330
The method 'addResourcesFromZip' has a Modified Cyclomatic Complexity of 11. 275330
The method addResourcesFromZip() has an NPath complexity of 288 275330
The method 'addResourcesFromZip' has a Standard Cyclomatic Complexity of 11. 275330
Avoid reassigning parameters such as 'fileName' 276
Avoid reassigning parameters such as 'fileName' 276
Avoid reassigning parameters such as 'fileName' 276
String.indexOf(char) is faster than String.indexOf(String). 280
Do not add empty strings 285
Position literals first in String comparisons 289
Position literals first in String comparisons 298
Avoid reassigning parameters such as 'resourcePath' 364
Avoid reassigning parameters such as 'resourcePath' 364
This call to String.startsWith can be rewritten using String.charAt(0) 365
Avoid declaring a variable if it is unreferenced before a possible exit point. 379
Avoid declaring a variable if it is unreferenced before a possible exit point. 386
Avoid if (x != y) ..; else ..; 440446

Tobago Core - org/apache/myfaces/tobago/internal/context/ResourceManagerFactory.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 37

Tobago Core - org/apache/myfaces/tobago/internal/context/ResourceManagerImpl.java

Violation Line
Possible God class (WMC=109, ATFD=52, TCC=0.05128205128205128) 20567
The class 'ResourceManagerImpl' has a Modified Cyclomatic Complexity of 5 (Highest = 21). 40567
The class 'ResourceManagerImpl' has a Cyclomatic Complexity of 6 (Highest = 35). 40567
The class 'ResourceManagerImpl' has a Standard Cyclomatic Complexity of 5 (Highest = 21). 40567
Avoid unused private fields such as 'JSP'. 44
Private field 'production' could be made final; it is only initialized in the declaration or constructor. 60
Private field 'tobagoConfig' could be made final; it is only initialized in the declaration or constructor. 80
Avoid if (x != y) ..; else ..; 123127
Avoid if (x != y) ..; else ..; 152158
Avoid if (x != y) ..; else ..; 217221
These nested if statements could be combined 225227
The method 'getPaths' has a Modified Cyclomatic Complexity of 21. 236332
The method 'getPaths' has a Standard Cyclomatic Complexity of 21. 236332
The method getPaths() has an NPath complexity of 7540 236332
The method 'getPaths' has a Cyclomatic Complexity of 35. 236332
Avoid long parameter lists. 236239
Avoid reassigning parameters such as 'ignoreMissing' 239
Avoid declaring a variable if it is unreferenced before a possible exit point. 241
Avoid declaring a variable if it is unreferenced before a possible exit point. 242
Avoid declaring a variable if it is unreferenced before a possible exit point. 243
The method 'checkPath' has a Cyclomatic Complexity of 11. 334376
The method 'checkPath' has a Standard Cyclomatic Complexity of 10. 334376
The method 'checkPath' has a Modified Cyclomatic Complexity of 10. 334376
Avoid long parameter lists. 334337
Avoid if (x != y) ..; else ..; 355374
The String literal "testing path: " appears 6 times in this file; the first occurrence is on line 360 360
The method 'checkPath' has a Modified Cyclomatic Complexity of 11. 378425
The method 'checkPath' has a Standard Cyclomatic Complexity of 11. 378425
The method 'checkPath' has a Cyclomatic Complexity of 14. 378425
Avoid long parameter lists. 378382
Avoid if (x != y) ..; else ..; 404423
Avoid long parameter lists. 427430
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 433
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 439
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 440
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 441
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 442
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 443
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 449
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 454
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 469
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 474
Prefer StringBuffer over += for concatenating strings 494
Avoid if (x != y) ..; else ..; 541549

Tobago Core - org/apache/myfaces/tobago/internal/context/ResponseWriterDivider.java

Violation Line
Private field 'writers' could be made final; it is only initialized in the declaration or constructor. 41
Private field 'buffers' could be made final; it is only initialized in the declaration or constructor. 42
Private field 'original' could be made final; it is only initialized in the declaration or constructor. 44
StringBuffer constructor is initialized with size 16, but has at least 175 characters appended. 139
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 140
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 141
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 142
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 143
Avoid appending characters as strings in StringBuffer.append. 144

Tobago Core - org/apache/myfaces/tobago/internal/context/StringValue.java

Violation Line
Private field 'value' could be made final; it is only initialized in the declaration or constructor. 29
Avoid if (x != y) ..; else ..; 50
Avoid if (x != y) ..; else ..; 59

Tobago Core - org/apache/myfaces/tobago/internal/context/ThemeConfigCacheKey.java

Violation Line
Avoid if (x != y) ..; else ..; 59
Avoid if (x != y) ..; else ..; 73

Tobago Core - org/apache/myfaces/tobago/internal/layout/Grid.java

Violation Line
The class 'Grid' has a Standard Cyclomatic Complexity of 6 (Highest = 51). 30447
The class 'Grid' has a Cyclomatic Complexity of 6 (Highest = 55). 30447
The class 'Grid' has a Modified Cyclomatic Complexity of 6 (Highest = 51). 30447
Private field 'cells' could be made final; it is only initialized in the declaration or constructor. 38
Private field 'columns' could be made final; it is only initialized in the declaration or constructor. 40
Private field 'rows' could be made final; it is only initialized in the declaration or constructor. 41
Private field 'columnCount' could be made final; it is only initialized in the declaration or constructor. 43
Avoid reassigning parameters such as 'columnSpan' 68
Avoid reassigning parameters such as 'columnSpan' 68
Avoid instantiating new objects inside loops 100
The method gridAsString() has an NCSS line count of 175 203431
The method gridAsString() has an NPath complexity of 11503356 203431
The method 'gridAsString' has a Standard Cyclomatic Complexity of 51. 203431
The method 'gridAsString' has a Modified Cyclomatic Complexity of 51. 203431
Avoid really long methods. 203431
The method 'gridAsString' has a Cyclomatic Complexity of 55. 203431
StringBuffer constructor is initialized with size 16, but has at least 17 characters appended. 205
Avoid if (x != y) ..; else ..; 210214
Avoid appending characters as strings in StringBuffer.append. 211
Avoid appending characters as strings in StringBuffer.append. 213
Avoid appending characters as strings in StringBuffer.append. 219
Avoid appending characters as strings in StringBuffer.append. 222
Avoid appending characters as strings in StringBuffer.append. 225
Avoid appending characters as strings in StringBuffer.append. 227
Avoid appending characters as strings in StringBuffer.append. 229
Avoid if (x != y) ..; else ..; 234238
Avoid appending characters as strings in StringBuffer.append. 235
Avoid appending characters as strings in StringBuffer.append. 237
Avoid if (x != y) ..; else ..; 241245
Avoid appending characters as strings in StringBuffer.append. 242
Avoid appending characters as strings in StringBuffer.append. 244
Avoid appending characters as strings in StringBuffer.append. 246
Avoid appending characters as strings in StringBuffer.append. 257
Avoid appending characters as strings in StringBuffer.append. 260
Avoid appending characters as strings in StringBuffer.append. 263
Avoid appending characters as strings in StringBuffer.append. 265
Avoid appending characters as strings in StringBuffer.append. 267
Avoid appending characters as strings in StringBuffer.append. 282
Avoid appending characters as strings in StringBuffer.append. 284
Avoid appending characters as strings in StringBuffer.append. 287
Avoid appending characters as strings in StringBuffer.append. 292
Avoid appending characters as strings in StringBuffer.append. 294
Avoid appending characters as strings in StringBuffer.append. 296
Avoid appending characters as strings in StringBuffer.append. 301
Avoid appending characters as strings in StringBuffer.append. 303
Avoid appending characters as strings in StringBuffer.append. 307
Avoid appending characters as strings in StringBuffer.append. 309
Avoid appending characters as strings in StringBuffer.append. 318
Avoid appending characters as strings in StringBuffer.append. 320
Avoid appending characters as strings in StringBuffer.append. 326
Avoid appending characters as strings in StringBuffer.append. 329
Avoid appending characters as strings in StringBuffer.append. 332
Avoid appending characters as strings in StringBuffer.append. 334
Avoid appending characters as strings in StringBuffer.append. 336
Avoid appending characters as strings in StringBuffer.append. 340
Avoid if (x != y) ..; else ..; 346350
Avoid appending characters as strings in StringBuffer.append. 347
Avoid appending characters as strings in StringBuffer.append. 349
Avoid appending characters as strings in StringBuffer.append. 355
Avoid appending characters as strings in StringBuffer.append. 357
Avoid appending characters as strings in StringBuffer.append. 361
Avoid appending characters as strings in StringBuffer.append. 364
Avoid appending characters as strings in StringBuffer.append. 367
Avoid appending characters as strings in StringBuffer.append. 372
Avoid appending characters as strings in StringBuffer.append. 374
Avoid appending characters as strings in StringBuffer.append. 378
Avoid if (x != y) ..; else ..; 382386
Avoid appending characters as strings in StringBuffer.append. 383
Avoid appending characters as strings in StringBuffer.append. 385
Avoid appending characters as strings in StringBuffer.append. 387
Avoid if (x != y) ..; else ..; 393397
Avoid appending characters as strings in StringBuffer.append. 394
Avoid appending characters as strings in StringBuffer.append. 396
Avoid appending characters as strings in StringBuffer.append. 402
Avoid appending characters as strings in StringBuffer.append. 405
Avoid appending characters as strings in StringBuffer.append. 408
Avoid appending characters as strings in StringBuffer.append. 410
Avoid appending characters as strings in StringBuffer.append. 412
Avoid if (x != y) ..; else ..; 417421
Avoid appending characters as strings in StringBuffer.append. 418
Avoid appending characters as strings in StringBuffer.append. 420
Avoid if (x != y) ..; else ..; 423427
Avoid appending characters as strings in StringBuffer.append. 424
Avoid appending characters as strings in StringBuffer.append. 426
Avoid appending characters as strings in StringBuffer.append. 428

Tobago Core - org/apache/myfaces/tobago/internal/layout/LayoutUtils.java

Violation Line
Parameter 'component' is not assigned and could be declared final 85
Avoid if (x != y) ..; else ..; 8993
Avoid if (x != y) ..; else ..; 96100

Tobago Core - org/apache/myfaces/tobago/internal/layout/MathUtils.java

Violation Line
Avoid reassigning parameters such as 'bias' 36
Avoid unused local variables such as 'ignored'. 37
Consider using varargs for methods or constructors which take an array the last parameter. 53
Consider using varargs for methods or constructors which take an array the last parameter. 70

Tobago Core - org/apache/myfaces/tobago/internal/layout/OriginCell.java

Violation Line
Private field 'component' could be made final; it is only initialized in the declaration or constructor. 26

Tobago Core - org/apache/myfaces/tobago/internal/layout/SpanCell.java

Violation Line
Private field 'origin' could be made final; it is only initialized in the declaration or constructor. 26
Private field 'horizontalFirst' could be made final; it is only initialized in the declaration or constructor. 27
Private field 'verticalFirst' could be made final; it is only initialized in the declaration or constructor. 28

Tobago Core - org/apache/myfaces/tobago/internal/taglib/component/SegmentLayoutTagDeclaration.java

Violation Line
The String literal "org.apache.myfaces.tobago.layout.ColumnPartition" appears 4 times in this file; the first occurrence is on line 49 49

Tobago Core - org/apache/myfaces/tobago/internal/taglib/component/SheetTagDeclaration.java

Violation Line
The String literal "boolean" appears 4 times in this file; the first occurrence is on line 73 73

Tobago Core - org/apache/myfaces/tobago/internal/taglib/component/StyleTagDeclaration.java

Violation Line
The String literal "org.apache.myfaces.tobago.layout.Measure" appears 18 times in this file; the first occurrence is on line 72 72

Tobago Core - org/apache/myfaces/tobago/internal/taglib/component/SuggestTagDeclaration.java

Violation Line
The String literal "java.lang.Integer" appears 4 times in this file; the first occurrence is on line 81 81

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/DateExtensionTag.java

Violation Line
The String literal "false" appears 5 times in this file; the first occurrence is on line 94 94
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 94 94

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/FileExtensionTag.java

Violation Line
The String literal "boolean" appears 4 times in this file; the first occurrence is on line 100 100

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/InExtensionTag.java

Violation Line
The String literal "false" appears 5 times in this file; the first occurrence is on line 96 96
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 96 96

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/MenuCheckboxExtensionTag.java

Violation Line
The String literal "boolean" appears 4 times in this file; the first occurrence is on line 102 102

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/MenuRadioExtensionTag.java

Violation Line
The String literal "boolean" appears 4 times in this file; the first occurrence is on line 106 106

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/SelectBooleanCheckboxExtensionTag.java

Violation Line
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 46
The String literal "boolean" appears 5 times in this file; the first occurrence is on line 65 65
The String literal "false" appears 4 times in this file; the first occurrence is on line 65 65
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 66
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 73
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 80
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 88
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 122
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 130
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 138
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 145
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 154
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 162
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 166
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 173
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 182
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 189
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 196
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 203

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/SelectManyCheckboxExtensionTag.java

Violation Line
The String literal "false" appears 5 times in this file; the first occurrence is on line 45 45
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 45 45

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/SelectManyListboxExtensionTag.java

Violation Line
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 45 45
The String literal "false" appears 5 times in this file; the first occurrence is on line 45 45

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/SelectManyShuttleExtensionTag.java

Violation Line
The String literal "false" appears 4 times in this file; the first occurrence is on line 45 45
The String literal "boolean" appears 5 times in this file; the first occurrence is on line 45 45

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/SelectOneChoiceExtensionTag.java

Violation Line
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 45 45
The String literal "false" appears 5 times in this file; the first occurrence is on line 45 45

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/SelectOneListboxExtensionTag.java

Violation Line
The String literal "false" appears 4 times in this file; the first occurrence is on line 45 45
The String literal "boolean" appears 5 times in this file; the first occurrence is on line 45 45

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/SelectOneRadioExtensionTag.java

Violation Line
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 47 47
The String literal "false" appears 5 times in this file; the first occurrence is on line 47 47

Tobago Core - org/apache/myfaces/tobago/internal/taglib/extension/TextareaExtensionTag.java

Violation Line
The String literal "false" appears 4 times in this file; the first occurrence is on line 84 84
The String literal "boolean" appears 5 times in this file; the first occurrence is on line 84 84

Tobago Core - org/apache/myfaces/tobago/internal/util/AccessKeyLogger.java

Violation Line
Parameter 'clientId' is not assigned and could be declared final 55
Local variable 'entry' could be declared final 76

Tobago Core - org/apache/myfaces/tobago/internal/util/ArrayUtils.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 2234

Tobago Core - org/apache/myfaces/tobago/internal/util/AuthorizationHelper.java

Violation Line
The class 'AuthorizationHelper' has a Cyclomatic Complexity of 5 (Highest = 13). 47230
The class 'AuthorizationHelper' has a Modified Cyclomatic Complexity of 5 (Highest = 12). 47230
The class 'AuthorizationHelper' has a Standard Cyclomatic Complexity of 5 (Highest = 13). 47230
The method 'isAuthorized' has a Modified Cyclomatic Complexity of 10. 99132
The method 'isAuthorized' has a Standard Cyclomatic Complexity of 10. 99132
The method 'isAuthorized' has a Cyclomatic Complexity of 10. 99132
Parameter 'expression' is not assigned and could be declared final 134
The method 'getSecurityAnnotation' has a Cyclomatic Complexity of 13. 134194
The method 'getSecurityAnnotation' has a Standard Cyclomatic Complexity of 13. 134194
The method 'getSecurityAnnotation' has a Modified Cyclomatic Complexity of 12. 134194
Avoid if (x != y) ..; else ..; 150161
Local variable 'entry' could be declared final 151
A switch statement does not contain a break 166176
Parameter 'name' is not assigned and could be declared final 212
Parameter 'bean' is not assigned and could be declared final 212
Local variable 'method' could be declared final 222

Tobago Core - org/apache/myfaces/tobago/internal/util/Deprecation.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 49

Tobago Core - org/apache/myfaces/tobago/internal/util/FastStringWriter.java

Violation Line
StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 27

Tobago Core - org/apache/myfaces/tobago/internal/util/HtmlWriterUtils.java

Violation Line
The class 'HtmlWriterUtils' has a Cyclomatic Complexity of 9 (Highest = 14). 25124
The class 'HtmlWriterUtils' has a Modified Cyclomatic Complexity of 7 (Highest = 10). 25124
The class 'HtmlWriterUtils' has a Standard Cyclomatic Complexity of 7 (Highest = 10). 25124
The method 'writeEncodedValue' has a Modified Cyclomatic Complexity of 10. 62123
The method 'writeEncodedValue' has a Standard Cyclomatic Complexity of 10. 62123
The method 'writeEncodedValue' has a Cyclomatic Complexity of 14. 62123
Avoid if (x != y) ..; else ..; 9599

Tobago Core - org/apache/myfaces/tobago/internal/util/JavascriptWriterUtils.java

Violation Line
The class 'JavascriptWriterUtils' has a Standard Cyclomatic Complexity of 7 (Highest = 10). 25120
The class 'JavascriptWriterUtils' has a Modified Cyclomatic Complexity of 7 (Highest = 10). 25120
The class 'JavascriptWriterUtils' has a Cyclomatic Complexity of 9 (Highest = 14). 25120
The method 'writeEncodedValue' has a Standard Cyclomatic Complexity of 10. 58119
The method 'writeEncodedValue' has a Cyclomatic Complexity of 14. 58119
The method 'writeEncodedValue' has a Modified Cyclomatic Complexity of 10. 58119
Avoid if (x != y) ..; else ..; 9195

Tobago Core - org/apache/myfaces/tobago/internal/util/JndiUtils.java

Violation Line
Prefer StringBuffer over += for concatenating strings 57

Tobago Core - org/apache/myfaces/tobago/internal/util/JsonWriterUtils.java

Violation Line
The class 'JsonWriterUtils' has a Cyclomatic Complexity of 9 (Highest = 14). 26125
The class 'JsonWriterUtils' has a Modified Cyclomatic Complexity of 7 (Highest = 10). 26125
The class 'JsonWriterUtils' has a Standard Cyclomatic Complexity of 7 (Highest = 10). 26125
The method 'writeEncodedValue' has a Modified Cyclomatic Complexity of 10. 63124
The method 'writeEncodedValue' has a Standard Cyclomatic Complexity of 10. 63124
The method 'writeEncodedValue' has a Cyclomatic Complexity of 14. 63124
Avoid if (x != y) ..; else ..; 96100

Tobago Core - org/apache/myfaces/tobago/internal/util/MimeTypeUtils.java

Violation Line
The class 'MimeTypeUtils' has a Cyclomatic Complexity of 8 (Highest = 21). 27106
The class 'MimeTypeUtils' has a Modified Cyclomatic Complexity of 8 (Highest = 21). 27106
The class 'MimeTypeUtils' has a Standard Cyclomatic Complexity of 8 (Highest = 21). 27106
Avoid using redundant field initializer for 'additionalMimeTypes' 29
The method 'getMimeTypeForFile' has a Modified Cyclomatic Complexity of 21. 3696
The method getMimeTypeForFile() has an NPath complexity of 290 3696
The method 'getMimeTypeForFile' has a Cyclomatic Complexity of 21. 3696
The method 'getMimeTypeForFile' has a Standard Cyclomatic Complexity of 21. 3696
Local variable 'length' could be declared final 37
These nested if statements could be combined 8486
Local variable 'extension' could be declared final 91
Parameter 'servletContext' is not assigned and could be declared final 98
Singleton is not thread safe 99104

Tobago Core - org/apache/myfaces/tobago/internal/util/NumberUtils.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 2234

Tobago Core - org/apache/myfaces/tobago/internal/util/ObjectUtils.java

Violation Line
A class which only has private constructors should be final 2232
Parameter 'o1' is not assigned and could be declared final 29
Parameter 'o2' is not assigned and could be declared final 29
Use equals() to compare object references. 30

Tobago Core - org/apache/myfaces/tobago/internal/util/PartUtils.java

Violation Line
Parameter 'part' is not assigned and could be declared final 45
Local variable 'cd' could be declared final 54
Local variable 'cdl' could be declared final 56
Local variable 'paramParser' could be declared final 58
Local variable 'params' could be declared final 6162
Avoid if (x != y) ..; else ..; 6574
Deeply nested if..then statements are hard to read 6574
The class 'ParameterParser' has a Cyclomatic Complexity of 4 (Highest = 10). 81330
Avoid using redundant field initializer for 'chars' 86
Avoid using redundant field initializer for 'pos' 91
Avoid using redundant field initializer for 'len' 96
Avoid using redundant field initializer for 'i1' 101
Avoid using redundant field initializer for 'i2' 106
Avoid using redundant field initializer for 'lowerCaseNames' 111
The method 'getToken' has a Cyclomatic Complexity of 10. 139161
Parameter 'quoted' is not assigned and could be declared final 139
Consider using varargs for methods or constructors which take an array the last parameter. 171
Parameter 'ch' is not assigned and could be declared final 171
Local variable 'element' could be declared final 173
Consider using varargs for methods or constructors which take an array the last parameter. 190
Consider using varargs for methods or constructors which take an array the last parameter. 214
Parameter 'b' is not assigned and could be declared final 244
Parameter 'separator' is not assigned and could be declared final 256
Parameter 'separator' is not assigned and could be declared final 272
The method 'parse' has a Cyclomatic Complexity of 10. 290329
Parameter 'offset' is not assigned and could be declared final 292
Parameter 'length' is not assigned and could be declared final 293
Parameter 'separator' is not assigned and could be declared final 294
Local variable 'params' could be declared final 299
Avoid instantiating new objects inside loops 307309
Avoid instantiating new objects inside loops 313315

Tobago Core - org/apache/myfaces/tobago/internal/util/ResponseUtils.java

Violation Line
The class 'ResponseUtils' has a Cyclomatic Complexity of 3 (Highest = 10). 33145
Avoid if (x != y) ..; else ..; 7283
The method 'ensureContentSecurityPolicyHeader' has a Cyclomatic Complexity of 10. 97132
Avoid declaring a variable if it is unreferenced before a possible exit point. 101
Avoid appending characters as strings in StringBuffer.append. 120
These nested if statements could be combined 128130

Tobago Core - org/apache/myfaces/tobago/internal/util/ResponseWriterBuffer.java

Violation Line
Consider using varargs for methods or constructors which take an array the last parameter. 62

Tobago Core - org/apache/myfaces/tobago/internal/util/StringUtils.java

Violation Line
Possible God class (WMC=114, ATFD=9, TCC=0.0) 20504
The class 'StringUtils' has a Modified Cyclomatic Complexity of 4 (Highest = 18). 27504
The class 'StringUtils' has a Standard Cyclomatic Complexity of 4 (Highest = 18). 27504
The class 'StringUtils' has a Cyclomatic Complexity of 4 (Highest = 18). 27504
This class has too many methods, consider refactoring it. 27504
Avoid instantiating new objects inside loops 45
Avoid instantiating Integer objects. Call Integer.valueOf() instead. 45
Avoid appending characters as strings in StringBuffer.append. 57
The method 'equalsIgnoreCaseAndWhitespace' has a Modified Cyclomatic Complexity of 11. 121175
The method 'equalsIgnoreCaseAndWhitespace' has a Cyclomatic Complexity of 14. 121175
The method 'equalsIgnoreCaseAndWhitespace' has a Standard Cyclomatic Complexity of 11. 121175
The method equalsIgnoreCaseAndWhitespace() has an NPath complexity of 378 121175
Use equals() to compare object references. 124
The method 'split' has a Cyclomatic Complexity of 18. 229304
The method 'split' has a Modified Cyclomatic Complexity of 18. 229304
The method 'split' has a Standard Cyclomatic Complexity of 18. 229304
Avoid declaring a variable if it is unreferenced before a possible exit point. 230
The method 'replace' has a Cyclomatic Complexity of 10. 365390
The method replace() has an NPath complexity of 1080 365390
Avoid declaring a variable if it is unreferenced before a possible exit point. 366
Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 379
No need to call String.valueOf to append to a string. 421
Parameter 'b' is not assigned and could be declared final 471
Parameter 'a' is not assigned and could be declared final 471
Local variable 'colon' could be declared final 482
Parameter 'prefix' is not assigned and could be declared final 494
Parameter 'string' is not assigned and could be declared final 494

Tobago Core - org/apache/myfaces/tobago/internal/webapp/DebugContentTypeFilter.java

Violation Line
Document empty method 6163

Tobago Core - org/apache/myfaces/tobago/internal/webapp/DebugContentTypeResponse.java

Violation Line
These nested if statements could be combined 5557
These nested if statements could be combined 6567

Tobago Core - org/apache/myfaces/tobago/internal/webapp/DebugResponseWriterWrapper.java

Violation Line
This class has too many methods, consider refactoring it. 37228
Private field 'stack' could be made final; it is only initialized in the declaration or constructor. 39
Parameter 'name' is not assigned and could be declared final 100
Parameter 'string' is not assigned and could be declared final 100

Tobago Core - org/apache/myfaces/tobago/internal/webapp/HtmlResponseWriter.java

Violation Line
This class has too many methods, consider refactoring it. 35169
Private field 'javascriptWriter' could be made final; it is only initialized in the declaration or constructor. 40

Tobago Core - org/apache/myfaces/tobago/internal/webapp/JsonResponseWriter.java

Violation Line
This class has too many methods, consider refactoring it. 35196
Private field 'javascriptWriter' could be made final; it is only initialized in the declaration or constructor. 39
Private field 'encodeInJavascriptBlock' could be made final; it is only initialized in the declaration or constructor. 41
Private field 'encodeOutsideJavascriptBlock' could be made final; it is only initialized in the declaration or constructor. 42
Consider using varargs for methods or constructors which take an array the last parameter. 76
Document empty method 190191
Document empty method 194195

Tobago Core - org/apache/myfaces/tobago/internal/webapp/LoggingMdcFilter.java

Violation Line
Local variable 'httpRequest' could be declared final 52
Local variable 'session' could be declared final 53
Local variable 'remoteAddr' could be declared final 57
Local variable 'remoteUser' could be declared final 61
Document empty method 7577

Tobago Core - org/apache/myfaces/tobago/internal/webapp/TobagoResponseWriterBase.java

Violation Line
Possible God class (WMC=56, ATFD=11, TCC=0.1339031339031339) 20341
This class has too many methods, consider refactoring it. 35340
Avoid using redundant field initializer for 'i' 43
Avoid if (x != y) ..; else ..; 58
Avoid if (x != y) ..; else ..; 82109
Avoid if (x != y) ..; else ..; 84109
Avoid if (x != y) ..; else ..; 85100
Avoid if (x != y) ..; else ..; 8791
Consider using varargs for methods or constructors which take an array the last parameter. 135
An empty method in an abstract class should be abstract instead 171173
An empty method in an abstract class should be abstract instead 176178

Tobago Core - org/apache/myfaces/tobago/internal/webapp/TobagoResponseWriterWrapper.java

Violation Line
This class has too many methods, consider refactoring it. 32148
Private field 'responseWriter' could be made final; it is only initialized in the declaration or constructor. 34
Parameter 'name' is not assigned and could be declared final 46
Parameter 'name' is not assigned and could be declared final 56
Parameter 'name' is not assigned and could be declared final 130
Parameter 'string' is not assigned and could be declared final 130

Tobago Core - org/apache/myfaces/tobago/internal/webapp/XmlResponseWriter.java

Violation Line
Avoid unused private fields such as 'XHTML_DOCTYPE'. 31

Tobago Core - org/apache/myfaces/tobago/layout/ColumnPartition.java

Violation Line
Parameter 'parts' is not assigned and could be declared final 42
Parameter 'string' is not assigned and could be declared final 46
Parameter 'object' is not assigned and could be declared final 51
Avoid if (x != y) ..; else ..; 5660
Consider using varargs for methods or constructors which take an array the last parameter. 63
Local variable 'summand' could be declared final 68
Parameter 'summands' is not assigned and could be declared final 80
Consider using varargs for methods or constructors which take an array the last parameter. 80
Parameter 'summands' is not assigned and could be declared final 84
Local variable 'list' could be declared final 85
Avoid declaring a variable if it is unreferenced before a possible exit point. 85
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 86
Consider using varargs for methods or constructors which take an array the last parameter. 110
Parameter 'parts' is not assigned and could be declared final 110

Tobago Core - org/apache/myfaces/tobago/layout/LayoutTokens.java

Violation Line
Private field 'tokens' could be made final; it is only initialized in the declaration or constructor. 35
Document empty constructor 3738
Consider using varargs for methods or constructors which take an array the last parameter. 69
Avoid appending characters as strings in StringBuffer.append. 147

Tobago Core - org/apache/myfaces/tobago/layout/Measure.java

Violation Line
Parameter 'i' is not assigned and could be declared final 45
Parameter 'unit' is not assigned and could be declared final 45
Parameter 'd' is not assigned and could be declared final 51
Parameter 'unit' is not assigned and could be declared final 51
Parameter 'string' is not assigned and could be declared final 57
Parameter 'unit' is not assigned and could be declared final 57
Avoid if (x != y) ..; else ..; 130
Do not add empty strings 130
Avoid if (x != y) ..; else ..; 130
Parameter 'o' is not assigned and could be declared final 138
Local variable 'measure' could be declared final 146
Avoid if (x != y) ..; else ..; 148
Avoid if (x != y) ..; else ..; 151
Avoid if (x != y) ..; else ..; 160
Avoid if (x != y) ..; else ..; 161

Tobago Core - org/apache/myfaces/tobago/layout/PercentLayoutToken.java

Violation Line
Private field 'percent' could be made final; it is only initialized in the declaration or constructor. 26
Avoid using redundant field initializer for 'percent' 26

Tobago Core - org/apache/myfaces/tobago/layout/PixelMeasure.java

Violation Line
Avoid instantiating new objects inside loops 35
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 39

Tobago Core - org/apache/myfaces/tobago/layout/RelativeLayoutToken.java

Violation Line
Private field 'factor' could be made final; it is only initialized in the declaration or constructor. 29

Tobago Core - org/apache/myfaces/tobago/lifecycle/SecretPhaseListener.java

Violation Line
Document empty method 5152

Tobago Core - org/apache/myfaces/tobago/model/CalendarModel.java

Violation Line
Private field 'calendarArray' could be made final; it is only initialized in the declaration or constructor. 28
Private field 'firstDayOffset' could be made final; it is only initialized in the declaration or constructor. 29
Avoid instantiating new objects inside loops 43
Avoid appending characters as strings in StringBuffer.append. 77
When instantiating a SimpleDateFormat object, specify a Locale 80
Avoid appending characters as strings in StringBuffer.append. 84
Avoid appending characters as strings in StringBuffer.append. 86
Avoid appending characters as strings in StringBuffer.append. 90
Avoid appending characters as strings in StringBuffer.append. 92

Tobago Core - org/apache/myfaces/tobago/model/DateModel.java

Violation Line
Avoid if (x != y) ..; else ..; 7273

Tobago Core - org/apache/myfaces/tobago/model/ExpandedState.java

Violation Line
This class has too many methods, consider refactoring it. 34251
Private field 'expandedSet' could be made final; it is only initialized in the declaration or constructor. 37
Private field 'collapsedSet' could be made final; it is only initialized in the declaration or constructor. 38
Avoid reassigning parameters such as 'level' 187

Tobago Core - org/apache/myfaces/tobago/model/ScrollPosition.java

Violation Line
Avoid if (x != y) ..; else ..; 36
Avoid if (x != y) ..; else ..; 36
String.indexOf(char) is faster than String.indexOf(String). 49
Parameter 'left' is not assigned and could be declared final 63
Parameter 'top' is not assigned and could be declared final 71

Tobago Core - org/apache/myfaces/tobago/model/SelectItem.java

Violation Line
Parameter 'tip' is not assigned and could be declared final 84
Parameter 'style' is not assigned and could be declared final 112
Parameter 'customClass' is not assigned and could be declared final 122

Tobago Core - org/apache/myfaces/tobago/model/SelectedState.java

Violation Line
Private field 'selected' could be made final; it is only initialized in the declaration or constructor. 32

Tobago Core - org/apache/myfaces/tobago/model/SheetState.java

Violation Line
Overridable method 'reset' called during object construction 51
Avoid if (x != y) ..; else ..; 6771
Parameter 'toBeSorted' is not assigned and could be declared final 172
String.indexOf(char) is faster than String.indexOf(String). 183
Do not add empty strings 185

Tobago Core - org/apache/myfaces/tobago/model/SuggestFilter.java

Violation Line
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 40

Tobago Core - org/apache/myfaces/tobago/model/TreeNodeDataModel.java

Violation Line
Possible God class (WMC=57, ATFD=27, TCC=0.10666666666666667) 20305
Private field 'mapping' could be made final; it is only initialized in the declaration or constructor. 43
Private field 'back' could be made final; it is only initialized in the declaration or constructor. 44
Private field 'showRoot' could be made final; it is only initialized in the declaration or constructor. 45
Overridable method 'reset' called during object construction 59
Avoid instantiating new objects inside loops 69
Avoid instantiation through private constructors from outside of the constructors class. 69
Avoid instantiation through private constructors from outside of the constructors class. 89
Avoid instantiating new objects inside loops 89
Avoid reassigning parameters such as 'node' 103
Avoid instantiating new objects inside loops 209
Avoid if (x != y) ..; else ..; 239243
Private field 'node' could be made final; it is only initialized in the declaration or constructor. 286

Tobago Core - org/apache/myfaces/tobago/model/TreePath.java

Violation Line
Avoid reassigning parameters such as 'node' 69
Avoid if (x != y) ..; else ..; 125

Tobago Core - org/apache/myfaces/tobago/renderkit/LabelWithAccessKey.java

Violation Line
The class 'LabelWithAccessKey' has a Cyclomatic Complexity of 3 (Highest = 17). 32220
The class 'LabelWithAccessKey' has a Standard Cyclomatic Complexity of 2 (Highest = 14). 32220
The class 'LabelWithAccessKey' has a Modified Cyclomatic Complexity of 2 (Highest = 14). 32220
Avoid unused private fields such as 'ESCAPED_INDICATOR'. 37
The constructor 'LabelWithAccessKey' has a Standard Cyclomatic Complexity of 14. 61134
The constructor 'LabelWithAccessKey' has a Modified Cyclomatic Complexity of 14. 61134
The constructor 'LabelWithAccessKey' has a Cyclomatic Complexity of 17. 61134
Local variable 'auto' could be declared final 86
These nested if statements could be combined 103111
Deeply nested if..then statements are hard to read 103111
Parameter 'accessKey' is not assigned and could be declared final 209

Tobago Core - org/apache/myfaces/tobago/renderkit/RendererBase.java

Violation Line
Avoid if (x != y) ..; else ..; 139143
Document empty method 147148
Use block level rather than method level synchronization 150155

Tobago Core - org/apache/myfaces/tobago/renderkit/TobagoRenderKit.java

Violation Line
The class 'TobagoRenderKit' has a Cyclomatic Complexity of 3 (Highest = 12). 48192
Private field 'renderers' could be made final; it is only initialized in the declaration or constructor. 54
Local variable 'rkFactory' could be declared final 57
Avoid instantiation through private constructors from outside of the constructors class. 68
The method 'createResponseWriter' has a Cyclomatic Complexity of 12. 82124
The method createResponseWriter() has an NPath complexity of 200 82124
Avoid declaring a variable if it is unreferenced before a possible exit point. 84
The String literal "text/html" appears 4 times in this file; the first occurrence is on line 92 92
Avoid instantiation through private constructors from outside of the constructors class. 128
Parameter 'renderer' is not assigned and could be declared final 142
Parameter 'type' is not assigned and could be declared final 142
Parameter 'type' is not assigned and could be declared final 147

Tobago Core - org/apache/myfaces/tobago/renderkit/css/Classes.java

Violation Line
Avoid unused private fields such as 'CACHE'. 64
Use block level rather than method level synchronization 86113
Avoid if (x != y) ..; else ..; 119
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 120
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 130

Tobago Core - org/apache/myfaces/tobago/renderkit/css/CustomClass.java

Violation Line
A class which only has private constructors should be final 3487
Parameter 'list' is not assigned and could be declared final 42
Parameter 'text' is not assigned and could be declared final 46
Avoid if (x != y) ..; else ..; 5458
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 63

Tobago Core - org/apache/myfaces/tobago/renderkit/css/FontAwesomeIconEncoder.java

Violation Line
Private field 'icons' could be made final; it is only initialized in the declaration or constructor. 40
Local variable 'icon' could be declared final 45
Avoid instantiating new objects inside loops 46
Local variable 'result' could be declared final 63
Private field 'name' could be made final; it is only initialized in the declaration or constructor. 90
Parameter 'name' is not assigned and could be declared final 92

Tobago Core - org/apache/myfaces/tobago/renderkit/css/Style.java

Violation Line
This class has a bunch of public methods and attributes 20679
Possible God class (WMC=113, ATFD=23, TCC=0.2970356926799758) 20679
The class 'Style' has a Standard Cyclomatic Complexity of 2 (Highest = 28). 34679
The class 'Style' has a Cyclomatic Complexity of 2 (Highest = 28). 34679
The class 'Style' has a Modified Cyclomatic Complexity of 2 (Highest = 28). 34679
Too many fields 34679
Document empty constructor 7374
Overridable method 'checkEmptiness' called during object construction 111
The method 'encode' has a Standard Cyclomatic Complexity of 27. 166300
The method encode() has an NPath complexity of 67108864 166300
The method 'encode' has a Cyclomatic Complexity of 27. 166300
Avoid really long methods. 166300
The method encode() has an NCSS line count of 107 166300
The method 'encode' has a Modified Cyclomatic Complexity of 27. 166300
StringBuffer constructor is initialized with size 16, but has at least 26 characters appended. 167
The method 'encodeJson' has a Cyclomatic Complexity of 28. 302441
The method encodeJson() has an NCSS line count of 110 302441
The method 'encodeJson' has a Modified Cyclomatic Complexity of 28. 302441
The method 'encodeJson' has a Standard Cyclomatic Complexity of 28. 302441
Avoid really long methods. 302441
The method encodeJson() has an NPath complexity of 134217728 302441
StringBuffer constructor is initialized with size 17, but has at least 78 characters appended. 303
The String literal "\"," appears 25 times in this file; the first occurrence is on line 307 307
Avoid appending characters as strings in StringBuffer.append. 427
Parameter 'minWidth' is not assigned and could be declared final 465
Parameter 'minHeight' is not assigned and could be declared final 474
Parameter 'maxWidth' is not assigned and could be declared final 483
Parameter 'maxHeight' is not assigned and could be declared final 492
Parameter 'right' is not assigned and could be declared final 510
Parameter 'bottom' is not assigned and could be declared final 528
Parameter 'backgroundImage' is not assigned and could be declared final 645
Parameter 'backgroundPosition' is not assigned and could be declared final 653

Tobago Core - org/apache/myfaces/tobago/renderkit/html/DataAttributes.java

Violation Line
Parameter 'value' is not assigned and could be declared final 210

Tobago Core - org/apache/myfaces/tobago/renderkit/html/HtmlAttributes.java

Violation Line
Parameter 'value' is not assigned and could be declared final 130

Tobago Core - org/apache/myfaces/tobago/renderkit/html/HtmlButtonTypes.java

Violation Line
Parameter 'value' is not assigned and could be declared final 30

Tobago Core - org/apache/myfaces/tobago/renderkit/html/HtmlInputTypes.java

Violation Line
Parameter 'value' is not assigned and could be declared final 37

Tobago Core - org/apache/myfaces/tobago/sanitizer/IgnoringSanitizer.java

Violation Line
Parameter 'html' is not assigned and could be declared final 35
Parameter 'configuration' is not assigned and could be declared final 40

Tobago Core - org/apache/myfaces/tobago/sanitizer/JsoupSanitizer.java

Violation Line
Avoid using redundant field initializer for 'unmodifiable' 39

Tobago Core - org/apache/myfaces/tobago/servlet/NonFacesRequestServlet.java

Violation Line
Avoid declaring a variable if it is unreferenced before a possible exit point. 61
An empty method in an abstract class should be abstract instead 9597

Tobago Core - org/apache/myfaces/tobago/servlet/ResourceServlet.java

Violation Line
The class 'ResourceServlet' has a Cyclomatic Complexity of 4 (Highest = 10). 69201
Private field 'resourceDirs' could be made final; it is only initialized in the declaration or constructor. 77
Avoid instantiating Long objects.Call Long.valueOf() instead 94
This call to String.startsWith can be rewritten using String.charAt(0) 119
The method 'doGet' has a Cyclomatic Complexity of 10. 127183
Avoid if (x != y) ..; else ..; 149159
Avoid if (x != y) ..; else ..; 173179
Avoid if (x != y) ..; else ..; 187191

Tobago Core - org/apache/myfaces/tobago/util/ApplyRequestValuesCallback.java

Violation Line
The class 'ApplyRequestValuesCallback' has a Cyclomatic Complexity of 6 (Highest = 11). 3571
Avoid unused private fields such as 'LOG'. 38
The method 'invokeContextCallback' has a Cyclomatic Complexity of 11. 4165
Deeply nested if..then statements are hard to read 5160

Tobago Core - org/apache/myfaces/tobago/util/BeanComparator.java

Violation Line
Private field 'property' could be made final; it is only initialized in the declaration or constructor. 35
Avoid if (x != y) ..; else ..; 69

Tobago Core - org/apache/myfaces/tobago/util/BundleMapWrapper.java

Violation Line
This class has too many methods, consider refactoring it. 30121
Private field 'basename' could be made final; it is only initialized in the declaration or constructor. 32

Tobago Core - org/apache/myfaces/tobago/util/ComparatorBase.java

Violation Line
The class 'ComparatorBase' has a Modified Cyclomatic Complexity of 3 (Highest = 12). 27126
The class 'ComparatorBase' has a Standard Cyclomatic Complexity of 3 (Highest = 12). 27126
The class 'ComparatorBase' has a Cyclomatic Complexity of 3 (Highest = 14). 27126
Document empty constructor 3334
The method 'internalCompare' has a Modified Cyclomatic Complexity of 12. 4989
The method 'internalCompare' has a Standard Cyclomatic Complexity of 12. 4989
The method internalCompare() has an NPath complexity of 1600 4989
The method 'internalCompare' has a Cyclomatic Complexity of 14. 4989
Avoid if (x != y) ..; else ..; 7787
Avoid if (x != y) ..; else ..; 120

Tobago Core - org/apache/myfaces/tobago/util/ComponentUtils.java

Violation Line
Possible God class (WMC=182, ATFD=111, TCC=8.163265306122449E-4) 20820
A high number of imports can indicate a high degree of coupling within an object. 20820
This class has a bunch of public methods and attributes 20820
The class 'ComponentUtils' has a Standard Cyclomatic Complexity of 3 (Highest = 10). 67820
The class 'ComponentUtils' has a Modified Cyclomatic Complexity of 3 (Highest = 10). 67820
This class has too many methods, consider refactoring it. 67820
The class 'ComponentUtils' has a Cyclomatic Complexity of 3 (Highest = 10). 67820
Avoid reassigning parameters such as 'component' 128
Avoid if (x != y) ..; else ..; 151155
Avoid if (x != y) ..; else ..; 160170
Avoid reassigning parameters such as 'component' 173
Avoid reassigning parameters such as 'component' 207
Avoid reassigning parameters such as 'component' 217
Avoid if (x != y) ..; else ..; 275283
Avoid reassigning parameters such as 'component' 388
Local variable 'bool' could be declared final 453
Avoid if (x != y) ..; else ..; 470
Consider simply returning the value vs storing it in local variable 'myRenderer' 572
The method 'findComponent' has a Cyclomatic Complexity of 10. 611646
These nested if statements could be combined 616618
Position literals first in String comparisons 616
Consider using varargs for methods or constructors which take an array the last parameter. 652
The method 'getConvertedValue' has a Modified Cyclomatic Complexity of 10. 694725
The method 'getConvertedValue' has a Standard Cyclomatic Complexity of 10. 694725
The method 'getConvertedValue' has a Cyclomatic Complexity of 10. 694725
Avoid if (x != y) ..; else ..; 698720
Avoid reassigning parameters such as 'markup' 727
Avoid reassigning parameters such as 'markup' 727
Avoid reassigning parameters such as 'markup' 727
Avoid reassigning parameters such as 'markup' 727
Avoid reassigning parameters such as 'markup' 727
Local variable 'map' could be declared final 817
Avoid if (x != y) ..; else ..; 818

Tobago Core - org/apache/myfaces/tobago/util/CreateComponentUtils.java

Violation Line
Avoid if (x != y) ..; else ..; 6468
Avoid if (x != y) ..; else ..; 8489

Tobago Core - org/apache/myfaces/tobago/util/DebugNavigationHandler.java

Violation Line
Private field 'navigationHandler' could be made final; it is only initialized in the declaration or constructor. 33

Tobago Core - org/apache/myfaces/tobago/util/DebugPhaseListener.java

Violation Line
The class 'DebugPhaseListener' has a Modified Cyclomatic Complexity of 8 (Highest = 13). 39157
The class 'DebugPhaseListener' has a Standard Cyclomatic Complexity of 8 (Highest = 13). 39157
The class 'DebugPhaseListener' has a Cyclomatic Complexity of 10 (Highest = 16). 39157
The method 'afterPhase' has a Modified Cyclomatic Complexity of 10. 4692
The method 'afterPhase' has a Cyclomatic Complexity of 11. 4692
The method 'afterPhase' has a Standard Cyclomatic Complexity of 10. 4692
Deeply nested if..then statements are hard to read 6873
Deeply nested if..then statements are hard to read 7678
The method 'beforePhase' has a Cyclomatic Complexity of 16. 95151
The method beforePhase() has an NPath complexity of 572 95151
The method 'beforePhase' has a Standard Cyclomatic Complexity of 13. 95151
The method 'beforePhase' has a Modified Cyclomatic Complexity of 13. 95151
Avoid declaring a variable if it is unreferenced before a possible exit point. 97
Deeply nested if..then statements are hard to read 113129
Deeply nested if..then statements are hard to read 135145
Deeply nested if..then statements are hard to read 146148

Tobago Core - org/apache/myfaces/tobago/util/DebugUtils.java

Violation Line
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 59
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 82
Avoid appending characters as strings in StringBuffer.append. 82
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 83
Avoid appending characters as strings in StringBuffer.append. 84
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 84
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 92
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 93

Tobago Core - org/apache/myfaces/tobago/util/EncodeAjaxCallback.java

Violation Line
Deeply nested if..then statements are hard to read 4244

Tobago Core - org/apache/myfaces/tobago/util/MessageFormat.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 2776
Parameter 'pattern' is not assigned and could be declared final 29
Parameter 'param0' is not assigned and could be declared final 29
Parameter 'param0' is not assigned and could be declared final 33
Parameter 'pattern' is not assigned and could be declared final 33
Parameter 'param1' is not assigned and could be declared final 33
Parameter 'param0' is not assigned and could be declared final 37
Parameter 'pattern' is not assigned and could be declared final 37
Parameter 'param1' is not assigned and could be declared final 37
Parameter 'param2' is not assigned and could be declared final 37
Parameter 'param3' is not assigned and could be declared final 41
Parameter 'param1' is not assigned and could be declared final 41
Parameter 'param2' is not assigned and could be declared final 41
Parameter 'pattern' is not assigned and could be declared final 41
Parameter 'param0' is not assigned and could be declared final 41
Parameter 'param3' is not assigned and could be declared final 45
Parameter 'param1' is not assigned and could be declared final 45
Parameter 'param0' is not assigned and could be declared final 45
Parameter 'pattern' is not assigned and could be declared final 45
Parameter 'param2' is not assigned and could be declared final 45
Parameter 'param4' is not assigned and could be declared final 46
Parameter 'param2' is not assigned and could be declared final 50
Parameter 'param0' is not assigned and could be declared final 50
Parameter 'param1' is not assigned and could be declared final 50
Parameter 'param3' is not assigned and could be declared final 50
Parameter 'pattern' is not assigned and could be declared final 50
Parameter 'param5' is not assigned and could be declared final 51
Parameter 'param4' is not assigned and could be declared final 51
Parameter 'param2' is not assigned and could be declared final 55
Parameter 'pattern' is not assigned and could be declared final 55
Parameter 'param1' is not assigned and could be declared final 55
Parameter 'param0' is not assigned and could be declared final 55
Parameter 'param3' is not assigned and could be declared final 55
Parameter 'param6' is not assigned and could be declared final 56
Parameter 'param4' is not assigned and could be declared final 56
Parameter 'param5' is not assigned and could be declared final 56
Parameter 'param1' is not assigned and could be declared final 60
Parameter 'pattern' is not assigned and could be declared final 60
Parameter 'param2' is not assigned and could be declared final 60
Parameter 'param3' is not assigned and could be declared final 60
Parameter 'param0' is not assigned and could be declared final 60
Parameter 'param5' is not assigned and could be declared final 61
Parameter 'param7' is not assigned and could be declared final 61
Parameter 'param6' is not assigned and could be declared final 61
Parameter 'param4' is not assigned and could be declared final 61
Avoid long parameter lists. 6667
Parameter 'param3' is not assigned and could be declared final 66
Parameter 'param1' is not assigned and could be declared final 66
Parameter 'param0' is not assigned and could be declared final 66
Parameter 'pattern' is not assigned and could be declared final 66
Parameter 'param2' is not assigned and could be declared final 66
Parameter 'param7' is not assigned and could be declared final 67
Parameter 'param5' is not assigned and could be declared final 67
Parameter 'param8' is not assigned and could be declared final 67
Parameter 'param6' is not assigned and could be declared final 67
Parameter 'param4' is not assigned and could be declared final 67
Parameter 'pattern' is not assigned and could be declared final 72

Tobago Core - org/apache/myfaces/tobago/util/MessageUtils.java

Violation Line
The class 'MessageUtils' has a Standard Cyclomatic Complexity of 3 (Highest = 11). 43160
The class 'MessageUtils' has a Cyclomatic Complexity of 4 (Highest = 15). 43160
The class 'MessageUtils' has a Modified Cyclomatic Complexity of 3 (Highest = 11). 43160
Consider using varargs for methods or constructors which take an array the last parameter. 52
The method 'getMessage' has a Modified Cyclomatic Complexity of 11. 57104
The method 'getMessage' has a Standard Cyclomatic Complexity of 11. 57104
The method 'getMessage' has a Cyclomatic Complexity of 15. 57104
The method getMessage() has an NPath complexity of 468 57104
Deeply nested if..then statements are hard to read 7678
Deeply nested if..then statements are hard to read 7981
Avoid if (x != y) ..; else ..; 116

Tobago Core - org/apache/myfaces/tobago/util/RequestUtils.java

Violation Line
Do not add empty strings 53

Tobago Core - org/apache/myfaces/tobago/util/ValueBindingComparator.java

Violation Line
The class 'ValueBindingComparator' has a Standard Cyclomatic Complexity of 3 (Highest = 10). 34131
The class 'ValueBindingComparator' has a Modified Cyclomatic Complexity of 3 (Highest = 10). 34131
The class 'ValueBindingComparator' has a Cyclomatic Complexity of 3 (Highest = 11). 34131
Private field 'facesContext' could be made final; it is only initialized in the declaration or constructor. 38
Private field 'var' could be made final; it is only initialized in the declaration or constructor. 40
Private field 'valueBinding' could be made final; it is only initialized in the declaration or constructor. 42
The method 'equals' has a Modified Cyclomatic Complexity of 10. 76100
The method 'equals' has a Standard Cyclomatic Complexity of 10. 76100
The method 'equals' has a Cyclomatic Complexity of 11. 76100
Avoid declaring a variable if it is unreferenced before a possible exit point. 84
Avoid if (x != y) ..; else ..; 89
Avoid if (x != y) ..; else ..; 92
Avoid if (x != y) ..; else ..; 95
Avoid if (x != y) ..; else ..; 104
Avoid if (x != y) ..; else ..; 105
Avoid if (x != y) ..; else ..; 106

Tobago Core - org/apache/myfaces/tobago/util/ValueExpressionComparator.java

Violation Line
The class 'ValueExpressionComparator' has a Cyclomatic Complexity of 3 (Highest = 11). 30128
The class 'ValueExpressionComparator' has a Modified Cyclomatic Complexity of 3 (Highest = 10). 30128
The class 'ValueExpressionComparator' has a Standard Cyclomatic Complexity of 3 (Highest = 10). 30128
Private field 'facesContext' could be made final; it is only initialized in the declaration or constructor. 34
Private field 'var' could be made final; it is only initialized in the declaration or constructor. 36
Private field 'valueExpression' could be made final; it is only initialized in the declaration or constructor. 38
The method 'equals' has a Modified Cyclomatic Complexity of 10. 7498
The method 'equals' has a Standard Cyclomatic Complexity of 10. 7498
The method 'equals' has a Cyclomatic Complexity of 11. 7498
Avoid declaring a variable if it is unreferenced before a possible exit point. 82
Avoid if (x != y) ..; else ..; 87
Avoid if (x != y) ..; else ..; 90
Avoid if (x != y) ..; else ..; 93
Avoid if (x != y) ..; else ..; 102
Avoid if (x != y) ..; else ..; 103
Avoid if (x != y) ..; else ..; 104

Tobago Core - org/apache/myfaces/tobago/util/XmlUtils.java

Violation Line
A switch statement does not contain a break 74100

Tobago Core - org/apache/myfaces/tobago/validator/FileItemValidator.java

Violation Line
The class 'FileItemValidator' has a Cyclomatic Complexity of 2 (Highest = 12). 40132
Avoid using redundant field initializer for 'maxSize' 44
Document empty constructor 5152
The method 'validate' has a Cyclomatic Complexity of 12. 5588
Deeply nested if..then statements are hard to read 6972
Deeply nested if..then statements are hard to read 7680
Consider using varargs for methods or constructors which take an array the last parameter. 104

Tobago Core - org/apache/myfaces/tobago/validator/SubmittedValueLengthValidator.java

Violation Line
Document empty constructor 4445
Avoid if (x != y) ..; else ..; 58
Avoid if (x != y) ..; else ..; 70
Avoid if (x != y) ..; else ..; 128
Avoid if (x != y) ..; else ..; 131
Avoid if (x != y) ..; else ..; 140
Avoid if (x != y) ..; else ..; 141

Tobago Core - org/apache/myfaces/tobago/webapp/LogoutActionListener.java

Violation Line
New exception is thrown in catch block, original stack trace may be lost 59

Tobago Core - org/apache/myfaces/tobago/webapp/Secret.java

Violation Line
Private field 'secret' could be made final; it is only initialized in the declaration or constructor. 57
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 57
Local variable 'secret' could be declared final 85
Parameter 'session' is not assigned and could be declared final 89

Tobago Core - org/apache/myfaces/tobago/webapp/SecretSessionListener.java

Violation Line
Document empty method 3839

Tobago Core - org/apache/myfaces/tobago/webapp/TobagoResponseWriter.java

Violation Line
Possible God class (WMC=49, ATFD=21, TCC=0.0) 20304
This class has too many methods, consider refactoring it. 48304
Private field 'iconEncoder' could be made final; it is only initialized in the declaration or constructor. 52
Local variable 'builder' could be declared final 175
Local variable 'other' could be declared final 184
Local variable 'builder' could be declared final 196
Local variable 'other' could be declared final 197

Tobago Core - org/apache/myfaces/tobago/webapp/TobagoServletContextListener.java

Violation Line
StringBuffer constructor is initialized with size 16, but has at least 87 characters appended. 60
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 61
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 61
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 62
StringBuffer (or StringBuilder).append is called 3 consecutive times with literal Strings. Use a single append with a single combined String. 65

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/Address.java

Violation Line
This class has a bunch of public methods and attributes 20327
Possible God class (WMC=58, ATFD=31, TCC=0.07312925170068027) 20327
Too many fields 43327
Avoid unused private fields such as 'revision'. 50
Avoid unused private methods such as 'store()'. 112
Avoid unused private methods such as 'load()'. 126

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/DerbyShutdownServletContextListener.java

Violation Line
Document empty method 3638

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/EmailAddress.java

Violation Line
Avoid if (x != y) ..; else ..; 59
Avoid if (x != y) ..; else ..; 67

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/InMemoryAddressDao.java

Violation Line
Private field 'addresses' could be made final; it is only initialized in the declaration or constructor. 39
Use block level rather than method level synchronization 4659
Use block level rather than method level synchronization 6770
Use block level rather than method level synchronization 7380

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/JpaAddressDao.java

Violation Line
Avoid unused private fields such as 'LOG'. 43
Avoid reassigning parameters such as 'filter' 70
StringBuffer constructor is initialized with size 16, but has at least 24 characters appended. 71
Avoid appending characters as strings in StringBuffer.append. 81
Avoid reassigning parameters such as 'address' 93

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/KillSession.java

Violation Line
Avoid if (x != y) ..; else ..; 4247

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/Picture.java

Violation Line
Document empty constructor 4748

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/web/AdminController.java

Violation Line
Unnecessary wrapper object creation 51
Unnecessary wrapper object creation 52

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/web/Controller.java

Violation Line
This class has a bunch of public methods and attributes 20434
Possible God class (WMC=67, ATFD=43, TCC=0.07474600870827286) 20434
A high number of imports can indicate a high degree of coupling within an object. 20434
This class has too many methods, consider refactoring it. 60434
Too many fields 60434
Avoid instantiating new objects inside loops 120
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 183
Avoid if (x != y) ..; else ..; 224226
Avoid instantiating new objects inside loops 428

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/web/Countries.java

Violation Line
Avoid instantiating new objects inside loops 39

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/web/EmailAddressConverter.java

Violation Line
Avoid unused private fields such as 'LOG'. 35

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/web/MessageUtils.java

Violation Line
A class which only has private constructors should be final 2761

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/web/RandomAddressGenerator.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 3575
Consider using varargs for methods or constructors which take an array the last parameter. 71

Tobago Example Addressbook (Java EE 6) - org/apache/myfaces/tobago/example/addressbook/web/StartupPhaseListener.java

Violation Line
Position literals first in String comparisons 55
New exception is thrown in catch block, original stack trace may be lost 79
New exception is thrown in catch block, original stack trace may be lost 79
Document empty method 8687

Tobago Example Data - org/apache/myfaces/tobago/example/data/CantorInterval.java

Violation Line
Private field 'begin' could be made final; it is only initialized in the declaration or constructor. 33
Private field 'end' could be made final; it is only initialized in the declaration or constructor. 34
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 81
Avoid appending characters as strings in StringBuffer.append. 81
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 82
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 83
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 84
Avoid appending characters as strings in StringBuffer.append. 85
Avoid reassigning parameters such as 'numerator' 109
Avoid reassigning parameters such as 'denominator' 109
Avoid appending characters as strings in StringBuffer.append. 144

Tobago Example Data - org/apache/myfaces/tobago/example/data/CategoryTree.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 2688
Avoid instantiating new objects inside loops 81

Tobago Example Data - org/apache/myfaces/tobago/example/data/CommandNode.java

Violation Line
Private field 'name' could be made final; it is only initialized in the declaration or constructor. 26
Parameter 'name' is not assigned and could be declared final 31
Parameter 'name' is not assigned and could be declared final 35
Parameter 'action' is not assigned and could be declared final 35
Parameter 'url' is not assigned and could be declared final 35
Parameter 'resource' is not assigned and could be declared final 35

Tobago Example Data - org/apache/myfaces/tobago/example/data/CommandNodeFactory.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 2248

Tobago Example Data - org/apache/myfaces/tobago/example/data/LocaleList.java

Violation Line
A class which only has private constructors should be final 3297
Avoid instantiating new objects inside loops 46

Tobago Example Data - org/apache/myfaces/tobago/example/data/MixedCommandTree.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 2246

Tobago Example Data - org/apache/myfaces/tobago/example/data/NamedNode.java

Violation Line
Private field 'name' could be made final; it is only initialized in the declaration or constructor. 32

Tobago Example Data - org/apache/myfaces/tobago/example/data/SmallTree.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 2434

Tobago Example Data - org/apache/myfaces/tobago/example/data/SolarObject.java

Violation Line
Position literals first in String comparisons 78
The String literal "Sun" appears 13 times in this file; the first occurrence is on line 78 78
Position literals first in String comparisons 81
Avoid if (x != y) ..; else ..; 162
Position literals first in String comparisons 174
Avoid instantiating new objects inside loops 192
Position literals first in String comparisons 195
Parameter 'name' is not assigned and could be declared final 216
Local variable 'solarObject' could be declared final 217
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 226
The String literal "III" appears 5 times in this file; the first occurrence is on line 227 227
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 227
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 228
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 230319
The String literal "Jupiter" appears 28 times in this file; the first occurrence is on line 236 236
The String literal "Saturn" appears 19 times in this file; the first occurrence is on line 237 237
The String literal "VII" appears 5 times in this file; the first occurrence is on line 238 238
The String literal "Uranus" appears 22 times in this file; the first occurrence is on line 238 238
The String literal "Herschel" appears 5 times in this file; the first occurrence is on line 238 238
The String literal "Neptune" appears 9 times in this file; the first occurrence is on line 239 239
The String literal "VIII" appears 5 times in this file; the first occurrence is on line 239 239
The String literal "Galileo" appears 4 times in this file; the first occurrence is on line 248 248
The String literal "Sheppard" appears 11 times in this file; the first occurrence is on line 252 252
The String literal "Nicholson" appears 5 times in this file; the first occurrence is on line 255 255
The String literal "Cassini" appears 4 times in this file; the first occurrence is on line 279 279
The String literal "Voyager 2" appears 15 times in this file; the first occurrence is on line 289 289
Local variable 'sun' could be declared final 322325
Local variable 'earth' could be declared final 327331
Local variable 'moon' could be declared final 333337

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/BirdController.java

Violation Line
Private field 'birds' could be made final; it is only initialized in the declaration or constructor. 33

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/Collapsible.java

Violation Line
Document empty constructor 3839
Parameter 'collapsed1' is not assigned and could be declared final 61
Parameter 'collapsed2' is not assigned and could be declared final 69

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/Controller.java

Violation Line
Private field 'vehicleOptionItems' could be made final; it is only initialized in the declaration or constructor. 43
Private field 'carOptionItems' could be made final; it is only initialized in the declaration or constructor. 45
Private field 'motorbikeOptionItems' could be made final; it is only initialized in the declaration or constructor. 47
Parameter 'event' is not assigned and could be declared final 85

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/ConversionController.java

Violation Line
Parameter 'amount' is not assigned and could be declared final 36

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/DateController.java

Violation Line
Avoid unused private fields such as 'LOG'. 34
Parameter 'once' is not assigned and could be declared final 48
Parameter 'onchange' is not assigned and could be declared final 56

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/DefaultCommandController.java

Violation Line
Parameter 'value1' is not assigned and could be declared final 40
Parameter 'value2' is not assigned and could be declared final 48
Parameter 'value3' is not assigned and could be declared final 56

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/FormController.java

Violation Line
Parameter 'outerValue' is not assigned and could be declared final 38
Parameter 'innerValue1' is not assigned and could be declared final 46
Parameter 'innerValue2' is not assigned and could be declared final 54

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/HtmlEditor.java

Violation Line
Private field 'ckeditorAvailable' could be made final; it is only initialized in the declaration or constructor. 34
Private field 'tinymceAvailable' could be made final; it is only initialized in the declaration or constructor. 35

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/InController.java

Violation Line
Parameter 'event' is not assigned and could be declared final 37
Parameter 'changeValue' is not assigned and could be declared final 45

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/InputSuggestController.java

Violation Line
Private field 'suggestData' could be made final; it is only initialized in the declaration or constructor. 51
The String literal "Niedersachsen" appears 6 times in this file; the first occurrence is on line 52 52
Avoid instantiating new objects inside loops 158
Avoid instantiating new objects inside loops 161
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 170
Avoid instantiating new objects inside loops 171
Avoid instantiating new objects inside loops 174

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/Login.java

Violation Line
Parameter 'username' is not assigned and could be declared final 71
Parameter 'password' is not assigned and could be declared final 79

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/ManifestReader.java

Violation Line
Avoid instantiating new objects inside loops 60
Avoid instantiating new objects inside loops 63
Avoid instantiating new objects inside loops 66

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/NavigationNode.java

Violation Line
Private field 'tree' could be made final; it is only initialized in the declaration or constructor. 37
Avoid unused local variables such as 'extension'. 52
Avoid if (x != y) ..; else ..; 55

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/NavigationState.java

Violation Line
Private field 'state' could be made final; it is only initialized in the declaration or constructor. 47
Avoid if (x != y) ..; else ..; 7782
Avoid if (x != y) ..; else ..; 9196
Parameter 'viewSource' is not assigned and could be declared final 140

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/NavigationTree.java

Violation Line
Avoid instantiating new objects inside loops 81
Position literals first in String comparisons 99
Parameter 'viewId' is not assigned and could be declared final 131
Avoid if (x != y) ..; else ..; 147151

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/ObjectController.java

Violation Line
Parameter 'collapsed' is not assigned and could be declared final 50
String.indexOf(char) is faster than String.indexOf(String). 62

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/PanelController.java

Violation Line
Parameter 'text' is not assigned and could be declared final 41

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/PartialReloadController.java

Violation Line
Do not add empty strings 123

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/PlanetExample.java

Violation Line
This abstract class does not have any abstract methods 2476
Private field 'planets' could be made final; it is only initialized in the declaration or constructor. 26
Private field 'earthMoons' could be made final; it is only initialized in the declaration or constructor. 27
Private field 'marsMoons' could be made final; it is only initialized in the declaration or constructor. 28
Private field 'jupiterMoons' could be made final; it is only initialized in the declaration or constructor. 29
Parameter 'planet' is not assigned and could be declared final 60

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/PopupController.java

Violation Line
Parameter 'text' is not assigned and could be declared final 37

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/PrettyUrlForMenuNavigationHandler.java

Violation Line
Private field 'navigationHandler' could be made final; it is only initialized in the declaration or constructor. 35
Parameter 'outcome' is not assigned and could be declared final 42

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/ProgressController.java

Violation Line
Avoid unused private fields such as 'LOG'. 36
Private field 'progress' could be made final; it is only initialized in the declaration or constructor. 38
Local variable 'value' could be declared final 45
When instantiating a SimpleDateFormat object, specify a Locale 59
Local variable 'sdf' could be declared final 59
When instantiating a SimpleDateFormat object, specify a Locale 64
Local variable 'sdf' could be declared final 64
When instantiating a SimpleDateFormat object, specify a Locale 69
Local variable 'sdf' could be declared final 69

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/RoleController.java

Violation Line
Parameter 'text' is not assigned and could be declared final 38

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SelectBooleanCheckboxController.java

Violation Line
Parameter 'a' is not assigned and could be declared final 41
Parameter 'b' is not assigned and could be declared final 49
Parameter 'c' is not assigned and could be declared final 57
Parameter 'd' is not assigned and could be declared final 65
Parameter 'e' is not assigned and could be declared final 73
Parameter 'f' is not assigned and could be declared final 81

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SelectController.java

Violation Line
Private field 'entries' could be made final; it is only initialized in the declaration or constructor. 32

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SelectManyCheckboxController.java

Violation Line
Parameter 'animals' is not assigned and could be declared final 39
Local variable 's' could be declared final 45
Parameter 'numbers' is not assigned and could be declared final 55
Local variable 'number' could be declared final 61

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SelectManyListboxController.java

Violation Line
Private field 'deserts' could be made final; it is only initialized in the declaration or constructor. 35
Parameter 'celestials' is not assigned and could be declared final 52
Local variable 's' could be declared final 58
Parameter 'selectedDeserts' is not assigned and could be declared final 72

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SelectManyShuttleController.java

Violation Line
Private field 'planets' could be made final; it is only initialized in the declaration or constructor. 35
Private field 'stars' could be made final; it is only initialized in the declaration or constructor. 37
Consider using varargs for methods or constructors which take an array the last parameter. 52
Parameter 'selectedPlanets' is not assigned and could be declared final 52
Consider using varargs for methods or constructors which take an array the last parameter. 68
Parameter 'selectedStars' is not assigned and could be declared final 68

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SelectOne.java

Violation Line
Parameter 'value' is not assigned and could be declared final 47
Parameter 'types' is not assigned and could be declared final 55

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SelectOneChoiceController.java

Violation Line
Parameter 'person' is not assigned and could be declared final 36

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SelectOneListboxController.java

Violation Line
Private field 'mountains' could be made final; it is only initialized in the declaration or constructor. 33
Parameter 'river' is not assigned and could be declared final 50
Parameter 'mountain' is not assigned and could be declared final 62

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SelectOneRadioController.java

Violation Line
Parameter 'numberOne' is not assigned and could be declared final 38
Parameter 'numberTwo' is not assigned and could be declared final 46

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/ServerInfo.java

Violation Line
Private field 'version' could be made final; it is only initialized in the declaration or constructor. 54
Private field 'enabled' could be made final; it is only initialized in the declaration or constructor. 59

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SheetController.java

Violation Line
Private field 'solarList' could be made final; it is only initialized in the declaration or constructor. 41
Parameter 'sheetState' is not assigned and could be declared final 57
Avoid if (x != y) ..; else ..; 6470
Avoid if (x != y) ..; else ..; 7478

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SheetFilter.java

Violation Line
The class 'SheetFilter' has a Cyclomatic Complexity of 2 (Highest = 10). 40216
Avoid instantiation through private constructors from outside of the constructors class. 42
Avoid unused private fields such as 'ANY'. 42
Private field 'distanceItems' could be made final; it is only initialized in the declaration or constructor. 47
Private field 'distanceRangeList' could be made final; it is only initialized in the declaration or constructor. 48
Private field 'converter' could be made final; it is only initialized in the declaration or constructor. 50
Private field 'filtered' could be made final; it is only initialized in the declaration or constructor. 57
Avoid instantiation through private constructors from outside of the constructors class. 61
Avoid instantiation through private constructors from outside of the constructors class. 62
Avoid instantiation through private constructors from outside of the constructors class. 63
Avoid instantiation through private constructors from outside of the constructors class. 64
Avoid instantiation through private constructors from outside of the constructors class. 65
Avoid instantiation through private constructors from outside of the constructors class. 66
Avoid instantiation through private constructors from outside of the constructors class. 67
Avoid instantiation through private constructors from outside of the constructors class. 68
The method 'filter' has a Cyclomatic Complexity of 10. 84104
Private field 'min' could be made final; it is only initialized in the declaration or constructor. 160
Private field 'max' could be made final; it is only initialized in the declaration or constructor. 161

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SheetSortingController.java

Violation Line
The class 'SheetSortingController' has a Modified Cyclomatic Complexity of 6 (Highest = 10). 4093
The class 'SheetSortingController' has a Cyclomatic Complexity of 6 (Highest = 10). 4093
The class 'SheetSortingController' has a Standard Cyclomatic Complexity of 6 (Highest = 10). 4093
The method 'sheetSorter' has a Standard Cyclomatic Complexity of 10. 5492
The method 'sheetSorter' has a Modified Cyclomatic Complexity of 10. 5492
The method 'sheetSorter' has a Cyclomatic Complexity of 10. 5492
Local variable 'period1' could be declared final 72
Local variable 'period2' could be declared final 73
Local variable 'discoverYear1' could be declared final 81
Avoid if (x != y) ..; else ..; 81
Avoid if (x != y) ..; else ..; 82
Local variable 'discoverYear2' could be declared final 82

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SheetTreeController.java

Violation Line
Private field 'solarTree' could be made final; it is only initialized in the declaration or constructor. 33

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/SuggestController.java

Violation Line
Parameter 'suggestInput' is not assigned and could be declared final 46

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/TabController.java

Violation Line
Avoid unused private fields such as 'LOG'. 35
Private field 'open' could be made final; it is only initialized in the declaration or constructor. 37
Private field 'close' could be made final; it is only initialized in the declaration or constructor. 38
Parameter 'index' is not assigned and could be declared final 46
Avoid if (x != y) ..; else ..; 5155
Avoid if (x != y) ..; else ..; 5963
Avoid if (x != y) ..; else ..; 6771

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/TestExceptionHandler.java

Violation Line
Private field 'wrapped' could be made final; it is only initialized in the declaration or constructor. 35
Avoid instantiating new objects inside loops 59

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/TestExceptionHandlerFactory.java

Violation Line
Private field 'parent' could be made final; it is only initialized in the declaration or constructor. 27

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/Textarea.java

Violation Line
Parameter 'changeValue' is not assigned and could be declared final 35

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/TextareaController.java

Violation Line
Parameter 'longText' is not assigned and could be declared final 54

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/ThemeController.java

Violation Line
Avoid instantiating new objects inside loops 49
Parameter 'theme' is not assigned and could be declared final 57
Consider using varargs for methods or constructors which take an array the last parameter. 65
Parameter 'themeItems' is not assigned and could be declared final 65

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/TobagoDemoController.java

Violation Line
Possible God class (WMC=109, ATFD=42, TCC=0.010126582278481013) 20654
This class has a bunch of public methods and attributes 20654
A high number of imports can indicate a high degree of coupling within an object. 20654
The String literal "demo" appears 5 times in this file; the first occurrence is on line 56 56
The class 'TobagoDemoController' has a Modified Cyclomatic Complexity of 1 (Highest = 14). 57654
The class 'TobagoDemoController' has a Standard Cyclomatic Complexity of 1 (Highest = 14). 57654
Too many fields 57654
The class 'TobagoDemoController' has a Cyclomatic Complexity of 1 (Highest = 14). 57654
This class has too many methods, consider refactoring it. 57654
Private field 'bool' could be made final; it is only initialized in the declaration or constructor. 66
Private field 'solarTree' could be made final; it is only initialized in the declaration or constructor. 80
Private field 'sheetTreeState' could be made final; it is only initialized in the declaration or constructor. 116
Private field 'toolbarIconItems' could be made final; it is only initialized in the declaration or constructor. 120
Private field 'toolbarTextItems' could be made final; it is only initialized in the declaration or constructor. 124
The String literal ".xhtml" appears 4 times in this file; the first occurrence is on line 196 196
String.indexOf(char) is faster than String.indexOf(String). 202
Avoid instantiating new objects inside loops 255
Document empty method 260262
Consider using varargs for methods or constructors which take an array the last parameter. 277
Consider using varargs for methods or constructors which take an array the last parameter. 285
Consider using varargs for methods or constructors which take an array the last parameter. 305
Consider using varargs for methods or constructors which take an array the last parameter. 313
Avoid if (x != y) ..; else ..; 354360
Consider using varargs for methods or constructors which take an array the last parameter. 432
The method 'sheetSorter' has a Standard Cyclomatic Complexity of 14. 522585
The method 'sheetSorter' has a Cyclomatic Complexity of 14. 522585
The method 'sheetSorter' has a Modified Cyclomatic Complexity of 14. 522585
Avoid instantiating Integer objects. Call Integer.valueOf() instead. 550
Avoid instantiating Integer objects. Call Integer.valueOf() instead. 556
Document empty method 635637
Parameter 'actionEvent' is not assigned and could be declared final 643
Parameter 'selectManyShuttle' is not assigned and could be declared final 651
Consider using varargs for methods or constructors which take an array the last parameter. 651

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/TreeCommandTypesController.java

Violation Line
Private field 'sample' could be made final; it is only initialized in the declaration or constructor. 33
Avoid using redundant field initializer for 'actionOneCount' 34
Avoid using redundant field initializer for 'actionTwoCount' 35
Parameter 'name' is not assigned and could be declared final 49

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/TreeController.java

Violation Line
Private field 'sample' could be made final; it is only initialized in the declaration or constructor. 33
Avoid using redundant field initializer for 'treeShowRoot' 34
Avoid using redundant field initializer for 'treeShowRootJunction' 35
Parameter 'treeShowRoot' is not assigned and could be declared final 51
Parameter 'treeShowRootJunction' is not assigned and could be declared final 59
Parameter 'treeIndentRendered' is not assigned and could be declared final 67
Parameter 'treeIndentShowJunction' is not assigned and could be declared final 75

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/TreeEditorController.java

Violation Line
This class has too many methods, consider refactoring it. 37208
Avoid unused private fields such as 'LOG'. 39
Parameter 'name' is not assigned and could be declared final 59
Avoid if (x != y) ..; else ..; 7479
Parameter 'node' is not assigned and could be declared final 104
Local variable 'nodeName' could be declared final 105
Local variable 'resultNode' could be declared final 106
Local variable 'children' could be declared final 108
Avoid if (x != y) ..; else ..; 119131
Parameter 'selected' is not assigned and could be declared final 135
Parameter 'base' is not assigned and could be declared final 135
Local variable 'children' could be declared final 139
Parameter 'node' is not assigned and could be declared final 151
Local variable 'children' could be declared final 154
Local variable 'previousSibling' could be declared final 164
Avoid if (x != y) ..; else ..; 165171
Local variable 'parent' could be declared final 166
Local variable 'nextSibling' could be declared final 179
Avoid if (x != y) ..; else ..; 180186
Local variable 'parent' could be declared final 181

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/TreeSelectController.java

Violation Line
Private field 'sample' could be made final; it is only initialized in the declaration or constructor. 34
Parameter 'selectable' is not assigned and could be declared final 49
Local variable 'stringBuilder' could be declared final 54
Parameter 'stringBuilder' is not assigned and could be declared final 63
Parameter 'node' is not assigned and could be declared final 63
Local variable 'userObject' could be declared final 64
Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 66
Local variable 'child' could be declared final 69

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/UniversalLoggingInfo.java

Violation Line
Private field 'jul' could be made final; it is only initialized in the declaration or constructor. 46
Private field 'slf4j' could be made final; it is only initialized in the declaration or constructor. 51
Private field 'log4j' could be made final; it is only initialized in the declaration or constructor. 53
Private field 'log4j2' could be made final; it is only initialized in the declaration or constructor. 55
Private field 'commonsLogging' could be made final; it is only initialized in the declaration or constructor. 57
The String literal "getLogger" appears 4 times in this file; the first occurrence is on line 62 62
The String literal "info" appears 5 times in this file; the first occurrence is on line 67 67
The String literal "trace" appears 4 times in this file; the first occurrence is on line 71 71
The String literal "debug" appears 4 times in this file; the first occurrence is on line 72 72
The String literal "warn" appears 4 times in this file; the first occurrence is on line 74 74
The String literal "error" appears 4 times in this file; the first occurrence is on line 75 75
Overridable method 'update' called during object construction 97
Parameter 'testCategory' is not assigned and could be declared final 133
Avoid if (x != y) ..; else ..; 134138
Private field 'id' could be made final; it is only initialized in the declaration or constructor. 143
Private field 'calls' could be made final; it is only initialized in the declaration or constructor. 145
Private field 'factoryClassName' could be made final; it is only initialized in the declaration or constructor. 147
Private field 'factoryMethod' could be made final; it is only initialized in the declaration or constructor. 148
Private field 'usesString' could be made final; it is only initialized in the declaration or constructor. 149
Parameter 'factoryMethod' is not assigned and could be declared final 151
Parameter 'id' is not assigned and could be declared final 151
Parameter 'factoryClassName' is not assigned and could be declared final 151
Parameter 'usesString' is not assigned and could be declared final 152
Parameter 'category' is not assigned and could be declared final 152
Parameter 'calls' is not assigned and could be declared final 152
Local variable 'call' could be declared final 162
Parameter 'logger' is not assigned and could be declared final 171
Parameter 'name' is not assigned and could be declared final 171
Parameter 'id' is not assigned and could be declared final 171
A method/constructor shouldnt explicitly throw java.lang.Exception 171
Parameter 'category' is not assigned and could be declared final 185
Avoid if (x != y) ..; else ..; 196213
Local variable 'call' could be declared final 198
Parameter 'category' is not assigned and could be declared final 216
Parameter 'level' is not assigned and could be declared final 216
Local variable 'methodName' could be declared final 235
Consider simply returning the value vs storing it in local variable 'hasLevel' 237
Parameter 'levelMethod' is not assigned and could be declared final 243
Parameter 'enabledMethod' is not assigned and could be declared final 243
Parameter 'level' is not assigned and could be declared final 243
Parameter 'c' is not assigned and could be declared final 243
Parameter 'clazz' is not assigned and could be declared final 243
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 248
Consider simply returning the value vs storing it in local variable 'hasLevel' 250

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/UploadController.java

Violation Line
Private field 'uploadItems' could be made final; it is only initialized in the declaration or constructor. 45
Parameter 'fileBasic' is not assigned and could be declared final 51
Parameter 'fileContentType' is not assigned and could be declared final 59
Parameter 'fileAjax' is not assigned and could be declared final 67
Parameter 'event' is not assigned and could be declared final 81
Parameter 'part' is not assigned and could be declared final 85

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/UploadItem.java

Violation Line
Avoid unused private fields such as 'LOG'. 29

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/ValidationJsr303Controller.java

Violation Line
Parameter 'required' is not assigned and could be declared final 42
Parameter 'length' is not assigned and could be declared final 50

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/actionlistener/SimpleTabChangeListener.java

Violation Line
Avoid using redundant field initializer for 'count' 27
Parameter 'tabChangeEvent' is not assigned and could be declared final 33

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/bestpractice/ToolBarCustomizer.java

Violation Line
Overridable method 'resetList' called during object construction 39
Avoid instantiation through private constructors from outside of the constructors class. 47
Avoid instantiation through private constructors from outside of the constructors class. 48
Avoid instantiation through private constructors from outside of the constructors class. 49
Private field 'label' could be made final; it is only initialized in the declaration or constructor. 79
Private field 'name' could be made final; it is only initialized in the declaration or constructor. 80

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/clientConfig/ClientConfigController.java

Violation Line
Avoid instantiating new objects inside loops 58
Overridable method 'loadFromClientProperties' called during object construction 67
Avoid if (x != y) ..; else ..; 127131
Consider using varargs for methods or constructors which take an array the last parameter. 162
Avoid if (x != y) ..; else ..; 171175

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/clientConfig/ClientConfigPhaseListener.java

Violation Line
Document empty method 3334

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/formatter/AbstractConverter.java

Violation Line
Private field 'matcher' could be made final; it is only initialized in the declaration or constructor. 38
Private field 'compiler' could be made final; it is only initialized in the declaration or constructor. 39
Private field 'util' could be made final; it is only initialized in the declaration or constructor. 40

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/formatter/JspTagConverter.java

Violation Line
Private field 'tagConverter' could be made final; it is only initialized in the declaration or constructor. 35
Private field 'tags' could be made final; it is only initialized in the declaration or constructor. 36
The String literal "</span>" appears 5 times in this file; the first occurrence is on line 65 65
Prefer StringBuffer over += for concatenating strings 65
Prefer StringBuffer over += for concatenating strings 67
Prefer StringBuffer over += for concatenating strings 72
Prefer StringBuffer over += for concatenating strings 75
Prefer StringBuffer over += for concatenating strings 80

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/formatter/SourceFormatter.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 2869
A method/constructor shouldnt explicitly throw java.lang.Exception 30
Consider using varargs for methods or constructors which take an array the last parameter. 30
Avoid appending characters as strings in StringBuffer.append. 63

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/formatter/StringExpression.java

Violation Line
Private field 'string' could be made final; it is only initialized in the declaration or constructor. 36
Use ArrayList instead of Vector 57
Use ArrayList instead of Vector 58
Avoid using implementation types like 'Vector'; use the interface instead 82
Avoid using implementation types like 'Vector'; use the interface instead 82
String.indexOf(char) is faster than String.indexOf(String). 85
Avoid if (x != y) ..; else ..; 92104

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/info/ActivityList.java

Violation Line
Private field 'data' could be made final; it is only initialized in the declaration or constructor. 42
Avoid if (x != y) ..; else ..; 6367
Avoid if (x != y) ..; else ..; 7276

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/info/ActivityPhaseListener.java

Violation Line
Document empty constructor 3839
Document empty method 5455

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/nonfacesrequest/FishPond.java

Violation Line
Private field 'fishes' could be made final; it is only initialized in the declaration or constructor. 38
Avoid using redundant field initializer for 'selectedFishId' 40
Parameter 'selectedFishId' is not assigned and could be declared final 79

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/overview/OverviewController.java

Violation Line
This class has a bunch of public methods and attributes 20382
Possible God class (WMC=63, ATFD=45, TCC=0.018501387604070305) 20382
Too many fields 49382
Private field 'currencyItems' could be made final; it is only initialized in the declaration or constructor. 69
Avoid instantiating new objects inside loops 124
Avoid instantiating new objects inside loops 132
Do not add empty strings 144
Avoid instantiating new objects inside loops 149
Do not add empty strings 160
Avoid instantiating new objects inside loops 165
Avoid if (x != y) ..; else ..; 177181
The String literal "overview" appears 4 times in this file; the first occurrence is on line 209 209
Consider using varargs for methods or constructors which take an array the last parameter. 248
Consider using varargs for methods or constructors which take an array the last parameter. 340
Consider using varargs for methods or constructors which take an array the last parameter. 356

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/overview/SheetConfig.java

Violation Line
Private field 'sheetDirectLinkCountItems' could be made final; it is only initialized in the declaration or constructor. 37
Private field 'sheetPagingPositionItems' could be made final; it is only initialized in the declaration or constructor. 38
Consider using varargs for methods or constructors which take an array the last parameter. 67
Avoid instantiating new objects inside loops 71
Consider using varargs for methods or constructors which take an array the last parameter. 166
Parameter 'showDirectLinksArrows' is not assigned and could be declared final 182
Parameter 'showPageRangeArrows' is not assigned and could be declared final 190

Tobago Example Demo - org/apache/myfaces/tobago/example/demo/sudoku/Sudoku.java

Violation Line
This class has too many methods, consider refactoring it. 36263
Avoid instantiation through private constructors from outside of the constructors class. 62
Avoid instantiation through private constructors from outside of the constructors class. 82
Avoid instantiating new objects inside loops 132
Avoid instantiating new objects inside loops 149
Avoid instantiating new objects inside loops 166
Consider using varargs for methods or constructors which take an array the last parameter. 182
Avoid instantiation through private constructors from outside of the constructors class. 183
Do not add empty strings 183
Do not add empty strings 187
Do not add empty strings 189
Avoid if (x != y) ..; else ..; 201205
Private field 'n' could be made final; it is only initialized in the declaration or constructor. 215
Private field 'list' could be made final; it is only initialized in the declaration or constructor. 216

Tobago Example Test - org/apache/myfaces/tobago/example/test/Bootstrap.java

Violation Line
Overridable method 'reset' called during object construction 27

Tobago Example Test - org/apache/myfaces/tobago/example/test/ClientConfigController.java

Violation Line
Avoid instantiating new objects inside loops 55
Overridable method 'loadFromClientProperties' called during object construction 64
Avoid if (x != y) ..; else ..; 109113
Consider using varargs for methods or constructors which take an array the last parameter. 138
Avoid if (x != y) ..; else ..; 147151

Tobago Example Test - org/apache/myfaces/tobago/example/test/DateController.java

Violation Line
When instantiating a SimpleDateFormat object, specify a Locale 38
Parameter 'once' is not assigned and could be declared final 48

Tobago Example Test - org/apache/myfaces/tobago/example/test/DirectoryBrowser.java

Violation Line
The class 'DirectoryBrowser' has a Modified Cyclomatic Complexity of 3 (Highest = 12). 33108
The class 'DirectoryBrowser' has a Standard Cyclomatic Complexity of 3 (Highest = 12). 33108
The class 'DirectoryBrowser' has a Cyclomatic Complexity of 4 (Highest = 14). 33108
Private field 'tree' could be made final; it is only initialized in the declaration or constructor. 37
The method 'locateResourcesInWar' has a Modified Cyclomatic Complexity of 12. 4995
The method 'locateResourcesInWar' has a Standard Cyclomatic Complexity of 12. 4995
The method 'locateResourcesInWar' has a Cyclomatic Complexity of 14. 4995
The method locateResourcesInWar() has an NPath complexity of 252 4995
These nested if statements could be combined 7176
Deeply nested if..then statements are hard to read 7274
Avoid instantiating new objects inside loops 75
Avoid instantiating new objects inside loops 81

Tobago Example Test - org/apache/myfaces/tobago/example/test/DynamicBean.java

Violation Line
Private field 'panels' could be made final; it is only initialized in the declaration or constructor. 30
A switch statement does not contain a break 3447

Tobago Example Test - org/apache/myfaces/tobago/example/test/DynamicPanel.java

Violation Line
This abstract class does not have any abstract methods 2240

Tobago Example Test - org/apache/myfaces/tobago/example/test/Hidden.java

Violation Line
Avoid unused private fields such as 'LOG'. 31

Tobago Example Test - org/apache/myfaces/tobago/example/test/Immediate.java

Violation Line
Private field 'selectItems' could be made final; it is only initialized in the declaration or constructor. 35
Consider using varargs for methods or constructors which take an array the last parameter. 73

Tobago Example Test - org/apache/myfaces/tobago/example/test/MeasureBean.java

Violation Line
Private field 'value' could be made final; it is only initialized in the declaration or constructor. 84

Tobago Example Test - org/apache/myfaces/tobago/example/test/PageItem.java

Violation Line
Private field 'name' could be made final; it is only initialized in the declaration or constructor. 31
Private field 'resource' could be made final; it is only initialized in the declaration or constructor. 32
Private field 'jsfResource' could be made final; it is only initialized in the declaration or constructor. 33
Private field 'folder' could be made final; it is only initialized in the declaration or constructor. 35
Private field 'disabled' could be made final; it is only initialized in the declaration or constructor. 36
Private field 'todo' could be made final; it is only initialized in the declaration or constructor. 37
String.indexOf(char) is faster than String.indexOf(String). 49

Tobago Example Test - org/apache/myfaces/tobago/example/test/Partially.java

Violation Line
StringBuffer constructor is initialized with size 16, but has at least 32 characters appended. 31
Avoid appending characters as strings in StringBuffer.append. 42

Tobago Example Test - org/apache/myfaces/tobago/example/test/PopupReferenceController.java

Violation Line
Private field 'sheet' could be made final; it is only initialized in the declaration or constructor. 38
Avoid instantiating new objects inside loops 44

Tobago Example Test - org/apache/myfaces/tobago/example/test/Progress.java

Violation Line
Private field 'range' could be made final; it is only initialized in the declaration or constructor. 28
Private field 'emptyRange' could be made final; it is only initialized in the declaration or constructor. 29

Tobago Example Test - org/apache/myfaces/tobago/example/test/ScriptEvent.java

Violation Line
Private field 'items' could be made final; it is only initialized in the declaration or constructor. 28

Tobago Example Test - org/apache/myfaces/tobago/example/test/ScriptEventItem.java

Violation Line
Private field 'name' could be made final; it is only initialized in the declaration or constructor. 31
Private field 'time' could be made final; it is only initialized in the declaration or constructor. 32

Tobago Example Test - org/apache/myfaces/tobago/example/test/ScriptEventServlet.java

Violation Line
Position literals first in String comparisons 44
Position literals first in String comparisons 46
Position literals first in String comparisons 48

Tobago Example Test - org/apache/myfaces/tobago/example/test/SelectItemModel.java

Violation Line
Avoid instantiating new objects inside loops 49

Tobago Example Test - org/apache/myfaces/tobago/example/test/SelectManyListboxBean.java

Violation Line
Private field 'currencyItems' could be made final; it is only initialized in the declaration or constructor. 42
Consider using varargs for methods or constructors which take an array the last parameter. 65
Consider using varargs for methods or constructors which take an array the last parameter. 97

Tobago Example Test - org/apache/myfaces/tobago/example/test/SheetController.java

Violation Line
Private field 'solarArray' could be made final; it is only initialized in the declaration or constructor. 33
Private field 'solarArray3' could be made final; it is only initialized in the declaration or constructor. 34
Private field 'undefined' could be made final; it is only initialized in the declaration or constructor. 35
System.arraycopy is more efficient 4345
Private field 'localeList' could be made final; it is only initialized in the declaration or constructor. 50
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 50

Tobago Example Test - org/apache/myfaces/tobago/example/test/SheetSortController.java

Violation Line
Avoid instantiation through private constructors from outside of the constructors class. 47
Avoid instantiating new objects inside loops 47
Avoid reassigning parameters such as 'i' 64
Avoid reassigning parameters such as 'i' 64
Avoid reassigning parameters such as 'i' 64
Avoid reassigning parameters such as 'i' 64
Avoid reassigning parameters such as 'i' 64
Do not add empty strings 65
Do not add empty strings 67
Do not add empty strings 69
Do not add empty strings 71
Do not add empty strings 82

Tobago Example Test - org/apache/myfaces/tobago/example/test/TestExceptionHandler.java

Violation Line
Private field 'wrapped' could be made final; it is only initialized in the declaration or constructor. 36
Avoid instantiating new objects inside loops 59

Tobago Example Test - org/apache/myfaces/tobago/example/test/TestExceptionHandlerFactory.java

Violation Line
Private field 'parent' could be made final; it is only initialized in the declaration or constructor. 27

Tobago Example Test - org/apache/myfaces/tobago/example/test/TestPageFilter.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 27132
This call to String.startsWith can be rewritten using String.charAt(0) 85

Tobago Example Test - org/apache/myfaces/tobago/example/test/TreeController.java

Violation Line
Private field 'tree' could be made final; it is only initialized in the declaration or constructor. 36
Private field 'small' could be made final; it is only initialized in the declaration or constructor. 38
Private field 'state' could be made final; it is only initialized in the declaration or constructor. 40
Private field 'treeInSheet' could be made final; it is only initialized in the declaration or constructor. 42
Private field 'infinite' could be made final; it is only initialized in the declaration or constructor. 44

Tobago Example Test - org/apache/myfaces/tobago/example/test/UndefinedRowCountDataModel.java

Violation Line
Consider using varargs for methods or constructors which take an array the last parameter. 28

Tobago Example Test - org/apache/myfaces/tobago/example/test/WaitFilter.java

Violation Line
Document empty method 4142
Document empty method 6263

Tobago Sandbox - org/apache/myfaces/tobago/internal/component/AbstractUISplitLayout.java

Violation Line
Avoid unused local variables such as 'firstComponent'. 39
Avoid unused local variables such as 'secondComponent'. 40
Avoid unused local variables such as 'oldPosition'. 41
Avoid unused local variables such as 'currentSize1'. 43
Avoid unused local variables such as 'currentSize2'. 44
Avoid reassigning parameters such as 'a' 66
Avoid reassigning parameters such as 'b' 66
Avoid reassigning parameters such as 'b' 66
Avoid reassigning parameters such as 'a' 66
An empty method in an abstract class should be abstract instead 100101
Document empty method 100101
Document empty method 113114
An empty method in an abstract class should be abstract instead 113114
Avoid if (x != y) ..; else ..; 122

Tobago Sandbox - org/apache/myfaces/tobago/renderkit/html/sandbox/standard/tag/NumberSliderRenderer.java

Violation Line
Avoid unused private fields such as 'SLIDER_WIDTH_PERCENT'. 44
Avoid really long methods. 47156

Tobago Sandbox - org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/SplitLayoutRenderer.java

Violation Line
Avoid unused method parameters such as 'layout'. 113
Avoid unused method parameters such as 'layout'. 142
Avoid unused private methods such as 'calculateHandleStyle(AbstractUISplitLayout)'. 142

Tobago Theme Example - org/apache/myfaces/tobago/renderkit/html/example/standard/tag/InRenderer.java

Violation Line
Position literals first in String comparisons 50
Position literals first in String comparisons 56

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/css/BootstrapClassGenerator.java

Violation Line
Private field 'extraSmall' could be made final; it is only initialized in the declaration or constructor. 53
Private field 'small' could be made final; it is only initialized in the declaration or constructor. 54
Private field 'medium' could be made final; it is only initialized in the declaration or constructor. 55
Private field 'large' could be made final; it is only initialized in the declaration or constructor. 56
Avoid using redundant field initializer for 'index' 58
Local variable 'result' could be declared final 78
Consider using varargs for methods or constructors which take an array the last parameter. 87

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/css/TobagoClass.java

Violation Line
Parameter 'alignItems' is not assigned and could be declared final 66

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/AjaxClientBehaviorRenderer.java

Violation Line
Parameter 'behaviorContext' is not assigned and could be declared final 41
Parameter 'behavior' is not assigned and could be declared final 41
Parameter 'behavior' is not assigned and could be declared final 87
Parameter 'context' is not assigned and could be declared final 87
Parameter 'component' is not assigned and could be declared final 87
Local variable 'ajaxBehavior' could be declared final 88
Parameter 'component' is not assigned and could be declared final 96
Parameter 'ajaxBehavior' is not assigned and could be declared final 96
Local variable 'event' could be declared final 98
Local variable 'isImmediate' could be declared final 99
Parameter 'component' is not assigned and could be declared final 104
Parameter 'ajaxBehavior' is not assigned and could be declared final 104

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/Command.java

Violation Line
The class 'Command' has a Cyclomatic Complexity of 1 (Highest = 10). 39221
Document empty constructor 5556
Avoid long parameter lists. 5860
Consider using varargs for methods or constructors which take an array the last parameter. 73
Parameter 'ids' is not assigned and could be declared final 73
The constructor 'Command' has a Cyclomatic Complexity of 10. 101133
Avoid reassigning parameters such as 'facetComponent' 101
Do not add empty strings 137
Avoid if (x != y) ..; else ..; 137

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/CommandMap.java

Violation Line
Document empty constructor 4041
Position literals first in String comparisons 59
Avoid if (x != y) ..; else ..; 7377

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/JsonUtils.java

Violation Line
Possible God class (WMC=50, ATFD=25, TCC=0.0) 20288
The class 'JsonUtils' has a Standard Cyclomatic Complexity of 3 (Highest = 12). 36288
The class 'JsonUtils' has a Modified Cyclomatic Complexity of 3 (Highest = 12). 36288
This class has too many methods, consider refactoring it. 36288
The class 'JsonUtils' has a Cyclomatic Complexity of 4 (Highest = 14). 36288
A class which only has private constructors should be final 36288
Consider using varargs for methods or constructors which take an array the last parameter. 43
Avoid appending characters as strings in StringBuffer.append. 44
Avoid appending characters as strings in StringBuffer.append. 48
Parameter 'value' is not assigned and could be declared final 51
Parameter 'builder' is not assigned and could be declared final 51
Consider using varargs for methods or constructors which take an array the last parameter. 51
Avoid appending characters as strings in StringBuffer.append. 52
Avoid appending characters as strings in StringBuffer.append. 56
Avoid appending characters as strings in StringBuffer.append. 58
Avoid appending characters as strings in StringBuffer.append. 60
Avoid appending characters as strings in StringBuffer.append. 63
Parameter 'builder' is not assigned and could be declared final 66
Parameter 'value' is not assigned and could be declared final 66
Avoid appending characters as strings in StringBuffer.append. 67
Avoid appending characters as strings in StringBuffer.append. 71
Avoid appending characters as strings in StringBuffer.append. 76
Avoid appending characters as strings in StringBuffer.append. 80
Avoid appending characters as strings in StringBuffer.append. 84
Avoid appending characters as strings in StringBuffer.append. 88
Avoid appending characters as strings in StringBuffer.append. 92
Avoid reassigning parameters such as 'value' 95
Avoid appending characters as strings in StringBuffer.append. 97
Avoid appending characters as strings in StringBuffer.append. 106
Avoid appending characters as strings in StringBuffer.append. 126
The method 'encode' has a Standard Cyclomatic Complexity of 12. 130183
The method encode() has an NPath complexity of 4608 130183
The method 'encode' has a Modified Cyclomatic Complexity of 12. 130183
The method 'encode' has a Cyclomatic Complexity of 14. 130183
Avoid appending characters as strings in StringBuffer.append. 131
Avoid appending characters as strings in StringBuffer.append. 186
Avoid appending characters as strings in StringBuffer.append. 209
Avoid appending characters as strings in StringBuffer.append. 223
Consider using varargs for methods or constructors which take an array the last parameter. 227
Avoid appending characters as strings in StringBuffer.append. 246
Avoid appending characters as strings in StringBuffer.append. 265
Avoid reassigning parameters such as 'json' 268
Avoid reassigning parameters such as 'json' 268

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/Popup.java

Violation Line
A class which only has private constructors should be final 2671
Private field 'command' could be made final; it is only initialized in the declaration or constructor. 31
Private field 'immediate' could be made final; it is only initialized in the declaration or constructor. 36
Avoid if (x != y) ..; else ..; 4856
Position literals first in String comparisons 50
Avoid if (x != y) ..; else ..; 5761

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/BoxRenderer.java

Violation Line
Avoid unused private fields such as 'LOG'. 42
Avoid if (x != y) ..; else ..; 7175

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/ButtonRenderer.java

Violation Line
The class 'ButtonRenderer' has a Cyclomatic Complexity of 10 (Highest = 9). 49118
The class 'ButtonRenderer' has a Modified Cyclomatic Complexity of 10 (Highest = 9). 49118
The class 'ButtonRenderer' has a Standard Cyclomatic Complexity of 10 (Highest = 9). 49118
Avoid unused private fields such as 'LOG'. 51
The method encodeEnd() has an NPath complexity of 900 54117
Local variable 'image' could be declared final 113

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/ColumnRenderer.java

Violation Line
Avoid unused private methods such as 'isPure(UIColumn)'. 39

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/CommandGroupRenderer.java

Violation Line
Parameter 'component' is not assigned and could be declared final 36
Parameter 'facesContext' is not assigned and could be declared final 36
Parameter 'component' is not assigned and could be declared final 50
Parameter 'facesContext' is not assigned and could be declared final 50
Local variable 'child' could be declared final 53
Parameter 'component' is not assigned and could be declared final 68
Parameter 'facesContext' is not assigned and could be declared final 68

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/CommandRenderer.java

Violation Line
The class 'CommandRenderer' has a Cyclomatic Complexity of 6 (Highest = 15). 45186
The class 'CommandRenderer' has a Standard Cyclomatic Complexity of 6 (Highest = 15). 45186
The class 'CommandRenderer' has a Modified Cyclomatic Complexity of 6 (Highest = 15). 45186
The method encodeBegin() has an NPath complexity of 55440 61143
The method 'encodeBegin' has a Modified Cyclomatic Complexity of 15. 61143
The method 'encodeBegin' has a Cyclomatic Complexity of 15. 61143
The method 'encodeBegin' has a Standard Cyclomatic Complexity of 15. 61143
Avoid if (x != y) ..; else ..; 117
Local variable 'child' could be declared final 158
Local variable 'submenu' could be declared final 161162

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/DateRenderer.java

Violation Line
Parameter 'facesContext' is not assigned and could be declared final 56
Parameter 'component' is not assigned and could be declared final 56
Parameter 'facesContext' is not assigned and could be declared final 69
Parameter 'component' is not assigned and could be declared final 69

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/DecodingRendererBase.java

Violation Line
Local variable 'textarea' could be declared final 64
Deeply nested if..then statements are hard to read 6569

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/FileRenderer.java

Violation Line
Parameter 'event' is not assigned and could be declared final 59
Local variable 'httpServletRequest' could be declared final 79
Parameter 'facesContext' is not assigned and could be declared final 96
Parameter 'component' is not assigned and could be declared final 96
Local variable 'validator' could be declared final 159
Avoid appending characters as strings in StringBuffer.append. 163
Parameter 'component' is not assigned and could be declared final 176
Parameter 'facesContext' is not assigned and could be declared final 176

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/FlexLayoutRenderer.java

Violation Line
Local variable 'b' could be declared final 46
Local variable 'vertically' could be declared final 84

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/FooterRenderer.java

Violation Line
Parameter 'component' is not assigned and could be declared final 38
Parameter 'facesContext' is not assigned and could be declared final 38
Parameter 'facesContext' is not assigned and could be declared final 53
Parameter 'component' is not assigned and could be declared final 53

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/GridLayoutRenderer.java

Violation Line
The class 'GridLayoutRenderer' has a Standard Cyclomatic Complexity of 5 (Highest = 10). 48160
The class 'GridLayoutRenderer' has a Modified Cyclomatic Complexity of 5 (Highest = 10). 48160
The class 'GridLayoutRenderer' has a Cyclomatic Complexity of 5 (Highest = 10). 48160
Avoid unused private fields such as 'LOG'. 50
StringBuffer constructor is initialized with size 16, but has at least 24 characters appended. 63
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 67
Avoid appending characters as strings in StringBuffer.append. 67
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 67
Avoid appending characters as strings in StringBuffer.append. 71
Local variable 'column' could be declared final 77
Avoid unused local variables such as 'column'. 77
The method 'encodeChildren' has a Standard Cyclomatic Complexity of 10. 86151
The method 'encodeChildren' has a Modified Cyclomatic Complexity of 10. 86151
The method 'encodeChildren' has a Cyclomatic Complexity of 10. 86151
Avoid instantiating new objects inside loops 117
Local variable 'builder' could be declared final 117
Avoid appending characters as strings in StringBuffer.append. 118
Avoid appending characters as strings in StringBuffer.append. 125
Avoid appending characters as strings in StringBuffer.append. 133
Avoid appending characters as strings in StringBuffer.append. 139

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/HeaderRenderer.java

Violation Line
Parameter 'facesContext' is not assigned and could be declared final 38
Parameter 'component' is not assigned and could be declared final 38
Parameter 'facesContext' is not assigned and could be declared final 57
Parameter 'component' is not assigned and could be declared final 57

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/HiddenRenderer.java

Violation Line
Parameter 'facesContext' is not assigned and could be declared final 39
Parameter 'component' is not assigned and could be declared final 39
Avoid if (x != y) ..; else ..; 60
Parameter 'component' is not assigned and could be declared final 64
Parameter 'facesContext' is not assigned and could be declared final 64

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/ImageRenderer.java

Violation Line
Avoid if (x != y) ..; else ..; 5269
Avoid if (x != y) ..; else ..; 92

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/InRenderer.java

Violation Line
The class 'InRenderer' has a Standard Cyclomatic Complexity of 5 (Highest = 11). 44129
The class 'InRenderer' has a Modified Cyclomatic Complexity of 5 (Highest = 11). 44129
The class 'InRenderer' has a Cyclomatic Complexity of 5 (Highest = 13). 44129
Parameter 'component' is not assigned and could be declared final 49
The method 'encodeBeginField' has a Cyclomatic Complexity of 13. 49119
The method 'encodeBeginField' has a Modified Cyclomatic Complexity of 11. 49119
Parameter 'facesContext' is not assigned and could be declared final 49
The method 'encodeBeginField' has a Standard Cyclomatic Complexity of 11. 49119
The method encodeBeginField() has an NPath complexity of 3840 49119
Avoid if (x != y) ..; else ..; 112116
Parameter 'facesContext' is not assigned and could be declared final 122
Document empty method 122123
Parameter 'component' is not assigned and could be declared final 122
Document empty method 127128

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/LabelLayoutRendererBase.java

Violation Line
A switch statement does not contain a break 110117
A switch statement does not contain a break 144161
Parameter 'writer' is not assigned and could be declared final 183
Parameter 'labelLayout' is not assigned and could be declared final 183
Parameter 'component' is not assigned and could be declared final 183
Parameter 'component' is not assigned and could be declared final 197

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/LabelRenderer.java

Violation Line
Parameter 'event' is not assigned and could be declared final 48
Avoid if (x != y) ..; else ..; 58

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/LinkRenderer.java

Violation Line
Local variable 'image' could be declared final 80

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/MediatorRenderer.java

Violation Line
Avoid unused private fields such as 'LOG'. 36

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/MenuCommandRenderer.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 20227
The class 'MenuCommandRenderer' has a Standard Cyclomatic Complexity of 5 (Highest = 12). 62227
The class 'MenuCommandRenderer' has a Cyclomatic Complexity of 6 (Highest = 15). 62227
The class 'MenuCommandRenderer' has a Modified Cyclomatic Complexity of 5 (Highest = 12). 62227
Avoid unused private fields such as 'LOG'. 64
Avoid if (x != y) ..; else ..; 75120
Avoid if (x != y) ..; else ..; 87120
Avoid instantiating new objects inside loops 96
Avoid instantiating new objects inside loops 9899
Avoid if (x != y) ..; else ..; 108115
The method 'encodeItem' has a Cyclomatic Complexity of 15. 134214
The method 'encodeItem' has a Modified Cyclomatic Complexity of 12. 134214
The method encodeItem() has an NPath complexity of 2592 134214
Avoid long parameter lists. 134138
The method 'encodeItem' has a Standard Cyclomatic Complexity of 12. 134214
Avoid if (x != y) ..; else ..; 153157
Document empty method 218219

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/MenuRenderer.java

Violation Line
The class 'MenuRenderer' has a Modified Cyclomatic Complexity of 6 (Highest = 10). 41126
The class 'MenuRenderer' has a Cyclomatic Complexity of 8 (Highest = 12). 41126
The class 'MenuRenderer' has a Standard Cyclomatic Complexity of 6 (Highest = 10). 41126
The method encodeBegin() has an NPath complexity of 2240 44110
The method 'encodeBegin' has a Cyclomatic Complexity of 12. 44110
The method 'encodeBegin' has a Standard Cyclomatic Complexity of 10. 44110
The method 'encodeBegin' has a Modified Cyclomatic Complexity of 10. 44110
Avoid appending characters as strings in StringBuffer.append. 74
Avoid appending characters as strings in StringBuffer.append. 75

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/MessagesRenderer.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 20246
The class 'MessagesRenderer' has a Cyclomatic Complexity of 8 (Highest = 12). 56246
The method encodeEnd() has an NPath complexity of 252 63162
The method 'encodeEnd' has a Cyclomatic Complexity of 12. 63162
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 79
Use equals() to compare object references. 104
Use equals() to compare object references. 108
Avoid if (x != y) ..; else ..; 169

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/NavRenderer.java

Violation Line
Parameter 'facesContext' is not assigned and could be declared final 45
Parameter 'component' is not assigned and could be declared final 45
Parameter 'component' is not assigned and could be declared final 75
Parameter 'facesContext' is not assigned and could be declared final 75
Local variable 'child' could be declared final 81
Parameter 'facesContext' is not assigned and could be declared final 100
Parameter 'component' is not assigned and could be declared final 100
Parameter 'navbarId' is not assigned and could be declared final 108
Parameter 'facesContext' is not assigned and could be declared final 108
Parameter 'nav' is not assigned and could be declared final 108
Parameter 'writer' is not assigned and could be declared final 108
Private field 'writer' could be made final; it is only initialized in the declaration or constructor. 159
Avoid using redundant field initializer for 'isInDiv' 161

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/ObjectRenderer.java

Violation Line
Avoid if (x != y) ..; else ..; 5256

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/PageRenderer.java

Violation Line
Possible God class (WMC=59, ATFD=148, TCC=0.0) 20506
A high number of imports can indicate a high degree of coupling within an object. 20506
The class 'PageRenderer' has a Modified Cyclomatic Complexity of 6 (Highest = 27). 70506
The class 'PageRenderer' has a Standard Cyclomatic Complexity of 6 (Highest = 27). 70506
The class 'PageRenderer' has a Cyclomatic Complexity of 7 (Highest = 29). 70506
The method 'encodeBegin' has a Cyclomatic Complexity of 29. 93294
Avoid really long methods. 93294
The method 'encodeBegin' has a Modified Cyclomatic Complexity of 27. 93294
The method 'encodeBegin' has a Standard Cyclomatic Complexity of 27. 93294
The method encodeBegin() has an NCSS line count of 124 93294
The method encodeBegin() has an NPath complexity of 387158400 93294
Avoid if (x != y) ..; else ..; 158
Avoid if (x != y) ..; else ..; 184198
Local variable 'root' could be declared final 200
Local variable 'componentResources' could be declared final 201
Local variable 'childCount' could be declared final 203
Local variable 'child' could be declared final 204
Avoid unused private methods such as 'writeEventFunction(TobagoResponseWriter,Collection,String,boolean)'. 410412
Avoid if (x != y) ..; else ..; 428432

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/PopupRenderer.java

Violation Line
Parameter 'event' is not assigned and could be declared final 45
Parameter 'component' is not assigned and could be declared final 52
Parameter 'facesContext' is not assigned and could be declared final 52
Parameter 'component' is not assigned and could be declared final 75
Parameter 'facesContext' is not assigned and could be declared final 75

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/ProgressRenderer.java

Violation Line
Avoid unused private fields such as 'LOG'. 39

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/ScriptRenderer.java

Violation Line
Parameter 'event' is not assigned and could be declared final 36

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SectionRenderer.java

Violation Line
Local variable 'label' could be declared final 46
A switch statement does not contain a break 4866

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SegmentLayoutRenderer.java

Violation Line
Avoid unused private fields such as 'LOG'. 48
Avoid declaring a variable if it is unreferenced before a possible exit point. 69
Local variable 'child' could be declared final 82
Local variable 'subChild' could be declared final 85
Parameter 'writer' is not assigned and could be declared final 120
Parameter 'generator' is not assigned and could be declared final 120
Parameter 'facesContext' is not assigned and could be declared final 120
Parameter 'child' is not assigned and could be declared final 120

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SelectBooleanCheckboxRenderer.java

Violation Line
The String literal "true" appears 4 times in this file; the first occurrence is on line 64 64
Avoid if (x != y) ..; else ..; 123127

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SelectManyCheckboxRenderer.java

Violation Line
Avoid if (x != y) ..; else ..; 96100

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SelectManyListboxRenderer.java

Violation Line
Avoid if (x != y) ..; else ..; 57
Avoid if (x != y) ..; else ..; 7478

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SelectManyRendererBase.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 20810
Possible God class (WMC=116, ATFD=71, TCC=0.0) 20810
The class 'SelectManyRendererBase' has a Standard Cyclomatic Complexity of 6 (Highest = 28). 57810
The class 'SelectManyRendererBase' has a Modified Cyclomatic Complexity of 6 (Highest = 28). 57810
The class 'SelectManyRendererBase' has a Cyclomatic Complexity of 6 (Highest = 31). 57810
Avoid if (x != y) ..; else ..; 74
Parameter 'input' is not assigned and could be declared final 90
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 122
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 123
Parameter 'facesContext' is not assigned and could be declared final 125
Parameter 'component' is not assigned and could be declared final 125
Consider using varargs for methods or constructors which take an array the last parameter. 126
Parameter 'submittedValue' is not assigned and could be declared final 126
The method 'getConvertedUISelectManyValue' has a Modified Cyclomatic Complexity of 28. 143320
The method 'getConvertedUISelectManyValue' has a Standard Cyclomatic Complexity of 28. 143320
Parameter 'component' is not assigned and could be declared final 143
Parameter 'facesContext' is not assigned and could be declared final 143
Avoid really long methods. 143320
The method getConvertedUISelectManyValue() has an NPath complexity of 2760 143320
The method 'getConvertedUISelectManyValue' has a Cyclomatic Complexity of 31. 143320
Parameter 'submittedValue' is not assigned and could be declared final 144
Parameter 'considerValueType' is not assigned and could be declared final 144
Local variable 'expression' could be declared final 154
Avoid if (x != y) ..; else ..; 165297
Local variable 'modelType' could be declared final 166167
Local variable 'componentType' could be declared final 173
Local variable 'iterator' could be declared final 203
Local variable 'collectionTypeAttr' could be declared final 207208
Avoid if (x != y) ..; else ..; 209289
Local variable 'collectionType' could be declared final 210
Local variable 'componentValue' could be declared final 238239
Local variable 'cloneMethod' could be declared final 244245
Local variable 'clone' could be declared final 246247
Avoid if (x != y) ..; else ..; 260262
Local variable 'isArray' could be declared final 301
Avoid if (x != y) ..; else ..; 305310
Parameter 'facesContext' is not assigned and could be declared final 333
Avoid reassigning parameters such as 'attribute' 334
Parameter 'facesContext' is not assigned and could be declared final 374
Parameter 'component' is not assigned and could be declared final 374
Local variable 'valueTypeAttr' could be declared final 377
Local variable 'valueType' could be declared final 380
Parameter 'facesContext' is not assigned and could be declared final 414
Parameter 'iterator' is not assigned and could be declared final 414
Local variable 'item' could be declared final 421
Local variable 'groupIterator' could be declared final 423424
Local variable 'selectItemsType' could be declared final 427
Parameter 'component' is not assigned and could be declared final 452
Local variable 'builder' could be declared final 453
The String literal "{Component-Path : " appears 4 times in this file; the first occurrence is on line 456 456
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 456
Avoid appending characters as strings in StringBuffer.append. 464
Parameter 'component' is not assigned and could be declared final 469
Parameter 'builder' is not assigned and could be declared final 469
Local variable 'intBuilder' could be declared final 474
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 476
Avoid appending characters as strings in StringBuffer.append. 485
The class 'SelectItemsIterator' has a Standard Cyclomatic Complexity of 6 (Highest = 23). 500779
The class 'SelectItemsIterator' has a Modified Cyclomatic Complexity of 6 (Highest = 23). 500779
The class 'SelectItemsIterator' has a Cyclomatic Complexity of 6 (Highest = 27). 500779
Private field 'facesContext' could be made final; it is only initialized in the declaration or constructor. 518
Parameter 'facesContext' is not assigned and could be declared final 520
Parameter 'selectItemsParent' is not assigned and could be declared final 520
The method hasNext() has an NPath complexity of 1662 529639
The method 'hasNext' has a Cyclomatic Complexity of 27. 529639
The method 'hasNext' has a Standard Cyclomatic Complexity of 23. 529639
The method 'hasNext' has a Modified Cyclomatic Complexity of 23. 529639
Avoid really long methods. 529639
Local variable 'uiSelectItem' could be declared final 560
Local variable 'itemValue' could be declared final 564
Local variable 'description' could be declared final 566
Local variable 'disabled' could be declared final 567
Local variable 'escape' could be declared final 568
Local variable 'noSelectionOption' could be declared final 569
Local variable 'expression' could be declared final 575
Local variable 'value' could be declared final 585
Local variable 'length' could be declared final 594
Local variable 'items' could be declared final 595
Local variable 'map' could be declared final 606
Local variable 'items' could be declared final 607
Local variable 'entry' could be declared final 608
Avoid instantiating new objects inside loops 609
Local variable 'expression' could be declared final 618
Local variable 'objects' could be declared final 619623
Local variable 'message' could be declared final 624626
The method 'next' has a Standard Cyclomatic Complexity of 11. 642715
The method next() has an NPath complexity of 296 642715
The method 'next' has a Cyclomatic Complexity of 12. 642715
The method 'next' has a Modified Cyclomatic Complexity of 11. 642715
Local variable 'value' could be declared final 647
Local variable 'attributeMap' could be declared final 658
Local variable 'var' could be declared final 663
Local variable 'itemDisabled' could be declared final 690
Local variable 'itemLabelEscaped' could be declared final 691
Local variable 'noSelectionValue' could be declared final 692
Avoid if (x != y) ..; else ..; 703709
Parameter 'attrName' is not assigned and could be declared final 726
Parameter 'defaultValue' is not assigned and could be declared final 726
Parameter 'component' is not assigned and could be declared final 726
Local variable 'value' could be declared final 727
Parameter 'component' is not assigned and could be declared final 740
Local variable 'builder' could be declared final 741
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 744
Avoid appending characters as strings in StringBuffer.append. 752
Parameter 'component' is not assigned and could be declared final 757
Parameter 'builder' is not assigned and could be declared final 757
Local variable 'intBuilder' could be declared final 762
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 764
Avoid appending characters as strings in StringBuffer.append. 773

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SelectManyShuttleRenderer.java

Violation Line
Avoid unused local variables such as 'hasLabel'. 56
Avoid if (x != y) ..; else ..; 69
Avoid if (x != y) ..; else ..; 130134

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SelectOneChoiceRenderer.java

Violation Line
Parameter 'component' is not assigned and could be declared final 47
Parameter 'facesContext' is not assigned and could be declared final 47
Avoid if (x != y) ..; else ..; 6771
Parameter 'facesContext' is not assigned and could be declared final 80
Parameter 'component' is not assigned and could be declared final 80

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SelectOneListboxRenderer.java

Violation Line
Avoid if (x != y) ..; else ..; 56
Avoid if (x != y) ..; else ..; 7377
Parameter 'facesContext' is not assigned and could be declared final 84
Parameter 'component' is not assigned and could be declared final 84

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SelectOneRadioRenderer.java

Violation Line
The class 'SelectOneRadioRenderer' has a Standard Cyclomatic Complexity of 6 (Highest = 10). 42134
The class 'SelectOneRadioRenderer' has a Modified Cyclomatic Complexity of 6 (Highest = 10). 42134
The class 'SelectOneRadioRenderer' has a Cyclomatic Complexity of 6 (Highest = 10). 42134
The method 'encodeBeginField' has a Standard Cyclomatic Complexity of 10. 45127
Parameter 'component' is not assigned and could be declared final 45
Parameter 'facesContext' is not assigned and could be declared final 45
The method 'encodeBeginField' has a Cyclomatic Complexity of 10. 45127
The method 'encodeBeginField' has a Modified Cyclomatic Complexity of 10. 45127
Avoid if (x != y) ..; else ..; 99103
Local variable 'tobagoItem' could be declared final 107
Parameter 'facesContext' is not assigned and could be declared final 130
Parameter 'component' is not assigned and could be declared final 130

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SelectReferenceRenderer.java

Violation Line
Avoid unused private fields such as 'LOG'. 35

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SeparatorRenderer.java

Violation Line
Avoid if (x != y) ..; else ..; 4765

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SheetRenderer.java

Violation Line
Possible God class (WMC=175, ATFD=467, TCC=0.008333333333333333) 201102
A high number of imports can indicate a high degree of coupling within an object. 201102
The class 'SheetRenderer' has a Cyclomatic Complexity of 11 (Highest = 71). 941102
Avoid really long classes. 941102
The class 'SheetRenderer' has a Modified Cyclomatic Complexity of 9 (Highest = 65). 941102
The class 'SheetRenderer' has a Standard Cyclomatic Complexity of 10 (Highest = 67). 941102
This class has too many methods, consider refactoring it. 941102
Avoid if (x != y) ..; else ..; 166170
Avoid if (x != y) ..; else ..; 172176
Avoid appending characters as strings in StringBuffer.append. 205
The method encodeEnd() has an NCSS line count of 336 216678
The method 'encodeEnd' has a Cyclomatic Complexity of 71. 216678
The method 'encodeEnd' has a Standard Cyclomatic Complexity of 67. 216678
Avoid really long methods. 216678
The method 'encodeEnd' has a Modified Cyclomatic Complexity of 65. 216678
Avoid if (x != y) ..; else ..; 304
Avoid appending characters as strings in StringBuffer.append. 350
Avoid instantiating new objects inside loops 370
Local variable 'selector' could be declared final 417
Avoid unused local variables such as 'ignored'. 457
Avoid instantiating new objects inside loops 463
The String literal "tobago" appears 13 times in this file; the first occurrence is on line 504 504
Avoid if (x != y) ..; else ..; 507556
Avoid if (x != y) ..; else ..; 604651
Avoid really long methods. 680862
The method encodeHeaderRows() has an NCSS line count of 132 680862
The method 'encodeHeaderRows' has a Standard Cyclomatic Complexity of 20. 680862
The method 'encodeHeaderRows' has a Modified Cyclomatic Complexity of 20. 680862
The method 'encodeHeaderRows' has a Cyclomatic Complexity of 23. 680862
The method encodeHeaderRows() has an NPath complexity of 13765 680862
String.indexOf(char) is faster than String.indexOf(String). 719
Avoid instantiating new objects inside loops 729
Avoid instantiating new objects inside loops 730731
Prefer StringBuffer over += for concatenating strings 738
Prefer StringBuffer over += for concatenating strings 740
Prefer StringBuffer over += for concatenating strings 757
Avoid instantiating new objects inside loops 852
Avoid if (x != y) ..; else ..; 871
The method encodeLink() has an NPath complexity of 200 931969
Parameter 'target' is not assigned and could be declared final 933
Parameter 'icon' is not assigned and could be declared final 933
Parameter 'liClass' is not assigned and could be declared final 933
Avoid if (x != y) ..; else ..; 962966
The method encodeDirectPagingLinks() has an NPath complexity of 648000 9841057
The method 'encodeDirectPagingLinks' has a Standard Cyclomatic Complexity of 20. 9841057
The method 'encodeDirectPagingLinks' has a Cyclomatic Complexity of 23. 9841057
The method 'encodeDirectPagingLinks' has a Modified Cyclomatic Complexity of 20. 9841057
Avoid unused local variables such as 'command'. 988
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 1029
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 1048
Avoid instantiating new objects inside loops 1088
Avoid instantiating new objects inside loops 1090

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/StyleRenderer.java

Violation Line
Parameter 'event' is not assigned and could be declared final 44

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SubviewRenderer.java

Violation Line
Document empty method 4547

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/SuggestRenderer.java

Violation Line
Local variable 'query' could be declared final 60
Parameter 'component' is not assigned and could be declared final 70
Parameter 'facesContext' is not assigned and could be declared final 70
Avoid instantiating new objects inside loops 129

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/TabGroupRenderer.java

Violation Line
Possible God class (WMC=53, ATFD=129, TCC=0.0) 20383
A high number of imports can indicate a high degree of coupling within an object. 20383
The class 'TabGroupRenderer' has a Cyclomatic Complexity of 6 (Highest = 21). 66383
The class 'TabGroupRenderer' has a Modified Cyclomatic Complexity of 5 (Highest = 18). 66383
The class 'TabGroupRenderer' has a Standard Cyclomatic Complexity of 5 (Highest = 18). 66383
The method 'ensureRenderedActiveIndex' has a Standard Cyclomatic Complexity of 10. 129163
The method 'ensureRenderedActiveIndex' has a Cyclomatic Complexity of 10. 129163
The method 'ensureRenderedActiveIndex' has a Modified Cyclomatic Complexity of 10. 129163
Deeply nested if..then statements are hard to read 140142
Avoid if (x != y) ..; else ..; 157161
The method 'encodeHeader' has a Standard Cyclomatic Complexity of 18. 165259
The method encodeHeader() has an NPath complexity of 69126 165259
The method 'encodeHeader' has a Modified Cyclomatic Complexity of 18. 165259
The method 'encodeHeader' has a Cyclomatic Complexity of 21. 165259
Avoid instantiating new objects inside loops 179
Avoid if (x != y) ..; else ..; 234
Avoid if (x != y) ..; else ..; 237241
Avoid declaring a variable if it is unreferenced before a possible exit point. 272
Avoid unused local variables such as 'clientId'. 272
Avoid instantiating new objects inside loops 311
Avoid if (x != y) ..; else ..; 312316
Parameter 'writer' is not assigned and could be declared final 349
Parameter 'tabGroup' is not assigned and could be declared final 349
Parameter 'facesContext' is not assigned and could be declared final 349
Parameter 'activeIndex' is not assigned and could be declared final 350
Parameter 'switchType' is not assigned and could be declared final 350
Parameter 'tab' is not assigned and could be declared final 380
Parameter 'facesContext' is not assigned and could be declared final 380

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/TextareaRenderer.java

Violation Line
The class 'TextareaRenderer' has a Modified Cyclomatic Complexity of 7 (Highest = 12). 44129
The class 'TextareaRenderer' has a Cyclomatic Complexity of 8 (Highest = 13). 44129
The class 'TextareaRenderer' has a Standard Cyclomatic Complexity of 7 (Highest = 12). 44129
The method encodeBeginField() has an NPath complexity of 624 49122
The method 'encodeBeginField' has a Standard Cyclomatic Complexity of 12. 49122
The method 'encodeBeginField' has a Cyclomatic Complexity of 13. 49122
The method 'encodeBeginField' has a Modified Cyclomatic Complexity of 12. 49122
Avoid if (x != y) ..; else ..; 9397
Prefer StringBuffer over += for concatenating strings 114
Prefer StringBuffer over += for concatenating strings 116
Prefer StringBuffer over += for concatenating strings 118
Parameter 'facesContext' is not assigned and could be declared final 125
Parameter 'component' is not assigned and could be declared final 125

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/ToolBarRenderer.java

Violation Line
Avoid unused private fields such as 'LOG'. 42
Avoid unused local variables such as 'toolBar'. 148

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/ToolBarRendererBase.java

Violation Line
Possible God class (WMC=81, ATFD=122, TCC=0.0) 20475
A high number of imports can indicate a high degree of coupling within an object. 20475
The class 'ToolBarRendererBase' has a Cyclomatic Complexity of 5 (Highest = 23). 62475
The class 'ToolBarRendererBase' has a Standard Cyclomatic Complexity of 4 (Highest = 20). 62475
The class 'ToolBarRendererBase' has a Modified Cyclomatic Complexity of 4 (Highest = 20). 62475
This class has too many methods, consider refactoring it. 62475
Parameter 'context' is not assigned and could be declared final 80
Parameter 'component' is not assigned and could be declared final 80
Avoid if (x != y) ..; else ..; 119127
Avoid if (x != y) ..; else ..; 121127
Avoid if (x != y) ..; else ..; 157161
Avoid instantiating new objects inside loops 186
The method 'renderToolbarButton' has a Standard Cyclomatic Complexity of 18. 227340
Avoid really long methods. 227340
The method 'renderToolbarButton' has a Cyclomatic Complexity of 19. 227340
The method 'renderToolbarButton' has a Modified Cyclomatic Complexity of 18. 227340
The method renderToolbarButton() has an NPath complexity of 268800 227340
Avoid if (x != y) ..; else ..; 310314
Avoid unused method parameters such as 'facesContext'. 343
The method getImage() has an NPath complexity of 53760 370444
The method 'getImage' has a Modified Cyclomatic Complexity of 20. 370444
The method 'getImage' has a Cyclomatic Complexity of 23. 370444
The method 'getImage' has a Standard Cyclomatic Complexity of 20. 370444
Avoid if (x != y) ..; else ..; 387391
The String literal "SelectedDisabled" appears 4 times in this file; the first occurrence is on line 388 388
Avoid if (x != y) ..; else ..; 393397
Avoid if (x != y) ..; else ..; 401405
The String literal "Disabled" appears 4 times in this file; the first occurrence is on line 402 402
Avoid if (x != y) ..; else ..; 407411
Avoid if (x != y) ..; else ..; 415419
The String literal "Selected" appears 4 times in this file; the first occurrence is on line 416 416
Avoid if (x != y) ..; else ..; 421425
Avoid if (x != y) ..; else ..; 429433
Avoid if (x != y) ..; else ..; 435439
An empty method in an abstract class should be abstract instead 457459
Document empty method 458459

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/TreeCommandRenderer.java

Violation Line
Avoid if (x != y) ..; else ..; 59

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/TreeIconRenderer.java

Violation Line
The class 'TreeIconRenderer' has a Standard Cyclomatic Complexity of 15 (Highest = 14). 40120
The class 'TreeIconRenderer' has a Modified Cyclomatic Complexity of 15 (Highest = 14). 40120
The class 'TreeIconRenderer' has a Cyclomatic Complexity of 15 (Highest = 14). 40120
The method encodeBegin() has an NPath complexity of 1296 47119
The method 'encodeBegin' has a Cyclomatic Complexity of 14. 47119
The method 'encodeBegin' has a Standard Cyclomatic Complexity of 14. 47119
The method 'encodeBegin' has a Modified Cyclomatic Complexity of 14. 47119
Avoid if (x != y) ..; else ..; 6876
Avoid if (x != y) ..; else ..; 6973
Avoid if (x != y) ..; else ..; 7886
Avoid if (x != y) ..; else ..; 7983
Avoid if (x != y) ..; else ..; 9098
Avoid if (x != y) ..; else ..; 9195

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/TreeIndentRenderer.java

Violation Line
Avoid unused method parameters such as 'node'. 83
Avoid unused local variables such as 'blank'. 89
Avoid unused local variables such as 'perpendicular'. 90
Avoid unused local variables such as 'junction'. 93
Avoid long parameter lists. 109112
Avoid unused method parameters such as 'node'. 110
Avoid unused method parameters such as 'facesContext'. 110
Avoid unused method parameters such as 'showLines'. 111
Avoid unused local variables such as 'hasNextSibling'. 117

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/TreeLabelRenderer.java

Violation Line
Avoid unused private fields such as 'LOG'. 42

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/TreeListboxRenderer.java

Violation Line
The method encodeEnd() has an NPath complexity of 250 53141
Avoid if (x != y) ..; else ..; 100
Avoid if (x != y) ..; else ..; 101103
Avoid if (x != y) ..; else ..; 164173
Avoid if (x != y) ..; else ..; 168172

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/TreeNodeRenderer.java

Violation Line
The class 'TreeNodeRenderer' has a Cyclomatic Complexity of 6 (Highest = 11). 52220
Avoid unused local variables such as 'expanded'. 77
Deeply nested if..then statements are hard to read 9092
Avoid if (x != y) ..; else ..; 98108
Avoid unused local variables such as 'markedValue'. 100
The method encodeBegin() has an NPath complexity of 217 113170
The method 'encodeBegin' has a Cyclomatic Complexity of 11. 113170
Parameter 'data' is not assigned and could be declared final 212
Parameter 'showRoot' is not assigned and could be declared final 212
Parameter 'level' is not assigned and could be declared final 212

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/TreeRenderer.java

Violation Line
The class 'TreeRenderer' has a Cyclomatic Complexity of 4 (Highest = 10). 46159
The method 'encodeEnd' has a Cyclomatic Complexity of 10. 74158
The method encodeEnd() has an NPath complexity of 500 74158
Avoid declaring a variable if it is unreferenced before a possible exit point. 78
Avoid appending characters as strings in StringBuffer.append. 117
Avoid appending characters as strings in StringBuffer.append. 122

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/TreeSelectRenderer.java

Violation Line
The class 'TreeSelectRenderer' has a Cyclomatic Complexity of 5 (Highest = 10). 44152
Avoid declaring a variable if it is unreferenced before a possible exit point. 53
The method 'encodeBegin' has a Cyclomatic Complexity of 10. 80142
Do not add empty strings 146
Do not add empty strings 146
Document empty method 150151

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/standard/standard/tag/VerbatimRenderer.java

Violation Line
Avoid declaring a variable if it is unreferenced before a possible exit point. 38

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/html/util/HtmlRendererUtils.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 20346
Possible God class (WMC=66, ATFD=68, TCC=0.0) 20346
The class 'HtmlRendererUtils' has a Cyclomatic Complexity of 4 (Highest = 20). 61346
The class 'HtmlRendererUtils' has a Standard Cyclomatic Complexity of 3 (Highest = 16). 61346
This class has too many methods, consider refactoring it. 61346
The class 'HtmlRendererUtils' has a Modified Cyclomatic Complexity of 3 (Highest = 16). 61346
Avoid unused private fields such as 'ERROR_FOCUS_KEY'. 64
Parameter 'label' is not assigned and could be declared final 105
Parameter 'writer' is not assigned and could be declared final 105
Parameter 'image' is not assigned and could be declared final 105
Parameter 'writer' is not assigned and could be declared final 117
Parameter 'disabled' is not assigned and could be declared final 117
Parameter 'facesContext' is not assigned and could be declared final 117
Avoid reassigning parameters such as 'image' 117
Parameter 'label' is not assigned and could be declared final 117
Avoid reassigning parameters such as 'title' 166
Avoid reassigning parameters such as 'title' 166
Avoid reassigning parameters such as 'title' 166
Avoid if (x != y) ..; else ..; 196
Avoid if (x != y) ..; else ..; 197
The method 'renderSelectItems' has a Standard Cyclomatic Complexity of 16. 200272
The method renderSelectItems() has an NPath complexity of 9006 200272
The method 'renderSelectItems' has a Cyclomatic Complexity of 20. 200272
The method 'renderSelectItems' has a Modified Cyclomatic Complexity of 16. 200272
Avoid instantiating new objects inside loops 250
Avoid instantiating new objects inside loops 302
Avoid instantiating new objects inside loops 305

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/util/HttpPartWrapper.java

Violation Line
Document empty constructor 3637
Parameter 'delegate' is not assigned and could be declared final 39
Parameter 'headerName' is not assigned and could be declared final 54
Parameter 'headerName' is not assigned and could be declared final 64
Parameter 'fileName' is not assigned and could be declared final 84
Local variable 'wrapped' could be declared final 89
Local variable 'm' could be declared final 91
Parameter 'context' is not assigned and could be declared final 107
Document empty method 112113
Parameter 'state' is not assigned and could be declared final 112
Parameter 'context' is not assigned and could be declared final 112
Document empty method 121122
Parameter 'newTransientValue' is not assigned and could be declared final 121

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/util/JQueryUtils.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 2232
Parameter 'id' is not assigned and could be declared final 24
Parameter 'id' is not assigned and could be declared final 28

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/util/RenderUtils.java

Violation Line
Possible God class (WMC=72, ATFD=57, TCC=0.0) 20357
The class 'RenderUtils' has a Modified Cyclomatic Complexity of 5 (Highest = 11). 53357
The class 'RenderUtils' has a Standard Cyclomatic Complexity of 5 (Highest = 11). 53357
The class 'RenderUtils' has a Cyclomatic Complexity of 5 (Highest = 11). 53357
Avoid if (x != y) ..; else ..; 177183
The method 'decodedStateOfTreeData' has a Modified Cyclomatic Complexity of 11. 189240
The method decodedStateOfTreeData() has an NPath complexity of 330 189240
The method 'decodedStateOfTreeData' has a Standard Cyclomatic Complexity of 11. 189240
The method 'decodedStateOfTreeData' has a Cyclomatic Complexity of 11. 189240
The method 'generateUrl' has a Cyclomatic Complexity of 11. 258313
The method 'generateUrl' has a Modified Cyclomatic Complexity of 11. 258313
The method 'generateUrl' has a Standard Cyclomatic Complexity of 11. 258313
Avoid if (x != y) ..; else ..; 266310
Avoid if (x != y) ..; else ..; 269279
This call to String.startsWith can be rewritten using String.charAt(0) 283
Avoid appending characters as strings in StringBuffer.append. 297
Avoid appending characters as strings in StringBuffer.append. 300
Avoid appending characters as strings in StringBuffer.append. 303
Avoid if (x != y) ..; else ..; 306
Local variable 'behavior' could be declared final 317
Local variable 'clientBehavior' could be declared final 321
Consider simply returning the value vs storing it in local variable 'commands' 327
Local variable 'clientBehavior' could be declared final 348

Tobago Theme Standard - org/apache/myfaces/tobago/renderkit/util/SelectItemUtils.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 45282
Avoid instantiation through private constructors from outside of the constructors class. 61
Local variable 'selectItem' could be declared final 80
The class 'SelectItemsIterator' has a Cyclomatic Complexity of 9 (Highest = 21). 87280
The class 'SelectItemsIterator' has a Standard Cyclomatic Complexity of 8 (Highest = 19). 87280
The class 'SelectItemsIterator' has a Modified Cyclomatic Complexity of 8 (Highest = 19). 87280
The method 'hasNext' has a Modified Cyclomatic Complexity of 19. 102194
The method hasNext() has an NPath complexity of 864 102194
The method 'hasNext' has a Cyclomatic Complexity of 21. 102194
The method 'hasNext' has a Standard Cyclomatic Complexity of 19. 102194
Local variable 'tobagoSelectItem' could be declared final 148149
Local variable 'expression' could be declared final 155
Local variable 'entry' could be declared final 186
Avoid instantiating new objects inside loops 187
The method 'next' has a Standard Cyclomatic Complexity of 12. 197274
The method next() has an NPath complexity of 584 197274
The method 'next' has a Modified Cyclomatic Complexity of 12. 197274
The method 'next' has a Cyclomatic Complexity of 13. 197274
Avoid unused local variables such as 'attributeMap'. 213
Avoid if (x != y) ..; else ..; 264268

Tobago Maven2 Theme Plugin - org/apache/myfaces/tobago/maven/plugin/IndexThemeMojo.java

Violation Line
The class 'IndexThemeMojo' has a Modified Cyclomatic Complexity of 5 (Highest = 12). 37147
The class 'IndexThemeMojo' has a Standard Cyclomatic Complexity of 5 (Highest = 12). 37147
The class 'IndexThemeMojo' has a Cyclomatic Complexity of 5 (Highest = 13). 37147
The method 'execute' has a Modified Cyclomatic Complexity of 12. 61119
The method execute() has an NPath complexity of 768 61119
The method 'execute' has a Cyclomatic Complexity of 13. 61119
The method 'execute' has a Standard Cyclomatic Complexity of 12. 61119
Avoid instantiation through private constructors from outside of the constructors class. 77
Avoid if (x != y) ..; else ..; 89118
These nested if statements could be combined 9294
Parameter 'file' is not assigned and could be declared final 121
Private field 'lastModified' could be made final; it is only initialized in the declaration or constructor. 131

Tobago Maven2 Theme Plugin - org/apache/myfaces/tobago/maven/plugin/UnPackThemeMojo.java

Violation Line
The class 'UnPackThemeMojo' has a Standard Cyclomatic Complexity of 11 (Highest = 23). 53230
The class 'UnPackThemeMojo' has a Cyclomatic Complexity of 13 (Highest = 28). 53230
The class 'UnPackThemeMojo' has a Modified Cyclomatic Complexity of 11 (Highest = 23). 53230
Position literals first in String comparisons 97
Position literals first in String comparisons 97
Avoid instantiating new objects inside loops 100
The method 'execute' has a Cyclomatic Complexity of 28. 123216
The method 'execute' has a Modified Cyclomatic Complexity of 23. 123216
The method 'execute' has a Standard Cyclomatic Complexity of 23. 123216
The method execute() has an NPath complexity of 4575 123216
Avoid instantiating new objects inside loops 136
Avoid instantiating new objects inside loops 142
Avoid if (x != y) ..; else ..; 144149
Deeply nested if..then statements are hard to read 147149
Avoid instantiating new objects inside loops 156
Deeply nested if..then statements are hard to read 158177
Avoid instantiating new objects inside loops 166
Avoid instantiating new objects inside loops 168
Deeply nested if..then statements are hard to read 183187
Avoid instantiating new objects inside loops 190
Deeply nested if..then statements are hard to read 192195
Deeply nested if..then statements are hard to read 196198
Avoid instantiating new objects inside loops 199

Tobago Tool Apt - org/apache/myfaces/tobago/apt/generate/ClassInfo.java

Violation Line
Private field 'className' could be made final; it is only initialized in the declaration or constructor. 27
Private field 'packageName' could be made final; it is only initialized in the declaration or constructor. 28
Private field 'imports' could be made final; it is only initialized in the declaration or constructor. 29
Private field 'interfaces' could be made final; it is only initialized in the declaration or constructor. 31
Private field 'sourceClass' could be made final; it is only initialized in the declaration or constructor. 32
Avoid if (x != y) ..; else ..; 5156
Avoid if (x != y) ..; else ..; 7782

Tobago Tool Apt - org/apache/myfaces/tobago/apt/generate/ClassUtils.java

Violation Line
A class which only has private constructors should be final 2571

Tobago Tool Apt - org/apache/myfaces/tobago/apt/generate/ComponentInfo.java

Violation Line
Private field 'nonTransientProperties' could be made final; it is only initialized in the declaration or constructor. 30
Private field 'transientProperties' could be made final; it is only initialized in the declaration or constructor. 31
Avoid using redundant field initializer for 'index' 35
Avoid using redundant field initializer for 'nonTransientIndex' 36
Avoid if (x != y) ..; else ..; 5561

Tobago Tool Apt - org/apache/myfaces/tobago/apt/generate/Imports.java

Violation Line
Private field 'imports' could be made final; it is only initialized in the declaration or constructor. 28
Document empty constructor 3233
Avoid reassigning parameters such as 'qualifiedName' 39
These nested if statements could be combined 4149
Deeply nested if..then statements are hard to read 4345
Deeply nested if..then statements are hard to read 4648

Tobago Tool Apt - org/apache/myfaces/tobago/apt/generate/PropertyInfo.java

Violation Line
Possible God class (WMC=54, ATFD=14, TCC=0.08478513356562137) 20262
Document empty constructor 4142
Position literals first in String comparisons 125
Consider using varargs for methods or constructors which take an array the last parameter. 135
Consider using varargs for methods or constructors which take an array the last parameter. 143

Tobago Tool Apt - org/apache/myfaces/tobago/apt/generate/TagInfo.java

Violation Line
Private field 'properties' could be made final; it is only initialized in the declaration or constructor. 27

Tobago Tool Apt - org/apache/myfaces/tobago/apt/processor/AbstractGenerator.java

Violation Line
The class 'AbstractGenerator' has a Modified Cyclomatic Complexity of 2 (Highest = 11). 40125
The class 'AbstractGenerator' has a Standard Cyclomatic Complexity of 2 (Highest = 11). 40125
The class 'AbstractGenerator' has a Cyclomatic Complexity of 2 (Highest = 11). 40125
The method 'process' has a Modified Cyclomatic Complexity of 11. 4695
The method 'process' has a Standard Cyclomatic Complexity of 11. 4695
The method 'process' has a Cyclomatic Complexity of 11. 4695
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 55
These nested if statements could be combined 6769
These nested if statements could be combined 7274
A method/constructor shouldnt explicitly throw java.lang.Exception 99

Tobago Tool Apt - org/apache/myfaces/tobago/apt/processor/CheckstyleConfigGenerator.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 20289
This class has too many methods, consider refactoring it. 70289
The String literal "name" appears 5 times in this file; the first occurrence is on line 96 96
Avoid if (x != y) ..; else ..; 152161
Avoid if (x != y) ..; else ..; 154161
Avoid if (x != y) ..; else ..; 156161

Tobago Tool Apt - org/apache/myfaces/tobago/apt/processor/ClassesGenerator.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 20291
The class 'ClassesGenerator' has a Cyclomatic Complexity of 4 (Highest = 13). 64291
The class 'ClassesGenerator' has a Modified Cyclomatic Complexity of 4 (Highest = 12). 64291
The class 'ClassesGenerator' has a Standard Cyclomatic Complexity of 4 (Highest = 12). 64291
Private field 'renderer' could be made final; it is only initialized in the declaration or constructor. 68
A method/constructor shouldnt explicitly throw java.lang.Exception 93
Position literals first in String comparisons 128
Position literals first in String comparisons 128
The method addProperty() has an NPath complexity of 1403 212261
The method 'addProperty' has a Modified Cyclomatic Complexity of 12. 212261
The method 'addProperty' has a Cyclomatic Complexity of 13. 212261
The method 'addProperty' has a Standard Cyclomatic Complexity of 12. 212261
Deeply nested if..then statements are hard to read 219221
Deeply nested if..then statements are hard to read 224227
Deeply nested if..then statements are hard to read 235237

Tobago Tool Apt - org/apache/myfaces/tobago/apt/processor/FacesConfigGenerator.java

Violation Line
Possible God class (WMC=82, ATFD=89, TCC=0.047619047619047616) 20639
A high number of imports can indicate a high degree of coupling within an object. 20639
The class 'FacesConfigGenerator' has a Standard Cyclomatic Complexity of 4 (Highest = 11). 74639
The class 'FacesConfigGenerator' has a Modified Cyclomatic Complexity of 4 (Highest = 11). 74639
This class has too many methods, consider refactoring it. 74639
The class 'FacesConfigGenerator' has a Cyclomatic Complexity of 4 (Highest = 11). 74639
The method 'generate' has a Modified Cyclomatic Complexity of 11. 145243
The method generate() has an NPath complexity of 241 145243
The method 'generate' has a Cyclomatic Complexity of 11. 145243
The method 'generate' has a Standard Cyclomatic Complexity of 11. 145243
A method/constructor shouldnt explicitly throw java.lang.Exception 145
Avoid if (x != y) ..; else ..; 177183
Avoid if (x != y) ..; else ..; 179183
Avoid unused private methods such as 'getIndexAfter(org.jdom.Element,String)'. 315
These nested if statements could be combined 326330
These nested if statements could be combined 327329
Position literals first in String comparisons 360
Prefer StringBuffer over += for concatenating strings 392
Prefer StringBuffer over += for concatenating strings 518
Avoid instantiating new objects inside loops 563
Avoid instantiating new objects inside loops 566
Avoid instantiating new objects inside loops 570
Avoid instantiating new objects inside loops 573
Avoid instantiating new objects inside loops 574
Prefer StringBuffer over += for concatenating strings 578
A method/constructor shouldnt explicitly throw java.lang.Exception 589
Avoid if (x != y) ..; else ..; 596625

Tobago Tool Apt - org/apache/myfaces/tobago/apt/processor/TaglibGenerator.java

Violation Line
Possible God class (WMC=83, ATFD=51, TCC=0.003952569169960474) 20547
A high number of imports can indicate a high degree of coupling within an object. 20547
The class 'TaglibGenerator' has a Cyclomatic Complexity of 4 (Highest = 21). 74547
The class 'TaglibGenerator' has a Modified Cyclomatic Complexity of 3 (Highest = 19). 74547
This class has too many methods, consider refactoring it. 74547
The class 'TaglibGenerator' has a Standard Cyclomatic Complexity of 3 (Highest = 19). 74547
Prefer StringBuffer over += for concatenating strings 156
Parameter 'functionName' is not assigned and could be declared final 172
Parameter 'taglib' is not assigned and could be declared final 172
Parameter 'functionClass' is not assigned and could be declared final 172
Parameter 'document' is not assigned and could be declared final 172
Parameter 'functionSignature' is not assigned and could be declared final 173
The method 'addDescription' has a Cyclomatic Complexity of 21. 239322
The method 'addDescription' has a Standard Cyclomatic Complexity of 19. 239322
The method addDescription() has an NPath complexity of 83520 239322
The method 'addDescription' has a Modified Cyclomatic Complexity of 19. 239322
StringBuffer constructor is initialized with size 16, but has at least 106 characters appended. 242
The String literal "</p>" appears 4 times in this file; the first occurrence is on line 257 257
Avoid reassigning parameters such as 'string' 324
Avoid reassigning parameters such as 'string' 324
String.indexOf(char) is faster than String.indexOf(String). 332
StringBuffer constructor is initialized with size 16, but has at least 65 characters appended. 352
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 353
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 354
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 355
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 355
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 356
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 357
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 361
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 366
Unnecessary use of fully qualified name 'org.w3c.dom.Element' due to existing import 'org.w3c.dom.Element' 438
Unnecessary use of fully qualified name 'org.w3c.dom.Element' due to existing import 'org.w3c.dom.Element' 444
Unnecessary use of fully qualified name 'org.w3c.dom.Element' due to existing import 'org.w3c.dom.Element' 445
Unnecessary use of fully qualified name 'org.w3c.dom.Element' due to existing import 'org.w3c.dom.Element' 451
Unnecessary use of fully qualified name 'org.w3c.dom.Element' due to existing import 'org.w3c.dom.Element' 452
The String literal "handler-class" appears 4 times in this file; the first occurrence is on line 486 486
Avoid if (x != y) ..; else ..; 518544