PMD Results

The following document contains the results of PMD 5.0.5.

Files

org/andromda/cartridges/ejb/EJBProfile.java

Violation Line
Avoid excessively long variable names like TAGGEDVALUE_EJB_TRANSACTION_TYPE 48
Avoid excessively long variable names like TAGGEDVALUE_EJB_NO_SYNTHETIC_CREATE_METHOD 52

org/andromda/cartridges/ejb/EJBScriptHelper.java

Violation Line
Parameter 'includeNames' is not assigned and could be declared final 27
Parameter 'attributes' is not assigned and could be declared final 27
Parameter 'includeTypes' is not assigned and could be declared final 27
A method should have only one exit point, and that should be the last statement in the method 31
Local variable 'sb' could be declared final 34
Avoid variables with short names like sb 34
Local variable 'attr' could be declared final 39
Parameter 'list' is not assigned and could be declared final 62
Parameter 'visibility' is not assigned and could be declared final 62
Local variable 'result' could be declared final 64
Local variable 'elem' could be declared final 65

org/andromda/cartridges/ejb/metafacades/EJBAssociationEndFacadeLogicImpl.java

Violation Line
Parameter 'metaObject' is not assigned and could be declared final 17
Parameter 'context' is not assigned and could be declared final 17

org/andromda/cartridges/ejb/metafacades/EJBAssociationFacadeLogicImpl.java

Violation Line
Parameter 'metaObject' is not assigned and could be declared final 20
Parameter 'context' is not assigned and could be declared final 20

org/andromda/cartridges/ejb/metafacades/EJBEntityAttributeFacadeLogicImpl.java

Violation Line
Parameter 'context' is not assigned and could be declared final 21
Parameter 'metaObject' is not assigned and could be declared final 21

org/andromda/cartridges/ejb/metafacades/EJBEntityFacadeLogicImpl.java

Violation Line
Parameter 'metaObject' is not assigned and could be declared final 39
Parameter 'context' is not assigned and could be declared final 40
Variables that are final and static should be all capitals, 'logger' is not all capitals. 48
Local variable 'iter' could be declared final 56
Local variable 'dep' could be declared final 59
A method should have only one exit point, and that should be the last statement in the method 66
Local variable 'attr' could be declared final 73
A method should have only one exit point, and that should be the last statement in the method 75
Local variable 'decorator' could be declared final 79
A method should have only one exit point, and that should be the last statement in the method 93
Local variable 'result' could be declared final 95
No need to check for null before an instanceof 98
Local variable 'entity' could be declared final 100
Local variable 'result' could be declared final 121
Local variable 'associationEnd' could be declared final 124
Local variable 'target' could be declared final 125
Local variable 'value' could be declared final 129–130
Local variable 'generateCmr' could be declared final 131
Parameter 'follow' is not assigned and could be declared final 166
Parameter 'follow' is not assigned and could be declared final 178
Local variable 'retval' could be declared final 180
Local variable 'ops' could be declared final 184
Local variable 'op' could be declared final 187
Avoid variables with short names like op 187
Parameter 'follow' is not assigned and could be declared final 220
Parameter 'follow' is not assigned and could be declared final 232
Local variable 'jndiName' could be declared final 245
Local variable 'jndiNamePrefix' could be declared final 246
Local variable 'businessOperations' could be declared final 286
Local variable 'operation' could be declared final 287
Local variable 'businessOperation' could be declared final 291
Local variable 'dependencies' could be declared final 306
Parameter 'object' is not assigned and could be declared final 311
Local variable 'dep' could be declared final 316
Parameter 'identifier' is not assigned and could be declared final 330
Local variable 'attr' could be declared final 332
A method should have only one exit point, and that should be the last statement in the method 336
Parameter 'strAttr' is not assigned and could be declared final 345
Local variable 'collAttrib' could be declared final 347
Local variable 'attr' could be declared final 348
A method should have only one exit point, and that should be the last statement in the method 352
Avoid variables with short names like op 361
Parameter 'op' is not assigned and could be declared final 361
Local variable 'collOps' could be declared final 363
Local variable 'operation' could be declared final 364
A method should have only one exit point, and that should be the last statement in the method 368
Local variable 'property' could be declared final 382
A catch statement should never catch throwable since it includes errors. 395
Local variable 'errMsg' could be declared final 397

org/andromda/cartridges/ejb/metafacades/EJBFinderMethodFacadeLogicImpl.java

Violation Line
Parameter 'context' is not assigned and could be declared final 23
Parameter 'metaObject' is not assigned and could be declared final 23
Local variable 'value' could be declared final 40
Local variable 'variableName' could be declared final 52
Prefer StringBuffer over += for concatenating strings 57
Local variable 'parameters' could be declared final 58
Deeply nested if..then statements are hard to read 59–72
Local variable 'parameterIt' could be declared final 61
Local variable 'test' could be declared final 64
Local variable 'param' could be declared final 65
Prefer StringBuffer over += for concatenating strings 66
Prefer StringBuffer over += for concatenating strings 69

org/andromda/cartridges/ejb/metafacades/EJBMetafacadeUtils.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. 23–233
Parameter 'follow' is not assigned and could be declared final 30
Parameter 'classifier' is not assigned and could be declared final 30
Use explicit scoping instead of the default package private level 30–51
The String literal "classifer" appears 8 times in this file; the first occurrence is on line 32 32
Local variable 'retval' could be declared final 33
Avoid variables with short names like op 37
Use explicit scoping instead of the default package private level 59–72
Parameter 'classifier' is not assigned and could be declared final 59
Use explicit scoping instead of the default package private level 80–109
Parameter 'classifier' is not assigned and could be declared final 80
Local variable 'viewTypeValue' could be declared final 86
Parameter 'object' is not assigned and could be declared final 92
Parameter 'classifier' is not assigned and could be declared final 117
Use explicit scoping instead of the default package private level 117–132
Local variable 'current' could be declared final 120
A method should have only one exit point, and that should be the last statement in the method 123
Local variable 'retval' could be declared final 125
Parameter 'classifier' is not assigned and could be declared final 141
Use explicit scoping instead of the default package private level 141–147
Local variable 'retval' could be declared final 144
Avoid reassigning parameters such as 'classifier' 159
Avoid reassigning parameters such as 'classifier' 159
Parameter 'follow' is not assigned and could be declared final 159
Use explicit scoping instead of the default package private level 159–183
Local variable 'attributes' could be declared final 163
Parameter 'object' is not assigned and could be declared final 176
Parameter 'follow' is not assigned and could be declared final 194
Avoid reassigning parameters such as 'classifier' 194
Use explicit scoping instead of the default package private level 194–218
Avoid reassigning parameters such as 'classifier' 194
Local variable 'attributes' could be declared final 198
Parameter 'object' is not assigned and could be declared final 211
Parameter 'classifier' is not assigned and could be declared final 226
Use explicit scoping instead of the default package private level 226–231

