Fork me on GitHub

Résultats PMD

Le document suivant contient les résultats de PMD 6.13.0.

Violations By Priority

Priority 1

fr/paris/lutece/plugins/consultation/business/Consultation.java

Rule Violation Ligne
FormalParameterNamingConventions The method parameter name '_nIdUnit' doesn't match '[a-z][a-zA-Z0-9]*' 471

fr/paris/lutece/plugins/consultation/business/ConsultationDAO.java

Rule Violation Ligne
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 382

fr/paris/lutece/plugins/consultation/business/ConsultationDTO.java

Rule Violation Ligne
FormalParameterNamingConventions The method parameter name '_nIdUnit' doesn't match '[a-z][a-zA-Z0-9]*' 536

fr/paris/lutece/plugins/consultation/business/ConsultationObjectMapper.java

Rule Violation Ligne
FormalParameterNamingConventions The method parameter name '_strPlace' doesn't match '[a-z][a-zA-Z0-9]*' 106
FormalParameterNamingConventions The method parameter name '_strTheme' doesn't match '[a-z][a-zA-Z0-9]*' 116
FormalParameterNamingConventions The method parameter name '_strImgRessource' doesn't match '[a-z][a-zA-Z0-9]*' 128
FormalParameterNamingConventions The method parameter name '_nId' doesn't match '[a-z][a-zA-Z0-9]*' 145
FormalParameterNamingConventions The method parameter name '_nIdForms' doesn't match '[a-z][a-zA-Z0-9]*' 145

fr/paris/lutece/plugins/consultation/business/IConsultationDAO.java

Rule Violation Ligne
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 138
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 149
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 160

fr/paris/lutece/plugins/consultation/service/ConsultationService.java

Rule Violation Ligne
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 44
FieldNamingConventions The field name 'consultationMapper' doesn't match '_[a-z][a-zA-Z0-9]*' 45
FormalParameterNamingConventions The method parameter name '_consultationDto' doesn't match '[a-z][a-zA-Z0-9]*' 133
FormalParameterNamingConventions The method parameter name '_consultationDto' doesn't match '[a-z][a-zA-Z0-9]*' 176
FormalParameterNamingConventions The method parameter name '_consultationDto' doesn't match '[a-z][a-zA-Z0-9]*' 200

fr/paris/lutece/plugins/consultation/util/ConsultationUtil.java

Rule Violation Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 15143

fr/paris/lutece/plugins/consultation/web/ConsultationJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'nIdWorkflow' doesn't match '_[a-z][a-zA-Z0-9]*' 201
FieldNamingConventions The field name 'strAscSort' doesn't match '_[a-z][a-zA-Z0-9]*' 207
FieldNamingConventions The field name 'referenceList' doesn't match '_[a-z][a-zA-Z0-9]*' 209
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 211
FieldNamingConventions The field name 'consultationMapper' doesn't match '_[a-z][a-zA-Z0-9]*' 212

Priority 3

fr/paris/lutece/plugins/consultation/business/Consultation.java

Rule Violation Ligne
ExcessivePublicCount This class has a bunch of public methods and attributes 34622
TooManyFields Too many fields 49622

fr/paris/lutece/plugins/consultation/business/ConsultationDAO.java

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 53489
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 330333
MissingOverride The method 'selectSortedConsultationsByIds(Plugin, List, ConsultationSort)' is missing an @Override annotation. 421487
CommentRequired Public method and constructor comments are required 421487
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 431434

fr/paris/lutece/plugins/consultation/business/ConsultationDTO.java

Rule Violation Ligne
ExcessivePublicCount This class has a bunch of public methods and attributes 34662
TooManyFields Too many fields 50662

fr/paris/lutece/plugins/consultation/business/ConsultationObjectMapper.java

