PMD Results

The following document contains the results of PMD 5.0.2.

Files

Tobago Core - org/apache/myfaces/tobago/ajax/AjaxUtils.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 51
Potential violation of Law of Demeter (object not created locally) 52
Potential violation of Law of Demeter (object not created locally) 59
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (object not created locally) 74
Potential violation of Law of Demeter (method chain calls) 79
Potential violation of Law of Demeter (object not created locally) 80
Potential violation of Law of Demeter (method chain calls) 110 - 111
Potential violation of Law of Demeter (method chain calls) 110 - 111
Potential violation of Law of Demeter (method chain calls) 127
Potential violation of Law of Demeter (object not created locally) 128
Potential violation of Law of Demeter (object not created locally) 133

Tobago Core - org/apache/myfaces/tobago/application/ActionListenerImpl.java

Violation Line
Private field 'base' could be made final; it is only initialized in the declaration or constructor. 41
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 53
Potential violation of Law of Demeter (object not created locally) 59
Potential violation of Law of Demeter (object not created locally) 62
Potential violation of Law of Demeter (object not created locally) 64
Potential violation of Law of Demeter (object not created locally) 65
Potential violation of Law of Demeter (method chain calls) 65
Potential violation of Law of Demeter (object not created locally) 66
Potential violation of Law of Demeter (object not created locally) 70
Potential violation of Law of Demeter (object not created locally) 71
Potential violation of Law of Demeter (object not created locally) 73
Potential violation of Law of Demeter (object not created locally) 75
Avoid if (x != y) ..; else ..; 75
Potential violation of Law of Demeter (object not created locally) 76
Potential violation of Law of Demeter (object not created locally) 78

Tobago Core - org/apache/myfaces/tobago/application/LabelValueExpressionFacesMessage.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 47 - 48
Potential violation of Law of Demeter (method chain calls) 47 - 48
Potential violation of Law of Demeter (method chain calls) 47 - 48
Potential violation of Law of Demeter (object not created locally) 47 - 48
Potential violation of Law of Demeter (object not created locally) 48
Potential violation of Law of Demeter (object not created locally) 49
Potential violation of Law of Demeter (object not created locally) 49
Potential violation of Law of Demeter (method chain calls) 55 - 56
Potential violation of Law of Demeter (method chain calls) 55 - 56
Potential violation of Law of Demeter (object not created locally) 55 - 56
Potential violation of Law of Demeter (method chain calls) 55 - 56
Potential violation of Law of Demeter (object not created locally) 56
Potential violation of Law of Demeter (object not created locally) 57
Potential violation of Law of Demeter (object not created locally) 57

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

Violation Line
This class has too many methods, consider refactoring it. 48 - 268
Useless parentheses. 71
Potential violation of Law of Demeter (object not created locally) 71
These nested if statements could be combined 72 - 74
Potential violation of Law of Demeter (object not created locally) 90
Potential violation of Law of Demeter (method chain calls) 135
Potential violation of Law of Demeter (method chain calls) 145
Potential violation of Law of Demeter (method chain calls) 150
Potential violation of Law of Demeter (object not created locally) 157
Potential violation of Law of Demeter (object not created locally) 165
Potential violation of Law of Demeter (method chain calls) 180
Potential violation of Law of Demeter (method chain calls) 180
Potential violation of Law of Demeter (object not created locally) 180
Potential violation of Law of Demeter (method chain calls) 195
Potential violation of Law of Demeter (object not created locally) 195
Potential violation of Law of Demeter (object not created locally) 196
Potential violation of Law of Demeter (object not created locally) 196
Avoid if (x != y) ..; else ..; 206
Potential violation of Law of Demeter (method chain calls) 206
Potential violation of Law of Demeter (method chain calls) 206
Potential violation of Law of Demeter (method chain calls) 206
Avoid if (x != y) ..; else ..; 216 - 221
Potential violation of Law of Demeter (object not created locally) 216
Potential violation of Law of Demeter (method chain calls) 217
Potential violation of Law of Demeter (method chain calls) 219 - 220
Potential violation of Law of Demeter (method chain calls) 219 - 220
Potential violation of Law of Demeter (method chain calls) 219 - 220
Potential violation of Law of Demeter (method chain calls) 219 - 220
Potential violation of Law of Demeter (object not created locally) 220

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

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

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

Violation Line
This class has too many methods, consider refactoring it. 37 - 231
Document empty constructor 47 - 48
Consider using varargs for methods or constructors which take an array the last parameter. 93
Potential violation of Law of Demeter (object not created locally) 112
Potential violation of Law of Demeter (object not created locally) 112
Potential violation of Law of Demeter (object not created locally) 112
Potential violation of Law of Demeter (method chain calls) 123
Potential violation of Law of Demeter (method chain calls) 124
Potential violation of Law of Demeter (method chain calls) 139
Potential violation of Law of Demeter (method chain calls) 139
Potential violation of Law of Demeter (method chain calls) 139
Potential violation of Law of Demeter (method chain calls) 139
Potential violation of Law of Demeter (method chain calls) 140
Potential violation of Law of Demeter (method chain calls) 140
Potential violation of Law of Demeter (method chain calls) 215
Potential violation of Law of Demeter (method chain calls) 215
Potential violation of Law of Demeter (method chain calls) 215
Potential violation of Law of Demeter (method chain calls) 218
Potential violation of Law of Demeter (method chain calls) 218
Potential violation of Law of Demeter (method chain calls) 218

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

Violation Line
Avoid using redundant field initializer for 'isTransient' 38
Potential violation of Law of Demeter (method chain calls) 60
Unnecessary use of fully qualified name 'javax.faces.el.MethodNotFoundException' due to existing import 'javax.faces.el.MethodNotFoundException' 62
Consider using varargs for methods or constructors which take an array the last parameter. 68
Unnecessary use of fully qualified name 'javax.faces.el.MethodNotFoundException' due to existing import 'javax.faces.el.MethodNotFoundException' 74

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

Violation Line
The class 'Sorter' has a Cyclomatic Complexity of 8 (Highest = 28). 49 - 238
Possible God class (WMC=48, ATFD=25, TCC=0.2) 49 - 238
The method 'perform' has a Cyclomatic Complexity of 28. 55 - 181
Avoid really long methods. 55 - 181
The method perform() has an NPath complexity of 34968 55 - 181
Potential violation of Law of Demeter (method chain calls) 57
Potential violation of Law of Demeter (method chain calls) 57
Potential violation of Law of Demeter (object not created locally) 62
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (object not created locally) 67
Potential violation of Law of Demeter (object not created locally) 75
Avoid if (x != y) ..; else ..; 76 - 117
Potential violation of Law of Demeter (object not created locally) 79
Avoid if (x != y) ..; else ..; 79 - 112
Potential violation of Law of Demeter (object not created locally) 80
Potential violation of Law of Demeter (method chain calls) 86
Potential violation of Law of Demeter (object not created locally) 86
Potential violation of Law of Demeter (object not created locally) 88
Potential violation of Law of Demeter (object not created locally) 89
Potential violation of Law of Demeter (object not created locally) 91 - 92
Potential violation of Law of Demeter (object not created locally) 92
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 97
Potential violation of Law of Demeter (object not created locally) 104
Potential violation of Law of Demeter (object not created locally) 105
Potential violation of Law of Demeter (object not created locally) 119
Potential violation of Law of Demeter (object not created locally) 133
Potential violation of Law of Demeter (method chain calls) 133
Potential violation of Law of Demeter (object not created locally) 134
Potential violation of Law of Demeter (method chain calls) 134
Potential violation of Law of Demeter (object not created locally) 136
Potential violation of Law of Demeter (method chain calls) 138
Potential violation of Law of Demeter (object not created locally) 142
Potential violation of Law of Demeter (object not created locally) 155
Potential violation of Law of Demeter (method chain calls) 155
Potential violation of Law of Demeter (method chain calls) 159
Potential violation of Law of Demeter (method chain calls) 160
Potential violation of Law of Demeter (object not created locally) 172
Potential violation of Law of Demeter (method chain calls) 172
Potential violation of Law of Demeter (method chain calls) 178 - 179
Potential violation of Law of Demeter (object not created locally) 179
Avoid if (x != y) ..; else ..; 179
Potential violation of Law of Demeter (method chain calls) 179
Potential violation of Law of Demeter (method chain calls) 179
Potential violation of Law of Demeter (method chain calls) 201
The method 'getFirstSortableChild' has a Cyclomatic Complexity of 13. 204 - 229
Potential violation of Law of Demeter (method chain calls) 212
Potential violation of Law of Demeter (object not created locally) 212
Useless parentheses. 212
Potential violation of Law of Demeter (method chain calls) 212
Useless parentheses. 213
Potential violation of Law of Demeter (static property access) 213
Potential violation of Law of Demeter (object not created locally) 213
Potential violation of Law of Demeter (object not created locally) 222

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/component/UIMenuSelectOne.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 33

Tobago Core - org/apache/myfaces/tobago/config/TobagoConfig.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 35
Potential violation of Law of Demeter (method chain calls) 35

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

Violation Line
Potential violation of Law of Demeter (static property access) 144
Potential violation of Law of Demeter (method chain calls) 145
Potential violation of Law of Demeter (method chain calls) 145
Potential violation of Law of Demeter (static property access) 154
Potential violation of Law of Demeter (method chain calls) 155
Potential violation of Law of Demeter (method chain calls) 155
Potential violation of Law of Demeter (object not created locally) 192
Potential violation of Law of Demeter (method chain calls) 192
Use equals() to compare object references. 194

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

Violation Line
This class has too many methods, consider refactoring it. 50 - 317
Consider using varargs for methods or constructors which take an array the last parameter. 96
Consider using varargs for methods or constructors which take an array the last parameter. 106
Potential violation of Law of Demeter (method chain calls) 110
Potential violation of Law of Demeter (method chain calls) 114
Potential violation of Law of Demeter (object not created locally) 114
Potential violation of Law of Demeter (method chain calls) 114
Avoid if (x != y) ..; else ..; 166
Avoid if (x != y) ..; else ..; 178
Avoid if (x != y) ..; else ..; 179
Use equals() to compare object references. 201
Avoid if (x != y) ..; else ..; 204 - 215
Potential violation of Law of Demeter (object not created locally) 211
Avoid if (x != y) ..; else ..; 245 - 254
Potential violation of Law of Demeter (object not created locally) 251
Potential violation of Law of Demeter (method chain calls) 276
Potential violation of Law of Demeter (method chain calls) 276

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
Potential violation of Law of Demeter (method chain calls) 38
Potential violation of Law of Demeter (method chain calls) 43
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (method chain calls) 68
Potential violation of Law of Demeter (method chain calls) 73
Potential violation of Law of Demeter (method chain calls) 74
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (method chain calls) 88
Consider using varargs for methods or constructors which take an array the last parameter. 92
Consider using varargs for methods or constructors which take an array the last parameter. 100
Potential violation of Law of Demeter (method chain calls) 131
Potential violation of Law of Demeter (method chain calls) 136
Potential violation of Law of Demeter (method chain calls) 141 - 142
Potential violation of Law of Demeter (object not created locally) 158 - 161
This call to String.startsWith can be rewritten using String.charAt(0) 158
Useless parentheses. 158 - 161
Potential violation of Law of Demeter (object not created locally) 158
Potential violation of Law of Demeter (object not created locally) 159
Potential violation of Law of Demeter (object not created locally) 160
Potential violation of Law of Demeter (object not created locally) 161

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

Violation Line
Possible God class (WMC=48, ATFD=16, TCC=0.07142857142857142) 33 - 261
Private field 'productionResources' could be made final; it is only initialized in the declaration or constructor. 53
Private field 'resources' could be made final; it is only initialized in the declaration or constructor. 55
Potential violation of Law of Demeter (object not created locally) 124
Potential violation of Law of Demeter (object not created locally) 141
Potential violation of Law of Demeter (object not created locally) 142
Deeply nested if..then statements are hard to read 143 - 148
Potential violation of Law of Demeter (object not created locally) 146
Potential violation of Law of Demeter (object not created locally) 163
Potential violation of Law of Demeter (object not created locally) 164
Potential violation of Law of Demeter (object not created locally) 165
Avoid appending characters as strings in StringBuffer.append. 174
Avoid appending characters as strings in StringBuffer.append. 177
Potential violation of Law of Demeter (method chain calls) 212
Potential violation of Law of Demeter (method chain calls) 213
Potential violation of Law of Demeter (method chain calls) 214
Potential violation of Law of Demeter (method chain calls) 214
Potential violation of Law of Demeter (method chain calls) 216
Potential violation of Law of Demeter (method chain calls) 217
Potential violation of Law of Demeter (method chain calls) 218
Potential violation of Law of Demeter (method chain calls) 218
Potential violation of Law of Demeter (method chain calls) 221
Potential violation of Law of Demeter (method chain calls) 222
Potential violation of Law of Demeter (method chain calls) 223
Potential violation of Law of Demeter (method chain calls) 223
Potential violation of Law of Demeter (method chain calls) 225
Potential violation of Law of Demeter (method chain calls) 226
Potential violation of Law of Demeter (method chain calls) 227
Potential violation of Law of Demeter (method chain calls) 227

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
Private field 'scriptList' could be made final; it is only initialized in the declaration or constructor. 34
Private field 'styleList' could be made final; it is only initialized in the declaration or constructor. 35
Potential violation of Law of Demeter (method chain calls) 41
Potential violation of Law of Demeter (method chain calls) 42
Potential violation of Law of Demeter (method chain calls) 46
Potential violation of Law of Demeter (method chain calls) 47

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
Potential violation of Law of Demeter (method chain calls) 56
Potential violation of Law of Demeter (method chain calls) 56

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 37
Potential violation of Law of Demeter (object not created locally) 47
Potential violation of Law of Demeter (object not created locally) 47

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

Violation Line
Document empty constructor 33 - 34

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

Violation Line
The class 'UserAgent' has a Cyclomatic Complexity of 3 (Highest = 18). 33 - 380
The String literal "msie" appears 11 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 103 103
The String literal "mozilla" appears 4 times in this file; the first occurrence is on line 127 127
The String literal "gecko" appears 5 times in this file; the first occurrence is on line 150 150
Potential violation of Law of Demeter (static property access) 223
Potential violation of Law of Demeter (static property access) 227
Potential violation of Law of Demeter (static property access) 227
Potential violation of Law of Demeter (static property access) 235
The method 'getInstance' has a Cyclomatic Complexity of 18. 271 - 324
Potential violation of Law of Demeter (static property access) 331
Avoid if (x != y) ..; else ..; 340 - 342
Consider using varargs for methods or constructors which take an array the last parameter. 354
Consider using varargs for methods or constructors which take an array the last parameter. 372

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 51

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

Violation Line
The class 'DurationConverter' has a Cyclomatic Complexity of 8 (Highest = 10). 37 - 150
Potential violation of Law of Demeter (method chain calls) 57
Potential violation of Law of Demeter (method chain calls) 67
Unnecessary wrapper object creation 67
Potential violation of Law of Demeter (object not created locally) 75
Potential violation of Law of Demeter (object not created locally) 76
Potential violation of Law of Demeter (object not created locally) 79
Avoid declaring a variable if it is unreferenced before a possible exit point. 90
Potential violation of Law of Demeter (object not created locally) 93
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 99
Potential violation of Law of Demeter (object not created locally) 101
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (object not created locally) 103
Potential violation of Law of Demeter (object not created locally) 106
Potential violation of Law of Demeter (object not created locally) 107
The method 'getUnitFactor' has a Cyclomatic Complexity of 10. 121 - 148
Potential violation of Law of Demeter (method chain calls) 124

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 43
Potential violation of Law of Demeter (method chain calls) 53

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

Violation Line
Document empty constructor 38 - 39
Consider using varargs for methods or constructors which take an array the last parameter. 41

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
Potential violation of Law of Demeter (object not created locally) 43
Potential violation of Law of Demeter (method chain calls) 43
Potential violation of Law of Demeter (object not created locally) 59
Potential violation of Law of Demeter (method chain calls) 59

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 39 - 40
Potential violation of Law of Demeter (method chain calls) 40
Potential violation of Law of Demeter (object not created locally) 42

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 35 - 36

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

Violation Line
Avoid using redundant field initializer for 'isTransient' 33
Document empty constructor 35 - 36

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

Violation Line
Avoid using redundant field initializer for 'isTransient' 34
Document empty constructor 36 - 37
Avoid unused private methods such as 'elContext()'. 47
Potential violation of Law of Demeter (method chain calls) 48

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

Violation Line
Avoid using redundant field initializer for 'isTransient' 37
Document empty constructor 39 - 40
Avoid unused private methods such as 'elContext()'. 50
Potential violation of Law of Demeter (method chain calls) 51

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

Violation Line
Avoid using redundant field initializer for 'isTransient' 33
Document empty constructor 35 - 36
Avoid unused private methods such as 'elContext()'. 46
Potential violation of Law of Demeter (method chain calls) 47

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

Violation Line
Avoid if (x != y) ..; else ..; 85 - 89

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 37 - 38
Potential violation of Law of Demeter (method chain calls) 52 - 53
Potential violation of Law of Demeter (method chain calls) 53

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 37
Avoid if (x != y) ..; else ..; 40 - 46
Potential violation of Law of Demeter (method chain calls) 41 - 42
Potential violation of Law of Demeter (method chain calls) 41 - 42
Potential violation of Law of Demeter (method chain calls) 41
Potential violation of Law of Demeter (object not created locally) 41
Potential violation of Law of Demeter (method chain calls) 42
Potential violation of Law of Demeter (method chain calls) 44 - 45
Potential violation of Law of Demeter (method chain calls) 45

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 32

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

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

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 40

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/TabChangeEvent.java

Violation Line
Overridable method 'setOldState' called during object construction 45
Overridable method 'setNewState' called during object construction 46
Potential violation of Law of Demeter (method chain calls) 55

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
Potential violation of Law of Demeter (method chain calls) 47
Potential violation of Law of Demeter (method chain calls) 49
Avoid empty if statements 50 - 52

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

Violation Line
Private field 'oldMarked' could be made final; it is only initialized in the declaration or constructor. 34
Private field 'newMarked' could be made final; it is only initialized in the declaration or constructor. 35
Potential violation of Law of Demeter (method chain calls) 50
Potential violation of Law of Demeter (method chain calls) 52
Avoid empty if statements 53 - 55

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
Potential violation of Law of Demeter (method chain calls) 46
Potential violation of Law of Demeter (method chain calls) 48
Avoid empty if statements 49 - 51

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

Violation Line
Document empty constructor 56 - 57
Potential violation of Law of Demeter (method chain calls) 61
Potential violation of Law of Demeter (method chain calls) 61
Potential violation of Law of Demeter (method chain calls) 66 - 69
Potential violation of Law of Demeter (method chain calls) 69
Potential violation of Law of Demeter (method chain calls) 80

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

Violation Line
Document empty constructor 44 - 45
Potential violation of Law of Demeter (method chain calls) 52
Potential violation of Law of Demeter (method chain calls) 52
Potential violation of Law of Demeter (method chain calls) 76

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

Violation Line
Document empty constructor 37 - 38
Avoid if (x != y) ..; else ..; 47 - 51
Potential violation of Law of Demeter (object not created locally) 48
Potential violation of Law of Demeter (method chain calls) 63
Potential violation of Law of Demeter (method chain calls) 79
Potential violation of Law of Demeter (method chain calls) 79
No need to check for null before an instanceof 80
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (method chain calls) 95
Potential violation of Law of Demeter (method chain calls) 95
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (object not created locally) 96

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

Violation Line
Document empty constructor 37 - 38
Avoid if (x != y) ..; else ..; 47 - 51
Potential violation of Law of Demeter (object not created locally) 48
Potential violation of Law of Demeter (method chain calls) 63
Potential violation of Law of Demeter (method chain calls) 79
Potential violation of Law of Demeter (method chain calls) 79
No need to check for null before an instanceof 80
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (method chain calls) 95
Potential violation of Law of Demeter (method chain calls) 95
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (object not created locally) 96

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. 20 - 366
The class 'AttributeHandler' has a Cyclomatic Complexity of 7 (Highest = 59). 56 - 366
Possible God class (WMC=79, ATFD=79, TCC=0.027777777777777776) 56 - 366
The method apply() has an NCSS line count of 165 73 - 283
The method 'apply' has a Cyclomatic Complexity of 59. 73 - 283
Avoid really long methods. 73 - 283
Avoid if (x != y) ..; else ..; 80 - 281
Potential violation of Law of Demeter (method chain calls) 81
Avoid if (x != y) ..; else ..; 84 - 102
Potential violation of Law of Demeter (method chain calls) 88
Potential violation of Law of Demeter (object not created locally) 93
Potential violation of Law of Demeter (method chain calls) 103
Potential violation of Law of Demeter (method chain calls) 104
Avoid if (x != y) ..; else ..; 107 - 125
Potential violation of Law of Demeter (method chain calls) 111
Potential violation of Law of Demeter (object not created locally) 116
Potential violation of Law of Demeter (method chain calls) 126
Potential violation of Law of Demeter (method chain calls) 127
Potential violation of Law of Demeter (method chain calls) 132
Potential violation of Law of Demeter (object not created locally) 141
Potential violation of Law of Demeter (object not created locally) 149 - 150
Potential violation of Law of Demeter (method chain calls) 151
Potential violation of Law of Demeter (method chain calls) 153
Potential violation of Law of Demeter (method chain calls) 158
Potential violation of Law of Demeter (object not created locally) 167
Potential violation of Law of Demeter (object not created locally) 178 - 179
Potential violation of Law of Demeter (method chain calls) 182
Potential violation of Law of Demeter (method chain calls) 187
Potential violation of Law of Demeter (method chain calls) 192
Avoid if (x != y) ..; else ..; 193 - 200
Potential violation of Law of Demeter (object not created locally) 195
Potential violation of Law of Demeter (method chain calls) 209
Potential violation of Law of Demeter (static property access) 218
Potential violation of Law of Demeter (static property access) 224
Potential violation of Law of Demeter (method chain calls) 229
Potential violation of Law of Demeter (static property access) 233
Potential violation of Law of Demeter (static property access) 236
Potential violation of Law of Demeter (method chain calls) 239
Potential violation of Law of Demeter (method chain calls) 245
Potential violation of Law of Demeter (method chain calls) 245
Potential violation of Law of Demeter (static property access) 247
Potential violation of Law of Demeter (static property access) 254
Potential violation of Law of Demeter (static property access) 261
Potential violation of Law of Demeter (static property access) 263
Potential violation of Law of Demeter (method chain calls) 266
Potential violation of Law of Demeter (static property access) 268
Potential violation of Law of Demeter (method chain calls) 274
Potential violation of Law of Demeter (method chain calls) 276
Potential violation of Law of Demeter (method chain calls) 303
Consider using varargs for methods or constructors which take an array the last parameter. 307
Potential violation of Law of Demeter (method chain calls) 311
Avoid if (x != y) ..; else ..; 313 - 319
Potential violation of Law of Demeter (object not created locally) 315 - 316
Potential violation of Law of Demeter (object not created locally) 316
Potential violation of Law of Demeter (object not created locally) 336 - 337
Potential violation of Law of Demeter (object not created locally) 338
Potential violation of Law of Demeter (method chain calls) 345
Potential violation of Law of Demeter (method chain calls) 360
Potential violation of Law of Demeter (method chain calls) 360
Potential violation of Law of Demeter (method chain calls) 361

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 36
Potential violation of Law of Demeter (method chain calls) 52

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

Violation Line
Avoid declaring a variable if it is unreferenced before a possible exit point. 53
Potential violation of Law of Demeter (object not created locally) 58
Potential violation of Law of Demeter (method chain calls) 62 - 63
Potential violation of Law of Demeter (method chain calls) 62 - 63
Potential violation of Law of Demeter (method chain calls) 62 - 63
Potential violation of Law of Demeter (method chain calls) 63
Potential violation of Law of Demeter (object not created locally) 65
New exception is thrown in catch block, original stack trace may be lost 65
Deeply nested if..then statements are hard to read 67 - 69
Potential violation of Law of Demeter (object not created locally) 68
Potential violation of Law of Demeter (object not created locally) 72

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

Violation Line
Potential violation of Law of Demeter (static property access) 39
Potential violation of Law of Demeter (static property access) 42
Potential violation of Law of Demeter (method chain calls) 57
Potential violation of Law of Demeter (method chain calls) 60
Potential violation of Law of Demeter (method chain calls) 74
Potential violation of Law of Demeter (method chain calls) 77

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

Violation Line
Document empty method 31 - 33
Document empty method 44 - 45

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

Violation Line
The class 'FlowLayoutRule' has a Cyclomatic Complexity of 10 (Highest = 9). 33 - 140
These nested if statements could be combined 39 - 58
Potential violation of Law of Demeter (static property access) 40
Deeply nested if..then statements are hard to read 40 - 42
Potential violation of Law of Demeter (static property access) 43
Deeply nested if..then statements are hard to read 43 - 45
Deeply nested if..then statements are hard to read 46 - 48
Potential violation of Law of Demeter (static property access) 46
Deeply nested if..then statements are hard to read 49 - 51
Potential violation of Law of Demeter (static property access) 49
Potential violation of Law of Demeter (static property access) 52
Deeply nested if..then statements are hard to read 52 - 54
Deeply nested if..then statements are hard to read 55 - 57
Potential violation of Law of Demeter (static property access) 55
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (object not created locally) 85
Potential violation of Law of Demeter (object not created locally) 98
Potential violation of Law of Demeter (object not created locally) 111
Potential violation of Law of Demeter (object not created locally) 124
Potential violation of Law of Demeter (object not created locally) 137

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

Violation Line
Possible God class (WMC=62, ATFD=138, TCC=0.0) 38 - 342
The class 'GridLayoutConstraintHandler' has a Cyclomatic Complexity of 32 (Highest = 62). 38 - 342
Too many fields 38 - 342
The method apply() has an NCSS line count of 140 98 - 341
Avoid reassigning parameters such as 'parent' 98
Avoid reassigning parameters such as 'parent' 98
Avoid really long methods. 98 - 341
The method 'apply' has a Cyclomatic Complexity of 62. 98 - 341
The method apply() has an NPath complexity of 2106404298 98 - 341
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (method chain calls) 103
Potential violation of Law of Demeter (method chain calls) 110
Potential violation of Law of Demeter (method chain calls) 119
Potential violation of Law of Demeter (object not created locally) 128
Potential violation of Law of Demeter (object not created locally) 137
Potential violation of Law of Demeter (object not created locally) 146
Potential violation of Law of Demeter (object not created locally) 155
Potential violation of Law of Demeter (object not created locally) 164
Potential violation of Law of Demeter (object not created locally) 173
Potential violation of Law of Demeter (object not created locally) 182
Potential violation of Law of Demeter (object not created locally) 191
Potential violation of Law of Demeter (object not created locally) 200
Potential violation of Law of Demeter (object not created locally) 209
Potential violation of Law of Demeter (object not created locally) 218
Potential violation of Law of Demeter (object not created locally) 227
Potential violation of Law of Demeter (object not created locally) 243
Potential violation of Law of Demeter (object not created locally) 252
Potential violation of Law of Demeter (object not created locally) 261
Potential violation of Law of Demeter (object not created locally) 270
Potential violation of Law of Demeter (object not created locally) 279
Potential violation of Law of Demeter (object not created locally) 288
Potential violation of Law of Demeter (object not created locally) 297
Potential violation of Law of Demeter (object not created locally) 306

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