org/andromda/cartridges/ejb/metafacades/EJBOperationFacadeLogicImpl.java

Violation Line
Parameter 'metaObject' is not assigned and could be declared final 22
Parameter 'context' is not assigned and could be declared final 22

org/andromda/cartridges/ejb/metafacades/EJBPrimaryKeyFacadeLogicImpl.java

Violation Line
Parameter 'context' is not assigned and could be declared final 20
Parameter 'metaObject' is not assigned and could be declared final 20
Local variable 'primaryKeys' could be declared final 43
Local variable 'pkField' could be declared final 46

org/andromda/cartridges/ejb/metafacades/EJBSessionAttributeFacadeLogicImpl.java

Violation Line
Parameter 'metaObject' is not assigned and could be declared final 20
Parameter 'context' is not assigned and could be declared final 20

org/andromda/cartridges/ejb/metafacades/EJBSessionFacadeLogicImpl.java

Violation Line
Parameter 'context' is not assigned and could be declared final 26
Parameter 'metaObject' is not assigned and could be declared final 26
Parameter 'follow' is not assigned and could be declared final 36
Unnecessary use of fully qualified name 'java.util.Collection' due to existing import 'java.util.Collection' 36
Parameter 'follow' is not assigned and could be declared final 80
Parameter 'follow' is not assigned and could be declared final 90
Local variable 'jndiName' could be declared final 101
Local variable 'jndiNamePrefix' could be declared final 102
Local variable 'operations' could be declared final 176
Parameter 'object' is not assigned and could be declared final 179

org/andromda/cartridges/ejb/metafacades/ValueObjectFacadeLogicImpl.java

Violation Line
Parameter 'metaObject' is not assigned and could be declared final 24
Parameter 'context' is not assigned and could be declared final 25
Local variable 'buffer' could be declared final 42
Local variable 'token' could be declared final 45
Local variable 'generalization' could be declared final 81
Assigning an Object to null is a code smell. Consider refactoring. 82

org/andromda/cartridges/ejb/metafacades/EJBAssociationEndFacade.java

Violation Line
Avoid modifiers which are implied by the context 22
Avoid modifiers which are implied by the context 30

org/andromda/cartridges/ejb/metafacades/EJBAssociationEndFacadeLogic.java

Violation Line
This class has a bunch of public methods and attributes 5–1068
Abstract classes should be named AbstractXXX 33–1068
Avoid really long classes. 33–1068
Found non-transient, non-static member. Please mark as transient or provide accessors. 41
Parameter 'context' is not assigned and could be declared final 47
Parameter 'metaObjectIn' is not assigned and could be declared final 47
Avoid reassigning parameters such as 'context' 64
Private field 'superEntityAssociationEnd' could be made final; it is only initialized in the declaration or constructor. 73
Found non-transient, non-static member. Please mark as transient or provide accessors. 73
Found non-transient, non-static member. Please mark as transient or provide accessors. 74
Avoid using redundant field initializer for 'superEntityAssociationEndInitialized' 74
Avoid excessively long variable names like superEntityAssociationEndInitialized 74
Avoid reassigning parameters such as 'context' 95
An empty method in an abstract class should be abstract instead 112–115
Found non-transient, non-static member. Please mark as transient or provide accessors. 125
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__relationType1a' is not final. 125
Found non-transient, non-static member. Please mark as transient or provide accessors. 126
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__relationType1aSet' is not final. 126
Avoid using redundant field initializer for '__relationType1aSet' 126
An empty method in an abstract class should be abstract instead 155–158
An empty method in an abstract class should be abstract instead 164–167
An empty method in an abstract class should be abstract instead 173–176
Parameter 'element' is not assigned and could be declared final 551
Parameter 'tagName' is not assigned and could be declared final 561
Parameter 'tagName' is not assigned and could be declared final 571
Parameter 'bindedElement' is not assigned and could be declared final 582
Parameter 'kind' is not assigned and could be declared final 601
Parameter 'indent' is not assigned and could be declared final 612
Parameter 'lineLength' is not assigned and could be declared final 624
Parameter 'indent' is not assigned and could be declared final 624
Parameter 'lineLength' is not assigned and could be declared final 635
Parameter 'indent' is not assigned and could be declared final 635
Parameter 'htmlStyle' is not assigned and could be declared final 635
Parameter 'modelName' is not assigned and could be declared final 656
Parameter 'modelName' is not assigned and could be declared final 753
Parameter 'parameterName' is not assigned and could be declared final 847
Parameter 'stereotypeName' is not assigned and could be declared final 876
Parameter 'keywordName' is not assigned and could be declared final 886
Parameter 'stereotypeName' is not assigned and could be declared final 903
Parameter 'name' is not assigned and could be declared final 972
Parameter 'translation' is not assigned and could be declared final 972
Parameter 'translation' is not assigned and could be declared final 981
Parameter 'translation' is not assigned and could be declared final 990
Parameter 'kind' is not assigned and could be declared final 990
Local variable 'owner' could be declared final 1011
Consider simply returning the value vs storing it in local variable 'owner' 1012
Local variable 'name' could be declared final 1022
Consider simply returning the value vs storing it in local variable 'name' 1023
Parameter 'validationMessages' is not assigned and could be declared final 1031
Avoid appending characters as strings in StringBuffer.append. 1049
A catch statement should never catch throwable since it includes errors. 1054
Avoid empty catch blocks 1060–1063
A catch statement should never catch throwable since it includes errors. 1060
Avoid appending characters as strings in StringBuffer.append. 1065

org/andromda/cartridges/ejb/metafacades/EJBAssociationFacade.java

Violation Line
Avoid modifiers which are implied by the context 22
Avoid modifiers which are implied by the context 28

org/andromda/cartridges/ejb/metafacades/EJBAssociationFacadeLogic.java

