PMD Results

The following document contains the results of PMD 6.29.0.

Violations By Priority

Priority 1

org/apache/fulcrum/intake/IntakeServiceFacade.java

Rule Violation Line
ClassNamingConventions The utility class name 'IntakeServiceFacade' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Constants)' 41–182

org/apache/fulcrum/intake/IntakeServiceImpl.java

Rule Violation Line
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 228
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 369
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 376
LocalVariableNamingConventions The local variable name 'READ_ERR' doesn't match '[a-z][a-zA-Z0-9]*' 751
VariableNamingConventions Variables should start with a lowercase character, 'READ_ERR' starts with uppercase character. 751
VariableNamingConventions Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'READ_ERR' is not final. 751
AvoidThrowingRawExceptionTypes Avoid throwing raw exception types. 755
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 803

org/apache/fulcrum/intake/model/Field.java

Rule Violation Line
FieldNamingConventions The public constant name 'defaultValidatorPackage' doesn't match '[A-Z][A-Z_0-9]*' 64
VariableNamingConventions Variables that are final and static should be all capitals, 'defaultValidatorPackage' is not all capitals. 64
FieldNamingConventions The public constant name 'defaultFieldPackage' doesn't match '[A-Z][A-Z_0-9]*' 67
VariableNamingConventions Variables that are final and static should be all capitals, 'defaultFieldPackage' is not all capitals. 67
ConstructorCallsOverridableMethod Overridable method 'enableLogging' called during object construction 183
ConstructorCallsOverridableMethod Overridable method 'getDisplayName' called during object construction 199
ConstructorCallsOverridableMethod Overridable method 'getDisplayName' called during object construction 210
ConstructorCallsOverridableMethod Overridable method 'getDefaultValidator' called during object construction 217

org/apache/fulcrum/intake/validator/FieldReference.java

Rule Violation Line
LocalVariableNamingConventions The local variable name 'comp_true' doesn't match '[a-z][a-zA-Z0-9]*' 202
VariableNamingConventions Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'comp_true' is not final. 202

org/apache/fulcrum/intake/validator/FileValidator.java

Rule Violation Line
FieldNamingConventions The constant name 'charsetPattern' doesn't match '[A-Z][A-Z_0-9]*' 45
VariableNamingConventions Variables that are final and static should be all capitals, 'charsetPattern' is not all capitals. 45

Priority 2

org/apache/fulcrum/intake/model/BooleanField.java

Rule Violation Line
GuardLogStatement Logger calls should be surrounded by log level guards. 163

org/apache/fulcrum/intake/model/Field.java

Rule Violation Line
LoggerIsNotStaticFinal The Logger variable declaration does not contain the static and final modifiers 170
GuardLogStatement Logger calls should be surrounded by log level guards. 198–200
GuardLogStatement Logger calls should be surrounded by log level guards. 209–211
GuardLogStatement Logger calls should be surrounded by log level guards. 269–273
GuardLogStatement Logger calls should be surrounded by log level guards. 281–285
GuardLogStatement Logger calls should be surrounded by log level guards. 640
GuardLogStatement Logger calls should be surrounded by log level guards. 654
GuardLogStatement Logger calls should be surrounded by log level guards. 698
GuardLogStatement Logger calls should be surrounded by log level guards. 703
GuardLogStatement Logger calls should be surrounded by log level guards. 820–821

org/apache/fulcrum/intake/model/FieldType.java

Rule Violation Line
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 41–44
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 49–52
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 57–60
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 65–68
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 74–77
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 82–85
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 90–93
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 98–101
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 115–118
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 123–126
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 131–134

org/apache/fulcrum/intake/model/Group.java

Rule Violation Line
LoggerIsNotStaticFinal The Logger variable declaration does not contain the static and final modifiers 72

org/apache/fulcrum/intake/model/XmlField.java

Rule Violation Line
LoggerIsNotStaticFinal The Logger variable declaration does not contain the static and final modifiers 102

org/apache/fulcrum/intake/validator/DefaultValidator.java

Rule Violation Line
LoggerIsNotStaticFinal The Logger variable declaration does not contain the static and final modifiers 75

org/apache/fulcrum/intake/validator/StringValidator.java

Rule Violation Line
GuardLogStatement Logger calls should be surrounded by log level guards. 112–113
GuardLogStatement Logger calls should be surrounded by log level guards. 153

Priority 3

org/apache/fulcrum/intake/IntakeError.java

Rule Violation Line
DoNotExtendJavaLangError Exceptions should not extend java.lang.Error 28
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 50
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 62
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 75
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 75

org/apache/fulcrum/intake/IntakeException.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 50
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 62
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 75
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 75

org/apache/fulcrum/intake/IntakeRuntimeException.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 50
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 62
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 75
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 75

org/apache/fulcrum/intake/IntakeService.java

Rule Violation Line
LongVariable Avoid excessively long variable names like SERIAL_XML_DEFAULT 69
LongVariable Avoid excessively long variable names like DEFAULT_POOL_CAPACITY 74

org/apache/fulcrum/intake/IntakeServiceFacade.java

Rule Violation Line
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 42–182
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 65
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 71
MethodArgumentCouldBeFinal Parameter 'instance' is not assigned and could be declared final 82
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 84
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 96
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 98
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 108
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 118
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 120
MethodArgumentCouldBeFinal Parameter 'groupKey' is not assigned and could be declared final 130
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 132
MethodArgumentCouldBeFinal Parameter 'className' is not assigned and could be declared final 146
MethodArgumentCouldBeFinal Parameter 'propName' is not assigned and could be declared final 146
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 148
MethodArgumentCouldBeFinal Parameter 'propName' is not assigned and could be declared final 162
MethodArgumentCouldBeFinal Parameter 'className' is not assigned and could be declared final 162
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 164
MethodArgumentCouldBeFinal Parameter 'service' is not assigned and could be declared final 177

org/apache/fulcrum/intake/IntakeServiceImpl.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 1–936
StdCyclomaticComplexity The class 'IntakeServiceImpl' has a Standard Cyclomatic Complexity of 5 (Highest = 16). 80–936
CyclomaticComplexity The class 'IntakeServiceImpl' has a total cyclomatic complexity of 89 (highest 16). 80–936
ModifiedCyclomaticComplexity The class 'IntakeServiceImpl' has a Modified Cyclomatic Complexity of 5 (Highest = 16). 80–936
AtLeastOneConstructor Each class should declare at least one constructor 80–936
GodClass Possible God Class (WMC=89, ATFD=52, TCC=28.571%) 80–936
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 85
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 88
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 91
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 94
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 97
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 100
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 103
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 106
RedundantFieldInitializer Avoid using redundant field initializer for 'xmlPathes' 106
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 109
RedundantFieldInitializer Avoid using redundant field initializer for 'serialDataPath' 109
AtLeastOneConstructor Each class should declare at least one constructor 115–131
MethodArgumentCouldBeFinal Parameter 'target' is not assigned and could be declared final 121
MethodArgumentCouldBeFinal Parameter 'parent' is not assigned and could be declared final 121
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 147
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 147
MethodArgumentCouldBeFinal Parameter 'appData' is not assigned and could be declared final 148
MethodArgumentCouldBeFinal Parameter 'checkKey' is not assigned and could be declared final 148
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 153
LocalVariableCouldBeFinal Local variable 'keyExists' could be declared final 156
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 161
LocalVariableCouldBeFinal Local variable 'fields' could be declared final 173
LocalVariableCouldBeFinal Local variable 'field' could be declared final 174
LocalVariableCouldBeFinal Local variable 'className' could be declared final 176
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 179
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 180
CyclomaticComplexity The method 'loadSerialized(String, long)' has a cyclomatic complexity of 16. 196–311
ExcessiveMethodLength Avoid really long methods. 196–311
MethodArgumentCouldBeFinal Parameter 'serialDataPath' is not assigned and could be declared final 196
StdCyclomaticComplexity The method 'loadSerialized' has a Standard Cyclomatic Complexity of 16. 196–311
NPathComplexity The method 'loadSerialized(String, long)' has an NPath complexity of 576, current threshold is 200 196–311
MethodArgumentCouldBeFinal Parameter 'timeStamp' is not assigned and could be declared final 196
ModifiedCyclomaticComplexity The method 'loadSerialized' has a Modified Cyclomatic Complexity of 16. 196–311
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 198–200
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 202
LocalVariableCouldBeFinal Local variable 'timer' could be declared final 202
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 206
LocalVariableCouldBeFinal Local variable 'serialDataFile' could be declared final 209
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 213
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 214
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 219
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 220
ShortVariable Avoid variables with short names like in 223
CloseResource Ensure that resources like this FileInputStream object are closed after use 228
LocalVariableCouldBeFinal Local variable 'fin' could be declared final 228
ShortVariable Avoid variables with short names like o 230
LocalVariableCouldBeFinal Local variable 'o' could be declared final 230
LocalVariableCouldBeFinal Local variable 'map' could be declared final 235
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 241
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 243
LocalVariableCouldBeFinal Local variable 'result' could be declared final 246
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 247
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 249
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 255
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 259
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 263
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 267
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 282
LocalVariableCouldBeFinal Local variable 'appData' could be declared final 289
LocalVariableCouldBeFinal Local variable 'group' could be declared final 291
LocalVariableCouldBeFinal Local variable 'field' could be declared final 298
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 308
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 309
StdCyclomaticComplexity The method 'saveSerialized' has a Standard Cyclomatic Complexity of 11. 323–420
ModifiedCyclomaticComplexity The method 'saveSerialized' has a Modified Cyclomatic Complexity of 11. 323–420
CyclomaticComplexity The method 'saveSerialized(String, Map)' has a cyclomatic complexity of 11. 323–420
MethodArgumentCouldBeFinal Parameter 'appDataElements' is not assigned and could be declared final 323
MethodArgumentCouldBeFinal Parameter 'serialDataPath' is not assigned and could be declared final 323
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 326–328
LocalVariableCouldBeFinal Local variable 'timer' could be declared final 330
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 330
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 334
LocalVariableCouldBeFinal Local variable 'serialData' could be declared final 337
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 342
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 344
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 349
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 351
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 357–359
ShortVariable Avoid variables with short names like in 364
LocalVariableCouldBeFinal Local variable 'fout' could be declared final 369
CloseResource Ensure that resources like this FileOutputStream object are closed after use 369
LocalVariableCouldBeFinal Local variable 'fin' could be declared final 376
CloseResource Ensure that resources like this FileInputStream object are closed after use 376
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 380
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 384–386
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 390–391
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 404
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 415
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 419
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 433
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 435
UnusedAssignment The initializer for variable 'group' is never used (overwritten on line 453) 435
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 437
LocalVariableCouldBeFinal Local variable 'appData' could be declared final 437
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 453
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 455
MethodArgumentCouldBeFinal Parameter 'instance' is not assigned and could be declared final 472
LocalVariableCouldBeFinal Local variable 'groupName' could be declared final 476
LocalVariableCouldBeFinal Local variable 'appData' could be declared final 477
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 489
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 491
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 508
LocalVariableCouldBeFinal Local variable 'appData' could be declared final 510
CloseResource Ensure that resources like this KeyedObjectPool object are closed after use 518
LocalVariableCouldBeFinal Local variable 'kop' could be declared final 518
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 520
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 520
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 542
MethodArgumentCouldBeFinal Parameter 'groupKey' is not assigned and could be declared final 555
MethodArgumentCouldBeFinal Parameter 'propName' is not assigned and could be declared final 572
MethodArgumentCouldBeFinal Parameter 'className' is not assigned and could be declared final 572
LocalVariableCouldBeFinal Local variable 'settersForClassName' could be declared final 575
LongVariable Avoid excessively long variable names like settersForClassName 575
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 583
LocalVariableCouldBeFinal Local variable 'pd' could be declared final 587–588
ShortVariable Avoid variables with short names like pd 587
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 595–597
LocalVariableCouldBeFinal Local variable 'gettersForClassName' could be declared final 604
LongVariable Avoid excessively long variable names like gettersForClassName 604
LocalVariableCouldBeFinal Local variable 'getter' could be declared final 608
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 611
MethodArgumentCouldBeFinal Parameter 'className' is not assigned and could be declared final 631
MethodArgumentCouldBeFinal Parameter 'propName' is not assigned and could be declared final 631
LocalVariableCouldBeFinal Local variable 'gettersForClassName' could be declared final 634
LongVariable Avoid excessively long variable names like gettersForClassName 634
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 642
ShortVariable Avoid variables with short names like pd 646
UnusedAssignment The initializer for variable 'pd' is never used (overwritten on line 649) 646
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 654–656
LocalVariableCouldBeFinal Local variable 'settersForClassName' could be declared final 663
LongVariable Avoid excessively long variable names like settersForClassName 663
LocalVariableCouldBeFinal Local variable 'setter' could be declared final 667
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 670
MethodArgumentCouldBeFinal Parameter 'conf' is not assigned and could be declared final 683
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 687
LocalVariableCouldBeFinal Local variable 'nameVal' could be declared final 695
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 696–700
LocalVariableCouldBeFinal Local variable 'val' could be declared final 698
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 703
ConfusingTernary Avoid if (x != y) ..; else ..; 705–712
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 705
LiteralsFirstInComparisons Position literals first in String comparisons 705
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 711
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 714
NcssCount The method 'initialize()' has a NCSS line count of 85. 725–904
NPathComplexity The method 'initialize()' has an NPath complexity of 840, current threshold is 200 725–904
ModifiedCyclomaticComplexity The method 'initialize' has a Modified Cyclomatic Complexity of 15. 725–904
ExcessiveMethodLength Avoid really long methods. 725–904
StdCyclomaticComplexity The method 'initialize' has a Standard Cyclomatic Complexity of 15. 725–904
CyclomaticComplexity The method 'initialize()' has a cyclomatic complexity of 16. 725–904
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 727
UnusedAssignment The initializer for variable 'appDataElements' is never used (overwritten on lines 774 and 795) 727
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 729
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 730
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 731
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 732
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 733
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 734
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 736
LocalVariableCouldBeFinal Local variable 'xmlFiles' could be declared final 736
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 740
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 740
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 740
LocalVariableCouldBeFinal Local variable 'xmlPath' could be declared final 742
LocalVariableCouldBeFinal Local variable 'xmlFile' could be declared final 745
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 745
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 747
LocalVariableCouldBeFinal Local variable 'READ_ERR' could be declared final 751–752
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 754
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 760
LocalVariableCouldBeFinal Local variable 'serializedMap' could be declared final 769
ConfusingTernary Avoid if (x != y) ..; else ..; 771–827
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 775
LocalVariableCouldBeFinal Local variable 'timer' could be declared final 779
LocalVariableCouldBeFinal Local variable 'jaxb' could be declared final 782
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 783
ShortVariable Avoid variables with short names like um 783
LocalVariableCouldBeFinal Local variable 'um' could be declared final 783
LocalVariableCouldBeFinal Local variable 'logEnabledListener' could be declared final 789
LongVariable Avoid excessively long variable names like logEnabledListener 789
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 790
LocalVariableCouldBeFinal Local variable 'schemaURL' could be declared final 792
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 792
LocalVariableCouldBeFinal Local variable 'schemaFactory' could be declared final 793
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 794
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 795
LocalVariableCouldBeFinal Local variable 'xmlFile' could be declared final 797
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 799
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 803
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 804
LocalVariableCouldBeFinal Local variable 'appData' could be declared final 804
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 807
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 819
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 825
LocalVariableCouldBeFinal Local variable 'entry' could be declared final 832
LocalVariableCouldBeFinal Local variable 'glist' could be declared final 839
LocalVariableCouldBeFinal Local variable 'groupPrefix' could be declared final 841
LocalVariableCouldBeFinal Local variable 'i' could be declared final 843
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 843
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 843
LocalVariableCouldBeFinal Local variable 'g' could be declared final 845
ShortVariable Avoid variables with short names like g 845
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 846
LocalVariableCouldBeFinal Local variable 'groupName' could be declared final 846
LongVariable Avoid excessively long variable names like registerUnqualified 848
LocalVariableCouldBeFinal Local variable 'registerUnqualified' could be declared final 848
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 852–855
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 854
LocalVariableCouldBeFinal Local variable 'qualifiedName' could be declared final 860
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 860
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 872–875
LocalVariableCouldBeFinal Local variable 'f' could be declared final 880
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 880
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 885
LocalVariableCouldBeFinal Local variable 'factory' could be declared final 888–889
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 889
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 891
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 891
LocalVariableCouldBeFinal Local variable 'poolConfig' could be declared final 891
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 897
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 900
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 902
MethodArgumentCouldBeFinal Parameter 'context' is not assigned and could be declared final 916
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 918
MethodArgumentCouldBeFinal Parameter 'manager' is not assigned and could be declared final 932