Violation Line
The class 'GridLayoutRule' has a Cyclomatic Complexity of 12 (Highest = 11). 32 - 171
The method applyRule() has an NPath complexity of 258 36 - 66
The method 'applyRule' has a Cyclomatic Complexity of 11. 36 - 66
These nested if statements could be combined 38 - 63
Potential violation of Law of Demeter (static property access) 39
Deeply nested if..then statements are hard to read 39 - 41
Potential violation of Law of Demeter (static property access) 42
Deeply nested if..then statements are hard to read 42 - 44
Potential violation of Law of Demeter (static property access) 45
Deeply nested if..then statements are hard to read 45 - 47
Potential violation of Law of Demeter (static property access) 48
Deeply nested if..then statements are hard to read 48 - 50
Potential violation of Law of Demeter (static property access) 51
Deeply nested if..then statements are hard to read 51 - 53
Potential violation of Law of Demeter (static property access) 54
Deeply nested if..then statements are hard to read 54 - 56
Deeply nested if..then statements are hard to read 57 - 59
Potential violation of Law of Demeter (static property access) 57
Potential violation of Law of Demeter (static property access) 60
Deeply nested if..then statements are hard to read 60 - 62
Potential violation of Law of Demeter (object not created locally) 77
Potential violation of Law of Demeter (object not created locally) 90
Potential violation of Law of Demeter (object not created locally) 103
Potential violation of Law of Demeter (object not created locally) 116
Potential violation of Law of Demeter (object not created locally) 129
Potential violation of Law of Demeter (object not created locally) 142
Potential violation of Law of Demeter (object not created locally) 155
Potential violation of Law of Demeter (object not created locally) 168

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 48 - 49
Potential violation of Law of Demeter (method chain calls) 48 - 49
Potential violation of Law of Demeter (method chain calls) 48 - 49

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

Violation Line
Avoid reassigning parameters such as 'path' 52
Avoid reassigning parameters such as 'path' 52
This call to String.startsWith can be rewritten using String.charAt(0) 58
Potential violation of Law of Demeter (method chain calls) 63
Potential violation of Law of Demeter (method chain calls) 63

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

Violation Line
Potential violation of Law of Demeter (static property access) 39
Potential violation of Law of Demeter (static property access) 42
Potential violation of Law of Demeter (method chain calls) 57
Potential violation of Law of Demeter (method chain calls) 60
Potential violation of Law of Demeter (method chain calls) 74
Potential violation of Law of Demeter (method chain calls) 77

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 61
These nested if statements could be combined 73 - 75
Potential violation of Law of Demeter (method chain calls) 84
Potential violation of Law of Demeter (object not created locally) 91
Potential violation of Law of Demeter (object not created locally) 94
Avoid empty catch blocks 95 - 98
Document empty method 113 - 114
Potential violation of Law of Demeter (object not created locally) 127
Potential violation of Law of Demeter (object not created locally) 134
Avoid if (x != y) ..; else ..; 134

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 39
Potential violation of Law of Demeter (method chain calls) 41
Potential violation of Law of Demeter (method chain calls) 43
Document empty method 48 - 49

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

Violation Line
These nested if statements could be combined 40 - 42
Potential violation of Law of Demeter (method chain calls) 40
Potential violation of Law of Demeter (method chain calls) 56 - 57

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

Violation Line
These nested if statements could be combined 41 - 43
Potential violation of Law of Demeter (method chain calls) 41
Potential violation of Law of Demeter (method chain calls) 57 - 58

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 40
Potential violation of Law of Demeter (method chain calls) 42
Potential violation of Law of Demeter (method chain calls) 44
Document empty method 49 - 50

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 41
These nested if statements could be combined 41 - 43
These nested if statements could be combined 46 - 50
These nested if statements could be combined 47 - 49
Potential violation of Law of Demeter (method chain calls) 47
Deeply nested if..then statements are hard to read 47 - 49
Potential violation of Law of Demeter (method chain calls) 63 - 64
Potential violation of Law of Demeter (method chain calls) 76

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

Violation Line
Potential violation of Law of Demeter (static property access) 40
These nested if statements could be combined 40 - 42
Potential violation of Law of Demeter (method chain calls) 57
Potential violation of Law of Demeter (method chain calls) 60

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

Violation Line
These nested if statements could be combined 40 - 42
Potential violation of Law of Demeter (static property access) 40
Potential violation of Law of Demeter (method chain calls) 58
Potential violation of Law of Demeter (method chain calls) 61

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 70
Potential violation of Law of Demeter (object not created locally) 76
New exception is thrown in catch block, original stack trace may be lost 76
Deeply nested if..then statements are hard to read 78 - 80
Potential violation of Law of Demeter (object not created locally) 79
Avoid if (x != y) ..; else ..; 82 - 87
Potential violation of Law of Demeter (object not created locally) 86

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

Violation Line
These nested if statements could be combined 37 - 39
Potential violation of Law of Demeter (method chain calls) 37
Potential violation of Law of Demeter (method chain calls) 53 - 54

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

Violation Line
New exception is thrown in catch block, original stack trace may be lost 63
New exception is thrown in catch block, original stack trace may be lost 65
New exception is thrown in catch block, original stack trace may be lost 67
Consider using varargs for methods or constructors which take an array the last parameter. 71
New exception is thrown in catch block, original stack trace may be lost 75
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
Potential violation of Law of Demeter (method chain calls) 100
Potential violation of Law of Demeter (method chain calls) 101

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

Violation Line
The class 'TobagoComponentHandler' has a Cyclomatic Complexity of 5 (Highest = 11). 52 - 147
The method 'createMetaRuleset' has a Cyclomatic Complexity of 11. 64 - 97
The method createMetaRuleset() has an NPath complexity of 1024 64 - 97
Potential violation of Law of Demeter (method chain calls) 66
Potential violation of Law of Demeter (object not created locally) 67
Potential violation of Law of Demeter (method chain calls) 69
Potential violation of Law of Demeter (object not created locally) 70
Potential violation of Law of Demeter (method chain calls) 72
Potential violation of Law of Demeter (object not created locally) 73
Potential violation of Law of Demeter (method chain calls) 75
Potential violation of Law of Demeter (object not created locally) 76
Potential violation of Law of Demeter (method chain calls) 78
Potential violation of Law of Demeter (object not created locally) 79
Potential violation of Law of Demeter (method chain calls) 81
Potential violation of Law of Demeter (object not created locally) 82
Potential violation of Law of Demeter (method chain calls) 84
Potential violation of Law of Demeter (object not created locally) 85
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (object not created locally) 88
Potential violation of Law of Demeter (method chain calls) 90
Potential violation of Law of Demeter (object not created locally) 91
Potential violation of Law of Demeter (method chain calls) 93
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (method chain calls) 103
Potential violation of Law of Demeter (method chain calls) 110
Potential violation of Law of Demeter (method chain calls) 111
Potential violation of Law of Demeter (method chain calls) 112
Potential violation of Law of Demeter (object not created locally) 122
Potential violation of Law of Demeter (object not created locally) 123
Potential violation of Law of Demeter (method chain calls) 131
Potential violation of Law of Demeter (method chain calls) 131
Potential violation of Law of Demeter (method chain calls) 131
Potential violation of Law of Demeter (method chain calls) 131
Potential violation of Law of Demeter (method chain calls) 131
Potential violation of Law of Demeter (method chain calls) 131
Potential violation of Law of Demeter (method chain calls) 133
Potential violation of Law of Demeter (object not created locally) 133
Potential violation of Law of Demeter (object not created locally) 136
Potential violation of Law of Demeter (object not created locally) 137
Potential violation of Law of Demeter (method chain calls) 137
Potential violation of Law of Demeter (method chain calls) 137
Potential violation of Law of Demeter (object not created locally) 141
Potential violation of Law of Demeter (method chain calls) 141
Potential violation of Law of Demeter (method chain calls) 143
Potential violation of Law of Demeter (method chain calls) 143

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 36
Potential violation of Law of Demeter (object not created locally) 37

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

Violation Line
Private field 'outcomeAttribute' could be made final; it is only initialized in the declaration or constructor. 30
Potential violation of Law of Demeter (method chain calls) 42

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

Violation Line
Private field 'pickerIdAttribute' could be made final; it is only initialized in the declaration or constructor. 41
Private field 'formIdAttribute' could be made final; it is only initialized in the declaration or constructor. 42
Private field 'markupAttribute' could be made final; it is only initialized in the declaration or constructor. 43
Potential violation of Law of Demeter (method chain calls) 64
Avoid unused local variables such as 'root'. 65
Potential violation of Law of Demeter (object not created locally) 67
Potential violation of Law of Demeter (object not created locally) 68
Avoid if (x != y) ..; else ..; 69 - 71
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (method chain calls) 73
Potential violation of Law of Demeter (object not created locally) 75
Potential violation of Law of Demeter (object not created locally) 76
Potential violation of Law of Demeter (object not created locally) 77
Avoid if (x != y) ..; else ..; 78 - 80
Potential violation of Law of Demeter (object not created locally) 81
Potential violation of Law of Demeter (method chain calls) 82
Potential violation of Law of Demeter (object not created locally) 82
Potential violation of Law of Demeter (object not created locally) 83
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (object not created locally) 84
Potential violation of Law of Demeter (object not created locally) 88
Potential violation of Law of Demeter (object not created locally) 91
Potential violation of Law of Demeter (method chain calls) 94
Potential violation of Law of Demeter (object not created locally) 94

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. 34
Avoid if (x != y) ..; else ..; 44 - 53
Potential violation of Law of Demeter (object not created locally) 46
Potential violation of Law of Demeter (object not created locally) 49
Potential violation of Law of Demeter (object not created locally) 52

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. 40
Potential violation of Law of Demeter (method chain calls) 50
Potential violation of Law of Demeter (method chain calls) 52
Potential violation of Law of Demeter (method chain calls) 60
Potential violation of Law of Demeter (object not created locally) 61
Potential violation of Law of Demeter (object not created locally) 62
Potential violation of Law of Demeter (object not created locally) 63
Potential violation of Law of Demeter (method chain calls) 65
Potential violation of Law of Demeter (object not created locally) 68
Potential violation of Law of Demeter (object not created locally) 71
Potential violation of Law of Demeter (method chain calls) 78
Potential violation of Law of Demeter (object not created locally) 79
Potential violation of Law of Demeter (method chain calls) 82
Potential violation of Law of Demeter (object not created locally) 85
Potential violation of Law of Demeter (object not created locally) 85

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

Violation Line
Private field 'labelWidthAttribute' could be made final; it is only initialized in the declaration or constructor. 59
Private field 'tipAttribute' could be made final; it is only initialized in the declaration or constructor. 60
Private field 'labelAttribute' could be made final; it is only initialized in the declaration or constructor. 61
Private field 'markupAttribute' could be made final; it is only initialized in the declaration or constructor. 62
Private field 'fieldIdAttribute' could be made final; it is only initialized in the declaration or constructor. 63
Potential violation of Law of Demeter (method chain calls) 92
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (method chain calls) 94
Potential violation of Law of Demeter (object not created locally) 97
Potential violation of Law of Demeter (method chain calls) 97
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 103
Potential violation of Law of Demeter (method chain calls) 105
Potential violation of Law of Demeter (method chain calls) 108
Potential violation of Law of Demeter (method chain calls) 116
Avoid if (x != y) ..; else ..; 122 - 126
Potential violation of Law of Demeter (object not created locally) 131
Potential violation of Law of Demeter (object not created locally) 132
Potential violation of Law of Demeter (object not created locally) 133
Potential violation of Law of Demeter (method chain calls) 138
An empty method in an abstract class should be abstract instead 141 - 142
Document empty method 141 - 142
Potential violation of Law of Demeter (object not created locally) 150
Potential violation of Law of Demeter (object not created locally) 151
Potential violation of Law of Demeter (method chain calls) 152
Potential violation of Law of Demeter (object not created locally) 152
Potential violation of Law of Demeter (object not created locally) 163
Potential violation of Law of Demeter (object not created locally) 166
Potential violation of Law of Demeter (object not created locally) 171
Potential violation of Law of Demeter (object not created locally) 174
Potential violation of Law of Demeter (method chain calls) 177
Potential violation of Law of Demeter (method chain calls) 195
Potential violation of Law of Demeter (method chain calls) 196
Potential violation of Law of Demeter (method chain calls) 201
Potential violation of Law of Demeter (object not created locally) 209
Avoid if (x != y) ..; else ..; 210 - 219
Potential violation of Law of Demeter (object not created locally) 216
Potential violation of Law of Demeter (object not created locally) 218
Potential violation of Law of Demeter (object not created locally) 220
Potential violation of Law of Demeter (object not created locally) 221
Potential violation of Law of Demeter (method chain calls) 223
Potential violation of Law of Demeter (method chain calls) 225
Potential violation of Law of Demeter (method chain calls) 227
Potential violation of Law of Demeter (method chain calls) 236
Potential violation of Law of Demeter (object not created locally) 245
Potential violation of Law of Demeter (object not created locally) 246
Potential violation of Law of Demeter (method chain calls) 247
Potential violation of Law of Demeter (object not created locally) 248
Potential violation of Law of Demeter (method chain calls) 250
Potential violation of Law of Demeter (object not created locally) 251
Potential violation of Law of Demeter (method chain calls) 258
Potential violation of Law of Demeter (method chain calls) 261
Potential violation of Law of Demeter (object not created locally) 261
Potential violation of Law of Demeter (object not created locally) 262
Potential violation of Law of Demeter (object not created locally) 262

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. 42
Potential violation of Law of Demeter (method chain calls) 57
Potential violation of Law of Demeter (method chain calls) 59
Potential violation of Law of Demeter (method chain calls) 68
Potential violation of Law of Demeter (object not created locally) 69
Avoid if (x != y) ..; else ..; 70 - 72
Potential violation of Law of Demeter (object not created locally) 73
Potential violation of Law of Demeter (object not created locally) 74
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (method chain calls) 84
Potential violation of Law of Demeter (method chain calls) 92
Potential violation of Law of Demeter (method chain calls) 95 - 96
Potential violation of Law of Demeter (object not created locally) 95
Potential violation of Law of Demeter (object not created locally) 95 - 96
Potential violation of Law of Demeter (method chain calls) 95
Potential violation of Law of Demeter (method chain calls) 95 - 96
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (method chain calls) 96
Potential violation of Law of Demeter (object not created locally) 97
Potential violation of Law of Demeter (object not created locally) 97
Potential violation of Law of Demeter (object not created locally) 105
Potential violation of Law of Demeter (object not created locally) 106

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 65
No need to check for null before an instanceof 79
Potential violation of Law of Demeter (method chain calls) 80
Potential violation of Law of Demeter (method chain calls) 86
Potential violation of Law of Demeter (method chain calls) 86
Potential violation of Law of Demeter (method chain calls) 88
Potential violation of Law of Demeter (object not created locally) 88
Potential violation of Law of Demeter (object not created locally) 89
Potential violation of Law of Demeter (method chain calls) 105
Potential violation of Law of Demeter (method chain calls) 109
Potential violation of Law of Demeter (method chain calls) 110
Potential violation of Law of Demeter (method chain calls) 110
Potential violation of Law of Demeter (object not created locally) 111
Potential violation of Law of Demeter (object not created locally) 112
Potential violation of Law of Demeter (method chain calls) 126
Potential violation of Law of Demeter (object not created locally) 127
Potential violation of Law of Demeter (method chain calls) 135
Potential violation of Law of Demeter (method chain calls) 135
Potential violation of Law of Demeter (object not created locally) 139
Potential violation of Law of Demeter (method chain calls) 155
Potential violation of Law of Demeter (method chain calls) 155
Potential violation of Law of Demeter (method chain calls) 160
Potential violation of Law of Demeter (method chain calls) 160
These nested if statements could be combined 162 - 164
Potential violation of Law of Demeter (object not created locally) 163
Use equals() to compare object references. 180
Potential violation of Law of Demeter (object not created locally) 184
Potential violation of Law of Demeter (object not created locally) 192
Potential violation of Law of Demeter (method chain calls) 204
Potential violation of Law of Demeter (method chain calls) 204
These nested if statements could be combined 214 - 216

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 29
Potential violation of Law of Demeter (method chain calls) 35

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. 43
Private field 'messages' could be made final; it is only initialized in the declaration or constructor. 45
Avoid if (x != y) ..; else ..; 104 - 108
Potential violation of Law of Demeter (object not created locally) 105
Potential violation of Law of Demeter (method chain calls) 110
Potential violation of Law of Demeter (method chain calls) 110
Avoid if (x != y) ..; else ..; 116 - 123
Potential violation of Law of Demeter (object not created locally) 118
Potential violation of Law of Demeter (object not created locally) 126
Potential violation of Law of Demeter (object not created locally) 127
Use equals() to compare object references. 128
Potential violation of Law of Demeter (object not created locally) 131
Potential violation of Law of Demeter (object not created locally) 131
Potential violation of Law of Demeter (method chain calls) 140
Potential violation of Law of Demeter (object not created locally) 142
Potential violation of Law of Demeter (object not created locally) 146
Parameter 'facesContext' is not assigned and could be declared final 151
Potential violation of Law of Demeter (method chain calls) 152
Potential violation of Law of Demeter (object not created locally) 153

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. 61
Potential violation of Law of Demeter (object not created locally) 71
Potential violation of Law of Demeter (object not created locally) 71
Potential violation of Law of Demeter (object not created locally) 85
Potential violation of Law of Demeter (object not created locally) 86
Potential violation of Law of Demeter (object not created locally) 87
Potential violation of Law of Demeter (object not created locally) 89
Potential violation of Law of Demeter (method chain calls) 90
Potential violation of Law of Demeter (object not created locally) 91
Potential violation of Law of Demeter (object not created locally) 93
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (object not created locally) 98
Potential violation of Law of Demeter (object not created locally) 99
Potential violation of Law of Demeter (object not created locally) 100
Potential violation of Law of Demeter (object not created locally) 101
Potential violation of Law of Demeter (object not created locally) 104
Potential violation of Law of Demeter (method chain calls) 112
Potential violation of Law of Demeter (object not created locally) 113
Potential violation of Law of Demeter (object not created locally) 114
Avoid reassigning parameters such as 'charset' 118
Potential violation of Law of Demeter (object not created locally) 123
Potential violation of Law of Demeter (object not created locally) 124
Potential violation of Law of Demeter (object not created locally) 125
Avoid if (x != y) ..; else ..; 135 - 139
Potential violation of Law of Demeter (object not created locally) 136
Potential violation of Law of Demeter (method chain calls) 136
Potential violation of Law of Demeter (object not created locally) 143
Potential violation of Law of Demeter (object not created locally) 144
Potential violation of Law of Demeter (object not created locally) 145
Potential violation of Law of Demeter (object not created locally) 150
Potential violation of Law of Demeter (object not created locally) 151
Potential violation of Law of Demeter (object not created locally) 152
Potential violation of Law of Demeter (object not created locally) 153
Potential violation of Law of Demeter (object not created locally) 154
Potential violation of Law of Demeter (object not created locally) 163
Potential violation of Law of Demeter (object not created locally) 164
Potential violation of Law of Demeter (object not created locally) 166
Potential violation of Law of Demeter (object not created locally) 169
Potential violation of Law of Demeter (object not created locally) 170
Potential violation of Law of Demeter (object not created locally) 171
Potential violation of Law of Demeter (method chain calls) 175
Potential violation of Law of Demeter (method chain calls) 177
Potential violation of Law of Demeter (method chain calls) 179
Potential violation of Law of Demeter (object not created locally) 191

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. 24 - 26

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

Violation Line
An empty method in an abstract class should be abstract instead 31 - 33

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. 27 - 28

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

Violation Line
An empty method in an abstract class should be abstract instead 27 - 29

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUICommand.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. 24 - 27

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

Violation Line
These nested if statements could be combined 40 - 42
Potential violation of Law of Demeter (object not created locally) 67

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 72 72
Avoid if (x != y) ..; else ..; 81 - 85
Avoid if (x != y) ..; else ..; 103
Avoid if (x != y) ..; else ..; 146 - 150
Avoid if (x != y) ..; else ..; 155
Avoid if (x != y) ..; else ..; 160
An empty method in an abstract class should be abstract instead 165 - 167
An empty method in an abstract class should be abstract instead 182 - 184
Potential violation of Law of Demeter (object not created locally) 195
Potential violation of Law of Demeter (object not created locally) 199
Potential violation of Law of Demeter (object not created locally) 200
String.indexOf(char) is faster than String.indexOf(String). 201
Potential violation of Law of Demeter (object not created locally) 201
Potential violation of Law of Demeter (object not created locally) 203
Potential violation of Law of Demeter (method chain calls) 231
Potential violation of Law of Demeter (method chain calls) 243

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIDate.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. 22 - 23

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

Violation Line
Avoid unused private fields such as 'LOG'. 37
Potential violation of Law of Demeter (method chain calls) 50
Potential violation of Law of Demeter (method chain calls) 51
Potential violation of Law of Demeter (object not created locally) 52
Potential violation of Law of Demeter (object not created locally) 53
Potential violation of Law of Demeter (object not created locally) 54
Potential violation of Law of Demeter (method chain calls) 54
Potential violation of Law of Demeter (object not created locally) 55
Potential violation of Law of Demeter (method chain calls) 57
Potential violation of Law of Demeter (object not created locally) 57
Potential violation of Law of Demeter (method chain calls) 65
An empty method in an abstract class should be abstract instead 69 - 71
An empty method in an abstract class should be abstract instead 73 - 75
An empty method in an abstract class should be abstract instead 77 - 79

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 37
Potential violation of Law of Demeter (method chain calls) 37

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

Violation Line
An empty method in an abstract class should be abstract instead 30 - 31
Document empty method 30 - 31
An empty method in an abstract class should be abstract instead 33 - 34
Document empty method 33 - 34
An empty method in an abstract class should be abstract instead 36 - 37
Document empty method 36 - 37
Document empty method 39 - 41
An empty method in an abstract class should be abstract instead 39 - 41
An empty method in an abstract class should be abstract instead 43 - 44
Document empty method 43 - 44

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 54
Avoid if (x != y) ..; else ..; 74 - 85
Potential violation of Law of Demeter (object not created locally) 83
Avoid if (x != y) ..; else ..; 94 - 105
Potential violation of Law of Demeter (object not created locally) 103
These nested if statements could be combined 113 - 115
Potential violation of Law of Demeter (static property access) 113
Potential violation of Law of Demeter (method chain calls) 113
Potential violation of Law of Demeter (method chain calls) 113
Potential violation of Law of Demeter (method chain calls) 117
Potential violation of Law of Demeter (method chain calls) 117

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

