PMD Results
The following document contains the results of PMD 6.13.0.
Violations By Priority
Priority 1
fr/paris/lutece/plugins/accesslogger/service/DefaultAccessLogger.java
Priority 2
fr/paris/lutece/plugins/accesslogger/service/DefaultAccessLogger.java
Rule |
Violation |
Line |
LoggerIsNotStaticFinal |
The Logger variable declaration does not contain the static and final modifiers |
67 |
Priority 3
fr/paris/lutece/plugins/accesslogger/service/AccessLogService.java
Rule |
Violation |
Line |
ImmutableField |
Private field '_accessLogger' could be made final; it is only initialized in the declaration or constructor. |
46 |
fr/paris/lutece/plugins/accesslogger/service/DefaultAccessLogger.java
Rule |
Violation |
Line |
MissingOverride |
The method 'info(String, String, String, Object)' is missing an @Override annotation. |
79–88 |
MissingOverride |
The method 'debug(String, String, String, Object)' is missing an @Override annotation. |
99–108 |
MissingOverride |
The method 'trace(String, String, String, Object)' is missing an @Override annotation. |
118–127 |
UseObjectForClearerAPI |
Rather than using a lot of String arguments, consider using a container object for those values. |
138 |
UseObjectForClearerAPI |
Rather than using a lot of String arguments, consider using a container object for those values. |
169–170 |
ShortVariable |
Avoid variables with short names like md |
199 |
ShortVariable |
Avoid variables with short names like sb |
203 |
fr/paris/lutece/plugins/accesslogger/service/IAccessLogger.java
Rule |
Violation |
Line |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_READ': the field is declared in an interface type |
43 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_CREATE': the field is declared in an interface type |
44 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_DELETE': the field is declared in an interface type |
45 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_MODIFY': the field is declared in an interface type |
46 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_ACTION': the field is declared in an interface type |
47 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_CONNECT': the field is declared in an interface type |
49 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_DISCONNECT': the field is declared in an interface type |
50 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_RIGHTS': the field is declared in an interface type |
51 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'PROPERTY_SITE_CODE': the field is declared in an interface type |
53 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'LOGGER_ACCESS_LOG': the field is declared in an interface type |
54 |
UnnecessaryModifier |
Unnecessary modifier 'public' on method 'info': the method is declared in an interface type |
65 |
UnnecessaryModifier |
Unnecessary modifier 'public' on method 'debug': the method is declared in an interface type |
76 |
UnnecessaryModifier |
Unnecessary modifier 'public' on method 'trace': the method is declared in an interface type |
86 |
Files
fr/paris/lutece/plugins/accesslogger/service/AccessLogService.java
Rule |
Violation |
Priority |
Line |
ImmutableField |
Private field '_accessLogger' could be made final; it is only initialized in the declaration or constructor. |
3 |
46 |
fr/paris/lutece/plugins/accesslogger/service/DefaultAccessLogger.java
Rule |
Violation |
Priority |
Line |
FieldNamingConventions |
The final field name '_bAddHashToLogs' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
63 |
FieldNamingConventions |
The final field name '_messageFormat' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
64 |
FieldNamingConventions |
The final field name '_messageFormatSeparator' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
65 |
LoggerIsNotStaticFinal |
The Logger variable declaration does not contain the static and final modifiers |
2 |
67 |
MissingOverride |
The method 'info(String, String, String, Object)' is missing an @Override annotation. |
3 |
79–88 |
MissingOverride |
The method 'debug(String, String, String, Object)' is missing an @Override annotation. |
3 |
99–108 |
MissingOverride |
The method 'trace(String, String, String, Object)' is missing an @Override annotation. |
3 |
118–127 |
UseObjectForClearerAPI |
Rather than using a lot of String arguments, consider using a container object for those values. |
3 |
138 |
UseObjectForClearerAPI |
Rather than using a lot of String arguments, consider using a container object for those values. |
3 |
169–170 |
ShortVariable |
Avoid variables with short names like md |
3 |
199 |
ShortVariable |
Avoid variables with short names like sb |
3 |
203 |
fr/paris/lutece/plugins/accesslogger/service/IAccessLogger.java
Rule |
Violation |
Priority |
Line |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_READ': the field is declared in an interface type |
3 |
43 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_CREATE': the field is declared in an interface type |
3 |
44 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_DELETE': the field is declared in an interface type |
3 |
45 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_MODIFY': the field is declared in an interface type |
3 |
46 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_ACTION': the field is declared in an interface type |
3 |
47 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_CONNECT': the field is declared in an interface type |
3 |
49 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_DISCONNECT': the field is declared in an interface type |
3 |
50 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'EVENT_TYPE_RIGHTS': the field is declared in an interface type |
3 |
51 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'PROPERTY_SITE_CODE': the field is declared in an interface type |
3 |
53 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'LOGGER_ACCESS_LOG': the field is declared in an interface type |
3 |
54 |
UnnecessaryModifier |
Unnecessary modifier 'public' on method 'info': the method is declared in an interface type |
3 |
65 |
UnnecessaryModifier |
Unnecessary modifier 'public' on method 'debug': the method is declared in an interface type |
3 |
76 |
UnnecessaryModifier |
Unnecessary modifier 'public' on method 'trace': the method is declared in an interface type |
3 |
86 |