org/apache/fulcrum/intake/Retrievable.java

Rule Violation Line
SignatureDeclareThrowsException A method/constructor should not explicitly throw java.lang.Exception 46

org/apache/fulcrum/intake/model/AppData.java

Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 43–204
DataClass The class 'AppData' is suspected to be a Data Class (WOC=25.000%, NOPA=0, NOAM=6, WMC=17) 43–204
MethodArgumentCouldBeFinal Parameter 'groups' is not assigned and could be declared final 79
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 92
LocalVariableCouldBeFinal Local variable 'groupPrefix' could be declared final 101
LocalVariableCouldBeFinal Local variable 'group' could be declared final 103
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 105
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 107
LocalVariableCouldBeFinal Local variable 'qualifiedGroupName' could be declared final 111
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 111
LongVariable Avoid excessively long variable names like qualifiedGroupName 111
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 119
MethodArgumentCouldBeFinal Parameter 'v' is not assigned and could be declared final 144
ShortVariable Avoid variables with short names like v 144
MethodArgumentCouldBeFinal Parameter 'groupPrefix' is not assigned and could be declared final 182
LocalVariableCouldBeFinal Local variable 'result' could be declared final 194
LocalVariableCouldBeFinal Local variable 'group' could be declared final 197

org/apache/fulcrum/intake/model/BigDecimalField.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 48
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 48
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 60
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 62
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 81
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 83

org/apache/fulcrum/intake/model/BooleanField.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 43
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 43
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 55
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 57
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 76
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 78
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 103
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 130
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 136
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 159
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 161

org/apache/fulcrum/intake/model/DateStringField.java

Rule Violation Line
RedundantFieldInitializer Avoid using redundant field initializer for 'df' 50
ShortVariable Avoid variables with short names like df 50
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 50
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 59
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 59
LocalVariableCouldBeFinal Local variable 'validator' could be declared final 64
SimplifyConditional No need to check for null before an instanceof 65
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 78
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 80
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 107
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 109
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 146
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 157
LocalVariableCouldBeFinal Local variable 'val' could be declared final 164
MethodArgumentCouldBeFinal Parameter 'dateString' is not assigned and could be declared final 185
UnusedAssignment The initializer for variable 'date' is never used (overwritten on lines 193 and 197) 188
LocalVariableCouldBeFinal Local variable 'validator' could be declared final 190
SimplifyConditional No need to check for null before an instanceof 191
UnusedAssignment The initializer for variable 's' is never used (overwritten on lines 215, 219, 223 and 227) 210
ShortVariable Avoid variables with short names like s 210
LocalVariableCouldBeFinal Local variable 'value' could be declared final 211
LocalVariableCouldBeFinal Local variable 'validator' could be declared final 212
SimplifyConditional No need to check for null before an instanceof 221

org/apache/fulcrum/intake/model/DoubleField.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 44
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 44
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 56
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 58
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 77
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 79
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 104
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 132
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 138

org/apache/fulcrum/intake/model/Field.java

Rule Violation Line
GodClass Possible God Class (WMC=116, ATFD=37, TCC=9.991%) 52–1149
StdCyclomaticComplexity The class 'Field' has a Standard Cyclomatic Complexity of 2 (Highest = 11). 52–1149
CyclomaticComplexity The class 'Field' has a total cyclomatic complexity of 116 (highest 13). 52–1149
AbstractNaming Abstract classes should be named AbstractXXX 52–1149
ExcessiveClassLength Avoid really long classes. 52–1149
ModifiedCyclomaticComplexity The class 'Field' has a Modified Cyclomatic Complexity of 2 (Highest = 11). 52–1149
TooManyFields Too many fields 52–1149
TooManyMethods This class has too many methods, consider refactoring it. 53–1149
LongVariable Avoid excessively long variable names like VALUE_IF_ABSENT_KEY 61
LongVariable Avoid excessively long variable names like defaultValidatorPackage 64
LongVariable Avoid excessively long variable names like defaultFieldPackage 67
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 81
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 84
ImmutableField Private field 'mapToProperty' could be made final; it is only initialized in the declaration or constructor. 84
LongVariable Avoid excessively long variable names like validatorClassName 87
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 87
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 93
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 96
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 99
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 102
AvoidFieldNameMatchingMethodName Field isMultiValued has the same name as a method 102
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 108
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 120
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 125
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 128
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 137
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 143
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 146
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 149
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 152
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 155
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 161
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 164
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 167
ImmutableField Private field 'ruleMap' could be made final; it is only initialized in the declaration or constructor. 167
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 181
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 181
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 196
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 207
LocalVariableCouldBeFinal Local variable 'reqRule' could be declared final 225
UnnecessaryWrapperObjectCreation Unnecessary wrapper object creation 228
LocalVariableCouldBeFinal Local variable 'maxLengthRule' could be declared final 232
MethodArgumentCouldBeFinal Parameter 'logger' is not assigned and could be declared final 248
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 250
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 267
AvoidDuplicateLiterals The String literal " in group " appears 4 times in this file; the first occurrence is on line 270 270
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 279
MethodArgumentCouldBeFinal Parameter 'pp' is not assigned and could be declared final 303
ShortVariable Avoid variables with short names like pp 303
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 345
MethodArgumentCouldBeFinal Parameter 'v' is not assigned and could be declared final 445
ShortVariable Avoid variables with short names like v 445
ShortVariable Avoid variables with short names like v 456
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 456
MethodArgumentCouldBeFinal Parameter 'v' is not assigned and could be declared final 456
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 483
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 489
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 490
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 492
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 493
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 494
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 495
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 496
MethodArgumentCouldBeFinal Parameter 'newDisplayName' is not assigned and could be declared final 577
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 597
MethodArgumentCouldBeFinal Parameter 'setFlag' is not assigned and could be declared final 608
MethodArgumentCouldBeFinal Parameter 'validFlag' is not assigned and could be declared final 618
MethodArgumentCouldBeFinal Parameter 'validated' is not assigned and could be declared final 628
StdCyclomaticComplexity The method 'validate' has a Standard Cyclomatic Complexity of 11. 638–716
CyclomaticComplexity The method 'validate()' has a cyclomatic complexity of 11. 638–716
ModifiedCyclomaticComplexity The method 'validate' has a Modified Cyclomatic Complexity of 11. 638–716
ShortVariable Avoid variables with short names like v 641
LocalVariableCouldBeFinal Local variable 'v' could be declared final 641
ConfusingTernary Avoid if (x != y) ..; else ..; 689–710
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the setInitialValue method if you want a default access modifier 747–750
DefaultPackage Use explicit scoping instead of the default package private level 747–750
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 747
ConfusingTernary Avoid if (x != y) ..; else ..; 766–773
DefaultPackage Use explicit scoping instead of the default package private level 784–787
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the setTestValue method if you want a default access modifier 784–787
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 784
ConfusingTernary Avoid if (x != y) ..; else ..; 843–850
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 849
LocalVariableCouldBeFinal Local variable 'res' could be declared final 864
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 876
LinguisticNaming Linguistics Antipattern - The getter 'getProperty' should not return void linguistically 876–900
LocalVariableCouldBeFinal Local variable 't' could be declared final 882
ShortVariable Avoid variables with short names like t 882
IdenticalCatchBranches 'catch' branch identical to 'IllegalAccessException' branch 890–894
IdenticalCatchBranches 'catch' branch identical to 'IllegalAccessException' branch 895–899
LinguisticNaming Linguistics Antipattern - The getter 'getDefault' should not return void linguistically 905–908
CyclomaticComplexity The method 'setProperty(Object)' has a cyclomatic complexity of 13. 918–984
ModifiedCyclomaticComplexity The method 'setProperty' has a Modified Cyclomatic Complexity of 11. 918–984
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 918
StdCyclomaticComplexity The method 'setProperty' has a Standard Cyclomatic Complexity of 11. 918–984
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 922
ConfusingTernary Avoid if (x != y) ..; else ..; 955–967
IdenticalCatchBranches 'catch' branch identical to 'IllegalAccessException' branch 974–978
IdenticalCatchBranches 'catch' branch identical to 'IllegalAccessException' branch 979–983
MethodArgumentCouldBeFinal Parameter 'type' is not assigned and could be declared final 996
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 996
MethodArgumentCouldBeFinal Parameter 'fieldName' is not assigned and could be declared final 997
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 997
MethodArgumentCouldBeFinal Parameter 'e' is not assigned and could be declared final 998
ShortVariable Avoid variables with short names like e 998
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 1003
LongVariable Avoid excessively long variable names like validatorClassName 1100
CyclomaticComplexity The method 'createValidator(String)' has a cyclomatic complexity of 10. 1100–1148
MethodArgumentCouldBeFinal Parameter 'validatorClassName' is not assigned and could be declared final 1100
ShortVariable Avoid variables with short names like v 1103
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 1108
IdenticalCatchBranches 'catch' branch identical to 'InstantiationException' branch 1116–1121

org/apache/fulcrum/intake/model/FieldAdapter.java

Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 32–68
MethodArgumentCouldBeFinal Parameter 'xmlField' is not assigned and could be declared final 41
UnusedAssignment The initializer for variable 'field' is never used (overwritten on line 53) 43
LocalVariableCouldBeFinal Local variable 'type' could be declared final 44
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 49
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 63

org/apache/fulcrum/intake/model/FieldType.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 41
ShortVariable Avoid variables with short names like g 41
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 41
ShortVariable Avoid variables with short names like f 41
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 49
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 49
ShortVariable Avoid variables with short names like g 49
ShortVariable Avoid variables with short names like f 49
ShortVariable Avoid variables with short names like g 57
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 57
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 57
ShortVariable Avoid variables with short names like f 57
ShortVariable Avoid variables with short names like f 65
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 65
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 65
ShortVariable Avoid variables with short names like g 65
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 74
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 74
ShortVariable Avoid variables with short names like f 74
ShortVariable Avoid variables with short names like g 74
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 82
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 82
ShortVariable Avoid variables with short names like g 82
ShortVariable Avoid variables with short names like f 82
ShortVariable Avoid variables with short names like f 90
ShortVariable Avoid variables with short names like g 90
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 90
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 90
ShortVariable Avoid variables with short names like f 98
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 98
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 98
ShortVariable Avoid variables with short names like g 98
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 106
ShortVariable Avoid variables with short names like g 106
ShortVariable Avoid variables with short names like f 106
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 106
ShortVariable Avoid variables with short names like f 115
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 115
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 115
ShortVariable Avoid variables with short names like g 115
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 123
ShortVariable Avoid variables with short names like f 123
ShortVariable Avoid variables with short names like g 123
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 123
ShortVariable Avoid variables with short names like g 131
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 131
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 131
ShortVariable Avoid variables with short names like f 131
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 139
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 139
ShortVariable Avoid variables with short names like f 139
ShortVariable Avoid variables with short names like g 139
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 144
ConfusingTernary Avoid if (x != y) ..; else ..; 149–179
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 156
LocalVariableCouldBeFinal Local variable 'constructor' could be declared final 156
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 158
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 167
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 194
ShortVariable Avoid variables with short names like g 218
ShortVariable Avoid variables with short names like f 218

org/apache/fulcrum/intake/model/FloatField.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 44
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 44
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 56
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 58
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 77
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 79
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 104
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 132
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 138

org/apache/fulcrum/intake/model/Group.java