Violation Line
Possible God class (WMC=121, ATFD=97, TCC=0.003076923076923077) 51 - 516
The class 'AbstractUIGridLayout' has a Cyclomatic Complexity of 5 (Highest = 27). 51 - 516
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (method chain calls) 70
Avoid instantiating new objects inside loops 74
Potential violation of Law of Demeter (method chain calls) 79
Potential violation of Law of Demeter (method chain calls) 79
The method fixRelativeInsideAuto() has an NPath complexity of 400 87 - 133
The method 'fixRelativeInsideAuto' has a Cyclomatic Complexity of 15. 87 - 133
Potential violation of Law of Demeter (method chain calls) 89
Potential violation of Law of Demeter (method chain calls) 98
Potential violation of Law of Demeter (method chain calls) 98
Deeply nested if..then statements are hard to read 99 - 102
Potential violation of Law of Demeter (method chain calls) 103
Potential violation of Law of Demeter (method chain calls) 103
Potential violation of Law of Demeter (object not created locally) 113
Potential violation of Law of Demeter (method chain calls) 113
Potential violation of Law of Demeter (object not created locally) 119
Potential violation of Law of Demeter (object not created locally) 120
Potential violation of Law of Demeter (method chain calls) 121
Potential violation of Law of Demeter (method chain calls) 124
Potential violation of Law of Demeter (object not created locally) 124
Potential violation of Law of Demeter (method chain calls) 124
Potential violation of Law of Demeter (object not created locally) 125
Potential violation of Law of Demeter (method chain calls) 130
Potential violation of Law of Demeter (method chain calls) 130
The method preProcessing() has an NPath complexity of 57620 135 - 248
Avoid really long methods. 135 - 248
The method 'preProcessing' has a Cyclomatic Complexity of 27. 135 - 248
Potential violation of Law of Demeter (method chain calls) 137
Potential violation of Law of Demeter (object not created locally) 148
Potential violation of Law of Demeter (object not created locally) 151
Potential violation of Law of Demeter (method chain calls) 152
Potential violation of Law of Demeter (method chain calls) 152
Potential violation of Law of Demeter (method chain calls) 152
Potential violation of Law of Demeter (method chain calls) 152
Potential violation of Law of Demeter (method chain calls) 154
Potential violation of Law of Demeter (method chain calls) 154
Avoid instantiating new objects inside loops 158
Potential violation of Law of Demeter (object not created locally) 163
Potential violation of Law of Demeter (object not created locally) 165
Potential violation of Law of Demeter (object not created locally) 165
Potential violation of Law of Demeter (method chain calls) 166
Potential violation of Law of Demeter (method chain calls) 166
Potential violation of Law of Demeter (object not created locally) 170
Potential violation of Law of Demeter (object not created locally) 170
These nested if statements could be combined 170 - 178
Useless parentheses. 170
Potential violation of Law of Demeter (object not created locally) 171
Avoid instantiating new objects inside loops 172
Deeply nested if..then statements are hard to read 174 - 176
Potential violation of Law of Demeter (object not created locally) 183
Potential violation of Law of Demeter (method chain calls) 185
Potential violation of Law of Demeter (method chain calls) 185
Potential violation of Law of Demeter (method chain calls) 188
Potential violation of Law of Demeter (method chain calls) 188
Potential violation of Law of Demeter (object not created locally) 189
Potential violation of Law of Demeter (object not created locally) 190
Potential violation of Law of Demeter (method chain calls) 190
Potential violation of Law of Demeter (method chain calls) 190
Potential violation of Law of Demeter (method chain calls) 192
Potential violation of Law of Demeter (method chain calls) 192
Potential violation of Law of Demeter (method chain calls) 199
Potential violation of Law of Demeter (method chain calls) 199
Potential violation of Law of Demeter (object not created locally) 223
Avoid empty if statements 224 - 227
Potential violation of Law of Demeter (object not created locally) 228
Potential violation of Law of Demeter (object not created locally) 235
Potential violation of Law of Demeter (object not created locally) 239
Potential violation of Law of Demeter (object not created locally) 240
Potential violation of Law of Demeter (object not created locally) 241
Potential violation of Law of Demeter (object not created locally) 242
Potential violation of Law of Demeter (object not created locally) 243
Potential violation of Law of Demeter (object not created locally) 244
Potential violation of Law of Demeter (object not created locally) 245
The method mainProcessing() has an NPath complexity of 30400 250 - 350
Avoid really long methods. 250 - 350
The method 'mainProcessing' has a Cyclomatic Complexity of 23. 250 - 350
Potential violation of Law of Demeter (method chain calls) 252
Potential violation of Law of Demeter (object not created locally) 262
Potential violation of Law of Demeter (object not created locally) 262
Potential violation of Law of Demeter (object not created locally) 263
Potential violation of Law of Demeter (method chain calls) 263
Potential violation of Law of Demeter (object not created locally) 263
Potential violation of Law of Demeter (method chain calls) 263
Potential violation of Law of Demeter (object not created locally) 263
Potential violation of Law of Demeter (object not created locally) 266
Avoid if (x != y) ..; else ..; 270 - 305
Potential violation of Law of Demeter (object not created locally) 272
Potential violation of Law of Demeter (object not created locally) 272
Potential violation of Law of Demeter (object not created locally) 274
Potential violation of Law of Demeter (object not created locally) 275
Potential violation of Law of Demeter (object not created locally) 276
Potential violation of Law of Demeter (object not created locally) 277
Potential violation of Law of Demeter (object not created locally) 278
Potential violation of Law of Demeter (object not created locally) 279
Potential violation of Law of Demeter (object not created locally) 280
Potential violation of Law of Demeter (object not created locally) 286
Potential violation of Law of Demeter (object not created locally) 286
Potential violation of Law of Demeter (object not created locally) 287
Potential violation of Law of Demeter (object not created locally) 290
Potential violation of Law of Demeter (object not created locally) 296
Potential violation of Law of Demeter (object not created locally) 296
Potential violation of Law of Demeter (object not created locally) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (object not created locally) 313
Potential violation of Law of Demeter (object not created locally) 315
Potential violation of Law of Demeter (method chain calls) 317
Potential violation of Law of Demeter (object not created locally) 322
Potential violation of Law of Demeter (method chain calls) 322
Potential violation of Law of Demeter (method chain calls) 322
Potential violation of Law of Demeter (method chain calls) 322
Potential violation of Law of Demeter (method chain calls) 322
Potential violation of Law of Demeter (object not created locally) 324
Potential violation of Law of Demeter (object not created locally) 331
Potential violation of Law of Demeter (object not created locally) 331
Potential violation of Law of Demeter (method chain calls) 332
Potential violation of Law of Demeter (method chain calls) 332
Potential violation of Law of Demeter (object not created locally) 339
Potential violation of Law of Demeter (object not created locally) 340
Potential violation of Law of Demeter (object not created locally) 341
Potential violation of Law of Demeter (object not created locally) 343
Potential violation of Law of Demeter (object not created locally) 343
Potential violation of Law of Demeter (object not created locally) 345
Potential violation of Law of Demeter (object not created locally) 346
Potential violation of Law of Demeter (object not created locally) 347
The method 'postProcessing' has a Cyclomatic Complexity of 15. 352 - 406
Potential violation of Law of Demeter (method chain calls) 354
Potential violation of Law of Demeter (object not created locally) 366
Potential violation of Law of Demeter (object not created locally) 368
Potential violation of Law of Demeter (object not created locally) 372
Potential violation of Law of Demeter (object not created locally) 373
Potential violation of Law of Demeter (method chain calls) 376
Potential violation of Law of Demeter (method chain calls) 376
Potential violation of Law of Demeter (method chain calls) 377
Potential violation of Law of Demeter (method chain calls) 377
Potential violation of Law of Demeter (method chain calls) 378
Potential violation of Law of Demeter (method chain calls) 378
Potential violation of Law of Demeter (method chain calls) 378
Potential violation of Law of Demeter (method chain calls) 379
Potential violation of Law of Demeter (method chain calls) 379
Potential violation of Law of Demeter (method chain calls) 379
Potential violation of Law of Demeter (object not created locally) 379
Potential violation of Law of Demeter (method chain calls) 379
Potential violation of Law of Demeter (object not created locally) 380
Potential violation of Law of Demeter (object not created locally) 384
Potential violation of Law of Demeter (object not created locally) 386
Potential violation of Law of Demeter (object not created locally) 390
Potential violation of Law of Demeter (object not created locally) 390
Potential violation of Law of Demeter (method chain calls) 391
Potential violation of Law of Demeter (method chain calls) 391
Potential violation of Law of Demeter (method chain calls) 397
Potential violation of Law of Demeter (method chain calls) 399
Useless parentheses. 435
Potential violation of Law of Demeter (method chain calls) 435
Potential violation of Law of Demeter (method chain calls) 435
Potential violation of Law of Demeter (method chain calls) 435
Potential violation of Law of Demeter (method chain calls) 435
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (method chain calls) 441
An empty method in an abstract class should be abstract instead 477 - 479
Potential violation of Law of Demeter (method chain calls) 483
Potential violation of Law of Demeter (method chain calls) 483
Potential violation of Law of Demeter (object not created locally) 483
Potential violation of Law of Demeter (method chain calls) 483
Avoid appending characters as strings in StringBuffer.append. 483
Potential violation of Law of Demeter (object not created locally) 484
Avoid appending characters as strings in StringBuffer.append. 485
Potential violation of Law of Demeter (object not created locally) 485
Potential violation of Law of Demeter (object not created locally) 487
Potential violation of Law of Demeter (object not created locally) 488
Potential violation of Law of Demeter (object not created locally) 492
Potential violation of Law of Demeter (method chain calls) 494
Potential violation of Law of Demeter (object not created locally) 494
Potential violation of Law of Demeter (object not created locally) 495
Avoid appending characters as strings in StringBuffer.append. 495
Potential violation of Law of Demeter (object not created locally) 497
Potential violation of Law of Demeter (object not created locally) 498
Potential violation of Law of Demeter (object not created locally) 502
Potential violation of Law of Demeter (object not created locally) 504
Potential violation of Law of Demeter (method chain calls) 504
Potential violation of Law of Demeter (object not created locally) 505
Avoid appending characters as strings in StringBuffer.append. 505
Potential violation of Law of Demeter (object not created locally) 508
Potential violation of Law of Demeter (object not created locally) 508
Potential violation of Law of Demeter (object not created locally) 509

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

Violation Line
These nested if statements could be combined 35 - 42
Potential violation of Law of Demeter (method chain calls) 38
Potential violation of Law of Demeter (object not created locally) 39
Potential violation of Law of Demeter (object not created locally) 40
Potential violation of Law of Demeter (object not created locally) 41

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUILabel.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. 24 - 26

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. 24 - 25

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

Violation Line
Avoid unused private fields such as 'LOG'. 34
Potential violation of Law of Demeter (method chain calls) 42
Potential violation of Law of Demeter (method chain calls) 42
Potential violation of Law of Demeter (method chain calls) 49
Potential violation of Law of Demeter (method chain calls) 49
Potential violation of Law of Demeter (method chain calls) 56
Potential violation of Law of Demeter (method chain calls) 56
Potential violation of Law of Demeter (object not created locally) 57
Potential violation of Law of Demeter (object not created locally) 61

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIMenu.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. 24 - 25

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 36

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

Violation Line
Avoid if (x != y) ..; else ..; 41 - 45
Potential violation of Law of Demeter (static property access) 50
Avoid declaring a variable if it is unreferenced before a possible exit point. 63
Potential violation of Law of Demeter (method chain calls) 67
Potential violation of Law of Demeter (method chain calls) 67
Potential violation of Law of Demeter (object not created locally) 67
Potential violation of Law of Demeter (method chain calls) 70
Potential violation of Law of Demeter (object not created locally) 70
Potential violation of Law of Demeter (method chain calls) 70
Avoid instantiating new objects inside loops 73
Potential violation of Law of Demeter (method chain calls) 108
Potential violation of Law of Demeter (method chain calls) 108
Potential violation of Law of Demeter (method chain calls) 108
Potential violation of Law of Demeter (method chain calls) 108
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 133

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. 26 - 27

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

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 20 - 390
Possible God class (WMC=66, ATFD=32, TCC=0.014245014245014245) 67 - 390
The class 'AbstractUIPage' has a Cyclomatic Complexity of 3 (Highest = 10). 67 - 390
This class has too many methods, consider refactoring it. 68 - 390
An empty method in an abstract class should be abstract instead 87 - 89
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (method chain calls) 111
Potential violation of Law of Demeter (method chain calls) 118
Potential violation of Law of Demeter (object not created locally) 134
Avoid if (x != y) ..; else ..; 144 - 162
Potential violation of Law of Demeter (object not created locally) 147
Potential violation of Law of Demeter (object not created locally) 148
Potential violation of Law of Demeter (object not created locally) 156
The method 'decodeActionComponent' has a Cyclomatic Complexity of 10. 165 - 195
Potential violation of Law of Demeter (object not created locally) 173
Avoid empty catch blocks 177 - 179
Use equals() to compare object references. 188
Potential violation of Law of Demeter (object not created locally) 191
Avoid if (x != y) ..; else ..; 205 - 212
Potential violation of Law of Demeter (object not created locally) 206
Avoid if (x != y) ..; else ..; 221 - 227
Potential violation of Law of Demeter (object not created locally) 222
The method 'markSubmittedForm' has a Cyclomatic Complexity of 10. 230 - 281
Potential violation of Law of Demeter (object not created locally) 243
Potential violation of Law of Demeter (object not created locally) 247
Potential violation of Law of Demeter (object not created locally) 252
Potential violation of Law of Demeter (object not created locally) 254
Avoid empty catch blocks 256 - 258
Avoid if (x != y) ..; else ..; 267 - 280
Potential violation of Law of Demeter (object not created locally) 269
Potential violation of Law of Demeter (object not created locally) 273
Potential violation of Law of Demeter (method chain calls) 285
Potential violation of Law of Demeter (method chain calls) 285
Potential violation of Law of Demeter (object not created locally) 286
Potential violation of Law of Demeter (method chain calls) 287
Potential violation of Law of Demeter (method chain calls) 292
An empty method in an abstract class should be abstract instead 311 - 312
Document empty method 311 - 312
Avoid if (x != y) ..; else ..; 320 - 330
Potential violation of Law of Demeter (static property access) 353
Potential violation of Law of Demeter (static property access) 361
Potential violation of Law of Demeter (method chain calls) 380

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 44
Potential violation of Law of Demeter (method chain calls) 50
Potential violation of Law of Demeter (method chain calls) 56
Potential violation of Law of Demeter (method chain calls) 76

Tobago Core - org/apache/myfaces/tobago/internal/component/AbstractUIPanelBase.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. 22 - 25

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

Violation Line
Avoid unused private fields such as 'LOG'. 54
Potential violation of Law of Demeter (method chain calls) 61
Potential violation of Law of Demeter (method chain calls) 61
Potential violation of Law of Demeter (method chain calls) 68
Potential violation of Law of Demeter (method chain calls) 68
Potential violation of Law of Demeter (object not created locally) 92
Avoid if (x != y) ..; else ..; 109 - 114
Potential violation of Law of Demeter (object not created locally) 111
Potential violation of Law of Demeter (object not created locally) 111
Potential violation of Law of Demeter (object not created locally) 120 - 121
Potential violation of Law of Demeter (method chain calls) 128 - 129
Potential violation of Law of Demeter (method chain calls) 128 - 129
No need to check for null before an instanceof 130
Potential violation of Law of Demeter (method chain calls) 131
Potential violation of Law of Demeter (object not created locally) 131
Potential violation of Law of Demeter (object not created locally) 148
Potential violation of Law of Demeter (object not created locally) 162
Potential violation of Law of Demeter (method chain calls) 193

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 41
These nested if statements could be combined 47 - 54

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

Violation Line
Avoid empty if statements 49 - 51
Potential violation of Law of Demeter (method chain calls) 52 - 53
Potential violation of Law of Demeter (method chain calls) 53

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 34

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 38
Avoid if (x != y) ..; else ..; 39 - 46

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. 22 - 23

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. 22 - 23

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. 22 - 23

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. 26 - 27

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

Violation Line
This class has a bunch of public methods and attributes 20 - 673
A high number of imports can indicate a high degree of coupling within an object. 20 - 673
The class 'AbstractUISheet' has a Cyclomatic Complexity of 3 (Highest = 19). 65 - 673
Possible God class (WMC=138, ATFD=57, TCC=0.013062409288824383) 65 - 673
This class has too many methods, consider refactoring it. 67 - 673
Potential violation of Law of Demeter (object not created locally) 106
Avoid if (x != y) ..; else ..; 109 - 113
Potential violation of Law of Demeter (object not created locally) 110
Potential violation of Law of Demeter (object not created locally) 165
Potential violation of Law of Demeter (method chain calls) 167
Useless parentheses. 209
Avoid if (x != y) ..; else ..; 276 - 280
Avoid if (x != y) ..; else ..; 286 - 294
Avoid if (x != y) ..; else ..; 320 - 328
Avoid if (x != y) ..; else ..; 327
Potential violation of Law of Demeter (object not created locally) 346
Potential violation of Law of Demeter (method chain calls) 347
Potential violation of Law of Demeter (object not created locally) 347
Potential violation of Law of Demeter (method chain calls) 347
Avoid if (x != y) ..; else ..; 347
Potential violation of Law of Demeter (object not created locally) 348
Potential violation of Law of Demeter (object not created locally) 348
Potential violation of Law of Demeter (object not created locally) 349
Potential violation of Law of Demeter (object not created locally) 349
Potential violation of Law of Demeter (object not created locally) 350
Potential violation of Law of Demeter (object not created locally) 351
Local variable 'result' could be declared final 372
Local variable 'result' could be declared final 378
Avoid empty if statements 388 - 390
Potential violation of Law of Demeter (object not created locally) 416
Potential violation of Law of Demeter (object not created locally) 419
Potential violation of Law of Demeter (object not created locally) 420
Potential violation of Law of Demeter (object not created locally) 421
Potential violation of Law of Demeter (method chain calls) 421
Potential violation of Law of Demeter (object not created locally) 421
Potential violation of Law of Demeter (method chain calls) 435
Potential violation of Law of Demeter (method chain calls) 440
Avoid if (x != y) ..; else ..; 446 - 453
Potential violation of Law of Demeter (method chain calls) 448
Potential violation of Law of Demeter (method chain calls) 449
Potential violation of Law of Demeter (method chain calls) 451
Potential violation of Law of Demeter (method chain calls) 480
Potential violation of Law of Demeter (method chain calls) 482
Avoid reassigning parameters such as 'searchId' 493
The method 'performPaging' has a Cyclomatic Complexity of 19. 509 - 571
Potential violation of Law of Demeter (method chain calls) 514
Potential violation of Law of Demeter (method chain calls) 514
Avoid if (x != y) ..; else ..; 563 - 567
Potential violation of Law of Demeter (object not created locally) 564
Potential violation of Law of Demeter (method chain calls) 564
Potential violation of Law of Demeter (method chain calls) 564
Potential violation of Law of Demeter (method chain calls) 569
The method 'getComponents' has a Cyclomatic Complexity of 11. 573 - 615
Avoid empty if statements 581 - 583
Potential violation of Law of Demeter (object not created locally) 587
Potential violation of Law of Demeter (object not created locally) 594
Potential violation of Law of Demeter (object not created locally) 595
Avoid if (x != y) ..; else ..; 599 - 611
Potential violation of Law of Demeter (method chain calls) 603 - 604
Potential violation of Law of Demeter (method chain calls) 604
Potential violation of Law of Demeter (object not created locally) 604
Potential violation of Law of Demeter (object not created locally) 605
Potential violation of Law of Demeter (object not created locally) 606
Potential violation of Law of Demeter (method chain calls) 606
Potential violation of Law of Demeter (object not created locally) 610
Potential violation of Law of Demeter (object not created locally) 610
Potential violation of Law of Demeter (method chain calls) 622
Potential violation of Law of Demeter (method chain calls) 635
An empty method in an abstract class should be abstract instead 642 - 644
Potential violation of Law of Demeter (method chain calls) 658
Potential violation of Law of Demeter (method chain calls) 663

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

Violation Line
The class 'AbstractUISheetLayout' has a Cyclomatic Complexity of 6 (Highest = 16). 54 - 412
Possible God class (WMC=84, ATFD=91, TCC=0.01098901098901099) 54 - 412
Potential violation of Law of Demeter (method chain calls) 65
Potential violation of Law of Demeter (method chain calls) 67
Potential violation of Law of Demeter (method chain calls) 67
Potential violation of Law of Demeter (method chain calls) 80
Potential violation of Law of Demeter (method chain calls) 82
Potential violation of Law of Demeter (method chain calls) 82
Avoid unused local variables such as 'intervals'. 90
Potential violation of Law of Demeter (method chain calls) 91
These nested if statements could be combined 94 - 96
Potential violation of Law of Demeter (method chain calls) 95
Potential violation of Law of Demeter (method chain calls) 95
The method 'mainProcessing' has a Cyclomatic Complexity of 12. 118 - 168
Potential violation of Law of Demeter (object not created locally) 132
Potential violation of Law of Demeter (object not created locally) 135
Potential violation of Law of Demeter (method chain calls) 143
Potential violation of Law of Demeter (object not created locally) 144
Potential violation of Law of Demeter (object not created locally) 150
Potential violation of Law of Demeter (object not created locally) 151
Potential violation of Law of Demeter (object not created locally) 152
Potential violation of Law of Demeter (object not created locally) 153
Potential violation of Law of Demeter (object not created locally) 154
Potential violation of Law of Demeter (object not created locally) 155
Potential violation of Law of Demeter (object not created locally) 156
Potential violation of Law of Demeter (object not created locally) 156
Potential violation of Law of Demeter (object not created locally) 158
Deeply nested if..then statements are hard to read 160 - 162
Potential violation of Law of Demeter (method chain calls) 161
Potential violation of Law of Demeter (method chain calls) 161
Potential violation of Law of Demeter (object not created locally) 176
Potential violation of Law of Demeter (method chain calls) 189
Potential violation of Law of Demeter (method chain calls) 189
An empty method in an abstract class should be abstract instead 203 - 205
The method 'ensureColumnWidthList' has a Cyclomatic Complexity of 16. 207 - 284
The method ensureColumnWidthList() has an NPath complexity of 1080 207 - 284
Potential violation of Law of Demeter (object not created locally) 216
Potential violation of Law of Demeter (object not created locally) 219
Potential violation of Law of Demeter (object not created locally) 229
Potential violation of Law of Demeter (object not created locally) 229
Potential violation of Law of Demeter (object not created locally) 237
Potential violation of Law of Demeter (object not created locally) 238
Potential violation of Law of Demeter (object not created locally) 239
Potential violation of Law of Demeter (object not created locally) 241
Potential violation of Law of Demeter (method chain calls) 242
Potential violation of Law of Demeter (object not created locally) 242
Potential violation of Law of Demeter (object not created locally) 242
Potential violation of Law of Demeter (method chain calls) 242
Potential violation of Law of Demeter (method chain calls) 242
Potential violation of Law of Demeter (object not created locally) 244
Potential violation of Law of Demeter (object not created locally) 247
Potential violation of Law of Demeter (object not created locally) 248
Potential violation of Law of Demeter (object not created locally) 248
Potential violation of Law of Demeter (object not created locally) 250
Potential violation of Law of Demeter (object not created locally) 258
Potential violation of Law of Demeter (object not created locally) 258
Potential violation of Law of Demeter (object not created locally) 259
Potential violation of Law of Demeter (object not created locally) 259
Potential violation of Law of Demeter (object not created locally) 261
Potential violation of Law of Demeter (object not created locally) 261
Potential violation of Law of Demeter (object not created locally) 268
Potential violation of Law of Demeter (object not created locally) 268
Potential violation of Law of Demeter (method chain calls) 270
Potential violation of Law of Demeter (object not created locally) 272
Potential violation of Law of Demeter (object not created locally) 272
Potential violation of Law of Demeter (object not created locally) 273
Potential violation of Law of Demeter (object not created locally) 274
Avoid if (x != y) ..; else ..; 277 - 283
Potential violation of Law of Demeter (object not created locally) 277
Potential violation of Law of Demeter (object not created locally) 277
Potential violation of Law of Demeter (object not created locally) 278
Potential violation of Law of Demeter (object not created locally) 279
The method needVerticalScrollbar() has an NPath complexity of 660 286 - 331
The method 'needVerticalScrollbar' has a Cyclomatic Complexity of 13. 286 - 331
Potential violation of Law of Demeter (method chain calls) 296
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 300
Potential violation of Law of Demeter (method chain calls) 302
Avoid if (x != y) ..; else ..; 312 - 327
Potential violation of Law of Demeter (method chain calls) 317
Potential violation of Law of Demeter (object not created locally) 319
Potential violation of Law of Demeter (object not created locally) 322
Potential violation of Law of Demeter (object not created locally) 324
Potential violation of Law of Demeter (object not created locally) 336
Potential violation of Law of Demeter (object not created locally) 337
Potential violation of Law of Demeter (object not created locally) 346
Avoid unused local variables such as 'component'. 346
Potential violation of Law of Demeter (method chain calls) 368
Potential violation of Law of Demeter (method chain calls) 373
Potential violation of Law of Demeter (method chain calls) 378
Potential violation of Law of Demeter (object not created locally) 384
Potential violation of Law of Demeter (object not created locally) 390
Avoid unused local variables such as 'ignored'. 391
Potential violation of Law of Demeter (object not created locally) 398
Potential violation of Law of Demeter (object not created locally) 402
Potential violation of Law of Demeter (object not created locally) 402
Avoid instantiating new objects inside loops 402
Potential violation of Law of Demeter (object not created locally) 410

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

Violation Line
Possible God class (WMC=62, ATFD=19, TCC=0.0) 51 - 324
This class has too many methods, consider refactoring it. 53 - 324
Potential violation of Law of Demeter (method chain calls) 68
An empty method in an abstract class should be abstract instead 72 - 75
Potential violation of Law of Demeter (method chain calls) 80
An empty method in an abstract class should be abstract instead 89 - 91
Potential violation of Law of Demeter (method chain calls) 107
Potential violation of Law of Demeter (method chain calls) 108
Avoid if (x != y) ..; else ..; 133 - 154
Potential violation of Law of Demeter (object not created locally) 142
Potential violation of Law of Demeter (method chain calls) 143
Avoid if (x != y) ..; else ..; 159 - 173
Potential violation of Law of Demeter (object not created locally) 167
Potential violation of Law of Demeter (method chain calls) 168
Avoid if (x != y) ..; else ..; 178 - 193
Potential violation of Law of Demeter (object not created locally) 186
Potential violation of Law of Demeter (method chain calls) 187
Potential violation of Law of Demeter (method chain calls) 214
Potential violation of Law of Demeter (method chain calls) 214
Potential violation of Law of Demeter (object not created locally) 216
Potential violation of Law of Demeter (object not created locally) 219
Avoid if (x != y) ..; else ..; 221 - 225
Potential violation of Law of Demeter (object not created locally) 222
Potential violation of Law of Demeter (method chain calls) 222
Potential violation of Law of Demeter (method chain calls) 222
Useless parentheses. 238
Potential violation of Law of Demeter (object not created locally) 238
Potential violation of Law of Demeter (method chain calls) 317

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

Violation Line
The class 'AbstractUITabGroupLayout' has a Cyclomatic Complexity of 5 (Highest = 10). 33 - 137
Potential violation of Law of Demeter (method chain calls) 41
Potential violation of Law of Demeter (method chain calls) 43
Potential violation of Law of Demeter (method chain calls) 43
Potential violation of Law of Demeter (method chain calls) 56
Potential violation of Law of Demeter (method chain calls) 58
Potential violation of Law of Demeter (method chain calls) 58
Potential violation of Law of Demeter (method chain calls) 67
Potential violation of Law of Demeter (method chain calls) 70
Potential violation of Law of Demeter (method chain calls) 70
Avoid instantiating new objects inside loops 75
The method 'mainProcessing' has a Cyclomatic Complexity of 10. 90 - 113
Avoid if (x != y) ..; else ..; 98 - 104
Potential violation of Law of Demeter (method chain calls) 99
Potential violation of Law of Demeter (method chain calls) 108
Potential violation of Law of Demeter (method chain calls) 110
Potential violation of Law of Demeter (method chain calls) 110
Potential violation of Law of Demeter (method chain calls) 118
Potential violation of Law of Demeter (method chain calls) 121
Potential violation of Law of Demeter (method chain calls) 121
An empty method in an abstract class should be abstract instead 134 - 136

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

Violation Line
An empty method in an abstract class should be abstract instead 25 - 27

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 36
Potential violation of Law of Demeter (object not created locally) 38
Potential violation of Law of Demeter (object not created locally) 39
Potential violation of Law of Demeter (object not created locally) 40
An empty method in an abstract class should be abstract instead 47 - 49

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. 24 - 38

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. 24 - 27

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

Violation Line
Potential violation of Law of Demeter (static property access) 114
An empty method in an abstract class should be abstract instead 119 - 121
Potential violation of Law of Demeter (method chain calls) 209
Potential violation of Law of Demeter (method chain calls) 227
Potential violation of Law of Demeter (method chain calls) 232

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

Violation Line
Potential violation of Law of Demeter (static property access) 47
Avoid if (x != y) ..; else ..; 50 - 54
Potential violation of Law of Demeter (method chain calls) 53
Avoid if (x != y) ..; else ..; 59 - 63
Potential violation of Law of Demeter (method chain calls) 62
Potential violation of Law of Demeter (static property access) 93
Potential violation of Law of Demeter (static property access) 101
Potential violation of Law of Demeter (static property access) 109

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. 26 - 27

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

Violation Line
An empty method in an abstract class should be abstract instead 25 - 27

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 45
Potential violation of Law of Demeter (object not created locally) 46
Potential violation of Law of Demeter (method chain calls) 56
Potential violation of Law of Demeter (object not created locally) 57
Potential violation of Law of Demeter (method chain calls) 61
Potential violation of Law of Demeter (object not created locally) 62
Potential violation of Law of Demeter (method chain calls) 66
Potential violation of Law of Demeter (object not created locally) 67
Potential violation of Law of Demeter (method chain calls) 71
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (object not created locally) 78
Potential violation of Law of Demeter (object not created locally) 79
Potential violation of Law of Demeter (object not created locally) 79
Potential violation of Law of Demeter (object not created locally) 91
Consider simply returning the value vs storing it in local variable 'rowData' 92
Potential violation of Law of Demeter (static property access) 100
The String literal "Doesn't work anymore." appears 10 times in this file; the first occurrence is on line 100 100
Potential violation of Law of Demeter (static property access) 108
Potential violation of Law of Demeter (static property access) 116
Potential violation of Law of Demeter (static property access) 124
Potential violation of Law of Demeter (static property access) 133
Potential violation of Law of Demeter (static property access) 141
Potential violation of Law of Demeter (static property access) 149
Potential violation of Law of Demeter (static property access) 157
Potential violation of Law of Demeter (static property access) 165
Potential violation of Law of Demeter (static property access) 174
Potential violation of Law of Demeter (static property access) 182
The String literal "treeExpansionListener!" appears 5 times in this file; the first occurrence is on line 182 182
Potential violation of Law of Demeter (static property access) 191
Potential violation of Law of Demeter (static property access) 199
Return an empty array rather than null. 206 - 209
Potential violation of Law of Demeter (static property access) 207
Potential violation of Law of Demeter (static property access) 216
The String literal "treeMarkedListener!" appears 5 times in this file; the first occurrence is on line 224 224
Potential violation of Law of Demeter (static property access) 224
Potential violation of Law of Demeter (static property access) 233
Potential violation of Law of Demeter (static property access) 241
Return an empty array rather than null. 248 - 251
Potential violation of Law of Demeter (static property access) 249
Potential violation of Law of Demeter (static property access) 258
Potential violation of Law of Demeter (static property access) 266
Potential violation of Law of Demeter (static property access) 275
Potential violation of Law of Demeter (static property access) 283
Potential violation of Law of Demeter (static property access) 292
Potential violation of Law of Demeter (static property access) 300
Potential violation of Law of Demeter (static property access) 309

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