Violation Line
This class has a bunch of public methods and attributes 5–916
Abstract classes should be named AbstractXXX 34–916
Found non-transient, non-static member. Please mark as transient or provide accessors. 42
Parameter 'context' is not assigned and could be declared final 48
Parameter 'metaObjectIn' is not assigned and could be declared final 48
Avoid reassigning parameters such as 'context' 65
Private field 'superEntityAssociation' could be made final; it is only initialized in the declaration or constructor. 74
Found non-transient, non-static member. Please mark as transient or provide accessors. 74
Avoid using redundant field initializer for 'superEntityAssociationInitialized' 75
Found non-transient, non-static member. Please mark as transient or provide accessors. 75
Avoid excessively long variable names like superEntityAssociationInitialized 75
Avoid reassigning parameters such as 'context' 96
An empty method in an abstract class should be abstract instead 113–116
Found non-transient, non-static member. Please mark as transient or provide accessors. 126
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType1a' is not final. 126
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType1aSet' is not final. 127
Avoid using redundant field initializer for '__transactionType1aSet' 127
Found non-transient, non-static member. Please mark as transient or provide accessors. 127
An empty method in an abstract class should be abstract instead 154–157
An empty method in an abstract class should be abstract instead 163–166
An empty method in an abstract class should be abstract instead 172–175
An empty method in an abstract class should be abstract instead 181–184
Parameter 'tagName' is not assigned and could be declared final 316
Parameter 'follow' is not assigned and could be declared final 316
Parameter 'element' is not assigned and could be declared final 399
Parameter 'tagName' is not assigned and could be declared final 409
Parameter 'tagName' is not assigned and could be declared final 419
Parameter 'bindedElement' is not assigned and could be declared final 430
Parameter 'kind' is not assigned and could be declared final 449
Parameter 'indent' is not assigned and could be declared final 460
Parameter 'indent' is not assigned and could be declared final 472
Parameter 'lineLength' is not assigned and could be declared final 472
Parameter 'indent' is not assigned and could be declared final 483
Parameter 'lineLength' is not assigned and could be declared final 483
Parameter 'htmlStyle' is not assigned and could be declared final 483
Parameter 'modelName' is not assigned and could be declared final 504
Parameter 'modelName' is not assigned and could be declared final 601
Parameter 'parameterName' is not assigned and could be declared final 695
Parameter 'stereotypeName' is not assigned and could be declared final 724
Parameter 'keywordName' is not assigned and could be declared final 734
Parameter 'stereotypeName' is not assigned and could be declared final 751
Parameter 'name' is not assigned and could be declared final 820
Parameter 'translation' is not assigned and could be declared final 820
Parameter 'translation' is not assigned and could be declared final 829
Parameter 'kind' is not assigned and could be declared final 838
Parameter 'translation' is not assigned and could be declared final 838
Local variable 'owner' could be declared final 859
Consider simply returning the value vs storing it in local variable 'owner' 860
Local variable 'name' could be declared final 870
Consider simply returning the value vs storing it in local variable 'name' 871
Parameter 'validationMessages' is not assigned and could be declared final 879
Avoid appending characters as strings in StringBuffer.append. 897
A catch statement should never catch throwable since it includes errors. 902
A catch statement should never catch throwable since it includes errors. 908
Avoid empty catch blocks 908–911
Avoid appending characters as strings in StringBuffer.append. 913

org/andromda/cartridges/ejb/metafacades/EJBEntityAttributeFacade.java

Violation Line
Avoid modifiers which are implied by the context 22
Avoid modifiers which are implied by the context 28

org/andromda/cartridges/ejb/metafacades/EJBEntityAttributeFacadeLogic.java

Violation Line
This class has a bunch of public methods and attributes 5–1034
Avoid really long classes. 32–1034
Abstract classes should be named AbstractXXX 32–1034
Found non-transient, non-static member. Please mark as transient or provide accessors. 40
Parameter 'context' is not assigned and could be declared final 46
Parameter 'metaObjectIn' is not assigned and could be declared final 46
Avoid reassigning parameters such as 'context' 63
Found non-transient, non-static member. Please mark as transient or provide accessors. 72
Private field 'superEntityAttribute' could be made final; it is only initialized in the declaration or constructor. 72
Avoid using redundant field initializer for 'superEntityAttributeInitialized' 73
Found non-transient, non-static member. Please mark as transient or provide accessors. 73
Avoid reassigning parameters such as 'context' 94
An empty method in an abstract class should be abstract instead 111–114
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType1a' is not final. 124
Found non-transient, non-static member. Please mark as transient or provide accessors. 124
Found non-transient, non-static member. Please mark as transient or provide accessors. 125
Avoid using redundant field initializer for '__transactionType1aSet' 125
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType1aSet' is not final. 125
An empty method in an abstract class should be abstract instead 152–155
An empty method in an abstract class should be abstract instead 161–164
An empty method in an abstract class should be abstract instead 170–173
Parameter 'follow' is not assigned and could be declared final 182
Parameter 'name' is not assigned and could be declared final 182
Parameter 'element' is not assigned and could be declared final 517
Parameter 'tagName' is not assigned and could be declared final 527
Parameter 'tagName' is not assigned and could be declared final 537
Parameter 'bindedElement' is not assigned and could be declared final 548
Parameter 'kind' is not assigned and could be declared final 567
Parameter 'indent' is not assigned and could be declared final 578
Parameter 'lineLength' is not assigned and could be declared final 590
Parameter 'indent' is not assigned and could be declared final 590
Parameter 'indent' is not assigned and could be declared final 601
Parameter 'lineLength' is not assigned and could be declared final 601
Parameter 'htmlStyle' is not assigned and could be declared final 601
Parameter 'modelName' is not assigned and could be declared final 622
Parameter 'modelName' is not assigned and could be declared final 719
Parameter 'parameterName' is not assigned and could be declared final 813
Parameter 'stereotypeName' is not assigned and could be declared final 842
Parameter 'keywordName' is not assigned and could be declared final 852
Parameter 'stereotypeName' is not assigned and could be declared final 869
Parameter 'name' is not assigned and could be declared final 938
Parameter 'translation' is not assigned and could be declared final 938
Parameter 'translation' is not assigned and could be declared final 947
Parameter 'translation' is not assigned and could be declared final 956
Parameter 'kind' is not assigned and could be declared final 956
Local variable 'owner' could be declared final 977
Consider simply returning the value vs storing it in local variable 'owner' 978
Local variable 'name' could be declared final 988
Consider simply returning the value vs storing it in local variable 'name' 989
Parameter 'validationMessages' is not assigned and could be declared final 997
Avoid appending characters as strings in StringBuffer.append. 1015
A catch statement should never catch throwable since it includes errors. 1020
Avoid empty catch blocks 1026–1029
A catch statement should never catch throwable since it includes errors. 1026
Avoid appending characters as strings in StringBuffer.append. 1031

org/andromda/cartridges/ejb/metafacades/EJBEntityFacade.java

Violation Line
Avoid modifiers which are implied by the context 24
Avoid modifiers which are implied by the context 48
Avoid modifiers which are implied by the context 56
Avoid modifiers which are implied by the context 65
Avoid modifiers which are implied by the context 73
Avoid modifiers which are implied by the context 80
Avoid modifiers which are implied by the context 90
Avoid modifiers which are implied by the context 96
Avoid modifiers which are implied by the context 104
Avoid modifiers which are implied by the context 111
Avoid modifiers which are implied by the context 119
Avoid modifiers which are implied by the context 126
Avoid modifiers which are implied by the context 132
Avoid modifiers which are implied by the context 139
Avoid modifiers which are implied by the context 145
Avoid modifiers which are implied by the context 153
Avoid modifiers which are implied by the context 161
Avoid variables with short names like id 161
Avoid modifiers which are implied by the context 169
Avoid variables with short names like op 169
Avoid modifiers which are implied by the context 175

org/andromda/cartridges/ejb/metafacades/EJBEntityFacadeLogic.java