Rule Violation Ligne
CommentRequired Header comments are required 21154
CommentRequired Public method and constructor comments are required 2327
CommentRequired Public method and constructor comments are required 3364
MissingOverride The method 'convert(MappingContext)' is missing an @Override annotation. 3364
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 45
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 50
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 51
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 58
MissingOverride The method 'convert(MappingContext)' is missing an @Override annotation. 73101
CommentRequired Public method and constructor comments are required 73101
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 85

fr/paris/lutece/plugins/consultation/business/IConsultationDAO.java

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 48161

fr/paris/lutece/plugins/consultation/business/Status.java

Rule Violation Ligne
CommentRequired Enum comments are required 754
CommentRequired Public method and constructor comments are required 2427
CommentRequired Public method and constructor comments are required 2932
CommentRequired Public method and constructor comments are required 3453

fr/paris/lutece/plugins/consultation/filter/ConsultationFilter.java

Rule Violation Ligne
CommentRequired Header comments are required 3655
CommentRequired Public method and constructor comments are required 5053

fr/paris/lutece/plugins/consultation/service/AccessControlType.java

Rule Violation Ligne
CommentRequired Header comments are required 2798
UncommentedEmptyMethodBody Document empty method body 7476
UncommentedEmptyMethodBody Document empty method body 7981

fr/paris/lutece/plugins/consultation/service/ConsultationIdService.java

Rule Violation Ligne
CommentRequired Header comments are required 1794
CommentRequired Public method and constructor comments are required 3034

fr/paris/lutece/plugins/consultation/service/ConsultationPlugin.java

Rule Violation Ligne
CommentRequired Header comments are required 925

fr/paris/lutece/plugins/consultation/service/ConsultationService.java

Rule Violation Ligne
CommentRequired Enum comments are required 35272
CommentRequired Public method and constructor comments are required 4799
CyclomaticComplexity The method 'deleteConsultation(int)' has a cyclomatic complexity of 14. 4799
ConfusingTernary Avoid if (x != y) ..; else ..; 6172
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 90
CommentRequired Public method and constructor comments are required 101131
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 125
CommentRequired Public method and constructor comments are required 133135
CommentRequired Public method and constructor comments are required 137174
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 168
CommentRequired Public method and constructor comments are required 176178
CommentRequired Public method and constructor comments are required 180198
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 192
CommentRequired Public method and constructor comments are required 200204
CommentRequired Public method and constructor comments are required 206218

fr/paris/lutece/plugins/consultation/service/forms/search/FormsLuceneReadIndex.java

Rule Violation Ligne
CommentRequired Enum comments are required 2578
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 45
ShortVariable Avoid variables with short names like ir 72

fr/paris/lutece/plugins/consultation/service/resource/ConsultationExtendableResourceService.java

Rule Violation Ligne
CommentRequired Header comments are required 1866

fr/paris/lutece/plugins/consultation/sort/ConsultationSort.java

Rule Violation Ligne
CommentRequired Header comments are required 36105
CommentRequired Public method and constructor comments are required 9294
CommentRequired Public method and constructor comments are required 101104

fr/paris/lutece/plugins/consultation/util/ConsultationException.java

Rule Violation Ligne
CommentRequired Header comments are required 322

fr/paris/lutece/plugins/consultation/util/ConsultationUtil.java

Rule Violation Ligne
CommentRequired Header comments are required 15143
CommentRequired Public method and constructor comments are required 5664
CommentRequired Public method and constructor comments are required 6568
CommentRequired Public method and constructor comments are required 7076
CommentRequired Public method and constructor comments are required 7781
CommentRequired Public method and constructor comments are required 8285
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 123
AssignmentInOperand Avoid assignments in operands 123

fr/paris/lutece/plugins/consultation/web/AbstractManageConsultationJspBean.java

Rule Violation Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 51118
DefaultPackage Use explicit scoping instead of the default package private level 117
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier 117

