PMD Results
The following document contains the results of PMD 6.13.0.
Violations By Priority
Priority 1
fr/paris/lutece/plugins/notifygru/modules/appointment/provider/AppointmentNotifyGruConstants.java
| Rule |
Violation |
Line |
| ClassNamingConventions |
The utility class name 'AppointmentNotifyGruConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' |
39–65 |
Priority 3
fr/paris/lutece/plugins/notifygru/modules/appointment/provider/AppointmentProvider.java
| Rule |
Violation |
Line |
| ExcessiveImports |
A high number of imports can indicate a high degree of coupling within an object. |
34–463 |
| TooManyMethods |
This class has too many methods, consider refactoring it. |
88–463 |
| ImmutableField |
Private field '_appointment' could be made final; it is only initialized in the declaration or constructor. |
124 |
| ImmutableField |
Private field '_appointmentForm' could be made final; it is only initialized in the declaration or constructor. |
126 |
| ImmutableField |
Private field '_appointmentGru' could be made final; it is only initialized in the declaration or constructor. |
128 |
| ImmutableField |
Private field '_resourceHistoryService' could be made final; it is only initialized in the declaration or constructor. |
131 |
| ImmutableField |
Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. |
132 |
| ImmutableField |
Private field '_commentService' could be made final; it is only initialized in the declaration or constructor. |
133 |
| UseStringBufferForStringAppends |
Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings |
200 |
| UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
275 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
417–424 |
| UnusedFormalParameter |
Avoid unused method parameters such as 'appointment'. |
436 |
fr/paris/lutece/plugins/notifygru/modules/appointment/provider/AppointmentProviderManager.java
| Rule |
Violation |
Line |
| ShortVariable |
Avoid variables with short names like wf |
90 |
Files
fr/paris/lutece/plugins/notifygru/modules/appointment/provider/AppointmentNotifyGruConstants.java
| Rule |
Violation |
Priority |
Line |
| ClassNamingConventions |
The utility class name 'AppointmentNotifyGruConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' |
1 |
39–65 |
fr/paris/lutece/plugins/notifygru/modules/appointment/provider/AppointmentProvider.java
| Rule |
Violation |
Priority |
Line |
| ExcessiveImports |
A high number of imports can indicate a high degree of coupling within an object. |
3 |
34–463 |
| TooManyMethods |
This class has too many methods, consider refactoring it. |
3 |
88–463 |
| ImmutableField |
Private field '_appointment' could be made final; it is only initialized in the declaration or constructor. |
3 |
124 |
| ImmutableField |
Private field '_appointmentForm' could be made final; it is only initialized in the declaration or constructor. |
3 |
126 |
| ImmutableField |
Private field '_appointmentGru' could be made final; it is only initialized in the declaration or constructor. |
3 |
128 |
| ImmutableField |
Private field '_resourceHistoryService' could be made final; it is only initialized in the declaration or constructor. |
3 |
131 |
| ImmutableField |
Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. |
3 |
132 |
| ImmutableField |
Private field '_commentService' could be made final; it is only initialized in the declaration or constructor. |
3 |
133 |
| UseStringBufferForStringAppends |
Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings |
3 |
200 |
| UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
3 |
275 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
417–424 |
| UnusedFormalParameter |
Avoid unused method parameters such as 'appointment'. |
3 |
436 |
fr/paris/lutece/plugins/notifygru/modules/appointment/provider/AppointmentProviderManager.java
| Rule |
Violation |
Priority |
Line |
| ShortVariable |
Avoid variables with short names like wf |
3 |
90 |
