The following document contains the results of PMD 7.9.0.
Violations By Priority
Priority 3
io/wcm/caravan/io/http/impl/CaravanHttpServiceConfig.java
Rule |
Violation |
Line |
SimplifyConditional |
No need to check for null before an instanceof |
239 |
io/wcm/caravan/io/http/impl/servletclient/HttpServletRequestMapper.java
Rule |
Violation |
Line |
PreserveStackTrace |
Thrown exception does not preserve the stack trace of exception 'ex' on all code paths |
88 |
OptimizableToArrayCall |
This call to Collection.toArray() may be optimizable |
160 |
io/wcm/caravan/io/http/request/CaravanHttpRequestBuilder.java
Rule |
Violation |
Line |
AvoidStringBufferField |
StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). |
55 |
Priority 4
io/wcm/caravan/io/http/impl/ribbon/CachingLoadBalancerFactory.java
Rule |
Violation |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'LoadBalancerFactory': 'SIMPLE' is already in scope because it is inherited by an enclosing type |
42 |
io/wcm/caravan/io/http/impl/servletclient/HttpServletResponseMapper.java
Rule |
Violation |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'HttpServletResponse': 'SC_OK' is already in scope because it is inherited by an enclosing type |
57 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'HttpServletResponse': 'SC_OK' is already in scope because it is inherited by an enclosing type |
141 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'HttpServletResponse': 'SC_MOVED_TEMPORARILY' is already in scope because it is inherited by an enclosing type |
199 |
io/wcm/caravan/io/http/request/CaravanHttpRequest.java
Rule |
Violation |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'CaravanHttpRequest': 'CORRELATION_ID_HEADER_NAME' is already in scope because it is declared in an enclosing type |
174 |
Priority 5
io/wcm/caravan/io/http/impl/CaravanHttpServiceConfig.java
Rule |
Violation |
Line |
InvalidLogMessageFormat |
Missing arguments, expected 5 arguments but found 4 |
335–336 |
io/wcm/caravan/io/http/impl/CaravanHttpServiceConfigValidator.java
Rule |
Violation |
Line |
InvalidLogMessageFormat |
Too many arguments, expected 0 argument but found 1 |
63 |
io/wcm/caravan/io/http/impl/CaravanHttpThreadPoolConfig.java
Rule |
Violation |
Line |
InvalidLogMessageFormat |
Too many arguments, expected 0 argument but found 1 |
109 |
Files
io/wcm/caravan/io/http/impl/CaravanHttpServiceConfigValidator.java
Rule |
Violation |
Priority |
Line |
InvalidLogMessageFormat |
Too many arguments, expected 0 argument but found 1 |
5 |
63 |
io/wcm/caravan/io/http/impl/CaravanHttpThreadPoolConfig.java
Rule |
Violation |
Priority |
Line |
InvalidLogMessageFormat |
Too many arguments, expected 0 argument but found 1 |
5 |
109 |
io/wcm/caravan/io/http/impl/ribbon/CachingLoadBalancerFactory.java
Rule |
Violation |
Priority |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'LoadBalancerFactory': 'SIMPLE' is already in scope because it is inherited by an enclosing type |
4 |
42 |
io/wcm/caravan/io/http/impl/servletclient/HttpServletRequestMapper.java
Rule |
Violation |
Priority |
Line |
PreserveStackTrace |
Thrown exception does not preserve the stack trace of exception 'ex' on all code paths |
3 |
88 |
OptimizableToArrayCall |
This call to Collection.toArray() may be optimizable |
3 |
160 |
io/wcm/caravan/io/http/impl/servletclient/HttpServletResponseMapper.java
Rule |
Violation |
Priority |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'HttpServletResponse': 'SC_OK' is already in scope because it is inherited by an enclosing type |
4 |
57 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'HttpServletResponse': 'SC_OK' is already in scope because it is inherited by an enclosing type |
4 |
141 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'HttpServletResponse': 'SC_MOVED_TEMPORARILY' is already in scope because it is inherited by an enclosing type |
4 |
199 |
io/wcm/caravan/io/http/request/CaravanHttpRequest.java
Rule |
Violation |
Priority |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'CaravanHttpRequest': 'CORRELATION_ID_HEADER_NAME' is already in scope because it is declared in an enclosing type |
4 |
174 |
io/wcm/caravan/io/http/request/CaravanHttpRequestBuilder.java
Rule |
Violation |
Priority |
Line |
AvoidStringBufferField |
StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). |
3 |
55 |