fr/paris/lutece/plugins/consultation/web/ConsultationJspBean.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 35879
ImmutableField Private field 'nIdWorkflow' could be made final; it is only initialized in the declaration or constructor. 201
SingularField Perhaps '_strSortedAttributeName' could be replaced by a local variable. 205
SingularField Perhaps 'strAscSort' could be replaced by a local variable. 207
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 211
ImmutableField Private field 'consultationMapper' could be made final; it is only initialized in the declaration or constructor. 212
CyclomaticComplexity The method 'getManageConsultations(HttpServletRequest)' has a cyclomatic complexity of 15. 222281
NPathComplexity The method 'getManageConsultations(HttpServletRequest)' has an NPath complexity of 720 222281
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 224
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getLibelleRefStatus method if you want a default access modifier 283288
DefaultPackage Use explicit scoping instead of the default package private level 283288
DefaultPackage Use explicit scoping instead of the default package private level 296316
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier 297316
NPathComplexity The method 'doCreateConsultation(HttpServletRequest)' has an NPath complexity of 432 366422
CyclomaticComplexity The method 'doCreateConsultation(HttpServletRequest)' has a cyclomatic complexity of 12. 366422
ConfusingTernary Avoid if (x != y) ..; else ..; 410417
NPathComplexity The method 'doModifyConsultation(HttpServletRequest)' has an NPath complexity of 1440 542601
CyclomaticComplexity The method 'doModifyConsultation(HttpServletRequest)' has a cyclomatic complexity of 16. 542601
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 579
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 662
ConfusingTernary Avoid if (x != y) ..; else ..; 670697
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 678
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 710
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 827
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 874

fr/paris/lutece/plugins/consultation/web/ConsultationPageInclude.java

Rule Violation Ligne
CommentRequired Header comments are required 2050
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3844

fr/paris/lutece/plugins/consultation/web/ConsultationXPage.java

Rule Violation Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 71233
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 128
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 161
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 210

Priority 4

fr/paris/lutece/plugins/consultation/service/ConsultationIdService.java

Rule Violation Ligne
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.accesscontrol.util.AccessControlUtils' 6

fr/paris/lutece/plugins/consultation/web/ConsultationJspBean.java

Rule Violation Ligne
UnusedImports Avoid unused imports such as 'fr.paris.lutece.portal.service.message.SiteMessage' 39
UnusedImports Avoid unused imports such as 'fr.paris.lutece.portal.service.message.SiteMessageService' 41

fr/paris/lutece/plugins/consultation/web/ConsultationPageInclude.java

Rule Violation Ligne
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.consultation.business.Consultation' 8
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.consultation.business.ConsultationHome' 10

Fichiers

fr/paris/lutece/plugins/consultation/business/Consultation.java

Rule Violation Priority Ligne
ExcessivePublicCount This class has a bunch of public methods and attributes 3 34622
TooManyFields Too many fields 3 49622
FormalParameterNamingConventions The method parameter name '_nIdUnit' doesn't match '[a-z][a-zA-Z0-9]*' 1 471

fr/paris/lutece/plugins/consultation/business/ConsultationDAO.java

Rule Violation Priority Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 53489
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 330333
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 1 382
MissingOverride The method 'selectSortedConsultationsByIds(Plugin, List, ConsultationSort)' is missing an @Override annotation. 3 421487
CommentRequired Public method and constructor comments are required 3 421487
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 431434

fr/paris/lutece/plugins/consultation/business/ConsultationDTO.java

Rule Violation Priority Ligne
ExcessivePublicCount This class has a bunch of public methods and attributes 3 34662
TooManyFields Too many fields 3 50662
FormalParameterNamingConventions The method parameter name '_nIdUnit' doesn't match '[a-z][a-zA-Z0-9]*' 1 536

