Fork me on GitHub

PMD Results

The following document contains the results of PMD 5.2.1.

Files

fr/paris/lutece/plugins/crmclient/business/CRMItem.java

Violation Line
Abstract classes should be named AbstractXXX 49–127
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 55
Document empty constructor 61–63
Avoid unnecessary constructors - the compiler will generate these for you 61–63
Avoid appending characters as strings in StringBuffer.append. 121

fr/paris/lutece/plugins/crmclient/service/ICRMClientService.java

Violation Line
Rather than using a lot of String arguments, consider using a container object for those values. 62
Rather than using a lot of String arguments, consider using a container object for those values. 72
Rather than using a lot of String arguments, consider using a container object for those values. 102
Rather than using a lot of String arguments, consider using a container object for those values. 114–115
Rather than using a lot of String arguments, consider using a container object for those values. 127–128
Rather than using a lot of String arguments, consider using a container object for those values. 141–142
Rather than using a lot of String arguments, consider using a container object for those values. 156–157
Rather than using a lot of String arguments, consider using a container object for those values. 171–172

fr/paris/lutece/plugins/crmclient/service/queue/DatabaseQueue.java

Violation Line
Use block level rather than method level synchronization 57–62
Use block level rather than method level synchronization 68–80

fr/paris/lutece/plugins/crmclient/util/CRMException.java

Violation Line
Avoid variables with short names like e 54