Violation Line
This class has a bunch of public methods and attributes 5–2245
Avoid really long classes. 39–2245
Too many fields 39–2245
Abstract classes should be named AbstractXXX 39–2245
Found non-transient, non-static member. Please mark as transient or provide accessors. 47
Parameter 'metaObjectIn' is not assigned and could be declared final 53
Parameter 'context' is not assigned and could be declared final 53
Avoid reassigning parameters such as 'context' 70
Private field 'superEntity' could be made final; it is only initialized in the declaration or constructor. 79
Found non-transient, non-static member. Please mark as transient or provide accessors. 79
Found non-transient, non-static member. Please mark as transient or provide accessors. 80
Avoid using redundant field initializer for 'superEntityInitialized' 80
Avoid reassigning parameters such as 'context' 101
An empty method in an abstract class should be abstract instead 118–121
Avoid excessively long variable names like __syntheticCreateMethodAllowed1a 131
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__syntheticCreateMethodAllowed1a' is not final. 131
Found non-transient, non-static member. Please mark as transient or provide accessors. 131
Avoid excessively long variable names like __syntheticCreateMethodAllowed1aSet 132
Avoid using redundant field initializer for '__syntheticCreateMethodAllowed1aSet' 132
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__syntheticCreateMethodAllowed1aSet' is not final. 132
Found non-transient, non-static member. Please mark as transient or provide accessors. 132
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__allEntityRelations2a' is not final. 161
Found non-transient, non-static member. Please mark as transient or provide accessors. 161
Found non-transient, non-static member. Please mark as transient or provide accessors. 162
Avoid using redundant field initializer for '__allEntityRelations2aSet' 162
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__allEntityRelations2aSet' is not final. 162
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__jndiName3a' is not final. 209
Found non-transient, non-static member. Please mark as transient or provide accessors. 209
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__jndiName3aSet' is not final. 210
Avoid using redundant field initializer for '__jndiName3aSet' 210
Found non-transient, non-static member. Please mark as transient or provide accessors. 210
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__viewType4a' is not final. 240
Found non-transient, non-static member. Please mark as transient or provide accessors. 240
Found non-transient, non-static member. Please mark as transient or provide accessors. 241
Avoid using redundant field initializer for '__viewType4aSet' 241
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__viewType4aSet' is not final. 241
Found non-transient, non-static member. Please mark as transient or provide accessors. 270
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__allInstanceAttributes5a' is not final. 270
Avoid using redundant field initializer for '__allInstanceAttributes5aSet' 271
Found non-transient, non-static member. Please mark as transient or provide accessors. 271
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__allInstanceAttributes5aSet' is not final. 271
Found non-transient, non-static member. Please mark as transient or provide accessors. 302
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__inheritedInstanceAttributes6a' is not final. 302
Avoid excessively long variable names like __inheritedInstanceAttributes6aSet 303
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__inheritedInstanceAttributes6aSet' is not final. 303
Avoid using redundant field initializer for '__inheritedInstanceAttributes6aSet' 303
Found non-transient, non-static member. Please mark as transient or provide accessors. 303
Found non-transient, non-static member. Please mark as transient or provide accessors. 334
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__homeInterfaceName7a' is not final. 334
Avoid using redundant field initializer for '__homeInterfaceName7aSet' 335
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__homeInterfaceName7aSet' is not final. 335
Found non-transient, non-static member. Please mark as transient or provide accessors. 335
Found non-transient, non-static member. Please mark as transient or provide accessors. 364
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__valueDependencies8a' is not final. 364
Avoid using redundant field initializer for '__valueDependencies8aSet' 365
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__valueDependencies8aSet' is not final. 365
Found non-transient, non-static member. Please mark as transient or provide accessors. 365
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__entityRelations9a' is not final. 395
Found non-transient, non-static member. Please mark as transient or provide accessors. 395
Avoid using redundant field initializer for '__entityRelations9aSet' 396
Found non-transient, non-static member. Please mark as transient or provide accessors. 396
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__entityRelations9aSet' is not final. 396
Found non-transient, non-static member. Please mark as transient or provide accessors. 426
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType10a' is not final. 426
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType10aSet' is not final. 427
Found non-transient, non-static member. Please mark as transient or provide accessors. 427
Avoid using redundant field initializer for '__transactionType10aSet' 427
Parameter 'follow' is not assigned and could be declared final 468
Local variable 'returnValue' could be declared final 471
Parameter 'follow' is not assigned and could be declared final 492
Local variable 'returnValue' could be declared final 495
Parameter 'follow' is not assigned and could be declared final 521
Local variable 'returnValue' could be declared final 524
Parameter 'follow' is not assigned and could be declared final 548
Local variable 'returnValue' could be declared final 551
Avoid variables with short names like op 563
Parameter 'op' is not assigned and could be declared final 573
Avoid variables with short names like op 573
Local variable 'returnValue' could be declared final 576
Parameter 'att' is not assigned and could be declared final 598
Local variable 'returnValue' could be declared final 601
Avoid variables with short names like id 613
Parameter 'id' is not assigned and could be declared final 623
Avoid variables with short names like id 623
Local variable 'returnValue' could be declared final 626
Local variable 'returnValue' could be declared final 647
An empty method in an abstract class should be abstract instead 656–659
An empty method in an abstract class should be abstract instead 665–668
An empty method in an abstract class should be abstract instead 674–677
An empty method in an abstract class should be abstract instead 683–686
Parameter 'name' is not assigned and could be declared final 693
Parameter 'follow' is not assigned and could be declared final 789
Parameter 'follow' is not assigned and could be declared final 886
Parameter 'follow' is not assigned and could be declared final 938
Parameter 'follow' is not assigned and could be declared final 1254
Parameter 'withIdentifiers' is not assigned and could be declared final 1254
Parameter 'withDerived' is not assigned and could be declared final 1266
Parameter 'follow' is not assigned and could be declared final 1266
Parameter 'withIdentifiers' is not assigned and could be declared final 1266
Parameter 'withIdentifiers' is not assigned and could be declared final 1277
Parameter 'follow' is not assigned and could be declared final 1277
Parameter 'withIdentifiers' is not assigned and could be declared final 1287
Parameter 'follow' is not assigned and could be declared final 1287
Parameter 'withDerived' is not assigned and could be declared final 1298
Parameter 'follow' is not assigned and could be declared final 1298
Parameter 'withIdentifiers' is not assigned and could be declared final 1298
Parameter 'follow' is not assigned and could be declared final 1417
Do not use the short type 1426
Parameter 'withIdentifiers' is not assigned and could be declared final 1436
Parameter 'follow' is not assigned and could be declared final 1448
Parameter 'withIdentifiers' is not assigned and could be declared final 1448
Parameter 'follow' is not assigned and could be declared final 1471
Parameter 'withIdentifiers' is not assigned and could be declared final 1471
Parameter 'follow' is not assigned and could be declared final 1492
Parameter 'withIdentifiers' is not assigned and could be declared final 1503
Parameter 'follow' is not assigned and could be declared final 1503
Parameter 'withIdentifiers' is not assigned and could be declared final 1514
Parameter 'follow' is not assigned and could be declared final 1514
Parameter 'follow' is not assigned and could be declared final 1526
Parameter 'withIdentifiers' is not assigned and could be declared final 1526
Parameter 'follow' is not assigned and could be declared final 1540
Parameter 'withIdentifiers' is not assigned and could be declared final 1540
Parameter 'withIdentifiers' is not assigned and could be declared final 1549
Parameter 'follow' is not assigned and could be declared final 1549
Parameter 'withIdentifiers' is not assigned and could be declared final 1558
Parameter 'follow' is not assigned and could be declared final 1558
Parameter 'tagName' is not assigned and could be declared final 1645
Parameter 'follow' is not assigned and could be declared final 1645
Parameter 'element' is not assigned and could be declared final 1728
Parameter 'tagName' is not assigned and could be declared final 1738
Parameter 'tagName' is not assigned and could be declared final 1748
Parameter 'bindedElement' is not assigned and could be declared final 1759
Parameter 'kind' is not assigned and could be declared final 1778
Parameter 'indent' is not assigned and could be declared final 1789
Parameter 'indent' is not assigned and could be declared final 1801
Parameter 'lineLength' is not assigned and could be declared final 1801
Parameter 'indent' is not assigned and could be declared final 1812
Parameter 'lineLength' is not assigned and could be declared final 1812
Parameter 'htmlStyle' is not assigned and could be declared final 1812
Parameter 'modelName' is not assigned and could be declared final 1833
Parameter 'modelName' is not assigned and could be declared final 1930
Parameter 'parameterName' is not assigned and could be declared final 2024
Parameter 'stereotypeName' is not assigned and could be declared final 2053
Parameter 'keywordName' is not assigned and could be declared final 2063
Parameter 'stereotypeName' is not assigned and could be declared final 2080
Parameter 'name' is not assigned and could be declared final 2149
Parameter 'translation' is not assigned and could be declared final 2149
Parameter 'translation' is not assigned and could be declared final 2158
Parameter 'translation' is not assigned and could be declared final 2167
Parameter 'kind' is not assigned and could be declared final 2167
Local variable 'owner' could be declared final 2188
Consider simply returning the value vs storing it in local variable 'owner' 2189
Local variable 'name' could be declared final 2199
Consider simply returning the value vs storing it in local variable 'name' 2200
Parameter 'validationMessages' is not assigned and could be declared final 2208
Avoid appending characters as strings in StringBuffer.append. 2226
A catch statement should never catch throwable since it includes errors. 2231
Avoid empty catch blocks 2237–2240
A catch statement should never catch throwable since it includes errors. 2237
Avoid appending characters as strings in StringBuffer.append. 2242