fr/paris/lutece/plugins/consultation/business/ConsultationObjectMapper.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 21154
CommentRequired Public method and constructor comments are required 3 2327
CommentRequired Public method and constructor comments are required 3 3364
MissingOverride The method 'convert(MappingContext)' is missing an @Override annotation. 3 3364
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 45
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 50
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 51
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 58
MissingOverride The method 'convert(MappingContext)' is missing an @Override annotation. 3 73101
CommentRequired Public method and constructor comments are required 3 73101
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 85
FormalParameterNamingConventions The method parameter name '_strPlace' doesn't match '[a-z][a-zA-Z0-9]*' 1 106
FormalParameterNamingConventions The method parameter name '_strTheme' doesn't match '[a-z][a-zA-Z0-9]*' 1 116
FormalParameterNamingConventions The method parameter name '_strImgRessource' doesn't match '[a-z][a-zA-Z0-9]*' 1 128
FormalParameterNamingConventions The method parameter name '_nId' doesn't match '[a-z][a-zA-Z0-9]*' 1 145
FormalParameterNamingConventions The method parameter name '_nIdForms' doesn't match '[a-z][a-zA-Z0-9]*' 1 145

fr/paris/lutece/plugins/consultation/business/IConsultationDAO.java

Rule Violation Priority Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 48161
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 1 138
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 1 149
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 1 160

fr/paris/lutece/plugins/consultation/business/Status.java

Rule Violation Priority Ligne
CommentRequired Enum comments are required 3 754
CommentRequired Public method and constructor comments are required 3 2427
CommentRequired Public method and constructor comments are required 3 2932
CommentRequired Public method and constructor comments are required 3 3453

fr/paris/lutece/plugins/consultation/filter/ConsultationFilter.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 3655
CommentRequired Public method and constructor comments are required 3 5053

fr/paris/lutece/plugins/consultation/service/AccessControlType.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 2798
UncommentedEmptyMethodBody Document empty method body 3 7476
UncommentedEmptyMethodBody Document empty method body 3 7981

fr/paris/lutece/plugins/consultation/service/ConsultationIdService.java

Rule Violation Priority Ligne
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.accesscontrol.util.AccessControlUtils' 4 6
CommentRequired Header comments are required 3 1794
CommentRequired Public method and constructor comments are required 3 3034

fr/paris/lutece/plugins/consultation/service/ConsultationPlugin.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 925

fr/paris/lutece/plugins/consultation/service/ConsultationService.java

Rule Violation Priority Ligne
CommentRequired Enum comments are required 3 35272
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 44
FieldNamingConventions The field name 'consultationMapper' doesn't match '_[a-z][a-zA-Z0-9]*' 1 45
CommentRequired Public method and constructor comments are required 3 4799
CyclomaticComplexity The method 'deleteConsultation(int)' has a cyclomatic complexity of 14. 3 4799
ConfusingTernary Avoid if (x != y) ..; else ..; 3 6172
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 90
CommentRequired Public method and constructor comments are required 3 101131
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 125
CommentRequired Public method and constructor comments are required 3 133135
FormalParameterNamingConventions The method parameter name '_consultationDto' doesn't match '[a-z][a-zA-Z0-9]*' 1 133
CommentRequired Public method and constructor comments are required 3 137174
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 168
FormalParameterNamingConventions The method parameter name '_consultationDto' doesn't match '[a-z][a-zA-Z0-9]*' 1 176
CommentRequired Public method and constructor comments are required 3 176178
CommentRequired Public method and constructor comments are required 3 180198
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 192
FormalParameterNamingConventions The method parameter name '_consultationDto' doesn't match '[a-z][a-zA-Z0-9]*' 1 200
CommentRequired Public method and constructor comments are required 3 200204
CommentRequired Public method and constructor comments are required 3 206218

fr/paris/lutece/plugins/consultation/service/forms/search/FormsLuceneReadIndex.java

Rule Violation Priority Ligne
CommentRequired Enum comments are required 3 2578
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 45
ShortVariable Avoid variables with short names like ir 3 72

fr/paris/lutece/plugins/consultation/service/resource/ConsultationExtendableResourceService.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 1866