Violation Line
Overriding method merely calls super 32 - 34

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 49
Potential violation of Law of Demeter (method chain calls) 49
Potential violation of Law of Demeter (method chain calls) 58
Potential violation of Law of Demeter (method chain calls) 58
Potential violation of Law of Demeter (method chain calls) 72
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (method chain calls) 72
Potential violation of Law of Demeter (method chain calls) 74
Potential violation of Law of Demeter (object not created locally) 75
Potential violation of Law of Demeter (method chain calls) 75
Potential violation of Law of Demeter (object not created locally) 77
Potential violation of Law of Demeter (method chain calls) 77
Potential violation of Law of Demeter (method chain calls) 77
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (method chain calls) 96
Potential violation of Law of Demeter (method chain calls) 96
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (method chain calls) 104
Potential violation of Law of Demeter (method chain calls) 104
Potential violation of Law of Demeter (method chain calls) 107
Potential violation of Law of Demeter (method chain calls) 107
Avoid if (x != y) ..; else ..; 133 - 137
Potential violation of Law of Demeter (object not created locally) 134
Unnecessary use of fully qualified name 'javax.faces.el.ValueBinding' due to existing import 'javax.faces.el.ValueBinding' 156
Potential violation of Law of Demeter (object not created locally) 158
Unnecessary use of fully qualified name 'javax.faces.el.ValueBinding' due to existing import 'javax.faces.el.ValueBinding' 172
Potential violation of Law of Demeter (object not created locally) 174
Unnecessary use of fully qualified name 'javax.faces.el.ValueBinding' due to existing import 'javax.faces.el.ValueBinding' 188
Potential violation of Law of Demeter (object not created locally) 190

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

Violation Line
Potential violation of Law of Demeter (static property access) 64
Potential violation of Law of Demeter (static property access) 66
Potential violation of Law of Demeter (static property access) 68

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

Violation Line
Private field 'rendererMap' could be made final; it is only initialized in the declaration or constructor. 38
Avoid using redundant field initializer for 'merged' 40
Potential violation of Law of Demeter (method chain calls) 59
Avoid if (x != y) ..; else ..; 70 - 75
Potential violation of Law of Demeter (object not created locally) 71

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'. 70
Potential violation of Law of Demeter (method chain calls) 95
Potential violation of Law of Demeter (object not created locally) 97
Avoid instantiating new objects inside loops 101
Potential violation of Law of Demeter (method chain calls) 109
Potential violation of Law of Demeter (method chain calls) 109
Potential violation of Law of Demeter (method chain calls) 109
Potential violation of Law of Demeter (method chain calls) 113

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

Violation Line
Position literals first in String comparisons 50
Potential violation of Law of Demeter (method chain calls) 51
Position literals first in String comparisons 52
Potential violation of Law of Demeter (method chain calls) 53
Position literals first in String comparisons 54
Potential violation of Law of Demeter (method chain calls) 55
Position literals first in String comparisons 56
Potential violation of Law of Demeter (method chain calls) 57
Avoid if (x != y) ..; else ..; 61 - 67

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

Violation Line
Private field 'before' could be made final; it is only initialized in the declaration or constructor. 36
Private field 'after' could be made final; it is only initialized in the declaration or constructor. 37
Private field 'supportedThemeNames' could be made final; it is only initialized in the declaration or constructor. 39
Private field 'resourceDirs' could be made final; it is only initialized in the declaration or constructor. 41
Private field 'themeDefinitions' could be made final; it is only initialized in the declaration or constructor. 47
Potential violation of Law of Demeter (static property access) 90
Potential violation of Law of Demeter (static property access) 168
Potential violation of Law of Demeter (static property access) 174
Avoid if (x != y) ..; else ..; 187

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

Violation Line
The class 'TobagoConfigImpl' has a Cyclomatic Complexity of 2 (Highest = 10). 44 - 325
Possible God class (WMC=56, ATFD=25, TCC=0.06666666666666667) 44 - 325
Private field 'supportedThemes' could be made final; it is only initialized in the declaration or constructor. 50
Private field 'supportedThemeNames' could be made final; it is only initialized in the declaration or constructor. 51
Private field 'resourceDirs' could be made final; it is only initialized in the declaration or constructor. 54
Private field 'contentSecurityPolicy' could be made final; it is only initialized in the declaration or constructor. 61
The method 'resolveThemes' has a Cyclomatic Complexity of 10. 79 - 119
Avoid if (x != y) ..; else ..; 80 - 107
Potential violation of Law of Demeter (object not created locally) 90
Potential violation of Law of Demeter (object not created locally) 91
Potential violation of Law of Demeter (method chain calls) 91
Potential violation of Law of Demeter (method chain calls) 93
Potential violation of Law of Demeter (object not created locally) 93
Potential violation of Law of Demeter (method chain calls) 136
Potential violation of Law of Demeter (method chain calls) 140
Potential violation of Law of Demeter (static property access) 172
The method 'initProjectState' has a Cyclomatic Complexity of 10. 199 - 237
Avoid empty catch blocks 212 - 214
Use block level rather than method level synchronization 239 - 255
Potential violation of Law of Demeter (object not created locally) 243
Potential violation of Law of Demeter (object not created locally) 244
Substitute calls to size() == 0 (or size() != 0) with calls to isEmpty() 245
Potential violation of Law of Demeter (object not created locally) 245
StringBuffer constructor is initialized with size 16, but has at least 177 characters appended. 297
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 298

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

Violation Line
Possible God class (WMC=83, ATFD=32, TCC=0.05454545454545454) 47 - 387
The class 'TobagoConfigParser' has a Cyclomatic Complexity of 4 (Highest = 23). 47 - 387
This class has too many methods, consider refactoring it. 47 - 387
StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 83
Document empty constructor 87 - 88
Potential violation of Law of Demeter (object not created locally) 101
Potential violation of Law of Demeter (object not created locally) 109
Potential violation of Law of Demeter (object not created locally) 110
Potential violation of Law of Demeter (object not created locally) 112
Potential violation of Law of Demeter (object not created locally) 113
The method 'startElement' has a Cyclomatic Complexity of 13. 139 - 224
This is an inefficient use of StringBuffer.toString; call StringBuffer.length instead. 143
Potential violation of Law of Demeter (method chain calls) 143
Potential violation of Law of Demeter (method chain calls) 143
Avoid if (x != y) ..; else ..; 160 - 164
Potential violation of Law of Demeter (method chain calls) 184
Potential violation of Law of Demeter (method chain calls) 186
Potential violation of Law of Demeter (method chain calls) 194
Potential violation of Law of Demeter (method chain calls) 196
Avoid really long methods. 232 - 349
The method 'endElement' has a Cyclomatic Complexity of 23. 232 - 349
Potential violation of Law of Demeter (method chain calls) 235
Potential violation of Law of Demeter (object not created locally) 242
Avoid if (x != y) ..; else ..; 258 - 262
Potential violation of Law of Demeter (method chain calls) 259
Potential violation of Law of Demeter (method chain calls) 261
Potential violation of Law of Demeter (method chain calls) 299
Potential violation of Law of Demeter (method chain calls) 299
Potential violation of Law of Demeter (method chain calls) 371
Potential violation of Law of Demeter (method chain calls) 372
Potential violation of Law of Demeter (object not created locally) 372
Potential violation of Law of Demeter (method chain calls) 372
Potential violation of Law of Demeter (method chain calls) 373
Potential violation of Law of Demeter (object not created locally) 375
Potential violation of Law of Demeter (method chain calls) 375
Potential violation of Law of Demeter (method chain calls) 375
Potential violation of Law of Demeter (method chain calls) 376
Potential violation of Law of Demeter (object not created locally) 377
Potential violation of Law of Demeter (method chain calls) 377
Potential violation of Law of Demeter (method chain calls) 377
Potential violation of Law of Demeter (object not created locally) 381
Potential violation of Law of Demeter (object not created locally) 384

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

Violation Line
The class 'TobagoConfigSorter' has a Cyclomatic Complexity of 4 (Highest = 12). 30 - 278
Private field 'list' could be made final; it is only initialized in the declaration or constructor. 34
Potential violation of Law of Demeter (object not created locally) 56
Prefer StringBuffer over += for concatenating strings 60
Potential violation of Law of Demeter (object not created locally) 62
The method merge() has an NPath complexity of 513 67 - 126
The method 'merge' has a Cyclomatic Complexity of 12. 67 - 126
Potential violation of Law of Demeter (method chain calls) 91
Potential violation of Law of Demeter (method chain calls) 110
Potential violation of Law of Demeter (method chain calls) 135
Potential violation of Law of Demeter (method chain calls) 135
Potential violation of Law of Demeter (method chain calls) 136
Potential violation of Law of Demeter (method chain calls) 136
Potential violation of Law of Demeter (method chain calls) 136
Potential violation of Law of Demeter (method chain calls) 136
Potential violation of Law of Demeter (method chain calls) 137
Potential violation of Law of Demeter (method chain calls) 137
Avoid instantiating new objects inside loops 137
Avoid instantiation through private constructors from outside of the constructors class. 137
Avoid instantiating new objects inside loops 148
The String literal "' < '" appears 4 times in this file; the first occurrence is on line 151 151
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 153
Avoid appending characters as strings in StringBuffer.append. 156
Avoid instantiating new objects inside loops 172
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 181
Avoid appending characters as strings in StringBuffer.append. 184
Avoid instantiating new objects inside loops 216
Avoid instantiation through private constructors from outside of the constructors class. 216
Avoid instantiating new objects inside loops 222
Avoid instantiation through private constructors from outside of the constructors class. 222

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 56

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

Violation Line
The class 'ClientPropertiesKey' has a Cyclomatic Complexity of 2 (Highest = 10). 33 - 136
Potential violation of Law of Demeter (object not created locally) 57
The method 'equals' has a Cyclomatic Complexity of 10. 69 - 99

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
Useless parentheses. 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) 61 - 454
This class has too many methods, consider refactoring it. 61 - 454
The class 'ResourceLocator' has a Cyclomatic Complexity of 6 (Highest = 14). 61 - 454
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
Private field 'themeBuilder' could be made final; it is only initialized in the declaration or constructor. 71
The method locateResourcesInWar() has an NPath complexity of 240 88 - 139
The method 'locateResourcesInWar' has a Cyclomatic Complexity of 14. 88 - 139
Avoid reassigning parameters such as 'path' 89
Potential violation of Law of Demeter (object not created locally) 100
Potential violation of Law of Demeter (method chain calls) 148
Potential violation of Law of Demeter (object not created locally) 149
Potential violation of Law of Demeter (object not created locally) 151
Potential violation of Law of Demeter (object not created locally) 152
Avoid instantiating new objects inside loops 153
Potential violation of Law of Demeter (object not created locally) 158
Potential violation of Law of Demeter (method chain calls) 160
Potential violation of Law of Demeter (method chain calls) 160
Potential violation of Law of Demeter (method chain calls) 160
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 167
Potential violation of Law of Demeter (method chain calls) 188
Potential violation of Law of Demeter (object not created locally) 189
Potential violation of Law of Demeter (object not created locally) 191
Potential violation of Law of Demeter (object not created locally) 192
Potential violation of Law of Demeter (method chain calls) 195
Potential violation of Law of Demeter (object not created locally) 195
Potential violation of Law of Demeter (object not created locally) 202
Potential violation of Law of Demeter (method chain calls) 206
Potential violation of Law of Demeter (method chain calls) 206
Potential violation of Law of Demeter (method chain calls) 206
Potential violation of Law of Demeter (object not created locally) 211
Potential violation of Law of Demeter (object not created locally) 212
Potential violation of Law of Demeter (object not created locally) 230
Potential violation of Law of Demeter (object not created locally) 230
Avoid if (x != y) ..; else ..; 240 - 244
Potential violation of Law of Demeter (object not created locally) 241
Potential violation of Law of Demeter (method chain calls) 254
Potential violation of Law of Demeter (method chain calls) 254
Potential violation of Law of Demeter (method chain calls) 254
Potential violation of Law of Demeter (method chain calls) 255
Potential violation of Law of Demeter (object not created locally) 256
Potential violation of Law of Demeter (object not created locally) 258
Potential violation of Law of Demeter (object not created locally) 259
Potential violation of Law of Demeter (object not created locally) 260
Potential violation of Law of Demeter (method chain calls) 260
Potential violation of Law of Demeter (method chain calls) 261
Potential violation of Law of Demeter (object not created locally) 261
Potential violation of Law of Demeter (object not created locally) 274
The method addResourcesFromZip() has an NPath complexity of 288 279 - 334
The method 'addResourcesFromZip' has a Cyclomatic Complexity of 12. 279 - 334
Avoid reassigning parameters such as 'fileName' 280
Avoid reassigning parameters such as 'fileName' 280
Avoid reassigning parameters such as 'fileName' 280
String.indexOf(char) is faster than String.indexOf(String). 284
Do not add empty strings 289
Position literals first in String comparisons 293
Position literals first in String comparisons 302
Potential violation of Law of Demeter (object not created locally) 319
Potential violation of Law of Demeter (object not created locally) 321
Potential violation of Law of Demeter (object not created locally) 321
Potential violation of Law of Demeter (object not created locally) 322
Potential violation of Law of Demeter (object not created locally) 325
Potential violation of Law of Demeter (object not created locally) 326
Avoid empty if statements 339 - 341
Potential violation of Law of Demeter (method chain calls) 346
Potential violation of Law of Demeter (method chain calls) 346
Potential violation of Law of Demeter (method chain calls) 357
Potential violation of Law of Demeter (method chain calls) 357
Avoid reassigning parameters such as 'resourcePath' 368
Avoid reassigning parameters such as 'resourcePath' 368
This call to String.startsWith can be rewritten using String.charAt(0) 369
Avoid declaring a variable if it is unreferenced before a possible exit point. 383
Potential violation of Law of Demeter (object not created locally) 386
Potential violation of Law of Demeter (object not created locally) 388
Potential violation of Law of Demeter (object not created locally) 390
Avoid declaring a variable if it is unreferenced before a possible exit point. 390
Potential violation of Law of Demeter (object not created locally) 416
Potential violation of Law of Demeter (object not created locally) 417
Potential violation of Law of Demeter (object not created locally) 428
Potential violation of Law of Demeter (object not created locally) 429
Potential violation of Law of Demeter (object not created locally) 435
Potential violation of Law of Demeter (object not created locally) 436
Potential violation of Law of Demeter (object not created locally) 437
Avoid if (x != y) ..; else ..; 444 - 450

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
Potential violation of Law of Demeter (method chain calls) 41 - 42
Potential violation of Law of Demeter (method chain calls) 41 - 42
Potential violation of Law of Demeter (object not created locally) 58
Potential violation of Law of Demeter (object not created locally) 59

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

Violation Line
The class 'ResourceManagerImpl' has a Cyclomatic Complexity of 5 (Highest = 30). 43 - 603
Possible God class (WMC=122, ATFD=73, TCC=0.0380952380952381) 43 - 603
Private field 'production' could be made final; it is only initialized in the declaration or constructor. 50
Private field 'tobagoConfig' could be made final; it is only initialized in the declaration or constructor. 68
Potential violation of Law of Demeter (method chain calls) 102 - 103
Potential violation of Law of Demeter (object not created locally) 106
The String literal "' clientProperties = '" appears 4 times in this file; the first occurrence is on line 106 106
The String literal "name = '" appears 4 times in this file; the first occurrence is on line 106 106
Avoid if (x != y) ..; else ..; 127 - 131
Potential violation of Law of Demeter (object not created locally) 128
Potential violation of Law of Demeter (object not created locally) 134
Potential violation of Law of Demeter (object not created locally) 160
Avoid if (x != y) ..; else ..; 160 - 166
Potential violation of Law of Demeter (object not created locally) 161
Potential violation of Law of Demeter (object not created locally) 162
Potential violation of Law of Demeter (object not created locally) 168
Potential violation of Law of Demeter (object not created locally) 170
Avoid if (x != y) ..; else ..; 205 - 209
Potential violation of Law of Demeter (object not created locally) 206
Potential violation of Law of Demeter (object not created locally) 212
Potential violation of Law of Demeter (object not created locally) 235
Potential violation of Law of Demeter (object not created locally) 243
Potential violation of Law of Demeter (object not created locally) 244
Avoid if (x != y) ..; else ..; 249 - 253
Potential violation of Law of Demeter (object not created locally) 256
Avoid if (x != y) ..; else ..; 287 - 291
Potential violation of Law of Demeter (object not created locally) 288
Potential violation of Law of Demeter (object not created locally) 295
These nested if statements could be combined 295 - 297
Potential violation of Law of Demeter (object not created locally) 300
The method getPaths() has an NPath complexity of 2100 306 - 391
The method 'getPaths' has a Cyclomatic Complexity of 30. 306 - 391
Avoid long parameter lists. 306 - 309
Avoid declaring a variable if it is unreferenced before a possible exit point. 311
Avoid declaring a variable if it is unreferenced before a possible exit point. 312
Avoid declaring a variable if it is unreferenced before a possible exit point. 313
Potential violation of Law of Demeter (object not created locally) 342
Potential violation of Law of Demeter (object not created locally) 344
Potential violation of Law of Demeter (object not created locally) 374
Potential violation of Law of Demeter (object not created locally) 379
The method 'checkPath' has a Cyclomatic Complexity of 11. 393 - 435
Avoid long parameter lists. 393 - 396
Avoid if (x != y) ..; else ..; 414 - 433
Potential violation of Law of Demeter (method chain calls) 416
Potential violation of Law of Demeter (object not created locally) 416
The String literal "testing path: " appears 6 times in this file; the first occurrence is on line 419 419
Avoid long parameter lists. 437 - 441
The method 'checkPath' has a Cyclomatic Complexity of 14. 437 - 484
Avoid if (x != y) ..; else ..; 463 - 482
Potential violation of Law of Demeter (method chain calls) 465
Potential violation of Law of Demeter (object not created locally) 465
Avoid long parameter lists. 486 - 489
Potential violation of Law of Demeter (method chain calls) 548
Prefer StringBuffer over += for concatenating strings 553
Potential violation of Law of Demeter (object not created locally) 554
Potential violation of Law of Demeter (object not created locally) 556
Potential violation of Law of Demeter (method chain calls) 556
Potential violation of Law of Demeter (method chain calls) 556

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
Potential violation of Law of Demeter (method chain calls) 51
Potential violation of Law of Demeter (object not created locally) 52
Potential violation of Law of Demeter (object not created locally) 55
Potential violation of Law of Demeter (method chain calls) 85
Potential violation of Law of Demeter (method chain calls) 91
Potential violation of Law of Demeter (method chain calls) 111
Potential violation of Law of Demeter (method chain calls) 117
Potential violation of Law of Demeter (method chain calls) 133
Potential violation of Law of Demeter (object not created locally) 134
StringBuffer constructor is initialized with size 16, but has at least 175 characters appended. 139
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
Useless parentheses. 59
Avoid if (x != y) ..; else ..; 59

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

Violation Line
Private field 'availableThemes' could be made final; it is only initialized in the declaration or constructor. 38
Private field 'tobagoConfig' could be made final; it is only initialized in the declaration or constructor. 39

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/FactorList.java

Violation Line
Potential violation of Law of Demeter (object not created locally) 47

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

Violation Line
The class 'Grid' has a Cyclomatic Complexity of 5 (Highest = 55). 32 - 498
Possible God class (WMC=104, ATFD=22, TCC=0.3006535947712418) 32 - 498
Private field 'cells' could be made final; it is only initialized in the declaration or constructor. 40
Private field 'columnHeads' could be made final; it is only initialized in the declaration or constructor. 42
Private field 'columnCount' could be made final; it is only initialized in the declaration or constructor. 45
Avoid instantiating new objects inside loops 65
Avoid instantiating new objects inside loops 69
Avoid reassigning parameters such as 'columnSpan' 78
Avoid reassigning parameters such as 'columnSpan' 78
Avoid instantiating new objects inside loops 110
Avoid instantiating new objects inside loops 176
The method gridAsString() has an NPath complexity of 11503356 246 - 474
Avoid really long methods. 246 - 474
The method gridAsString() has an NCSS line count of 175 246 - 474
The method 'gridAsString' has a Cyclomatic Complexity of 55. 246 - 474
StringBuffer constructor is initialized with size 16, but has at least 17 characters appended. 248
Avoid if (x != y) ..; else ..; 253 - 257
Avoid appending characters as strings in StringBuffer.append. 254
Avoid appending characters as strings in StringBuffer.append. 256
Avoid appending characters as strings in StringBuffer.append. 262
Avoid appending characters as strings in StringBuffer.append. 265
Avoid appending characters as strings in StringBuffer.append. 268
Avoid appending characters as strings in StringBuffer.append. 270
Avoid appending characters as strings in StringBuffer.append. 272
Avoid if (x != y) ..; else ..; 277 - 281
Avoid appending characters as strings in StringBuffer.append. 278
Avoid appending characters as strings in StringBuffer.append. 280
Avoid if (x != y) ..; else ..; 284 - 288
Avoid appending characters as strings in StringBuffer.append. 285
Avoid appending characters as strings in StringBuffer.append. 287
Avoid appending characters as strings in StringBuffer.append. 289
Avoid appending characters as strings in StringBuffer.append. 300
Avoid appending characters as strings in StringBuffer.append. 303
Avoid appending characters as strings in StringBuffer.append. 306
Avoid appending characters as strings in StringBuffer.append. 308
Avoid appending characters as strings in StringBuffer.append. 310
Avoid appending characters as strings in StringBuffer.append. 325
Avoid appending characters as strings in StringBuffer.append. 327
Avoid appending characters as strings in StringBuffer.append. 330
Avoid appending characters as strings in StringBuffer.append. 335
Avoid appending characters as strings in StringBuffer.append. 337
Avoid appending characters as strings in StringBuffer.append. 339
Avoid appending characters as strings in StringBuffer.append. 344
Avoid appending characters as strings in StringBuffer.append. 346
Avoid appending characters as strings in StringBuffer.append. 350
Avoid appending characters as strings in StringBuffer.append. 352
Avoid appending characters as strings in StringBuffer.append. 361
Avoid appending characters as strings in StringBuffer.append. 363
Avoid appending characters as strings in StringBuffer.append. 369
Avoid appending characters as strings in StringBuffer.append. 372
Avoid appending characters as strings in StringBuffer.append. 375
Avoid appending characters as strings in StringBuffer.append. 377
Avoid appending characters as strings in StringBuffer.append. 379
Avoid appending characters as strings in StringBuffer.append. 383
Avoid if (x != y) ..; else ..; 389 - 393
Avoid appending characters as strings in StringBuffer.append. 390
Avoid appending characters as strings in StringBuffer.append. 392
Avoid appending characters as strings in StringBuffer.append. 398
Avoid appending characters as strings in StringBuffer.append. 400
Avoid appending characters as strings in StringBuffer.append. 404
Avoid appending characters as strings in StringBuffer.append. 407
Avoid appending characters as strings in StringBuffer.append. 410
Avoid appending characters as strings in StringBuffer.append. 415
Avoid appending characters as strings in StringBuffer.append. 417
Avoid appending characters as strings in StringBuffer.append. 421
Avoid if (x != y) ..; else ..; 425 - 429
Avoid appending characters as strings in StringBuffer.append. 426
Avoid appending characters as strings in StringBuffer.append. 428
Avoid appending characters as strings in StringBuffer.append. 430
Avoid if (x != y) ..; else ..; 436 - 440
Avoid appending characters as strings in StringBuffer.append. 437
Avoid appending characters as strings in StringBuffer.append. 439
Avoid appending characters as strings in StringBuffer.append. 445
Avoid appending characters as strings in StringBuffer.append. 448
Avoid appending characters as strings in StringBuffer.append. 451
Avoid appending characters as strings in StringBuffer.append. 453
Avoid appending characters as strings in StringBuffer.append. 455
Avoid if (x != y) ..; else ..; 460 - 464
Avoid appending characters as strings in StringBuffer.append. 461
Avoid appending characters as strings in StringBuffer.append. 463
Avoid if (x != y) ..; else ..; 466 - 470
Avoid appending characters as strings in StringBuffer.append. 467
Avoid appending characters as strings in StringBuffer.append. 469
Avoid appending characters as strings in StringBuffer.append. 471
StringBuffer constructor is initialized with size 16, but has at least 25 characters appended. 478
Avoid appending characters as strings in StringBuffer.append. 482
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 482
Avoid appending characters as strings in StringBuffer.append. 485
Potential violation of Law of Demeter (method chain calls) 496

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

Violation Line
Avoid if (x != y) ..; else ..; 66 - 67
Avoid if (x != y) ..; else ..; 68
Avoid if (x != y) ..; else ..; 69
Avoid if (x != y) ..; else ..; 70
Avoid appending characters as strings in StringBuffer.append. 92
Avoid appending characters as strings in StringBuffer.append. 98
Avoid appending characters as strings in StringBuffer.append. 100

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 49
Avoid if (x != y) ..; else ..; 49 - 53
Potential violation of Law of Demeter (object not created locally) 83
Potential violation of Law of Demeter (object not created locally) 85
Potential violation of Law of Demeter (object not created locally) 87
Potential violation of Law of Demeter (object not created locally) 87
Potential violation of Law of Demeter (object not created locally) 88

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

Violation Line
The class 'LayoutContext' has a Cyclomatic Complexity of 5 (Highest = 12). 59 - 159
Private field 'container' could be made final; it is only initialized in the declaration or constructor. 63
The method 'layout' has a Cyclomatic Complexity of 12. 69 - 120
The method layout() has an NPath complexity of 2048 69 - 120
Potential violation of Law of Demeter (object not created locally) 79
Potential violation of Law of Demeter (object not created locally) 83
Potential violation of Law of Demeter (object not created locally) 87
Potential violation of Law of Demeter (object not created locally) 91
Potential violation of Law of Demeter (object not created locally) 95
Potential violation of Law of Demeter (object not created locally) 99
Potential violation of Law of Demeter (object not created locally) 103
Potential violation of Law of Demeter (object not created locally) 107
Potential violation of Law of Demeter (object not created locally) 111
Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 123
Potential violation of Law of Demeter (method chain calls) 135
Potential violation of Law of Demeter (method chain calls) 135
Avoid appending characters as strings in StringBuffer.append. 136
Avoid appending characters as strings in StringBuffer.append. 139
Potential violation of Law of Demeter (method chain calls) 140
Potential violation of Law of Demeter (method chain calls) 140
Potential violation of Law of Demeter (method chain calls) 142
Potential violation of Law of Demeter (method chain calls) 142
Avoid appending characters as strings in StringBuffer.append. 143
Potential violation of Law of Demeter (method chain calls) 146
Avoid appending characters as strings in StringBuffer.append. 148
Potential violation of Law of Demeter (method chain calls) 151
Potential violation of Law of Demeter (method chain calls) 151
Avoid appending characters as strings in StringBuffer.append. 154

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