Rule Violation Line
CyclomaticComplexity The class 'Group' has a total cyclomatic complexity of 80 (highest 9). 59–755
GodClass Possible God Class (WMC=80, ATFD=75, TCC=20.923%) 59–755
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 79
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 85
ImmutableField Private field 'poolCapacity' could be made final; it is only initialized in the declaration or constructor. 91
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 91
LongVariable Avoid excessively long variable names like defaultMapToObject 97
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 97
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 102
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 107
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 112
LooseCoupling Avoid using implementation types like 'LinkedList'; use the interface instead 117
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 123
ShortVariable Avoid variables with short names like pp 128
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 133
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 141
MethodArgumentCouldBeFinal Parameter 'logger' is not assigned and could be declared final 148
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 150
MethodArgumentCouldBeFinal Parameter 'pp' is not assigned and could be declared final 160
ShortVariable Avoid variables with short names like pp 160
MethodArgumentCouldBeFinal Parameter 'key' is not assigned and could be declared final 174
MethodArgumentCouldBeFinal Parameter 'pp' is not assigned and could be declared final 174
ShortVariable Avoid variables with short names like pp 174
LocalVariableCouldBeFinal Local variable 'i' could be declared final 178
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 180
LocalVariableCouldBeFinal Local variable 'i' could be declared final 182
LocalVariableCouldBeFinal Local variable 'field' could be declared final 184
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 185
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 185
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 187
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 199
LocalVariableCouldBeFinal Local variable 'flds' could be declared final 206
LocalVariableCouldBeFinal Local variable 'interfaces' could be declared final 216
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 217–228
LocalVariableCouldBeFinal Local variable 'interfaceFields' could be declared final 219–220
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 223–226
ShortVariable Avoid variables with short names like i 244
LocalVariableCouldBeFinal Local variable 'f' could be declared final 245
MethodArgumentCouldBeFinal Parameter 'pp' is not assigned and could be declared final 322
ShortVariable Avoid variables with short names like pp 322
LocalVariableCouldBeFinal Local variable 'oids' could be declared final 325
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 328
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 331
MethodArgumentCouldBeFinal Parameter 'fieldName' is not assigned and could be declared final 344
MethodArgumentCouldBeFinal Parameter 'inputFields' is not assigned and could be declared final 374
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 376
LocalVariableCouldBeFinal Local variable 'i' could be declared final 387
LocalVariableCouldBeFinal Local variable 'field' could be declared final 389
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 390
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 391
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 394
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 395
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 408
LocalVariableCouldBeFinal Local variable 'flds' could be declared final 419
LocalVariableCouldBeFinal Local variable 'interfaces' could be declared final 429
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 430–441
LocalVariableCouldBeFinal Local variable 'interfaceFields' could be declared final 432–433
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 436–439
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 458
LocalVariableCouldBeFinal Local variable 'flds' could be declared final 463
EmptyCatchBlock Avoid empty catch blocks 472–475
LocalVariableCouldBeFinal Local variable 'interfaces' could be declared final 480
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 481–499
LocalVariableCouldBeFinal Local variable 'interfaceFields' could be declared final 483–484
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 487–497
EmptyCatchBlock Avoid empty catch blocks 493–496
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 515
LinguisticNaming Linguistics Antipattern - The getter 'getProperties' should not return void linguistically 515–547
LocalVariableCouldBeFinal Local variable 'flds' could be declared final 521
LocalVariableCouldBeFinal Local variable 'interfaces' could be declared final 531
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 532–543
LocalVariableCouldBeFinal Local variable 'interfaceFields' could be declared final 534–535
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 538–541
LocalVariableCouldBeFinal Local variable 'groups' could be declared final 557
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 561–567
LocalVariableCouldBeFinal Local variable 'i' could be declared final 568
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 570
LocalVariableCouldBeFinal Local variable 'sb' could be declared final 593
ShortVariable Avoid variables with short names like sb 593
MethodArgumentCouldBeFinal Parameter 'sb' is not assigned and could be declared final 604
ShortVariable Avoid variables with short names like sb 604
LocalVariableCouldBeFinal Local variable 'result' could be declared final 624
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 626
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 627
LocalVariableCouldBeFinal Local variable 'field' could be declared final 632
MethodArgumentCouldBeFinal Parameter 'parent' is not assigned and could be declared final 659
ShortVariable Avoid variables with short names like um 659
MethodArgumentCouldBeFinal Parameter 'um' is not assigned and could be declared final 659
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 664
LocalVariableCouldBeFinal Local variable 'field' could be declared final 666
LocalVariableCouldBeFinal Local variable 'mapToObjectFieldLists' could be declared final 671–672
LongVariable Avoid excessively long variable names like mapToObjectFieldLists 671
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 671
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 672
LocalVariableCouldBeFinal Local variable 'field' could be declared final 675
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 686
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 686
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 694
LocalVariableCouldBeFinal Local variable 'entry' could be declared final 696
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 699
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 699
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 708
CallSuperInConstructor It is a good practice to call super() in a constructor 710–713
MethodArgumentCouldBeFinal Parameter 'appData' is not assigned and could be declared final 710
MethodArgumentCouldBeFinal Parameter 'key' is not assigned and could be declared final 722
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 731
MethodArgumentCouldBeFinal Parameter 'key' is not assigned and could be declared final 742
MethodArgumentCouldBeFinal Parameter 'pooledGroup' is not assigned and could be declared final 742
LocalVariableCouldBeFinal Local variable 'group' could be declared final 744
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 745
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 745
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 746
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 746
LawOfDemeter Potential violation of Law of Demeter (static property access) 747
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 747
LocalVariableCouldBeFinal Local variable 'i' could be declared final 747
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 747
LawOfDemeter Potential violation of Law of Demeter (static property access) 747
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 750
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 752

org/apache/fulcrum/intake/model/IntegerField.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 43
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 43
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 55
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 57
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 76
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 78
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 103
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 131
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 137

org/apache/fulcrum/intake/model/LongField.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 43
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 43
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 55
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 57
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 76
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 78
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 103
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 130
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 136

org/apache/fulcrum/intake/model/Rule.java

Rule Violation Line
AtLeastOneConstructor Each class should declare at least one constructor 41–109
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 49
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 52
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 55
LocalVariableCouldBeFinal Local variable 'result' could be declared final 91

org/apache/fulcrum/intake/model/ShortField.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 43
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 43
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 55
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 57
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 76
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 78
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 103
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 131
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 137
LocalVariableCouldBeFinal Local variable 'value' could be declared final 145
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 153

org/apache/fulcrum/intake/model/StringField.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 47
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 47
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 70
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 84
LocalVariableCouldBeFinal Local variable 'ss' could be declared final 97
ShortVariable Avoid variables with short names like ss 97
LocalVariableCouldBeFinal Local variable 'val' could be declared final 107
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 119
MethodArgumentCouldBeFinal Parameter 'v' is not assigned and could be declared final 119
CyclomaticComplexity The method 'setRequired(boolean, String)' has a cyclomatic complexity of 10. 119–159
ShortVariable Avoid variables with short names like v 119
ShortVariable Avoid variables with short names like ss 126
LocalVariableCouldBeFinal Local variable 'ss' could be declared final 126
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 135–142

org/apache/fulcrum/intake/model/UploadPartField.java

Rule Violation Line
StdCyclomaticComplexity The class 'UploadPartField' has a Standard Cyclomatic Complexity of 3 (Highest = 12). 37–228
ModifiedCyclomaticComplexity The class 'UploadPartField' has a Modified Cyclomatic Complexity of 3 (Highest = 12). 37–228
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 50
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 50
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 65
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 74
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 86
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 95
ShortVariable Avoid variables with short names like vp 120
MethodArgumentCouldBeFinal Parameter 'vp' is not assigned and could be declared final 120
CyclomaticComplexity The method 'validate()' has a cyclomatic complexity of 14. 146–210
StdCyclomaticComplexity The method 'validate' has a Standard Cyclomatic Complexity of 12. 146–210
ModifiedCyclomaticComplexity The method 'validate' has a Modified Cyclomatic Complexity of 12. 146–210
LocalVariableCouldBeFinal Local variable 'pp' could be declared final 148
ShortVariable Avoid variables with short names like pp 148
ShortVariable Avoid variables with short names like ss 151
LocalVariableCouldBeFinal Local variable 'ss' could be declared final 151
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 161–171
ShortVariable Avoid variables with short names like s 181
LocalVariableCouldBeFinal Local variable 's' could be declared final 181
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 182
ConfusingTernary Avoid if (x != y) ..; else ..; 187–206
ShortVariable Avoid variables with short names like pp 218
LocalVariableCouldBeFinal Local variable 'pp' could be declared final 218

org/apache/fulcrum/intake/model/XmlField.java

Rule Violation Line
DataClass The class 'XmlField' is suspected to be a Data Class (WOC=27.273%, NOPA=0, NOAM=15, WMC=38) 53–408
TooManyFields Too many fields 53–408
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 62
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 65
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 68
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 71
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 74
ImmutableField Private field 'type' could be made final; it is only initialized in the declaration or constructor. 74
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 77
RedundantFieldInitializer Avoid using redundant field initializer for 'multiValued' 77
ImmutableField Private field 'multiValued' could be made final; it is only initialized in the declaration or constructor. 77
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 80
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 83
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 86
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 89
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 92
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 95
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 98
ImmutableField Private field 'ruleMap' could be made final; it is only initialized in the declaration or constructor. 98
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 100
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 102
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 109
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 110
MethodArgumentCouldBeFinal Parameter 'logger' is not assigned and could be declared final 117
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 222
MethodArgumentCouldBeFinal Parameter 'rules' is not assigned and could be declared final 296
ShortVariable Avoid variables with short names like um 318
MethodArgumentCouldBeFinal Parameter 'parent' is not assigned and could be declared final 318
MethodArgumentCouldBeFinal Parameter 'um' is not assigned and could be declared final 318
LocalVariableCouldBeFinal Local variable 'rule' could be declared final 324
LocalVariableCouldBeFinal Local variable 'result' could be declared final 348
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 379
LocalVariableCouldBeFinal Local variable 'rule' could be declared final 386
MethodArgumentCouldBeFinal Parameter 'stream' is not assigned and could be declared final 397
MethodArgumentCouldBeFinal Parameter 'stream' is not assigned and could be declared final 403

org/apache/fulcrum/intake/validator/BigDecimalValidator.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 63
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 63
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 63
LocalVariableCouldBeFinal Local variable 'number' could be declared final 65
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 66
AvoidDecimalLiteralsInBigDecimalConstructor Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal 66

org/apache/fulcrum/intake/validator/BooleanValidator.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 69–72
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 83
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 96
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 108

org/apache/fulcrum/intake/validator/DateRangeValidator.java

Rule Violation Line
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the fieldReferences field if you want a default access modifier 70
DefaultPackage Use explicit scoping instead of the default package private level 70
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 70
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the compareCallback field if you want a default access modifier 73
DefaultPackage Use explicit scoping instead of the default package private level 73
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 73
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 78–81
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 90
MethodArgumentCouldBeFinal Parameter 'refValue' is not assigned and could be declared final 107
MethodArgumentCouldBeFinal Parameter 'thisValue' is not assigned and could be declared final 107
MethodArgumentCouldBeFinal Parameter 'compare' is not assigned and could be declared final 107
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 134
LocalVariableCouldBeFinal Local variable 'entry' could be declared final 136
LocalVariableCouldBeFinal Local variable 'key' could be declared final 138
LocalVariableCouldBeFinal Local variable 'constraint' could be declared final 139
LocalVariableCouldBeFinal Local variable 'compare' could be declared final 141
LocalVariableCouldBeFinal Local variable 'fieldref' could be declared final 146
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 146
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 148
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 149
MethodArgumentCouldBeFinal Parameter 'testField' is not assigned and could be declared final 170
LocalVariableCouldBeFinal Local variable 'thisGroup' could be declared final 175
LocalVariableCouldBeFinal Local variable 'stringValues' could be declared final 179
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 181–184
LocalVariableCouldBeFinal Local variable 'testValue' could be declared final 188
UnusedAssignment The initializer for variable 'testDate' is never used (overwritten on line 213) 209
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 220

org/apache/fulcrum/intake/validator/DateStringValidator.java

Rule Violation Line
LongVariable Avoid excessively long variable names like DEFAULT_DATE_MESSAGE 59
RedundantFieldInitializer Avoid using redundant field initializer for 'dateFormats' 63
UnusedAssignment The field initializer for 'dateFormats' is never used (overwritten on line 77) 63
RedundantFieldInitializer Avoid using redundant field initializer for 'dateFormatMessage' 66
RedundantFieldInitializer Avoid using redundant field initializer for 'flexible' 69
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 77
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 87
UnnecessaryWrapperObjectCreation Unnecessary wrapper object creation 122
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 122
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 135
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 149
CyclomaticComplexity The method 'parse(String)' has a cyclomatic complexity of 11. 166–221
MethodArgumentCouldBeFinal Parameter 's' is not assigned and could be declared final 166
ShortVariable Avoid variables with short names like s 166
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 169
LocalVariableCouldBeFinal Local variable 'sdf' could be declared final 176
EmptyCatchBlock Avoid empty catch blocks 187–190
EmptyCatchBlock Avoid empty catch blocks 201–204
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 210
MethodArgumentCouldBeFinal Parameter 'date' is not assigned and could be declared final 230
ShortVariable Avoid variables with short names like s 232
LocalVariableCouldBeFinal Local variable 'sdf' could be declared final 236
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 236
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 265
MethodArgumentCouldBeFinal Parameter 'formats' is not assigned and could be declared final 288
MethodArgumentCouldBeFinal Parameter 'flexible' is not assigned and could be declared final 308

org/apache/fulcrum/intake/validator/DefaultValidator.java

Rule Violation Line
AbstractNaming Abstract classes should be named AbstractXXX 50–383
RedundantFieldInitializer Avoid using redundant field initializer for 'required' 54
RedundantFieldInitializer Avoid using redundant field initializer for 'requiredMessage' 57
RedundantFieldInitializer Avoid using redundant field initializer for 'minLength' 60
RedundantFieldInitializer Avoid using redundant field initializer for 'minLengthMessage' 63
RedundantFieldInitializer Avoid using redundant field initializer for 'maxLength' 66
RedundantFieldInitializer Avoid using redundant field initializer for 'maxLengthMessage' 69
RedundantFieldInitializer Avoid using redundant field initializer for 'errorMessage' 72
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 72
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 75
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 80–83
MethodArgumentCouldBeFinal Parameter 'logger' is not assigned and could be declared final 89
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 91
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 103
LocalVariableCouldBeFinal Local variable 'param' could be declared final 109
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 110
UnnecessaryWrapperObjectCreation Unnecessary wrapper object creation 110
LocalVariableCouldBeFinal Local variable 'param' could be declared final 117
LocalVariableCouldBeFinal Local variable 'param' could be declared final 125
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 139
UnusedAssignment The initializer for variable 'valid' is never used (overwritten on lines 145 and 149) 141
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 163
LocalVariableCouldBeFinal Local variable 'stringValues' could be declared final 168
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 170–173
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 192
UnusedAssignment The initializer for variable 'valid' is never used (overwritten on lines 198 and 202) 194
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 216
CyclomaticComplexity The method 'assertValidity(String)' has a cyclomatic complexity of 12. 216–239
MethodArgumentCouldBeFinal Parameter 'required' is not assigned and could be declared final 279
MethodArgumentCouldBeFinal Parameter 'requiredMessage' is not assigned and could be declared final 299
MethodArgumentCouldBeFinal Parameter 'minLength' is not assigned and could be declared final 319
MethodArgumentCouldBeFinal Parameter 'minLengthMessage' is not assigned and could be declared final 339
MethodArgumentCouldBeFinal Parameter 'maxLength' is not assigned and could be declared final 359
MethodArgumentCouldBeFinal Parameter 'maxLengthMessage' is not assigned and could be declared final 379

org/apache/fulcrum/intake/validator/DoubleValidator.java

Rule Violation Line
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 61
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 61
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 61
LocalVariableCouldBeFinal Local variable 'number' could be declared final 63
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 64

org/apache/fulcrum/intake/validator/FieldReference.java

Rule Violation Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Comparison': nested enums are implicitly static 64–106
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Comparison(String)': enum constructors are implicitly private 80–83
MethodArgumentCouldBeFinal Parameter 'text' is not assigned and could be declared final 80
MethodArgumentCouldBeFinal Parameter 'string' is not assigned and could be declared final 91
LocalVariableCouldBeFinal Local variable 'c' could be declared final 95
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 99
RedundantFieldInitializer Avoid using redundant field initializer for 'compare' 109
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 109
RedundantFieldInitializer Avoid using redundant field initializer for 'fieldName' 112
RedundantFieldInitializer Avoid using redundant field initializer for 'message' 115
MethodArgumentCouldBeFinal Parameter 'compare' is not assigned and could be declared final 136
MethodArgumentCouldBeFinal Parameter 'fieldName' is not assigned and could be declared final 152
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 168
MethodArgumentCouldBeFinal Parameter 'key' is not assigned and could be declared final 179
MethodArgumentCouldBeFinal Parameter 'compareCallback' is not assigned and could be declared final 196
MethodArgumentCouldBeFinal Parameter 'fieldReferences' is not assigned and could be declared final 196
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 197
MethodArgumentCouldBeFinal Parameter 'value' is not assigned and could be declared final 197
LocalVariableCouldBeFinal Local variable 'ref' could be declared final 200
LocalVariableCouldBeFinal Local variable 'refField' could be declared final 207
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 209
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 216
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 218
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 221
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 225
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 231
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 234

