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 |