org/andromda/cartridges/ejb/metafacades/EJBFinderMethodFacade.java

Violation Line
Avoid modifiers which are implied by the context 22
Avoid modifiers which are implied by the context 28
Avoid modifiers which are implied by the context 34

org/andromda/cartridges/ejb/metafacades/EJBFinderMethodFacadeLogic.java

Violation Line
This class has a bunch of public methods and attributes 5–1142
Avoid really long classes. 33–1142
Abstract classes should be named AbstractXXX 33–1142
Found non-transient, non-static member. Please mark as transient or provide accessors. 41
Parameter 'metaObjectIn' is not assigned and could be declared final 47
Parameter 'context' is not assigned and could be declared final 47
Avoid reassigning parameters such as 'context' 64
Found non-transient, non-static member. Please mark as transient or provide accessors. 73
Private field 'superEntityQueryOperation' could be made final; it is only initialized in the declaration or constructor. 73
Avoid using redundant field initializer for 'superEntityQueryOperationInitialized' 74
Avoid excessively long variable names like superEntityQueryOperationInitialized 74
Found non-transient, non-static member. Please mark as transient or provide accessors. 74
Avoid reassigning parameters such as 'context' 95
An empty method in an abstract class should be abstract instead 112–115
Found non-transient, non-static member. Please mark as transient or provide accessors. 125
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__query1a' is not final. 125
Found non-transient, non-static member. Please mark as transient or provide accessors. 126
Avoid using redundant field initializer for '__query1aSet' 126
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__query1aSet' is not final. 126
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType2a' is not final. 155
Found non-transient, non-static member. Please mark as transient or provide accessors. 155
Found non-transient, non-static member. Please mark as transient or provide accessors. 156
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType2aSet' is not final. 156
Avoid using redundant field initializer for '__transactionType2aSet' 156
An empty method in an abstract class should be abstract instead 183–186
An empty method in an abstract class should be abstract instead 192–195
An empty method in an abstract class should be abstract instead 201–204
Parameter 'translation' is not assigned and could be declared final 213
Parameter 'element' is not assigned and could be declared final 222
Parameter 'tagName' is not assigned and could be declared final 232
Parameter 'tagName' is not assigned and could be declared final 242
Parameter 'bindedElement' is not assigned and could be declared final 253
Parameter 'kind' is not assigned and could be declared final 272
Parameter 'indent' is not assigned and could be declared final 283
Parameter 'indent' is not assigned and could be declared final 295
Parameter 'lineLength' is not assigned and could be declared final 295
Parameter 'htmlStyle' is not assigned and could be declared final 306
Parameter 'lineLength' is not assigned and could be declared final 306
Parameter 'indent' is not assigned and could be declared final 306
Parameter 'modelName' is not assigned and could be declared final 327
Parameter 'modelName' is not assigned and could be declared final 424
Parameter 'parameterName' is not assigned and could be declared final 518
Parameter 'stereotypeName' is not assigned and could be declared final 547
Parameter 'keywordName' is not assigned and could be declared final 557
Parameter 'stereotypeName' is not assigned and could be declared final 574
Parameter 'name' is not assigned and could be declared final 643
Parameter 'translation' is not assigned and could be declared final 643
Parameter 'translation' is not assigned and could be declared final 652
Parameter 'kind' is not assigned and could be declared final 661
Parameter 'translation' is not assigned and could be declared final 661
Parameter 'name' is not assigned and could be declared final 671
Parameter 'name' is not assigned and could be declared final 682
Parameter 'follow' is not assigned and could be declared final 682
Parameter 'initialExceptions' is not assigned and could be declared final 753
Parameter 'withArgumentNames' is not assigned and could be declared final 910
Parameter 'argumentModifier' is not assigned and could be declared final 920
Parameter 'modifier' is not assigned and could be declared final 939
Local variable 'owner' could be declared final 1085
Consider simply returning the value vs storing it in local variable 'owner' 1086
Local variable 'name' could be declared final 1096
Consider simply returning the value vs storing it in local variable 'name' 1097
Parameter 'validationMessages' is not assigned and could be declared final 1105
Avoid appending characters as strings in StringBuffer.append. 1123
A catch statement should never catch throwable since it includes errors. 1128
A catch statement should never catch throwable since it includes errors. 1134
Avoid empty catch blocks 1134–1137
Avoid appending characters as strings in StringBuffer.append. 1139