org/apache/fulcrum/intake/validator/FileValidator.java

Rule Violation Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 50–53
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 63
LocalVariableCouldBeFinal Local variable 'contentType' could be declared final 67
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 68
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 70
LocalVariableCouldBeFinal Local variable 'matcher' could be declared final 72
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 73
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 75
LocalVariableCouldBeFinal Local variable 'byteSize' could be declared final 81
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 90
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 103

org/apache/fulcrum/intake/validator/FloatValidator.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 61
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 61
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 61
LocalVariableCouldBeFinal Local variable 'number' could be declared final 63
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 64

org/apache/fulcrum/intake/validator/IntegerRangeValidator.java

Rule Violation Line
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the fieldReferences field if you want a default access modifier 69
DefaultPackage Use explicit scoping instead of the default package private level 69
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 69
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the compareCallback field if you want a default access modifier 72
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 72
DefaultPackage Use explicit scoping instead of the default package private level 72
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 77–80
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 89
MethodArgumentCouldBeFinal Parameter 'refValue' is not assigned and could be declared final 106
MethodArgumentCouldBeFinal Parameter 'thisValue' is not assigned and could be declared final 106
MethodArgumentCouldBeFinal Parameter 'compare' is not assigned and could be declared final 106
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 133
LocalVariableCouldBeFinal Local variable 'entry' could be declared final 135
LocalVariableCouldBeFinal Local variable 'key' could be declared final 137
LocalVariableCouldBeFinal Local variable 'constraint' could be declared final 138
LocalVariableCouldBeFinal Local variable 'compare' could be declared final 140
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 145
LocalVariableCouldBeFinal Local variable 'fieldref' could be declared final 145
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 147
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 148
LocalVariableCouldBeFinal Local variable 'thisGroup' could be declared final 174
LocalVariableCouldBeFinal Local variable 'locale' could be declared final 175
LocalVariableCouldBeFinal Local variable 'stringValues' could be declared final 179
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 181–184
LocalVariableCouldBeFinal Local variable 'testValue' could be declared final 188
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 219

org/apache/fulcrum/intake/validator/IntegerValidator.java

Rule Violation Line
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 60
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 60
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 60

org/apache/fulcrum/intake/validator/InvalidMaskException.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 44
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 55
MethodArgumentCouldBeFinal Parameter 'cause' is not assigned and could be declared final 55

org/apache/fulcrum/intake/validator/LongValidator.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 60
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 60
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 60

org/apache/fulcrum/intake/validator/NumberValidator.java

Rule Violation Line
DataClass The class 'NumberValidator' is suspected to be a Data Class (WOC=21.429%, NOPA=0, NOAM=10, WMC=38) 52–341
AbstractNaming Abstract classes should be named AbstractXXX 52–341
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the minValueMessage field if you want a default access modifier 56
DefaultPackage Use explicit scoping instead of the default package private level 56
RedundantFieldInitializer Avoid using redundant field initializer for 'minValueMessage' 56
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the maxValueMessage field if you want a default access modifier 59
DefaultPackage Use explicit scoping instead of the default package private level 59
RedundantFieldInitializer Avoid using redundant field initializer for 'maxValueMessage' 59
RedundantFieldInitializer Avoid using redundant field initializer for 'invalidNumberMessage' 62
LongVariable Avoid excessively long variable names like invalidNumberMessage 62
DefaultPackage Use explicit scoping instead of the default package private level 62
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the invalidNumberMessage field if you want a default access modifier 62
RedundantFieldInitializer Avoid using redundant field initializer for 'minValue' 64
RedundantFieldInitializer Avoid using redundant field initializer for 'maxValue' 65
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 70–73
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 84
LocalVariableCouldBeFinal Local variable 'param' could be declared final 99
LocalVariableCouldBeFinal Local variable 'param' could be declared final 114
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 137
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 149
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 149
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 149
LocalVariableCouldBeFinal Local variable 'nf' could be declared final 151
ShortVariable Avoid variables with short names like nf 151
LocalVariableCouldBeFinal Local variable 'pos' could be declared final 155
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 156
LocalVariableCouldBeFinal Local variable 'number' could be declared final 156
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 167
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 180
LocalVariableCouldBeFinal Local variable 'locale' could be declared final 182
LocalVariableCouldBeFinal Local variable 'stringValues' could be declared final 186
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 188–191
CyclomaticComplexity The method 'assertValidity(String, Locale)' has a cyclomatic complexity of 11. 208–236
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 208
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 208
UnusedAssignment The initializer for variable 'number' is never used (overwritten on line 217) 214
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 222
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 225
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 230
MethodArgumentCouldBeFinal Parameter 'minValueMessage' is not assigned and could be declared final 257
MethodArgumentCouldBeFinal Parameter 'maxValueMessage' is not assigned and could be declared final 277
MethodArgumentCouldBeFinal Parameter 'invalidNumberMessage' is not assigned and could be declared final 297
LongVariable Avoid excessively long variable names like invalidNumberMessage 297
MethodArgumentCouldBeFinal Parameter 'minValue' is not assigned and could be declared final 317
MethodArgumentCouldBeFinal Parameter 'maxValue' is not assigned and could be declared final 337

org/apache/fulcrum/intake/validator/ShortValidator.java

Rule Violation Line
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 60
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 60
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 60

org/apache/fulcrum/intake/validator/StringValidator.java

Rule Violation Line
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 53
RedundantFieldInitializer Avoid using redundant field initializer for 'maskString' 53
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 56
RedundantFieldInitializer Avoid using redundant field initializer for 'maskPattern' 56
RedundantFieldInitializer Avoid using redundant field initializer for 'maskMessage' 59
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 64–67
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 78
LocalVariableCouldBeFinal Local variable 'constraint' could be declared final 83
LocalVariableCouldBeFinal Local variable 'param' could be declared final 86
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 102
LocalVariableCouldBeFinal Local variable 'patternMatch' could be declared final 110
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 110
MethodArgumentCouldBeFinal Parameter 'mask' is not assigned and could be declared final 143
LocalVariableCouldBeFinal Local variable 'maskOptions' could be declared final 149
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 177

org/apache/fulcrum/intake/validator/ValidationException.java

Rule Violation Line
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 45

org/apache/fulcrum/intake/validator/Validator.java

Rule Violation Line
LongVariable Avoid excessively long variable names like FLEXIBLE_RULE_NAME 34
LongVariable Avoid excessively long variable names like INVALID_NUMBER_RULE_NAME 40
LongVariable Avoid excessively long variable names like MAX_LENGTH_RULE_NAME 46
LongVariable Avoid excessively long variable names like MAX_VALUE_RULE_NAME 49
LongVariable Avoid excessively long variable names like MIN_LENGTH_RULE_NAME 52
LongVariable Avoid excessively long variable names like MIN_VALUE_RULE_NAME 55
LongVariable Avoid excessively long variable names like REQUIRED_RULE_NAME 58

Priority 4

org/apache/fulcrum/intake/model/Field.java

Rule Violation Line
UselessParentheses Useless parentheses. 1061
UselessParentheses Useless parentheses. 1073

org/apache/fulcrum/intake/model/Rule.java

Rule Violation Line
ShortClassName Avoid short class names like Rule 41–109

Priority 5

org/apache/fulcrum/intake/IntakeServiceImpl.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'timer' (lines '202'-'311'). 202–311
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'in' (lines '223'-'229'). 223–229
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'serialData' (lines '224'-'236'). 224–236
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'serialData' (lines '236'-'259'). 236–259
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'serialData' (lines '259'-'267'). 259–267
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'timer' (lines '330'-'420'). 330–420
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'group' (lines '435'-'460'). 435–460
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'group' (lines '435'-'453'). 435–453
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'appData' (lines '437'-'460'). 437–460
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'group' (lines '453'-'460'). 453–460
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'pd' (lines '646'-'649'). 646–649
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'appDataElements' (lines '727'-'774'). 727–774
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'appDataElements' (lines '727'-'904'). 727–904
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'xmlFiles' (lines '736'-'904'). 736–904
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'timeStamp' (lines '738'-'904'). 738–904
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'timeStamp' (lines '765'-'904'). 765–904
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'dataFile' (lines '836'-'904'). 836–904
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'groupPrefix' (lines '841'-'904'). 841–904

org/apache/fulcrum/intake/model/AppData.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'groupPrefix' (lines '101'-'124'). 101–124

org/apache/fulcrum/intake/model/BooleanField.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '131'-'135'). 131–135
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '135'-'135'). 135
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '156'-'159'). 156–159

org/apache/fulcrum/intake/model/DateStringField.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '147'-'152'). 147–152
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '152'-'157'). 152–157
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '157'-'152'). 157–152
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'date' (lines '188'-'197'). 188–197
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'date' (lines '188'-'193'). 188–193
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 's' (lines '210'-'215'). 210–215
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 's' (lines '210'-'219'). 210–219
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 's' (lines '210'-'227'). 210–227
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 's' (lines '210'-'223'). 210–223
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'validator' (lines '212'-'230'). 212–230

org/apache/fulcrum/intake/model/DoubleField.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '133'-'137'). 133–137
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '137'-'137'). 137

org/apache/fulcrum/intake/model/Field.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'tmpGetter' (lines '258'-'265'). 258–265
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'tmpSetter' (lines '259'-'277'). 259–277
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'val' (lines '813'-'816'). 813–816
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'val' (lines '816'-'826'). 816–826
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'res' (lines '841'-'849'). 841–849
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'res' (lines '841'-'845'). 841–845
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'v' (lines '1.107'-'1.148'). 1107–1148

org/apache/fulcrum/intake/model/FieldAdapter.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'field' (lines '43'-'57'). 43–57
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'field' (lines '43'-'53'). 43–53

org/apache/fulcrum/intake/model/FloatField.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '133'-'137'). 133–137
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '137'-'137'). 137

org/apache/fulcrum/intake/model/Group.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'nameList' (lines '243'-'247'). 243–247
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'i' (lines '244'-'250'). 244–250
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'nameList' (lines '247'-'247'). 247
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'objs' (lines '324'-'328'). 324–328

org/apache/fulcrum/intake/model/IntegerField.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '132'-'136'). 132–136
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '136'-'136'). 136

org/apache/fulcrum/intake/model/LongField.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '131'-'135'). 131–135
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '135'-'135'). 135

org/apache/fulcrum/intake/model/ShortField.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '132'-'136'). 132–136
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '136'-'136'). 136

org/apache/fulcrum/intake/model/StringField.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'sval' (lines '98'-'101'). 98–101
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'sval' (lines '101'-'101'). 101

org/apache/fulcrum/intake/validator/BooleanValidator.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '111'-'127'). 111–127
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '111'-'120'). 111–120

org/apache/fulcrum/intake/validator/DateRangeValidator.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'126'). 109–126
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'122'). 109–122
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'114'). 109–114
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'114'). 109–114
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'118'). 109–118
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'126'). 109–126
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'118'). 109–118
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'122'). 109–122
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'thisGroup' (lines '175'-'192'). 175–192
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'testDate' (lines '209'-'213'). 209–213
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'testDate' (lines '213'-'234'). 213–234

org/apache/fulcrum/intake/validator/DateStringValidator.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'i' (lines '100'-'100'). 100
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'i' (lines '100'-'124'). 100–124
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'date' (lines '169'-'221'). 169–221
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 's' (lines '232'-'237'). 232–237

org/apache/fulcrum/intake/validator/DefaultValidator.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'valid' (lines '141'-'145'). 141–145
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'valid' (lines '145'-'149'). 145–149
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'valid' (lines '194'-'198'). 194–198
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'valid' (lines '198'-'202'). 198–202
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'retValue' (lines '250'-'254'). 250–254

org/apache/fulcrum/intake/validator/FieldReference.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'comp_true' (lines '202'-'239'). 202–239
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'comp_true' (lines '202'-'223'). 202–223
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'comp_true' (lines '223'-'239'). 223–239

org/apache/fulcrum/intake/validator/FileValidator.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'charset' (lines '68'-'75'). 68–75
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'charset' (lines '68'-'107'). 68–107
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'charset' (lines '75'-'107'). 75–107
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'content' (lines '93'-'98'). 93–98
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'content' (lines '93'-'107'). 93–107
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'content' (lines '98'-'107'). 98–107

org/apache/fulcrum/intake/validator/IntegerRangeValidator.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'113'). 108–113
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'121'). 108–121
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'117'). 108–117
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'113'). 108–113
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'117'). 108–117
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'121'). 108–121
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'125'). 108–125
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'125'). 108–125
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'thisGroup' (lines '174'-'192'). 174–192
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'locale' (lines '175'-'192'). 175–192
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'testInt' (lines '214'-'233'). 214–233

org/apache/fulcrum/intake/validator/NumberValidator.java

Rule Violation Line
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'number' (lines '156'-'169'). 156–169
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'locale' (lines '182'-'197'). 182–197
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'number' (lines '214'-'217'). 214–217
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'number' (lines '217'-'236'). 217–236

Files

org/apache/fulcrum/intake/IntakeError.java

Rule Violation Priority Line
DoNotExtendJavaLangError Exceptions should not extend java.lang.Error 3 28
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 3 50
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 3 62
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 3 75
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 3 75

org/apache/fulcrum/intake/IntakeException.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 3 50
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 3 62
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 3 75
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 3 75

org/apache/fulcrum/intake/IntakeRuntimeException.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 3 50
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 3 62
MethodArgumentCouldBeFinal Parameter 'nested' is not assigned and could be declared final 3 75
MethodArgumentCouldBeFinal Parameter 'msg' is not assigned and could be declared final 3 75

org/apache/fulcrum/intake/IntakeService.java

Rule Violation Priority Line
LongVariable Avoid excessively long variable names like SERIAL_XML_DEFAULT 3 69
LongVariable Avoid excessively long variable names like DEFAULT_POOL_CAPACITY 3 74

org/apache/fulcrum/intake/IntakeServiceFacade.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'IntakeServiceFacade' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Constants)' 1 41–182
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 3 42–182
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 3 65
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 71
MethodArgumentCouldBeFinal Parameter 'instance' is not assigned and could be declared final 3 82
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 84
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 3 96
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 98
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 108
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 3 118
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 120
MethodArgumentCouldBeFinal Parameter 'groupKey' is not assigned and could be declared final 3 130
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 132
MethodArgumentCouldBeFinal Parameter 'className' is not assigned and could be declared final 3 146
MethodArgumentCouldBeFinal Parameter 'propName' is not assigned and could be declared final 3 146
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 148
MethodArgumentCouldBeFinal Parameter 'propName' is not assigned and could be declared final 3 162
MethodArgumentCouldBeFinal Parameter 'className' is not assigned and could be declared final 3 162
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 164
MethodArgumentCouldBeFinal Parameter 'service' is not assigned and could be declared final 3 177

