PMD Results
The following document contains the results of PMD 6.29.0.
Violations By Priority
Priority 3
org/apache/fulcrum/cache/CachedObject.java
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'CachedObject' is suspected to be a Data Class (WOC=0.000%, NOPA=2, NOAM=6, WMC=12) | 34–185 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'contents' | 54 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 54 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 60 |
org/apache/fulcrum/cache/RefreshableCachedObject.java
| Rule | Violation | Line |
|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 53 |
| MethodArgumentCouldBeFinal | Parameter 'lastAccess' is not assigned and could be declared final | 158 |
org/apache/fulcrum/cache/impl/DefaultGlobalCacheService.java
| Rule | Violation | Line |
|---|---|---|
| LongVariable | Avoid excessively long variable names like DEFAULT_INITIAL_CACHE_SIZE | 65 |
| LongVariable | Avoid excessively long variable names like INITIAL_CACHE_SIZE | 70 |
| LongVariable | Avoid excessively long variable names like CACHE_CHECK_FREQUENCY | 75 |
| LongVariable | Avoid excessively long variable names like DEFAULT_CACHE_CHECK_FREQUENCY | 81 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'cache' | 84 |
| LongVariable | Avoid excessively long variable names like cacheCheckFrequency | 89 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 97 |
| LongVariable | Avoid excessively long variable names like houseKeepingThread | 97 |
| SingularField | Perhaps 'houseKeepingThread' could be replaced by a local variable. | 97 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 102–105 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 102–105 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 130 |
| CyclomaticComplexity | The method 'getObject(String)' has a cyclomatic complexity of 11. | 130–168 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 139 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObj | 143 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObj' could be declared final | 143 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 144 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 149 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 150 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObj | 164 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObj' could be declared final | 164 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 165 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 197 |
| LocalVariableCouldBeFinal | Local variable 'keys' could be declared final | 213 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 214 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 239 |
| UnusedAssignment | The initializer for variable 'cachedObject' is never used (overwritten on lines 241 and 245) | 241 |
| LocalVariableCouldBeFinal | Local variable 'refreshThese' could be declared final | 292 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 295 |
| LocalVariableCouldBeFinal | Local variable 'refreshableObject' could be declared final | 300 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 301 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 305 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 311 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 317 |
| LocalVariableCouldBeFinal | Local variable 'cachedObject' could be declared final | 319 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObject' could be declared final | 320 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObject | 320 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 321 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 353 |
| MethodArgumentCouldBeFinal | Parameter 'conf' is not assigned and could be declared final | 370 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 387 |
org/apache/fulcrum/cache/impl/EHCacheService.java
| Rule | Violation | Line |
|---|---|---|
| LongVariable | Avoid excessively long variable names like DEFAULT_CACHE_CHECK_FREQUENCY | 56 |
| LongVariable | Avoid excessively long variable names like cacheCheckFrequency | 61 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 61 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 66 |
| LongVariable | Avoid excessively long variable names like DEFAULT_CACHE_NAME | 71 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 76 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 79 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 79 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 82 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 85 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 88 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 90–93 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 90–93 |
| MethodArgumentCouldBeFinal | Parameter 'config' is not assigned and could be declared final | 100 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 102–103 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 104 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 105 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 128 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 138 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 151 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 152 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 153 |
| MethodArgumentCouldBeFinal | Parameter 'object' is not assigned and could be declared final | 160 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 160 |
| LocalVariableCouldBeFinal | Local variable 'cacheElement' could be declared final | 162 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 192 |
| LocalVariableCouldBeFinal | Local variable 'values' could be declared final | 192 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 194 |
| LocalVariableCouldBeFinal | Local variable 'cachedElement' could be declared final | 196 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 200 |
| LocalVariableCouldBeFinal | Local variable 'keysWithExpiryCheck' could be declared final | 224 |
| LongVariable | Avoid excessively long variable names like keysWithExpiryCheck | 224 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 234 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 241 |
| CyclomaticComplexity | The method 'getObject(String)' has a cyclomatic complexity of 11. | 241–291 |
| LocalVariableCouldBeFinal | Local variable 'cachedElement' could be declared final | 243 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 252 |
| LocalVariableCouldBeFinal | Local variable 'cachedObject' could be declared final | 252 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 254 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObject' could be declared final | 258 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObject | 258 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 259 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 267 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 268 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObject | 286 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObject' could be declared final | 286 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 287 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 297 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 325 |
| LocalVariableCouldBeFinal | Local variable 'cachedElement' could be declared final | 327 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 335 |
| LocalVariableCouldBeFinal | Local variable 'object' could be declared final | 335 |
| LocalVariableCouldBeFinal | Local variable 'refreshableObject' could be declared final | 339 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 340 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 344 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 346 |
org/apache/fulcrum/cache/impl/JCSCacheService.java
| Rule | Violation | Line |
|---|---|---|
| LongVariable | Avoid excessively long variable names like DEFAULT_CACHE_CHECK_FREQUENCY | 61 |
| LongVariable | Avoid excessively long variable names like cacheCheckFrequency | 66 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 66 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 71 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 76 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 81 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 89 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 89 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 92 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 94–97 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 94–97 |
| MethodArgumentCouldBeFinal | Parameter 'config' is not assigned and could be declared final | 105 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 107–108 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 109 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 110–111 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 125 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 135 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 148 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 149 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 151 |
| CyclomaticComplexity | The method 'getObject(String)' has a cyclomatic complexity of 11. | 158–206 |
| LocalVariableCouldBeFinal | Local variable 'cachedObject' could be declared final | 161 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 169 |
| LocalVariableCouldBeFinal | Local variable 'refreshableObject' could be declared final | 173 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 174 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 182 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 183 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObject' could be declared final | 201 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObject | 201 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 202 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 213 |
| MethodArgumentCouldBeFinal | Parameter 'cachedObject' is not assigned and could be declared final | 213 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 219–224 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 224 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 224 |
| LocalVariableCouldBeFinal | Local variable 'attrib' could be declared final | 227 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 231 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 235 |
| LocalVariableCouldBeFinal | Local variable 'tmp0' could be declared final | 237 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 239 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 242 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 243 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 249 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 257 |
| LocalVariableCouldBeFinal | Local variable 'keys' could be declared final | 268 |
| LocalVariableCouldBeFinal | Local variable 'values' could be declared final | 279 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 280 |
| LocalVariableCouldBeFinal | Local variable 'cachedObject' could be declared final | 282 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 315 |
| LocalVariableCouldBeFinal | Local variable 'cachedObject' could be declared final | 317 |
| LocalVariableCouldBeFinal | Local variable 'refreshableObject' could be declared final | 326 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 327 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 331 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 333 |
| LocalVariableCouldBeFinal | Local variable 'baos' could be declared final | 348 |
| LocalVariableCouldBeFinal | Local variable 'out' could be declared final | 349 |
| LocalVariableCouldBeFinal | Local variable 'keys' could be declared final | 350 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 352 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 364 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 364 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 375 |
Priority 5
org/apache/fulcrum/cache/CachedObject.java
| Rule | Violation | Line |
|---|---|---|
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'currentState' (lines '177'-'181'). | 177–181 |
org/apache/fulcrum/cache/RefreshableCachedObject.java
| Rule | Violation | Line |
|---|---|---|
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'untouched' (lines '130'-'134'). | 130–134 |
org/apache/fulcrum/cache/impl/DefaultGlobalCacheService.java
| Rule | Violation | Line |
|---|---|---|
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'cachedObject' (lines '241'-'245'). | 241–245 |
org/apache/fulcrum/cache/impl/JCSCacheService.java
| Rule | Violation | Line |
|---|---|---|
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'count' (lines '373'-'379'). | 373–379 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'count' (lines '379'-'379'). | 379 |
Files
org/apache/fulcrum/cache/CachedObject.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'CachedObject' is suspected to be a Data Class (WOC=0.000%, NOPA=2, NOAM=6, WMC=12) | 3 | 34–185 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'contents' | 3 | 54 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 54 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 60 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'currentState' (lines '177'-'181'). | 5 | 177–181 |
org/apache/fulcrum/cache/RefreshableCachedObject.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 53 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'untouched' (lines '130'-'134'). | 5 | 130–134 |
| MethodArgumentCouldBeFinal | Parameter 'lastAccess' is not assigned and could be declared final | 3 | 158 |
org/apache/fulcrum/cache/impl/DefaultGlobalCacheService.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| LongVariable | Avoid excessively long variable names like DEFAULT_INITIAL_CACHE_SIZE | 3 | 65 |
| LongVariable | Avoid excessively long variable names like INITIAL_CACHE_SIZE | 3 | 70 |
| LongVariable | Avoid excessively long variable names like CACHE_CHECK_FREQUENCY | 3 | 75 |
| LongVariable | Avoid excessively long variable names like DEFAULT_CACHE_CHECK_FREQUENCY | 3 | 81 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'cache' | 3 | 84 |
| LongVariable | Avoid excessively long variable names like cacheCheckFrequency | 3 | 89 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 3 | 97 |
| LongVariable | Avoid excessively long variable names like houseKeepingThread | 3 | 97 |
| SingularField | Perhaps 'houseKeepingThread' could be replaced by a local variable. | 3 | 97 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 102–105 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 102–105 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 3 | 130 |
| CyclomaticComplexity | The method 'getObject(String)' has a cyclomatic complexity of 11. | 3 | 130–168 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 139 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObj | 3 | 143 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObj' could be declared final | 3 | 143 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 144 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 149 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 150 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObj | 3 | 164 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObj' could be declared final | 3 | 164 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 165 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 3 | 197 |
| LocalVariableCouldBeFinal | Local variable 'keys' could be declared final | 3 | 213 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 214 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 239 |
| UnusedAssignment | The initializer for variable 'cachedObject' is never used (overwritten on lines 241 and 245) | 3 | 241 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'cachedObject' (lines '241'-'245'). | 5 | 241–245 |
| LocalVariableCouldBeFinal | Local variable 'refreshThese' could be declared final | 3 | 292 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 295 |
| LocalVariableCouldBeFinal | Local variable 'refreshableObject' could be declared final | 3 | 300 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 301 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 305 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 311 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 317 |
| LocalVariableCouldBeFinal | Local variable 'cachedObject' could be declared final | 3 | 319 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObject' could be declared final | 3 | 320 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObject | 3 | 320 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 321 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 353 |
| MethodArgumentCouldBeFinal | Parameter 'conf' is not assigned and could be declared final | 3 | 370 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 3 | 387 |
org/apache/fulcrum/cache/impl/EHCacheService.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| LongVariable | Avoid excessively long variable names like DEFAULT_CACHE_CHECK_FREQUENCY | 3 | 56 |
| LongVariable | Avoid excessively long variable names like cacheCheckFrequency | 3 | 61 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 61 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 66 |
| LongVariable | Avoid excessively long variable names like DEFAULT_CACHE_NAME | 3 | 71 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 76 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 3 | 79 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 79 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 82 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 85 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 88 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 90–93 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 90–93 |
| MethodArgumentCouldBeFinal | Parameter 'config' is not assigned and could be declared final | 3 | 100 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 102–103 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 104 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 105 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 3 | 128 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 138 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 151 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 152 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 153 |
| MethodArgumentCouldBeFinal | Parameter 'object' is not assigned and could be declared final | 3 | 160 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 3 | 160 |
| LocalVariableCouldBeFinal | Local variable 'cacheElement' could be declared final | 3 | 162 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 192 |
| LocalVariableCouldBeFinal | Local variable 'values' could be declared final | 3 | 192 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 194 |
| LocalVariableCouldBeFinal | Local variable 'cachedElement' could be declared final | 3 | 196 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 200 |
| LocalVariableCouldBeFinal | Local variable 'keysWithExpiryCheck' could be declared final | 3 | 224 |
| LongVariable | Avoid excessively long variable names like keysWithExpiryCheck | 3 | 224 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 234 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 3 | 241 |
| CyclomaticComplexity | The method 'getObject(String)' has a cyclomatic complexity of 11. | 3 | 241–291 |
| LocalVariableCouldBeFinal | Local variable 'cachedElement' could be declared final | 3 | 243 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 252 |
| LocalVariableCouldBeFinal | Local variable 'cachedObject' could be declared final | 3 | 252 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 254 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObject' could be declared final | 3 | 258 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObject | 3 | 258 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 259 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 267 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 268 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObject | 3 | 286 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObject' could be declared final | 3 | 286 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 287 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 3 | 297 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 325 |
| LocalVariableCouldBeFinal | Local variable 'cachedElement' could be declared final | 3 | 327 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 335 |
| LocalVariableCouldBeFinal | Local variable 'object' could be declared final | 3 | 335 |
| LocalVariableCouldBeFinal | Local variable 'refreshableObject' could be declared final | 3 | 339 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 340 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 344 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 346 |
org/apache/fulcrum/cache/impl/JCSCacheService.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| LongVariable | Avoid excessively long variable names like DEFAULT_CACHE_CHECK_FREQUENCY | 3 | 61 |
| LongVariable | Avoid excessively long variable names like cacheCheckFrequency | 3 | 66 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 66 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 71 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 76 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 81 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 89 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 3 | 89 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 92 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 94–97 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 94–97 |
| MethodArgumentCouldBeFinal | Parameter 'config' is not assigned and could be declared final | 3 | 105 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 107–108 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 109 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 110–111 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 3 | 125 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 135 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 148 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 149 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 151 |
| CyclomaticComplexity | The method 'getObject(String)' has a cyclomatic complexity of 11. | 3 | 158–206 |
| LocalVariableCouldBeFinal | Local variable 'cachedObject' could be declared final | 3 | 161 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 169 |
| LocalVariableCouldBeFinal | Local variable 'refreshableObject' could be declared final | 3 | 173 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 174 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 182 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 183 |
| LocalVariableCouldBeFinal | Local variable 'refreshableCachedObject' could be declared final | 3 | 201 |
| LongVariable | Avoid excessively long variable names like refreshableCachedObject | 3 | 201 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 202 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 3 | 213 |
| MethodArgumentCouldBeFinal | Parameter 'cachedObject' is not assigned and could be declared final | 3 | 213 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 219–224 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 224 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 224 |
| LocalVariableCouldBeFinal | Local variable 'attrib' could be declared final | 3 | 227 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 231 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 235 |
| LocalVariableCouldBeFinal | Local variable 'tmp0' could be declared final | 3 | 237 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 239 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 242 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 243 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 249 |
| MethodArgumentCouldBeFinal | Parameter 'objectId' is not assigned and could be declared final | 3 | 257 |
| LocalVariableCouldBeFinal | Local variable 'keys' could be declared final | 3 | 268 |
| LocalVariableCouldBeFinal | Local variable 'values' could be declared final | 3 | 279 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 280 |
| LocalVariableCouldBeFinal | Local variable 'cachedObject' could be declared final | 3 | 282 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 315 |
| LocalVariableCouldBeFinal | Local variable 'cachedObject' could be declared final | 3 | 317 |
| LocalVariableCouldBeFinal | Local variable 'refreshableObject' could be declared final | 3 | 326 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 327 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 331 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 333 |
| LocalVariableCouldBeFinal | Local variable 'baos' could be declared final | 3 | 348 |
| LocalVariableCouldBeFinal | Local variable 'out' could be declared final | 3 | 349 |
| LocalVariableCouldBeFinal | Local variable 'keys' could be declared final | 3 | 350 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 352 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 364 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 364 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'count' (lines '373'-'379'). | 5 | 373–379 |
| LocalVariableCouldBeFinal | Local variable 'key' could be declared final | 3 | 375 |
| DataflowAnomalyAnalysis | Found 'DD'-anomaly for variable 'count' (lines '379'-'379'). | 5 | 379 |