Violation Line
Avoid unused private fields such as 'LOG'. 39
Potential violation of Law of Demeter (method chain calls) 51
Avoid reassigning parameters such as 'size' 96
Avoid reassigning parameters such as 'size' 96
Avoid reassigning parameters such as 'size' 96
Avoid reassigning parameters such as 'size' 96
Avoid reassigning parameters such as 'size' 96
Avoid reassigning parameters such as 'size' 96
Avoid if (x != y) ..; else ..; 99 - 110
Avoid if (x != y) ..; else ..; 114 - 125

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
Potential violation of Law of Demeter (method chain calls) 57
Consider using varargs for methods or constructors which take an array the last parameter. 70
Potential violation of Law of Demeter (method chain calls) 74

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. 27

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/TobagoBodyTag.java

Violation Line
Avoid unused private fields such as 'LOG'. 31
Document empty method 39 - 40
An empty method in an abstract class should be abstract instead 39 - 40
Potential violation of Law of Demeter (object not created locally) 51
Potential violation of Law of Demeter (object not created locally) 52
String.trim().length()==0 is an inefficient way to validate an empty String. 52
Potential violation of Law of Demeter (method chain calls) 52

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 38
Potential violation of Law of Demeter (method chain calls) 38
Potential violation of Law of Demeter (method chain calls) 39
Potential violation of Law of Demeter (object not created locally) 39
Potential violation of Law of Demeter (method chain calls) 40
Potential violation of Law of Demeter (object not created locally) 40
Potential violation of Law of Demeter (method chain calls) 50
Potential violation of Law of Demeter (object not created locally) 50
Potential violation of Law of Demeter (object not created locally) 51
Potential violation of Law of Demeter (method chain calls) 51
Potential violation of Law of Demeter (object not created locally) 52
Potential violation of Law of Demeter (method chain calls) 52
Potential violation of Law of Demeter (object not created locally) 80
Potential violation of Law of Demeter (method chain calls) 81
Potential violation of Law of Demeter (object not created locally) 81
String.trim().length()==0 is an inefficient way to validate an empty String. 81

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 36
Potential violation of Law of Demeter (object not created locally) 36
Potential violation of Law of Demeter (object not created locally) 37
Potential violation of Law of Demeter (method chain calls) 37
Potential violation of Law of Demeter (object not created locally) 38
Potential violation of Law of Demeter (method chain calls) 38

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 70
Useless parentheses. 71
Potential violation of Law of Demeter (object not created locally) 74
Potential violation of Law of Demeter (method chain calls) 80
Avoid if (x != y) ..; else ..; 84 - 94
Potential violation of Law of Demeter (object not created locally) 85
Potential violation of Law of Demeter (static property access) 86
Potential violation of Law of Demeter (static property access) 88
Potential violation of Law of Demeter (method chain calls) 91
Potential violation of Law of Demeter (method chain calls) 93
Potential violation of Law of Demeter (object not created locally) 93
Useless parentheses. 95

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

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

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

Violation Line
The class 'ConverterTag' has a Cyclomatic Complexity of 4 (Highest = 12). 46 - 138
The method doStartTag() has an NPath complexity of 576 62 - 114
The method 'doStartTag' has a Cyclomatic Complexity of 12. 62 - 114
Potential violation of Law of Demeter (object not created locally) 72
Useless parentheses. 73
Potential violation of Law of Demeter (object not created locally) 76
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (object not created locally) 83
Potential violation of Law of Demeter (object not created locally) 90
Potential violation of Law of Demeter (object not created locally) 99
Potential violation of Law of Demeter (method chain calls) 99
Avoid if (x != y) ..; else ..; 103 - 110
Potential violation of Law of Demeter (object not created locally) 104
Potential violation of Law of Demeter (object not created locally) 106
Potential violation of Law of Demeter (object not created locally) 109
Useless parentheses. 113

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

Violation Line
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 54
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 55
Potential violation of Law of Demeter (object not created locally) 77
Useless parentheses. 78
Potential violation of Law of Demeter (object not created locally) 81
Potential violation of Law of Demeter (method chain calls) 87
Useless parentheses. 94

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

Violation Line
Possible God class (WMC=82, ATFD=101, TCC=0.16304347826086957) 50 - 543
Too many fields 50 - 543
The class 'GridLayoutConstraintTag' has a Cyclomatic Complexity of 3 (Highest = 59). 50 - 543
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 56
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 57
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 58
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 59
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 60
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 61
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 62
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 63
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 64
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 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' 67
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 68
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 69
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 70
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 71
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 72
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' 74
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 75
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 76
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 77
Avoid really long methods. 80 - 319
The method doStartTag() has an NCSS line count of 151 80 - 319
The method 'doStartTag' has a Cyclomatic Complexity of 59. 80 - 319
Potential violation of Law of Demeter (object not created locally) 90
Useless parentheses. 91
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 101
Potential violation of Law of Demeter (object not created locally) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (method chain calls) 109
Potential violation of Law of Demeter (object not created locally) 113
The String literal "Ignoring '" appears 10 times in this file; the first occurrence is on line 115 115
Potential violation of Law of Demeter (object not created locally) 121
Potential violation of Law of Demeter (object not created locally) 129
Potential violation of Law of Demeter (object not created locally) 131
Potential violation of Law of Demeter (object not created locally) 137
Potential violation of Law of Demeter (object not created locally) 139
Potential violation of Law of Demeter (object not created locally) 145
Potential violation of Law of Demeter (object not created locally) 147
Potential violation of Law of Demeter (object not created locally) 153
Potential violation of Law of Demeter (object not created locally) 155
Potential violation of Law of Demeter (object not created locally) 161
Potential violation of Law of Demeter (object not created locally) 163
Potential violation of Law of Demeter (object not created locally) 169
Potential violation of Law of Demeter (object not created locally) 171
Potential violation of Law of Demeter (object not created locally) 177
Potential violation of Law of Demeter (object not created locally) 179
Potential violation of Law of Demeter (object not created locally) 185
Potential violation of Law of Demeter (object not created locally) 187
Potential violation of Law of Demeter (object not created locally) 193
Potential violation of Law of Demeter (object not created locally) 195
Potential violation of Law of Demeter (object not created locally) 201
Potential violation of Law of Demeter (object not created locally) 203
Potential violation of Law of Demeter (object not created locally) 209
Potential violation of Law of Demeter (object not created locally) 211
Potential violation of Law of Demeter (object not created locally) 217
Potential violation of Law of Demeter (object not created locally) 219
Potential violation of Law of Demeter (object not created locally) 226
Potential violation of Law of Demeter (object not created locally) 228
The String literal "', because the parent is not a LayoutContainer!" appears 8 times in this file; the first occurrence is on line 231 231
Potential violation of Law of Demeter (object not created locally) 238
Potential violation of Law of Demeter (object not created locally) 240
Potential violation of Law of Demeter (object not created locally) 250
Potential violation of Law of Demeter (object not created locally) 252
Potential violation of Law of Demeter (object not created locally) 262
Potential violation of Law of Demeter (object not created locally) 264
Potential violation of Law of Demeter (object not created locally) 274
Potential violation of Law of Demeter (object not created locally) 276
Potential violation of Law of Demeter (object not created locally) 286
Potential violation of Law of Demeter (object not created locally) 288
Potential violation of Law of Demeter (object not created locally) 297
Potential violation of Law of Demeter (object not created locally) 299
Potential violation of Law of Demeter (object not created locally) 309
Potential violation of Law of Demeter (object not created locally) 311
Useless parentheses. 318
The String literal "java.lang.Object" appears 40 times in this file; the first occurrence is on line 367 367

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 61
Potential violation of Law of Demeter (method chain calls) 65
Potential violation of Law of Demeter (object not created locally) 65
Potential violation of Law of Demeter (method chain calls) 65

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

Violation Line
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 49
Potential violation of Law of Demeter (object not created locally) 61
Useless parentheses. 62
Potential violation of Law of Demeter (object not created locally) 65
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (method chain calls) 72
Potential violation of Law of Demeter (object not created locally) 76
Potential violation of Law of Demeter (method chain calls) 80
Potential violation of Law of Demeter (method chain calls) 80
Potential violation of Law of Demeter (object not created locally) 82
Useless parentheses. 84

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 65
Useless parentheses. 66
Potential violation of Law of Demeter (object not created locally) 69
Potential violation of Law of Demeter (object not created locally) 76
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (method chain calls) 79
Useless parentheses. 90
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 104

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

Violation Line
The String literal "center" appears 4 times in this file; the first occurrence is on line 152 152
The String literal "right" appears 4 times in this file; the first occurrence is on line 153 153
The String literal "left" appears 4 times in this file; the first occurrence is on line 153 153
The String literal "none" appears 4 times in this file; the first occurrence is on line 153 153

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 84 84

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

Violation Line
The class 'TabChangeListenerTag' has a Cyclomatic Complexity of 5 (Highest = 14). 46 - 147
The method doStartTag() has an NPath complexity of 960 79 - 131
The method 'doStartTag' has a Cyclomatic Complexity of 14. 79 - 131
Potential violation of Law of Demeter (object not created locally) 89
Useless parentheses. 90
Potential violation of Law of Demeter (object not created locally) 93
Potential violation of Law of Demeter (object not created locally) 100
Potential violation of Law of Demeter (method chain calls) 100
Potential violation of Law of Demeter (method chain calls) 103
Avoid if (x != y) ..; else ..; 122 - 127
Potential violation of Law of Demeter (object not created locally) 126
Useless parentheses. 130
Potential violation of Law of Demeter (method chain calls) 141
Potential violation of Law of Demeter (method chain calls) 141
Potential violation of Law of Demeter (object not created locally) 142
Potential violation of Law of Demeter (object not created locally) 142

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

Violation Line
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 52
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 53
Potential violation of Law of Demeter (object not created locally) 58
Potential violation of Law of Demeter (object not created locally) 59
Potential violation of Law of Demeter (object not created locally) 60
Potential violation of Law of Demeter (object not created locally) 64
Potential violation of Law of Demeter (object not created locally) 70

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 53
Potential violation of Law of Demeter (object not created locally) 55
Potential violation of Law of Demeter (method chain calls) 56
Potential violation of Law of Demeter (object not created locally) 60
Avoid empty catch blocks 61 - 63
Potential violation of Law of Demeter (object not created locally) 67
Avoid empty catch blocks 68 - 70

Tobago Core - org/apache/myfaces/tobago/internal/taglib/declaration/HasMargins.java

Violation Line
The String literal "org.apache.myfaces.tobago.layout.Measure" appears 4 times in this file; the first occurrence is on line 32 32
The String literal "getMargin() != null\n" appears 4 times in this file; the first occurrence is on line 33 33
The String literal " ? getMargin()\n : getRendererType() != null" appears 4 times in this file; the first occurrence is on line 34 34

Tobago Core - org/apache/myfaces/tobago/internal/taglib/declaration/IsGridLayoutComponentBase.java

Violation Line
The String literal "org.apache.myfaces.tobago.layout.Measure" appears 14 times in this file; the first occurrence is on line 35 35
The String literal "java.lang.Integer" appears 4 times in this file; the first occurrence is on line 51 51
The String literal "getRendererType() != null ? ((LayoutComponentRenderer) " appears 10 times in this file; the first occurrence is on line 64 64

Tobago Core - org/apache/myfaces/tobago/internal/taglib/declaration/IsGridLayoutContainer.java

Violation Line
The String literal "org.apache.myfaces.tobago.layout.Measure" appears 8 times in this file; the first occurrence is on line 32 32
The String literal "getRendererType() != null ? ((org.apache.myfaces.tobago.renderkit.LayoutComponentRenderer)\n" appears 8 times in this file; the first occurrence is on line 33 33

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

Violation Line
Possible God class (WMC=57, ATFD=40, TCC=0.1402116402116402) 56 - 505
Too many fields 56 - 505
The class 'DateExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 26). 56 - 505
The method doStartTag() has an NPath complexity of 33554432 89 - 182
The method 'doStartTag' has a Cyclomatic Complexity of 26. 89 - 182
Avoid if (x != y) ..; else ..; 99 - 103
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 296 296
The String literal "false" appears 5 times in this file; the first occurrence is on line 296 296

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

Violation Line
Avoid unused private fields such as 'LOG'. 33
Avoid unused local variables such as 'application'. 51
Potential violation of Law of Demeter (object not created locally) 51
Avoid if (x != y) ..; else ..; 53 - 57
Potential violation of Law of Demeter (object not created locally) 54
Potential violation of Law of Demeter (object not created locally) 56
Potential violation of Law of Demeter (static property access) 56
Potential violation of Law of Demeter (object not created locally) 60

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

Violation Line
The class 'FileExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 20). 53 - 363
Too many fields 53 - 363
The method doStartTag() has an NPath complexity of 524288 77 - 149
The method 'doStartTag' has a Cyclomatic Complexity of 20. 77 - 149
Avoid empty if statements 132 - 134
The String literal "boolean" appears 4 times in this file; the first occurrence is on line 242 242

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

Violation Line
Too many fields 57 - 515
Possible God class (WMC=57, ATFD=37, TCC=0.12561576354679804) 57 - 515
The class 'InExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 29). 57 - 515
The method doStartTag() has an NPath complexity of 268435456 92 - 191
The method 'doStartTag' has a Cyclomatic Complexity of 29. 92 - 191
Avoid unused private fields such as 'LOG'. 200
The String literal "false" appears 5 times in this file; the first occurrence is on line 284 284
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 284 284

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

Violation Line
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 74
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 75
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 76
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 77
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 79
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 80
Avoid if (x != y) ..; else ..; 109 - 113
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 118
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 168
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 177
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 187
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 191
Avoid if (x != y) ..; else ..; 192 - 198
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 222

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

Violation Line
Too many fields 58 - 339
The class 'MenuCheckboxExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 15). 58 - 339
The method 'doStartTag' has a Cyclomatic Complexity of 15. 78 - 141
The method doStartTag() has an NPath complexity of 16384 78 - 141
Potential violation of Law of Demeter (object not created locally) 150
Avoid if (x != y) ..; else ..; 151 - 156
Potential violation of Law of Demeter (object not created locally) 152
Potential violation of Law of Demeter (method chain calls) 154
Potential violation of Law of Demeter (object not created locally) 154
Potential violation of Law of Demeter (object not created locally) 155
The String literal "boolean" appears 4 times in this file; the first occurrence is on line 252 252

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

Violation Line
The class 'MenuRadioExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 16). 62 - 361
Too many fields 62 - 361
The method doStartTag() has an NPath complexity of 32768 84 - 150
The method 'doStartTag' has a Cyclomatic Complexity of 16. 84 - 150
Potential violation of Law of Demeter (object not created locally) 159
Avoid if (x != y) ..; else ..; 160 - 165
Potential violation of Law of Demeter (object not created locally) 161
Potential violation of Law of Demeter (method chain calls) 163
Potential violation of Law of Demeter (object not created locally) 163
Potential violation of Law of Demeter (object not created locally) 164
The String literal "boolean" appears 4 times in this file; the first occurrence is on line 258 258

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

Violation Line
The class 'SelectBooleanCheckboxExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 25). 40 - 421
Too many fields 40 - 421
Possible God class (WMC=49, ATFD=30, TCC=0.14333333333333334) 40 - 421
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 42
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 44
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 45
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 46
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 47
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 48
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 49
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 50
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 51
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 52
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 54
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 55
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 56
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 57
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 58
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 59
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 60
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 61
The method 'doStartTag' has a Cyclomatic Complexity of 25. 68 - 157
The method doStartTag() has an NPath complexity of 16777216 68 - 157
Avoid if (x != y) ..; else ..; 132 - 136
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 199
The String literal "false" appears 4 times in this file; the first occurrence is on line 224 224
The String literal "boolean" appears 5 times in this file; the first occurrence is on line 224 224
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 225
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 234
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 243
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 253
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 293
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 303
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 313
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 322
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 333
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 343
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 349
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 358
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 369
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 378
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 387
Unnecessary use of fully qualified name 'javax.el.ValueExpression' due to existing import 'javax.el.ValueExpression' 396

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

Violation Line
The class 'SelectManyCheckboxExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 26). 38 - 438
Too many fields 38 - 438
Possible God class (WMC=51, ATFD=32, TCC=0.13846153846153847) 38 - 438
The method doStartTag() has an NPath complexity of 33554432 68 - 162
The method 'doStartTag' has a Cyclomatic Complexity of 26. 68 - 162
The String literal "false" appears 5 times in this file; the first occurrence is on line 207 207
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 207 207

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

Violation Line
Possible God class (WMC=49, ATFD=31, TCC=0.14333333333333334) 38 - 419
Too many fields 38 - 419
The class 'SelectManyListboxExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 25). 38 - 419
The method doStartTag() has an NPath complexity of 16777216 66 - 154
The method 'doStartTag' has a Cyclomatic Complexity of 25. 66 - 154
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 197 197
The String literal "false" appears 5 times in this file; the first occurrence is on line 197 197

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

Violation Line
The class 'SelectManyShuttleExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 26). 38 - 432
Too many fields 38 - 432
Possible God class (WMC=51, ATFD=32, TCC=0.13846153846153847) 38 - 432
The method 'doStartTag' has a Cyclomatic Complexity of 26. 67 - 158
The method doStartTag() has an NPath complexity of 33554432 67 - 158
The String literal "boolean" appears 5 times in this file; the first occurrence is on line 203 203
The String literal "false" appears 4 times in this file; the first occurrence is on line 203 203

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

Violation Line
Too many fields 39 - 419
The class 'SelectOneChoiceExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 25). 39 - 419
Possible God class (WMC=49, ATFD=30, TCC=0.14333333333333334) 39 - 419
The method 'doStartTag' has a Cyclomatic Complexity of 25. 67 - 154
The method doStartTag() has an NPath complexity of 16777216 67 - 154
The String literal "false" appears 5 times in this file; the first occurrence is on line 197 197
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 197 197

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

Violation Line
Possible God class (WMC=47, ATFD=30, TCC=0.14855072463768115) 38 - 407
Too many fields 38 - 407
The class 'SelectOneListboxExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 24). 38 - 407
The method doStartTag() has an NPath complexity of 8388608 65 - 153
The method 'doStartTag' has a Cyclomatic Complexity of 24. 65 - 153
The String literal "false" appears 4 times in this file; the first occurrence is on line 196 196
The String literal "boolean" appears 5 times in this file; the first occurrence is on line 196 196

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

Violation Line
Possible God class (WMC=51, ATFD=31, TCC=0.13846153846153847) 40 - 432
The class 'SelectOneRadioExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 26). 40 - 432
Too many fields 40 - 432
The method doStartTag() has an NPath complexity of 33554432 69 - 160
The method 'doStartTag' has a Cyclomatic Complexity of 26. 69 - 160
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 204 204
The String literal "false" appears 5 times in this file; the first occurrence is on line 204 204

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

Violation Line
The class 'TextareaExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 24). 53 - 418
Too many fields 53 - 418
Possible God class (WMC=47, ATFD=30, TCC=0.14855072463768115) 53 - 418
The method 'doStartTag' has a Cyclomatic Complexity of 24. 80 - 165
The method doStartTag() has an NPath complexity of 8388608 80 - 165
The String literal "false" appears 4 times in this file; the first occurrence is on line 241 241
The String literal "boolean" appears 5 times in this file; the first occurrence is on line 241 241

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

Violation Line
Too many fields 52 - 432
The class 'TimeExtensionTag' has a Cyclomatic Complexity of 2 (Highest = 24). 52 - 432
Possible God class (WMC=48, ATFD=29, TCC=0.14) 52 - 432
Avoid unused private fields such as 'converter'. 55
The method 'doStartTag' has a Cyclomatic Complexity of 24. 80 - 167
The method doStartTag() has an NPath complexity of 8388608 80 - 167
The String literal "boolean" appears 6 times in this file; the first occurrence is on line 254 254
The String literal "false" appears 5 times in this file; the first occurrence is on line 254 254

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 35 - 36
Potential violation of Law of Demeter (method chain calls) 35 - 36
Potential violation of Law of Demeter (method chain calls) 35 - 36

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

Violation Line
Avoid using implementation types like 'HashSet'; use the interface instead 35
Private field 'set' could be made final; it is only initialized in the declaration or constructor. 35
Avoid using implementation types like 'HashSet'; use the interface instead 35
StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 36
Potential violation of Law of Demeter (method chain calls) 40
Avoid using implementation types like 'HashSet'; use the interface instead 53
Avoid using implementation types like 'HashSet'; use the interface instead 53
Avoid reassigning parameters such as 'key' 65
Potential violation of Law of Demeter (method chain calls) 66
Potential violation of Law of Demeter (method chain calls) 66
Potential violation of Law of Demeter (method chain calls) 68
Potential violation of Law of Demeter (object not created locally) 68
Potential violation of Law of Demeter (object not created locally) 69
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (method chain calls) 72
Potential violation of Law of Demeter (method chain calls) 78
Potential violation of Law of Demeter (method chain calls) 82
Potential violation of Law of Demeter (object not created locally) 85

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

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

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 38
Potential violation of Law of Demeter (method chain calls) 48
Useless parentheses. 60
Potential violation of Law of Demeter (method chain calls) 68
Potential violation of Law of Demeter (method chain calls) 68

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 51
Potential violation of Law of Demeter (method chain calls) 51

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 63
Potential violation of Law of Demeter (object not created locally) 86

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/FacesContextUtils.java

Violation Line
This class has too many methods, consider refactoring it. 33 - 280
Potential violation of Law of Demeter (method chain calls) 54
Potential violation of Law of Demeter (method chain calls) 58
Potential violation of Law of Demeter (method chain calls) 62
Potential violation of Law of Demeter (method chain calls) 66
Potential violation of Law of Demeter (method chain calls) 70
Potential violation of Law of Demeter (method chain calls) 74
Potential violation of Law of Demeter (method chain calls) 79
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (method chain calls) 91
Potential violation of Law of Demeter (method chain calls) 95
Potential violation of Law of Demeter (method chain calls) 103
Potential violation of Law of Demeter (method chain calls) 106
Potential violation of Law of Demeter (method chain calls) 113
Potential violation of Law of Demeter (method chain calls) 121
Potential violation of Law of Demeter (method chain calls) 124
Potential violation of Law of Demeter (method chain calls) 131
Potential violation of Law of Demeter (method chain calls) 139
Potential violation of Law of Demeter (method chain calls) 142
Potential violation of Law of Demeter (method chain calls) 148
Potential violation of Law of Demeter (method chain calls) 156
Potential violation of Law of Demeter (method chain calls) 159
Potential violation of Law of Demeter (method chain calls) 165
Potential violation of Law of Demeter (method chain calls) 174
Potential violation of Law of Demeter (method chain calls) 177
Potential violation of Law of Demeter (method chain calls) 183
Potential violation of Law of Demeter (method chain calls) 186
Potential violation of Law of Demeter (method chain calls) 192
Potential violation of Law of Demeter (method chain calls) 200
Potential violation of Law of Demeter (method chain calls) 203
Potential violation of Law of Demeter (method chain calls) 209
Potential violation of Law of Demeter (method chain calls) 217
Potential violation of Law of Demeter (method chain calls) 220
Potential violation of Law of Demeter (method chain calls) 226
Potential violation of Law of Demeter (method chain calls) 234
Potential violation of Law of Demeter (method chain calls) 237
Potential violation of Law of Demeter (method chain calls) 243
Potential violation of Law of Demeter (object not created locally) 244
Potential violation of Law of Demeter (method chain calls) 248
Potential violation of Law of Demeter (method chain calls) 256
Potential violation of Law of Demeter (method chain calls) 259
Potential violation of Law of Demeter (method chain calls) 265
Potential violation of Law of Demeter (method chain calls) 273
Potential violation of Law of Demeter (method chain calls) 276

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/FindComponentUtils.java

Violation Line
The class 'FindComponentUtils' has a Cyclomatic Complexity of 6 (Highest = 10). 28 - 67
Avoid reassigning parameters such as 'from' 33
Avoid reassigning parameters such as 'relativeId' 33
The method 'findComponent' has a Cyclomatic Complexity of 10. 33 - 66
These nested if statements could be combined 36 - 38
Position literals first in String comparisons 36

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

Violation Line
The class 'HtmlWriterUtils' has a Cyclomatic Complexity of 9 (Highest = 14). 25 - 122
The method 'writeEncodedValue' has a Cyclomatic Complexity of 14. 60 - 121
Potential violation of Law of Demeter (object not created locally) 77
Potential violation of Law of Demeter (object not created locally) 80
Useless parentheses. 89
Potential violation of Law of Demeter (object not created locally) 92
Avoid if (x != y) ..; else ..; 93 - 97
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (method chain calls) 94
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (object not created locally) 99
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (object not created locally) 104
Potential violation of Law of Demeter (object not created locally) 106
Potential violation of Law of Demeter (object not created locally) 107
Potential violation of Law of Demeter (object not created locally) 109
Potential violation of Law of Demeter (object not created locally) 119

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

Violation Line
Avoid empty catch blocks 35 - 37

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

Violation Line
The class 'JavascriptWriterUtils' has a Cyclomatic Complexity of 9 (Highest = 14). 25 - 120
The method 'writeEncodedValue' has a Cyclomatic Complexity of 14. 58 - 119
Potential violation of Law of Demeter (object not created locally) 75
Potential violation of Law of Demeter (object not created locally) 78
Useless parentheses. 87
Potential violation of Law of Demeter (object not created locally) 90
Avoid if (x != y) ..; else ..; 91 - 95
Potential violation of Law of Demeter (object not created locally) 92
Potential violation of Law of Demeter (method chain calls) 92
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 97
Potential violation of Law of Demeter (object not created locally) 100
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (object not created locally) 104
Potential violation of Law of Demeter (object not created locally) 105
Potential violation of Law of Demeter (object not created locally) 107
Potential violation of Law of Demeter (object not created locally) 117

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 46
Potential violation of Law of Demeter (object not created locally) 51
Prefer StringBuffer over += for concatenating strings 57
Potential violation of Law of Demeter (object not created locally) 66
Potential violation of Law of Demeter (object not created locally) 67
Potential violation of Law of Demeter (object not created locally) 69
Potential violation of Law of Demeter (object not created locally) 71

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

Violation Line
The class 'JsonWriterUtils' has a Cyclomatic Complexity of 9 (Highest = 14). 26 - 125
The method 'writeEncodedValue' has a Cyclomatic Complexity of 14. 63 - 124
Potential violation of Law of Demeter (object not created locally) 80
Potential violation of Law of Demeter (object not created locally) 83
Useless parentheses. 92
Potential violation of Law of Demeter (object not created locally) 95
Avoid if (x != y) ..; else ..; 96 - 100
Potential violation of Law of Demeter (method chain calls) 97
Potential violation of Law of Demeter (object not created locally) 97
Potential violation of Law of Demeter (object not created locally) 99
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (object not created locally) 105
Potential violation of Law of Demeter (object not created locally) 107
Potential violation of Law of Demeter (object not created locally) 109
Potential violation of Law of Demeter (object not created locally) 110
Potential violation of Law of Demeter (object not created locally) 112
Potential violation of Law of Demeter (object not created locally) 122

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