org/apache/fulcrum/intake/IntakeServiceImpl.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 1–936
StdCyclomaticComplexity The class 'IntakeServiceImpl' has a Standard Cyclomatic Complexity of 5 (Highest = 16). 3 80–936
CyclomaticComplexity The class 'IntakeServiceImpl' has a total cyclomatic complexity of 89 (highest 16). 3 80–936
ModifiedCyclomaticComplexity The class 'IntakeServiceImpl' has a Modified Cyclomatic Complexity of 5 (Highest = 16). 3 80–936
AtLeastOneConstructor Each class should declare at least one constructor 3 80–936
GodClass Possible God Class (WMC=89, ATFD=52, TCC=28.571%) 3 80–936
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 85
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 88
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 91
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 94
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 97
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 100
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 103
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 106
RedundantFieldInitializer Avoid using redundant field initializer for 'xmlPathes' 3 106
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 109
RedundantFieldInitializer Avoid using redundant field initializer for 'serialDataPath' 3 109
AtLeastOneConstructor Each class should declare at least one constructor 3 115–131
MethodArgumentCouldBeFinal Parameter 'target' is not assigned and could be declared final 3 121
MethodArgumentCouldBeFinal Parameter 'parent' is not assigned and could be declared final 3 121
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 147
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 3 147
MethodArgumentCouldBeFinal Parameter 'appData' is not assigned and could be declared final 3 148
MethodArgumentCouldBeFinal Parameter 'checkKey' is not assigned and could be declared final 3 148
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 153
LocalVariableCouldBeFinal Local variable 'keyExists' could be declared final 3 156
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 161
LocalVariableCouldBeFinal Local variable 'fields' could be declared final 3 173
LocalVariableCouldBeFinal Local variable 'field' could be declared final 3 174
LocalVariableCouldBeFinal Local variable 'className' could be declared final 3 176
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 179
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 180
CyclomaticComplexity The method 'loadSerialized(String, long)' has a cyclomatic complexity of 16. 3 196–311
ExcessiveMethodLength Avoid really long methods. 3 196–311
MethodArgumentCouldBeFinal Parameter 'serialDataPath' is not assigned and could be declared final 3 196
StdCyclomaticComplexity The method 'loadSerialized' has a Standard Cyclomatic Complexity of 16. 3 196–311
NPathComplexity The method 'loadSerialized(String, long)' has an NPath complexity of 576, current threshold is 200 3 196–311
MethodArgumentCouldBeFinal Parameter 'timeStamp' is not assigned and could be declared final 3 196
ModifiedCyclomaticComplexity The method 'loadSerialized' has a Modified Cyclomatic Complexity of 16. 3 196–311
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 198–200
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 202
LocalVariableCouldBeFinal Local variable 'timer' could be declared final 3 202
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'timer' (lines '202'-'311'). 5 202–311
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 206
LocalVariableCouldBeFinal Local variable 'serialDataFile' could be declared final 3 209
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 213
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 214
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 219
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 220
ShortVariable Avoid variables with short names like in 3 223
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'in' (lines '223'-'229'). 5 223–229
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'serialData' (lines '224'-'236'). 5 224–236
CloseResource Ensure that resources like this FileInputStream object are closed after use 3 228
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 228
LocalVariableCouldBeFinal Local variable 'fin' could be declared final 3 228
ShortVariable Avoid variables with short names like o 3 230
LocalVariableCouldBeFinal Local variable 'o' could be declared final 3 230
LocalVariableCouldBeFinal Local variable 'map' could be declared final 3 235
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'serialData' (lines '236'-'259'). 5 236–259
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 241
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 243
LocalVariableCouldBeFinal Local variable 'result' could be declared final 3 246
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 3 247
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 249
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 255
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'serialData' (lines '259'-'267'). 5 259–267
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 259
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 263
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 267
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 282
LocalVariableCouldBeFinal Local variable 'appData' could be declared final 3 289
LocalVariableCouldBeFinal Local variable 'group' could be declared final 3 291
LocalVariableCouldBeFinal Local variable 'field' could be declared final 3 298
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 308
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 309
StdCyclomaticComplexity The method 'saveSerialized' has a Standard Cyclomatic Complexity of 11. 3 323–420
ModifiedCyclomaticComplexity The method 'saveSerialized' has a Modified Cyclomatic Complexity of 11. 3 323–420
CyclomaticComplexity The method 'saveSerialized(String, Map)' has a cyclomatic complexity of 11. 3 323–420
MethodArgumentCouldBeFinal Parameter 'appDataElements' is not assigned and could be declared final 3 323
MethodArgumentCouldBeFinal Parameter 'serialDataPath' is not assigned and could be declared final 3 323
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 326–328
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'timer' (lines '330'-'420'). 5 330–420
LocalVariableCouldBeFinal Local variable 'timer' could be declared final 3 330
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 330
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 334
LocalVariableCouldBeFinal Local variable 'serialData' could be declared final 3 337
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 3 342
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 344
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 3 349
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 351
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 357–359
ShortVariable Avoid variables with short names like in 3 364
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 369
LocalVariableCouldBeFinal Local variable 'fout' could be declared final 3 369
CloseResource Ensure that resources like this FileOutputStream object are closed after use 3 369
LocalVariableCouldBeFinal Local variable 'fin' could be declared final 3 376
CloseResource Ensure that resources like this FileInputStream object are closed after use 3 376
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 376
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 380
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 384–386
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 390–391
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 404
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 415
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 419
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 3 433
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 435
UnusedAssignment The initializer for variable 'group' is never used (overwritten on line 453) 3 435
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'group' (lines '435'-'460'). 5 435–460
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'group' (lines '435'-'453'). 5 435–453
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 437
LocalVariableCouldBeFinal Local variable 'appData' could be declared final 3 437
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'appData' (lines '437'-'460'). 5 437–460
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 453
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'group' (lines '453'-'460'). 5 453–460
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 455
MethodArgumentCouldBeFinal Parameter 'instance' is not assigned and could be declared final 3 472
LocalVariableCouldBeFinal Local variable 'groupName' could be declared final 3 476
LocalVariableCouldBeFinal Local variable 'appData' could be declared final 3 477
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 489
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 491
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 3 508
LocalVariableCouldBeFinal Local variable 'appData' could be declared final 3 510
CloseResource Ensure that resources like this KeyedObjectPool object are closed after use 3 518
LocalVariableCouldBeFinal Local variable 'kop' could be declared final 3 518
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 520
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 520
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 3 542
MethodArgumentCouldBeFinal Parameter 'groupKey' is not assigned and could be declared final 3 555
MethodArgumentCouldBeFinal Parameter 'propName' is not assigned and could be declared final 3 572
MethodArgumentCouldBeFinal Parameter 'className' is not assigned and could be declared final 3 572
LocalVariableCouldBeFinal Local variable 'settersForClassName' could be declared final 3 575
LongVariable Avoid excessively long variable names like settersForClassName 3 575
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 583
LocalVariableCouldBeFinal Local variable 'pd' could be declared final 3 587–588
ShortVariable Avoid variables with short names like pd 3 587
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 595–597
LocalVariableCouldBeFinal Local variable 'gettersForClassName' could be declared final 3 604
LongVariable Avoid excessively long variable names like gettersForClassName 3 604
LocalVariableCouldBeFinal Local variable 'getter' could be declared final 3 608
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 611
MethodArgumentCouldBeFinal Parameter 'className' is not assigned and could be declared final 3 631
MethodArgumentCouldBeFinal Parameter 'propName' is not assigned and could be declared final 3 631
LocalVariableCouldBeFinal Local variable 'gettersForClassName' could be declared final 3 634
LongVariable Avoid excessively long variable names like gettersForClassName 3 634
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 642
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'pd' (lines '646'-'649'). 5 646–649
ShortVariable Avoid variables with short names like pd 3 646
UnusedAssignment The initializer for variable 'pd' is never used (overwritten on line 649) 3 646
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 654–656
LocalVariableCouldBeFinal Local variable 'settersForClassName' could be declared final 3 663
LongVariable Avoid excessively long variable names like settersForClassName 3 663
LocalVariableCouldBeFinal Local variable 'setter' could be declared final 3 667
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 670
MethodArgumentCouldBeFinal Parameter 'conf' is not assigned and could be declared final 3 683
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 687
LocalVariableCouldBeFinal Local variable 'nameVal' could be declared final 3 695
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 696–700
LocalVariableCouldBeFinal Local variable 'val' could be declared final 3 698
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 703
ConfusingTernary Avoid if (x != y) ..; else ..; 3 705–712
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 705
LiteralsFirstInComparisons Position literals first in String comparisons 3 705
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 711
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 714
NcssCount The method 'initialize()' has a NCSS line count of 85. 3 725–904
NPathComplexity The method 'initialize()' has an NPath complexity of 840, current threshold is 200 3 725–904
ModifiedCyclomaticComplexity The method 'initialize' has a Modified Cyclomatic Complexity of 15. 3 725–904
ExcessiveMethodLength Avoid really long methods. 3 725–904
StdCyclomaticComplexity The method 'initialize' has a Standard Cyclomatic Complexity of 15. 3 725–904
CyclomaticComplexity The method 'initialize()' has a cyclomatic complexity of 16. 3 725–904
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 727
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'appDataElements' (lines '727'-'774'). 5 727–774
UnusedAssignment The initializer for variable 'appDataElements' is never used (overwritten on lines 774 and 795) 3 727
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'appDataElements' (lines '727'-'904'). 5 727–904
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 729
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 730
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 731
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 732
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 733
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 734
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 736
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'xmlFiles' (lines '736'-'904'). 5 736–904
LocalVariableCouldBeFinal Local variable 'xmlFiles' could be declared final 3 736
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'timeStamp' (lines '738'-'904'). 5 738–904
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 740
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 740
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 740
LocalVariableCouldBeFinal Local variable 'xmlPath' could be declared final 3 742
LocalVariableCouldBeFinal Local variable 'xmlFile' could be declared final 3 745
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 745
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 747
LocalVariableNamingConventions The local variable name 'READ_ERR' doesn't match '[a-z][a-zA-Z0-9]*' 1 751
LocalVariableCouldBeFinal Local variable 'READ_ERR' could be declared final 3 751–752
VariableNamingConventions Variables should start with a lowercase character, 'READ_ERR' starts with uppercase character. 1 751
VariableNamingConventions Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'READ_ERR' is not final. 1 751
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 754
AvoidThrowingRawExceptionTypes Avoid throwing raw exception types. 1 755
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 760
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'timeStamp' (lines '765'-'904'). 5 765–904
LocalVariableCouldBeFinal Local variable 'serializedMap' could be declared final 3 769
ConfusingTernary Avoid if (x != y) ..; else ..; 3 771–827
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 775
LocalVariableCouldBeFinal Local variable 'timer' could be declared final 3 779
LocalVariableCouldBeFinal Local variable 'jaxb' could be declared final 3 782
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 783
ShortVariable Avoid variables with short names like um 3 783
LocalVariableCouldBeFinal Local variable 'um' could be declared final 3 783
LocalVariableCouldBeFinal Local variable 'logEnabledListener' could be declared final 3 789
LongVariable Avoid excessively long variable names like logEnabledListener 3 789
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 790
LocalVariableCouldBeFinal Local variable 'schemaURL' could be declared final 3 792
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 792
LocalVariableCouldBeFinal Local variable 'schemaFactory' could be declared final 3 793
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 794
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 795
LocalVariableCouldBeFinal Local variable 'xmlFile' could be declared final 3 797
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 799
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 803
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 803
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 804
LocalVariableCouldBeFinal Local variable 'appData' could be declared final 3 804
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 807
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 819
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 825
LocalVariableCouldBeFinal Local variable 'entry' could be declared final 3 832
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'dataFile' (lines '836'-'904'). 5 836–904
LocalVariableCouldBeFinal Local variable 'glist' could be declared final 3 839
LocalVariableCouldBeFinal Local variable 'groupPrefix' could be declared final 3 841
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'groupPrefix' (lines '841'-'904'). 5 841–904
LocalVariableCouldBeFinal Local variable 'i' could be declared final 3 843
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 843
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 843
LocalVariableCouldBeFinal Local variable 'g' could be declared final 3 845
ShortVariable Avoid variables with short names like g 3 845
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 846
LocalVariableCouldBeFinal Local variable 'groupName' could be declared final 3 846
LongVariable Avoid excessively long variable names like registerUnqualified 3 848
LocalVariableCouldBeFinal Local variable 'registerUnqualified' could be declared final 3 848
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 852–855
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 854
LocalVariableCouldBeFinal Local variable 'qualifiedName' could be declared final 3 860
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 860
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 872–875
LocalVariableCouldBeFinal Local variable 'f' could be declared final 3 880
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 880
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 885
LocalVariableCouldBeFinal Local variable 'factory' could be declared final 3 888–889
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 889
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 891
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 891
LocalVariableCouldBeFinal Local variable 'poolConfig' could be declared final 3 891
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 897
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 900
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 902
MethodArgumentCouldBeFinal Parameter 'context' is not assigned and could be declared final 3 916
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 918
MethodArgumentCouldBeFinal Parameter 'manager' is not assigned and could be declared final 3 932

org/apache/fulcrum/intake/Retrievable.java

Rule Violation Priority Line
SignatureDeclareThrowsException A method/constructor should not explicitly throw java.lang.Exception 3 46

org/apache/fulcrum/intake/model/AppData.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 43–204
DataClass The class 'AppData' is suspected to be a Data Class (WOC=25.000%, NOPA=0, NOAM=6, WMC=17) 3 43–204
MethodArgumentCouldBeFinal Parameter 'groups' is not assigned and could be declared final 3 79
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 3 92
LocalVariableCouldBeFinal Local variable 'groupPrefix' could be declared final 3 101
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'groupPrefix' (lines '101'-'124'). 5 101–124
LocalVariableCouldBeFinal Local variable 'group' could be declared final 3 103
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 105
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 107
LocalVariableCouldBeFinal Local variable 'qualifiedGroupName' could be declared final 3 111
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 111
LongVariable Avoid excessively long variable names like qualifiedGroupName 3 111
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 119
MethodArgumentCouldBeFinal Parameter 'v' is not assigned and could be declared final 3 144
ShortVariable Avoid variables with short names like v 3 144
MethodArgumentCouldBeFinal Parameter 'groupPrefix' is not assigned and could be declared final 3 182
LocalVariableCouldBeFinal Local variable 'result' could be declared final 3 194
LocalVariableCouldBeFinal Local variable 'group' could be declared final 3 197

org/apache/fulcrum/intake/model/BigDecimalField.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 48
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 48
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 60
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 62
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 81
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 83

