The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'certificateDate' doesn't match '_[a-z][a-zA-Z0-9]*' | 4 |
| FieldNamingConventions | The field name 'lastUpdateClientCode' doesn't match '_[a-z][a-zA-Z0-9]*' | 5 |
| FieldNamingConventions | The field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 6 |
| FieldNamingConventions | The field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 7 |
| FieldNamingConventions | The field name 'pivot' doesn't match '_[a-z][a-zA-Z0-9]*' | 8 |
| FieldNamingConventions | The field name 'certifierCode' doesn't match '_[a-z][a-zA-Z0-9]*' | 9 |
| FieldNamingConventions | The field name 'certificateExpirationDate' doesn't match '_[a-z][a-zA-Z0-9]*' | 10 |
| FieldNamingConventions | The field name 'type' doesn't match '_[a-z][a-zA-Z0-9]*' | 11 |
| FieldNamingConventions | The field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 12 |
| FieldNamingConventions | The field name 'certifierName' doesn't match '_[a-z][a-zA-Z0-9]*' | 13 |
| FieldNamingConventions | The field name 'key' doesn't match '_[a-z][a-zA-Z0-9]*' | 14 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'took' doesn't match '_[a-z][a-zA-Z0-9]*' | 6 |
| FieldNamingConventions | The field name 'timedOut' doesn't match '_[a-z][a-zA-Z0-9]*' | 7 |
| FieldNamingConventions | The field name 'shards' doesn't match '_[a-z][a-zA-Z0-9]*' | 8 |
| FieldNamingConventions | The field name 'hits' doesn't match '_[a-z][a-zA-Z0-9]*' | 9 |
| FieldNamingConventions | The field name '_scroll_id' doesn't match '_[a-z][a-zA-Z0-9]*' | 10 |
| FieldNamingConventions | The field name 'pit_id' doesn't match '_[a-z][a-zA-Z0-9]*' | 11 |
| MethodNamingConventions | The instance method name 'get_scroll_id' doesn't match '[a-z][a-zA-Z0-9]*' | 13–15 |
| MethodNamingConventions | The instance method name 'set_scroll_id' doesn't match '[a-z][a-zA-Z0-9]*' | 17–19 |
| FormalParameterNamingConventions | The method parameter name '_scroll_id' doesn't match '[a-z][a-zA-Z0-9]*' | 17 |
| MethodNamingConventions | The instance method name 'getPit_id' doesn't match '[a-z][a-zA-Z0-9]*' | 53–55 |
| FormalParameterNamingConventions | The method parameter name 'pit_id' doesn't match '[a-z][a-zA-Z0-9]*' | 57 |
| MethodNamingConventions | The instance method name 'setPit_id' doesn't match '[a-z][a-zA-Z0-9]*' | 57–59 |
| FieldNamingConventions | The field name 'total' doesn't match '_[a-z][a-zA-Z0-9]*' | 66 |
| FieldNamingConventions | The field name 'hits' doesn't match '_[a-z][a-zA-Z0-9]*' | 67 |
| FieldNamingConventions | The field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 87 |
| FieldNamingConventions | The field name 'relation' doesn't match '_[a-z][a-zA-Z0-9]*' | 88 |
| FieldNamingConventions | The field name 'sort' doesn't match '_[a-z][a-zA-Z0-9]*' | 98 |
| MethodNamingConventions | The instance method name 'get_source' doesn't match '[a-z][a-zA-Z0-9]*' | 100–102 |
| MethodNamingConventions | The instance method name 'set_source' doesn't match '[a-z][a-zA-Z0-9]*' | 103–105 |
| FormalParameterNamingConventions | The method parameter name '_source' doesn't match '[a-z][a-zA-Z0-9]*' | 103 |
| MethodNamingConventions | The instance method name 'get_score' doesn't match '[a-z][a-zA-Z0-9]*' | 106–108 |
| MethodNamingConventions | The instance method name 'set_score' doesn't match '[a-z][a-zA-Z0-9]*' | 109–111 |
| FormalParameterNamingConventions | The method parameter name '_score' doesn't match '[a-z][a-zA-Z0-9]*' | 109 |
| MethodNamingConventions | The instance method name 'get_id' doesn't match '[a-z][a-zA-Z0-9]*' | 112–114 |
| FormalParameterNamingConventions | The method parameter name '_id' doesn't match '[a-z][a-zA-Z0-9]*' | 115 |
| MethodNamingConventions | The instance method name 'set_id' doesn't match '[a-z][a-zA-Z0-9]*' | 115–117 |
| FieldNamingConventions | The field name 'customerId' doesn't match '_[a-z][a-zA-Z0-9]*' | 128 |
| FieldNamingConventions | The field name 'connectionId' doesn't match '_[a-z][a-zA-Z0-9]*' | 129 |
| FieldNamingConventions | The field name 'attributes' doesn't match '_[a-z][a-zA-Z0-9]*' | 130 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 106 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 106 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 106 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_bAutoExtract' doesn't match '[a-z][a-zA-Z0-9]*' | 158 |
| FormalParameterNamingConventions | The method parameter name '_strPassword' doesn't match '[a-z][a-zA-Z0-9]*' | 174 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 3–23 |
| Rule | Violation | Line |
|---|---|---|
| AvoidFileStream | Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter | 39 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'exportModel' doesn't match '_[a-z][a-zA-Z0-9]*' | 24 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 99 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 130 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 171 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 26 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 81 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 150 |
| Rule | Violation | Line |
|---|---|---|
| SystemPrintln | System.out.println is used | 215 |
| SystemPrintln | System.out.println is used | 224 |
| Rule | Violation | Line |
|---|---|---|
| SystemPrintln | System.out.println is used | 228 |
| SystemPrintln | System.out.println is used | 232 |
| SystemPrintln | System.out.println is used | 380 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 1–325 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 44–325 |
| CommentRequired | Header comments are required | 44–325 |
| NPathComplexity | The method 'generateExport(int, String)' has an NPath complexity of 1728 | 56–173 |
| CyclomaticComplexity | The method 'generateExport(int, String)' has a cyclomatic complexity of 15. | 56–173 |
| ExcessiveMethodLength | Avoid really long methods. | 56–173 |
| NcssCount | The method 'generateExport(int, String)' has a NCSS line count of 60. | 56–173 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 59 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 123–142 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 163 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 211 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 214 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 215 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 218 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 236–265 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 286 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 3–85 |
| DataClass | The class 'Attribute' is suspected to be a Data Class (WOC=0.000%, NOPA=11, NOAM=22, WMC=22) | 3–85 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 5–154 |
| DataClass | The class 'ElasticsearchResponseJSON' is suspected to be a Data Class (WOC=16.667%, NOPA=0, NOAM=10, WMC=12) | 5–154 |
| CommentRequired | Public method and constructor comments are required | 13–15 |
| CommentRequired | Public method and constructor comments are required | 17–19 |
| CommentRequired | Header comments are required | 61–63 |
| DataClass | The class 'Hits' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=4) | 65–84 |
| CommentRequired | Header comments are required | 65–84 |
| AvoidFieldNameMatchingTypeName | It is somewhat confusing to have a field name matching the declaring class name | 67 |
| CommentRequired | Header comments are required | 86–90 |
| UnusedPrivateField | Avoid unused private fields such as 'value'. | 87 |
| UnusedPrivateField | Avoid unused private fields such as 'relation'. | 88 |
| CommentRequired | Header comments are required | 92–125 |
| UnusedPrivateField | Avoid unused private fields such as '_index'. | 93 |
| UnusedPrivateField | Avoid unused private fields such as '_type'. | 94 |
| CommentRequired | Public method and constructor comments are required | 100–102 |
| CommentRequired | Public method and constructor comments are required | 103–105 |
| CommentRequired | Public method and constructor comments are required | 106–108 |
| CommentRequired | Public method and constructor comments are required | 109–111 |
| CommentRequired | Public method and constructor comments are required | 112–114 |
| CommentRequired | Public method and constructor comments are required | 115–117 |
| ArrayIsStoredDirectly | The user-supplied array 'sort' is stored directly. | 121 |
| CommentRequired | Header comments are required | 127–152 |
| DataClass | The class 'Source' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=6, WMC=6) | 127–152 |
| Rule | Violation | Line |
|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 190–192 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 51–317 |
| UnusedPrivateField | Avoid unused private fields such as 'SQL_QUERY_SELECTALL_ID_BY_ID_PROFIL'. | 62 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 233–235 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 46–130 |
| Rule | Violation | Line |
|---|---|---|
| AvoidDuplicateLiterals | The String literal "SELECT " appears 4 times in this file; the first occurrence is on line 55 | 55 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 75 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 240–242 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 146–148 |
| CommentRequired | Public method and constructor comments are required | 150–152 |
| CommentRequired | Public method and constructor comments are required | 154–156 |
| CommentRequired | Public method and constructor comments are required | 158–160 |
| CommentRequired | Public method and constructor comments are required | 162–164 |
| CommentRequired | Public method and constructor comments are required | 166–168 |
| CommentRequired | Public method and constructor comments are required | 170–172 |
| CommentRequired | Public method and constructor comments are required | 174–176 |
| CommentRequired | Public method and constructor comments are required | 178–180 |
| CommentRequired | Public method and constructor comments are required | 182–184 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 15–40 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 25–27 |
| CollapsibleIfStatements | These nested if statements could be combined | 25–27 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 13–51 |
| ShortVariable | Avoid variables with short names like sb | 21 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 35 characters appended. | 21 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 37 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 3–23 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 23–196 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 23–196 |
| AvoidDuplicateLiterals | The String literal " {\n" appears 4 times in this file; the first occurrence is on line 59 | 59 |
| IdenticalCatchBranches | 'catch' branch identical to 'JsonMappingException' branch | 184–187 |
| IdenticalCatchBranches | 'catch' branch identical to 'JsonMappingException' branch | 188–191 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 17–176 |
| ImmutableField | Private field '_profile' could be made final; it is only initialized in the declaration or constructor. | 23 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 28–31 |
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 128–131 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 137–140 |
| MissingOverride | The method 'getCertification()' is missing an @Override annotation. | 146–149 |
| MissingOverride | The method 'getFileName()' is missing an @Override annotation. | 155–158 |
| MissingOverride | The method 'isAutoExtract()' is missing an @Override annotation. | 163–166 |
| MissingOverride | The method 'getPassword()' is missing an @Override annotation. | 171–174 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 5–26 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 14–58 |
| ShortVariable | Avoid variables with short names like rt | 24 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 18–38 |
| UncommentedEmptyConstructor | Document empty constructor | 20–22 |
| CommentRequired | Public method and constructor comments are required | 20–22 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 20–22 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 27–32 |
| CommentRequired | Public method and constructor comments are required | 27–36 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 8–13 |
| CommentRequired | Public method and constructor comments are required | 9–12 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like e | 26 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like e | 25 |
| ShortVariable | Avoid variables with short names like e | 35 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 15–44 |
| ShortVariable | Avoid variables with short names like e | 20 |
| ShortVariable | Avoid variables with short names like e | 29 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like e | 24 |
| ShortVariable | Avoid variables with short names like e | 34 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 21–80 |
| CommentRequired | Public method and constructor comments are required | 26–30 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_storageDir' could be made final; it is only initialized in the declaration or constructor. | 47 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 238 |
| CommentRequired | Public method and constructor comments are required | 278–281 |
| CommentRequired | Public method and constructor comments are required | 283–286 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 50–110 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 109 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 109 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 36–416 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 81–416 |
| UnusedPrivateField | Avoid unused private fields such as 'ACTION_EXTRACTION_DATA'. | 119 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 141 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 152 |
| UnusedLocalVariable | Avoid unused local variables such as 'profilsReferenceList'. | 152 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 166–175 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 167–175 |
| AvoidDuplicateLiterals | The String literal "You don't have the right to modify this export profile." appears 6 times in this file; the first occurrence is on line 196 | 196 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 199 |
| UnusedLocalVariable | Avoid unused local variables such as 'lstProfils'. | 204 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 204 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 229 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 286 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 310 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 368 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 36–560 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 90–560 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 153 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 160 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 164 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 164 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 183–192 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 184–192 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 215 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 239 |
| AvoidDuplicateLiterals | The String literal "You don't have the right to modify this export profile." appears 4 times in this file; the first occurrence is on line 318 | 318 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 386 |
| ExcessiveMethodLength | Avoid really long methods. | 432–559 |
| CommentRequired | Public method and constructor comments are required | 432–559 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like Hits | 65–84 |
| ShortClassName | Avoid short class names like Hit | 92–125 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 121 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.Date' | 5 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.mail.MailService' | 9 |
| UnusedImports | Avoid unused imports such as 'org.apache.commons.lang3.StringUtils' | 11 |
| Rule | Violation | Line |
|---|---|---|
| DuplicateImports | Avoid duplicate imports such as 'com.fasterxml.jackson.core.JsonProcessingException' | 12 |
| DuplicateImports | Avoid duplicate imports such as 'com.fasterxml.jackson.databind.DeserializationFeature' | 13 |
| DuplicateImports | Avoid duplicate imports such as 'com.fasterxml.jackson.databind.JsonMappingException' | 14 |
| DuplicateImports | Avoid duplicate imports such as 'com.fasterxml.jackson.databind.JsonNode' | 15 |
| DuplicateImports | Avoid duplicate imports such as 'com.fasterxml.jackson.databind.ObjectMapper' | 16 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppLogService' | 11 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.commons.lang3.StringUtils' | 75 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.Collections' | 76 |
| UnusedImports | Avoid unused imports such as 'java.util.Iterator' | 79 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 1–325 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 44–325 |
| CommentRequired | Header comments are required | 3 | 44–325 |
| NPathComplexity | The method 'generateExport(int, String)' has an NPath complexity of 1728 | 3 | 56–173 |
| CyclomaticComplexity | The method 'generateExport(int, String)' has a cyclomatic complexity of 15. | 3 | 56–173 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 56–173 |
| NcssCount | The method 'generateExport(int, String)' has a NCSS line count of 60. | 3 | 56–173 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 59 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 99 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 123–142 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 130 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 163 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 171 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 211 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 214 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 215 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 218 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 236–265 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 286 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 3–85 |
| DataClass | The class 'Attribute' is suspected to be a Data Class (WOC=0.000%, NOPA=11, NOAM=22, WMC=22) | 3 | 3–85 |
| FieldNamingConventions | The field name 'certificateDate' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 4 |
| FieldNamingConventions | The field name 'lastUpdateClientCode' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 5 |
| FieldNamingConventions | The field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 6 |
| FieldNamingConventions | The field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 7 |
| FieldNamingConventions | The field name 'pivot' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 8 |
| FieldNamingConventions | The field name 'certifierCode' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 9 |
| FieldNamingConventions | The field name 'certificateExpirationDate' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 10 |
| FieldNamingConventions | The field name 'type' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 11 |
| FieldNamingConventions | The field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 12 |
| FieldNamingConventions | The field name 'certifierName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 13 |
| FieldNamingConventions | The field name 'key' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 14 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 5–154 |
| DataClass | The class 'ElasticsearchResponseJSON' is suspected to be a Data Class (WOC=16.667%, NOPA=0, NOAM=10, WMC=12) | 3 | 5–154 |
| FieldNamingConventions | The field name 'took' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 6 |
| FieldNamingConventions | The field name 'timedOut' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 7 |
| FieldNamingConventions | The field name 'shards' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 8 |
| FieldNamingConventions | The field name 'hits' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 9 |
| FieldNamingConventions | The field name '_scroll_id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 10 |
| FieldNamingConventions | The field name 'pit_id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 11 |
| CommentRequired | Public method and constructor comments are required | 3 | 13–15 |
| MethodNamingConventions | The instance method name 'get_scroll_id' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 13–15 |
| CommentRequired | Public method and constructor comments are required | 3 | 17–19 |
| MethodNamingConventions | The instance method name 'set_scroll_id' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 17–19 |
| FormalParameterNamingConventions | The method parameter name '_scroll_id' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 17 |
| MethodNamingConventions | The instance method name 'getPit_id' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 53–55 |
| FormalParameterNamingConventions | The method parameter name 'pit_id' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 57 |
| MethodNamingConventions | The instance method name 'setPit_id' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 57–59 |
| CommentRequired | Header comments are required | 3 | 61–63 |
| ShortClassName | Avoid short class names like Hits | 4 | 65–84 |
| DataClass | The class 'Hits' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=4) | 3 | 65–84 |
| CommentRequired | Header comments are required | 3 | 65–84 |
| FieldNamingConventions | The field name 'total' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 66 |
| AvoidFieldNameMatchingTypeName | It is somewhat confusing to have a field name matching the declaring class name | 3 | 67 |
| FieldNamingConventions | The field name 'hits' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 67 |
| CommentRequired | Header comments are required | 3 | 86–90 |
| UnusedPrivateField | Avoid unused private fields such as 'value'. | 3 | 87 |
| FieldNamingConventions | The field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 87 |
| FieldNamingConventions | The field name 'relation' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 88 |
| UnusedPrivateField | Avoid unused private fields such as 'relation'. | 3 | 88 |
| ShortClassName | Avoid short class names like Hit | 4 | 92–125 |
| CommentRequired | Header comments are required | 3 | 92–125 |
| UnusedPrivateField | Avoid unused private fields such as '_index'. | 3 | 93 |
| UnusedPrivateField | Avoid unused private fields such as '_type'. | 3 | 94 |
| FieldNamingConventions | The field name 'sort' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 98 |
| MethodNamingConventions | The instance method name 'get_source' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 100–102 |
| CommentRequired | Public method and constructor comments are required | 3 | 100–102 |
| CommentRequired | Public method and constructor comments are required | 3 | 103–105 |
| MethodNamingConventions | The instance method name 'set_source' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 103–105 |
| FormalParameterNamingConventions | The method parameter name '_source' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 103 |
| CommentRequired | Public method and constructor comments are required | 3 | 106–108 |
| MethodNamingConventions | The instance method name 'get_score' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 106–108 |
| MethodNamingConventions | The instance method name 'set_score' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 109–111 |
| FormalParameterNamingConventions | The method parameter name '_score' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 109 |
| CommentRequired | Public method and constructor comments are required | 3 | 109–111 |
| CommentRequired | Public method and constructor comments are required | 3 | 112–114 |
| MethodNamingConventions | The instance method name 'get_id' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 112–114 |
| FormalParameterNamingConventions | The method parameter name '_id' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 115 |
| CommentRequired | Public method and constructor comments are required | 3 | 115–117 |
| MethodNamingConventions | The instance method name 'set_id' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 115–117 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 121 |
| ArrayIsStoredDirectly | The user-supplied array 'sort' is stored directly. | 3 | 121 |
| CommentRequired | Header comments are required | 3 | 127–152 |
| DataClass | The class 'Source' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=6, WMC=6) | 3 | 127–152 |
| FieldNamingConventions | The field name 'customerId' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 128 |
| FieldNamingConventions | The field name 'connectionId' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 129 |
| FieldNamingConventions | The field name 'attributes' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 130 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 190–192 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 51–317 |
| UnusedPrivateField | Avoid unused private fields such as 'SQL_QUERY_SELECTALL_ID_BY_ID_PROFIL'. | 3 | 62 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 233–235 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 106 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 46–130 |
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 106 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 106 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidDuplicateLiterals | The String literal "SELECT " appears 4 times in this file; the first occurrence is on line 55 | 3 | 55 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 75 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 240–242 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 146–148 |
| CommentRequired | Public method and constructor comments are required | 3 | 150–152 |
| CommentRequired | Public method and constructor comments are required | 3 | 154–156 |
| FormalParameterNamingConventions | The method parameter name '_bAutoExtract' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 158 |
| CommentRequired | Public method and constructor comments are required | 3 | 158–160 |
| CommentRequired | Public method and constructor comments are required | 3 | 162–164 |
| CommentRequired | Public method and constructor comments are required | 3 | 166–168 |
| CommentRequired | Public method and constructor comments are required | 3 | 170–172 |
| FormalParameterNamingConventions | The method parameter name '_strPassword' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 174 |
| CommentRequired | Public method and constructor comments are required | 3 | 174–176 |
| CommentRequired | Public method and constructor comments are required | 3 | 178–180 |
| CommentRequired | Public method and constructor comments are required | 3 | 182–184 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.Date' | 4 | 5 |
| CommentRequired | Header comments are required | 3 | 15–40 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 25–27 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 25–27 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.mail.MailService' | 4 | 9 |
| UnusedImports | Avoid unused imports such as 'org.apache.commons.lang3.StringUtils' | 4 | 11 |
| CommentRequired | Header comments are required | 3 | 13–51 |
| ShortVariable | Avoid variables with short names like sb | 3 | 21 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 35 characters appended. | 3 | 21 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 26 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 37 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 3–23 |
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 3–23 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DuplicateImports | Avoid duplicate imports such as 'com.fasterxml.jackson.core.JsonProcessingException' | 4 | 12 |
| DuplicateImports | Avoid duplicate imports such as 'com.fasterxml.jackson.databind.DeserializationFeature' | 4 | 13 |
| DuplicateImports | Avoid duplicate imports such as 'com.fasterxml.jackson.databind.JsonMappingException' | 4 | 14 |
| DuplicateImports | Avoid duplicate imports such as 'com.fasterxml.jackson.databind.JsonNode' | 4 | 15 |
| DuplicateImports | Avoid duplicate imports such as 'com.fasterxml.jackson.databind.ObjectMapper' | 4 | 16 |
| CommentRequired | Header comments are required | 3 | 23–196 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 23–196 |
| AvoidDuplicateLiterals | The String literal " {\n" appears 4 times in this file; the first occurrence is on line 59 | 3 | 59 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 81 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 150 |
| IdenticalCatchBranches | 'catch' branch identical to 'JsonMappingException' branch | 3 | 184–187 |
| IdenticalCatchBranches | 'catch' branch identical to 'JsonMappingException' branch | 3 | 188–191 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppLogService' | 4 | 11 |
| CommentRequired | Header comments are required | 3 | 17–176 |
| ImmutableField | Private field '_profile' could be made final; it is only initialized in the declaration or constructor. | 3 | 23 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 28–31 |
| AvoidFileStream | Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter | 1 | 39 |
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 3 | 128–131 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 3 | 137–140 |
| MissingOverride | The method 'getCertification()' is missing an @Override annotation. | 3 | 146–149 |
| MissingOverride | The method 'getFileName()' is missing an @Override annotation. | 3 | 155–158 |
| MissingOverride | The method 'isAutoExtract()' is missing an @Override annotation. | 3 | 163–166 |
| MissingOverride | The method 'getPassword()' is missing an @Override annotation. | 3 | 171–174 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 5–26 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 14–58 |
| ShortVariable | Avoid variables with short names like rt | 3 | 24 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 18–38 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 20–22 |
| CommentRequired | Public method and constructor comments are required | 3 | 20–22 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 20–22 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 27–32 |
| CommentRequired | Public method and constructor comments are required | 3 | 27–36 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 8–13 |
| CommentRequired | Public method and constructor comments are required | 3 | 9–12 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like e | 3 | 26 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like e | 3 | 25 |
| ShortVariable | Avoid variables with short names like e | 3 | 35 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 15–44 |
| ShortVariable | Avoid variables with short names like e | 3 | 20 |
| ShortVariable | Avoid variables with short names like e | 3 | 29 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like e | 3 | 24 |
| ShortVariable | Avoid variables with short names like e | 3 | 34 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 21–80 |
| FieldNamingConventions | The field name 'exportModel' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 24 |
| CommentRequired | Public method and constructor comments are required | 3 | 26–30 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_storageDir' could be made final; it is only initialized in the declaration or constructor. | 3 | 47 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 238 |
| CommentRequired | Public method and constructor comments are required | 3 | 278–281 |
| CommentRequired | Public method and constructor comments are required | 3 | 283–286 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 50–110 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 109 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 109 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 36–416 |
| UnusedImports | Avoid unused imports such as 'org.apache.commons.lang3.StringUtils' | 4 | 75 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 81–416 |
| UnusedPrivateField | Avoid unused private fields such as 'ACTION_EXTRACTION_DATA'. | 3 | 119 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 141 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 152 |
| UnusedLocalVariable | Avoid unused local variables such as 'profilsReferenceList'. | 3 | 152 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 166–175 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 167–175 |
| AvoidDuplicateLiterals | The String literal "You don't have the right to modify this export profile." appears 6 times in this file; the first occurrence is on line 196 | 3 | 196 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 199 |
| UnusedLocalVariable | Avoid unused local variables such as 'lstProfils'. | 3 | 204 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 204 |
| SystemPrintln | System.out.println is used | 2 | 215 |
| SystemPrintln | System.out.println is used | 2 | 224 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 229 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 286 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 310 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 368 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 36–560 |
| UnusedImports | Avoid unused imports such as 'java.util.Collections' | 4 | 76 |
| UnusedImports | Avoid unused imports such as 'java.util.Iterator' | 4 | 79 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 90–560 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 153 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 160 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 164 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 164 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 183–192 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 184–192 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 215 |
| SystemPrintln | System.out.println is used | 2 | 228 |
| SystemPrintln | System.out.println is used | 2 | 232 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 239 |
| AvoidDuplicateLiterals | The String literal "You don't have the right to modify this export profile." appears 4 times in this file; the first occurrence is on line 318 | 3 | 318 |
| SystemPrintln | System.out.println is used | 2 | 380 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 386 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 432–559 |
| CommentRequired | Public method and constructor comments are required | 3 | 432–559 |