Violation Line
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 22 - 34
Useless parentheses. 24
Useless parentheses. 24

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

Violation Line
A class which only has private constructors should be final 22 - 32
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/ResponseUtils.java

Violation Line
The class 'ResponseUtils' has a Cyclomatic Complexity of 4 (Highest = 10). 33 - 133
Potential violation of Law of Demeter (method chain calls) 42
Potential violation of Law of Demeter (method chain calls) 59
Potential violation of Law of Demeter (method chain calls) 63
Avoid if (x != y) ..; else ..; 72 - 83
The method 'ensureContentSecurityPolicyHeader' has a Cyclomatic Complexity of 10. 97 - 132
Potential violation of Law of Demeter (method chain calls) 99
Avoid declaring a variable if it is unreferenced before a possible exit point. 101
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (object not created locally) 109
Potential violation of Law of Demeter (object not created locally) 112
Avoid appending characters as strings in StringBuffer.append. 120
Potential violation of Law of Demeter (object not created locally) 124
These nested if statements could be combined 128 - 130

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
The class 'StringUtils' has a Cyclomatic Complexity of 5 (Highest = 18). 27 - 465
This class has too many methods, consider refactoring it. 27 - 465
Possible God class (WMC=102, ATFD=9, TCC=0.0) 27 - 465
Potential violation of Law of Demeter (method chain calls) 43
Potential violation of Law of Demeter (object not created locally) 44
Avoid instantiating Integer objects. Call Integer.valueOf() instead. 45
Avoid instantiating new objects inside loops 45
Avoid appending characters as strings in StringBuffer.append. 57
Potential violation of Law of Demeter (object not created locally) 66
Potential violation of Law of Demeter (method chain calls) 67
Potential violation of Law of Demeter (object not created locally) 67
Potential violation of Law of Demeter (object not created locally) 68
Potential violation of Law of Demeter (object not created locally) 70
Potential violation of Law of Demeter (method chain calls) 72
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (method chain calls) 72
Potential violation of Law of Demeter (method chain calls) 73
Potential violation of Law of Demeter (object not created locally) 73
Potential violation of Law of Demeter (method chain calls) 73
Potential violation of Law of Demeter (object not created locally) 76
Potential violation of Law of Demeter (object not created locally) 80
Potential violation of Law of Demeter (object not created locally) 86
Potential violation of Law of Demeter (object not created locally) 88
Potential violation of Law of Demeter (method chain calls) 98
Potential violation of Law of Demeter (method chain calls) 100
Potential violation of Law of Demeter (method chain calls) 102
Useless parentheses. 102
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (method chain calls) 102
The method 'equalsIgnoreCaseAndWhitespace' has a Cyclomatic Complexity of 14. 113 - 167
The method equalsIgnoreCaseAndWhitespace() has an NPath complexity of 378 113 - 167
Use equals() to compare object references. 116
Potential violation of Law of Demeter (method chain calls) 133
Potential violation of Law of Demeter (method chain calls) 133
Potential violation of Law of Demeter (method chain calls) 137
Potential violation of Law of Demeter (method chain calls) 140
Potential violation of Law of Demeter (method chain calls) 149
Potential violation of Law of Demeter (method chain calls) 158
Useless parentheses. 174
Potential violation of Law of Demeter (object not created locally) 203
Potential violation of Law of Demeter (object not created locally) 213
Potential violation of Law of Demeter (object not created locally) 215
The method 'split' has a Cyclomatic Complexity of 18. 221 - 296
Avoid declaring a variable if it is unreferenced before a possible exit point. 222
Potential violation of Law of Demeter (object not created locally) 247
Potential violation of Law of Demeter (object not created locally) 265
Potential violation of Law of Demeter (object not created locally) 282
Potential violation of Law of Demeter (object not created locally) 293
Potential violation of Law of Demeter (object not created locally) 295
Useless parentheses. 340
The method 'replace' has a Cyclomatic Complexity of 10. 357 - 382
The method replace() has an NPath complexity of 1080 357 - 382
Avoid declaring a variable if it is unreferenced before a possible exit point. 358
Useless parentheses. 369
Useless parentheses. 370
Useless parentheses. 370
Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 371
Potential violation of Law of Demeter (object not created locally) 373
Potential violation of Law of Demeter (method chain calls) 373
Potential violation of Law of Demeter (object not created locally) 380
Potential violation of Law of Demeter (object not created locally) 381
No need to call String.valueOf to append to a string. 400
Potential violation of Law of Demeter (method chain calls) 425
Local variable 'colon' could be declared final 454

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

Violation Line
Useless parentheses. 172
Useless parentheses. 174
Useless parentheses. 176
Useless parentheses. 178
Useless parentheses. 182
Useless parentheses. 184
Useless parentheses. 186
Useless parentheses. 190
Useless parentheses. 192

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 39
Potential violation of Law of Demeter (method chain calls) 39
Potential violation of Law of Demeter (method chain calls) 46
Potential violation of Law of Demeter (method chain calls) 46
Potential violation of Law of Demeter (method chain calls) 50
Potential violation of Law of Demeter (method chain calls) 50
Document empty method 58 - 60

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 54
These nested if statements could be combined 55 - 57
Potential violation of Law of Demeter (method chain calls) 64
These nested if statements could be combined 65 - 67

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

Violation Line
This class has too many methods, consider refactoring it. 37 - 188
Private field 'stack' could be made final; it is only initialized in the declaration or constructor. 39
Potential violation of Law of Demeter (object not created locally) 55
Potential violation of Law of Demeter (static property access) 97
Potential violation of Law of Demeter (object not created locally) 182

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

Violation Line
This class has too many methods, consider refactoring it. 39 - 183
Private field 'javascriptWriter' could be made final; it is only initialized in the declaration or constructor. 45
Avoid empty if statements 119 - 121
Potential violation of Law of Demeter (method chain calls) 128
Potential violation of Law of Demeter (method chain calls) 136
Avoid reassigning parameters such as 'styles' 149
Potential violation of Law of Demeter (method chain calls) 152
Potential violation of Law of Demeter (method chain calls) 159
Potential violation of Law of Demeter (method chain calls) 160

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

Violation Line
This class has too many methods, consider refactoring it. 34 - 191
Private field 'javascriptWriter' could be made final; it is only initialized in the declaration or constructor. 38
Private field 'encodeInJavascriptBlock' could be made final; it is only initialized in the declaration or constructor. 40
Private field 'encodeOutsideJavascriptBlock' could be made final; it is only initialized in the declaration or constructor. 41
Consider using varargs for methods or constructors which take an array the last parameter. 75
Potential violation of Law of Demeter (method chain calls) 81
Potential violation of Law of Demeter (method chain calls) 82
Potential violation of Law of Demeter (method chain calls) 82
Potential violation of Law of Demeter (method chain calls) 82
Potential violation of Law of Demeter (method chain calls) 130
Potential violation of Law of Demeter (object not created locally) 144
Potential violation of Law of Demeter (object not created locally) 144
Potential violation of Law of Demeter (method chain calls) 155
Potential violation of Law of Demeter (method chain calls) 167
Document empty method 185 - 186
Document empty method 189 - 190

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

Violation Line
The class 'TobagoMultipartFormdataRequest' has a Cyclomatic Complexity of 3 (Highest = 13). 43 - 200
The method 'init' has a Cyclomatic Complexity of 13. 65 - 134
The method init() has an NPath complexity of 401 65 - 134
Avoid if (x != y) ..; else ..; 66 - 133
Potential violation of Law of Demeter (object not created locally) 79
Potential violation of Law of Demeter (object not created locally) 81
Potential violation of Law of Demeter (object not created locally) 83
Potential violation of Law of Demeter (object not created locally) 87
Potential violation of Law of Demeter (method chain calls) 93
Potential violation of Law of Demeter (method chain calls) 93
Potential violation of Law of Demeter (object not created locally) 93
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (object not created locally) 98
Potential violation of Law of Demeter (object not created locally) 99
Potential violation of Law of Demeter (object not created locally) 100
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (object not created locally) 103
Potential violation of Law of Demeter (object not created locally) 105
Potential violation of Law of Demeter (object not created locally) 109
Potential violation of Law of Demeter (object not created locally) 111
Potential violation of Law of Demeter (object not created locally) 112
Potential violation of Law of Demeter (object not created locally) 123
Potential violation of Law of Demeter (object not created locally) 124
Potential violation of Law of Demeter (object not created locally) 181
Potential violation of Law of Demeter (object not created locally) 183
Potential violation of Law of Demeter (object not created locally) 183
Potential violation of Law of Demeter (object not created locally) 184
Potential violation of Law of Demeter (object not created locally) 186
Potential violation of Law of Demeter (object not created locally) 186
Potential violation of Law of Demeter (object not created locally) 187
Potential violation of Law of Demeter (object not created locally) 189
Potential violation of Law of Demeter (object not created locally) 189
Potential violation of Law of Demeter (object not created locally) 192
Potential violation of Law of Demeter (method chain calls) 194 - 195
Potential violation of Law of Demeter (method chain calls) 195

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

Violation Line
This class has too many methods, consider refactoring it. 36 - 329
Possible God class (WMC=49, ATFD=12, TCC=0.07142857142857142) 36 - 329
Avoid if (x != y) ..; else ..; 75
Avoid if (x != y) ..; else ..; 99 - 126
Avoid if (x != y) ..; else ..; 101 - 126
Avoid if (x != y) ..; else ..; 102 - 117
Potential violation of Law of Demeter (method chain calls) 103
Avoid if (x != y) ..; else ..; 104 - 108
Potential violation of Law of Demeter (object not created locally) 105
Potential violation of Law of Demeter (object not created locally) 113
Potential violation of Law of Demeter (object not created locally) 113
Potential violation of Law of Demeter (object not created locally) 122
Potential violation of Law of Demeter (object not created locally) 122
Consider using varargs for methods or constructors which take an array the last parameter. 151
An empty method in an abstract class should be abstract instead 189 - 191
An empty method in an abstract class should be abstract instead 193 - 195
Potential violation of Law of Demeter (method chain calls) 246
Potential violation of Law of Demeter (method chain calls) 246
Potential violation of Law of Demeter (method chain calls) 246
Potential violation of Law of Demeter (method chain calls) 249
Potential violation of Law of Demeter (method chain calls) 249
Potential violation of Law of Demeter (object not created locally) 256
Potential violation of Law of Demeter (static property access) 276
Potential violation of Law of Demeter (static property access) 285
Potential violation of Law of Demeter (object not created locally) 313
Potential violation of Law of Demeter (object not created locally) 313

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

Violation Line
This class has too many methods, consider refactoring it. 32 - 125
Private field 'responseWriter' could be made final; it is only initialized in the declaration or constructor. 34
Potential violation of Law of Demeter (static property access) 90

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 51
Avoid empty if statements 57 - 59
Potential violation of Law of Demeter (method chain calls) 71
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (method chain calls) 81
Potential violation of Law of Demeter (method chain calls) 82

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 49
Potential violation of Law of Demeter (method chain calls) 56

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

Violation Line
The class 'LayoutTokens' has a Cyclomatic Complexity of 2 (Highest = 10). 31 - 170
This class has too many methods, consider refactoring it. 31 - 170
Private field 'tokens' could be made final; it is only initialized in the declaration or constructor. 35
Document empty constructor 37 - 38
Consider using varargs for methods or constructors which take an array the last parameter. 80
Potential violation of Law of Demeter (method chain calls) 101
The method 'parseToken' has a Cyclomatic Complexity of 10. 114 - 139
Potential violation of Law of Demeter (method chain calls) 116
Potential violation of Law of Demeter (method chain calls) 121
Potential violation of Law of Demeter (method chain calls) 121
Potential violation of Law of Demeter (method chain calls) 123
Avoid appending characters as strings in StringBuffer.append. 165

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

Violation Line
This class has too many methods, consider refactoring it. 32 - 196
Possible God class (WMC=49, ATFD=8, TCC=0.008333333333333333) 32 - 196
Avoid unused private fields such as 'LOG'. 36
Potential violation of Law of Demeter (object not created locally) 155
Potential violation of Law of Demeter (object not created locally) 168
Avoid if (x != y) ..; else ..; 179 - 183
Avoid if (x != y) ..; else ..; 182
Avoid if (x != y) ..; else ..; 190 - 194
Avoid if (x != y) ..; else ..; 193

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
This class has too many methods, consider refactoring it. 25 - 131
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/layout/TextAlign.java

Violation Line
Avoid if (x != y) ..; else ..; 64 - 68

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 38
Potential violation of Law of Demeter (object not created locally) 39
Potential violation of Law of Demeter (method chain calls) 40
Potential violation of Law of Demeter (object not created locally) 45
Document empty method 49 - 50

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
Potential violation of Law of Demeter (method chain calls) 54
Potential violation of Law of Demeter (method chain calls) 54
Potential violation of Law of Demeter (method chain calls) 54
Potential violation of Law of Demeter (method chain calls) 58
Potential violation of Law of Demeter (method chain calls) 58
Potential violation of Law of Demeter (method chain calls) 58
Potential violation of Law of Demeter (object not created locally) 67
Potential violation of Law of Demeter (object not created locally) 68
Potential violation of Law of Demeter (object not created locally) 69
Potential violation of Law of Demeter (object not created locally) 70
Avoid appending characters as strings in StringBuffer.append. 77
Potential violation of Law of Demeter (method chain calls) 77
Potential violation of Law of Demeter (method chain calls) 77
When instantiating a SimpleDateFormat object, specify a Locale 80
Potential violation of Law of Demeter (object not created locally) 83
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (object not created locally) 84
Avoid appending characters as strings in StringBuffer.append. 84
Potential violation of Law of Demeter (method chain calls) 84
Avoid appending characters as strings in StringBuffer.append. 86
Avoid appending characters as strings in StringBuffer.append. 90
Potential violation of Law of Demeter (object not created locally) 90
Potential violation of Law of Demeter (method chain calls) 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 ..; 72 - 73
Potential violation of Law of Demeter (object not created locally) 74

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

Violation Line
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
Potential violation of Law of Demeter (object not created locally) 112
Potential violation of Law of Demeter (object not created locally) 120
Avoid reassigning parameters such as 'level' 164
Potential violation of Law of Demeter (object not created locally) 173
Potential violation of Law of Demeter (object not created locally) 181

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

Violation Line
Private field 'junctions' could be made final; it is only initialized in the declaration or constructor. 39
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (object not created locally) 71
Potential violation of Law of Demeter (object not created locally) 77

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 65
Potential violation of Law of Demeter (object not created locally) 93
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (object not created locally) 98

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

Violation Line
Avoid using redundant field initializer for 'currentMarkup' 31
Consider using varargs for methods or constructors which take an array the last parameter. 58
Consider using varargs for methods or constructors which take an array the last parameter. 77
Parameter 'tip' is not assigned and could be declared final 99

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

Violation Line
Avoid if (x != y) ..; else ..; 127 - 131

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 'resetSelected' called during object construction 50
Potential violation of Law of Demeter (method chain calls) 102
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (object not created locally) 106
Consider using varargs for methods or constructors which take an array the last parameter. 114
String.indexOf(char) is faster than String.indexOf(String). 147
Do not add empty strings 149

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 33
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 40

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

Violation Line
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 instantiation through private constructors from outside of the constructors class. 69
Avoid instantiating new objects inside loops 69
Potential violation of Law of Demeter (object not created locally) 73
Potential violation of Law of Demeter (object not created locally) 74
Avoid instantiating new objects inside loops 89
Avoid instantiation through private constructors from outside of the constructors class. 89
Potential violation of Law of Demeter (object not created locally) 95
Potential violation of Law of Demeter (object not created locally) 96
Avoid reassigning parameters such as 'node' 103
Potential violation of Law of Demeter (object not created locally) 124
Potential violation of Law of Demeter (object not created locally) 125
Potential violation of Law of Demeter (object not created locally) 126
Potential violation of Law of Demeter (method chain calls) 138
Potential violation of Law of Demeter (object not created locally) 149
Potential violation of Law of Demeter (method chain calls) 163
Potential violation of Law of Demeter (method chain calls) 170
Potential violation of Law of Demeter (object not created locally) 199
Potential violation of Law of Demeter (object not created locally) 202
Potential violation of Law of Demeter (method chain calls) 205
Potential violation of Law of Demeter (object not created locally) 205
Avoid instantiating new objects inside loops 208
Potential violation of Law of Demeter (object not created locally) 211
Potential violation of Law of Demeter (method chain calls) 218
Potential violation of Law of Demeter (method chain calls) 226
Potential violation of Law of Demeter (method chain calls) 234
Potential violation of Law of Demeter (method chain calls) 234
Avoid if (x != y) ..; else ..; 235 - 239
Potential violation of Law of Demeter (method chain calls) 236
Potential violation of Law of Demeter (object not created locally) 247
Potential violation of Law of Demeter (object not created locally) 250
Potential violation of Law of Demeter (object not created locally) 263
Potential violation of Law of Demeter (object not created locally) 264
Potential violation of Law of Demeter (object not created locally) 272
Potential violation of Law of Demeter (object not created locally) 272
Private field 'node' could be made final; it is only initialized in the declaration or constructor. 281

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

Violation Line
Avoid reassigning parameters such as 'node' 90
Avoid appending characters as strings in StringBuffer.append. 129
Avoid appending characters as strings in StringBuffer.append. 142
Potential violation of Law of Demeter (method chain calls) 143
Avoid reassigning parameters such as 'tree' 156
Avoid if (x != y) ..; else ..; 185

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

Violation Line
Avoid if (x != y) ..; else ..; 70 - 74

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

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

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

Violation Line
Overridable method 'reset' called during object construction 38
Useless parentheses. 48
Potential violation of Law of Demeter (method chain calls) 48
Potential violation of Law of Demeter (method chain calls) 48
Potential violation of Law of Demeter (method chain calls) 48
Potential violation of Law of Demeter (method chain calls) 48
Potential violation of Law of Demeter (method chain calls) 59

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

Violation Line
StringBuffer constructor is initialized with size 16, but has at least 21 characters appended. 57
Avoid appending characters as strings in StringBuffer.append. 58
Avoid appending characters as strings in StringBuffer.append. 64

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 32

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 32
Potential violation of Law of Demeter (method chain calls) 36
Potential violation of Law of Demeter (method chain calls) 40
Potential violation of Law of Demeter (method chain calls) 44
Potential violation of Law of Demeter (method chain calls) 48
Potential violation of Law of Demeter (method chain calls) 52

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 32
Potential violation of Law of Demeter (method chain calls) 36
Potential violation of Law of Demeter (method chain calls) 40
Potential violation of Law of Demeter (method chain calls) 44
Potential violation of Law of Demeter (method chain calls) 48
Potential violation of Law of Demeter (method chain calls) 52
Potential violation of Law of Demeter (method chain calls) 56
Potential violation of Law of Demeter (method chain calls) 60
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (method chain calls) 68
Potential violation of Law of Demeter (method chain calls) 72
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (method chain calls) 80
Potential violation of Law of Demeter (method chain calls) 84
Potential violation of Law of Demeter (method chain calls) 88
Potential violation of Law of Demeter (method chain calls) 92
Potential violation of Law of Demeter (method chain calls) 96
Potential violation of Law of Demeter (method chain calls) 100
Potential violation of Law of Demeter (method chain calls) 104
Potential violation of Law of Demeter (method chain calls) 108
Potential violation of Law of Demeter (method chain calls) 112
Potential violation of Law of Demeter (method chain calls) 116

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

Violation Line
Avoid unused private fields such as 'LOG'. 35
Potential violation of Law of Demeter (method chain calls) 40
Potential violation of Law of Demeter (method chain calls) 45
Potential violation of Law of Demeter (method chain calls) 49
Potential violation of Law of Demeter (method chain calls) 53
Potential violation of Law of Demeter (method chain calls) 58
Potential violation of Law of Demeter (method chain calls) 63
Potential violation of Law of Demeter (method chain calls) 68
Potential violation of Law of Demeter (method chain calls) 73
Potential violation of Law of Demeter (method chain calls) 78
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (method chain calls) 90
Potential violation of Law of Demeter (method chain calls) 95
Potential violation of Law of Demeter (method chain calls) 100
Potential violation of Law of Demeter (method chain calls) 105
Potential violation of Law of Demeter (method chain calls) 110
Potential violation of Law of Demeter (method chain calls) 114 - 115

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 45
Useless parentheses. 45
Potential violation of Law of Demeter (method chain calls) 52
Avoid if (x != y) ..; else ..; 53 - 54
Potential violation of Law of Demeter (object not created locally) 54
Potential violation of Law of Demeter (method chain calls) 77
Potential violation of Law of Demeter (object not created locally) 79

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 51
Potential violation of Law of Demeter (object not created locally) 52
Potential violation of Law of Demeter (object not created locally) 53
Potential violation of Law of Demeter (object not created locally) 58
Potential violation of Law of Demeter (object not created locally) 60

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

Violation Line
Avoid reassigning parameters such as 'escapedIndicatorCount' 57
Avoid reassigning parameters such as 'index' 57
Avoid if (x != y) ..; else ..; 78 - 86
Potential violation of Law of Demeter (method chain calls) 83 - 84

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 33
Potential violation of Law of Demeter (method chain calls) 37
Potential violation of Law of Demeter (method chain calls) 41
Potential violation of Law of Demeter (method chain calls) 45
Potential violation of Law of Demeter (method chain calls) 49
Potential violation of Law of Demeter (method chain calls) 53
Potential violation of Law of Demeter (method chain calls) 57
Potential violation of Law of Demeter (method chain calls) 61
Potential violation of Law of Demeter (method chain calls) 65
Potential violation of Law of Demeter (method chain calls) 69
Potential violation of Law of Demeter (method chain calls) 73
Potential violation of Law of Demeter (method chain calls) 77
Potential violation of Law of Demeter (method chain calls) 81
Potential violation of Law of Demeter (method chain calls) 85
Potential violation of Law of Demeter (method chain calls) 89
Potential violation of Law of Demeter (method chain calls) 93
Potential violation of Law of Demeter (method chain calls) 97
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 105
Potential violation of Law of Demeter (method chain calls) 109
Potential violation of Law of Demeter (method chain calls) 113
Potential violation of Law of Demeter (object not created locally) 118
Avoid if (x != y) ..; else ..; 119 - 123
Potential violation of Law of Demeter (method chain calls) 122

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 58
Potential violation of Law of Demeter (object not created locally) 59
Document empty method 67 - 68
Potential violation of Law of Demeter (method chain calls) 75
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (object not created locally) 98
Potential violation of Law of Demeter (object not created locally) 99
Potential violation of Law of Demeter (method chain calls) 122
Potential violation of Law of Demeter (method chain calls) 129
Potential violation of Law of Demeter (method chain calls) 132
Potential violation of Law of Demeter (object not created locally) 138
Potential violation of Law of Demeter (method chain calls) 144
Potential violation of Law of Demeter (method chain calls) 153
Potential violation of Law of Demeter (object not created locally) 158
Potential violation of Law of Demeter (method chain calls) 164
Avoid if (x != y) ..; else ..; 180 - 184
Potential violation of Law of Demeter (object not created locally) 181
Document empty method 188 - 189
Use block level rather than method level synchronization 191 - 196

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

Violation Line
Private field 'renderer' could be made final; it is only initialized in the declaration or constructor. 29
Document empty method 35 - 36

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

Violation Line
The class 'SelectManyRendererBase' has a Cyclomatic Complexity of 13 (Highest = 26). 39 - 200
Potential violation of Law of Demeter (method chain calls) 51
Potential violation of Law of Demeter (object not created locally) 51
Potential violation of Law of Demeter (method chain calls) 51
Avoid if (x != y) ..; else ..; 55
Deeply nested if..then statements are hard to read 56 - 60
Potential violation of Law of Demeter (object not created locally) 66
The method getConvertedUISelectManyValue() has an NPath complexity of 823680 86 - 198
The method 'getConvertedUISelectManyValue' has a Cyclomatic Complexity of 26. 86 - 198
Avoid really long methods. 86 - 198
Consider using varargs for methods or constructors which take an array the last parameter. 88
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (object not created locally) 103
Potential violation of Law of Demeter (object not created locally) 104
Potential violation of Law of Demeter (method chain calls) 116
Potential violation of Law of Demeter (method chain calls) 127
Potential violation of Law of Demeter (method chain calls) 130
Potential violation of Law of Demeter (method chain calls) 135
Potential violation of Law of Demeter (object not created locally) 137
Potential violation of Law of Demeter (object not created locally) 145
These nested if statements could be combined 146 - 148
Potential violation of Law of Demeter (method chain calls) 147
Potential violation of Law of Demeter (object not created locally) 147
Potential violation of Law of Demeter (method chain calls) 147
Potential violation of Law of Demeter (method chain calls) 159
Potential violation of Law of Demeter (object not created locally) 159
Potential violation of Law of Demeter (method chain calls) 164
Potential violation of Law of Demeter (object not created locally) 171
Potential violation of Law of Demeter (method chain calls) 171
Potential violation of Law of Demeter (method chain calls) 171
Potential violation of Law of Demeter (object not created locally) 180
Potential violation of Law of Demeter (method chain calls) 185 - 186
Potential violation of Law of Demeter (object not created locally) 186
Potential violation of Law of Demeter (method chain calls) 186
Potential violation of Law of Demeter (method chain calls) 194
Potential violation of Law of Demeter (method chain calls) 194
Potential violation of Law of Demeter (object not created locally) 194

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 41
Potential violation of Law of Demeter (method chain calls) 43
Potential violation of Law of Demeter (method chain calls) 43
Potential violation of Law of Demeter (object not created locally) 47

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

Violation Line
The class 'TobagoRenderKit' has a Cyclomatic Complexity of 4 (Highest = 13). 48 - 195
Private field 'renderers' could be made final; it is only initialized in the declaration or constructor. 56
Avoid instantiation through private constructors from outside of the constructors class. 66
Potential violation of Law of Demeter (object not created locally) 87
The method 'createResponseWriter' has a Cyclomatic Complexity of 13. 93 - 142
The method createResponseWriter() has an NPath complexity of 400 93 - 142
Avoid declaring a variable if it is unreferenced before a possible exit point. 95
The String literal "text/html" appears 5 times in this file; the first occurrence is on line 101 101
Potential violation of Law of Demeter (method chain calls) 119
Potential violation of Law of Demeter (method chain calls) 120
Potential violation of Law of Demeter (method chain calls) 121
Potential violation of Law of Demeter (method chain calls) 122
Potential violation of Law of Demeter (method chain calls) 127 - 128
Potential violation of Law of Demeter (method chain calls) 127 - 128
Potential violation of Law of Demeter (method chain calls) 138
Avoid instantiation through private constructors from outside of the constructors class. 146
Potential violation of Law of Demeter (method chain calls) 151
Potential violation of Law of Demeter (method chain calls) 156

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