org/apache/fulcrum/intake/model/BooleanField.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 43
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 43
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 55
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 57
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 76
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 78
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 103
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 3 130
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '131'-'135'). 5 131–135
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '135'-'135'). 5 135
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 136
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '156'-'159'). 5 156–159
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 159
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 161
GuardLogStatement Logger calls should be surrounded by log level guards. 2 163

org/apache/fulcrum/intake/model/DateStringField.java

Rule Violation Priority Line
RedundantFieldInitializer Avoid using redundant field initializer for 'df' 3 50
ShortVariable Avoid variables with short names like df 3 50
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 50
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 59
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 59
LocalVariableCouldBeFinal Local variable 'validator' could be declared final 3 64
SimplifyConditional No need to check for null before an instanceof 3 65
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 78
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 80
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 107
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 109
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 3 146
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '147'-'152'). 5 147–152
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '152'-'157'). 5 152–157
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 157
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '157'-'152'). 5 157–152
LocalVariableCouldBeFinal Local variable 'val' could be declared final 3 164
MethodArgumentCouldBeFinal Parameter 'dateString' is not assigned and could be declared final 3 185
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'date' (lines '188'-'197'). 5 188–197
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'date' (lines '188'-'193'). 5 188–193
UnusedAssignment The initializer for variable 'date' is never used (overwritten on lines 193 and 197) 3 188
LocalVariableCouldBeFinal Local variable 'validator' could be declared final 3 190
SimplifyConditional No need to check for null before an instanceof 3 191
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 's' (lines '210'-'215'). 5 210–215
UnusedAssignment The initializer for variable 's' is never used (overwritten on lines 215, 219, 223 and 227) 3 210
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 's' (lines '210'-'219'). 5 210–219
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 's' (lines '210'-'227'). 5 210–227
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 's' (lines '210'-'223'). 5 210–223
ShortVariable Avoid variables with short names like s 3 210
LocalVariableCouldBeFinal Local variable 'value' could be declared final 3 211
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'validator' (lines '212'-'230'). 5 212–230
LocalVariableCouldBeFinal Local variable 'validator' could be declared final 3 212
SimplifyConditional No need to check for null before an instanceof 3 221

org/apache/fulcrum/intake/model/DoubleField.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 44
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 44
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 56
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 58
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 77
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 79
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 104
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 3 132
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '133'-'137'). 5 133–137
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '137'-'137'). 5 137
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 138

org/apache/fulcrum/intake/model/Field.java

Rule Violation Priority Line
GodClass Possible God Class (WMC=116, ATFD=37, TCC=9.991%) 3 52–1149
StdCyclomaticComplexity The class 'Field' has a Standard Cyclomatic Complexity of 2 (Highest = 11). 3 52–1149
CyclomaticComplexity The class 'Field' has a total cyclomatic complexity of 116 (highest 13). 3 52–1149
AbstractNaming Abstract classes should be named AbstractXXX 3 52–1149
ExcessiveClassLength Avoid really long classes. 3 52–1149
ModifiedCyclomaticComplexity The class 'Field' has a Modified Cyclomatic Complexity of 2 (Highest = 11). 3 52–1149
TooManyFields Too many fields 3 52–1149
TooManyMethods This class has too many methods, consider refactoring it. 3 53–1149
LongVariable Avoid excessively long variable names like VALUE_IF_ABSENT_KEY 3 61
LongVariable Avoid excessively long variable names like defaultValidatorPackage 3 64
FieldNamingConventions The public constant name 'defaultValidatorPackage' doesn't match '[A-Z][A-Z_0-9]*' 1 64
VariableNamingConventions Variables that are final and static should be all capitals, 'defaultValidatorPackage' is not all capitals. 1 64
FieldNamingConventions The public constant name 'defaultFieldPackage' doesn't match '[A-Z][A-Z_0-9]*' 1 67
VariableNamingConventions Variables that are final and static should be all capitals, 'defaultFieldPackage' is not all capitals. 1 67
LongVariable Avoid excessively long variable names like defaultFieldPackage 3 67
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 81
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 84
ImmutableField Private field 'mapToProperty' could be made final; it is only initialized in the declaration or constructor. 3 84
LongVariable Avoid excessively long variable names like validatorClassName 3 87
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 87
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 93
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 96
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 99
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 102
AvoidFieldNameMatchingMethodName Field isMultiValued has the same name as a method 3 102
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 108
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 120
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 125
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 128
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 137
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 143
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 146
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 149
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 152
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 155
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 161
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 164
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 167
ImmutableField Private field 'ruleMap' could be made final; it is only initialized in the declaration or constructor. 3 167
LoggerIsNotStaticFinal The Logger variable declaration does not contain the static and final modifiers 2 170
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 181
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 181
ConstructorCallsOverridableMethod Overridable method 'enableLogging' called during object construction 1 183
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 196
GuardLogStatement Logger calls should be surrounded by log level guards. 2 198–200
ConstructorCallsOverridableMethod Overridable method 'getDisplayName' called during object construction 1 199
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 207
GuardLogStatement Logger calls should be surrounded by log level guards. 2 209–211
ConstructorCallsOverridableMethod Overridable method 'getDisplayName' called during object construction 1 210
ConstructorCallsOverridableMethod Overridable method 'getDefaultValidator' called during object construction 1 217
LocalVariableCouldBeFinal Local variable 'reqRule' could be declared final 3 225
UnnecessaryWrapperObjectCreation Unnecessary wrapper object creation 3 228
LocalVariableCouldBeFinal Local variable 'maxLengthRule' could be declared final 3 232
MethodArgumentCouldBeFinal Parameter 'logger' is not assigned and could be declared final 3 248
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 250
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'tmpGetter' (lines '258'-'265'). 5 258–265
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'tmpSetter' (lines '259'-'277'). 5 259–277
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 267
GuardLogStatement Logger calls should be surrounded by log level guards. 2 269–273
AvoidDuplicateLiterals The String literal " in group " appears 4 times in this file; the first occurrence is on line 270 3 270
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 279
GuardLogStatement Logger calls should be surrounded by log level guards. 2 281–285
MethodArgumentCouldBeFinal Parameter 'pp' is not assigned and could be declared final 3 303
ShortVariable Avoid variables with short names like pp 3 303
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 3 345
MethodArgumentCouldBeFinal Parameter 'v' is not assigned and could be declared final 3 445
ShortVariable Avoid variables with short names like v 3 445
ShortVariable Avoid variables with short names like v 3 456
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 3 456
MethodArgumentCouldBeFinal Parameter 'v' is not assigned and could be declared final 3 456
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 483
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 489
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 490
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 492
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 493
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 494
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 495
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 496
MethodArgumentCouldBeFinal Parameter 'newDisplayName' is not assigned and could be declared final 3 577
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 3 597
MethodArgumentCouldBeFinal Parameter 'setFlag' is not assigned and could be declared final 3 608
MethodArgumentCouldBeFinal Parameter 'validFlag' is not assigned and could be declared final 3 618
MethodArgumentCouldBeFinal Parameter 'validated' is not assigned and could be declared final 3 628
StdCyclomaticComplexity The method 'validate' has a Standard Cyclomatic Complexity of 11. 3 638–716
CyclomaticComplexity The method 'validate()' has a cyclomatic complexity of 11. 3 638–716
ModifiedCyclomaticComplexity The method 'validate' has a Modified Cyclomatic Complexity of 11. 3 638–716
GuardLogStatement Logger calls should be surrounded by log level guards. 2 640
ShortVariable Avoid variables with short names like v 3 641
LocalVariableCouldBeFinal Local variable 'v' could be declared final 3 641
GuardLogStatement Logger calls should be surrounded by log level guards. 2 654
ConfusingTernary Avoid if (x != y) ..; else ..; 3 689–710
GuardLogStatement Logger calls should be surrounded by log level guards. 2 698
GuardLogStatement Logger calls should be surrounded by log level guards. 2 703
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the setInitialValue method if you want a default access modifier 3 747–750
DefaultPackage Use explicit scoping instead of the default package private level 3 747–750
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 3 747
ConfusingTernary Avoid if (x != y) ..; else ..; 3 766–773
DefaultPackage Use explicit scoping instead of the default package private level 3 784–787
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the setTestValue method if you want a default access modifier 3 784–787
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 3 784
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'val' (lines '813'-'816'). 5 813–816
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'val' (lines '816'-'826'). 5 816–826
GuardLogStatement Logger calls should be surrounded by log level guards. 2 820–821
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'res' (lines '841'-'849'). 5 841–849
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'res' (lines '841'-'845'). 5 841–845
ConfusingTernary Avoid if (x != y) ..; else ..; 3 843–850
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 849
LocalVariableCouldBeFinal Local variable 'res' could be declared final 3 864
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 3 876
LinguisticNaming Linguistics Antipattern - The getter 'getProperty' should not return void linguistically 3 876–900
LocalVariableCouldBeFinal Local variable 't' could be declared final 3 882
ShortVariable Avoid variables with short names like t 3 882
IdenticalCatchBranches 'catch' branch identical to 'IllegalAccessException' branch 3 890–894
IdenticalCatchBranches 'catch' branch identical to 'IllegalAccessException' branch 3 895–899
LinguisticNaming Linguistics Antipattern - The getter 'getDefault' should not return void linguistically 3 905–908
CyclomaticComplexity The method 'setProperty(Object)' has a cyclomatic complexity of 13. 3 918–984
ModifiedCyclomaticComplexity The method 'setProperty' has a Modified Cyclomatic Complexity of 11. 3 918–984
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 3 918
StdCyclomaticComplexity The method 'setProperty' has a Standard Cyclomatic Complexity of 11. 3 918–984
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 922
ConfusingTernary Avoid if (x != y) ..; else ..; 3 955–967
IdenticalCatchBranches 'catch' branch identical to 'IllegalAccessException' branch 3 974–978
IdenticalCatchBranches 'catch' branch identical to 'IllegalAccessException' branch 3 979–983
MethodArgumentCouldBeFinal Parameter 'type' is not assigned and could be declared final 3 996
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 3 996
MethodArgumentCouldBeFinal Parameter 'fieldName' is not assigned and could be declared final 3 997
MethodArgumentCouldBeFinal Parameter 'groupName' is not assigned and could be declared final 3 997
MethodArgumentCouldBeFinal Parameter 'e' is not assigned and could be declared final 3 998
ShortVariable Avoid variables with short names like e 3 998
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 1003
UselessParentheses Useless parentheses. 4 1061
UselessParentheses Useless parentheses. 4 1073
LongVariable Avoid excessively long variable names like validatorClassName 3 1100
CyclomaticComplexity The method 'createValidator(String)' has a cyclomatic complexity of 10. 3 1100–1148
MethodArgumentCouldBeFinal Parameter 'validatorClassName' is not assigned and could be declared final 3 1100
ShortVariable Avoid variables with short names like v 3 1103
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'v' (lines '1.107'-'1.148'). 5 1107–1148
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 1108
IdenticalCatchBranches 'catch' branch identical to 'InstantiationException' branch 3 1116–1121

org/apache/fulcrum/intake/model/FieldAdapter.java

Rule Violation Priority Line
AtLeastOneConstructor Each class should declare at least one constructor 3 32–68
MethodArgumentCouldBeFinal Parameter 'xmlField' is not assigned and could be declared final 3 41
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'field' (lines '43'-'57'). 5 43–57
UnusedAssignment The initializer for variable 'field' is never used (overwritten on line 53) 3 43
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'field' (lines '43'-'53'). 5 43–53
LocalVariableCouldBeFinal Local variable 'type' could be declared final 3 44
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 49
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 63

org/apache/fulcrum/intake/model/FieldType.java

Rule Violation Priority Line
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 41–44
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 41
ShortVariable Avoid variables with short names like g 3 41
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 41
ShortVariable Avoid variables with short names like f 3 41
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 49
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 49
ShortVariable Avoid variables with short names like g 3 49
ShortVariable Avoid variables with short names like f 3 49
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 49–52
ShortVariable Avoid variables with short names like g 3 57
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 57–60
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 57
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 57
ShortVariable Avoid variables with short names like f 3 57
ShortVariable Avoid variables with short names like f 3 65
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 65
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 65
ShortVariable Avoid variables with short names like g 3 65
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 65–68
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 74
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 74
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 74–77
ShortVariable Avoid variables with short names like f 3 74
ShortVariable Avoid variables with short names like g 3 74
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 82–85
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 82
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 82
ShortVariable Avoid variables with short names like g 3 82
ShortVariable Avoid variables with short names like f 3 82
ShortVariable Avoid variables with short names like f 3 90
ShortVariable Avoid variables with short names like g 3 90
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 90
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 90
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 90–93
ShortVariable Avoid variables with short names like f 3 98
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 98
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 98–101
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 98
ShortVariable Avoid variables with short names like g 3 98
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 106
ShortVariable Avoid variables with short names like g 3 106
ShortVariable Avoid variables with short names like f 3 106
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 106
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 115–118
ShortVariable Avoid variables with short names like f 3 115
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 115
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 115
ShortVariable Avoid variables with short names like g 3 115
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 123
ShortVariable Avoid variables with short names like f 3 123
ShortVariable Avoid variables with short names like g 3 123
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 123–126
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 123
SingletonClassReturningNewInstance getInstance method always creates a new object and hence does not comply to Singleton Design Pattern behaviour. Please review 2 131–134
ShortVariable Avoid variables with short names like g 3 131
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 131
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 131
ShortVariable Avoid variables with short names like f 3 131
MethodArgumentCouldBeFinal Parameter 'g' is not assigned and could be declared final 3 139
MethodArgumentCouldBeFinal Parameter 'f' is not assigned and could be declared final 3 139
ShortVariable Avoid variables with short names like f 3 139
ShortVariable Avoid variables with short names like g 3 139
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 144
ConfusingTernary Avoid if (x != y) ..; else ..; 3 149–179
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 156
LocalVariableCouldBeFinal Local variable 'constructor' could be declared final 3 156
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 158
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 167
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 3 194
ShortVariable Avoid variables with short names like g 3 218
ShortVariable Avoid variables with short names like f 3 218

org/apache/fulcrum/intake/model/FloatField.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 44
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 44
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 56
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 58
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 77
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 79
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 104
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 3 132
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '133'-'137'). 5 133–137
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '137'-'137'). 5 137
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 138

org/apache/fulcrum/intake/model/Group.java