fr/paris/lutece/plugins/consultation/sort/ConsultationSort.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 36105
CommentRequired Public method and constructor comments are required 3 9294
CommentRequired Public method and constructor comments are required 3 101104

fr/paris/lutece/plugins/consultation/util/ConsultationException.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 322

fr/paris/lutece/plugins/consultation/util/ConsultationUtil.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 15143
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 15143
CommentRequired Public method and constructor comments are required 3 5664
CommentRequired Public method and constructor comments are required 3 6568
CommentRequired Public method and constructor comments are required 3 7076
CommentRequired Public method and constructor comments are required 3 7781
CommentRequired Public method and constructor comments are required 3 8285
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 123
AssignmentInOperand Avoid assignments in operands 3 123

fr/paris/lutece/plugins/consultation/web/AbstractManageConsultationJspBean.java

Rule Violation Priority Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 51118
DefaultPackage Use explicit scoping instead of the default package private level 3 117
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier 3 117

fr/paris/lutece/plugins/consultation/web/ConsultationJspBean.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 35879
UnusedImports Avoid unused imports such as 'fr.paris.lutece.portal.service.message.SiteMessage' 4 39
UnusedImports Avoid unused imports such as 'fr.paris.lutece.portal.service.message.SiteMessageService' 4 41
FieldNamingConventions The field name 'nIdWorkflow' doesn't match '_[a-z][a-zA-Z0-9]*' 1 201
ImmutableField Private field 'nIdWorkflow' could be made final; it is only initialized in the declaration or constructor. 3 201
SingularField Perhaps '_strSortedAttributeName' could be replaced by a local variable. 3 205
FieldNamingConventions The field name 'strAscSort' doesn't match '_[a-z][a-zA-Z0-9]*' 1 207
SingularField Perhaps 'strAscSort' could be replaced by a local variable. 3 207
FieldNamingConventions The field name 'referenceList' doesn't match '_[a-z][a-zA-Z0-9]*' 1 209
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 211
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 211
ImmutableField Private field 'consultationMapper' could be made final; it is only initialized in the declaration or constructor. 3 212
FieldNamingConventions The field name 'consultationMapper' doesn't match '_[a-z][a-zA-Z0-9]*' 1 212
CyclomaticComplexity The method 'getManageConsultations(HttpServletRequest)' has a cyclomatic complexity of 15. 3 222281
NPathComplexity The method 'getManageConsultations(HttpServletRequest)' has an NPath complexity of 720 3 222281
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 224
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getLibelleRefStatus method if you want a default access modifier 3 283288
DefaultPackage Use explicit scoping instead of the default package private level 3 283288
DefaultPackage Use explicit scoping instead of the default package private level 3 296316
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier 3 297316
NPathComplexity The method 'doCreateConsultation(HttpServletRequest)' has an NPath complexity of 432 3 366422
CyclomaticComplexity The method 'doCreateConsultation(HttpServletRequest)' has a cyclomatic complexity of 12. 3 366422
ConfusingTernary Avoid if (x != y) ..; else ..; 3 410417
NPathComplexity The method 'doModifyConsultation(HttpServletRequest)' has an NPath complexity of 1440 3 542601
CyclomaticComplexity The method 'doModifyConsultation(HttpServletRequest)' has a cyclomatic complexity of 16. 3 542601
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 579
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 662
ConfusingTernary Avoid if (x != y) ..; else ..; 3 670697
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 678
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 710
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 827
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 874

fr/paris/lutece/plugins/consultation/web/ConsultationPageInclude.java

Rule Violation Priority Ligne
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.consultation.business.Consultation' 4 8
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.consultation.business.ConsultationHome' 4 10
CommentRequired Header comments are required 3 2050
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 3844

fr/paris/lutece/plugins/consultation/web/ConsultationXPage.java

Rule Violation Priority Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 71233
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 128
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 161
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 210