Violation Line
Use block level rather than method level synchronization 86 - 101
Potential violation of Law of Demeter (method chain calls) 90
Avoid if (x != y) ..; else ..; 108
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 119

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

Violation Line
This class has a bunch of public methods and attributes 20 - 556
The class 'Style' has a Cyclomatic Complexity of 2 (Highest = 24). 33 - 556
Too many fields 33 - 556
Possible God class (WMC=101, ATFD=48, TCC=0.10638297872340426) 33 - 556
Document empty constructor 60 - 61
The constructor 'Style' has a Cyclomatic Complexity of 10. 88 - 133
Overridable method 'getDisplay' called during object construction 126
Avoid really long methods. 144 - 258
The method 'encode' has a Cyclomatic Complexity of 23. 144 - 258
The method encode() has an NPath complexity of 4194304 144 - 258
StringBuffer constructor is initialized with size 16, but has at least 22 characters appended. 145
Avoid really long methods. 260 - 379
The method 'encodeJson' has a Cyclomatic Complexity of 24. 260 - 379
The method encodeJson() has an NPath complexity of 8388608 260 - 379
StringBuffer constructor is initialized with size 17, but has at least 66 characters appended. 261
The String literal "\"," appears 21 times in this file; the first occurrence is on line 265 265
Avoid appending characters as strings in StringBuffer.append. 365
Parameter 'backgroundPosition' is not assigned and could be declared final 537

Tobago Core - org/apache/myfaces/tobago/renderkit/util/EncodeUtils.java

Violation Line
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 30 - 53
Potential violation of Law of Demeter (object not created locally) 40
Potential violation of Law of Demeter (object not created locally) 41
Potential violation of Law of Demeter (object not created locally) 44

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 50
Potential violation of Law of Demeter (object not created locally) 53
Avoid declaring a variable if it is unreferenced before a possible exit point. 57
Potential violation of Law of Demeter (object not created locally) 59
Potential violation of Law of Demeter (object not created locally) 66
Potential violation of Law of Demeter (object not created locally) 67
Potential violation of Law of Demeter (object not created locally) 68
Potential violation of Law of Demeter (object not created locally) 71
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (object not created locally) 75
Potential violation of Law of Demeter (object not created locally) 77
Potential violation of Law of Demeter (method chain calls) 82
Potential violation of Law of Demeter (method chain calls) 82
An empty method in an abstract class should be abstract instead 91 - 93

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

Violation Line
Private field 'resourceDirs' could be made final; it is only initialized in the declaration or constructor. 76
Potential violation of Law of Demeter (object not created locally) 82
Potential violation of Law of Demeter (object not created locally) 85
Potential violation of Law of Demeter (object not created locally) 86
Potential violation of Law of Demeter (object not created locally) 87
Avoid instantiating Long objects.Call Long.valueOf() instead 92
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 103
This call to String.startsWith can be rewritten using String.charAt(0) 116
Potential violation of Law of Demeter (object not created locally) 116
Potential violation of Law of Demeter (object not created locally) 117
Potential violation of Law of Demeter (object not created locally) 129
Potential violation of Law of Demeter (method chain calls) 129
Potential violation of Law of Demeter (method chain calls) 129
Potential violation of Law of Demeter (object not created locally) 131
Potential violation of Law of Demeter (object not created locally) 132
Potential violation of Law of Demeter (object not created locally) 132
Potential violation of Law of Demeter (object not created locally) 133
Potential violation of Law of Demeter (object not created locally) 133
Potential violation of Law of Demeter (object not created locally) 133
Avoid if (x != y) ..; else ..; 144 - 151
Potential violation of Law of Demeter (method chain calls) 155
Potential violation of Law of Demeter (object not created locally) 158
Potential violation of Law of Demeter (object not created locally) 162
Avoid if (x != y) ..; else ..; 165 - 171
Avoid if (x != y) ..; else ..; 179 - 183

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

Violation Line
The class 'ApplyRequestValuesCallback' has a Cyclomatic Complexity of 8 (Highest = 13). 35 - 68
Avoid unused private fields such as 'LOG'. 38
The method 'invokeContextCallback' has a Cyclomatic Complexity of 13. 40 - 63
Potential violation of Law of Demeter (object not created locally) 44
Potential violation of Law of Demeter (object not created locally) 44
Potential violation of Law of Demeter (object not created locally) 45
Potential violation of Law of Demeter (object not created locally) 46
Potential violation of Law of Demeter (method chain calls) 46
Potential violation of Law of Demeter (object not created locally) 48
Potential violation of Law of Demeter (method chain calls) 48
Deeply nested if..then statements are hard to read 49 - 58
Potential violation of Law of Demeter (method chain calls) 50
Potential violation of Law of Demeter (method chain calls) 52
Avoid empty if statements 53 - 55

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
Potential violation of Law of Demeter (method chain calls) 65
Potential violation of Law of Demeter (method chain calls) 65
Avoid if (x != y) ..; else ..; 73
Useless parentheses. 73
Potential violation of Law of Demeter (object not created locally) 93

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

Violation Line
This class has too many methods, consider refactoring it. 30 - 109
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 Cyclomatic Complexity of 3 (Highest = 14). 27 - 126
Document empty constructor 33 - 34
The method internalCompare() has an NPath complexity of 1600 49 - 89
The method 'internalCompare' has a Cyclomatic Complexity of 14. 49 - 89
Potential violation of Law of Demeter (method chain calls) 62
Potential violation of Law of Demeter (method chain calls) 63
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (method chain calls) 72
Potential violation of Law of Demeter (method chain calls) 74
Potential violation of Law of Demeter (object not created locally) 75
Avoid if (x != y) ..; else ..; 77 - 87
Potential violation of Law of Demeter (method chain calls) 81
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (method chain calls) 85
Potential violation of Law of Demeter (method chain calls) 116
Potential violation of Law of Demeter (method chain calls) 116
Avoid if (x != y) ..; else ..; 120
Useless parentheses. 120

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

Violation Line
High amount of different objects as members denotes a high coupling 20 - 1014
This class has a bunch of public methods and attributes 20 - 1014
A high number of imports can indicate a high degree of coupling within an object. 20 - 1014
The class 'ComponentUtils' has a Cyclomatic Complexity of 3 (Highest = 10). 76 - 1014
This class has too many methods, consider refactoring it. 76 - 1014
Possible God class (WMC=213, ATFD=135, TCC=4.2625745950554135E-4) 76 - 1014
Potential violation of Law of Demeter (object not created locally) 105
Potential violation of Law of Demeter (static property access) 105
Potential violation of Law of Demeter (object not created locally) 128
Avoid reassigning parameters such as 'component' 137
Potential violation of Law of Demeter (method chain calls) 150
Potential violation of Law of Demeter (object not created locally) 150
Avoid if (x != y) ..; else ..; 160 - 164
Potential violation of Law of Demeter (method chain calls) 163
Avoid if (x != y) ..; else ..; 169 - 179
Potential violation of Law of Demeter (method chain calls) 170
Potential violation of Law of Demeter (method chain calls) 174
Potential violation of Law of Demeter (method chain calls) 174
Avoid reassigning parameters such as 'component' 182
Avoid reassigning parameters such as 'component' 216
Avoid reassigning parameters such as 'component' 226
Potential violation of Law of Demeter (object not created locally) 303
Potential violation of Law of Demeter (method chain calls) 316
Potential violation of Law of Demeter (method chain calls) 332
Potential violation of Law of Demeter (method chain calls) 336
Potential violation of Law of Demeter (method chain calls) 337
Potential violation of Law of Demeter (method chain calls) 352
Potential violation of Law of Demeter (method chain calls) 356
Potential violation of Law of Demeter (method chain calls) 358
Potential violation of Law of Demeter (method chain calls) 364
Potential violation of Law of Demeter (method chain calls) 364
Potential violation of Law of Demeter (method chain calls) 382
Avoid reassigning parameters such as 'component' 388
Potential violation of Law of Demeter (object not created locally) 392
Potential violation of Law of Demeter (method chain calls) 402
Potential violation of Law of Demeter (object not created locally) 405
Potential violation of Law of Demeter (object not created locally) 408
Potential violation of Law of Demeter (object not created locally) 408
Potential violation of Law of Demeter (method chain calls) 408
Potential violation of Law of Demeter (object not created locally) 409
Potential violation of Law of Demeter (method chain calls) 418
Potential violation of Law of Demeter (object not created locally) 418
Potential violation of Law of Demeter (method chain calls) 444
Potential violation of Law of Demeter (method chain calls) 449
Potential violation of Law of Demeter (method chain calls) 459 - 461
Potential violation of Law of Demeter (object not created locally) 459
Potential violation of Law of Demeter (method chain calls) 459
Potential violation of Law of Demeter (method chain calls) 471
Potential violation of Law of Demeter (method chain calls) 471
Potential violation of Law of Demeter (static property access) 481
Potential violation of Law of Demeter (static property access) 489
Potential violation of Law of Demeter (method chain calls) 493
Potential violation of Law of Demeter (method chain calls) 495
Potential violation of Law of Demeter (method chain calls) 514
Potential violation of Law of Demeter (method chain calls) 516
Potential violation of Law of Demeter (method chain calls) 527 - 528
Potential violation of Law of Demeter (method chain calls) 527
Potential violation of Law of Demeter (object not created locally) 527
Potential violation of Law of Demeter (method chain calls) 534
Potential violation of Law of Demeter (object not created locally) 541
Potential violation of Law of Demeter (object not created locally) 541
Potential violation of Law of Demeter (method chain calls) 543
Potential violation of Law of Demeter (method chain calls) 543 - 544
Potential violation of Law of Demeter (object not created locally) 543
Potential violation of Law of Demeter (method chain calls) 550
Potential violation of Law of Demeter (method chain calls) 569
Potential violation of Law of Demeter (object not created locally) 571
Potential violation of Law of Demeter (object not created locally) 572
Potential violation of Law of Demeter (object not created locally) 572
Potential violation of Law of Demeter (object not created locally) 577
Potential violation of Law of Demeter (object not created locally) 577
Potential violation of Law of Demeter (object not created locally) 590
Potential violation of Law of Demeter (method chain calls) 590
Potential violation of Law of Demeter (method chain calls) 590
Potential violation of Law of Demeter (object not created locally) 591
Consider simply returning the value vs storing it in local variable 'myRenderer' 592
Potential violation of Law of Demeter (object not created locally) 600
Potential violation of Law of Demeter (object not created locally) 601
Potential violation of Law of Demeter (object not created locally) 603
Potential violation of Law of Demeter (object not created locally) 604
Potential violation of Law of Demeter (method chain calls) 614
Potential violation of Law of Demeter (method chain calls) 616
Potential violation of Law of Demeter (object not created locally) 618
Potential violation of Law of Demeter (object not created locally) 619
Potential violation of Law of Demeter (static property access) 657
Avoid reassigning parameters such as 'value' 660
Potential violation of Law of Demeter (method chain calls) 669
Potential violation of Law of Demeter (object not created locally) 669
Potential violation of Law of Demeter (object not created locally) 670
Potential violation of Law of Demeter (method chain calls) 693
Potential violation of Law of Demeter (object not created locally) 732
Potential violation of Law of Demeter (method chain calls) 736
Potential violation of Law of Demeter (object not created locally) 737
Potential violation of Law of Demeter (object not created locally) 738
Potential violation of Law of Demeter (object not created locally) 740
Potential violation of Law of Demeter (object not created locally) 741
Potential violation of Law of Demeter (object not created locally) 744
Potential violation of Law of Demeter (object not created locally) 746
Potential violation of Law of Demeter (method chain calls) 746
Potential violation of Law of Demeter (method chain calls) 749
Potential violation of Law of Demeter (static property access) 760
Potential violation of Law of Demeter (static property access) 768
Potential violation of Law of Demeter (static property access) 776
Potential violation of Law of Demeter (static property access) 784
Potential violation of Law of Demeter (static property access) 792
Potential violation of Law of Demeter (static property access) 800
Potential violation of Law of Demeter (method chain calls) 808
Potential violation of Law of Demeter (object not created locally) 810
Potential violation of Law of Demeter (method chain calls) 817
String.trim().length()==0 is an inefficient way to validate an empty String. 817
Potential violation of Law of Demeter (object not created locally) 825
The method 'getConvertedValue' has a Cyclomatic Complexity of 10. 852 - 883
Avoid if (x != y) ..; else ..; 856 - 878
Potential violation of Law of Demeter (method chain calls) 859
Potential violation of Law of Demeter (object not created locally) 861
Potential violation of Law of Demeter (method chain calls) 864
Potential violation of Law of Demeter (object not created locally) 869
Potential violation of Law of Demeter (method chain calls) 871
Potential violation of Law of Demeter (object not created locally) 876
Avoid reassigning parameters such as 'markup' 885
Avoid reassigning parameters such as 'markup' 885
Avoid reassigning parameters such as 'markup' 885
Avoid reassigning parameters such as 'markup' 885
Avoid reassigning parameters such as 'markup' 885
Potential violation of Law of Demeter (object not created locally) 901
Potential violation of Law of Demeter (static property access) 909
Potential violation of Law of Demeter (static property access) 911
Potential violation of Law of Demeter (static property access) 913
Potential violation of Law of Demeter (static property access) 915
Potential violation of Law of Demeter (method chain calls) 927
Potential violation of Law of Demeter (object not created locally) 930
Potential violation of Law of Demeter (method chain calls) 941
Potential violation of Law of Demeter (object not created locally) 945
Potential violation of Law of Demeter (object not created locally) 949
Potential violation of Law of Demeter (method chain calls) 949
Potential violation of Law of Demeter (object not created locally) 949
Potential violation of Law of Demeter (object not created locally) 950
Potential violation of Law of Demeter (method chain calls) 963
Potential violation of Law of Demeter (object not created locally) 966
Potential violation of Law of Demeter (method chain calls) 995
Potential violation of Law of Demeter (method chain calls) 1002

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

Violation Line
This class has too many methods, consider refactoring it. 41 - 177
Potential violation of Law of Demeter (method chain calls) 65
Potential violation of Law of Demeter (object not created locally) 66
Potential violation of Law of Demeter (object not created locally) 67
Potential violation of Law of Demeter (object not created locally) 83
Potential violation of Law of Demeter (method chain calls) 97
Potential violation of Law of Demeter (object not created locally) 97
Potential violation of Law of Demeter (method chain calls) 127
Avoid if (x != y) ..; else ..; 129 - 133
Potential violation of Law of Demeter (object not created locally) 130
Potential violation of Law of Demeter (object not created locally) 132
Potential violation of Law of Demeter (method chain calls) 144
Potential violation of Law of Demeter (method chain calls) 144
Potential violation of Law of Demeter (method chain calls) 152
Avoid if (x != y) ..; else ..; 154 - 159
Potential violation of Law of Demeter (object not created locally) 155
Potential violation of Law of Demeter (object not created locally) 158
Potential violation of Law of Demeter (method chain calls) 167 - 168
Potential violation of Law of Demeter (method chain calls) 168
Potential violation of Law of Demeter (method chain calls) 170
Potential violation of Law of Demeter (method chain calls) 173

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 Cyclomatic Complexity of 10 (Highest = 16). 39 - 154
The method 'afterPhase' has a Cyclomatic Complexity of 11. 45 - 91
Potential violation of Law of Demeter (method chain calls) 47
Potential violation of Law of Demeter (object not created locally) 48
Potential violation of Law of Demeter (object not created locally) 54
Potential violation of Law of Demeter (method chain calls) 54
Potential violation of Law of Demeter (method chain calls) 55
Potential violation of Law of Demeter (object not created locally) 55
Potential violation of Law of Demeter (method chain calls) 55
Potential violation of Law of Demeter (method chain calls) 62
Potential violation of Law of Demeter (object not created locally) 62
Potential violation of Law of Demeter (method chain calls) 62
Potential violation of Law of Demeter (method chain calls) 66
Deeply nested if..then statements are hard to read 67 - 72
Potential violation of Law of Demeter (method chain calls) 68
Potential violation of Law of Demeter (object not created locally) 68
Potential violation of Law of Demeter (object not created locally) 69
Potential violation of Law of Demeter (object not created locally) 70
Potential violation of Law of Demeter (object not created locally) 71
Potential violation of Law of Demeter (object not created locally) 74
Deeply nested if..then statements are hard to read 75 - 77
Potential violation of Law of Demeter (object not created locally) 79
Potential violation of Law of Demeter (object not created locally) 82
Potential violation of Law of Demeter (object not created locally) 87
Potential violation of Law of Demeter (object not created locally) 87
Potential violation of Law of Demeter (object not created locally) 87
The method 'beforePhase' has a Cyclomatic Complexity of 16. 93 - 149
The method beforePhase() has an NPath complexity of 572 93 - 149
Avoid declaring a variable if it is unreferenced before a possible exit point. 95
Potential violation of Law of Demeter (method chain calls) 95
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (object not created locally) 107
Potential violation of Law of Demeter (object not created locally) 109
Deeply nested if..then statements are hard to read 111 - 127
Potential violation of Law of Demeter (object not created locally) 114
Potential violation of Law of Demeter (object not created locally) 117
Potential violation of Law of Demeter (object not created locally) 119
Potential violation of Law of Demeter (object not created locally) 120
Potential violation of Law of Demeter (object not created locally) 121
Potential violation of Law of Demeter (object not created locally) 123
Potential violation of Law of Demeter (object not created locally) 124
Potential violation of Law of Demeter (object not created locally) 125
Potential violation of Law of Demeter (object not created locally) 130
Potential violation of Law of Demeter (object not created locally) 131
Potential violation of Law of Demeter (object not created locally) 131
Deeply nested if..then statements are hard to read 133 - 143
Potential violation of Law of Demeter (object not created locally) 135
Potential violation of Law of Demeter (object not created locally) 137
Potential violation of Law of Demeter (object not created locally) 140
Deeply nested if..then statements are hard to read 144 - 146

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

Violation Line
Substitute calls to size() == 0 (or size() != 0) with calls to isEmpty() 59
Potential violation of Law of Demeter (object not created locally) 59
Potential violation of Law of Demeter (object not created locally) 60
Potential violation of Law of Demeter (object not created locally) 71
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (method chain calls) 79
Avoid appending characters as strings in StringBuffer.append. 82
Avoid appending characters as strings in StringBuffer.append. 84
Potential violation of Law of Demeter (method chain calls) 90
Potential violation of Law of Demeter (method chain calls) 90
Potential violation of Law of Demeter (method chain calls) 105

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 41
Potential violation of Law of Demeter (method chain calls) 42
Potential violation of Law of Demeter (object not created locally) 42
Potential violation of Law of Demeter (object not created locally) 44
Potential violation of Law of Demeter (method chain calls) 44
Deeply nested if..then statements are hard to read 45 - 47
Potential violation of Law of Demeter (object not created locally) 53
Potential violation of Law of Demeter (object not created locally) 54
Potential violation of Law of Demeter (object not created locally) 55
Potential violation of Law of Demeter (object not created locally) 56
Potential violation of Law of Demeter (object not created locally) 57
Potential violation of Law of Demeter (object not created locally) 58
Potential violation of Law of Demeter (object not created locally) 60
Potential violation of Law of Demeter (object not created locally) 61

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

Violation Line
Avoid empty catch blocks 48 - 50
Potential violation of Law of Demeter (method chain calls) 60
Potential violation of Law of Demeter (method chain calls) 60
Potential violation of Law of Demeter (method chain calls) 65
Potential violation of Law of Demeter (method chain calls) 65
Avoid empty catch blocks 67 - 69
Avoid empty catch blocks 71 - 73

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 43
Potential violation of Law of Demeter (method chain calls) 54

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

Violation Line
The class 'LayoutInfo' has a Cyclomatic Complexity of 4 (Highest = 13). 39 - 390
This class has too many methods, consider refactoring it. 39 - 390
The String literal " clientId='" appears 17 times in this file; the first occurrence is on line 73 73
Avoid reassigning parameters such as 'space' 108
The method 'update' has a Cyclomatic Complexity of 10. 108 - 143
Avoid reassigning parameters such as 'space' 108
These nested if statements could be combined 131 - 137
Potential violation of Law of Demeter (method chain calls) 166
Consider using varargs for methods or constructors which take an array the last parameter. 171
Potential violation of Law of Demeter (method chain calls) 174
Consider using varargs for methods or constructors which take an array the last parameter. 179
Avoid appending characters as strings in StringBuffer.append. 183
Avoid appending characters as strings in StringBuffer.append. 188
The method 'handleSpaceLeft' has a Cyclomatic Complexity of 13. 216 - 250
The method handleSpaceLeft() has an NPath complexity of 330 216 - 250
Consider using varargs for methods or constructors which take an array the last parameter. 253
These nested if statements could be combined 279 - 301
Potential violation of Law of Demeter (method chain calls) 284
The String literal " from " appears 5 times in this file; the first occurrence is on line 289 289
The String literal "set column " appears 5 times in this file; the first occurrence is on line 289 289
The String literal " to with " appears 4 times in this file; the first occurrence is on line 290 290
The method 'parseColumnLayout' has a Cyclomatic Complexity of 12. 339 - 379
Potential violation of Law of Demeter (method chain calls) 353
Potential violation of Law of Demeter (method chain calls) 360
Potential violation of Law of Demeter (method chain calls) 362

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 61
Potential violation of Law of Demeter (object not created locally) 63
Potential violation of Law of Demeter (object not created locally) 64
Potential violation of Law of Demeter (object not created locally) 65
Potential violation of Law of Demeter (object not created locally) 85
Potential violation of Law of Demeter (object not created locally) 87
Potential violation of Law of Demeter (object not created locally) 88
Potential violation of Law of Demeter (object not created locally) 89
Potential violation of Law of Demeter (object not created locally) 92

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

Violation Line
The class 'MessageUtils' has a Cyclomatic Complexity of 4 (Highest = 15). 43 - 160
Consider using varargs for methods or constructors which take an array the last parameter. 52
Potential violation of Law of Demeter (method chain calls) 53 - 54
Potential violation of Law of Demeter (method chain calls) 54
Potential violation of Law of Demeter (method chain calls) 54
The method getMessage() has an NPath complexity of 468 57 - 104
The method 'getMessage' has a Cyclomatic Complexity of 15. 57 - 104
Deeply nested if..then statements are hard to read 76 - 78
Deeply nested if..then statements are hard to read 79 - 81
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (object not created locally) 95
Potential violation of Law of Demeter (object not created locally) 100
Potential violation of Law of Demeter (method chain calls) 115
Avoid if (x != y) ..; else ..; 116
Potential violation of Law of Demeter (method chain calls) 126
Potential violation of Law of Demeter (method chain calls) 126
Potential violation of Law of Demeter (method chain calls) 129
Potential violation of Law of Demeter (method chain calls) 129
Potential violation of Law of Demeter (method chain calls) 131
Potential violation of Law of Demeter (method chain calls) 138
Potential violation of Law of Demeter (object not created locally) 140
Potential violation of Law of Demeter (object not created locally) 144

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 31
Potential violation of Law of Demeter (method chain calls) 31
Potential violation of Law of Demeter (method chain calls) 32
Potential violation of Law of Demeter (method chain calls) 32
Potential violation of Law of Demeter (method chain calls) 32
Potential violation of Law of Demeter (method chain calls) 32

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 39
Potential violation of Law of Demeter (object not created locally) 43
Potential violation of Law of Demeter (object not created locally) 44
Potential violation of Law of Demeter (object not created locally) 48
Potential violation of Law of Demeter (object not created locally) 49
Do not add empty strings 53

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 31
Potential violation of Law of Demeter (method chain calls) 31
Potential violation of Law of Demeter (method chain calls) 32
Potential violation of Law of Demeter (method chain calls) 32
Potential violation of Law of Demeter (method chain calls) 32
Potential violation of Law of Demeter (method chain calls) 32

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

Violation Line
The class 'ValueBindingComparator' has a Cyclomatic Complexity of 3 (Highest = 11). 34 - 130
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 Cyclomatic Complexity of 11. 76 - 100
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
Useless parentheses. 104
Avoid if (x != y) ..; else ..; 105
Avoid if (x != y) ..; else ..; 106
Potential violation of Law of Demeter (method chain calls) 117
Potential violation of Law of Demeter (object not created locally) 118
Potential violation of Law of Demeter (object not created locally) 120
Potential violation of Law of Demeter (object not created locally) 124

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

Violation Line
The class 'ValueExpressionComparator' has a Cyclomatic Complexity of 3 (Highest = 11). 30 - 132
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 Cyclomatic Complexity of 11. 74 - 98
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
Useless parentheses. 102
Avoid if (x != y) ..; else ..; 102
Avoid if (x != y) ..; else ..; 103
Avoid if (x != y) ..; else ..; 104
Potential violation of Law of Demeter (method chain calls) 120
Potential violation of Law of Demeter (object not created locally) 121
Potential violation of Law of Demeter (object not created locally) 123
Potential violation of Law of Demeter (object not created locally) 127

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 38
Potential violation of Law of Demeter (method chain calls) 38
Potential violation of Law of Demeter (object not created locally) 56
Potential violation of Law of Demeter (method chain calls) 56

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 65
Potential violation of Law of Demeter (method chain calls) 113 - 114
Potential violation of Law of Demeter (object not created locally) 113 - 114
Potential violation of Law of Demeter (method chain calls) 113 - 114
Potential violation of Law of Demeter (method chain calls) 114
Potential violation of Law of Demeter (object not created locally) 114
Potential violation of Law of Demeter (object not created locally) 121
Potential violation of Law of Demeter (object not created locally) 122
Potential violation of Law of Demeter (object not created locally) 123
Potential violation of Law of Demeter (object not created locally) 124
Potential violation of Law of Demeter (object not created locally) 126
Potential violation of Law of Demeter (object not created locally) 129
Potential violation of Law of Demeter (object not created locally) 137
Potential violation of Law of Demeter (method chain calls) 139
Potential violation of Law of Demeter (object not created locally) 139
Potential violation of Law of Demeter (method chain calls) 139
Potential violation of Law of Demeter (object not created locally) 141
Potential violation of Law of Demeter (object not created locally) 144
Potential violation of Law of Demeter (object not created locally) 145
Potential violation of Law of Demeter (object not created locally) 146
Potential violation of Law of Demeter (object not created locally) 147

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 58
Potential violation of Law of Demeter (object not created locally) 59
Potential violation of Law of Demeter (object not created locally) 61
Potential violation of Law of Demeter (method chain calls) 71
Potential violation of Law of Demeter (object not created locally) 71

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

Violation Line
The class 'FileItemValidator' has a Cyclomatic Complexity of 2 (Highest = 12). 40 - 127
Avoid using redundant field initializer for 'maxSize' 44
Document empty constructor 51 - 52
The method 'validate' has a Cyclomatic Complexity of 12. 54 - 87
Potential violation of Law of Demeter (object not created locally) 58
Potential violation of Law of Demeter (method chain calls) 59 - 61
Potential violation of Law of Demeter (method chain calls) 60
Potential violation of Law of Demeter (object not created locally) 65
Potential violation of Law of Demeter (method chain calls) 68
Deeply nested if..then statements are hard to read 68 - 71
Potential violation of Law of Demeter (object not created locally) 68
Deeply nested if..then statements are hard to read 75 - 79
Potential violation of Law of Demeter (method chain calls) 80 - 82
Potential violation of Law of Demeter (method chain calls) 81
Consider using varargs for methods or constructors which take an array the last parameter. 103

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