Rule Violation Priority Line
CyclomaticComplexity The class 'Group' has a total cyclomatic complexity of 80 (highest 9). 3 59–755
GodClass Possible God Class (WMC=80, ATFD=75, TCC=20.923%) 3 59–755
LoggerIsNotStaticFinal The Logger variable declaration does not contain the static and final modifiers 2 72
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 79
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 85
ImmutableField Private field 'poolCapacity' could be made final; it is only initialized in the declaration or constructor. 3 91
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 91
LongVariable Avoid excessively long variable names like defaultMapToObject 3 97
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 97
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 102
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 107
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 112
LooseCoupling Avoid using implementation types like 'LinkedList'; use the interface instead 3 117
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 123
ShortVariable Avoid variables with short names like pp 3 128
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 133
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 141
MethodArgumentCouldBeFinal Parameter 'logger' is not assigned and could be declared final 3 148
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 150
MethodArgumentCouldBeFinal Parameter 'pp' is not assigned and could be declared final 3 160
ShortVariable Avoid variables with short names like pp 3 160
MethodArgumentCouldBeFinal Parameter 'key' is not assigned and could be declared final 3 174
MethodArgumentCouldBeFinal Parameter 'pp' is not assigned and could be declared final 3 174
ShortVariable Avoid variables with short names like pp 3 174
LocalVariableCouldBeFinal Local variable 'i' could be declared final 3 178
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 180
LocalVariableCouldBeFinal Local variable 'i' could be declared final 3 182
LocalVariableCouldBeFinal Local variable 'field' could be declared final 3 184
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 185
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 185
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 187
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 3 199
LocalVariableCouldBeFinal Local variable 'flds' could be declared final 3 206
LocalVariableCouldBeFinal Local variable 'interfaces' could be declared final 3 216
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 217–228
LocalVariableCouldBeFinal Local variable 'interfaceFields' could be declared final 3 219–220
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 223–226
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'nameList' (lines '243'-'247'). 5 243–247
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'i' (lines '244'-'250'). 5 244–250
ShortVariable Avoid variables with short names like i 3 244
LocalVariableCouldBeFinal Local variable 'f' could be declared final 3 245
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'nameList' (lines '247'-'247'). 5 247
MethodArgumentCouldBeFinal Parameter 'pp' is not assigned and could be declared final 3 322
ShortVariable Avoid variables with short names like pp 3 322
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'objs' (lines '324'-'328'). 5 324–328
LocalVariableCouldBeFinal Local variable 'oids' could be declared final 3 325
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 328
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 331
MethodArgumentCouldBeFinal Parameter 'fieldName' is not assigned and could be declared final 3 344
MethodArgumentCouldBeFinal Parameter 'inputFields' is not assigned and could be declared final 3 374
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 376
LocalVariableCouldBeFinal Local variable 'i' could be declared final 3 387
LocalVariableCouldBeFinal Local variable 'field' could be declared final 3 389
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 390
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 391
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 394
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 395
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 3 408
LocalVariableCouldBeFinal Local variable 'flds' could be declared final 3 419
LocalVariableCouldBeFinal Local variable 'interfaces' could be declared final 3 429
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 430–441
LocalVariableCouldBeFinal Local variable 'interfaceFields' could be declared final 3 432–433
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 436–439
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 3 458
LocalVariableCouldBeFinal Local variable 'flds' could be declared final 3 463
EmptyCatchBlock Avoid empty catch blocks 3 472–475
LocalVariableCouldBeFinal Local variable 'interfaces' could be declared final 3 480
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 481–499
LocalVariableCouldBeFinal Local variable 'interfaceFields' could be declared final 3 483–484
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 487–497
EmptyCatchBlock Avoid empty catch blocks 3 493–496
MethodArgumentCouldBeFinal Parameter 'obj' is not assigned and could be declared final 3 515
LinguisticNaming Linguistics Antipattern - The getter 'getProperties' should not return void linguistically 3 515–547
LocalVariableCouldBeFinal Local variable 'flds' could be declared final 3 521
LocalVariableCouldBeFinal Local variable 'interfaces' could be declared final 3 531
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 532–543
LocalVariableCouldBeFinal Local variable 'interfaceFields' could be declared final 3 534–535
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 538–541
LocalVariableCouldBeFinal Local variable 'groups' could be declared final 3 557
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 561–567
LocalVariableCouldBeFinal Local variable 'i' could be declared final 3 568
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 570
LocalVariableCouldBeFinal Local variable 'sb' could be declared final 3 593
ShortVariable Avoid variables with short names like sb 3 593
MethodArgumentCouldBeFinal Parameter 'sb' is not assigned and could be declared final 3 604
ShortVariable Avoid variables with short names like sb 3 604
LocalVariableCouldBeFinal Local variable 'result' could be declared final 3 624
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 626
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 627
LocalVariableCouldBeFinal Local variable 'field' could be declared final 3 632
MethodArgumentCouldBeFinal Parameter 'parent' is not assigned and could be declared final 3 659
ShortVariable Avoid variables with short names like um 3 659
MethodArgumentCouldBeFinal Parameter 'um' is not assigned and could be declared final 3 659
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 664
LocalVariableCouldBeFinal Local variable 'field' could be declared final 3 666
LocalVariableCouldBeFinal Local variable 'mapToObjectFieldLists' could be declared final 3 671–672
LongVariable Avoid excessively long variable names like mapToObjectFieldLists 3 671
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 671
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 672
LocalVariableCouldBeFinal Local variable 'field' could be declared final 3 675
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 686
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 686
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 694
LocalVariableCouldBeFinal Local variable 'entry' could be declared final 3 696
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 699
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 699
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 708
CallSuperInConstructor It is a good practice to call super() in a constructor 3 710–713
MethodArgumentCouldBeFinal Parameter 'appData' is not assigned and could be declared final 3 710
MethodArgumentCouldBeFinal Parameter 'key' is not assigned and could be declared final 3 722
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 731
MethodArgumentCouldBeFinal Parameter 'key' is not assigned and could be declared final 3 742
MethodArgumentCouldBeFinal Parameter 'pooledGroup' is not assigned and could be declared final 3 742
LocalVariableCouldBeFinal Local variable 'group' could be declared final 3 744
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 745
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 745
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 746
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 746
LawOfDemeter Potential violation of Law of Demeter (static property access) 3 747
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 747
LocalVariableCouldBeFinal Local variable 'i' could be declared final 3 747
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 747
LawOfDemeter Potential violation of Law of Demeter (static property access) 3 747
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 750
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 752

org/apache/fulcrum/intake/model/IntegerField.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 43
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 43
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 55
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 57
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 76
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 78
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 103
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 3 131
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '132'-'136'). 5 132–136
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '136'-'136'). 5 136
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 137

org/apache/fulcrum/intake/model/LongField.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 43
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 43
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 55
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 57
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 76
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 78
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 103
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 3 130
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '131'-'135'). 5 131–135
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '135'-'135'). 5 135
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 136

org/apache/fulcrum/intake/model/Rule.java

Rule Violation Priority Line
ShortClassName Avoid short class names like Rule 4 41–109
AtLeastOneConstructor Each class should declare at least one constructor 3 41–109
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 49
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 52
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 55
LocalVariableCouldBeFinal Local variable 'result' could be declared final 3 91

org/apache/fulcrum/intake/model/ShortField.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 43
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 43
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 55
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 57
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 76
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 78
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 103
LocalVariableCouldBeFinal Local variable 'inputs' could be declared final 3 131
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '132'-'136'). 5 132–136
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'values' (lines '136'-'136'). 5 136
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 137
LocalVariableCouldBeFinal Local variable 'value' could be declared final 3 145
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 153

org/apache/fulcrum/intake/model/StringField.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 47
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 47
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 70
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 84
LocalVariableCouldBeFinal Local variable 'ss' could be declared final 3 97
ShortVariable Avoid variables with short names like ss 3 97
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'sval' (lines '98'-'101'). 5 98–101
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'sval' (lines '101'-'101'). 5 101
LocalVariableCouldBeFinal Local variable 'val' could be declared final 3 107
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 3 119
MethodArgumentCouldBeFinal Parameter 'v' is not assigned and could be declared final 3 119
CyclomaticComplexity The method 'setRequired(boolean, String)' has a cyclomatic complexity of 10. 3 119–159
ShortVariable Avoid variables with short names like v 3 119
ShortVariable Avoid variables with short names like ss 3 126
LocalVariableCouldBeFinal Local variable 'ss' could be declared final 3 126
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 135–142

org/apache/fulcrum/intake/model/UploadPartField.java

Rule Violation Priority Line
StdCyclomaticComplexity The class 'UploadPartField' has a Standard Cyclomatic Complexity of 3 (Highest = 12). 3 37–228
ModifiedCyclomaticComplexity The class 'UploadPartField' has a Modified Cyclomatic Complexity of 3 (Highest = 12). 3 37–228
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 50
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 50
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 65
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 74
MethodArgumentCouldBeFinal Parameter 'prop' is not assigned and could be declared final 3 86
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 95
ShortVariable Avoid variables with short names like vp 3 120
MethodArgumentCouldBeFinal Parameter 'vp' is not assigned and could be declared final 3 120
CyclomaticComplexity The method 'validate()' has a cyclomatic complexity of 14. 3 146–210
StdCyclomaticComplexity The method 'validate' has a Standard Cyclomatic Complexity of 12. 3 146–210
ModifiedCyclomaticComplexity The method 'validate' has a Modified Cyclomatic Complexity of 12. 3 146–210
LocalVariableCouldBeFinal Local variable 'pp' could be declared final 3 148
ShortVariable Avoid variables with short names like pp 3 148
ShortVariable Avoid variables with short names like ss 3 151
LocalVariableCouldBeFinal Local variable 'ss' could be declared final 3 151
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 161–171
ShortVariable Avoid variables with short names like s 3 181
LocalVariableCouldBeFinal Local variable 's' could be declared final 3 181
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 182
ConfusingTernary Avoid if (x != y) ..; else ..; 3 187–206
ShortVariable Avoid variables with short names like pp 3 218
LocalVariableCouldBeFinal Local variable 'pp' could be declared final 3 218

org/apache/fulcrum/intake/model/XmlField.java

Rule Violation Priority Line
DataClass The class 'XmlField' is suspected to be a Data Class (WOC=27.273%, NOPA=0, NOAM=15, WMC=38) 3 53–408
TooManyFields Too many fields 3 53–408
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 62
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 65
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 68
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 71
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 74
ImmutableField Private field 'type' could be made final; it is only initialized in the declaration or constructor. 3 74
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 77
RedundantFieldInitializer Avoid using redundant field initializer for 'multiValued' 3 77
ImmutableField Private field 'multiValued' could be made final; it is only initialized in the declaration or constructor. 3 77
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 80
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 83
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 86
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 89
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 92
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 95
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 98
ImmutableField Private field 'ruleMap' could be made final; it is only initialized in the declaration or constructor. 3 98
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 100
LoggerIsNotStaticFinal The Logger variable declaration does not contain the static and final modifiers 2 102
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 102
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 109
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 110
MethodArgumentCouldBeFinal Parameter 'logger' is not assigned and could be declared final 3 117
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 222
MethodArgumentCouldBeFinal Parameter 'rules' is not assigned and could be declared final 3 296
ShortVariable Avoid variables with short names like um 3 318
MethodArgumentCouldBeFinal Parameter 'parent' is not assigned and could be declared final 3 318
MethodArgumentCouldBeFinal Parameter 'um' is not assigned and could be declared final 3 318
LocalVariableCouldBeFinal Local variable 'rule' could be declared final 3 324
LocalVariableCouldBeFinal Local variable 'result' could be declared final 3 348
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 3 379
LocalVariableCouldBeFinal Local variable 'rule' could be declared final 3 386
MethodArgumentCouldBeFinal Parameter 'stream' is not assigned and could be declared final 3 397
MethodArgumentCouldBeFinal Parameter 'stream' is not assigned and could be declared final 3 403

org/apache/fulcrum/intake/validator/BigDecimalValidator.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 3 63
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 63
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 3 63
LocalVariableCouldBeFinal Local variable 'number' could be declared final 3 65
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 66
AvoidDecimalLiteralsInBigDecimalConstructor Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal 3 66

org/apache/fulcrum/intake/validator/BooleanValidator.java

Rule Violation Priority Line
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 69–72
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 3 83
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 96
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 3 108
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '111'-'127'). 5 111–127
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '111'-'120'). 5 111–120

org/apache/fulcrum/intake/validator/DateRangeValidator.java

Rule Violation Priority Line
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the fieldReferences field if you want a default access modifier 3 70
DefaultPackage Use explicit scoping instead of the default package private level 3 70
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 70
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the compareCallback field if you want a default access modifier 3 73
DefaultPackage Use explicit scoping instead of the default package private level 3 73
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 73
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 78–81
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 3 90
MethodArgumentCouldBeFinal Parameter 'refValue' is not assigned and could be declared final 3 107
MethodArgumentCouldBeFinal Parameter 'thisValue' is not assigned and could be declared final 3 107
MethodArgumentCouldBeFinal Parameter 'compare' is not assigned and could be declared final 3 107
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'126'). 5 109–126
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'122'). 5 109–122
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'114'). 5 109–114
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'114'). 5 109–114
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'118'). 5 109–118
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'126'). 5 109–126
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'118'). 5 109–118
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '109'-'122'). 5 109–122
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 134
LocalVariableCouldBeFinal Local variable 'entry' could be declared final 3 136
LocalVariableCouldBeFinal Local variable 'key' could be declared final 3 138
LocalVariableCouldBeFinal Local variable 'constraint' could be declared final 3 139
LocalVariableCouldBeFinal Local variable 'compare' could be declared final 3 141
LocalVariableCouldBeFinal Local variable 'fieldref' could be declared final 3 146
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 146
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 148
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 149
MethodArgumentCouldBeFinal Parameter 'testField' is not assigned and could be declared final 3 170
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'thisGroup' (lines '175'-'192'). 5 175–192
LocalVariableCouldBeFinal Local variable 'thisGroup' could be declared final 3 175
LocalVariableCouldBeFinal Local variable 'stringValues' could be declared final 3 179
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 181–184
LocalVariableCouldBeFinal Local variable 'testValue' could be declared final 3 188
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'testDate' (lines '209'-'213'). 5 209–213
UnusedAssignment The initializer for variable 'testDate' is never used (overwritten on line 213) 3 209
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'testDate' (lines '213'-'234'). 5 213–234
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 220

org/apache/fulcrum/intake/validator/DateStringValidator.java

Rule Violation Priority Line
LongVariable Avoid excessively long variable names like DEFAULT_DATE_MESSAGE 3 59
RedundantFieldInitializer Avoid using redundant field initializer for 'dateFormats' 3 63
UnusedAssignment The field initializer for 'dateFormats' is never used (overwritten on line 77) 3 63
RedundantFieldInitializer Avoid using redundant field initializer for 'dateFormatMessage' 3 66
RedundantFieldInitializer Avoid using redundant field initializer for 'flexible' 3 69
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 77
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 3 87
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'i' (lines '100'-'100'). 5 100
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'i' (lines '100'-'124'). 5 100–124
UnnecessaryWrapperObjectCreation Unnecessary wrapper object creation 3 122
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 122
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 3 135
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 149
CyclomaticComplexity The method 'parse(String)' has a cyclomatic complexity of 11. 3 166–221
MethodArgumentCouldBeFinal Parameter 's' is not assigned and could be declared final 3 166
ShortVariable Avoid variables with short names like s 3 166
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'date' (lines '169'-'221'). 5 169–221
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 169
LocalVariableCouldBeFinal Local variable 'sdf' could be declared final 3 176
EmptyCatchBlock Avoid empty catch blocks 3 187–190
EmptyCatchBlock Avoid empty catch blocks 3 201–204
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 210
MethodArgumentCouldBeFinal Parameter 'date' is not assigned and could be declared final 3 230
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 's' (lines '232'-'237'). 5 232–237
ShortVariable Avoid variables with short names like s 3 232
LocalVariableCouldBeFinal Local variable 'sdf' could be declared final 3 236
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 3 236
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 3 265
MethodArgumentCouldBeFinal Parameter 'formats' is not assigned and could be declared final 3 288
MethodArgumentCouldBeFinal Parameter 'flexible' is not assigned and could be declared final 3 308

