The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'X' doesn't match '_[a-z][a-zA-Z0-9]*' | 38 |
| FieldNamingConventions | The field name 'Y' doesn't match '_[a-z][a-zA-Z0-9]*' | 39 |
| FieldNamingConventions | The field name 'W' doesn't match '_[a-z][a-zA-Z0-9]*' | 40 |
| FieldNamingConventions | The field name 'H' doesn't match '_[a-z][a-zA-Z0-9]*' | 41 |
| FormalParameterNamingConventions | The method parameter name 'H' doesn't match '[a-z][a-zA-Z0-9]*' | 48 |
| FormalParameterNamingConventions | The method parameter name 'X' doesn't match '[a-z][a-zA-Z0-9]*' | 48 |
| FormalParameterNamingConventions | The method parameter name 'W' doesn't match '[a-z][a-zA-Z0-9]*' | 48 |
| FormalParameterNamingConventions | The method parameter name 'Y' doesn't match '[a-z][a-zA-Z0-9]*' | 48 |
| FormalParameterNamingConventions | The method parameter name 'X' doesn't match '[a-z][a-zA-Z0-9]*' | 76 |
| FormalParameterNamingConventions | The method parameter name 'Y' doesn't match '[a-z][a-zA-Z0-9]*' | 81 |
| FormalParameterNamingConventions | The method parameter name 'W' doesn't match '[a-z][a-zA-Z0-9]*' | 86 |
| FormalParameterNamingConventions | The method parameter name 'H' doesn't match '[a-z][a-zA-Z0-9]*' | 91 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 61–335 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'KibanaConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 39–80 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 579 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'strObject' | 695 |
| Rule | Violation | Line |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 162 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 189 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 257 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'DashboardReference' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=6, WMC=8) | 38–87 |
| CommentRequired | Header comments are required | 38–87 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 40 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _id field if you want a default access modifier | 40 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 41 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _name field if you want a default access modifier | 41 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 42 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _type field if you want a default access modifier | 42 |
| CommentRequired | Public method and constructor comments are required | 44–49 |
| CommentRequired | Public method and constructor comments are required | 51–56 |
| ShortVariable | Avoid variables with short names like id | 63 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'Gird' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=8, WMC=10) | 36–95 |
| CommentRequired | Header comments are required | 36–95 |
| ShortVariable | Avoid variables with short names like X | 38 |
| ShortVariable | Avoid variables with short names like Y | 39 |
| ShortVariable | Avoid variables with short names like W | 40 |
| ShortVariable | Avoid variables with short names like H | 41 |
| CommentRequired | Public method and constructor comments are required | 43–46 |
| CommentRequired | Public method and constructor comments are required | 48–54 |
| ShortVariable | Avoid variables with short names like Y | 48 |
| ShortVariable | Avoid variables with short names like X | 48 |
| ShortVariable | Avoid variables with short names like W | 48 |
| ShortVariable | Avoid variables with short names like H | 48 |
| ShortVariable | Avoid variables with short names like X | 76 |
| ShortVariable | Avoid variables with short names like Y | 81 |
| ShortVariable | Avoid variables with short names like W | 86 |
| ShortVariable | Avoid variables with short names like H | 91 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like _x | 45 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_x' | 45 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_y' | 46 |
| ShortVariable | Avoid variables with short names like _y | 46 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_preX' | 47 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_preY' | 48 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 57 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 47–93 |
| ShortVariable | Avoid variables with short names like rt | 58 |
| ShortVariable | Avoid variables with short names like p | 64 |
| Rule | Violation | Line |
|---|---|---|
| RedundantFieldInitializer | Avoid using redundant field initializer for '_instance' | 90 |
| CommentRequired | Public method and constructor comments are required | 98–105 |
| NonThreadSafeSingleton | Singleton is not thread safe | 100–103 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 118 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 120 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 129–136 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 143 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 147 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 162 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 310 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 39–51 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 39–51 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 61–711 |
| GodClass | Possible God Class (WMC=51, ATFD=33, TCC=32.900%) | 61–711 |
| 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. | 62–711 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 62–711 |
| UnusedPrivateField | Avoid unused private fields such as 'NOT_FOUND'. | 63 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the SavedObjectService constructor if you want a default access modifier | 87–89 |
| UncommentedEmptyConstructor | Document empty constructor | 87–89 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 101 |
| AvoidDuplicateLiterals | The String literal "kbn-xsrf" appears 12 times in this file; the first occurrence is on line 102 | 102 |
| AvoidDuplicateLiterals | The String literal "true" appears 12 times in this file; the first occurrence is on line 102 | 102 |
| AvoidDuplicateLiterals | The String literal "/s/" appears 11 times in this file; the first occurrence is on line 106 | 106 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 110–117 |
| CommentRequired | Public method and constructor comments are required | 125–150 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 129 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 164 |
| AvoidDuplicateLiterals | The String literal "Unable to connect to Elasticsearch / Kibana servers" appears 5 times in this file; the first occurrence is on line 175 | 175 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 191 |
| SimplifyBooleanReturns | Avoid unnecessary if..then..else statements when returning booleans | 196–199 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 222 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 247 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 291 |
| AvoidDuplicateLiterals | The String literal "version" appears 5 times in this file; the first occurrence is on line 387 | 387 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'doDeleteSavedObject': private methods cannot be overridden | 445–459 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 447 |
| AvoidDuplicateLiterals | The String literal "Unable to connect to Kibana server" appears 6 times in this file; the first occurrence is on line 457 | 457 |
| UnusedPrivateMethod | Avoid unused private methods such as 'getIndexPatternFieldsMapping(String)'. | 468 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 475 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 531 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'doPost': private methods cannot be overridden | 611–626 |
| UnusedPrivateMethod | Avoid unused private methods such as 'doPost(JSONObject,String)'. | 611 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 614 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'doPostSavedObject': private methods cannot be overridden | 634–655 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 636 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 665 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'doPostSavedStringObject': private methods cannot be overridden | 695–710 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 697 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 40–154 |
| 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. | 41–154 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 48 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 84–85 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 104–105 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 134 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 45–105 |
| UncommentedEmptyMethodBody | Document empty method body | 83–84 |
| UncommentedEmptyMethodBody | Document empty method body | 91–92 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 105 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 137 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 105 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 179–190 |
| Rule | Violation | Line |
|---|---|---|
| SingularField | Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. | 68 |
| AddEmptyString | Do not add empty strings | 99 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.elasticdata.service.DataSourceService' | 36 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like Gird | 36–95 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.stream.Collectors' | 40 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.elasticdata.service.DataSourceService' | 4 | 36 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 162 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 189 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 257 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'DashboardReference' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=6, WMC=8) | 3 | 38–87 |
| CommentRequired | Header comments are required | 3 | 38–87 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 40 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _id field if you want a default access modifier | 3 | 40 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 41 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _name field if you want a default access modifier | 3 | 41 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 42 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _type field if you want a default access modifier | 3 | 42 |
| CommentRequired | Public method and constructor comments are required | 3 | 44–49 |
| CommentRequired | Public method and constructor comments are required | 3 | 51–56 |
| ShortVariable | Avoid variables with short names like id | 3 | 63 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'Gird' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=8, WMC=10) | 3 | 36–95 |
| ShortClassName | Avoid short class names like Gird | 4 | 36–95 |
| CommentRequired | Header comments are required | 3 | 36–95 |
| FieldNamingConventions | The field name 'X' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 38 |
| ShortVariable | Avoid variables with short names like X | 3 | 38 |
| ShortVariable | Avoid variables with short names like Y | 3 | 39 |
| FieldNamingConventions | The field name 'Y' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 39 |
| ShortVariable | Avoid variables with short names like W | 3 | 40 |
| FieldNamingConventions | The field name 'W' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 40 |
| ShortVariable | Avoid variables with short names like H | 3 | 41 |
| FieldNamingConventions | The field name 'H' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 41 |
| CommentRequired | Public method and constructor comments are required | 3 | 43–46 |
| FormalParameterNamingConventions | The method parameter name 'H' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 48 |
| CommentRequired | Public method and constructor comments are required | 3 | 48–54 |
| ShortVariable | Avoid variables with short names like Y | 3 | 48 |
| FormalParameterNamingConventions | The method parameter name 'X' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 48 |
| FormalParameterNamingConventions | The method parameter name 'W' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 48 |
| ShortVariable | Avoid variables with short names like X | 3 | 48 |
| ShortVariable | Avoid variables with short names like W | 3 | 48 |
| FormalParameterNamingConventions | The method parameter name 'Y' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 48 |
| ShortVariable | Avoid variables with short names like H | 3 | 48 |
| ShortVariable | Avoid variables with short names like X | 3 | 76 |
| FormalParameterNamingConventions | The method parameter name 'X' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 76 |
| ShortVariable | Avoid variables with short names like Y | 3 | 81 |
| FormalParameterNamingConventions | The method parameter name 'Y' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 81 |
| ShortVariable | Avoid variables with short names like W | 3 | 86 |
| FormalParameterNamingConventions | The method parameter name 'W' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 86 |
| FormalParameterNamingConventions | The method parameter name 'H' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 91 |
| ShortVariable | Avoid variables with short names like H | 3 | 91 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like _x | 3 | 45 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_x' | 3 | 45 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_y' | 3 | 46 |
| ShortVariable | Avoid variables with short names like _y | 3 | 46 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_preX' | 3 | 47 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_preY' | 3 | 48 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 57 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 47–93 |
| ShortVariable | Avoid variables with short names like rt | 3 | 58 |
| ShortVariable | Avoid variables with short names like p | 3 | 64 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 61–335 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_instance' | 3 | 90 |
| CommentRequired | Public method and constructor comments are required | 3 | 98–105 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 100–103 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 118 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 120 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 129–136 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 3 | 143 |
| PreserveStackTrace | New exception is thrown in catch block, original stack trace may be lost | 3 | 147 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 162 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 310 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 39–51 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 39–51 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 61–711 |
| GodClass | Possible God Class (WMC=51, ATFD=33, TCC=32.900%) | 3 | 61–711 |
| 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 | 62–711 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 62–711 |
| UnusedPrivateField | Avoid unused private fields such as 'NOT_FOUND'. | 3 | 63 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the SavedObjectService constructor if you want a default access modifier | 3 | 87–89 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 87–89 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 101 |
| AvoidDuplicateLiterals | The String literal "kbn-xsrf" appears 12 times in this file; the first occurrence is on line 102 | 3 | 102 |
| AvoidDuplicateLiterals | The String literal "true" appears 12 times in this file; the first occurrence is on line 102 | 3 | 102 |
| AvoidDuplicateLiterals | The String literal "/s/" appears 11 times in this file; the first occurrence is on line 106 | 3 | 106 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 110–117 |
| CommentRequired | Public method and constructor comments are required | 3 | 125–150 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 129 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 164 |
| AvoidDuplicateLiterals | The String literal "Unable to connect to Elasticsearch / Kibana servers" appears 5 times in this file; the first occurrence is on line 175 | 3 | 175 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 191 |
| SimplifyBooleanReturns | Avoid unnecessary if..then..else statements when returning booleans | 3 | 196–199 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 222 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 247 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 291 |
| AvoidDuplicateLiterals | The String literal "version" appears 5 times in this file; the first occurrence is on line 387 | 3 | 387 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'doDeleteSavedObject': private methods cannot be overridden | 3 | 445–459 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 447 |
| AvoidDuplicateLiterals | The String literal "Unable to connect to Kibana server" appears 6 times in this file; the first occurrence is on line 457 | 3 | 457 |
| UnusedPrivateMethod | Avoid unused private methods such as 'getIndexPatternFieldsMapping(String)'. | 3 | 468 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 475 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 2 | 579 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 2 | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 2 | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 2 | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 2 | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 2 | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 2 | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 2 | 592 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'jsString' | 2 | 592 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'doPost': private methods cannot be overridden | 3 | 611–626 |
| UnusedPrivateMethod | Avoid unused private methods such as 'doPost(JSONObject,String)'. | 3 | 611 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 614 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'doPostSavedObject': private methods cannot be overridden | 3 | 634–655 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 636 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 665 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'strObject' | 2 | 695 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'doPostSavedStringObject': private methods cannot be overridden | 3 | 695–710 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 697 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 40–154 |
| 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 | 41–154 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 48 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 84–85 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 104–105 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 134 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 45–105 |
| UncommentedEmptyMethodBody | Document empty method body | 3 | 83–84 |
| UncommentedEmptyMethodBody | Document empty method body | 3 | 91–92 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'KibanaConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 39–80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 105 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 137 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.stream.Collectors' | 4 | 40 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 105 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 179–190 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SingularField | Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. | 3 | 68 |
| AddEmptyString | Do not add empty strings | 3 | 99 |