org/andromda/cartridges/ejb/metafacades/EJBOperationFacade.java

Violation Line
Avoid modifiers which are implied by the context 22
Avoid modifiers which are implied by the context 28
Avoid modifiers which are implied by the context 34
Avoid modifiers which are implied by the context 40

org/andromda/cartridges/ejb/metafacades/EJBOperationFacadeLogic.java

Violation Line
This class has a bunch of public methods and attributes 5–1151
Avoid really long classes. 32–1151
Abstract classes should be named AbstractXXX 32–1151
Found non-transient, non-static member. Please mark as transient or provide accessors. 40
Parameter 'context' is not assigned and could be declared final 46
Parameter 'metaObjectIn' is not assigned and could be declared final 46
Avoid reassigning parameters such as 'context' 63
Private field 'superOperationFacade' could be made final; it is only initialized in the declaration or constructor. 72
Found non-transient, non-static member. Please mark as transient or provide accessors. 72
Avoid using redundant field initializer for 'superOperationFacadeInitialized' 73
Found non-transient, non-static member. Please mark as transient or provide accessors. 73
Avoid reassigning parameters such as 'context' 94
An empty method in an abstract class should be abstract instead 111–114
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType1a' is not final. 124
Found non-transient, non-static member. Please mark as transient or provide accessors. 124
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType1aSet' is not final. 125
Avoid using redundant field initializer for '__transactionType1aSet' 125
Found non-transient, non-static member. Please mark as transient or provide accessors. 125
Found non-transient, non-static member. Please mark as transient or provide accessors. 154
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__businessOperation2a' is not final. 154
Avoid using redundant field initializer for '__businessOperation2aSet' 155
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__businessOperation2aSet' is not final. 155
Found non-transient, non-static member. Please mark as transient or provide accessors. 155
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__selectMethod3a' is not final. 184
Found non-transient, non-static member. Please mark as transient or provide accessors. 184
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__selectMethod3aSet' is not final. 185
Avoid using redundant field initializer for '__selectMethod3aSet' 185
Found non-transient, non-static member. Please mark as transient or provide accessors. 185
An empty method in an abstract class should be abstract instead 212–215
An empty method in an abstract class should be abstract instead 221–224
Parameter 'element' is not assigned and could be declared final 231
Parameter 'tagName' is not assigned and could be declared final 241
Parameter 'tagName' is not assigned and could be declared final 251
Parameter 'bindedElement' is not assigned and could be declared final 262
Parameter 'kind' is not assigned and could be declared final 281
Parameter 'indent' is not assigned and could be declared final 292
Parameter 'lineLength' is not assigned and could be declared final 304
Parameter 'indent' is not assigned and could be declared final 304
Parameter 'lineLength' is not assigned and could be declared final 315
Parameter 'htmlStyle' is not assigned and could be declared final 315
Parameter 'indent' is not assigned and could be declared final 315
Parameter 'modelName' is not assigned and could be declared final 336
Parameter 'modelName' is not assigned and could be declared final 433
Parameter 'parameterName' is not assigned and could be declared final 527
Parameter 'stereotypeName' is not assigned and could be declared final 556
Parameter 'keywordName' is not assigned and could be declared final 566
Parameter 'stereotypeName' is not assigned and could be declared final 583
Parameter 'translation' is not assigned and could be declared final 652
Parameter 'name' is not assigned and could be declared final 652
Parameter 'translation' is not assigned and could be declared final 661
Parameter 'translation' is not assigned and could be declared final 670
Parameter 'kind' is not assigned and could be declared final 670
Parameter 'name' is not assigned and could be declared final 680
Parameter 'follow' is not assigned and could be declared final 691
Parameter 'name' is not assigned and could be declared final 691
Parameter 'initialExceptions' is not assigned and could be declared final 762
Parameter 'withArgumentNames' is not assigned and could be declared final 919
Parameter 'argumentModifier' is not assigned and could be declared final 929
Parameter 'modifier' is not assigned and could be declared final 948
Local variable 'owner' could be declared final 1094
Consider simply returning the value vs storing it in local variable 'owner' 1095
Local variable 'name' could be declared final 1105
Consider simply returning the value vs storing it in local variable 'name' 1106
Parameter 'validationMessages' is not assigned and could be declared final 1114
Avoid appending characters as strings in StringBuffer.append. 1132
A catch statement should never catch throwable since it includes errors. 1137
A catch statement should never catch throwable since it includes errors. 1143
Avoid empty catch blocks 1143–1146
Avoid appending characters as strings in StringBuffer.append. 1148

org/andromda/cartridges/ejb/metafacades/EJBPrimaryKeyFacade.java

Violation Line
Avoid modifiers which are implied by the context 20
Avoid modifiers which are implied by the context 26

org/andromda/cartridges/ejb/metafacades/EJBPrimaryKeyFacadeLogic.java

Violation Line
Abstract classes should be named AbstractXXX 16–113
Found non-transient, non-static member. Please mark as transient or provide accessors. 24
Parameter 'context' is not assigned and could be declared final 30
Parameter 'metaObjectIn' is not assigned and could be declared final 30
Avoid reassigning parameters such as 'context' 41
Avoid reassigning parameters such as 'context' 54
An empty method in an abstract class should be abstract instead 67–70
Found non-transient, non-static member. Please mark as transient or provide accessors. 80
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__complex1a' is not final. 80
Found non-transient, non-static member. Please mark as transient or provide accessors. 81
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__complex1aSet' is not final. 81
Avoid using redundant field initializer for '__complex1aSet' 81
Overriding method merely calls super 109–112
Parameter 'validationMessages' is not assigned and could be declared final 109

org/andromda/cartridges/ejb/metafacades/EJBSessionAttributeFacade.java

Violation Line
Avoid modifiers which are implied by the context 22
Avoid modifiers which are implied by the context 28

org/andromda/cartridges/ejb/metafacades/EJBSessionAttributeFacadeLogic.java

