PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/userban/bean/AbstractBean.java
Violation |
Line |
Avoid variables with short names like id |
57 |
fr/paris/lutece/plugins/userban/bean/AbstractFilter.java
Violation |
Line |
Avoid variables with short names like id |
98 |
fr/paris/lutece/plugins/userban/bean/user/User.java
Violation |
Line |
Avoid short class names like User |
55–160 |
Avoid variables with short names like id |
139 |
fr/paris/lutece/plugins/userban/bean/user/UserFilter.java
Violation |
Line |
Avoid variables with short names like id |
130 |
fr/paris/lutece/plugins/userban/bean/user/User_.java
Violation |
Line |
Use of modifier volatile is not recommended. |
47 |
Use of modifier volatile is not recommended. |
48 |
Use of modifier volatile is not recommended. |
49 |
Use of modifier volatile is not recommended. |
50 |
Use of modifier volatile is not recommended. |
51 |
fr/paris/lutece/plugins/userban/dao/AbstractDAO.java
Violation |
Line |
Avoid variables with short names like em |
94 |
Avoid variables with short names like cb |
95 |
Avoid variables with short names like pq |
111 |
Avoid variables with short names like pq |
131 |
Avoid variables with short names like em |
197 |
Avoid variables with short names like cb |
198 |
Avoid variables with short names like cq |
199 |
fr/paris/lutece/plugins/userban/dao/commons/PagedQuery.java
Violation |
Line |
Perhaps '_paginationProperties' could be replaced by a local variable. |
63 |
fr/paris/lutece/plugins/userban/service/AbstractService.java
Violation |
Line |
Avoid variables with short names like id |
66 |
Consider simply returning the value vs storing it in local variable 'bean' |
69 |
Avoid variables with short names like id |
99 |
fr/paris/lutece/plugins/userban/service/IAbstractService.java
Violation |
Line |
Avoid modifiers which are implied by the context |
78 |
Avoid modifiers which are implied by the context |
86 |
fr/paris/lutece/plugins/userban/service/UserbanPlugin.java
Violation |
Line |
Overriding method merely calls super |
45–48 |
fr/paris/lutece/plugins/userban/service/user/IUserService.java
Violation |
Line |
Avoid variables with short names like id |
57 |
fr/paris/lutece/plugins/userban/service/user/UserService.java
Violation |
Line |
Avoid variables with short names like id |
73 |
fr/paris/lutece/plugins/userban/web/AbstractJspBean.java
Violation |
Line |
Avoid unused private fields such as '_strCurrentPageIndex'. |
63 |
Private field '_strCurrentPageIndex' could be made final; it is only initialized in the declaration or constructor. |
63 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
73 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
85 |
fr/paris/lutece/plugins/userban/web/user/UserJspBean.java
Violation |
Line |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
218 |