Violation Line
Document empty constructor 44 - 45
Overridable method 'setMaximum' called during object construction 48
Overridable method 'setMaximum' called during object construction 52
Overridable method 'setMinimum' called during object construction 53
Avoid if (x != y) ..; else ..; 57
Avoid if (x != y) ..; else ..; 67
Potential violation of Law of Demeter (method chain calls) 79
Potential violation of Law of Demeter (method chain calls) 79
Potential violation of Law of Demeter (object not created locally) 80
Potential violation of Law of Demeter (method chain calls) 82 - 83
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (object not created locally) 86
Potential violation of Law of Demeter (method chain calls) 88 - 89
Potential violation of Law of Demeter (method chain calls) 89
Avoid if (x != y) ..; else ..; 121
Avoid if (x != y) ..; else ..; 124
Avoid if (x != y) ..; else ..; 133
Useless parentheses. 133
Avoid if (x != y) ..; else ..; 134

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 42
Potential violation of Law of Demeter (object not created locally) 43
Potential violation of Law of Demeter (method chain calls) 46
Potential violation of Law of Demeter (method chain calls) 49
Potential violation of Law of Demeter (object not created locally) 52
Potential violation of Law of Demeter (object not created locally) 54
New exception is thrown in catch block, original stack trace may be lost 58
Potential violation of Law of Demeter (object not created locally) 60

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

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 57
Private field 'secret' could be made final; it is only initialized in the declaration or constructor. 57
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. 69
Potential violation of Law of Demeter (method chain calls) 82
Potential violation of Law of Demeter (object not created locally) 83
Avoid declaring a variable if it is unreferenced before a possible exit point. 83
Potential violation of Law of Demeter (method chain calls) 84
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (method chain calls) 89
Potential violation of Law of Demeter (static property access) 93
Potential violation of Law of Demeter (method chain calls) 104
Potential violation of Law of Demeter (method chain calls) 107
Potential violation of Law of Demeter (method chain calls) 109

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

Violation Line
Potential violation of Law of Demeter (method chain calls) 31
Potential violation of Law of Demeter (method chain calls) 31
Potential violation of Law of Demeter (method chain calls) 31
Document empty method 36 - 37

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

Violation Line
Avoid if (x != y) ..; else ..; 80 - 86
Potential violation of Law of Demeter (object not created locally) 80
Potential violation of Law of Demeter (method chain calls) 81
Potential violation of Law of Demeter (method chain calls) 81
Potential violation of Law of Demeter (object not created locally) 82
Avoid if (x != y) ..; else ..; 82 - 86
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (method chain calls) 83
Potential violation of Law of Demeter (method chain calls) 92
Potential violation of Law of Demeter (method chain calls) 92
Potential violation of Law of Demeter (method chain calls) 92
Potential violation of Law of Demeter (method chain calls) 93
Potential violation of Law of Demeter (method chain calls) 93
Potential violation of Law of Demeter (method chain calls) 93
Potential violation of Law of Demeter (method chain calls) 107
Potential violation of Law of Demeter (object not created locally) 107
Potential violation of Law of Demeter (method chain calls) 109
Potential violation of Law of Demeter (method chain calls) 109 - 111
Document empty method 127 - 128

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

Violation Line
This class has too many methods, consider refactoring it. 40 - 237
Potential violation of Law of Demeter (object not created locally) 159

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

Violation Line
Potential violation of Law of Demeter (object not created locally) 45
Potential violation of Law of Demeter (method chain calls) 54
Potential violation of Law of Demeter (object not created locally) 54
StringBuffer constructor is initialized with size 16, but has at least 87 characters appended. 55
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 56
StringBuffer (or StringBuilder).append is called 3 consecutive times with literal Strings. Use a single append with a single combined String. 60
Potential violation of Law of Demeter (object not created locally) 79

Tobago Deprecation - org/apache/myfaces/tobago/ajax/api/AjaxUtils.java

Violation Line
This class has too many methods, consider refactoring it. 36 - 160
A class which only has private constructors should be final 36 - 160
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 45
Potential violation of Law of Demeter (static property access) 86
Potential violation of Law of Demeter (static property access) 94
Potential violation of Law of Demeter (static property access) 102
Potential violation of Law of Demeter (static property access) 110
Potential violation of Law of Demeter (static property access) 142
Potential violation of Law of Demeter (static property access) 150
Potential violation of Law of Demeter (static property access) 158

Tobago Deprecation - org/apache/myfaces/tobago/application/LabelValueBindingFacesMessage.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 62
Potential violation of Law of Demeter (method chain calls) 62
Potential violation of Law of Demeter (method chain calls) 62
Potential violation of Law of Demeter (method chain calls) 62
Potential violation of Law of Demeter (object not created locally) 64
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (object not created locally) 65
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (object not created locally) 78
Potential violation of Law of Demeter (method chain calls) 78
Potential violation of Law of Demeter (object not created locally) 79

Tobago Deprecation - org/apache/myfaces/tobago/bean/ResourceMap.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 49
Potential violation of Law of Demeter (method chain calls) 49
Potential violation of Law of Demeter (method chain calls) 70

Tobago Deprecation - org/apache/myfaces/tobago/compat/FacesInvokeOnComponent12.java

Violation Line
A class which only has private constructors should be final 33 - 83
Potential violation of Law of Demeter (object not created locally) 55
Useless parentheses. 55
These nested if statements could be combined 56 - 58
Potential violation of Law of Demeter (object not created locally) 77

Tobago Deprecation - org/apache/myfaces/tobago/compat/FacesUtils.java

Violation Line
A class which only has private constructors should be final 42 - 245
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 51

Tobago Deprecation - org/apache/myfaces/tobago/component/ComponentUtil.java

Violation Line
This class has a bunch of public methods and attributes 20 - 743
This class has too many methods, consider refactoring it. 56 - 743
Possible God class (WMC=86, ATFD=55, TCC=0.0) 56 - 743
A class which only has private constructors should be final 56 - 743
Unnecessary use of fully qualified name 'javax.faces.component.UICommand' due to existing import 'javax.faces.component.UICommand' 98
Consider using varargs for methods or constructors which take an array the last parameter. 146
Potential violation of Law of Demeter (method chain calls) 147
Potential violation of Law of Demeter (method chain calls) 147
Consider using varargs for methods or constructors which take an array the last parameter. 154
Potential violation of Law of Demeter (method chain calls) 155
Potential violation of Law of Demeter (method chain calls) 155
Consider using varargs for methods or constructors which take an array the last parameter. 162
Potential violation of Law of Demeter (method chain calls) 163
Potential violation of Law of Demeter (method chain calls) 163
Potential violation of Law of Demeter (static property access) 413
Avoid instantiating Integer objects. Call Integer.valueOf() instead. 450
Potential violation of Law of Demeter (method chain calls) 464
Potential violation of Law of Demeter (method chain calls) 478
Avoid reassigning parameters such as 'script' 503
Potential violation of Law of Demeter (object not created locally) 506
Potential violation of Law of Demeter (object not created locally) 713
Potential violation of Law of Demeter (object not created locally) 719

Tobago Deprecation - org/apache/myfaces/tobago/component/UIViewRoot.java

Violation Line
The class 'UIViewRoot' has a Cyclomatic Complexity of 5 (Highest = 12). 57 - 341
Possible God class (WMC=55, ATFD=34, TCC=0.06060606060606061) 57 - 341
Potential violation of Law of Demeter (object not created locally) 74
Potential violation of Law of Demeter (object not created locally) 75
The method 'broadcastForPhase' has a Cyclomatic Complexity of 12. 147 - 199
Potential violation of Law of Demeter (object not created locally) 157
Potential violation of Law of Demeter (method chain calls) 157
Potential violation of Law of Demeter (static property access) 158
Potential violation of Law of Demeter (object not created locally) 159
Potential violation of Law of Demeter (object not created locally) 161
Deeply nested if..then statements are hard to read 166 - 168
Potential violation of Law of Demeter (object not created locally) 175
Avoid if (x != y) ..; else ..; 177 - 181
The String literal "context" appears 4 times in this file; the first occurrence is on line 210 210
Avoid if (x != y) ..; else ..; 213 - 231
Potential violation of Law of Demeter (object not created locally) 216
Potential violation of Law of Demeter (object not created locally) 217
Potential violation of Law of Demeter (object not created locally) 225
The method 'decodeActionComponent' has a Cyclomatic Complexity of 10. 238 - 267
Potential violation of Law of Demeter (object not created locally) 245
Avoid empty catch blocks 249 - 251
Use equals() to compare object references. 260
Potential violation of Law of Demeter (object not created locally) 263
Avoid if (x != y) ..; else ..; 277 - 284
Potential violation of Law of Demeter (object not created locally) 278
Avoid if (x != y) ..; else ..; 297 - 303
Potential violation of Law of Demeter (object not created locally) 298

Tobago Deprecation - org/apache/myfaces/tobago/config/ThemeConfig.java

Violation Line
A class which only has private constructors should be final 29 - 53
Potential violation of Law of Demeter (static property access) 39
Potential violation of Law of Demeter (method chain calls) 40 - 41
Potential violation of Law of Demeter (method chain calls) 40 - 41
Potential violation of Law of Demeter (static property access) 49
Potential violation of Law of Demeter (method chain calls) 50 - 51

Tobago Deprecation - org/apache/myfaces/tobago/context/FacesContextFactoryWrapper.java

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

Tobago Deprecation - org/apache/myfaces/tobago/context/FacesContextWrapper.java

Violation Line
Unnecessary use of fully qualified name 'javax.faces.context.FacesContext' due to existing import 'javax.faces.context.FacesContext' 36
Private field 'context' could be made final; it is only initialized in the declaration or constructor. 37

Tobago Deprecation - org/apache/myfaces/tobago/context/ResourceManagerFactory.java

Violation Line
A class which only has private constructors should be final 32 - 76
Potential violation of Law of Demeter (static property access) 49
Potential violation of Law of Demeter (static property access) 57
Potential violation of Law of Demeter (static property access) 66
Potential violation of Law of Demeter (static property access) 74

Tobago Deprecation - org/apache/myfaces/tobago/context/ResourceManagerUtil.java

Violation Line
A class which only has private constructors should be final 33 - 200
Potential violation of Law of Demeter (method chain calls) 44
Potential violation of Law of Demeter (method chain calls) 53
Potential violation of Law of Demeter (method chain calls) 66
Potential violation of Law of Demeter (method chain calls) 67
Potential violation of Law of Demeter (method chain calls) 77
Potential violation of Law of Demeter (method chain calls) 81
Potential violation of Law of Demeter (method chain calls) 90
Potential violation of Law of Demeter (method chain calls) 91
Potential violation of Law of Demeter (method chain calls) 112
Potential violation of Law of Demeter (method chain calls) 113
Consider using varargs for methods or constructors which take an array the last parameter. 121
Consider using varargs for methods or constructors which take an array the last parameter. 133
Potential violation of Law of Demeter (method chain calls) 180

Tobago Deprecation - org/apache/myfaces/tobago/context/TobagoFacesContext.java

Violation Line
Private field 'scriptFiles' could be made final; it is only initialized in the declaration or constructor. 40
Private field 'scriptBlocks' could be made final; it is only initialized in the declaration or constructor. 42
Private field 'styleFiles' could be made final; it is only initialized in the declaration or constructor. 44
Private field 'styleBlocks' could be made final; it is only initialized in the declaration or constructor. 46
Private field 'onloadScripts' could be made final; it is only initialized in the declaration or constructor. 48
Private field 'onunloadScripts' could be made final; it is only initialized in the declaration or constructor. 50
Private field 'onexitScripts' could be made final; it is only initialized in the declaration or constructor. 52
Private field 'onsubmitScripts' could be made final; it is only initialized in the declaration or constructor. 54
Private field 'popups' could be made final; it is only initialized in the declaration or constructor. 56
Potential violation of Law of Demeter (method chain calls) 140
Potential violation of Law of Demeter (method chain calls) 141
Potential violation of Law of Demeter (method chain calls) 142
Potential violation of Law of Demeter (method chain calls) 143
Potential violation of Law of Demeter (method chain calls) 144
Potential violation of Law of Demeter (method chain calls) 149

Tobago Deprecation - org/apache/myfaces/tobago/event/PageActionUtil.java

Violation Line
A class which only has private constructors should be final 29 - 52
Avoid if (x != y) ..; else ..; 46 - 50

Tobago Deprecation - org/apache/myfaces/tobago/event/ValueBindingPopupActionListener.java

Violation Line
Document empty constructor 47 - 48
Potential violation of Law of Demeter (method chain calls) 61 - 64
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (method chain calls) 75

Tobago Deprecation - org/apache/myfaces/tobago/event/ValueBindingResetInputActionListener.java

Violation Line
Document empty constructor 48 - 49
Potential violation of Law of Demeter (method chain calls) 80

Tobago Deprecation - org/apache/myfaces/tobago/event/ValueBindingTabChangeListener.java

Violation Line
Document empty constructor 39 - 40
Avoid if (x != y) ..; else ..; 49 - 53
Potential violation of Law of Demeter (object not created locally) 50
Potential violation of Law of Demeter (method chain calls) 65
No need to check for null before an instanceof 82
Potential violation of Law of Demeter (method chain calls) 97
Potential violation of Law of Demeter (method chain calls) 97
Potential violation of Law of Demeter (object not created locally) 98
Potential violation of Law of Demeter (object not created locally) 98

Tobago Deprecation - org/apache/myfaces/tobago/internal/application/ViewHandlerImpl.java

Violation Line
Private field 'base' could be made final; it is only initialized in the declaration or constructor. 44
Consider simply returning the value vs storing it in local variable 'viewRoot' 102

Tobago Deprecation - org/apache/myfaces/tobago/internal/lifecycle/ApplyRequestValuesExecutor.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 38

Tobago Deprecation - org/apache/myfaces/tobago/internal/lifecycle/InvokeApplicationExecutor.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 37

Tobago Deprecation - org/apache/myfaces/tobago/internal/lifecycle/PhaseListenerManager.java

Violation Line
Private field 'lifecycle' could be made final; it is only initialized in the declaration or constructor. 47
Private field 'facesContext' could be made final; it is only initialized in the declaration or constructor. 48
Private field 'phaseListeners' could be made final; it is only initialized in the declaration or constructor. 49
Private field 'listenerSuccessMap' could be made final; it is only initialized in the declaration or constructor. 54
Consider using varargs for methods or constructors which take an array the last parameter. 60
Potential violation of Law of Demeter (method chain calls) 67
Useless parentheses. 68 - 69
Potential violation of Law of Demeter (static property access) 68
Potential violation of Law of Demeter (object not created locally) 85
Avoid declaring a variable if it is unreferenced before a possible exit point. 97
Potential violation of Law of Demeter (object not created locally) 109

Tobago Deprecation - org/apache/myfaces/tobago/internal/lifecycle/ProcessValidationsExecutor.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 37

Tobago Deprecation - org/apache/myfaces/tobago/internal/lifecycle/RenderResponseExecutor.java

Violation Line
Potential violation of Law of Demeter (object not created locally) 45
Avoid if (x != y) ..; else ..; 49 - 58
Potential violation of Law of Demeter (object not created locally) 49
Potential violation of Law of Demeter (object not created locally) 50
Potential violation of Law of Demeter (method chain calls) 52
Potential violation of Law of Demeter (object not created locally) 55

Tobago Deprecation - org/apache/myfaces/tobago/internal/lifecycle/RestoreViewExecutor.java

Violation Line
The class 'RestoreViewExecutor' has a Cyclomatic Complexity of 6 (Highest = 16). 57 - 257
The method 'execute' has a Cyclomatic Complexity of 16. 61 - 143
The method execute() has an NPath complexity of 3360 61 - 143
Potential violation of Law of Demeter (object not created locally) 64
Potential violation of Law of Demeter (object not created locally) 65
Potential violation of Law of Demeter (object not created locally) 68
Potential violation of Law of Demeter (object not created locally) 70
Potential violation of Law of Demeter (method chain calls) 73
Potential violation of Law of Demeter (method chain calls) 73
Potential violation of Law of Demeter (object not created locally) 76
Potential violation of Law of Demeter (object not created locally) 77
Potential violation of Law of Demeter (method chain calls) 77
Potential violation of Law of Demeter (object not created locally) 77
Potential violation of Law of Demeter (method chain calls) 78
Potential violation of Law of Demeter (method chain calls) 78
Potential violation of Law of Demeter (object not created locally) 78
Potential violation of Law of Demeter (method chain calls) 85
Potential violation of Law of Demeter (method chain calls) 85
Potential violation of Law of Demeter (method chain calls) 85
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (object not created locally) 100
Potential violation of Law of Demeter (method chain calls) 100
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (object not created locally) 102
Potential violation of Law of Demeter (object not created locally) 112
Potential violation of Law of Demeter (object not created locally) 116
Potential violation of Law of Demeter (object not created locally) 119
Potential violation of Law of Demeter (object not created locally) 120
Potential violation of Law of Demeter (method chain calls) 141
Potential violation of Law of Demeter (method chain calls) 141
Potential violation of Law of Demeter (method chain calls) 146
Potential violation of Law of Demeter (object not created locally) 147
Potential violation of Law of Demeter (method chain calls) 151
Potential violation of Law of Demeter (object not created locally) 171
Potential violation of Law of Demeter (object not created locally) 174
Potential violation of Law of Demeter (object not created locally) 183
Avoid if (x != y) ..; else ..; 184
Potential violation of Law of Demeter (object not created locally) 185
Potential violation of Law of Demeter (object not created locally) 191
Potential violation of Law of Demeter (object not created locally) 196
Avoid if (x != y) ..; else ..; 211 - 233
Potential violation of Law of Demeter (object not created locally) 222
Potential violation of Law of Demeter (object not created locally) 223
Potential violation of Law of Demeter (object not created locally) 224
Potential violation of Law of Demeter (method chain calls) 228
Potential violation of Law of Demeter (method chain calls) 242
Potential violation of Law of Demeter (method chain calls) 245
Avoid instantiating new objects inside loops 247
Avoid empty catch blocks 248 - 250

Tobago Deprecation - org/apache/myfaces/tobago/internal/lifecycle/TobagoLifecycle.java

Violation Line
Private field 'lifecycleExecutors' could be made final; it is only initialized in the declaration or constructor. 50
Private field 'renderExecutor' could be made final; it is only initialized in the declaration or constructor. 51
Avoid using redundant field initializer for 'phaseListenerArray' 58
Potential violation of Law of Demeter (method chain calls) 94
The String literal " in " appears 4 times in this file; the first occurrence is on line 94 94
Potential violation of Law of Demeter (method chain calls) 94
Potential violation of Law of Demeter (method chain calls) 123
Potential violation of Law of Demeter (method chain calls) 123
Potential violation of Law of Demeter (method chain calls) 135
Potential violation of Law of Demeter (method chain calls) 135
Potential violation of Law of Demeter (method chain calls) 153
Potential violation of Law of Demeter (method chain calls) 153

Tobago Deprecation - org/apache/myfaces/tobago/internal/lifecycle/TobagoLifecycleFactory.java

Violation Line
Avoid unused private fields such as 'LOG'. 38
Private field 'factory' could be made final; it is only initialized in the declaration or constructor. 40
Private field 'defaultLifecycle' could be made final; it is only initialized in the declaration or constructor. 41
Do not add empty strings 51
Potential violation of Law of Demeter (method chain calls) 51
Potential violation of Law of Demeter (static property access) 51
Potential violation of Law of Demeter (method chain calls) 51
Potential violation of Law of Demeter (static property access) 55
Potential violation of Law of Demeter (static property access) 56
Potential violation of Law of Demeter (static property access) 59
Potential violation of Law of Demeter (static property access) 65

Tobago Deprecation - org/apache/myfaces/tobago/internal/lifecycle/UpdateModelValuesExecutor.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 38

Tobago Deprecation - org/apache/myfaces/tobago/internal/taglib/TagUtils.java

Violation Line
A class which only has private constructors should be final 52 - 323
Possible God class (WMC=60, ATFD=24, TCC=0.0) 52 - 323
Avoid reassigning parameters such as 'value' 62
Potential violation of Law of Demeter (static property access) 70
Potential violation of Law of Demeter (static property access) 70
Potential violation of Law of Demeter (static property access) 70
Deeply nested if..then statements are hard to read 71 - 73
These nested if statements could be combined 71 - 73
Avoid instantiating Integer objects. Call Integer.valueOf() instead. 75
Potential violation of Law of Demeter (method chain calls) 89
Potential violation of Law of Demeter (method chain calls) 103
Potential violation of Law of Demeter (object not created locally) 115
Potential violation of Law of Demeter (object not created locally) 117
Potential violation of Law of Demeter (object not created locally) 120
Potential violation of Law of Demeter (method chain calls) 121
Potential violation of Law of Demeter (method chain calls) 155
Potential violation of Law of Demeter (method chain calls) 155
Consider using varargs for methods or constructors which take an array the last parameter. 163
Potential violation of Law of Demeter (method chain calls) 165 - 166
Potential violation of Law of Demeter (method chain calls) 165 - 166
Potential violation of Law of Demeter (method chain calls) 167
Consider using varargs for methods or constructors which take an array the last parameter. 176
Potential violation of Law of Demeter (method chain calls) 178 - 179
Potential violation of Law of Demeter (method chain calls) 178 - 179
Potential violation of Law of Demeter (method chain calls) 180
Consider using varargs for methods or constructors which take an array the last parameter. 189
Potential violation of Law of Demeter (method chain calls) 191 - 192
Potential violation of Law of Demeter (method chain calls) 191 - 192
Potential violation of Law of Demeter (method chain calls) 193
Consider using varargs for methods or constructors which take an array the last parameter. 202
Potential violation of Law of Demeter (method chain calls) 204 - 205
Potential violation of Law of Demeter (method chain calls) 204 - 205
Potential violation of Law of Demeter (method chain calls) 206
Consider using varargs for methods or constructors which take an array the last parameter. 214
Potential violation of Law of Demeter (method chain calls) 216 - 217
Potential violation of Law of Demeter (method chain calls) 216 - 217
Potential violation of Law of Demeter (method chain calls) 218
Consider using varargs for methods or constructors which take an array the last parameter. 227
Potential violation of Law of Demeter (method chain calls) 229 - 230
Potential violation of Law of Demeter (method chain calls) 229 - 230
Potential violation of Law of Demeter (method chain calls) 231
Consider using varargs for methods or constructors which take an array the last parameter. 239
Potential violation of Law of Demeter (method chain calls) 243
Potential violation of Law of Demeter (method chain calls) 243
Potential violation of Law of Demeter (method chain calls) 244
Consider using varargs for methods or constructors which take an array the last parameter. 256
Potential violation of Law of Demeter (method chain calls) 258 - 259
Potential violation of Law of Demeter (method chain calls) 258 - 259
Potential violation of Law of Demeter (method chain calls) 260
Potential violation of Law of Demeter (method chain calls) 307
Potential violation of Law of Demeter (static property access) 307
Avoid reassigning parameters such as 'script' 316
Potential violation of Law of Demeter (object not created locally) 319

Tobago Deprecation - org/apache/myfaces/tobago/model/DefaultMutableTreeNodeAnalyser.java

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

Tobago Deprecation - org/apache/myfaces/tobago/renderkit/RenderUtil.java

Violation Line
A class which only has private constructors should be final 37 - 134
The String literal "Using deprecated API" appears 12 times in this file; the first occurrence is on line 46 46
Potential violation of Law of Demeter (static property access) 46
Potential violation of Law of Demeter (static property access) 59
Potential violation of Law of Demeter (static property access) 65
Potential violation of Law of Demeter (static property access) 72
Potential violation of Law of Demeter (static property access) 78
Potential violation of Law of Demeter (static property access) 83
Potential violation of Law of Demeter (static property access) 88
Potential violation of Law of Demeter (static property access) 96
Potential violation of Law of Demeter (static property access) 102
Potential violation of Law of Demeter (static property access) 108
Potential violation of Law of Demeter (static property access) 121
Potential violation of Law of Demeter (static property access) 126
Potential violation of Law of Demeter (static property access) 131

Tobago Deprecation - org/apache/myfaces/tobago/renderkit/TobagoResponseStateManager.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 56
Potential violation of Law of Demeter (object not created locally) 57
Potential violation of Law of Demeter (object not created locally) 58
Potential violation of Law of Demeter (object not created locally) 63
Potential violation of Law of Demeter (method chain calls) 68
Potential violation of Law of Demeter (object not created locally) 69
Avoid if (x != y) ..; else ..; 78 - 91
Potential violation of Law of Demeter (object not created locally) 80
Potential violation of Law of Demeter (object not created locally) 81
Potential violation of Law of Demeter (object not created locally) 82
Potential violation of Law of Demeter (object not created locally) 83
Potential violation of Law of Demeter (object not created locally) 84
Potential violation of Law of Demeter (object not created locally) 85
Potential violation of Law of Demeter (object not created locally) 93
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 95
Potential violation of Law of Demeter (object not created locally) 96
Avoid if (x != y) ..; else ..; 97 - 108
Potential violation of Law of Demeter (object not created locally) 99
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (object not created locally) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (object not created locally) 104
Potential violation of Law of Demeter (object not created locally) 109
Potential violation of Law of Demeter (object not created locally) 111
Potential violation of Law of Demeter (object not created locally) 112
Potential violation of Law of Demeter (object not created locally) 113
Potential violation of Law of Demeter (object not created locally) 114
Potential violation of Law of Demeter (method chain calls) 115
Potential violation of Law of Demeter (method chain calls) 115
Potential violation of Law of Demeter (object not created locally) 115
Potential violation of Law of Demeter (object not created locally) 116

Tobago Deprecation - org/apache/myfaces/tobago/renderkit/html/HtmlRendererUtil.java

Violation Line
This class has a bunch of public methods and attributes 20 - 661
This class has too many methods, consider refactoring it. 51 - 661
Possible God class (WMC=84, ATFD=29, TCC=0.0) 51 - 661
Avoid unused private methods such as 'renderErrorFocusId(FacesContext,UIInput)'. 64
Potential violation of Law of Demeter (method chain calls) 66
Avoid if (x != y) ..; else ..; 66 - 74
Potential violation of Law of Demeter (method chain calls) 66
Potential violation of Law of Demeter (method chain calls) 66
Potential violation of Law of Demeter (method chain calls) 67
Potential violation of Law of Demeter (method chain calls) 67
Potential violation of Law of Demeter (method chain calls) 67
Potential violation of Law of Demeter (object not created locally) 70
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (method chain calls) 76
Potential violation of Law of Demeter (static property access) 200
The String literal "HtmlRendererUtils.replaceStyleAttribute() no longer supported. Use setter." appears 4 times in this file; the first occurrence is on line 200 200
Potential violation of Law of Demeter (static property access) 209
Potential violation of Law of Demeter (static property access) 217
Potential violation of Law of Demeter (static property access)</