Violation Line
This class has a bunch of public methods and attributes 5–935
Abstract classes should be named AbstractXXX 32–935
Found non-transient, non-static member. Please mark as transient or provide accessors. 40
Parameter 'metaObjectIn' is not assigned and could be declared final 46
Parameter 'context' is not assigned and could be declared final 46
Avoid reassigning parameters such as 'context' 63
Private field 'superAttributeFacade' could be made final; it is only initialized in the declaration or constructor. 72
Found non-transient, non-static member. Please mark as transient or provide accessors. 72
Found non-transient, non-static member. Please mark as transient or provide accessors. 73
Avoid using redundant field initializer for 'superAttributeFacadeInitialized' 73
Avoid reassigning parameters such as 'context' 94
An empty method in an abstract class should be abstract instead 111–114
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType1a' is not final. 124
Found non-transient, non-static member. Please mark as transient or provide accessors. 124
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType1aSet' is not final. 125
Found non-transient, non-static member. Please mark as transient or provide accessors. 125
Avoid using redundant field initializer for '__transactionType1aSet' 125
An empty method in an abstract class should be abstract instead 152–155
An empty method in an abstract class should be abstract instead 161–164
Parameter 'follow' is not assigned and could be declared final 173
Parameter 'name' is not assigned and could be declared final 173
Parameter 'element' is not assigned and could be declared final 418
Parameter 'tagName' is not assigned and could be declared final 428
Parameter 'tagName' is not assigned and could be declared final 438
Parameter 'bindedElement' is not assigned and could be declared final 449
Parameter 'kind' is not assigned and could be declared final 468
Parameter 'indent' is not assigned and could be declared final 479
Parameter 'lineLength' is not assigned and could be declared final 491
Parameter 'indent' is not assigned and could be declared final 491
Parameter 'htmlStyle' is not assigned and could be declared final 502
Parameter 'lineLength' is not assigned and could be declared final 502
Parameter 'indent' is not assigned and could be declared final 502
Parameter 'modelName' is not assigned and could be declared final 523
Parameter 'modelName' is not assigned and could be declared final 620
Parameter 'parameterName' is not assigned and could be declared final 714
Parameter 'stereotypeName' is not assigned and could be declared final 743
Parameter 'keywordName' is not assigned and could be declared final 753
Parameter 'stereotypeName' is not assigned and could be declared final 770
Parameter 'translation' is not assigned and could be declared final 839
Parameter 'name' is not assigned and could be declared final 839
Parameter 'translation' is not assigned and could be declared final 848
Parameter 'kind' is not assigned and could be declared final 857
Parameter 'translation' is not assigned and could be declared final 857
Local variable 'owner' could be declared final 878
Consider simply returning the value vs storing it in local variable 'owner' 879
Local variable 'name' could be declared final 889
Consider simply returning the value vs storing it in local variable 'name' 890
Parameter 'validationMessages' is not assigned and could be declared final 898
Avoid appending characters as strings in StringBuffer.append. 916
A catch statement should never catch throwable since it includes errors. 921
A catch statement should never catch throwable since it includes errors. 927
Avoid empty catch blocks 927–930
Avoid appending characters as strings in StringBuffer.append. 932

org/andromda/cartridges/ejb/metafacades/EJBSessionFacade.java

Violation Line
Avoid modifiers which are implied by the context 24
Avoid modifiers which are implied by the context 32
Avoid modifiers which are implied by the context 39
Avoid modifiers which are implied by the context 48
Avoid modifiers which are implied by the context 56
Avoid modifiers which are implied by the context 66
Avoid modifiers which are implied by the context 72
Avoid modifiers which are implied by the context 80
Avoid modifiers which are implied by the context 86
Avoid modifiers which are implied by the context 92
Avoid modifiers which are implied by the context 98
Avoid modifiers which are implied by the context 104
Avoid modifiers which are implied by the context 110
Avoid modifiers which are implied by the context 116
Avoid modifiers which are implied by the context 122

org/andromda/cartridges/ejb/metafacades/EJBSessionFacadeLogic.java

Violation Line
This class has a bunch of public methods and attributes 5–1807
Avoid really long classes. 39–1807
Too many fields 39–1807
Abstract classes should be named AbstractXXX 39–1807
Found non-transient, non-static member. Please mark as transient or provide accessors. 47
Parameter 'metaObjectIn' is not assigned and could be declared final 53
Parameter 'context' is not assigned and could be declared final 53
Avoid reassigning parameters such as 'context' 70
Found non-transient, non-static member. Please mark as transient or provide accessors. 79
Private field 'superService' could be made final; it is only initialized in the declaration or constructor. 79
Found non-transient, non-static member. Please mark as transient or provide accessors. 80
Avoid using redundant field initializer for 'superServiceInitialized' 80
Avoid reassigning parameters such as 'context' 101
An empty method in an abstract class should be abstract instead 118–121
Avoid excessively long variable names like __syntheticCreateMethodAllowed1a 131
Found non-transient, non-static member. Please mark as transient or provide accessors. 131
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__syntheticCreateMethodAllowed1a' is not final. 131
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__syntheticCreateMethodAllowed1aSet' is not final. 132
Found non-transient, non-static member. Please mark as transient or provide accessors. 132
Avoid using redundant field initializer for '__syntheticCreateMethodAllowed1aSet' 132
Avoid excessively long variable names like __syntheticCreateMethodAllowed1aSet 132
Found non-transient, non-static member. Please mark as transient or provide accessors. 161
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__businessOperations2a' is not final. 161
Avoid using redundant field initializer for '__businessOperations2aSet' 162
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__businessOperations2aSet' is not final. 162
Found non-transient, non-static member. Please mark as transient or provide accessors. 162
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__allInstanceAttributes3a' is not final. 192
Found non-transient, non-static member. Please mark as transient or provide accessors. 192
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__allInstanceAttributes3aSet' is not final. 193
Avoid using redundant field initializer for '__allInstanceAttributes3aSet' 193
Found non-transient, non-static member. Please mark as transient or provide accessors. 193
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__inheritedInstanceAttributes4a' is not final. 224
Found non-transient, non-static member. Please mark as transient or provide accessors. 224
Avoid excessively long variable names like __inheritedInstanceAttributes4aSet 225
Found non-transient, non-static member. Please mark as transient or provide accessors. 225
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__inheritedInstanceAttributes4aSet' is not final. 225
Avoid using redundant field initializer for '__inheritedInstanceAttributes4aSet' 225
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__jndiName5a' is not final. 256
Found non-transient, non-static member. Please mark as transient or provide accessors. 256
Found non-transient, non-static member. Please mark as transient or provide accessors. 257
Avoid using redundant field initializer for '__jndiName5aSet' 257
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__jndiName5aSet' is not final. 257
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__stateful6a' is not final. 286
Found non-transient, non-static member. Please mark as transient or provide accessors. 286
Avoid using redundant field initializer for '__stateful6aSet' 287
Found non-transient, non-static member. Please mark as transient or provide accessors. 287
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__stateful6aSet' is not final. 287
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__stateless7a' is not final. 316
Found non-transient, non-static member. Please mark as transient or provide accessors. 316
Avoid using redundant field initializer for '__stateless7aSet' 317
Found non-transient, non-static member. Please mark as transient or provide accessors. 317
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__stateless7aSet' is not final. 317
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__type8a' is not final. 346
Found non-transient, non-static member. Please mark as transient or provide accessors. 346
Found non-transient, non-static member. Please mark as transient or provide accessors. 347
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__type8aSet' is not final. 347
Avoid using redundant field initializer for '__type8aSet' 347
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__viewType9a' is not final. 376
Found non-transient, non-static member. Please mark as transient or provide accessors. 376
Avoid using redundant field initializer for '__viewType9aSet' 377
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__viewType9aSet' is not final. 377
Found non-transient, non-static member. Please mark as transient or provide accessors. 377
Found non-transient, non-static member. Please mark as transient or provide accessors. 406
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__homeInterfaceName10a' is not final. 406
Avoid using redundant field initializer for '__homeInterfaceName10aSet' 407
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__homeInterfaceName10aSet' is not final. 407
Found non-transient, non-static member. Please mark as transient or provide accessors. 407
Found non-transient, non-static member. Please mark as transient or provide accessors. 436
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType11a' is not final. 436
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '__transactionType11aSet' is not final. 437
Found non-transient, non-static member. Please mark as transient or provide accessors. 437
Avoid using redundant field initializer for '__transactionType11aSet' 437
Parameter 'follow' is not assigned and could be declared final 478
Local variable 'returnValue' could be declared final 481
Parameter 'follow' is not assigned and could be declared final 507
Local variable 'returnValue' could be declared final 510
Parameter 'follow' is not assigned and could be declared final 534
Local variable 'returnValue' could be declared final 537
An empty method in an abstract class should be abstract instead 546–549
An empty method in an abstract class should be abstract instead 555–558
An empty method in an abstract class should be abstract instead 564–567
An empty method in an abstract class should be abstract instead 573–576
Parameter 'name' is not assigned and could be declared final 583
Parameter 'follow' is not assigned and could be declared final 679
Parameter 'follow' is not assigned and could be declared final 776
Parameter 'follow' is not assigned and could be declared final 828
Parameter 'tagName' is not assigned and could be declared final 1134
Parameter 'follow' is not assigned and could be declared final 1134
Parameter 'element' is not assigned and could be declared final 1217
Parameter 'tagName' is not assigned and could be declared final 1227
Parameter 'tagName' is not assigned and could be declared final 1237
Parameter 'bindedElement' is not assigned and could be declared final 1248
Parameter 'kind' is not assigned and could be declared final 1267
Parameter 'indent' is not assigned and could be declared final 1278
Parameter 'indent' is not assigned and could be declared final 1290
Parameter 'lineLength' is not assigned and could be declared final 1290
Parameter 'htmlStyle' is not assigned and could be declared final 1301
Parameter 'indent' is not assigned and could be declared final 1301
Parameter 'lineLength' is not assigned and could be declared final 1301
Parameter 'modelName' is not assigned and could be declared final 1322
Parameter 'modelName' is not assigned and could be declared final 1419
Parameter 'parameterName' is not assigned and could be declared final 1513
Parameter 'stereotypeName' is not assigned and could be declared final 1542
Parameter 'keywordName' is not assigned and could be declared final 1552
Parameter 'stereotypeName' is not assigned and could be declared final 1569
Parameter 'translation' is not assigned and could be declared final 1638
Parameter 'name' is not assigned and could be declared final 1638
Parameter 'translation' is not assigned and could be declared final 1647
Parameter 'kind' is not assigned and could be declared final 1656
Parameter 'translation' is not assigned and could be declared final 1656
Local variable 'owner' could be declared final 1750
Consider simply returning the value vs storing it in local variable 'owner' 1751
Local variable 'name' could be declared final 1761
Consider simply returning the value vs storing it in local variable 'name' 1762
Parameter 'validationMessages' is not assigned and could be declared final 1770
Avoid appending characters as strings in StringBuffer.append. 1788
A catch statement should never catch throwable since it includes errors. 1793
A catch statement should never catch throwable since it includes errors. 1799
Avoid empty catch blocks 1799–1802
Avoid appending characters as strings in StringBuffer.append. 1804