org/apache/fulcrum/intake/validator/DefaultValidator.java

Rule Violation Priority Line
AbstractNaming Abstract classes should be named AbstractXXX 3 50–383
RedundantFieldInitializer Avoid using redundant field initializer for 'required' 3 54
RedundantFieldInitializer Avoid using redundant field initializer for 'requiredMessage' 3 57
RedundantFieldInitializer Avoid using redundant field initializer for 'minLength' 3 60
RedundantFieldInitializer Avoid using redundant field initializer for 'minLengthMessage' 3 63
RedundantFieldInitializer Avoid using redundant field initializer for 'maxLength' 3 66
RedundantFieldInitializer Avoid using redundant field initializer for 'maxLengthMessage' 3 69
RedundantFieldInitializer Avoid using redundant field initializer for 'errorMessage' 3 72
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 72
LoggerIsNotStaticFinal The Logger variable declaration does not contain the static and final modifiers 2 75
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 75
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 80–83
MethodArgumentCouldBeFinal Parameter 'logger' is not assigned and could be declared final 3 89
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 91
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 3 103
LocalVariableCouldBeFinal Local variable 'param' could be declared final 3 109
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 110
UnnecessaryWrapperObjectCreation Unnecessary wrapper object creation 3 110
LocalVariableCouldBeFinal Local variable 'param' could be declared final 3 117
LocalVariableCouldBeFinal Local variable 'param' could be declared final 3 125
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 139
UnusedAssignment The initializer for variable 'valid' is never used (overwritten on lines 145 and 149) 3 141
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'valid' (lines '141'-'145'). 5 141–145
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'valid' (lines '145'-'149'). 5 145–149
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 163
LocalVariableCouldBeFinal Local variable 'stringValues' could be declared final 3 168
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 170–173
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 3 192
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'valid' (lines '194'-'198'). 5 194–198
UnusedAssignment The initializer for variable 'valid' is never used (overwritten on lines 198 and 202) 3 194
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'valid' (lines '198'-'202'). 5 198–202
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 3 216
CyclomaticComplexity The method 'assertValidity(String)' has a cyclomatic complexity of 12. 3 216–239
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'retValue' (lines '250'-'254'). 5 250–254
MethodArgumentCouldBeFinal Parameter 'required' is not assigned and could be declared final 3 279
MethodArgumentCouldBeFinal Parameter 'requiredMessage' is not assigned and could be declared final 3 299
MethodArgumentCouldBeFinal Parameter 'minLength' is not assigned and could be declared final 3 319
MethodArgumentCouldBeFinal Parameter 'minLengthMessage' is not assigned and could be declared final 3 339
MethodArgumentCouldBeFinal Parameter 'maxLength' is not assigned and could be declared final 3 359
MethodArgumentCouldBeFinal Parameter 'maxLengthMessage' is not assigned and could be declared final 3 379

org/apache/fulcrum/intake/validator/DoubleValidator.java

Rule Violation Priority Line
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 61
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 3 61
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 3 61
LocalVariableCouldBeFinal Local variable 'number' could be declared final 3 63
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 64

org/apache/fulcrum/intake/validator/FieldReference.java

Rule Violation Priority Line
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Comparison': nested enums are implicitly static 3 64–106
UnnecessaryModifier Unnecessary modifier 'private' on constructor 'Comparison(String)': enum constructors are implicitly private 3 80–83
MethodArgumentCouldBeFinal Parameter 'text' is not assigned and could be declared final 3 80
MethodArgumentCouldBeFinal Parameter 'string' is not assigned and could be declared final 3 91
LocalVariableCouldBeFinal Local variable 'c' could be declared final 3 95
OnlyOneReturn A method should have only one exit point, and that should be the last statement in the method 3 99
RedundantFieldInitializer Avoid using redundant field initializer for 'compare' 3 109
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 109
RedundantFieldInitializer Avoid using redundant field initializer for 'fieldName' 3 112
RedundantFieldInitializer Avoid using redundant field initializer for 'message' 3 115
MethodArgumentCouldBeFinal Parameter 'compare' is not assigned and could be declared final 3 136
MethodArgumentCouldBeFinal Parameter 'fieldName' is not assigned and could be declared final 3 152
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 3 168
MethodArgumentCouldBeFinal Parameter 'key' is not assigned and could be declared final 3 179
MethodArgumentCouldBeFinal Parameter 'compareCallback' is not assigned and could be declared final 3 196
MethodArgumentCouldBeFinal Parameter 'fieldReferences' is not assigned and could be declared final 3 196
MethodArgumentCouldBeFinal Parameter 'group' is not assigned and could be declared final 3 197
MethodArgumentCouldBeFinal Parameter 'value' is not assigned and could be declared final 3 197
LocalVariableCouldBeFinal Local variable 'ref' could be declared final 3 200
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'comp_true' (lines '202'-'239'). 5 202–239
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'comp_true' (lines '202'-'223'). 5 202–223
LocalVariableNamingConventions The local variable name 'comp_true' doesn't match '[a-z][a-zA-Z0-9]*' 1 202
VariableNamingConventions Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'comp_true' is not final. 1 202
LocalVariableCouldBeFinal Local variable 'refField' could be declared final 3 207
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 209
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 216
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 218
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 221
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'comp_true' (lines '223'-'239'). 5 223–239
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 225
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 231
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 3 234

org/apache/fulcrum/intake/validator/FileValidator.java

Rule Violation Priority Line
FieldNamingConventions The constant name 'charsetPattern' doesn't match '[A-Z][A-Z_0-9]*' 1 45
VariableNamingConventions Variables that are final and static should be all capitals, 'charsetPattern' is not all capitals. 1 45
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 50–53
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 3 63
LocalVariableCouldBeFinal Local variable 'contentType' could be declared final 3 67
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'charset' (lines '68'-'75'). 5 68–75
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 68
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'charset' (lines '68'-'107'). 5 68–107
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 70
LocalVariableCouldBeFinal Local variable 'matcher' could be declared final 3 72
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 73
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'charset' (lines '75'-'107'). 5 75–107
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 75
LocalVariableCouldBeFinal Local variable 'byteSize' could be declared final 3 81
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 90
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'content' (lines '93'-'98'). 5 93–98
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'content' (lines '93'-'107'). 5 93–107
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'content' (lines '98'-'107'). 5 98–107
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 103

org/apache/fulcrum/intake/validator/FloatValidator.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 3 61
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 3 61
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 61
LocalVariableCouldBeFinal Local variable 'number' could be declared final 3 63
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 64

org/apache/fulcrum/intake/validator/IntegerRangeValidator.java

Rule Violation Priority Line
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the fieldReferences field if you want a default access modifier 3 69
DefaultPackage Use explicit scoping instead of the default package private level 3 69
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 69
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the compareCallback field if you want a default access modifier 3 72
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 72
DefaultPackage Use explicit scoping instead of the default package private level 3 72
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 77–80
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 3 89
MethodArgumentCouldBeFinal Parameter 'refValue' is not assigned and could be declared final 3 106
MethodArgumentCouldBeFinal Parameter 'thisValue' is not assigned and could be declared final 3 106
MethodArgumentCouldBeFinal Parameter 'compare' is not assigned and could be declared final 3 106
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'113'). 5 108–113
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'121'). 5 108–121
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'117'). 5 108–117
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'113'). 5 108–113
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'117'). 5 108–117
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'121'). 5 108–121
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'125'). 5 108–125
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'result' (lines '108'-'125'). 5 108–125
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 133
LocalVariableCouldBeFinal Local variable 'entry' could be declared final 3 135
LocalVariableCouldBeFinal Local variable 'key' could be declared final 3 137
LocalVariableCouldBeFinal Local variable 'constraint' could be declared final 3 138
LocalVariableCouldBeFinal Local variable 'compare' could be declared final 3 140
AvoidInstantiatingObjectsInLoops Avoid instantiating new objects inside loops 3 145
LocalVariableCouldBeFinal Local variable 'fieldref' could be declared final 3 145
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 147
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 148
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'thisGroup' (lines '174'-'192'). 5 174–192
LocalVariableCouldBeFinal Local variable 'thisGroup' could be declared final 3 174
LocalVariableCouldBeFinal Local variable 'locale' could be declared final 3 175
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'locale' (lines '175'-'192'). 5 175–192
LocalVariableCouldBeFinal Local variable 'stringValues' could be declared final 3 179
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 181–184
LocalVariableCouldBeFinal Local variable 'testValue' could be declared final 3 188
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'testInt' (lines '214'-'233'). 5 214–233
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 219

org/apache/fulcrum/intake/validator/IntegerValidator.java

Rule Violation Priority Line
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 60
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 3 60
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 3 60

org/apache/fulcrum/intake/validator/InvalidMaskException.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 3 44
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 3 55
MethodArgumentCouldBeFinal Parameter 'cause' is not assigned and could be declared final 3 55

org/apache/fulcrum/intake/validator/LongValidator.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 3 60
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 3 60
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 60

org/apache/fulcrum/intake/validator/NumberValidator.java

Rule Violation Priority Line
DataClass The class 'NumberValidator' is suspected to be a Data Class (WOC=21.429%, NOPA=0, NOAM=10, WMC=38) 3 52–341
AbstractNaming Abstract classes should be named AbstractXXX 3 52–341
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the minValueMessage field if you want a default access modifier 3 56
DefaultPackage Use explicit scoping instead of the default package private level 3 56
RedundantFieldInitializer Avoid using redundant field initializer for 'minValueMessage' 3 56
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the maxValueMessage field if you want a default access modifier 3 59
DefaultPackage Use explicit scoping instead of the default package private level 3 59
RedundantFieldInitializer Avoid using redundant field initializer for 'maxValueMessage' 3 59
RedundantFieldInitializer Avoid using redundant field initializer for 'invalidNumberMessage' 3 62
LongVariable Avoid excessively long variable names like invalidNumberMessage 3 62
DefaultPackage Use explicit scoping instead of the default package private level 3 62
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the invalidNumberMessage field if you want a default access modifier 3 62
RedundantFieldInitializer Avoid using redundant field initializer for 'minValue' 3 64
RedundantFieldInitializer Avoid using redundant field initializer for 'maxValue' 3 65
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 70–73
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 3 84
LocalVariableCouldBeFinal Local variable 'param' could be declared final 3 99
LocalVariableCouldBeFinal Local variable 'param' could be declared final 3 114
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 137
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 3 149
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 3 149
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 149
LocalVariableCouldBeFinal Local variable 'nf' could be declared final 3 151
ShortVariable Avoid variables with short names like nf 3 151
LocalVariableCouldBeFinal Local variable 'pos' could be declared final 3 155
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 156
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'number' (lines '156'-'169'). 5 156–169
LocalVariableCouldBeFinal Local variable 'number' could be declared final 3 156
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 167
MethodArgumentCouldBeFinal Parameter 'field' is not assigned and could be declared final 3 180
LocalVariableCouldBeFinal Local variable 'locale' could be declared final 3 182
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'locale' (lines '182'-'197'). 5 182–197
LocalVariableCouldBeFinal Local variable 'stringValues' could be declared final 3 186
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 188–191
CyclomaticComplexity The method 'assertValidity(String, Locale)' has a cyclomatic complexity of 11. 3 208–236
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 3 208
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 3 208
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'number' (lines '214'-'217'). 5 214–217
UnusedAssignment The initializer for variable 'number' is never used (overwritten on line 217) 3 214
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'number' (lines '217'-'236'). 5 217–236
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 222
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 225
LawOfDemeter Potential violation of Law of Demeter (object not created locally) 3 230
MethodArgumentCouldBeFinal Parameter 'minValueMessage' is not assigned and could be declared final 3 257
MethodArgumentCouldBeFinal Parameter 'maxValueMessage' is not assigned and could be declared final 3 277
MethodArgumentCouldBeFinal Parameter 'invalidNumberMessage' is not assigned and could be declared final 3 297
LongVariable Avoid excessively long variable names like invalidNumberMessage 3 297
MethodArgumentCouldBeFinal Parameter 'minValue' is not assigned and could be declared final 3 317
MethodArgumentCouldBeFinal Parameter 'maxValue' is not assigned and could be declared final 3 337

org/apache/fulcrum/intake/validator/ShortValidator.java

Rule Violation Priority Line
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 60
MethodArgumentCouldBeFinal Parameter 'stringValue' is not assigned and could be declared final 3 60
MethodArgumentCouldBeFinal Parameter 'locale' is not assigned and could be declared final 3 60

org/apache/fulcrum/intake/validator/StringValidator.java

Rule Violation Priority Line
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 53
RedundantFieldInitializer Avoid using redundant field initializer for 'maskString' 3 53
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 56
RedundantFieldInitializer Avoid using redundant field initializer for 'maskPattern' 3 56
RedundantFieldInitializer Avoid using redundant field initializer for 'maskMessage' 3 59
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 64–67
MethodArgumentCouldBeFinal Parameter 'paramMap' is not assigned and could be declared final 3 78
LocalVariableCouldBeFinal Local variable 'constraint' could be declared final 3 83
LocalVariableCouldBeFinal Local variable 'param' could be declared final 3 86
MethodArgumentCouldBeFinal Parameter 'testValue' is not assigned and could be declared final 3 102
LocalVariableCouldBeFinal Local variable 'patternMatch' could be declared final 3 110
LawOfDemeter Potential violation of Law of Demeter (method chain calls) 3 110
GuardLogStatement Logger calls should be surrounded by log level guards. 2 112–113
MethodArgumentCouldBeFinal Parameter 'mask' is not assigned and could be declared final 3 143
LocalVariableCouldBeFinal Local variable 'maskOptions' could be declared final 3 149
GuardLogStatement Logger calls should be surrounded by log level guards. 2 153
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 3 177

org/apache/fulcrum/intake/validator/ValidationException.java

Rule Violation Priority Line
MethodArgumentCouldBeFinal Parameter 'message' is not assigned and could be declared final 3 45

org/apache/fulcrum/intake/validator/Validator.java

Rule Violation Priority Line
LongVariable Avoid excessively long variable names like FLEXIBLE_RULE_NAME 3 34
LongVariable Avoid excessively long variable names like INVALID_NUMBER_RULE_NAME 3 40
LongVariable Avoid excessively long variable names like MAX_LENGTH_RULE_NAME 3 46
LongVariable Avoid excessively long variable names like MAX_VALUE_RULE_NAME 3 49
LongVariable Avoid excessively long variable names like MIN_LENGTH_RULE_NAME 3 52
LongVariable Avoid excessively long variable names like MIN_VALUE_RULE_NAME 3 55
LongVariable Avoid excessively long variable names like REQUIRED_RULE_NAME 3 58