PMD Results
The following document contains the results of PMD 6.29.0.
Violations By Priority
Priority 1
org/apache/fulcrum/json/jackson/Jackson2MapperService.java
| Rule | Violation | Line |
|---|---|---|
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 608 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 618 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 769 |
Priority 2
org/apache/fulcrum/json/jackson/CacheService.java
| Rule | Violation | Line |
|---|---|---|
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 39 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 49 |
org/apache/fulcrum/json/jackson/SimpleNameIntrospector.java
| Rule | Violation | Line |
|---|---|---|
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 57 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 104 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 111 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 117 |
Priority 3
org/apache/fulcrum/json/jackson/CacheService.java
| Rule | Violation | Line |
|---|---|---|
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the primary field if you want a default access modifier | 36 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 36 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 36 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the filters field if you want a default access modifier | 37 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 37 |
| MethodArgumentCouldBeFinal | Parameter 'primary' is not assigned and could be declared final | 41 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the removeFilter method if you want a default access modifier | 45–56 |
| MethodArgumentCouldBeFinal | Parameter 'excludeType' is not assigned and could be declared final | 45 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 45 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 45–56 |
| IfStmtsMustUseBraces | Avoid using if statements without curly braces | 46–47 |
| ControlStatementBraces | This statement should have braces | 47 |
| LocalVariableCouldBeFinal | Local variable 'smpfilter' could be declared final | 51–52 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 53 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 58–68 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the removeCustomIntrospectorWithExternalFilterId method if you want a default access modifier | 58–68 |
| MethodArgumentCouldBeFinal | Parameter 'externalFilterId' is not assigned and could be declared final | 59 |
| MethodArgumentCouldBeFinal | Parameter 'excludeType' is not assigned and could be declared final | 59 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the cleanSerializerCache method if you want a default access modifier | 70–82 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 70–82 |
| MethodArgumentCouldBeFinal | Parameter 'mapper' is not assigned and could be declared final | 70 |
| LocalVariableCouldBeFinal | Local variable 'cachedSerProvs' could be declared final | 72–73 |
| MethodArgumentCouldBeFinal | Parameter 'filters' is not assigned and could be declared final | 88 |
| MethodArgumentCouldBeFinal | Parameter 'logger' is not assigned and could be declared final | 93 |
org/apache/fulcrum/json/jackson/CustomModule.java
| Rule | Violation | Line |
|---|---|---|
| MethodArgumentCouldBeFinal | Parameter 'name' is not assigned and could be declared final | 38 |
| MethodArgumentCouldBeFinal | Parameter 'targetClazz' is not assigned and could be declared final | 38 |
| MethodArgumentCouldBeFinal | Parameter 'stdDeser' is not assigned and could be declared final | 39 |
| MethodArgumentCouldBeFinal | Parameter 'stdSer' is not assigned and could be declared final | 39 |
org/apache/fulcrum/json/jackson/Jackson2MapperService.java
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 1–889 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 91–889 |
| AtLeastOneConstructor | Each class should declare at least one constructor | 91–889 |
| GodClass | Possible God Class (WMC=124, ATFD=141, TCC=27.926%) | 91–889 |
| CyclomaticComplexity | The class 'Jackson2MapperService' has a total cyclomatic complexity of 124 (highest 15). | 91–889 |
| ModifiedCyclomaticComplexity | The class 'Jackson2MapperService' has a Modified Cyclomatic Complexity of 2 (Highest = 10). | 91–889 |
| StdCyclomaticComplexity | The class 'Jackson2MapperService' has a Standard Cyclomatic Complexity of 2 (Highest = 10). | 91–889 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 99 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the mapper field if you want a default access modifier | 99 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 100 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 100 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the primary field if you want a default access modifier | 100 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 101 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 101 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the secondary field if you want a default access modifier | 101 |
| LongVariable | Avoid excessively long variable names like ANNOTATIONINSPECTOR | 103 |
| LongVariable | Avoid excessively long variable names like annotationInspectors | 105 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'annotationInspectors' | 105 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 105 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 106 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'features' | 106 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 107 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'featureTypes' | 107 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 109 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the defaultTypeDefs field if you want a default access modifier | 120 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'defaultTypeDefs' | 120 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 120 |
| 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. | 121 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 122 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'escapeCharsGlobal' | 122 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'useJsonPath' | 124 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 124 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'escapeCharsClass' | 125 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 125 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 128 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 133 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 133 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 137 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 137 |
| MethodArgumentCouldBeFinal | Parameter 'filter' is not assigned and could be declared final | 137 |
| MethodArgumentCouldBeFinal | Parameter 'cleanCache' is not assigned and could be declared final | 149 |
| MethodArgumentCouldBeFinal | Parameter 'filter' is not assigned and could be declared final | 149 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 149 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 149 |
| LocalVariableCouldBeFinal | Local variable 'serResult' could be declared final | 150 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 152 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 153 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 156 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 157 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 159 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 159 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 162 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 162 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 165 |
| LocalVariableCouldBeFinal | Local variable 'res' could be declared final | 167 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 167 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 175 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 175 |
| UnusedAssignment | The initializer for variable 'reader' is never used (overwritten on lines 178 and 180) | 176 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 177–180 |
| ControlStatementBraces | This statement should have braces | 178 |
| IfElseStmtsMustUseBraces | Avoid using if...else statements without curly braces | 178 |
| ControlStatementBraces | This statement should have braces | 180 |
| IfElseStmtsMustUseBraces | Avoid using if...else statements without curly braces | 180 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 182 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 192 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 192 |
| MethodArgumentCouldBeFinal | Parameter 'module' is not assigned and could be declared final | 209 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 209 |
| MethodArgumentCouldBeFinal | Parameter 'name' is not assigned and could be declared final | 209 |
| LocalVariableCouldBeFinal | Local variable 'cmw' could be declared final | 212 |
| ShortVariable | Avoid variables with short names like cm | 213 |
| LocalVariableCouldBeFinal | Local variable 'cm' could be declared final | 213–214 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 213 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 214 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 215 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 218–219 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 227 |
| LocalVariableCouldBeFinal | Local variable 'mixin' could be declared final | 228 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 229 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 233 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 233 |
| MethodArgumentCouldBeFinal | Parameter 'elementType' is not assigned and could be declared final | 233 |
| MethodArgumentCouldBeFinal | Parameter 'targetList' is not assigned and could be declared final | 233 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 234 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 237 |
| MethodArgumentCouldBeFinal | Parameter 'valueClass' is not assigned and could be declared final | 237 |
| MethodArgumentCouldBeFinal | Parameter 'mapClass' is not assigned and could be declared final | 237 |
| MethodArgumentCouldBeFinal | Parameter 'keyClass' is not assigned and could be declared final | 237 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 238 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 239 |
| MethodArgumentCouldBeFinal | Parameter 'collectionType' is not assigned and could be declared final | 242 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 242 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 243 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 247 |
| MethodArgumentCouldBeFinal | Parameter 'collectionClass' is not assigned and could be declared final | 247 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 248 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 248 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 249 |
| MethodArgumentCouldBeFinal | Parameter 'elementType' is not assigned and could be declared final | 253 |
| MethodArgumentCouldBeFinal | Parameter 'collectionType' is not assigned and could be declared final | 253 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 253 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 256 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 271 |
| MethodArgumentCouldBeFinal | Parameter 'collectionType' is not assigned and could be declared final | 271 |
| MethodArgumentCouldBeFinal | Parameter 'cleanCache' is not assigned and could be declared final | 272 |
| LocalVariableCouldBeFinal | Local variable 'res' could be declared final | 273 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 273 |
| MethodArgumentCouldBeFinal | Parameter 'name' is not assigned and could be declared final | 289 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 289 |
| MethodArgumentCouldBeFinal | Parameter 'mixin' is not assigned and could be declared final | 289 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 290–291 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 308 |
| MethodArgumentCouldBeFinal | Parameter 'mixin' is not assigned and could be declared final | 308 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 308 |
| MethodArgumentCouldBeFinal | Parameter 'name' is not assigned and could be declared final | 308 |
| LocalVariableCouldBeFinal | Local variable 'mx' could be declared final | 309 |
| ShortVariable | Avoid variables with short names like mx | 309 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 310 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 311 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 311 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 325 |
| MethodArgumentCouldBeFinal | Parameter 'mixin' is not assigned and could be declared final | 325 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 325 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 326 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 326 |
| LinguisticNaming | Linguistics Antipattern - The setter 'setMixins' should not return any type except void linguistically | 335–343 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 335 |
| MethodArgumentCouldBeFinal | Parameter 'mixin' is not assigned and could be declared final | 335 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 341 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 346 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 346 |
| MethodArgumentCouldBeFinal | Parameter 'cache' is not assigned and could be declared final | 351 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 351 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 351 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 351–354 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 356–359 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 356 |
| MethodArgumentCouldBeFinal | Parameter 'filterClasses' is not assigned and could be declared final | 356 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 356 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 357 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 362 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 362 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 362 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 362–365 |
| MethodArgumentCouldBeFinal | Parameter 'cleanFilter' is not assigned and could be declared final | 368 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 368 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 368 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 369 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 384 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 384–396 |
| MethodArgumentCouldBeFinal | Parameter 'clean' is not assigned and could be declared final | 384 |
| MethodArgumentCouldBeFinal | Parameter 'filterClasses' is not assigned and could be declared final | 384 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 385 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 385 |
| ShortVariable | Avoid variables with short names like pf | 386 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 387–394 |
| IfElseStmtsMustUseBraces | Avoid using if...else statements without curly braces | 388 |
| ControlStatementBraces | This statement should have braces | 388 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 390 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 399 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttrs' is not assigned and could be declared final | 399 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 404 |
| MethodArgumentCouldBeFinal | Parameter 'cache' is not assigned and could be declared final | 404 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 404 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 404–406 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 409–412 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 409 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 409 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 409 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 415 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 415 |
| MethodArgumentCouldBeFinal | Parameter 'refresh' is not assigned and could be declared final | 415 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 415–418 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 416 |
| MethodArgumentCouldBeFinal | Parameter 'filterClasses' is not assigned and could be declared final | 420 |
| MethodArgumentCouldBeFinal | Parameter 'refresh' is not assigned and could be declared final | 420 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 420 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 420–433 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 421 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 421 |
| UnusedAssignment | The initializer for variable 'pf' is never used (overwritten on lines 424 and 428) | 422 |
| ShortVariable | Avoid variables with short names like pf | 422 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 425 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 427 |
| IfStmtsMustUseBraces | Avoid using if statements without curly braces | 430–431 |
| ControlStatementBraces | This statement should have braces | 431 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 436 |
| MethodArgumentCouldBeFinal | Parameter 'cleanCache' is not assigned and could be declared final | 436 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 437 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 437 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 438–439 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 439 |
| LocalVariableCouldBeFinal | Local variable 'filter' could be declared final | 440–441 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 440–441 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 441 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 442 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 444 |
| LocalVariableCouldBeFinal | Local variable 'res' could be declared final | 444 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 452 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 452 |
| MethodArgumentCouldBeFinal | Parameter 'cleanCache' is not assigned and could be declared final | 452 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 453 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 454 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 454 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 455–456 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 456 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 461–462 |
| LocalVariableCouldBeFinal | Local variable 'filter' could be declared final | 461–462 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 462 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 463 |
| LocalVariableCouldBeFinal | Local variable 'res' could be declared final | 466–467 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 466–467 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 466 |
| MethodArgumentCouldBeFinal | Parameter 'pf' is not assigned and could be declared final | 491 |
| ShortVariable | Avoid variables with short names like pf | 491 |
| MethodArgumentCouldBeFinal | Parameter 'excludeClasses' is not assigned and could be declared final | 491 |
| MethodArgumentCouldBeFinal | Parameter 'filterClasses' is not assigned and could be declared final | 491 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 491 |
| MethodArgumentCouldBeFinal | Parameter 'clean' is not assigned and could be declared final | 492 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 492 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 497 |
| LocalVariableCouldBeFinal | Local variable 'serialized' could be declared final | 498 |
| CollapsibleIfStatements | These nested if statements could be combined | 500–503 |
| LocalVariableCouldBeFinal | Local variable 'exclude' could be declared final | 501 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 501 |
| MethodArgumentCouldBeFinal | Parameter 'pf' is not assigned and could be declared final | 508 |
| ShortVariable | Avoid variables with short names like pf | 508 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 508 |
| MethodArgumentCouldBeFinal | Parameter 'excludeClasses' is not assigned and could be declared final | 509 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 516 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 516–526 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 517 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 520 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 523 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 528 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 542 |
| MethodArgumentCouldBeFinal | Parameter 'externalFilterClassIds' is not assigned and could be declared final | 543 |
| LongVariable | Avoid excessively long variable names like externalFilterClassIds | 543 |
| LocalVariableCouldBeFinal | Local variable 'filterClazz' could be declared final | 549 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 550 |
| MethodArgumentCouldBeFinal | Parameter 'module' is not assigned and could be declared final | 557 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 562 |
| MethodArgumentCouldBeFinal | Parameter 'ser' is not assigned and could be declared final | 562 |
| MethodArgumentCouldBeFinal | Parameter 'module' is not assigned and could be declared final | 562 |
| MethodArgumentCouldBeFinal | Parameter 'deSer' is not assigned and could be declared final | 566 |
| MethodArgumentCouldBeFinal | Parameter 'module' is not assigned and could be declared final | 566 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 566 |
| ShortVariable | Avoid variables with short names like df | 574 |
| MethodArgumentCouldBeFinal | Parameter 'conf' is not assigned and could be declared final | 582 |
| StdCyclomaticComplexity | The method 'configure' has a Standard Cyclomatic Complexity of 10. | 582–649 |
| ModifiedCyclomaticComplexity | The method 'configure' has a Modified Cyclomatic Complexity of 10. | 582–649 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 583 |
| LongVariable | Avoid excessively long variable names like configuredAnnotationInspectors | 586 |
| LocalVariableCouldBeFinal | Local variable 'nameVal' could be declared final | 589 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 590–622 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 592 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 594 |
| LiteralsFirstInComparisons | Position literals first in String comparisons | 594 |
| LocalVariableCouldBeFinal | Local variable 'featureValue' could be declared final | 598 |
| UnusedAssignment | The initializer for variable 'featureType' is never used (overwritten on line 602) | 599 |
| UnusedAssignment | The initializer for variable 'feature' is never used (overwritten on line 603) | 600 |
| LongVariable | Avoid excessively long variable names like configuredDateFormat | 624 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 625 |
| LongVariable | Avoid excessively long variable names like configuredKeepFilter | 627 |
| LongVariable | Avoid excessively long variable names like configuredEscapeChars | 631 |
| LongVariable | Avoid excessively long variable names like configuredEscapeCharClass | 635 |
| LongVariable | Avoid excessively long variable names like configuredDefaultType | 640 |
| LongVariable | Avoid excessively long variable names like configuredjsonPath | 645 |
| CyclomaticComplexity | The method 'initialize()' has a cyclomatic complexity of 10. | 652–708 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 661 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 662 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 667 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 670 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 674 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 674 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 675 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 676–677 |
| AvoidDuplicateLiterals | The String literal "JsonMapperService: Error instantiating " appears 5 times in this file; the first occurrence is on line 677 | 677 |
| AvoidDuplicateLiterals | The String literal " for " appears 4 times in this file; the first occurrence is on line 677 | 677 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 681 |
| MethodArgumentCouldBeFinal | Parameter 'jgen' is not assigned and could be declared final | 685 |
| MethodArgumentCouldBeFinal | Parameter 'value' is not assigned and could be declared final | 685 |
| MethodArgumentCouldBeFinal | Parameter 'provider' is not assigned and could be declared final | 685 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 692 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 692 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 693 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 693 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 701 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 702 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 703–704 |
| LocalVariableCouldBeFinal | Local variable 'defaultTyping' could be declared final | 712 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 714 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 718 |
| StdCyclomaticComplexity | The method 'initFeatures' has a Standard Cyclomatic Complexity of 10. | 718–776 |
| ModifiedCyclomaticComplexity | The method 'initFeatures' has a Modified Cyclomatic Complexity of 10. | 718–776 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 720–774 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 720–774 |
| LocalVariableCouldBeFinal | Local variable 'featureKey' could be declared final | 721 |
| LocalVariableCouldBeFinal | Local variable 'featureValue' could be declared final | 722 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 722 |
| LocalVariableCouldBeFinal | Local variable 'featureType' could be declared final | 723 |
| UnusedAssignment | The initializer for variable 'configFeature' is never used (overwritten on line 727) | 724 |
| UnusedAssignment | The initializer for variable 'feature' is never used (overwritten on lines 736, 743 and 750) | 731 |
| AvoidDuplicateLiterals | The String literal " with " appears 6 times in this file; the first occurrence is on line 740 | 740 |
| LocalVariableCouldBeFinal | Local variable 'parserFeature' could be declared final | 759 |
| LocalVariableCouldBeFinal | Local variable 'genFeature' could be declared final | 763–764 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 768 |
| ModifiedCyclomaticComplexity | The method 'initAnnotationInspectors' has a Modified Cyclomatic Complexity of 10. | 778–815 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 778 |
| CyclomaticComplexity | The method 'initAnnotationInspectors()' has a cyclomatic complexity of 15. | 778–815 |
| StdCyclomaticComplexity | The method 'initAnnotationInspectors' has a Standard Cyclomatic Complexity of 10. | 778–815 |
| LocalVariableCouldBeFinal | Local variable 'entry' could be declared final | 779 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 780 |
| LocalVariableCouldBeFinal | Local variable 'avClass' could be declared final | 781 |
| LiteralsFirstInComparisons | Position literals first in String comparisons | 782 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 782 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 782 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 784 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 784 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 785 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 786 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 788 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 790 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 790 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 791 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 792 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 798 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 798 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 800–805 |
| LocalVariableCouldBeFinal | Local variable 'pair' could be declared final | 801 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 808 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 808 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 809 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 809 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 812 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 812 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 813 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 813 |
| MethodArgumentCouldBeFinal | Parameter 'className' is not assigned and could be declared final | 825 |
| LocalVariableCouldBeFinal | Local variable 'loader' could be declared final | 826 |
| UseProperClassLoader | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | 826 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 830–834 |
| AvoidRethrowingException | A catch statement that catches an exception only to rethrow it should be avoided. | 837–840 |
| MethodArgumentCouldBeFinal | Parameter 'mapper' is not assigned and could be declared final | 847 |
| MethodArgumentCouldBeFinal | Parameter 'cacheFilters' is not assigned and could be declared final | 855 |
| IfStmtsMustUseBraces | Avoid using if statements without curly braces | 857–858 |
| ControlStatementBraces | This statement should have braces | 858 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the characterEscapes field if you want a default access modifier | 861–888 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 861–888 |
| ShortVariable | Avoid variables with short names like ch | 881 |
org/apache/fulcrum/json/jackson/MixinModule.java
| Rule | Violation | Line |
|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 29 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 30 |
| MethodArgumentCouldBeFinal | Parameter 'clazz' is not assigned and could be declared final | 32 |
| MethodArgumentCouldBeFinal | Parameter 'mixin' is not assigned and could be declared final | 32 |
| MethodArgumentCouldBeFinal | Parameter 'name' is not assigned and could be declared final | 32 |
| MethodArgumentCouldBeFinal | Parameter 'context' is not assigned and could be declared final | 39 |
org/apache/fulcrum/json/jackson/SimpleNameIntrospector.java
| Rule | Violation | Line |
|---|---|---|
| AtLeastOneConstructor | Each class should declare at least one constructor | 45–201 |
| ImmutableField | Private field 'filteredClasses' could be made final; it is only initialized in the declaration or constructor. | 51 |
| LongVariable | Avoid excessively long variable names like externalFilterExcludeClasses | 52 |
| ImmutableField | Private field 'externalFilterExcludeClasses' could be made final; it is only initialized in the declaration or constructor. | 52 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 52 |
| LongVariable | Avoid excessively long variable names like externalFilterIncludeClasses | 53 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 53 |
| ImmutableField | Private field 'externalFilterIncludeClasses' could be made final; it is only initialized in the declaration or constructor. | 53 |
| ImmutableField | Private field 'isExludeType' could be made final; it is only initialized in the declaration or constructor. | 55 |
| MethodArgumentCouldBeFinal | Parameter 'ac' is not assigned and could be declared final | 64 |
| ShortVariable | Avoid variables with short names like ac | 64 |
| EmptyCatchBlock | Avoid empty catch blocks | 79–81 |
| MethodArgumentCouldBeFinal | Parameter 'ac' is not assigned and could be declared final | 93 |
| ShortVariable | Avoid variables with short names like ac | 93 |
| ShortVariable | Avoid variables with short names like id | 94 |
| LocalVariableCouldBeFinal | Local variable 'name' could be declared final | 99 |
| LocalVariableCouldBeFinal | Local variable 'targetClazz' could be declared final | 100 |
| LocalVariableCouldBeFinal | Local variable 'filterClazz' could be declared final | 108 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 116 |
| MethodArgumentCouldBeFinal | Parameter 'filteredClass' is not assigned and could be declared final | 131 |
| MethodArgumentCouldBeFinal | Parameter 'classes' is not assigned and could be declared final | 137 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 139–149 |
| MethodArgumentCouldBeFinal | Parameter 'filteredClass' is not assigned and could be declared final | 152 |
| MethodArgumentCouldBeFinal | Parameter 'classes' is not assigned and could be declared final | 158 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 160–165 |
| MethodArgumentCouldBeFinal | Parameter 'externalFilterClass' is not assigned and could be declared final | 168 |
| LongVariable | Avoid excessively long variable names like externalFilterClass | 168 |
| MethodArgumentCouldBeFinal | Parameter 'classes' is not assigned and could be declared final | 174 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 176–181 |
| MethodArgumentCouldBeFinal | Parameter 'externalFilterClass' is not assigned and could be declared final | 184 |
| LongVariable | Avoid excessively long variable names like externalFilterClass | 184 |
| MethodArgumentCouldBeFinal | Parameter 'isExludeType' is not assigned and could be declared final | 193 |
| MethodArgumentCouldBeFinal | Parameter 'logger' is not assigned and could be declared final | 197 |
org/apache/fulcrum/json/jackson/filters/CustomModuleWrapper.java
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'CustomModuleWrapper' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=5) | 34–79 |
| UnusedAssignment | The field initializer for 'ser' is never used (overwritten on line 47) | 37 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the ser field if you want a default access modifier | 37 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 37 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'ser' | 37 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'deSer' | 38 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 38 |
| UnusedAssignment | The field initializer for 'deSer' is never used (overwritten on line 48) | 38 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the deSer field if you want a default access modifier | 38 |
| MethodArgumentCouldBeFinal | Parameter 'ser' is not assigned and could be declared final | 46 |
| MethodArgumentCouldBeFinal | Parameter 'deSer' is not assigned and could be declared final | 46 |
| MethodArgumentCouldBeFinal | Parameter 'ser' is not assigned and could be declared final | 61 |
| MethodArgumentCouldBeFinal | Parameter 'deSer' is not assigned and could be declared final | 75 |
org/apache/fulcrum/json/jackson/filters/FilterContext.java
| Rule | Violation | Line |
|---|---|---|
| AtLeastOneConstructor | Each class should declare at least one constructor | 25–37 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the filter field if you want a default access modifier | 27 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 27 |
| MethodArgumentCouldBeFinal | Parameter 'bpf' is not assigned and could be declared final | 33 |
org/apache/fulcrum/json/jackson/jsonpath/DefaultJsonPathWrapper.java
| Rule | Violation | Line |
|---|---|---|
| MethodArgumentCouldBeFinal | Parameter 'mapper' is not assigned and could be declared final | 37 |
org/apache/fulcrum/json/jackson/jsonpath/JsonPathDefault.java
| Rule | Violation | Line |
|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 45 |
| AvoidFieldNameMatchingMethodName | Field jsonProvider has the same name as a method | 45 |
| AvoidFieldNameMatchingMethodName | Field mappingProvider has the same name as a method | 46 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 46 |
| MethodArgumentCouldBeFinal | Parameter 'mapper' is not assigned and could be declared final | 49 |
Priority 4
org/apache/fulcrum/json/jackson/Jackson2MapperService.java
| Rule | Violation | Line |
|---|---|---|
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 509 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 543 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.fasterxml.jackson.core.JsonGenerator.Feature' due to existing import 'com.fasterxml.jackson.core.JsonGenerator' | 763 |
org/apache/fulcrum/json/jackson/SimpleNameIntrospector.java
| Rule | Violation | Line |
|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 190–192 |
Priority 5
org/apache/fulcrum/json/jackson/Jackson2MapperService.java
| Rule | Violation | Line |
|---|---|---|
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'serResult' (lines '150'-'172'). | 150–172 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'reader' (lines '176'-'180'). | 176–180 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'reader' (lines '176'-'178'). | 176–178 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'sourceMixins' (lines '336'-'338'). | 336–338 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'pf' (lines '386'-'396'). | 386–396 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'pf' (lines '386'-'388'). | 386–388 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'pf' (lines '422'-'428'). | 422–428 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'pf' (lines '422'-'424'). | 422–424 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'pf' (lines '424'-'433'). | 424–433 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'pf' (lines '428'-'433'). | 428–433 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'filter' (lines '493'-'495'). | 493–495 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'filter' (lines '510'-'512'). | 510–512 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'filter' (lines '510'-'523'). | 510–523 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'featureType' (lines '599'-'602'). | 599–602 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'feature' (lines '600'-'603'). | 600–603 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'djpw' (lines '698'-'700'). | 698–700 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'djpw' (lines '698'-'700'). | 698–700 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'featureValue' (lines '722'-'776'). | 722–776 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'configFeature' (lines '724'-'727'). | 724–727 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'configFeature' (lines '727'-'776'). | 727–776 |
org/apache/fulcrum/json/jackson/SimpleNameIntrospector.java
| Rule | Violation | Line |
|---|---|---|
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'name' (lines '99'-'125'). | 99–125 |
Files
org/apache/fulcrum/json/jackson/CacheService.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the primary field if you want a default access modifier | 3 | 36 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 36 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 36 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the filters field if you want a default access modifier | 3 | 37 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 37 |
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 2 | 39 |
| MethodArgumentCouldBeFinal | Parameter 'primary' is not assigned and could be declared final | 3 | 41 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the removeFilter method if you want a default access modifier | 3 | 45–56 |
| MethodArgumentCouldBeFinal | Parameter 'excludeType' is not assigned and could be declared final | 3 | 45 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 3 | 45 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 45–56 |
| IfStmtsMustUseBraces | Avoid using if statements without curly braces | 3 | 46–47 |
| ControlStatementBraces | This statement should have braces | 3 | 47 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 49 |
| LocalVariableCouldBeFinal | Local variable 'smpfilter' could be declared final | 3 | 51–52 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 53 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 58–68 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the removeCustomIntrospectorWithExternalFilterId method if you want a default access modifier | 3 | 58–68 |
| MethodArgumentCouldBeFinal | Parameter 'externalFilterId' is not assigned and could be declared final | 3 | 59 |
| MethodArgumentCouldBeFinal | Parameter 'excludeType' is not assigned and could be declared final | 3 | 59 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the cleanSerializerCache method if you want a default access modifier | 3 | 70–82 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 70–82 |
| MethodArgumentCouldBeFinal | Parameter 'mapper' is not assigned and could be declared final | 3 | 70 |
| LocalVariableCouldBeFinal | Local variable 'cachedSerProvs' could be declared final | 3 | 72–73 |
| MethodArgumentCouldBeFinal | Parameter 'filters' is not assigned and could be declared final | 3 | 88 |
| MethodArgumentCouldBeFinal | Parameter 'logger' is not assigned and could be declared final | 3 | 93 |
org/apache/fulcrum/json/jackson/CustomModule.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MethodArgumentCouldBeFinal | Parameter 'name' is not assigned and could be declared final | 3 | 38 |
| MethodArgumentCouldBeFinal | Parameter 'targetClazz' is not assigned and could be declared final | 3 | 38 |
| MethodArgumentCouldBeFinal | Parameter 'stdDeser' is not assigned and could be declared final | 3 | 39 |
| MethodArgumentCouldBeFinal | Parameter 'stdSer' is not assigned and could be declared final | 3 | 39 |
org/apache/fulcrum/json/jackson/Jackson2MapperService.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 1–889 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 91–889 |
| AtLeastOneConstructor | Each class should declare at least one constructor | 3 | 91–889 |
| GodClass | Possible God Class (WMC=124, ATFD=141, TCC=27.926%) | 3 | 91–889 |
| CyclomaticComplexity | The class 'Jackson2MapperService' has a total cyclomatic complexity of 124 (highest 15). | 3 | 91–889 |
| ModifiedCyclomaticComplexity | The class 'Jackson2MapperService' has a Modified Cyclomatic Complexity of 2 (Highest = 10). | 3 | 91–889 |
| StdCyclomaticComplexity | The class 'Jackson2MapperService' has a Standard Cyclomatic Complexity of 2 (Highest = 10). | 3 | 91–889 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 99 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the mapper field if you want a default access modifier | 3 | 99 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 100 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 100 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the primary field if you want a default access modifier | 3 | 100 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 101 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 101 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the secondary field if you want a default access modifier | 3 | 101 |
| LongVariable | Avoid excessively long variable names like ANNOTATIONINSPECTOR | 3 | 103 |
| LongVariable | Avoid excessively long variable names like annotationInspectors | 3 | 105 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'annotationInspectors' | 3 | 105 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 105 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 106 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'features' | 3 | 106 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 107 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'featureTypes' | 3 | 107 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 109 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the defaultTypeDefs field if you want a default access modifier | 3 | 120 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'defaultTypeDefs' | 3 | 120 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 120 |
| 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 | 121 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 122 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'escapeCharsGlobal' | 3 | 122 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'useJsonPath' | 3 | 124 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 124 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'escapeCharsClass' | 3 | 125 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 125 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 128 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 133 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 3 | 133 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 137 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 137 |
| MethodArgumentCouldBeFinal | Parameter 'filter' is not assigned and could be declared final | 3 | 137 |
| MethodArgumentCouldBeFinal | Parameter 'cleanCache' is not assigned and could be declared final | 3 | 149 |
| MethodArgumentCouldBeFinal | Parameter 'filter' is not assigned and could be declared final | 3 | 149 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 149 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 149 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'serResult' (lines '150'-'172'). | 5 | 150–172 |
| LocalVariableCouldBeFinal | Local variable 'serResult' could be declared final | 3 | 150 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 152 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 153 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 156 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 157 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 159 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 159 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 162 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 162 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 165 |
| LocalVariableCouldBeFinal | Local variable 'res' could be declared final | 3 | 167 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 167 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 3 | 175 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 3 | 175 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'reader' (lines '176'-'180'). | 5 | 176–180 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'reader' (lines '176'-'178'). | 5 | 176–178 |
| UnusedAssignment | The initializer for variable 'reader' is never used (overwritten on lines 178 and 180) | 3 | 176 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 177–180 |
| ControlStatementBraces | This statement should have braces | 3 | 178 |
| IfElseStmtsMustUseBraces | Avoid using if...else statements without curly braces | 3 | 178 |
| ControlStatementBraces | This statement should have braces | 3 | 180 |
| IfElseStmtsMustUseBraces | Avoid using if...else statements without curly braces | 3 | 180 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 182 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 3 | 192 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 192 |
| MethodArgumentCouldBeFinal | Parameter 'module' is not assigned and could be declared final | 3 | 209 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 3 | 209 |
| MethodArgumentCouldBeFinal | Parameter 'name' is not assigned and could be declared final | 3 | 209 |
| LocalVariableCouldBeFinal | Local variable 'cmw' could be declared final | 3 | 212 |
| ShortVariable | Avoid variables with short names like cm | 3 | 213 |
| LocalVariableCouldBeFinal | Local variable 'cm' could be declared final | 3 | 213–214 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 213 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 214 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 215 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 218–219 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 3 | 227 |
| LocalVariableCouldBeFinal | Local variable 'mixin' could be declared final | 3 | 228 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 229 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 3 | 233 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 233 |
| MethodArgumentCouldBeFinal | Parameter 'elementType' is not assigned and could be declared final | 3 | 233 |
| MethodArgumentCouldBeFinal | Parameter 'targetList' is not assigned and could be declared final | 3 | 233 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 234 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 3 | 237 |
| MethodArgumentCouldBeFinal | Parameter 'valueClass' is not assigned and could be declared final | 3 | 237 |
| MethodArgumentCouldBeFinal | Parameter 'mapClass' is not assigned and could be declared final | 3 | 237 |
| MethodArgumentCouldBeFinal | Parameter 'keyClass' is not assigned and could be declared final | 3 | 237 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 238 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 239 |
| MethodArgumentCouldBeFinal | Parameter 'collectionType' is not assigned and could be declared final | 3 | 242 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 3 | 242 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 243 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 3 | 247 |
| MethodArgumentCouldBeFinal | Parameter 'collectionClass' is not assigned and could be declared final | 3 | 247 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 3 | 248 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 248 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 249 |
| MethodArgumentCouldBeFinal | Parameter 'elementType' is not assigned and could be declared final | 3 | 253 |
| MethodArgumentCouldBeFinal | Parameter 'collectionType' is not assigned and could be declared final | 3 | 253 |
| MethodArgumentCouldBeFinal | Parameter 'json' is not assigned and could be declared final | 3 | 253 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 256 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 271 |
| MethodArgumentCouldBeFinal | Parameter 'collectionType' is not assigned and could be declared final | 3 | 271 |
| MethodArgumentCouldBeFinal | Parameter 'cleanCache' is not assigned and could be declared final | 3 | 272 |
| LocalVariableCouldBeFinal | Local variable 'res' could be declared final | 3 | 273 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 273 |
| MethodArgumentCouldBeFinal | Parameter 'name' is not assigned and could be declared final | 3 | 289 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 3 | 289 |
| MethodArgumentCouldBeFinal | Parameter 'mixin' is not assigned and could be declared final | 3 | 289 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 290–291 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 3 | 308 |
| MethodArgumentCouldBeFinal | Parameter 'mixin' is not assigned and could be declared final | 3 | 308 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 308 |
| MethodArgumentCouldBeFinal | Parameter 'name' is not assigned and could be declared final | 3 | 308 |
| LocalVariableCouldBeFinal | Local variable 'mx' could be declared final | 3 | 309 |
| ShortVariable | Avoid variables with short names like mx | 3 | 309 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 310 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 311 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 311 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 325 |
| MethodArgumentCouldBeFinal | Parameter 'mixin' is not assigned and could be declared final | 3 | 325 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 3 | 325 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 326 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 326 |
| LinguisticNaming | Linguistics Antipattern - The setter 'setMixins' should not return any type except void linguistically | 3 | 335–343 |
| MethodArgumentCouldBeFinal | Parameter 'target' is not assigned and could be declared final | 3 | 335 |
| MethodArgumentCouldBeFinal | Parameter 'mixin' is not assigned and could be declared final | 3 | 335 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'sourceMixins' (lines '336'-'338'). | 5 | 336–338 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 341 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 346 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 3 | 346 |
| MethodArgumentCouldBeFinal | Parameter 'cache' is not assigned and could be declared final | 3 | 351 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 351 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 3 | 351 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 351–354 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 356–359 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 356 |
| MethodArgumentCouldBeFinal | Parameter 'filterClasses' is not assigned and could be declared final | 3 | 356 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 3 | 356 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 357 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 3 | 362 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 3 | 362 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 362 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 362–365 |
| MethodArgumentCouldBeFinal | Parameter 'cleanFilter' is not assigned and could be declared final | 3 | 368 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 3 | 368 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 368 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 3 | 369 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 384 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 384–396 |
| MethodArgumentCouldBeFinal | Parameter 'clean' is not assigned and could be declared final | 3 | 384 |
| MethodArgumentCouldBeFinal | Parameter 'filterClasses' is not assigned and could be declared final | 3 | 384 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 3 | 385 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 385 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'pf' (lines '386'-'396'). | 5 | 386–396 |
| ShortVariable | Avoid variables with short names like pf | 3 | 386 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'pf' (lines '386'-'388'). | 5 | 386–388 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 387–394 |
| IfElseStmtsMustUseBraces | Avoid using if...else statements without curly braces | 3 | 388 |
| ControlStatementBraces | This statement should have braces | 3 | 388 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 390 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 399 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttrs' is not assigned and could be declared final | 3 | 399 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 404 |
| MethodArgumentCouldBeFinal | Parameter 'cache' is not assigned and could be declared final | 3 | 404 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 3 | 404 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 404–406 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 409–412 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 3 | 409 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 409 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 3 | 409 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 3 | 415 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 415 |
| MethodArgumentCouldBeFinal | Parameter 'refresh' is not assigned and could be declared final | 3 | 415 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 415–418 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 3 | 416 |
| MethodArgumentCouldBeFinal | Parameter 'filterClasses' is not assigned and could be declared final | 3 | 420 |
| MethodArgumentCouldBeFinal | Parameter 'refresh' is not assigned and could be declared final | 3 | 420 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 420 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 420–433 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 421 |
| MethodArgumentCouldBeFinal | Parameter 'filterAttr' is not assigned and could be declared final | 3 | 421 |
| UnusedAssignment | The initializer for variable 'pf' is never used (overwritten on lines 424 and 428) | 3 | 422 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'pf' (lines '422'-'428'). | 5 | 422–428 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'pf' (lines '422'-'424'). | 5 | 422–424 |
| ShortVariable | Avoid variables with short names like pf | 3 | 422 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'pf' (lines '424'-'433'). | 5 | 424–433 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 425 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 427 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'pf' (lines '428'-'433'). | 5 | 428–433 |
| IfStmtsMustUseBraces | Avoid using if statements without curly braces | 3 | 430–431 |
| ControlStatementBraces | This statement should have braces | 3 | 431 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 436 |
| MethodArgumentCouldBeFinal | Parameter 'cleanCache' is not assigned and could be declared final | 3 | 436 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 437 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 437 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 438–439 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 439 |
| LocalVariableCouldBeFinal | Local variable 'filter' could be declared final | 3 | 440–441 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 440–441 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 441 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 442 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 444 |
| LocalVariableCouldBeFinal | Local variable 'res' could be declared final | 3 | 444 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 3 | 452 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 452 |
| MethodArgumentCouldBeFinal | Parameter 'cleanCache' is not assigned and could be declared final | 3 | 452 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 453 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 454 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 454 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 455–456 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 456 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 461–462 |
| LocalVariableCouldBeFinal | Local variable 'filter' could be declared final | 3 | 461–462 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 462 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 463 |
| LocalVariableCouldBeFinal | Local variable 'res' could be declared final | 3 | 466–467 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 466–467 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 466 |
| MethodArgumentCouldBeFinal | Parameter 'pf' is not assigned and could be declared final | 3 | 491 |
| ShortVariable | Avoid variables with short names like pf | 3 | 491 |
| MethodArgumentCouldBeFinal | Parameter 'excludeClasses' is not assigned and could be declared final | 3 | 491 |
| MethodArgumentCouldBeFinal | Parameter 'filterClasses' is not assigned and could be declared final | 3 | 491 |
| MethodArgumentCouldBeFinal | Parameter 'src' is not assigned and could be declared final | 3 | 491 |
| MethodArgumentCouldBeFinal | Parameter 'clean' is not assigned and could be declared final | 3 | 492 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 492 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'filter' (lines '493'-'495'). | 5 | 493–495 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 497 |
| LocalVariableCouldBeFinal | Local variable 'serialized' could be declared final | 3 | 498 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 500–503 |
| LocalVariableCouldBeFinal | Local variable 'exclude' could be declared final | 3 | 501 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 501 |
| MethodArgumentCouldBeFinal | Parameter 'pf' is not assigned and could be declared final | 3 | 508 |
| ShortVariable | Avoid variables with short names like pf | 3 | 508 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 3 | 508 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 509 |
| MethodArgumentCouldBeFinal | Parameter 'excludeClasses' is not assigned and could be declared final | 3 | 509 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'filter' (lines '510'-'512'). | 5 | 510–512 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'filter' (lines '510'-'523'). | 5 | 510–523 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 516 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 516–526 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 517 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 520 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 523 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 528 |
| MethodArgumentCouldBeFinal | Parameter 'filterClass' is not assigned and could be declared final | 3 | 542 |
| MethodArgumentCouldBeFinal | Parameter 'externalFilterClassIds' is not assigned and could be declared final | 3 | 543 |
| LongVariable | Avoid excessively long variable names like externalFilterClassIds | 3 | 543 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 543 |
| LocalVariableCouldBeFinal | Local variable 'filterClazz' could be declared final | 3 | 549 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 550 |
| MethodArgumentCouldBeFinal | Parameter 'module' is not assigned and could be declared final | 3 | 557 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 3 | 562 |
| MethodArgumentCouldBeFinal | Parameter 'ser' is not assigned and could be declared final | 3 | 562 |
| MethodArgumentCouldBeFinal | Parameter 'module' is not assigned and could be declared final | 3 | 562 |
| MethodArgumentCouldBeFinal | Parameter 'deSer' is not assigned and could be declared final | 3 | 566 |
| MethodArgumentCouldBeFinal | Parameter 'module' is not assigned and could be declared final | 3 | 566 |
| MethodArgumentCouldBeFinal | Parameter 'type' is not assigned and could be declared final | 3 | 566 |
| ShortVariable | Avoid variables with short names like df | 3 | 574 |
| MethodArgumentCouldBeFinal | Parameter 'conf' is not assigned and could be declared final | 3 | 582 |
| StdCyclomaticComplexity | The method 'configure' has a Standard Cyclomatic Complexity of 10. | 3 | 582–649 |
| ModifiedCyclomaticComplexity | The method 'configure' has a Modified Cyclomatic Complexity of 10. | 3 | 582–649 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 583 |
| LongVariable | Avoid excessively long variable names like configuredAnnotationInspectors | 3 | 586 |
| LocalVariableCouldBeFinal | Local variable 'nameVal' could be declared final | 3 | 589 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 590–622 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 592 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 594 |
| LiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 594 |
| LocalVariableCouldBeFinal | Local variable 'featureValue' could be declared final | 3 | 598 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'featureType' (lines '599'-'602'). | 5 | 599–602 |
| UnusedAssignment | The initializer for variable 'featureType' is never used (overwritten on line 602) | 3 | 599 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'feature' (lines '600'-'603'). | 5 | 600–603 |
| UnusedAssignment | The initializer for variable 'feature' is never used (overwritten on line 603) | 3 | 600 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 608 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 618 |
| LongVariable | Avoid excessively long variable names like configuredDateFormat | 3 | 624 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 625 |
| LongVariable | Avoid excessively long variable names like configuredKeepFilter | 3 | 627 |
| LongVariable | Avoid excessively long variable names like configuredEscapeChars | 3 | 631 |
| LongVariable | Avoid excessively long variable names like configuredEscapeCharClass | 3 | 635 |
| LongVariable | Avoid excessively long variable names like configuredDefaultType | 3 | 640 |
| LongVariable | Avoid excessively long variable names like configuredjsonPath | 3 | 645 |
| CyclomaticComplexity | The method 'initialize()' has a cyclomatic complexity of 10. | 3 | 652–708 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 661 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 662 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 3 | 667 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 670 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 674 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 674 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 675 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 3 | 676–677 |
| AvoidDuplicateLiterals | The String literal "JsonMapperService: Error instantiating " appears 5 times in this file; the first occurrence is on line 677 | 3 | 677 |
| AvoidDuplicateLiterals | The String literal " for " appears 4 times in this file; the first occurrence is on line 677 | 3 | 677 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 681 |
| MethodArgumentCouldBeFinal | Parameter 'jgen' is not assigned and could be declared final | 3 | 685 |
| MethodArgumentCouldBeFinal | Parameter 'value' is not assigned and could be declared final | 3 | 685 |
| MethodArgumentCouldBeFinal | Parameter 'provider' is not assigned and could be declared final | 3 | 685 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 692 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 692 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 693 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 693 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'djpw' (lines '698'-'700'). | 5 | 698–700 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'djpw' (lines '698'-'700'). | 5 | 698–700 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 701 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 702 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 3 | 703–704 |
| LocalVariableCouldBeFinal | Local variable 'defaultTyping' could be declared final | 3 | 712 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 714 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 718 |
| StdCyclomaticComplexity | The method 'initFeatures' has a Standard Cyclomatic Complexity of 10. | 3 | 718–776 |
| ModifiedCyclomaticComplexity | The method 'initFeatures' has a Modified Cyclomatic Complexity of 10. | 3 | 718–776 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 720–774 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 720–774 |
| LocalVariableCouldBeFinal | Local variable 'featureKey' could be declared final | 3 | 721 |
| LocalVariableCouldBeFinal | Local variable 'featureValue' could be declared final | 3 | 722 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'featureValue' (lines '722'-'776'). | 5 | 722–776 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 722 |
| LocalVariableCouldBeFinal | Local variable 'featureType' could be declared final | 3 | 723 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'configFeature' (lines '724'-'727'). | 5 | 724–727 |
| UnusedAssignment | The initializer for variable 'configFeature' is never used (overwritten on line 727) | 3 | 724 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'configFeature' (lines '727'-'776'). | 5 | 727–776 |
| UnusedAssignment | The initializer for variable 'feature' is never used (overwritten on lines 736, 743 and 750) | 3 | 731 |
| AvoidDuplicateLiterals | The String literal " with " appears 6 times in this file; the first occurrence is on line 740 | 3 | 740 |
| LocalVariableCouldBeFinal | Local variable 'parserFeature' could be declared final | 3 | 759 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.fasterxml.jackson.core.JsonGenerator.Feature' due to existing import 'com.fasterxml.jackson.core.JsonGenerator' | 4 | 763 |
| LocalVariableCouldBeFinal | Local variable 'genFeature' could be declared final | 3 | 763–764 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 768 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 769 |
| ModifiedCyclomaticComplexity | The method 'initAnnotationInspectors' has a Modified Cyclomatic Complexity of 10. | 3 | 778–815 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 778 |
| CyclomaticComplexity | The method 'initAnnotationInspectors()' has a cyclomatic complexity of 15. | 3 | 778–815 |
| StdCyclomaticComplexity | The method 'initAnnotationInspectors' has a Standard Cyclomatic Complexity of 10. | 3 | 778–815 |
| LocalVariableCouldBeFinal | Local variable 'entry' could be declared final | 3 | 779 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 780 |
| LocalVariableCouldBeFinal | Local variable 'avClass' could be declared final | 3 | 781 |
| LiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 782 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 782 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 782 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 784 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 784 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 785 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 3 | 786 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 788 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 790 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 790 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 791 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 3 | 792 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 798 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 798 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 800–805 |
| LocalVariableCouldBeFinal | Local variable 'pair' could be declared final | 3 | 801 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 808 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 808 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 809 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 809 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 812 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 812 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 813 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 813 |
| MethodArgumentCouldBeFinal | Parameter 'className' is not assigned and could be declared final | 3 | 825 |
| LocalVariableCouldBeFinal | Local variable 'loader' could be declared final | 3 | 826 |
| UseProperClassLoader | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | 3 | 826 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 830–834 |
| AvoidRethrowingException | A catch statement that catches an exception only to rethrow it should be avoided. | 3 | 837–840 |
| MethodArgumentCouldBeFinal | Parameter 'mapper' is not assigned and could be declared final | 3 | 847 |
| MethodArgumentCouldBeFinal | Parameter 'cacheFilters' is not assigned and could be declared final | 3 | 855 |
| IfStmtsMustUseBraces | Avoid using if statements without curly braces | 3 | 857–858 |
| ControlStatementBraces | This statement should have braces | 3 | 858 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the characterEscapes field if you want a default access modifier | 3 | 861–888 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 861–888 |
| ShortVariable | Avoid variables with short names like ch | 3 | 881 |
org/apache/fulcrum/json/jackson/MixinModule.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 29 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 30 |
| MethodArgumentCouldBeFinal | Parameter 'clazz' is not assigned and could be declared final | 3 | 32 |
| MethodArgumentCouldBeFinal | Parameter 'mixin' is not assigned and could be declared final | 3 | 32 |
| MethodArgumentCouldBeFinal | Parameter 'name' is not assigned and could be declared final | 3 | 32 |
| MethodArgumentCouldBeFinal | Parameter 'context' is not assigned and could be declared final | 3 | 39 |
org/apache/fulcrum/json/jackson/SimpleNameIntrospector.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AtLeastOneConstructor | Each class should declare at least one constructor | 3 | 45–201 |
| ImmutableField | Private field 'filteredClasses' could be made final; it is only initialized in the declaration or constructor. | 3 | 51 |
| LongVariable | Avoid excessively long variable names like externalFilterExcludeClasses | 3 | 52 |
| ImmutableField | Private field 'externalFilterExcludeClasses' could be made final; it is only initialized in the declaration or constructor. | 3 | 52 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 52 |
| LongVariable | Avoid excessively long variable names like externalFilterIncludeClasses | 3 | 53 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 53 |
| ImmutableField | Private field 'externalFilterIncludeClasses' could be made final; it is only initialized in the declaration or constructor. | 3 | 53 |
| ImmutableField | Private field 'isExludeType' could be made final; it is only initialized in the declaration or constructor. | 3 | 55 |
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 2 | 57 |
| MethodArgumentCouldBeFinal | Parameter 'ac' is not assigned and could be declared final | 3 | 64 |
| ShortVariable | Avoid variables with short names like ac | 3 | 64 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 79–81 |
| MethodArgumentCouldBeFinal | Parameter 'ac' is not assigned and could be declared final | 3 | 93 |
| ShortVariable | Avoid variables with short names like ac | 3 | 93 |
| ShortVariable | Avoid variables with short names like id | 3 | 94 |
| LocalVariableCouldBeFinal | Local variable 'name' could be declared final | 3 | 99 |
| DataflowAnomalyAnalysis | Found 'DU'-anomaly for variable 'name' (lines '99'-'125'). | 5 | 99–125 |
| LocalVariableCouldBeFinal | Local variable 'targetClazz' could be declared final | 3 | 100 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 104 |
| LocalVariableCouldBeFinal | Local variable 'filterClazz' could be declared final | 3 | 108 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 111 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 116 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 117 |
| MethodArgumentCouldBeFinal | Parameter 'filteredClass' is not assigned and could be declared final | 3 | 131 |
| MethodArgumentCouldBeFinal | Parameter 'classes' is not assigned and could be declared final | 3 | 137 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 139–149 |
| MethodArgumentCouldBeFinal | Parameter 'filteredClass' is not assigned and could be declared final | 3 | 152 |
| MethodArgumentCouldBeFinal | Parameter 'classes' is not assigned and could be declared final | 3 | 158 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 160–165 |
| MethodArgumentCouldBeFinal | Parameter 'externalFilterClass' is not assigned and could be declared final | 3 | 168 |
| LongVariable | Avoid excessively long variable names like externalFilterClass | 3 | 168 |
| MethodArgumentCouldBeFinal | Parameter 'classes' is not assigned and could be declared final | 3 | 174 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 176–181 |
| MethodArgumentCouldBeFinal | Parameter 'externalFilterClass' is not assigned and could be declared final | 3 | 184 |
| LongVariable | Avoid excessively long variable names like externalFilterClass | 3 | 184 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 190–192 |
| MethodArgumentCouldBeFinal | Parameter 'isExludeType' is not assigned and could be declared final | 3 | 193 |
| MethodArgumentCouldBeFinal | Parameter 'logger' is not assigned and could be declared final | 3 | 197 |
org/apache/fulcrum/json/jackson/filters/CustomModuleWrapper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'CustomModuleWrapper' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=5) | 3 | 34–79 |
| UnusedAssignment | The field initializer for 'ser' is never used (overwritten on line 47) | 3 | 37 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the ser field if you want a default access modifier | 3 | 37 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 37 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'ser' | 3 | 37 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'deSer' | 3 | 38 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 38 |
| UnusedAssignment | The field initializer for 'deSer' is never used (overwritten on line 48) | 3 | 38 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the deSer field if you want a default access modifier | 3 | 38 |
| MethodArgumentCouldBeFinal | Parameter 'ser' is not assigned and could be declared final | 3 | 46 |
| MethodArgumentCouldBeFinal | Parameter 'deSer' is not assigned and could be declared final | 3 | 46 |
| MethodArgumentCouldBeFinal | Parameter 'ser' is not assigned and could be declared final | 3 | 61 |
| MethodArgumentCouldBeFinal | Parameter 'deSer' is not assigned and could be declared final | 3 | 75 |
org/apache/fulcrum/json/jackson/filters/FilterContext.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AtLeastOneConstructor | Each class should declare at least one constructor | 3 | 25–37 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the filter field if you want a default access modifier | 3 | 27 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 27 |
| MethodArgumentCouldBeFinal | Parameter 'bpf' is not assigned and could be declared final | 3 | 33 |
org/apache/fulcrum/json/jackson/jsonpath/DefaultJsonPathWrapper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MethodArgumentCouldBeFinal | Parameter 'mapper' is not assigned and could be declared final | 3 | 37 |
org/apache/fulcrum/json/jackson/jsonpath/JsonPathDefault.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 45 |
| AvoidFieldNameMatchingMethodName | Field jsonProvider has the same name as a method | 3 | 45 |
| AvoidFieldNameMatchingMethodName | Field mappingProvider has the same name as a method | 3 | 46 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 46 |
| MethodArgumentCouldBeFinal | Parameter 'mapper' is not assigned and could be declared final | 3 | 49 |