org/andromda/cartridges/ejb/metafacades/ValueObjectFacade.java

Violation Line
Avoid modifiers which are implied by the context 22

org/andromda/cartridges/ejb/metafacades/ValueObjectFacadeLogic.java

Violation Line
This class has a bunch of public methods and attributes 5–1303
Abstract classes should be named AbstractXXX 36–1303
Avoid really long classes. 36–1303
Found non-transient, non-static member. Please mark as transient or provide accessors. 44
Parameter 'context' is not assigned and could be declared final 50
Parameter 'metaObjectIn' is not assigned and could be declared final 50
Avoid reassigning parameters such as 'context' 67
Found non-transient, non-static member. Please mark as transient or provide accessors. 76
Private field 'superClassifierFacade' could be made final; it is only initialized in the declaration or constructor. 76
Avoid excessively long variable names like superClassifierFacadeInitialized 77
Avoid using redundant field initializer for 'superClassifierFacadeInitialized' 77
Found non-transient, non-static member. Please mark as transient or provide accessors. 77
Avoid reassigning parameters such as 'context' 98
An empty method in an abstract class should be abstract instead 115–118
An empty method in an abstract class should be abstract instead 124–127
An empty method in an abstract class should be abstract instead 133–136
An empty method in an abstract class should be abstract instead 142–145
Parameter 'name' is not assigned and could be declared final 152
Parameter 'follow' is not assigned and could be declared final 248
Parameter 'follow' is not assigned and could be declared final 345
Parameter 'follow' is not assigned and could be declared final 397
Parameter 'follow' is not assigned and could be declared final 703
Parameter 'tagName' is not assigned and could be declared final 703
Parameter 'element' is not assigned and could be declared final 786
Parameter 'tagName' is not assigned and could be declared final 796
Parameter 'tagName' is not assigned and could be declared final 806
Parameter 'bindedElement' is not assigned and could be declared final 817
Parameter 'kind' is not assigned and could be declared final 836
Parameter 'indent' is not assigned and could be declared final 847
Parameter 'indent' is not assigned and could be declared final 859
Parameter 'lineLength' is not assigned and could be declared final 859
Parameter 'htmlStyle' is not assigned and could be declared final 870
Parameter 'lineLength' is not assigned and could be declared final 870
Parameter 'indent' is not assigned and could be declared final 870
Parameter 'modelName' is not assigned and could be declared final 891
Parameter 'modelName' is not assigned and could be declared final 988
Parameter 'parameterName' is not assigned and could be declared final 1082
Parameter 'stereotypeName' is not assigned and could be declared final 1111
Parameter 'keywordName' is not assigned and could be declared final 1121
Parameter 'stereotypeName' is not assigned and could be declared final 1138
Parameter 'name' is not assigned and could be declared final 1207
Parameter 'translation' is not assigned and could be declared final 1207
Parameter 'translation' is not assigned and could be declared final 1216
Parameter 'translation' is not assigned and could be declared final 1225
Parameter 'kind' is not assigned and could be declared final 1225
Local variable 'owner' could be declared final 1246
Consider simply returning the value vs storing it in local variable 'owner' 1247
Local variable 'name' could be declared final 1257
Consider simply returning the value vs storing it in local variable 'name' 1258
Parameter 'validationMessages' is not assigned and could be declared final 1266
Avoid appending characters as strings in StringBuffer.append. 1284
A catch statement should never catch throwable since it includes errors. 1289
A catch statement should never catch throwable since it includes errors. 1295
Avoid empty catch blocks 1295–1298
Avoid appending characters as strings in StringBuffer.append. 1300