The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The constant name '_listPrefixToRemove' doesn't match '[A-Z][A-Z_0-9]*' | 73 |
| FieldNamingConventions | The constant name '_listTypeAllowedForSearch' doesn't match '[A-Z][A-Z_0-9]*' | 74 |
| FormalParameterNamingConventions | The method parameter name 'SQL_QUERY_SELECTALL_ID' doesn't match '[a-z][a-zA-Z0-9]*' | 85 |
| LocalVariableNamingConventions | The local variable name 'WhereClauses' doesn't match '[a-z][a-zA-Z0-9]*' | 107 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'mapAttributes' doesn't match '_[a-z][a-zA-Z0-9]*' | 11 |
| FieldNamingConventions | The field name 'strFormName' doesn't match '_[a-z][a-zA-Z0-9]*' | 12 |
| FieldNamingConventions | The field name 'strStatus' doesn't match '_[a-z][a-zA-Z0-9]*' | 13 |
| FieldNamingConventions | The field name 'dateForm' doesn't match '_[a-z][a-zA-Z0-9]*' | 14 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 110 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'strName' doesn't match '_[a-z][a-zA-Z0-9]*' | 5 |
| FieldNamingConventions | The field name 'strFirstName' doesn't match '_[a-z][a-zA-Z0-9]*' | 6 |
| FieldNamingConventions | The field name 'strRegistrationNumber' doesn't match '_[a-z][a-zA-Z0-9]*' | 7 |
| Rule | Violation | Line |
|---|---|---|
| SystemPrintln | System.out.println is used | 451 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 48–248 |
| CommentRequired | Header comments are required | 48–248 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 51 |
| UnusedPrivateField | Avoid unused private fields such as 'SQL_ASC'. | 63 |
| UnusedPrivateField | Avoid unused private fields such as 'SQL_DESC'. | 64 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 90 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 91 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 119–124 |
| CollapsibleIfStatements | These nested if statements could be combined | 119–124 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 148 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 149 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 71–74 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 233–235 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 9–59 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 11 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 17 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 20 |
| CommentRequired | Public method and constructor comments are required | 23–25 |
| CommentRequired | Public method and constructor comments are required | 26–28 |
| CommentRequired | Public method and constructor comments are required | 29–31 |
| CommentRequired | Public method and constructor comments are required | 32–34 |
| CommentRequired | Public method and constructor comments are required | 42–47 |
| CommentRequired | Public method and constructor comments are required | 49–57 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 3–46 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the strName field if you want a default access modifier | 5 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 5 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the strFirstName field if you want a default access modifier | 6 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 6 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the strRegistrationNumber field if you want a default access modifier | 7 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 7 |
| CommentRequired | Public method and constructor comments are required | 9–13 |
| CommentRequired | Public method and constructor comments are required | 20–22 |
| CommentRequired | Public method and constructor comments are required | 24–26 |
| CommentRequired | Public method and constructor comments are required | 28–30 |
| CommentRequired | Public method and constructor comments are required | 32–34 |
| CommentRequired | Public method and constructor comments are required | 36–38 |
| CommentRequired | Public method and constructor comments are required | 40–42 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 52–269 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 52–269 |
| AddEmptyString | Do not add empty strings | 163 |
| CommentRequired | Protected method constructor comments are required | 170–185 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 172 |
| CommentRequired | Protected method constructor comments are required | 187–200 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 212 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 212 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getPluginDefaultNumberOfItemPerPage method if you want a default access modifier | 214 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 214 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 220 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 232 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 264 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 36–748 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 93–748 |
| UnusedPrivateField | Avoid unused private fields such as '_listIdFieldsearchs'. | 151 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 152 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 158 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _unitUserService field if you want a default access modifier | 158 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _unitService field if you want a default access modifier | 159 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 159 |
| NPathComplexity | The method 'getManageFieldsearchs(HttpServletRequest)' has an NPath complexity of 660960 | 167–478 |
| NcssCount | The method 'getManageFieldsearchs(HttpServletRequest)' has a NCSS line count of 130. | 167–478 |
| CyclomaticComplexity | The method 'getManageFieldsearchs(HttpServletRequest)' has a cyclomatic complexity of 32. | 167–478 |
| ExcessiveMethodLength | Avoid really long methods. | 167–478 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 169 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 175 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 176 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 190 |
| AvoidDuplicateLiterals | The String literal "timestamp" appears 5 times in this file; the first occurrence is on line 190 | 190 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 190–193 |
| AvoidDuplicateLiterals | The String literal "\" } } ]" appears 4 times in this file; the first occurrence is on line 191 | 191 |
| ControlStatementBraces | This statement should have braces | 191 |
| AvoidDuplicateLiterals | The String literal ", \"sort\": [ { \"" appears 4 times in this file; the first occurrence is on line 191 | 191 |
| ControlStatementBraces | This statement should have braces | 193 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 213–216 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 213 |
| ControlStatementBraces | This statement should have braces | 214 |
| ControlStatementBraces | This statement should have braces | 216 |
| UnusedLocalVariable | Avoid unused local variables such as 'searchRequest'. | 226 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 226 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 230 |
| AvoidDuplicateLiterals | The String literal "filter_" appears 4 times in this file; the first occurrence is on line 237 | 237 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 276 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 286 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 294 |
| ControlStatementBraces | This statement should have braces | 294 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 297 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 303 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 308 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 315–324 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 317 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 323 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 330 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 334 |
| UnusedLocalVariable | Avoid unused local variables such as 'strPrettyJson'. | 334 |
| UnusedLocalVariable | Avoid unused local variables such as 'strUnitCodeUGD'. | 384 |
| ControlStatementBraces | This statement should have braces | 433 |
| AvoidDuplicateLiterals | The String literal "Nom" appears 5 times in this file; the first occurrence is on line 438 | 438 |
| ControlStatementBraces | This statement should have braces | 439 |
| ControlStatementBraces | This statement should have braces | 441 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 455 |
| ControlStatementBraces | This statement should have braces | 462 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 494 |
| ControlStatementBraces | This statement should have braces | 501 |
| ControlStatementBraces | This statement should have braces | 507 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 562 |
| IdenticalCatchBranches | 'catch' branch identical to 'ElasticClientException' branch | 564–567 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 566 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 582–591 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 583–591 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 593–596 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getPluginDefaultNumberOfItemPerPage method if you want a default access modifier | 594–596 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 615 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.ArrayList' | 38 |
| Rule | Violation | Line |
|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 96–99 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppPropertiesService' | 37 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.Collections' | 63 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.ArrayList' | 4 | 38 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 48–248 |
| CommentRequired | Header comments are required | 3 | 48–248 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 51 |
| UnusedPrivateField | Avoid unused private fields such as 'SQL_ASC'. | 3 | 63 |
| UnusedPrivateField | Avoid unused private fields such as 'SQL_DESC'. | 3 | 64 |
| FieldNamingConventions | The constant name '_listPrefixToRemove' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 73 |
| FieldNamingConventions | The constant name '_listTypeAllowedForSearch' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 74 |
| FormalParameterNamingConventions | The method parameter name 'SQL_QUERY_SELECTALL_ID' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 85 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 90 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 91 |
| LocalVariableNamingConventions | The local variable name 'WhereClauses' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 107 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 119–124 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 119–124 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 148 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 149 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 96–99 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 71–74 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 233–235 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 9–59 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 11 |
| FieldNamingConventions | The field name 'mapAttributes' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 11 |
| FieldNamingConventions | The field name 'strFormName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 12 |
| FieldNamingConventions | The field name 'strStatus' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 13 |
| FieldNamingConventions | The field name 'dateForm' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 14 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 17 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 20 |
| CommentRequired | Public method and constructor comments are required | 3 | 23–25 |
| CommentRequired | Public method and constructor comments are required | 3 | 26–28 |
| CommentRequired | Public method and constructor comments are required | 3 | 29–31 |
| CommentRequired | Public method and constructor comments are required | 3 | 32–34 |
| CommentRequired | Public method and constructor comments are required | 3 | 42–47 |
| CommentRequired | Public method and constructor comments are required | 3 | 49–57 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 110 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 3–46 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the strName field if you want a default access modifier | 3 | 5 |
| FieldNamingConventions | The field name 'strName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 5 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 5 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the strFirstName field if you want a default access modifier | 3 | 6 |
| FieldNamingConventions | The field name 'strFirstName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 6 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 6 |
| FieldNamingConventions | The field name 'strRegistrationNumber' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 7 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the strRegistrationNumber field if you want a default access modifier | 3 | 7 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 7 |
| CommentRequired | Public method and constructor comments are required | 3 | 9–13 |
| CommentRequired | Public method and constructor comments are required | 3 | 20–22 |
| CommentRequired | Public method and constructor comments are required | 3 | 24–26 |
| CommentRequired | Public method and constructor comments are required | 3 | 28–30 |
| CommentRequired | Public method and constructor comments are required | 3 | 32–34 |
| CommentRequired | Public method and constructor comments are required | 3 | 36–38 |
| CommentRequired | Public method and constructor comments are required | 3 | 40–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppPropertiesService' | 4 | 37 |
| CommentRequired | Header comments are required | 3 | 52–269 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 52–269 |
| AddEmptyString | Do not add empty strings | 3 | 163 |
| CommentRequired | Protected method constructor comments are required | 3 | 170–185 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 172 |
| CommentRequired | Protected method constructor comments are required | 3 | 187–200 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 212 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 212 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getPluginDefaultNumberOfItemPerPage method if you want a default access modifier | 3 | 214 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 214 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 220 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 232 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 264 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 36–748 |
| UnusedImports | Avoid unused imports such as 'java.util.Collections' | 4 | 63 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 93–748 |
| UnusedPrivateField | Avoid unused private fields such as '_listIdFieldsearchs'. | 3 | 151 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 152 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 158 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _unitUserService field if you want a default access modifier | 3 | 158 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _unitService field if you want a default access modifier | 3 | 159 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 159 |
| NPathComplexity | The method 'getManageFieldsearchs(HttpServletRequest)' has an NPath complexity of 660960 | 3 | 167–478 |
| NcssCount | The method 'getManageFieldsearchs(HttpServletRequest)' has a NCSS line count of 130. | 3 | 167–478 |
| CyclomaticComplexity | The method 'getManageFieldsearchs(HttpServletRequest)' has a cyclomatic complexity of 32. | 3 | 167–478 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 167–478 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 169 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 175 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 176 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 190 |
| AvoidDuplicateLiterals | The String literal "timestamp" appears 5 times in this file; the first occurrence is on line 190 | 3 | 190 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 190–193 |
| AvoidDuplicateLiterals | The String literal "\" } } ]" appears 4 times in this file; the first occurrence is on line 191 | 3 | 191 |
| ControlStatementBraces | This statement should have braces | 3 | 191 |
| AvoidDuplicateLiterals | The String literal ", \"sort\": [ { \"" appears 4 times in this file; the first occurrence is on line 191 | 3 | 191 |
| ControlStatementBraces | This statement should have braces | 3 | 193 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 213–216 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 213 |
| ControlStatementBraces | This statement should have braces | 3 | 214 |
| ControlStatementBraces | This statement should have braces | 3 | 216 |
| UnusedLocalVariable | Avoid unused local variables such as 'searchRequest'. | 3 | 226 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 226 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 230 |
| AvoidDuplicateLiterals | The String literal "filter_" appears 4 times in this file; the first occurrence is on line 237 | 3 | 237 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 276 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 286 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 294 |
| ControlStatementBraces | This statement should have braces | 3 | 294 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 297 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 303 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 308 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 315–324 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 317 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 323 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 330 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 334 |
| UnusedLocalVariable | Avoid unused local variables such as 'strPrettyJson'. | 3 | 334 |
| UnusedLocalVariable | Avoid unused local variables such as 'strUnitCodeUGD'. | 3 | 384 |
| ControlStatementBraces | This statement should have braces | 3 | 433 |
| AvoidDuplicateLiterals | The String literal "Nom" appears 5 times in this file; the first occurrence is on line 438 | 3 | 438 |
| ControlStatementBraces | This statement should have braces | 3 | 439 |
| ControlStatementBraces | This statement should have braces | 3 | 441 |
| SystemPrintln | System.out.println is used | 2 | 451 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 455 |
| ControlStatementBraces | This statement should have braces | 3 | 462 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 494 |
| ControlStatementBraces | This statement should have braces | 3 | 501 |
| ControlStatementBraces | This statement should have braces | 3 | 507 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 562 |
| IdenticalCatchBranches | 'catch' branch identical to 'ElasticClientException' branch | 3 | 564–567 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 566 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 582–591 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 583–591 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 593–596 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getPluginDefaultNumberOfItemPerPage method if you want a default access modifier | 3 | 594–596 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 615 |
