Fork me on GitHub

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 3

fr/paris/lutece/plugins/avatar/service/AbstractAvatarProvider.java

Rule Violation Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 64
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 64
ShortVariable Avoid variables with short names like t 67
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 82
ShortVariable Avoid variables with short names like md 83
ShortVariable Avoid variables with short names like sb 90
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 9295

fr/paris/lutece/plugins/avatar/service/LuteceAvatarProvider.java

Rule Violation Line
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strUrl' 57

Priority 4

fr/paris/lutece/plugins/avatar/service/LuteceAvatarProvider.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.util.url.UrlItem' 37

Files

fr/paris/lutece/plugins/avatar/service/AbstractAvatarProvider.java

Rule Violation Priority Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 64
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 64
ShortVariable Avoid variables with short names like t 3 67
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 82
ShortVariable Avoid variables with short names like md 3 83
ShortVariable Avoid variables with short names like sb 3 90
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 9295

fr/paris/lutece/plugins/avatar/service/LuteceAvatarProvider.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.util.url.UrlItem' 4 37
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strUrl' 3 57