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/ctv/authentification/MokeLuteceAuthentication.java

Rule Violation Ligne
ReturnEmptyArrayRatherThanNull Return an empty array rather than null. 9699

fr/paris/lutece/plugins/ctv/bo/controller/conference/coordination/ConferenceCoordinationJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 80
FieldNamingConventions The static field name 'conferenceService' doesn't match '_[a-z][a-zA-Z0-9]*' 82
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 83
LocalVariableNamingConventions The local variable name 'TypeDemandeOccupationEtIntervention' doesn't match '[a-z][a-zA-Z0-9]*' 317
FormalParameterNamingConventions The method parameter name 'DemandeType' doesn't match '[a-z][a-zA-Z0-9]*' 377

fr/paris/lutece/plugins/ctv/bo/controller/conference/coordination/GestionConferenceCoordinationJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'conferenceService' doesn't match '_[a-z][a-zA-Z0-9]*' 67
FieldNamingConventions The constant name 'ctvConferenceCoordinationUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 68
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 71

fr/paris/lutece/plugins/ctv/bo/controller/conventions/GestionConventionJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'conventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 78
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 79
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 80
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 81
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 82
FieldNamingConventions The field name 'emailTask' doesn't match '_[a-z][a-zA-Z0-9]*' 84
FieldNamingConventions The field name 'validationConvention' doesn't match '_[a-z][a-zA-Z0-9]*' 86

fr/paris/lutece/plugins/ctv/bo/controller/demandes/AbstractCtvJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 148
FieldNamingConventions The field name 'ctvProfileService' doesn't match '_[a-z][a-zA-Z0-9]*' 149
FieldNamingConventions The constant name 'actionProfilStatutService' doesn't match '[A-Z][A-Z_0-9]*' 150
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 151
FieldNamingConventions The static field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 152
FieldNamingConventions The static field name 'alerteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 153
FieldNamingConventions The constant name 'infosSuperviseurService' doesn't match '[A-Z][A-Z_0-9]*' 154
LocalVariableNamingConventions The local variable name 'EmpriseId' doesn't match '[a-z][a-zA-Z0-9]*' 303

fr/paris/lutece/plugins/ctv/bo/controller/demandes/AbstractGestionConflitsJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 86
FieldNamingConventions The constant name 'conflitService' doesn't match '[A-Z][A-Z_0-9]*' 87
FieldNamingConventions The field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 88
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 97

fr/paris/lutece/plugins/ctv/bo/controller/demandes/AbstractSuperviseurJpsBean.java

Rule Violation Ligne
FieldNamingConventions The constant name 'emailTask' doesn't match '[A-Z][A-Z_0-9]*' 120
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 122
FieldNamingConventions The constant name 'infosSuperviseurService' doesn't match '[A-Z][A-Z_0-9]*' 124
FieldNamingConventions The field name 'saisineService' doesn't match '_[a-z][a-zA-Z0-9]*' 130
FieldNamingConventions The field name 'conflitService' doesn't match '_[a-z][a-zA-Z0-9]*' 131
FieldNamingConventions The constant name 'unitService' doesn't match '[A-Z][A-Z_0-9]*' 132
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 134

fr/paris/lutece/plugins/ctv/bo/controller/demandes/ConsultationCRVisiteJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'ctvUploadCompteRenduVisiteHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 67
FieldNamingConventions The static field name 'compteRenduVisiteService' doesn't match '_[a-z][a-zA-Z0-9]*' 70
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 72
FieldNamingConventions The static field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 74

fr/paris/lutece/plugins/ctv/bo/controller/demandes/CreationCRVisiteJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'ctvUploadCompteRenduVisiteHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 67
FieldNamingConventions The static field name 'compteRenduVisiteService' doesn't match '_[a-z][a-zA-Z0-9]*' 70
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 72
FieldNamingConventions The static field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 74
FieldNamingConventions The static field name 'alerteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 76

fr/paris/lutece/plugins/ctv/bo/controller/demandes/ProjetJspBean.java

Rule Violation Ligne
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 113
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 114
FieldNamingConventions The field name 'demandeFinTravauxService' doesn't match '_[a-z][a-zA-Z0-9]*' 115
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 116
FieldNamingConventions The field name 'prolongationService' doesn't match '_[a-z][a-zA-Z0-9]*' 118
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 119
FieldNamingConventions The static field name 'alerteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 120
FieldNamingConventions The constant name 'ctvDemandeUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 121
FieldNamingConventions The constant name 'ctvEmpriseUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 123
FieldNamingConventions The constant name 'emailTask' doesn't match '[A-Z][A-Z_0-9]*' 126

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/ChantierJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'chantierIntentionControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 65
FieldNamingConventions The field name 'formulaireIntentionService' doesn't match '_[a-z][a-zA-Z0-9]*' 66
FieldNamingConventions The field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 67

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/DemandeIntentionJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireIntentionService' doesn't match '_[a-z][a-zA-Z0-9]*' 121
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 123
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 125

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/DemandeurJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 76
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 77
FieldNamingConventions The field name 'formulaireIntentionService' doesn't match '_[a-z][a-zA-Z0-9]*' 78
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 81

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/EmpriseJspBean.java

Rule Violation Ligne
FieldNamingConventions The constant name 'uploadBaseDirectory' doesn't match '[A-Z][A-Z_0-9]*' 111
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 118
FieldNamingConventions The field name 'empriseCtrlService' doesn't match '_[a-z][a-zA-Z0-9]*' 119
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 120
FieldNamingConventions The field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 121
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 122
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 124
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 126
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 127
FieldNamingConventions The static field name 'alerteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 128

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/ChantierJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireInterventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 61
FieldNamingConventions The field name 'chantierControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 62

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/DemandeInterventionJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireInterventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 85
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 87
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 89

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/DemandeurJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 75
FieldNamingConventions The field name 'formulaireInterventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 77
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 80

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/DocumentsJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 82
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 85

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/EmpriseJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'empriseCtrlService' doesn't match '_[a-z][a-zA-Z0-9]*' 78
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 79
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 80
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 82
FieldNamingConventions The field name 'formulaireInterventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 84

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/ExecutantJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireInterventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 63
FieldNamingConventions The field name 'executantControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 65
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 70
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 71

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/GestionConflitsJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 58

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/SuperviseurJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 71
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 73
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 83
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 84

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/ValidationJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 102
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 103
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 104
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 106
FieldNamingConventions The field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 107
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 108
FieldNamingConventions The field name 'saisineService' doesn't match '_[a-z][a-zA-Z0-9]*' 109
FieldNamingConventions The field name 'emailTask' doesn't match '_[a-z][a-zA-Z0-9]*' 110
FieldNamingConventions The constant name 'infosSuperviseurService' doesn't match '[A-Z][A-Z_0-9]*' 112

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/ChantierJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'chantierControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 66
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 67

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DemandeOccupationJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 91
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 93
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 95

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DemandeurJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'entiteControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 72
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 73
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 74
FieldNamingConventions The constant name 'ctvDemandeUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 77

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DocumentsJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 76
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 78

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DownloadJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 56

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/EmpriseJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'empriseControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 68
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 88
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 90
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 92

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/ExecutantJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'executantControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 58
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 59

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/GestionConflitsJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireServiceBo' doesn't match '_[a-z][a-zA-Z0-9]*' 59

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/SuperviseurJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 69
FieldNamingConventions The field name 'formulaireServiceBo' doesn't match '_[a-z][a-zA-Z0-9]*' 70
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 71
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 76
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 77

fr/paris/lutece/plugins/ctv/bo/controller/emails/GestionEmailJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'emailConfigService' doesn't match '_[a-z][a-zA-Z0-9]*' 56

fr/paris/lutece/plugins/ctv/bo/controller/referentiels/ManageNumericParametersJspBean.java

Rule Violation Ligne
LocalVariableNamingConventions The local variable name 'entity_key' doesn't match '[a-z][a-zA-Z0-9]*' 62
LocalVariableNamingConventions The local variable name 'entity_value' doesn't match '[a-z][a-zA-Z0-9]*' 63

fr/paris/lutece/plugins/ctv/bo/controller/referentiels/ManageReferentielsJspBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 61

fr/paris/lutece/plugins/ctv/bo/controller/tdb/TableauDeBordJspBean.java

Rule Violation Ligne
FieldNamingConventions The static field name 'tableauDeBordInstructionControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 62
FieldNamingConventions The static field name 'tableauDeBordConcessionnaireControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 64

fr/paris/lutece/plugins/ctv/cas/web/CustomLuteceCASFilter.java

Rule Violation Ligne
FieldNamingConventions The field name 'noCookieMessageKey' doesn't match '_[a-z][a-zA-Z0-9]*' 41
FieldNamingConventions The field name 'redirectAfterGateway' doesn't match '_[a-z][a-zA-Z0-9]*' 53

fr/paris/lutece/plugins/ctv/constants/CtvConstants.java

Rule Violation Ligne
ClassNamingConventions The utility class name 'CtvConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 36297
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 36297

fr/paris/lutece/plugins/ctv/controller/front/ChantierApp.java

Rule Violation Ligne
FieldNamingConventions The field name 'chantierControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 80
FieldNamingConventions The field name 'formulaireDemandeOccupationServiceImpl' doesn't match '_[a-z][a-zA-Z0-9]*' 81
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 82

fr/paris/lutece/plugins/ctv/controller/front/ConfirmationApp.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 70
FieldNamingConventions The constant name 'demandeService' doesn't match '[A-Z][A-Z_0-9]*' 72

fr/paris/lutece/plugins/ctv/controller/front/DemandeurApp.java

Rule Violation Ligne
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 99
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 100
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 101
FieldNamingConventions The static field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 102
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 104

fr/paris/lutece/plugins/ctv/controller/front/DocumentApp.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 94
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 95
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 97
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 99
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 100

fr/paris/lutece/plugins/ctv/controller/front/EditionApp.java

Rule Violation Ligne
FieldNamingConventions The constant name 'editionService' doesn't match '[A-Z][A-Z_0-9]*' 84
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 85
FieldNamingConventions The constant name 'ctvEmpriseUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 90
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 172
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 187
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 222

fr/paris/lutece/plugins/ctv/controller/front/EmpriseApp.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 95
FieldNamingConventions The field name 'empriseCtrlService' doesn't match '_[a-z][a-zA-Z0-9]*' 96
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 97
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 98

fr/paris/lutece/plugins/ctv/controller/front/ExecutantApp.java

Rule Violation Ligne
FieldNamingConventions The field name 'executantControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 69
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 70

fr/paris/lutece/plugins/ctv/controller/front/MonEspaceDemandeurApp.java

Rule Violation Ligne
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 105
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 106
FieldNamingConventions The field name 'prolongationService' doesn't match '_[a-z][a-zA-Z0-9]*' 107
FieldNamingConventions The field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 108
FieldNamingConventions The field name 'demandeFinTravauxService' doesn't match '_[a-z][a-zA-Z0-9]*' 110
FieldNamingConventions The static field name 'alerteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 112
FieldNamingConventions The constant name 'ctvDemandeUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 114
FieldNamingConventions The constant name 'ctvEmpriseUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 116
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 119

fr/paris/lutece/plugins/ctv/controller/front/ValidationApp.java

Rule Violation Ligne
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 87
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 88
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 90
FieldNamingConventions The field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 91
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 92
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 93
FieldNamingConventions The field name 'emailTask' doesn't match '_[a-z][a-zA-Z0-9]*' 94
FieldNamingConventions The constant name 'infosSuperviseurService' doesn't match '[A-Z][A-Z_0-9]*' 96

fr/paris/lutece/plugins/ctv/form/GestionConflitsFormBean.java

Rule Violation Ligne
FieldNamingConventions The field name 'dateDebutRecherche' doesn't match '_[a-z][a-zA-Z0-9]*' 40
FieldNamingConventions The field name 'dateFinRecherche' doesn't match '_[a-z][a-zA-Z0-9]*' 41
FieldNamingConventions The field name 'rayon' doesn't match '_[a-z][a-zA-Z0-9]*' 42
FieldNamingConventions The field name 'hasConflits' doesn't match '_[a-z][a-zA-Z0-9]*' 43

fr/paris/lutece/plugins/ctv/handler/AbstractCtvUploadHandler.java

Rule Violation Ligne
FieldNamingConventions The field name 'uploadDirectory' doesn't match '_[a-z][a-zA-Z0-9]*' 77
FieldNamingConventions The field name 'maxFileSize' doesn't match '_[a-z][a-zA-Z0-9]*' 80
FieldNamingConventions The field name 'activeRemoteFilesSynchro' doesn't match '_[a-z][a-zA-Z0-9]*' 83
FieldNamingConventions The static field name 'extensionList' doesn't match '_[a-z][a-zA-Z0-9]*' 98
FieldNamingConventions The field name 'currentDir' doesn't match '_[a-z][a-zA-Z0-9]*' 399

fr/paris/lutece/plugins/ctv/handler/DeleteFilesOnEndUploadCleaningTracker.java

Rule Violation Ligne
FieldNamingConventions The field name 'filesToDelete' doesn't match '_[a-z][a-zA-Z0-9]*' 49

fr/paris/lutece/plugins/ctv/mailtask/EmailTask.java

Rule Violation Ligne
FieldNamingConventions The field name 'emailConfigService' doesn't match '_[a-z][a-zA-Z0-9]*' 91
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 92
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 93
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 94
FieldNamingConventions The field name 'unitUserService' doesn't match '_[a-z][a-zA-Z0-9]*' 95
FieldNamingConventions The field name 'infosSuperviseurService' doesn't match '_[a-z][a-zA-Z0-9]*' 96
FieldNamingConventions The constant name 'referentielsService' doesn't match '[A-Z][A-Z_0-9]*' 97
FieldNamingConventions The field name 'conventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 98

fr/paris/lutece/plugins/ctv/service/AbstractEmpriseControllerService.java

Rule Violation Ligne
FieldNamingConventions The static field name 'PATH_FOLDER_IMAGE_EMPRISE' doesn't match '_[a-z][a-zA-Z0-9]*' 94
FieldNamingConventions The constant name 'empriseService' doesn't match '[A-Z][A-Z_0-9]*' 96
FieldNamingConventions The constant name 'referentielsService' doesn't match '[A-Z][A-Z_0-9]*' 97
FieldNamingConventions The constant name 'formulaireService' doesn't match '[A-Z][A-Z_0-9]*' 98
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 99
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 100
FieldNamingConventions The constant name 'stockageService' doesn't match '[A-Z][A-Z_0-9]*' 102

fr/paris/lutece/plugins/ctv/service/AbstractFormulaireService.java

Rule Violation Ligne
FieldNamingConventions The constant name 'demandeService' doesn't match '[A-Z][A-Z_0-9]*' 83
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 85
FieldNamingConventions The constant name 'entiteDemandeService' doesn't match '[A-Z][A-Z_0-9]*' 87
FieldNamingConventions The constant name 'empriseService' doesn't match '[A-Z][A-Z_0-9]*' 89
FieldNamingConventions The constant name 'conventionService' doesn't match '[A-Z][A-Z_0-9]*' 91
FieldNamingConventions The constant name 'ctvDemandeUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 93
FieldNamingConventions The constant name 'ctvEmpriseUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 96
FieldNamingConventions The constant name 'ctvConferenceCoordinationUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 99

fr/paris/lutece/plugins/ctv/service/AbstractTableauDeBordControllerService.java

Rule Violation Ligne
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 79
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 80

fr/paris/lutece/plugins/ctv/service/ChantierControllerService.java

Rule Violation Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 60123
FieldNamingConventions The constant name 'ctvDemandeUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 63
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 66
FieldNamingConventions The constant name 'formulaireService' doesn't match '[A-Z][A-Z_0-9]*' 68
FieldNamingConventions The constant name 'chantierControllerService' doesn't match '[A-Z][A-Z_0-9]*' 70
FieldNamingConventions The constant name 'referentielsService' doesn't match '[A-Z][A-Z_0-9]*' 72

fr/paris/lutece/plugins/ctv/service/ChantierIntentionControllerService.java

Rule Violation Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 58119
FieldNamingConventions The static field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 61
FieldNamingConventions The static field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 64
FieldNamingConventions The static field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 65
FieldNamingConventions The static field name 'chantierControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 67

fr/paris/lutece/plugins/ctv/service/ChantierInterventionControllerService.java

Rule Violation Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 56111
FieldNamingConventions The static field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 59
FieldNamingConventions The static field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 60
FieldNamingConventions The static field name 'chantierControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 61
FieldNamingConventions The static field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 63

fr/paris/lutece/plugins/ctv/service/CtvMapService.java

Rule Violation Ligne
FieldNamingConventions The constant name 'empriseService' doesn't match '[A-Z][A-Z_0-9]*' 75
FieldNamingConventions The constant name 'formulaireService' doesn't match '[A-Z][A-Z_0-9]*' 77
FieldNamingConventions The field name 'empriseControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 79
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 81

fr/paris/lutece/plugins/ctv/service/EditionServiceImpl.java

Rule Violation Ligne
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 139

fr/paris/lutece/plugins/ctv/service/EmpriseControllerService.java

Rule Violation Ligne
FieldNamingConventions The static field name 'PATH_FOLDER_IMAGE_EMPRISE' doesn't match '_[a-z][a-zA-Z0-9]*' 60
FieldNamingConventions The constant name 'formulaireService' doesn't match '[A-Z][A-Z_0-9]*' 61
FieldNamingConventions The constant name 'empriseCtrlService' doesn't match '[A-Z][A-Z_0-9]*' 62
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 63
FieldNamingConventions The constant name 'stockageService' doesn't match '[A-Z][A-Z_0-9]*' 64

fr/paris/lutece/plugins/ctv/service/EmpriseIntentionControllerService.java

Rule Violation Ligne
FieldNamingConventions The static field name 'PATH_FOLDER_IMAGE_EMPRISE' doesn't match '_[a-z][a-zA-Z0-9]*' 62
FieldNamingConventions The constant name 'formulaireService' doesn't match '[A-Z][A-Z_0-9]*' 63
FieldNamingConventions The constant name 'empriseCtrlService' doesn't match '[A-Z][A-Z_0-9]*' 64
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 65
FieldNamingConventions The constant name 'stockageService' doesn't match '[A-Z][A-Z_0-9]*' 66

fr/paris/lutece/plugins/ctv/service/EmpriseInterventionControllerService.java

Rule Violation Ligne
FieldNamingConventions The constant name 'formulaireService' doesn't match '[A-Z][A-Z_0-9]*' 50
FieldNamingConventions The constant name 'empriseCtrlService' doesn't match '[A-Z][A-Z_0-9]*' 51
FieldNamingConventions The constant name 'demandeService' doesn't match '[A-Z][A-Z_0-9]*' 52

fr/paris/lutece/plugins/ctv/service/ExecutantControllerService.java

Rule Violation Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 52100
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 55
FieldNamingConventions The constant name 'formulaireService' doesn't match '[A-Z][A-Z_0-9]*' 57
FieldNamingConventions The constant name 'executantCtrlService' doesn't match '[A-Z][A-Z_0-9]*' 59

fr/paris/lutece/plugins/ctv/service/ExecutantInterventionControllerService.java

Rule Violation Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 52104
FieldNamingConventions The constant name 'executantCtrlService' doesn't match '[A-Z][A-Z_0-9]*' 55
FieldNamingConventions The constant name 'formulaireService' doesn't match '[A-Z][A-Z_0-9]*' 57
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 59

fr/paris/lutece/plugins/ctv/service/FluxDonneesEmprisesTaxationSavoi2.java

Rule Violation Ligne
FieldNamingConventions The field name 'fluxService' doesn't match '_[a-z][a-zA-Z0-9]*' 18
FieldNamingConventions The field name 'file' doesn't match '_[a-z][a-zA-Z0-9]*' 19
FieldNamingConventions The constant name 'uploadBaseDirectory' doesn't match '[A-Z][A-Z_0-9]*' 20
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 45

fr/paris/lutece/plugins/ctv/service/FormulaireDemandeIntentionServiceImpl.java

Rule Violation Ligne
FieldNamingConventions The constant name 'attributeService' doesn't match '[A-Z][A-Z_0-9]*' 99
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 101
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 103
FieldNamingConventions The public constant name 'formulaireDemandeIntentionService' doesn't match '[A-Z][A-Z_0-9]*' 109
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 111

fr/paris/lutece/plugins/ctv/service/FormulaireDemandeInterventionServiceImpl.java

Rule Violation Ligne
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 92
FieldNamingConventions The constant name 'conflitService' doesn't match '[A-Z][A-Z_0-9]*' 94
FieldNamingConventions The constant name 'infosSuperviseurService' doesn't match '[A-Z][A-Z_0-9]*' 96
FieldNamingConventions The public constant name 'formulaireDemandeInterventionService' doesn't match '[A-Z][A-Z_0-9]*' 102

fr/paris/lutece/plugins/ctv/service/FormulaireDemandeOccupationBoServiceImpl.java

Rule Violation Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 56125
FieldNamingConventions The constant name 'demandeService' doesn't match '[A-Z][A-Z_0-9]*' 60
FieldNamingConventions The constant name 'conflitService' doesn't match '[A-Z][A-Z_0-9]*' 62
FieldNamingConventions The constant name 'infosSuperviseurService' doesn't match '[A-Z][A-Z_0-9]*' 64
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 66
FieldNamingConventions The public constant name 'formulaireDemandeOccupationService' doesn't match '[A-Z][A-Z_0-9]*' 73

fr/paris/lutece/plugins/ctv/service/FormulaireDemandeOccupationServiceImpl.java

Rule Violation Ligne
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 89
FieldNamingConventions The public constant name 'formulaireDemandeOccupationService' doesn't match '[A-Z][A-Z_0-9]*' 96
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 98

fr/paris/lutece/plugins/ctv/service/StatutsEmprisesDaemon.java

Rule Violation Ligne
FieldNamingConventions The constant name 'empriseService' doesn't match '[A-Z][A-Z_0-9]*' 50
FieldNamingConventions The constant name 'demandeFinTravauxService' doesn't match '[A-Z][A-Z_0-9]*' 51
FieldNamingConventions The constant name 'demandeService' doesn't match '[A-Z][A-Z_0-9]*' 52

fr/paris/lutece/plugins/ctv/service/TableauDeBordConcessionnaireControllerService.java

Rule Violation Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 81512
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 86
FieldNamingConventions The static field name 'tableauDeBordConcessionnaireControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 90

fr/paris/lutece/plugins/ctv/service/TableauDeBordInstructionControllerService.java

Rule Violation Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 88576
FieldNamingConventions The static field name 'tableauDeBordInstructionControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 93

fr/paris/lutece/plugins/ctv/util/GestionPDFUtil.java

Rule Violation Ligne
FieldNamingConventions The constant name 'unitService' doesn't match '[A-Z][A-Z_0-9]*' 72
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 73
FieldNamingConventions The constant name 'referentielsService' doesn't match '[A-Z][A-Z_0-9]*' 74
FieldNamingConventions The constant name 'empriseService' doesn't match '[A-Z][A-Z_0-9]*' 75
FieldNamingConventions The constant name 'editionService' doesn't match '[A-Z][A-Z_0-9]*' 76
FieldNamingConventions The constant name 'prolongationService' doesn't match '[A-Z][A-Z_0-9]*' 77
FieldNamingConventions The constant name 'infosSuperviseurService' doesn't match '[A-Z][A-Z_0-9]*' 78
FieldNamingConventions The constant name 'demandeService' doesn't match '[A-Z][A-Z_0-9]*' 79
FieldNamingConventions The static field name 'nbDocToGen' doesn't match '_[a-z][a-zA-Z0-9]*' 88

fr/paris/lutece/plugins/ctv/util/RemoteFilesUtils.java

Rule Violation Ligne
FieldNamingConventions The static field name 'HOST' doesn't match '_[a-z][a-zA-Z0-9]*' 61
FieldNamingConventions The static field name 'USER' doesn't match '_[a-z][a-zA-Z0-9]*' 63
FieldNamingConventions The static field name 'PASSWORD' doesn't match '_[a-z][a-zA-Z0-9]*' 65
FieldNamingConventions The static field name 'UPLOAD_DIRECTORY' doesn't match '_[a-z][a-zA-Z0-9]*' 67
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 110

fr/paris/lutece/plugins/ctv/util/SslContextSecureProtocolSocketFactory.java

Rule Violation Ligne
FieldNamingConventions The field name 'sslContext' doesn't match '_[a-z][a-zA-Z0-9]*' 62
FieldNamingConventions The field name 'verifyHostname' doesn't match '_[a-z][a-zA-Z0-9]*' 63
LocalVariableNamingConventions The local variable name 'i$' doesn't match '[a-z][a-zA-Z0-9]*' 179

fr/paris/lutece/plugins/ctv/util/Utils.java

Rule Violation Ligne
ClassNamingConventions The utility class name 'Utils' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 60649
FieldNamingConventions The static field name 'PATH_LABEL_KEY' doesn't match '_[a-z][a-zA-Z0-9]*' 63
FieldNamingConventions The static field name 'PATH_LABEL_KEY_CHAMP' doesn't match '_[a-z][a-zA-Z0-9]*' 64

fr/paris/lutece/plugins/ctv/util/sort/CustomAttributeComparator.java

Rule Violation Ligne
FieldNamingConventions The field name 'sortedAttribute' doesn't match '_[a-z][a-zA-Z0-9]*' 50
FieldNamingConventions The field name 'isASC' doesn't match '_[a-z][a-zA-Z0-9]*' 51

fr/paris/lutece/plugins/ctv/web/bean/BeanGisEntry.java

Rule Violation Ligne
FieldNamingConventions The field name 'idEntry' doesn't match '_[a-z][a-zA-Z0-9]*' 39

fr/paris/lutece/plugins/ctv/web/bean/BlocLabelWithDate.java

Rule Violation Ligne
FieldNamingConventions The field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' 41
FieldNamingConventions The field name 'date' doesn't match '_[a-z][a-zA-Z0-9]*' 42
FieldNamingConventions The field name 'dateToUpdate' doesn't match '_[a-z][a-zA-Z0-9]*' 43

fr/paris/lutece/plugins/ctv/web/bean/DemandeWithFiles.java

Rule Violation Ligne
FieldNamingConventions The field name 'listOfFiles' doesn't match '_[a-z][a-zA-Z0-9]*' 46

Priority 2

fr/paris/lutece/plugins/ctv/controller/front/MonEspaceDemandeurApp.java

Rule Violation Ligne
IntegerInstantiation Avoid instantiating Integer objects. Call Integer.valueOf() instead. 235
IntegerInstantiation Avoid instantiating Integer objects. Call Integer.valueOf() instead. 235
IntegerInstantiation Avoid instantiating Integer objects. Call Integer.valueOf() instead. 261
IntegerInstantiation Avoid instantiating Integer objects. Call Integer.valueOf() instead. 261

fr/paris/lutece/plugins/ctv/handler/AbstractCtvUploadHandler.java

Rule Violation Ligne
AvoidReassigningParameters Avoid reassigning parameters such as 'uploadDirectory' 314
AvoidReassigningParameters Avoid reassigning parameters such as 'uploadDirectory' 314

fr/paris/lutece/plugins/ctv/handler/DeleteFilesOnEndUploadCleaningTracker.java

Rule Violation Ligne
DoNotCallGarbageCollectionExplicitly Do not explicitly trigger a garbage collection. 59

fr/paris/lutece/plugins/ctv/util/EmpriseMappingUtils.java

Rule Violation Ligne
AvoidReassigningParameters Avoid reassigning parameters such as 'value' 119

fr/paris/lutece/plugins/ctv/util/GestionPDFUtil.java

Rule Violation Ligne
AvoidReassigningParameters Avoid reassigning parameters such as 'uploadDirectory' 391
AvoidReassigningParameters Avoid reassigning parameters such as 'uploadDirectory' 391
AvoidReassigningParameters Avoid reassigning parameters such as 'uploadDirectory' 391
AvoidReassigningParameters Avoid reassigning parameters such as 'uploadDirectory' 391
AvoidReassigningParameters Avoid reassigning parameters such as 'uploadDirectory' 391

fr/paris/lutece/plugins/ctv/util/RemoteFilesUtils.java

Rule Violation Ligne
AvoidReassigningParameters Avoid reassigning parameters such as 'srcRemoteDirectory' 69
AvoidReassigningParameters Avoid reassigning parameters such as 'srcRemoteDirectory' 126
AvoidReassigningParameters Avoid reassigning parameters such as 'remoteFilePath' 202

fr/paris/lutece/plugins/ctv/util/Utils.java

Rule Violation Ligne
AvoidReassigningParameters Avoid reassigning parameters such as 'msgArgs' 474
AvoidReassigningParameters Avoid reassigning parameters such as 'msgArgs' 512
AvoidReassigningParameters Avoid reassigning parameters such as 'fields' 637

fr/paris/lutece/plugins/ctv/util/sort/CustomAttributeComparator.java

Rule Violation Ligne
AvoidReassigningParameters Avoid reassigning parameters such as 'obj' 163

Priority 3

fr/paris/lutece/plugins/ctv/authentification/MokeLuteceAuthentication.java

Rule Violation Ligne
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 5052
UncommentedEmptyConstructor Document empty constructor 5052
MissingOverride The method 'logout(LuteceUser)' is missing an @Override annotation. 7577
CommentRequired Public method and constructor comments are required 7577
UncommentedEmptyMethodBody Document empty method body 7677
MissingOverride The method 'isUserInRole(LuteceUser, HttpServletRequest, String)' is missing an @Override annotation. 9194
CommentRequired Public method and constructor comments are required 9194
MissingOverride The method 'getRolesByUser(LuteceUser)' is missing an @Override annotation. 9699
CommentRequired Public method and constructor comments are required 9699
MissingOverride The method 'updateDateLastLogin(LuteceUser, HttpServletRequest)' is missing an @Override annotation. 215217
CommentRequired Public method and constructor comments are required 215217
UncommentedEmptyMethodBody Document empty method body 216217

fr/paris/lutece/plugins/ctv/bo/controller/conference/coordination/ConferenceCoordinationJspBean.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34408
CommentRequired Header comments are required 77408
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 80
CommentRequired Public method and constructor comments are required 9195
CommentRequired Public method and constructor comments are required 98113
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 123
NPathComplexity The method 'putDemandes(HttpServletRequest, Map)' has an NPath complexity of 361 193307
ExcessiveMethodLength Avoid really long methods. 193307
NcssCount The method 'putDemandes(HttpServletRequest, Map)' has a NCSS line count of 62. 193307
CyclomaticComplexity The method 'putDemandes(HttpServletRequest, Map)' has a cyclomatic complexity of 15. 193307
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 200
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'typesDemande' 385
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'statutEmprise' 397
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'statutDemande' 405

fr/paris/lutece/plugins/ctv/bo/controller/conference/coordination/GestionConferenceCoordinationJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 63195
ImmutableField Private field 'conferenceService' could be made final; it is only initialized in the declaration or constructor. 67
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 71
CommentRequired Public method and constructor comments are required 74115
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 102
AvoidDuplicateLiterals The String literal "conferenceDocument" appears 6 times in this file; the first occurrence is on line 110 110
CommentRequired Public method and constructor comments are required 118153
ConfusingTernary Avoid if (x != y) ..; else ..; 131147
CommentRequired Public method and constructor comments are required 156164
CommentRequired Public method and constructor comments are required 167173
CommentRequired Public method and constructor comments are required 176186

fr/paris/lutece/plugins/ctv/bo/controller/conventions/GestionConventionJspBean.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34240
CommentRequired Header comments are required 74240
ImmutableField Private field 'conventionService' could be made final; it is only initialized in the declaration or constructor. 78
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 79
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 80
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 81
ImmutableField Private field 'ctvDemandeUploadHandler' could be made final; it is only initialized in the declaration or constructor. 82
ImmutableField Private field 'emailTask' could be made final; it is only initialized in the declaration or constructor. 84
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the validationConvention field if you want a default access modifier 86
DefaultPackage Use explicit scoping instead of the default package private level 86
CommentRequired Public method and constructor comments are required 89107
CommentRequired Public method and constructor comments are required 110130
CommentRequired Public method and constructor comments are required 133151
CommentRequired Public method and constructor comments are required 154191

fr/paris/lutece/plugins/ctv/bo/controller/demandes/AbstractCtvJspBean.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34322
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 151
CommentRequired Protected method constructor comments are required 193200
NPathComplexity The method 'fillUserPossibleActions(HttpServletRequest, Map, int)' has an NPath complexity of 288 207253
CyclomaticComplexity The method 'fillUserPossibleActions(HttpServletRequest, Map, int)' has a cyclomatic complexity of 18. 207253
CommentRequired Protected method constructor comments are required 255269
CommentRequired Public method and constructor comments are required 271279
CommentRequired Protected method constructor comments are required 281294
CommentRequired Public method and constructor comments are required 296315
CommentRequired Protected method constructor comments are required 317320

fr/paris/lutece/plugins/ctv/bo/controller/demandes/AbstractGestionConflitsJspBean.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34219
CommentRequired Header comments are required 64219
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 86
CommentRequired Protected method constructor comments are required 95
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 97
CommentRequired Protected method constructor comments are required 99
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 120
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 121
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 122
CommentRequired Protected method constructor comments are required 178186
CommentRequired Protected method constructor comments are required 188202
CommentRequired Protected method constructor comments are required 204209
CommentRequired Protected method constructor comments are required 211216
CommentRequired Protected method constructor comments are required 218

fr/paris/lutece/plugins/ctv/bo/controller/demandes/AbstractSuperviseurJpsBean.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34537
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34537
GodClass Possible God Class (WMC=55, ATFD=94, TCC=8.974%) 91537
CommentRequired Header comments are required 91537
TooManyMethods This class has too many methods, consider refactoring it. 92537
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 122
CommentRequired Protected method constructor comments are required 126
CommentRequired Protected method constructor comments are required 128
ImmutableField Private field 'saisineService' could be made final; it is only initialized in the declaration or constructor. 130
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 130
ImmutableField Private field 'conflitService' could be made final; it is only initialized in the declaration or constructor. 131
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 131
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 132
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 134
ProperLogger Logger should be defined private static final and have the correct class 136
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 136
CommentRequired Protected method constructor comments are required 138163
CyclomaticComplexity The method 'doSave(HttpServletRequest)' has a cyclomatic complexity of 17. 165247
NPathComplexity The method 'doSave(HttpServletRequest)' has an NPath complexity of 2016 165247
CommentRequired Protected method constructor comments are required 165247
CommentRequired Protected method constructor comments are required 249316
CommentRequired Protected method constructor comments are required 327352
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 329
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 329
ConfusingTernary Avoid if (x != y) ..; else ..; 339347
CommentRequired Protected method constructor comments are required 367383
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 413
CommentRequired Protected method constructor comments are required 443477
ConfusingTernary Avoid if (x != y) ..; else ..; 458474
CommentRequired Protected method constructor comments are required 479487
CommentRequired Protected method constructor comments are required 489510
MissingBreakInSwitch A switch statement does not contain a break 497507
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 506
CommentRequired Protected method constructor comments are required 512535

fr/paris/lutece/plugins/ctv/bo/controller/demandes/ConsultationCRVisiteJspBean.java

Rule Violation Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 60128
CommentRequired Header comments are required 60128
ImmutableField Private field 'ctvUploadCompteRenduVisiteHandler' could be made final; it is only initialized in the declaration or constructor. 67
CommentRequired Public method and constructor comments are required 7798
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 79
ShortVariable Avoid variables with short names like id 85
CommentRequired Public method and constructor comments are required 101112
ShortVariable Avoid variables with short names like id 103
CommentRequired Public method and constructor comments are required 115119

fr/paris/lutece/plugins/ctv/bo/controller/demandes/CreationCRVisiteJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 62160
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 62160
ImmutableField Private field 'ctvUploadCompteRenduVisiteHandler' could be made final; it is only initialized in the declaration or constructor. 67
CommentRequired Public method and constructor comments are required 7982
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 84
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 86
AvoidDuplicateLiterals The String literal "num_demande" appears 4 times in this file; the first occurrence is on line 94 94
CommentRequired Public method and constructor comments are required 119123
CommentRequired Public method and constructor comments are required 126151

fr/paris/lutece/plugins/ctv/bo/controller/demandes/ProjetJspBean.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34986
TooManyStaticImports Too many static imports may lead to messy code 34986
CommentRequired Header comments are required 106986
TooManyMethods This class has too many methods, consider refactoring it. 107986
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 114
ImmutableField Private field 'demandeFinTravauxService' could be made final; it is only initialized in the declaration or constructor. 115
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 116
ImmutableField Private field 'prolongationService' could be made final; it is only initialized in the declaration or constructor. 118
CommentRequired Public method and constructor comments are required 129134
CyclomaticComplexity The method 'initModel(HttpServletRequest)' has a cyclomatic complexity of 15. 136242
ExcessiveMethodLength Avoid really long methods. 136242
NPathComplexity The method 'initModel(HttpServletRequest)' has an NPath complexity of 273 136242
NcssCount The method 'initModel(HttpServletRequest)' has a NCSS line count of 66. 136242
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 138
ConfusingTernary Avoid if (x != y) ..; else ..; 163
ConfusingTernary Avoid if (x != y) ..; else ..; 166228
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 198
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 198
CommentRequired Public method and constructor comments are required 251279
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 259
CommentRequired Public method and constructor comments are required 282286
AvoidDuplicateLiterals The String literal "empriseId" appears 14 times in this file; the first occurrence is on line 284 284
CommentRequired Public method and constructor comments are required 289326
CommentRequired Public method and constructor comments are required 329339
CommentRequired Public method and constructor comments are required 342352
CommentRequired Public method and constructor comments are required 355392
CommentRequired Public method and constructor comments are required 395408
CommentRequired Public method and constructor comments are required 411424
CommentRequired Public method and constructor comments are required 427433
CommentRequired Public method and constructor comments are required 436488
ConfusingTernary Avoid if (x != y) ..; else ..; 452459
ConfusingTernary Avoid if (x != y) ..; else ..; 461468
ConfusingTernary Avoid if (x != y) ..; else ..; 470477
CommentRequired Public method and constructor comments are required 491534
ConfusingTernary Avoid if (x != y) ..; else ..; 504511
ConfusingTernary Avoid if (x != y) ..; else ..; 513520
CommentRequired Public method and constructor comments are required 537569
ConfusingTernary Avoid if (x != y) ..; else ..; 549556
CommentRequired Public method and constructor comments are required 572665
CyclomaticComplexity The method 'refusProlongation(HttpServletRequest)' has a cyclomatic complexity of 19. 572665
NPathComplexity The method 'refusProlongation(HttpServletRequest)' has an NPath complexity of 604 572665
CyclomaticComplexity The method 'acceptationProlongation(HttpServletRequest)' has a cyclomatic complexity of 20. 668763
CommentRequired Public method and constructor comments are required 668763
NPathComplexity The method 'acceptationProlongation(HttpServletRequest)' has an NPath complexity of 1190 668763
CyclomaticComplexity The method 'acceptationFinDeTravaux(HttpServletRequest, TypeFinTravaux, StatutEmprise, int)' has a cyclomatic complexity of 18. 805892
NPathComplexity The method 'acceptationFinDeTravaux(HttpServletRequest, TypeFinTravaux, StatutEmprise, int)' has an NPath complexity of 1057 805892
CollapsibleIfStatements These nested if statements could be combined 865878
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 868
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 868
CommentRequired Public method and constructor comments are required 920931
CommentRequired Public method and constructor comments are required 934948
CommentRequired Public method and constructor comments are required 951970
CommentRequired Public method and constructor comments are required 979984

fr/paris/lutece/plugins/ctv/bo/controller/demandes/generic/DemandeGenericJspBean.java

Rule Violation Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 4796
CommentRequired Header comments are required 4796
CommentRequired Public method and constructor comments are required 5394
ConfusingTernary Avoid if (x != y) ..; else ..; 5992

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/ChantierJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 60150
ImmutableField Private field 'chantierIntentionControllerService' could be made final; it is only initialized in the declaration or constructor. 65
CommentRequired Public method and constructor comments are required 7086
CommentRequired Public method and constructor comments are required 98111
CommentRequired Public method and constructor comments are required 114118
CommentRequired Public method and constructor comments are required 121128
CommentRequired Public method and constructor comments are required 131140

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/DemandeIntentionJspBean.java

Rule Violation Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 101257
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 123
CyclomaticComplexity The method 'getInstructionDemandeIntention(HttpServletRequest)' has a cyclomatic complexity of 12. 136200
NPathComplexity The method 'getInstructionDemandeIntention(HttpServletRequest)' has an NPath complexity of 432 136200
ConfusingTernary Avoid if (x != y) ..; else ..; 144152
ConfusingTernary Avoid if (x != y) ..; else ..; 173
ConfusingTernary Avoid if (x != y) ..; else ..; 189197
CommentRequired Public method and constructor comments are required 203208
CommentRequired Public method and constructor comments are required 211216
CommentRequired Public method and constructor comments are required 219228
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 239

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/DemandeurJspBean.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34240
CommentRequired Header comments are required 71240
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 76
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 77
CommentRequired Public method and constructor comments are required 85102
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 88
CommentRequired Public method and constructor comments are required 105119
CommentRequired Public method and constructor comments are required 122128
CommentRequired Public method and constructor comments are required 131137
CommentRequired Public method and constructor comments are required 140149

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/EmpriseJspBean.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34637
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34637
CommentRequired Header comments are required 108637
TooManyMethods This class has too many methods, consider refactoring it. 109637
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 118
ImmutableField Private field 'empriseCtrlService' could be made final; it is only initialized in the declaration or constructor. 119
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 120
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 124
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 127
CommentRequired Public method and constructor comments are required 133162
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 164
CommentRequired Public method and constructor comments are required 165177
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 179
CommentRequired Public method and constructor comments are required 180190
CommentRequired Public method and constructor comments are required 193213
ConfusingTernary Avoid if (x != y) ..; else ..; 197205
AvoidDuplicateLiterals The String literal "profile_default" appears 6 times in this file; the first occurrence is on line 207 207
CommentRequired Public method and constructor comments are required 216225
CommentRequired Public method and constructor comments are required 228237
CommentRequired Public method and constructor comments are required 240250
CommentRequired Public method and constructor comments are required 256273
CommentRequired Public method and constructor comments are required 276293
CommentRequired Public method and constructor comments are required 313319
CommentRequired Public method and constructor comments are required 322342
ConfusingTernary Avoid if (x != y) ..; else ..; 334341
CommentRequired Public method and constructor comments are required 348352
CommentRequired Public method and constructor comments are required 360369
CommentRequired Public method and constructor comments are required 372401
CommentRequired Public method and constructor comments are required 414447
CommentRequired Public method and constructor comments are required 450462
CommentRequired Public method and constructor comments are required 465490
CommentRequired Public method and constructor comments are required 493503
CommentRequired Public method and constructor comments are required 506516
CommentRequired Public method and constructor comments are required 519536
ShortVariable Avoid variables with short names like x 552
ShortVariable Avoid variables with short names like y 553
CommentRequired Public method and constructor comments are required 577607
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 596
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 596
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 611
CommentRequired Public method and constructor comments are required 622627

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/ChantierJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 56134
ImmutableField Private field 'chantierControllerService' could be made final; it is only initialized in the declaration or constructor. 62
CommentRequired Public method and constructor comments are required 6574
CommentRequired Public method and constructor comments are required 7781
CommentRequired Public method and constructor comments are required 8489
CommentRequired Public method and constructor comments are required 92102
CommentRequired Public method and constructor comments are required 105114

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/DemandeInterventionJspBean.java

Rule Violation Ligne
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 87
CyclomaticComplexity The method 'getInstructionDemandeIntervention(HttpServletRequest)' has a cyclomatic complexity of 10. 100156
UnusedLocalVariable Avoid unused local variables such as 'canAccessDemande'. 125
ConfusingTernary Avoid if (x != y) ..; else ..; 128136
ConfusingTernary Avoid if (x != y) ..; else ..; 141
CommentRequired Public method and constructor comments are required 159164
CommentRequired Public method and constructor comments are required 167172
CommentRequired Public method and constructor comments are required 175184
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 194

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/DemandeurJspBean.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34203
CommentRequired Header comments are required 70203
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 75
CommentRequired Public method and constructor comments are required 8497
CommentRequired Public method and constructor comments are required 100105
CommentRequired Public method and constructor comments are required 108119
CommentRequired Public method and constructor comments are required 122128
CommentRequired Public method and constructor comments are required 131140
ConfusingTernary Avoid if (x != y) ..; else ..; 173185
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 178
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 199

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/DocumentsJspBean.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34189
CommentRequired Header comments are required 67189
CommentRequired Public method and constructor comments are required 8898
CommentRequired Public method and constructor comments are required 101113
CommentRequired Public method and constructor comments are required 116131
CommentRequired Public method and constructor comments are required 134143
CommentRequired Public method and constructor comments are required 146151

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/EmpriseJspBean.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34222
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34222
CommentRequired Header comments are required 73222
ImmutableField Private field 'empriseCtrlService' could be made final; it is only initialized in the declaration or constructor. 78
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 82
CommentRequired Public method and constructor comments are required 8797
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 99
CommentRequired Public method and constructor comments are required 100108
CommentRequired Public method and constructor comments are required 111124
CommentRequired Public method and constructor comments are required 127148
CommentRequired Public method and constructor comments are required 163172
CommentRequired Public method and constructor comments are required 175181
CommentRequired Public method and constructor comments are required 184187
CommentRequired Public method and constructor comments are required 190193
CommentRequired Public method and constructor comments are required 196200
CommentRequired Public method and constructor comments are required 203220

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/ExecutantJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 55149
ImmutableField Private field 'executantControllerService' could be made final; it is only initialized in the declaration or constructor. 65
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 70
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 71
CommentRequired Public method and constructor comments are required 7684
CommentRequired Public method and constructor comments are required 8793
CommentRequired Public method and constructor comments are required 96105
CommentRequired Public method and constructor comments are required 108115
CommentRequired Public method and constructor comments are required 118130

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/GestionConflitsJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 54131
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 58
CommentRequired Public method and constructor comments are required 6468
CommentRequired Public method and constructor comments are required 7175
CommentRequired Public method and constructor comments are required 7898
CommentRequired Public method and constructor comments are required 101110

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/SuperviseurJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 67187
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 71
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 83
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 84
CommentRequired Public method and constructor comments are required 87103
CommentRequired Public method and constructor comments are required 106117
CommentRequired Public method and constructor comments are required 120123
CommentRequired Public method and constructor comments are required 126140
CommentRequired Public method and constructor comments are required 143154
CommentRequired Public method and constructor comments are required 169173
CommentRequired Public method and constructor comments are required 176185

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/ValidationJspBean.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34325
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34325
CommentRequired Header comments are required 98325
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 102
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 103
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 106
ImmutableField Private field 'saisineService' could be made final; it is only initialized in the declaration or constructor. 109
ImmutableField Private field 'emailTask' could be made final; it is only initialized in the declaration or constructor. 110
CommentRequired Public method and constructor comments are required 115135
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 117
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 117
ConfusingTernary Avoid if (x != y) ..; else ..; 126133
CommentRequired Public method and constructor comments are required 138142
CommentRequired Public method and constructor comments are required 145185
CommentRequired Public method and constructor comments are required 188199
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 195
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 195
ShortVariable Avoid variables with short names like x 216
ShortVariable Avoid variables with short names like y 217
CommentRequired Public method and constructor comments are required 259311
CommentRequired Public method and constructor comments are required 314323

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/ChantierJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 52124
ImmutableField Private field 'chantierControllerService' could be made final; it is only initialized in the declaration or constructor. 66
CommentRequired Public method and constructor comments are required 7081
CommentRequired Public method and constructor comments are required 8498
CommentRequired Public method and constructor comments are required 101110

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DemandeOccupationJspBean.java

Rule Violation Ligne
UnusedPrivateField Avoid unused private fields such as 'MARK_NUM_DEMANDE'. 85
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 91
ConfusingTernary Avoid if (x != y) ..; else ..; 145
CommentRequired Public method and constructor comments are required 162167
CommentRequired Public method and constructor comments are required 170175
CommentRequired Public method and constructor comments are required 178187
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 197

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DemandeurJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 66160
ImmutableField Private field 'entiteControllerService' could be made final; it is only initialized in the declaration or constructor. 72
CommentRequired Public method and constructor comments are required 8899
CommentRequired Public method and constructor comments are required 102123
CommentRequired Public method and constructor comments are required 126135

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DocumentsJspBean.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34144
CommentRequired Header comments are required 63144
CommentRequired Public method and constructor comments are required 8186
CommentRequired Public method and constructor comments are required 89104
CommentRequired Public method and constructor comments are required 107116

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DownloadJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 5180
CommentRequired Public method and constructor comments are required 5966
CommentRequired Public method and constructor comments are required 6978

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/EmpriseJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 62284
TooManyMethods This class has too many methods, consider refactoring it. 63284
ImmutableField Private field 'empriseControllerService' could be made final; it is only initialized in the declaration or constructor. 68
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 92
CommentRequired Public method and constructor comments are required 96107
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 109
CommentRequired Public method and constructor comments are required 110121
CommentRequired Public method and constructor comments are required 124145
CommentRequired Public method and constructor comments are required 148158
CommentRequired Public method and constructor comments are required 161171
CommentRequired Public method and constructor comments are required 174181
CommentRequired Public method and constructor comments are required 194200
CommentRequired Public method and constructor comments are required 203206
CommentRequired Public method and constructor comments are required 209226
CommentRequired Public method and constructor comments are required 229236
CommentRequired Public method and constructor comments are required 239246

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/ExecutantJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 52119
ImmutableField Private field 'executantControllerService' could be made final; it is only initialized in the declaration or constructor. 58
CommentRequired Public method and constructor comments are required 7077
CommentRequired Public method and constructor comments are required 8093
CommentRequired Public method and constructor comments are required 96105

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/GestionConflitsJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 54132
ImmutableField Private field 'formulaireServiceBo' could be made final; it is only initialized in the declaration or constructor. 59
CommentRequired Public method and constructor comments are required 6569
CommentRequired Public method and constructor comments are required 7276
CommentRequired Public method and constructor comments are required 7999
CommentRequired Public method and constructor comments are required 102111

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/SuperviseurJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 66180
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 69
ImmutableField Private field 'formulaireServiceBo' could be made final; it is only initialized in the declaration or constructor. 70
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 76
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 77
CommentRequired Public method and constructor comments are required 8395
CommentRequired Public method and constructor comments are required 98109
CommentRequired Public method and constructor comments are required 112115
CommentRequired Public method and constructor comments are required 118133
CommentRequired Public method and constructor comments are required 136148
CommentRequired Public method and constructor comments are required 151160
CommentRequired Public method and constructor comments are required 175178

fr/paris/lutece/plugins/ctv/bo/controller/emails/GestionEmailJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 5299
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the emailConfigService field if you want a default access modifier 56
DefaultPackage Use explicit scoping instead of the default package private level 56
CommentRequired Public method and constructor comments are required 5965
CommentRequired Public method and constructor comments are required 6878
ShortVariable Avoid variables with short names like id 71
CommentRequired Public method and constructor comments are required 8191
CommentRequired Public method and constructor comments are required 9397

fr/paris/lutece/plugins/ctv/bo/controller/referentiels/ManageNumericParametersJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 4580
DefaultPackage Use explicit scoping instead of the default package private level 49
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the MANAGE_PARAMETERS_JSP field if you want a default access modifier 49
DefaultPackage Use explicit scoping instead of the default package private level 50
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the CTV_GESTION_PARAMETRES_NUMERIC_PATH field if you want a default access modifier 50
CommentRequired Public method and constructor comments are required 5356
CommentRequired Public method and constructor comments are required 5973

fr/paris/lutece/plugins/ctv/bo/controller/referentiels/ManageReferentielsJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 52152
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the MANAGE_REFERENTIELS_JSP field if you want a default access modifier 55
DefaultPackage Use explicit scoping instead of the default package private level 55
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the CTV_GESTION_REFERENTIELS_PATH field if you want a default access modifier 57
DefaultPackage Use explicit scoping instead of the default package private level 57
CommentRequired Public method and constructor comments are required 6473
CommentRequired Public method and constructor comments are required 8489
ShortVariable Avoid variables with short names like id 86
AvoidDuplicateLiterals The String literal "itemId" appears 5 times in this file; the first occurrence is on line 86 86
CommentRequired Public method and constructor comments are required 9196
ShortVariable Avoid variables with short names like id 93
CommentRequired Public method and constructor comments are required 98106
ShortVariable Avoid variables with short names like id 100
CommentRequired Public method and constructor comments are required 108116
ShortVariable Avoid variables with short names like id 110
CommentRequired Public method and constructor comments are required 118125
CommentRequired Public method and constructor comments are required 127134
ShortVariable Avoid variables with short names like id 130
CommentRequired Public method and constructor comments are required 136144

fr/paris/lutece/plugins/ctv/bo/controller/tdb/TableauDeBordJspBean.java

Rule Violation Ligne
CommentRequired Header comments are required 57182
CommentRequired Public method and constructor comments are required 6797
AvoidDuplicateLiterals The String literal "profile_default" appears 8 times in this file; the first occurrence is on line 69 69
CommentRequired Public method and constructor comments are required 100129
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 110
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 111
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 116
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 116
CyclomaticComplexity The method 'selectDefaultProfileByPriority(HttpServletRequest, List)' has a cyclomatic complexity of 11. 131180
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 134

fr/paris/lutece/plugins/ctv/cas/web/CustomLuteceCASFilter.java

Rule Violation Ligne
RedundantFieldInitializer Avoid using redundant field initializer for 'noCookieMessageKey' 41
RedundantFieldInitializer Avoid using redundant field initializer for 'redirectAfterGateway' 53
CyclomaticComplexity The method 'doFilter(ServletRequest, ServletResponse, FilterChain)' has a cyclomatic complexity of 13. 70135
NPathComplexity The method 'doFilter(ServletRequest, ServletResponse, FilterChain)' has an NPath complexity of 240 70135
IdenticalCatchBranches 'catch' branch identical to 'LoginException' branch 121124

fr/paris/lutece/plugins/ctv/constants/CtvConstants.java

Rule Violation Ligne
CommentRequired Header comments are required 36297

fr/paris/lutece/plugins/ctv/controller/AccueilApp.java

Rule Violation Ligne
CommentRequired Header comments are required 4757
CommentRequired Public method and constructor comments are required 5356

fr/paris/lutece/plugins/ctv/controller/front/ChantierApp.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34175
TooManyStaticImports Too many static imports may lead to messy code 34175
CommentRequired Header comments are required 73175
ImmutableField Private field 'chantierControllerService' could be made final; it is only initialized in the declaration or constructor. 80
ImmutableField Private field 'formulaireDemandeOccupationServiceImpl' could be made final; it is only initialized in the declaration or constructor. 81
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 82
CommentRequired Public method and constructor comments are required 8693
CommentRequired Public method and constructor comments are required 96101
ConfusingTernary Avoid if (x != y) ..; else ..; 106113
CommentRequired Public method and constructor comments are required 117121
CommentRequired Public method and constructor comments are required 124135
CommentRequired Public method and constructor comments are required 138148
CommentRequired Public method and constructor comments are required 151159
ConfusingTernary Avoid if (x != y) ..; else ..; 165172

fr/paris/lutece/plugins/ctv/controller/front/ConfirmationApp.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34104
CommentRequired Header comments are required 62104
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 70
CommentRequired Public method and constructor comments are required 7581
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 85
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 85
CommentRequired Public method and constructor comments are required 99102

fr/paris/lutece/plugins/ctv/controller/front/DemandeurApp.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34309
TooManyStaticImports Too many static imports may lead to messy code 34309
CommentRequired Header comments are required 92309
TooManyMethods This class has too many methods, consider refactoring it. 93309
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 99
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 100
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 101
ImmutableField Private field 'ctvDemandeUploadHandler' could be made final; it is only initialized in the declaration or constructor. 104
UnusedPrivateField Avoid unused private fields such as 'LOGGER'. 105
ProperLogger Logger should be defined private static final and have the correct class 105
CommentRequired Public method and constructor comments are required 108133
CommentRequired Public method and constructor comments are required 136142
ConfusingTernary Avoid if (x != y) ..; else ..; 146152
CommentRequired Public method and constructor comments are required 156162
CommentRequired Public method and constructor comments are required 165177
CommentRequired Public method and constructor comments are required 180190
CommentRequired Public method and constructor comments are required 193201
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 203
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 266
ConfusingTernary Avoid if (x != y) ..; else ..; 295301

fr/paris/lutece/plugins/ctv/controller/front/DocumentApp.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34300
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34300
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 91300
CommentRequired Header comments are required 91300
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 94
ImmutableField Private field 'ctvDemandeUploadHandler' could be made final; it is only initialized in the declaration or constructor. 95
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 97
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 99
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 100
CommentRequired Public method and constructor comments are required 103122
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 106
UnusedLocalVariable Avoid unused local variables such as 'guid'. 106
CommentRequired Public method and constructor comments are required 185201
CommentRequired Public method and constructor comments are required 204208
CommentRequired Public method and constructor comments are required 211215
CommentRequired Public method and constructor comments are required 218228
CommentRequired Public method and constructor comments are required 231242
CommentRequired Public method and constructor comments are required 245269
ConfusingTernary Avoid if (x != y) ..; else ..; 249267
ShortVariable Avoid variables with short names like id 256
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 271

fr/paris/lutece/plugins/ctv/controller/front/EditionApp.java

Rule Violation Ligne
CommentRequired Header comments are required 71240
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 85
ProperLogger Logger should be defined private static final and have the correct class 86
CommentRequired Public method and constructor comments are required 94102
LinguisticNaming Linguistics Antipattern - The getter 'getPermisStationnement' should not return void linguistically 94102
CloseResource Ensure that resources like this Connection object are closed after use 96
LinguisticNaming Linguistics Antipattern - The getter 'getPermissionVoirie' should not return void linguistically 105113
CommentRequired Public method and constructor comments are required 105113
CloseResource Ensure that resources like this Connection object are closed after use 107
LinguisticNaming Linguistics Antipattern - The getter 'getPermissionAutorisationProlongationOccupation' should not return void linguistically 116125
CommentRequired Public method and constructor comments are required 116125
CloseResource Ensure that resources like this Connection object are closed after use 118
CommentRequired Public method and constructor comments are required 128137
LinguisticNaming Linguistics Antipattern - The getter 'getPermissionAutorisationProlongationIntervention' should not return void linguistically 128137
CloseResource Ensure that resources like this Connection object are closed after use 130
CommentRequired Public method and constructor comments are required 140148
LinguisticNaming Linguistics Antipattern - The getter 'getAutorisationIntervention' should not return void linguistically 140148
CloseResource Ensure that resources like this Connection object are closed after use 142
CommentRequired Public method and constructor comments are required 151159
LinguisticNaming Linguistics Antipattern - The getter 'getAnnulationEmprise' should not return void linguistically 151159
CloseResource Ensure that resources like this Connection object are closed after use 153
LinguisticNaming Linguistics Antipattern - The getter 'getEmprisesPdf' should not return void linguistically 162204
CommentRequired Public method and constructor comments are required 162204
CloseResource Ensure that resources like this Connection object are closed after use 164
LinguisticNaming Linguistics Antipattern - The getter 'getDocumentsAutorisations' should not return void linguistically 207216
CommentRequired Public method and constructor comments are required 207216
AssignmentInOperand Avoid assignments in operands 232

fr/paris/lutece/plugins/ctv/controller/front/EmpriseApp.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34281
TooManyStaticImports Too many static imports may lead to messy code 34281
CommentRequired Header comments are required 91281
TooManyMethods This class has too many methods, consider refactoring it. 92281
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 95
ImmutableField Private field 'empriseCtrlService' could be made final; it is only initialized in the declaration or constructor. 96
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 97
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 98
CommentRequired Public method and constructor comments are required 104121
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 123
CommentRequired Public method and constructor comments are required 124133
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 135
CommentRequired Public method and constructor comments are required 136143
CommentRequired Public method and constructor comments are required 146149
CommentRequired Public method and constructor comments are required 152155
CommentRequired Public method and constructor comments are required 158168
CommentRequired Public method and constructor comments are required 171180
CommentRequired Public method and constructor comments are required 183189
CommentRequired Public method and constructor comments are required 192196
CommentRequired Public method and constructor comments are required 199212
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 204
ConfusingTernary Avoid if (x != y) ..; else ..; 204211
CommentRequired Public method and constructor comments are required 215220
CommentRequired Public method and constructor comments are required 235254
CommentRequired Public method and constructor comments are required 257275

fr/paris/lutece/plugins/ctv/controller/front/ExecutantApp.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34122
CommentRequired Header comments are required 66122
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 66122
ImmutableField Private field 'executantControllerService' could be made final; it is only initialized in the declaration or constructor. 69
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 70
CommentRequired Public method and constructor comments are required 7380
CommentRequired Public method and constructor comments are required 8387
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 89
CommentRequired Public method and constructor comments are required 9094
CommentRequired Public method and constructor comments are required 97107
CommentRequired Public method and constructor comments are required 110120

fr/paris/lutece/plugins/ctv/controller/front/MonEspaceDemandeurApp.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34497
TooManyStaticImports Too many static imports may lead to messy code 34497
CommentRequired Header comments are required 98497
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 105
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 106
ImmutableField Private field 'prolongationService' could be made final; it is only initialized in the declaration or constructor. 107
ImmutableField Private field 'chantierService' could be made final; it is only initialized in the declaration or constructor. 108
UnusedPrivateField Avoid unused private fields such as 'chantierService'. 108
ImmutableField Private field 'demandeFinTravauxService' could be made final; it is only initialized in the declaration or constructor. 110
CommentRequired Public method and constructor comments are required 122215
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 134
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 152
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 152
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 170
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 170
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 184
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 184
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 202
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 202
LinguisticNaming Linguistics Antipattern - The setter 'setFilesFromDemande' should not return any type except void linguistically 217241
UnusedFormalParameter Avoid unused method parameters such as 'request'. 217
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 220
UnusedLocalVariable Avoid unused local variables such as 'demandeNumero'. 223
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 230
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 230237
ShortVariable Avoid variables with short names like d2 233
ShortVariable Avoid variables with short names like d1 233
LinguisticNaming Linguistics Antipattern - The setter 'setFilesFromDemandeReading' should not return any type except void linguistically 243267
UnusedFormalParameter Avoid unused method parameters such as 'request'. 243
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 246
UnusedLocalVariable Avoid unused local variables such as 'demandeNumero'. 249
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 256
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 256263
ShortVariable Avoid variables with short names like d2 259
ShortVariable Avoid variables with short names like d1 259
CommentRequired Public method and constructor comments are required 270275
AvoidDuplicateLiterals The String literal "demande_id" appears 4 times in this file; the first occurrence is on line 272 272
CommentRequired Public method and constructor comments are required 278287
CommentRequired Public method and constructor comments are required 290298
CommentRequired Public method and constructor comments are required 301306
CyclomaticComplexity The method 'prolongation(HttpServletRequest)' has a cyclomatic complexity of 11. 309371
CommentRequired Public method and constructor comments are required 309371
ConfusingTernary Avoid if (x != y) ..; else ..; 332340
ConfusingTernary Avoid if (x != y) ..; else ..; 343351
ConfusingTernary Avoid if (x != y) ..; else ..; 354362
CommentRequired Public method and constructor comments are required 374377
CommentRequired Public method and constructor comments are required 380397
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 383
UnusedLocalVariable Avoid unused local variables such as 'numDemande'. 383
CommentRequired Public method and constructor comments are required 400443
CommentRequired Public method and constructor comments are required 474480
UnusedPrivateMethod Avoid unused private methods such as 'getFileItemList(HttpServletRequest,int)'. 482

fr/paris/lutece/plugins/ctv/controller/front/ValidationApp.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34201
TooManyStaticImports Too many static imports may lead to messy code 34201
CommentRequired Header comments are required 80201
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 87
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 90
ImmutableField Private field 'emailTask' could be made final; it is only initialized in the declaration or constructor. 94
CommentRequired Public method and constructor comments are required 99110
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 102
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 102
CommentRequired Public method and constructor comments are required 113118
CommentRequired Public method and constructor comments are required 121149
CyclomaticComplexity The method 'validationDemande(HttpServletRequest)' has a cyclomatic complexity of 10. 151200
ShortVariable Avoid variables with short names like x 168
ShortVariable Avoid variables with short names like y 169

fr/paris/lutece/plugins/ctv/converter/CustomDateConverter.java

Rule Violation Ligne
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 5457

fr/paris/lutece/plugins/ctv/converter/DoubleConverter.java

Rule Violation Ligne
CommentRequired Header comments are required 3955

fr/paris/lutece/plugins/ctv/form/GestionConflitsFormBean.java

Rule Violation Ligne
CommentRequired Header comments are required 3884
DataClass The class 'GestionConflitsFormBean' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=8, WMC=8) 3884

fr/paris/lutece/plugins/ctv/handler/AbstractCtvUploadHandler.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34474
CommentRequired Header comments are required 73474
TooManyMethods This class has too many methods, consider refactoring it. 74474
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 92
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 93
ProperLogger Logger should be defined private static final and have the correct class 93
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 94
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 95
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 96
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 9899
CallSuperInConstructor It is a good practice to call super() in a constructor 101103
CommentRequired Public method and constructor comments are required 101103
UncommentedEmptyConstructor Document empty constructor 101103
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 101103
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 144
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 144
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 157
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getMapAsynchronousUpload method if you want a default access modifier 186
DefaultPackage Use explicit scoping instead of the default package private level 186
CommentRequired Public method and constructor comments are required 195202
CommentRequired Public method and constructor comments are required 204214
CommentRequired Public method and constructor comments are required 216233
CommentRequired Protected method constructor comments are required 268306
ShortVariable Avoid variables with short names like id 270
ShortVariable Avoid variables with short names like id 308
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 319
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 320
ShortVariable Avoid variables with short names like id 331
CommentRequired Public method and constructor comments are required 389393
ConfusingTernary Avoid if (x != y) ..; else ..; 425432
CommentRequired Public method and constructor comments are required 441452
CommentRequired Public method and constructor comments are required 454458
ConfusingTernary Avoid if (x != y) ..; else ..; 457
CommentRequired Public method and constructor comments are required 462
CommentRequired Public method and constructor comments are required 464

fr/paris/lutece/plugins/ctv/handler/CtvUploadCompteRenduVisiteHandler.java

Rule Violation Ligne
CommentRequired Header comments are required 5298
DefaultPackage Use explicit scoping instead of the default package private level 7175
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getMapAsynchronousUpload method if you want a default access modifier 7275
ShortVariable Avoid variables with short names like id 91

fr/paris/lutece/plugins/ctv/handler/CtvUploadConferenceCoordinationDocumentHandler.java

Rule Violation Ligne
CommentRequired Header comments are required 5198
DefaultPackage Use explicit scoping instead of the default package private level 7074
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getMapAsynchronousUpload method if you want a default access modifier 7174
ShortVariable Avoid variables with short names like id 88

fr/paris/lutece/plugins/ctv/handler/CtvUploadDemandeDocumentHandler.java

Rule Violation Ligne
CommentRequired Header comments are required 52101
DefaultPackage Use explicit scoping instead of the default package private level 7175
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getMapAsynchronousUpload method if you want a default access modifier 7275

fr/paris/lutece/plugins/ctv/handler/CtvUploadEmpriseDocumentHandler.java

Rule Violation Ligne
CommentRequired Header comments are required 52100
MissingOverride The method 'getUploadDirectory()' is missing an @Override annotation. 5962
CommentRequired Public method and constructor comments are required 5962
DefaultPackage Use explicit scoping instead of the default package private level 7074
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getMapAsynchronousUpload method if you want a default access modifier 7174

fr/paris/lutece/plugins/ctv/handler/DeleteFilesOnEndUploadCleaningTracker.java

Rule Violation Ligne
ImmutableField Private field 'filesToDelete' could be made final; it is only initialized in the declaration or constructor. 49
CommentRequired Public method and constructor comments are required 5160
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 6366

fr/paris/lutece/plugins/ctv/mailtask/EmailTask.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34334
CommentRequired Header comments are required 81334
ProperLogger Logger should be defined private static final and have the correct class 84
ImmutableField Private field 'emailConfigService' could be made final; it is only initialized in the declaration or constructor. 91
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 92
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 93
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 94
ImmutableField Private field 'unitUserService' could be made final; it is only initialized in the declaration or constructor. 95
ImmutableField Private field 'infosSuperviseurService' could be made final; it is only initialized in the declaration or constructor. 96
ImmutableField Private field 'conventionService' could be made final; it is only initialized in the declaration or constructor. 98
NPathComplexity The method 'sendEmail(int, int, Locale, HttpServletRequest, int)' has an NPath complexity of 320 101173
CyclomaticComplexity The method 'sendEmail(int, int, Locale, HttpServletRequest, int)' has a cyclomatic complexity of 11. 101173
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 138
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 138
NPathComplexity The method 'processMessageAndSubject(EmailConfig, EntiteDemande, EntiteDemande, Locale, int, HttpServletRequest, int)' has an NPath complexity of 8100 187333
ExcessiveMethodLength Avoid really long methods. 187333
CyclomaticComplexity The method 'processMessageAndSubject(EmailConfig, EntiteDemande, EntiteDemande, Locale, int, HttpServletRequest, int)' has a cyclomatic complexity of 18. 187333
NcssCount The method 'processMessageAndSubject(EmailConfig, EntiteDemande, EntiteDemande, Locale, int, HttpServletRequest, int)' has a NCSS line count of 81. 187333
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 190
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 190
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 191
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 191
ConfusingTernary Avoid if (x != y) ..; else ..; 198217
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 275

fr/paris/lutece/plugins/ctv/mailtask/IEmailTask.java

Rule Violation Ligne
CommentRequired Header comments are required 4045
CommentRequired Public method and constructor comments are required 43

fr/paris/lutece/plugins/ctv/service/AbstractEmpriseControllerService.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34294
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34294
CommentRequired Header comments are required 91294
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 100
CommentRequired Protected method constructor comments are required 140184
ConfusingTernary Avoid if (x != y) ..; else ..; 190201
CommentRequired Protected method constructor comments are required 205215
CommentRequired Protected method constructor comments are required 283290

fr/paris/lutece/plugins/ctv/service/AbstractFormulaireService.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34334
TooManyStaticImports Too many static imports may lead to messy code 34334
CommentRequired Header comments are required 76334
ShortVariable Avoid variables with short names like id 156
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 170
NPathComplexity The method 'putMessagesErrorsForEmprise(Emprise, DetailsEmprise, Map, int)' has an NPath complexity of 58320 184272
CyclomaticComplexity The method 'putMessagesErrorsForEmprise(Emprise, DetailsEmprise, Map, int)' has a cyclomatic complexity of 25. 184272
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 188
CommentRequired Protected method constructor comments are required 286293
CommentRequired Protected method constructor comments are required 295309
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 297
CommentRequired Protected method constructor comments are required 311324
CommentRequired Protected method constructor comments are required 326332

fr/paris/lutece/plugins/ctv/service/AbstractTableauDeBordControllerService.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34297
TooManyStaticImports Too many static imports may lead to messy code 34297
CommentRequired Header comments are required 74297
AvoidDuplicateLiterals The String literal "statut" appears 4 times in this file; the first occurrence is on line 87 87
CommentRequired Protected method constructor comments are required 92109
ExcessiveMethodLength Avoid really long methods. 141262
CyclomaticComplexity The method 'putDemandes(HttpServletRequest, Map, AdminUser)' has a cyclomatic complexity of 22. 141262
NcssCount The method 'putDemandes(HttpServletRequest, Map, AdminUser)' has a NCSS line count of 64. 141262
ConfusingTernary Avoid if (x != y) ..; else ..; 146256
ConfusingTernary Avoid if (x != y) ..; else ..; 160243
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 162
ConfusingTernary Avoid if (x != y) ..; else ..; 166186
ConfusingTernary Avoid if (x != y) ..; else ..; 199208
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 247
LinguisticNaming Linguistics Antipattern - The setter 'setUnitIds' should not return any type except void linguistically 264276
CommentRequired Protected method constructor comments are required 264276
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 266
CommentRequired Protected method constructor comments are required 278
CommentRequired Protected method constructor comments are required 280
CommentRequired Protected method constructor comments are required 282283
CommentRequired Protected method constructor comments are required 285
CommentRequired Protected method constructor comments are required 287
CommentRequired Protected method constructor comments are required 289
CommentRequired Protected method constructor comments are required 291
CommentRequired Protected method constructor comments are required 293
CommentRequired Protected method constructor comments are required 295

fr/paris/lutece/plugins/ctv/service/ChantierControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 60123
AvoidFieldNameMatchingTypeName It is somewhat confusing to have a field name matching the declaring class name 70
CommentRequired Public method and constructor comments are required 7881

fr/paris/lutece/plugins/ctv/service/ChantierIntentionControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 58119
CommentRequired Public method and constructor comments are required 7376

fr/paris/lutece/plugins/ctv/service/ChantierInterventionControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 56111
CommentRequired Public method and constructor comments are required 7073

fr/paris/lutece/plugins/ctv/service/CtvMapService.java

Rule Violation Ligne
ImmutableField Private field 'empriseControllerService' could be made final; it is only initialized in the declaration or constructor. 79
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 81
CyclomaticComplexity The method 'getMapTemplate(HttpServletRequest, int)' has a cyclomatic complexity of 12. 122201
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 124
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 124
AvoidDuplicateLiterals The String literal "edit" appears 4 times in this file; the first occurrence is on line 129 129
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 129
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 160163
ConfusingTernary Avoid if (x != y) ..; else ..; 176196
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 219
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'colorCode' 220
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 225
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 225
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 317

fr/paris/lutece/plugins/ctv/service/CtvProfileService.java

Rule Violation Ligne
CommentRequired Header comments are required 50210
ImmutableField Private field '_profilesService' could be made final; it is only initialized in the declaration or constructor. 53
CommentRequired Public method and constructor comments are required 5760
CommentRequired Header comments are required 6283
CommentRequired Public method and constructor comments are required 6569
ShortVariable Avoid variables with short names like o 72
UselessOverridingMethod Overriding method merely calls super 7881
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'hasProfile' 90
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 99
AvoidDuplicateLiterals The String literal "profile_default" appears 5 times in this file; the first occurrence is on line 104 104
ConfusingTernary Avoid if (x != y) ..; else ..; 104111
CyclomaticComplexity The method 'selectDefaultProfileByPriority(HttpServletRequest, List)' has a cyclomatic complexity of 11. 116165
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 119

fr/paris/lutece/plugins/ctv/service/EditionServiceImpl.java

Rule Violation Ligne
CommentRequired Header comments are required 52187
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 80
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 80
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 91
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 91
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 127
UseTryWithResources Consider using a try-with-resources statement instead of explicitly closing the resource 137158
ShortVariable Avoid variables with short names like e 161
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 162
ConfusingTernary Avoid if (x != y) ..; else ..; 164168

fr/paris/lutece/plugins/ctv/service/EmpriseControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 57129
UnusedPrivateField Avoid unused private fields such as 'referentielsService'. 63
CommentRequired Public method and constructor comments are required 6669

fr/paris/lutece/plugins/ctv/service/EmpriseIntentionControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 59146
CommentRequired Public method and constructor comments are required 6871
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 88

fr/paris/lutece/plugins/ctv/service/EmpriseInterventionControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 4794
CommentRequired Public method and constructor comments are required 5457
UncommentedEmptyMethodBody Document empty method body 7475

fr/paris/lutece/plugins/ctv/service/EntiteControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 4663

fr/paris/lutece/plugins/ctv/service/ExecutantControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 52100
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 55
CommentRequired Public method and constructor comments are required 6568

fr/paris/lutece/plugins/ctv/service/ExecutantInterventionControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 52104
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 59
CommentRequired Public method and constructor comments are required 6568

fr/paris/lutece/plugins/ctv/service/FluxDonneesEmprisesTaxationSavoi2.java

Rule Violation Ligne
CommentRequired Header comments are required 1670
ImmutableField Private field 'fluxService' could be made final; it is only initialized in the declaration or constructor. 18
SingularField Perhaps 'file' could be replaced by a local variable. 19
CommentRequired Public method and constructor comments are required 2968
ShortVariable Avoid variables with short names like df 32
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 32
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 100 characters appended. 35
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 49
AvoidPrintStackTrace Avoid printStackTrace(); use a logger call instead. 61
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 62

fr/paris/lutece/plugins/ctv/service/FormulaireDemandeIntentionServiceImpl.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34332
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34332
CommentRequired Header comments are required 95332
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 101
UncommentedEmptyConstructor Document empty constructor 105107
CallSuperInConstructor It is a good practice to call super() in a constructor 105107
CommentRequired Protected method constructor comments are required 105107
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 109
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 111112
UnusedPrivateField Avoid unused private fields such as 'ctvUploadEmpriseDocumentHandler'. 111
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 111
CommentRequired Public method and constructor comments are required 114117
CyclomaticComplexity The method 'alimentationRecapitulatif(HttpServletRequest, Map)' has a cyclomatic complexity of 11. 223300
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 226
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 227
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 227
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 228
ShortVariable Avoid variables with short names like id 229
ConfusingTernary Avoid if (x != y) ..; else ..; 233
ConfusingTernary Avoid if (x != y) ..; else ..; 238
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 241
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 242
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 316
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 318
ConfusingTernary Avoid if (x != y) ..; else ..; 324328

fr/paris/lutece/plugins/ctv/service/FormulaireDemandeInterventionServiceImpl.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34279
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34279
CommentRequired Header comments are required 88279
CommentRequired Protected method constructor comments are required 98100
UncommentedEmptyConstructor Document empty constructor 98100
CallSuperInConstructor It is a good practice to call super() in a constructor 98100
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 102
CommentRequired Public method and constructor comments are required 104107
NPathComplexity The method 'alimentationRecapitulatif(HttpServletRequest, Map)' has an NPath complexity of 264 150233
CyclomaticComplexity The method 'alimentationRecapitulatif(HttpServletRequest, Map)' has a cyclomatic complexity of 12. 150233
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 153
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 154
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 154
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 155
ShortVariable Avoid variables with short names like id 157
ConfusingTernary Avoid if (x != y) ..; else ..; 161
ConfusingTernary Avoid if (x != y) ..; else ..; 166
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 169
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 171
ConfusingTernary Avoid if (x != y) ..; else ..; 219
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 244
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 246
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 262
ConfusingTernary Avoid if (x != y) ..; else ..; 265269
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 275

fr/paris/lutece/plugins/ctv/service/FormulaireDemandeOccupationBoServiceImpl.java

Rule Violation Ligne
CommentRequired Header comments are required 56125
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 73
CommentRequired Public method and constructor comments are required 7578
ConfusingTernary Avoid if (x != y) ..; else ..; 8598
ConfusingTernary Avoid if (x != y) ..; else ..; 9198

fr/paris/lutece/plugins/ctv/service/FormulaireDemandeOccupationServiceImpl.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34248
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34248
CommentRequired Header comments are required 84248
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 89
UnusedPrivateField Avoid unused private fields such as 'ctvUploadEmpriseDocumentHandler'. 89
CommentRequired Protected method constructor comments are required 9294
CallSuperInConstructor It is a good practice to call super() in a constructor 9294
UncommentedEmptyConstructor Document empty constructor 9294
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 96
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 98
CommentRequired Public method and constructor comments are required 100103
CyclomaticComplexity The method 'alimentationRecapitulatif(HttpServletRequest, Map)' has a cyclomatic complexity of 11. 128195
NPathComplexity The method 'alimentationRecapitulatif(HttpServletRequest, Map)' has an NPath complexity of 240 128195
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 131
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 132
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 132
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 133
ConfusingTernary Avoid if (x != y) ..; else ..; 138
ConfusingTernary Avoid if (x != y) ..; else ..; 143
ConfusingTernary Avoid if (x != y) ..; else ..; 147
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 151
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 153
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 206
NPathComplexity The method 'controlRequiredFiles(HttpServletRequest, HashMap, ChantierOccupation)' has an NPath complexity of 210 206246
CyclomaticComplexity The method 'controlRequiredFiles(HttpServletRequest, HashMap, ChantierOccupation)' has a cyclomatic complexity of 14. 206246
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 208
ConfusingTernary Avoid if (x != y) ..; else ..; 221225
ConfusingTernary Avoid if (x != y) ..; else ..; 233240

fr/paris/lutece/plugins/ctv/service/IChantierControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 4279

fr/paris/lutece/plugins/ctv/service/ICtvProfileService.java

Rule Violation Ligne
CommentRequired Header comments are required 4457
CommentRequired Public method and constructor comments are required 47
CommentRequired Public method and constructor comments are required 49
CommentRequired Public method and constructor comments are required 51
CommentRequired Public method and constructor comments are required 53
CommentRequired Public method and constructor comments are required 55

fr/paris/lutece/plugins/ctv/service/IEditionService.java

Rule Violation Ligne
CommentRequired Header comments are required 3962
CommentRequired Public method and constructor comments are required 42
CommentRequired Public method and constructor comments are required 44
CommentRequired Public method and constructor comments are required 46
CommentRequired Public method and constructor comments are required 48
CommentRequired Public method and constructor comments are required 50
CommentRequired Public method and constructor comments are required 52
CommentRequired Public method and constructor comments are required 54
CommentRequired Public method and constructor comments are required 56
CommentRequired Public method and constructor comments are required 58
CommentRequired Public method and constructor comments are required 60

fr/paris/lutece/plugins/ctv/service/IEmpriseControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 4072
CommentRequired Public method and constructor comments are required 64
CommentRequired Public method and constructor comments are required 66
CommentRequired Public method and constructor comments are required 68
CommentRequired Public method and constructor comments are required 70

fr/paris/lutece/plugins/ctv/service/IEntiteControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 4044
CommentRequired Public method and constructor comments are required 43

fr/paris/lutece/plugins/ctv/service/IExecutantControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 4070

fr/paris/lutece/plugins/ctv/service/IFormulaire.java

Rule Violation Ligne
CommentRequired Header comments are required 4563
CommentRequired Public method and constructor comments are required 48
CommentRequired Public method and constructor comments are required 50
CommentRequired Public method and constructor comments are required 52
CommentRequired Public method and constructor comments are required 54
CommentRequired Public method and constructor comments are required 56
CommentRequired Public method and constructor comments are required 58
CommentRequired Public method and constructor comments are required 6061

fr/paris/lutece/plugins/ctv/service/IFormulaireDemandeOccupationBoService.java

Rule Violation Ligne
CommentRequired Header comments are required 4045
CommentRequired Public method and constructor comments are required 43

fr/paris/lutece/plugins/ctv/service/ITableauDeBordControllerService.java

Rule Violation Ligne
CommentRequired Header comments are required 4043
CommentRequired Public method and constructor comments are required 42

fr/paris/lutece/plugins/ctv/service/PluginCite.java

Rule Violation Ligne
CommentRequired Header comments are required 613

fr/paris/lutece/plugins/ctv/service/StatutsEmprisesDaemon.java

Rule Violation Ligne
CommentRequired Header comments are required 4797
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 78
AvoidDuplicateLiterals The String literal " lignes." appears 4 times in this file; the first occurrence is on line 78 78
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 78

fr/paris/lutece/plugins/ctv/service/TableauDeBordConcessionnaireControllerService.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34512
TooManyStaticImports Too many static imports may lead to messy code 34512
CommentRequired Header comments are required 81512
TooManyMethods This class has too many methods, consider refactoring it. 82512
AvoidFieldNameMatchingTypeName It is somewhat confusing to have a field name matching the declaring class name 90
CommentRequired Public method and constructor comments are required 9295
CallSuperInConstructor It is a good practice to call super() in a constructor 9799
AvoidDuplicateLiterals The String literal "profile_default" appears 4 times in this file; the first occurrence is on line 161 161
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 310
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 310
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 326
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 326
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 348
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 348
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 366
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 366
ExcessiveMethodLength Avoid really long methods. 383483
CyclomaticComplexity The method 'putDemandes(HttpServletRequest, Map, AdminUser)' has a cyclomatic complexity of 15. 383483
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 386
ConfusingTernary Avoid if (x != y) ..; else ..; 388476
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 400
ConfusingTernary Avoid if (x != y) ..; else ..; 402470
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 407414
ConfusingTernary Avoid if (x != y) ..; else ..; 427436

fr/paris/lutece/plugins/ctv/service/TableauDeBordInstructionControllerService.java

Rule Violation Ligne
TooManyStaticImports Too many static imports may lead to messy code 34576
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34576
CommentRequired Header comments are required 88576
TooManyMethods This class has too many methods, consider refactoring it. 89576
AvoidFieldNameMatchingTypeName It is somewhat confusing to have a field name matching the declaring class name 93
CommentRequired Public method and constructor comments are required 9598
CallSuperInConstructor It is a good practice to call super() in a constructor 100102
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 122
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 122
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 170
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 170
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 194
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 194
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 232
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 232
UnusedFormalParameter Avoid unused method parameters such as 'userType'. 326
LinguisticNaming Linguistics Antipattern - The getter 'getCompteurFromProfileEnCours' should not return void linguistically 424433
LinguisticNaming Linguistics Antipattern - The getter 'getCompteurFromProfileATraiter' should not return void linguistically 435444
CyclomaticComplexity The method 'getStatusByBlocs(SearchBeanForEmprises, HttpServletRequest, List, AdminUser, String)' has a cyclomatic complexity of 11. 453534
NcssCount The method 'getStatusByBlocs(SearchBeanForEmprises, HttpServletRequest, List, AdminUser, String)' has a NCSS line count of 63. 453534
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 545

fr/paris/lutece/plugins/ctv/service/fileimage/CiteFileImageService.java

Rule Violation Ligne
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 6871
MissingOverride The method 'getImageResource(int)' is missing an @Override annotation. 98117
MissingOverride The method 'getResourceTypeId()' is missing an @Override annotation. 124127

fr/paris/lutece/plugins/ctv/util/EditionUtils.java

Rule Violation Ligne
CommentRequired Header comments are required 5392
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. 5492
ProperLogger Logger should be defined private static final and have the correct class 56
CommentRequired Public method and constructor comments are required 5891

fr/paris/lutece/plugins/ctv/util/EmpriseMappingUtils.java

Rule Violation Ligne
CommentRequired Header comments are required 51153
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. 52153
ProperLogger Logger should be defined private static final and have the correct class 54
CommentRequired Public method and constructor comments are required 56103
CyclomaticComplexity The method 'map(HttpServletRequest, DetailsEmprise)' has a cyclomatic complexity of 12. 56103
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 67
ShortVariable Avoid variables with short names like e 73
ShortVariable Avoid variables with short names like e 93

fr/paris/lutece/plugins/ctv/util/GestionPDFUtil.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34407
GodClass Possible God Class (WMC=55, ATFD=58, TCC=20.000%) 69407
CommentRequired Header comments are required 69407
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. 70407
RedundantFieldInitializer Avoid using redundant field initializer for 'nbDocToGen' 88
CommentRequired Public method and constructor comments are required 90130
CyclomaticComplexity The method 'gestionPDF(DemandeReading, String, String, String)' has a cyclomatic complexity of 10. 90130
ConfusingTernary Avoid if (x != y) ..; else ..; 102
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 105
CyclomaticComplexity The method 'generationAutorisations(int, Enum, String, String, String)' has a cyclomatic complexity of 14. 132209
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 134
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 134
CloseResource Ensure that resources like this Connection object are closed after use 135
PositionLiteralsFirstInComparisons Position literals first in String comparisons 214
SimplifyBooleanReturns Avoid unnecessary if..then..else statements when returning booleans 216223
CyclomaticComplexity The method 'writeEditionsOnServer(Map, DemandeReading, String, String)' has a cyclomatic complexity of 23. 231380
NcssCount The method 'writeEditionsOnServer(Map, DemandeReading, String, String)' has a NCSS line count of 99. 231380
ExcessiveMethodLength Avoid really long methods. 231380
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 237
PositionLiteralsFirstInComparisons Position literals first in String comparisons 260
AvoidDuplicateLiterals The String literal ".pdf" appears 8 times in this file; the first occurrence is on line 273 273
AvoidDuplicateLiterals The String literal "-N_" appears 8 times in this file; the first occurrence is on line 273 273
PositionLiteralsFirstInComparisons Position literals first in String comparisons 298
PositionLiteralsFirstInComparisons Position literals first in String comparisons 337
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 396
SimpleDateFormatNeedsLocale When instantiating a SimpleDateFormat object, specify a Locale 397

fr/paris/lutece/plugins/ctv/util/RemoteFilesUtils.java

Rule Violation Ligne
CommentRequired Header comments are required 58219
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. 59219
LinguisticNaming Linguistics Antipattern - The getter 'getRemoteFilesInLocalDirectory' should not return void linguistically 69101
CommentRequired Public method and constructor comments are required 69101
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 77
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 83
EmptyCatchBlock Avoid empty catch blocks 9799
UseTryWithResources Consider using a try-with-resources statement instead of explicitly closing the resource 106123
EmptyCatchBlock Avoid empty catch blocks 113115
EmptyCatchBlock Avoid empty catch blocks 116118
IdenticalCatchBranches 'catch' branch identical to 'FileNotFoundException' branch 116118
CommentRequired Public method and constructor comments are required 126168
UseTryWithResources Consider using a try-with-resources statement instead of explicitly closing the resource 135164
EmptyCatchBlock Avoid empty catch blocks 158160
EmptyCatchBlock Avoid empty catch blocks 179181
EmptyCatchBlock Avoid empty catch blocks 196198
CommentRequired Public method and constructor comments are required 202217
EmptyCatchBlock Avoid empty catch blocks 213215

fr/paris/lutece/plugins/ctv/util/SslContextSecureProtocolSocketFactory.java

Rule Violation Ligne
CommentRequired Header comments are required 59242
AvoidFieldNameMatchingMethodName Field verifyHostname has the same name as a method 63
CommentRequired Public method and constructor comments are required 6570
CommentRequired Public method and constructor comments are required 7275
CommentRequired Public method and constructor comments are required 7780
CommentRequired Public method and constructor comments are required 8285
CommentRequired Public method and constructor comments are required 8790
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 8790
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 9295
CommentRequired Public method and constructor comments are required 9295
MissingOverride The method 'createSocket(String, int, InetAddress, int)' is missing an @Override annotation. 97103
CommentRequired Public method and constructor comments are required 97103
ShortVariable Avoid variables with short names like sf 99
MissingOverride The method 'createSocket(String, int, InetAddress, int, HttpConnectionParams)' is missing an @Override annotation. 105133
CommentRequired Public method and constructor comments are required 105133
MissingOverride The method 'createSocket(String, int)' is missing an @Override annotation. 135141
CommentRequired Public method and constructor comments are required 135141
ShortVariable Avoid variables with short names like sf 137
MissingOverride The method 'createSocket(Socket, String, int, boolean)' is missing an @Override annotation. 143149
CommentRequired Public method and constructor comments are required 143149
ShortVariable Avoid variables with short names like sf 145
CyclomaticComplexity The method 'verifyHostname(SSLSocket)' has a cyclomatic complexity of 11. 151200
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 170
ConfusingTernary Avoid if (x != y) ..; else ..; 174199
AvoidDollarSigns Avoid using dollar signs in variable/method/class/interface names 179
ShortVariable Avoid variables with short names like i$ 179
ShortVariable Avoid variables with short names like cn 183
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 184
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 184
ShortVariable Avoid variables with short names like st 205
CommentRequired Protected method constructor comments are required 219236
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 238241

fr/paris/lutece/plugins/ctv/util/UnitTreeUtils.java

Rule Violation Ligne
CommentRequired Header comments are required 46140
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. 47140
CommentRequired Public method and constructor comments are required 5156
CommentRequired Public method and constructor comments are required 5861
CommentRequired Public method and constructor comments are required 6381
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 65
ConfusingTernary Avoid if (x != y) ..; else ..; 65
ShortVariable Avoid variables with short names like id 93
CollapsibleIfStatements These nested if statements could be combined 130133

fr/paris/lutece/plugins/ctv/util/UserInclude.java

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

fr/paris/lutece/plugins/ctv/util/Utils.java

Rule Violation Ligne
CommentRequired Header comments are required 60649
CyclomaticComplexity The class 'Utils' has a total cyclomatic complexity of 115 (highest 14). 60649
GodClass Possible God Class (WMC=115, ATFD=132, TCC=0.000%) 60649
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. 61649
ProperLogger Logger should be defined private static final and have the correct class 66
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 68
CommentRequired Public method and constructor comments are required 68113
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 72
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 74
ConfusingTernary Avoid if (x != y) ..; else ..; 8793
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 87
ConfusingTernary Avoid if (x != y) ..; else ..; 98104
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 98
CommentRequired Public method and constructor comments are required 115167
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 115
CyclomaticComplexity The method 'controlRequiredFieldsIntExecutant(String, Object, HashMap)' has a cyclomatic complexity of 13. 115167
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 119
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 121
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 133
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 133
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 133
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 134
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 134
ConfusingTernary Avoid if (x != y) ..; else ..; 140146
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 140
ConfusingTernary Avoid if (x != y) ..; else ..; 151157
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 151
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 169
CyclomaticComplexity The method 'controlRequiredFieldsIntDemandeur(String, Object, HashMap)' has a cyclomatic complexity of 11. 169221
CommentRequired Public method and constructor comments are required 169221
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 173
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 175
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 187
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 187
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 187
ConfusingTernary Avoid if (x != y) ..; else ..; 194200
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 194
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 205
ConfusingTernary Avoid if (x != y) ..; else ..; 205211
CommentRequired Public method and constructor comments are required 223273
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 223
CyclomaticComplexity The method 'controlRequiredFieldsOccupationExecutant(String, Object, HashMap)' has a cyclomatic complexity of 11. 223273
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 227
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 229
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 238
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 238
UseEqualsToCompareStrings Use equals() to compare strings instead of '==' or '!=' 238
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 246
ConfusingTernary Avoid if (x != y) ..; else ..; 246252
ConfusingTernary Avoid if (x != y) ..; else ..; 257263
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 257
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 275
CommentRequired Public method and constructor comments are required 275322
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 279
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 281
ConfusingTernary Avoid if (x != y) ..; else ..; 295301
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 295
ConfusingTernary Avoid if (x != y) ..; else ..; 306312
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 306
NPathComplexity The method 'controlConditionnal(Object, List, Class, Field, String, Class)' has an NPath complexity of 360 324381
CyclomaticComplexity The method 'controlConditionnal(Object, List, Class, Field, String, Class)' has a cyclomatic complexity of 14. 324381
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 360
CyclomaticComplexity The method 'control(Object, Field, List, Class)' has a cyclomatic complexity of 11. 383422
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 436
UseCorrectExceptionLogging Use the correct logging statement for logging exceptions 460
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 482
CommentRequired Public method and constructor comments are required 512557
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 521
CommentRequired Public method and constructor comments are required 559579
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 559
CommentRequired Public method and constructor comments are required 599602

fr/paris/lutece/plugins/ctv/util/sort/CustomAttributeComparator.java

Rule Violation Ligne
CommentRequired Header comments are required 46226
ImmutableField Private field 'sortedAttribute' could be made final; it is only initialized in the declaration or constructor. 50
ImmutableField Private field 'isASC' could be made final; it is only initialized in the declaration or constructor. 51
CyclomaticComplexity The method 'compare(Object, Object)' has a cyclomatic complexity of 13. 89154
ShortVariable Avoid variables with short names like o1 89
ShortVariable Avoid variables with short names like o2 89
PositionLiteralsFirstInComparisons Position literals first in String comparisons 125
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 127
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 127
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 166
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 181
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 188

fr/paris/lutece/plugins/ctv/web/EmpriseImageServlet.java

Rule Violation Ligne
CommentRequired Header comments are required 2162

fr/paris/lutece/plugins/ctv/web/bean/BeanGisEntry.java

Rule Violation Ligne
CommentRequired Header comments are required 3656
CommentRequired Public method and constructor comments are required 4144

fr/paris/lutece/plugins/ctv/web/bean/BlocLabelWithDate.java

Rule Violation Ligne
CommentRequired Header comments are required 3883
DataClass The class 'BlocLabelWithDate' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=6, WMC=7) 3883
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 date field if you want a default access modifier 42
DefaultPackage Use explicit scoping instead of the default package private level 43
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the dateToUpdate field if you want a default access modifier 43
CommentRequired Public method and constructor comments are required 4551

fr/paris/lutece/plugins/ctv/web/bean/DemandeWithFiles.java

Rule Violation Ligne
CommentRequired Header comments are required 4258

Priority 4

fr/paris/lutece/plugins/ctv/bo/controller/conference/coordination/ConferenceCoordinationJspBean.java

Rule Violation Ligne
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 377

fr/paris/lutece/plugins/ctv/bo/controller/demandes/AbstractSuperviseurJpsBean.java

Rule Violation Ligne
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.ROC_PV' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.ROC_PV' 428
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.ROC_PV' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.ROC_PV' 428
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.ROC_AUTRES' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.ROC_AUTRES' 429
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.ROC_AUTRES' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.ROC_AUTRES' 429

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/DemandeurJspBean.java

Rule Violation Ligne
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.JUSTIFICATIFS_CONVENTION' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.JUSTIFICATIFS_CONVENTION' 94
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.JUSTIFICATIFS_CONVENTION' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.JUSTIFICATIFS_CONVENTION' 94

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DemandeurJspBean.java

Rule Violation Ligne
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.JUSTIFICATIFS_CONVENTION' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.JUSTIFICATIFS_CONVENTION' 152
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.JUSTIFICATIFS_CONVENTION' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.JUSTIFICATIFS_CONVENTION' 152

fr/paris/lutece/plugins/ctv/controller/front/MonEspaceDemandeurApp.java

Rule Violation Ligne
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'CtvConstants.SESSION_NUM_DEMANDE_ID' due to existing static import 'fr.paris.lutece.plugins.ctv.constants.CtvConstants.SESSION_NUM_DEMANDE_ID' 485
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'CtvConstants.SESSION_NUM_DEMANDE_ID' due to existing static import 'fr.paris.lutece.plugins.ctv.constants.CtvConstants.SESSION_NUM_DEMANDE_ID' 492

fr/paris/lutece/plugins/ctv/service/AbstractTableauDeBordControllerService.java

Rule Violation Ligne
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'TypeProfileCtv.SUPERVISEUR_2' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.TypeProfileCtv.SUPERVISEUR_2' 170
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'TypeProfileCtv.SUPERVISEUR_1' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.TypeProfileCtv.SUPERVISEUR_1' 170

fr/paris/lutece/plugins/ctv/service/TableauDeBordConcessionnaireControllerService.java

Rule Violation Ligne
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 240

fr/paris/lutece/plugins/ctv/service/TableauDeBordInstructionControllerService.java

Rule Violation Ligne
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'TypeDemande.INTENTION_TRAVAUX' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.TypeDemande.INTENTION_TRAVAUX' 176
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'TypeDemande.BARRAGE_US' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.TypeDemande.BARRAGE_US' 176
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'TypeDemande.TRAVAUX_IMPERATIFS_NON_PROGRAMMES' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.TypeDemande.TRAVAUX_IMPERATIFS_NON_PROGRAMMES' 177
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'TypeProfileCtv.INSTRUCTEUR' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.TypeProfileCtv.INSTRUCTEUR' 186
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'StatutEmprise.A_COORDONNEE' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.StatutEmprise.A_COORDONNEE' 335
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'StatutEmprise.A_TRAITER' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.StatutEmprise.A_TRAITER' 335
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'TypeProfileCtv.INSTRUCTEUR' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.TypeProfileCtv.INSTRUCTEUR' 476
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'TypeProfileCtv.INSTRUCTEUR' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.TypeProfileCtv.INSTRUCTEUR' 481
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'TypeProfileCtv.INSTRUCTEUR' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.TypeProfileCtv.INSTRUCTEUR' 487

fr/paris/lutece/plugins/ctv/util/SslContextSecureProtocolSocketFactory.java

Rule Violation Ligne
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 9295

Fichiers

fr/paris/lutece/plugins/ctv/authentification/MokeLuteceAuthentication.java

Rule Violation Priority Ligne
UnnecessaryConstructor Avoid unnecessary constructors - the compiler will generate these for you 3 5052
UncommentedEmptyConstructor Document empty constructor 3 5052
MissingOverride The method 'logout(LuteceUser)' is missing an @Override annotation. 3 7577
CommentRequired Public method and constructor comments are required 3 7577
UncommentedEmptyMethodBody Document empty method body 3 7677
MissingOverride The method 'isUserInRole(LuteceUser, HttpServletRequest, String)' is missing an @Override annotation. 3 9194
CommentRequired Public method and constructor comments are required 3 9194
ReturnEmptyArrayRatherThanNull Return an empty array rather than null. 1 9699
MissingOverride The method 'getRolesByUser(LuteceUser)' is missing an @Override annotation. 3 9699
CommentRequired Public method and constructor comments are required 3 9699
MissingOverride The method 'updateDateLastLogin(LuteceUser, HttpServletRequest)' is missing an @Override annotation. 3 215217
CommentRequired Public method and constructor comments are required 3 215217
UncommentedEmptyMethodBody Document empty method body 3 216217

fr/paris/lutece/plugins/ctv/bo/controller/conference/coordination/ConferenceCoordinationJspBean.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34408
CommentRequired Header comments are required 3 77408
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 80
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 80
FieldNamingConventions The static field name 'conferenceService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 82
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 83
CommentRequired Public method and constructor comments are required 3 9195
CommentRequired Public method and constructor comments are required 3 98113
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 123
NPathComplexity The method 'putDemandes(HttpServletRequest, Map)' has an NPath complexity of 361 3 193307
ExcessiveMethodLength Avoid really long methods. 3 193307
NcssCount The method 'putDemandes(HttpServletRequest, Map)' has a NCSS line count of 62. 3 193307
CyclomaticComplexity The method 'putDemandes(HttpServletRequest, Map)' has a cyclomatic complexity of 15. 3 193307
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 200
LocalVariableNamingConventions The local variable name 'TypeDemandeOccupationEtIntervention' doesn't match '[a-z][a-zA-Z0-9]*' 1 317
FormalParameterNamingConventions The method parameter name 'DemandeType' doesn't match '[a-z][a-zA-Z0-9]*' 1 377
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 377
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'typesDemande' 3 385
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'statutEmprise' 3 397
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'statutDemande' 3 405

fr/paris/lutece/plugins/ctv/bo/controller/conference/coordination/GestionConferenceCoordinationJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 63195
FieldNamingConventions The field name 'conferenceService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 67
ImmutableField Private field 'conferenceService' could be made final; it is only initialized in the declaration or constructor. 3 67
FieldNamingConventions The constant name 'ctvConferenceCoordinationUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 1 68
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 71
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 71
CommentRequired Public method and constructor comments are required 3 74115
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 102
AvoidDuplicateLiterals The String literal "conferenceDocument" appears 6 times in this file; the first occurrence is on line 110 3 110
CommentRequired Public method and constructor comments are required 3 118153
ConfusingTernary Avoid if (x != y) ..; else ..; 3 131147
CommentRequired Public method and constructor comments are required 3 156164
CommentRequired Public method and constructor comments are required 3 167173
CommentRequired Public method and constructor comments are required 3 176186

fr/paris/lutece/plugins/ctv/bo/controller/conventions/GestionConventionJspBean.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34240
CommentRequired Header comments are required 3 74240
FieldNamingConventions The field name 'conventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 78
ImmutableField Private field 'conventionService' could be made final; it is only initialized in the declaration or constructor. 3 78
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 3 79
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 79
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 3 80
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 80
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 81
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 3 81
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 82
ImmutableField Private field 'ctvDemandeUploadHandler' could be made final; it is only initialized in the declaration or constructor. 3 82
FieldNamingConventions The field name 'emailTask' doesn't match '_[a-z][a-zA-Z0-9]*' 1 84
ImmutableField Private field 'emailTask' could be made final; it is only initialized in the declaration or constructor. 3 84
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the validationConvention field if you want a default access modifier 3 86
FieldNamingConventions The field name 'validationConvention' doesn't match '_[a-z][a-zA-Z0-9]*' 1 86
DefaultPackage Use explicit scoping instead of the default package private level 3 86
CommentRequired Public method and constructor comments are required 3 89107
CommentRequired Public method and constructor comments are required 3 110130
CommentRequired Public method and constructor comments are required 3 133151
CommentRequired Public method and constructor comments are required 3 154191

fr/paris/lutece/plugins/ctv/bo/controller/demandes/AbstractCtvJspBean.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34322
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 148
FieldNamingConventions The field name 'ctvProfileService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 149
FieldNamingConventions The constant name 'actionProfilStatutService' doesn't match '[A-Z][A-Z_0-9]*' 1 150
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 151
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 151
FieldNamingConventions The static field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 152
FieldNamingConventions The static field name 'alerteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 153
FieldNamingConventions The constant name 'infosSuperviseurService' doesn't match '[A-Z][A-Z_0-9]*' 1 154
CommentRequired Protected method constructor comments are required 3 193200
NPathComplexity The method 'fillUserPossibleActions(HttpServletRequest, Map, int)' has an NPath complexity of 288 3 207253
CyclomaticComplexity The method 'fillUserPossibleActions(HttpServletRequest, Map, int)' has a cyclomatic complexity of 18. 3 207253
CommentRequired Protected method constructor comments are required 3 255269
CommentRequired Public method and constructor comments are required 3 271279
CommentRequired Protected method constructor comments are required 3 281294
CommentRequired Public method and constructor comments are required 3 296315
LocalVariableNamingConventions The local variable name 'EmpriseId' doesn't match '[a-z][a-zA-Z0-9]*' 1 303
CommentRequired Protected method constructor comments are required 3 317320

fr/paris/lutece/plugins/ctv/bo/controller/demandes/AbstractGestionConflitsJspBean.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34219
CommentRequired Header comments are required 3 64219
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 3 86
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 86
FieldNamingConventions The constant name 'conflitService' doesn't match '[A-Z][A-Z_0-9]*' 1 87
FieldNamingConventions The field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 88
CommentRequired Protected method constructor comments are required 3 95
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 97
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 97
CommentRequired Protected method constructor comments are required 3 99
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 120
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 121
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 122
CommentRequired Protected method constructor comments are required 3 178186
CommentRequired Protected method constructor comments are required 3 188202
CommentRequired Protected method constructor comments are required 3 204209
CommentRequired Protected method constructor comments are required 3 211216
CommentRequired Protected method constructor comments are required 3 218

fr/paris/lutece/plugins/ctv/bo/controller/demandes/AbstractSuperviseurJpsBean.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34537
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34537
GodClass Possible God Class (WMC=55, ATFD=94, TCC=8.974%) 3 91537
CommentRequired Header comments are required 3 91537
TooManyMethods This class has too many methods, consider refactoring it. 3 92537
FieldNamingConventions The constant name 'emailTask' doesn't match '[A-Z][A-Z_0-9]*' 1 120
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 122
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 3 122
FieldNamingConventions The constant name 'infosSuperviseurService' doesn't match '[A-Z][A-Z_0-9]*' 1 124
CommentRequired Protected method constructor comments are required 3 126
CommentRequired Protected method constructor comments are required 3 128
FieldNamingConventions The field name 'saisineService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 130
ImmutableField Private field 'saisineService' could be made final; it is only initialized in the declaration or constructor. 3 130
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 130
ImmutableField Private field 'conflitService' could be made final; it is only initialized in the declaration or constructor. 3 131
FieldNamingConventions The field name 'conflitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 131
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 131
FieldNamingConventions The constant name 'unitService' doesn't match '[A-Z][A-Z_0-9]*' 1 132
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 132
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 134
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 134
ProperLogger Logger should be defined private static final and have the correct class 3 136
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 136
CommentRequired Protected method constructor comments are required 3 138163
CyclomaticComplexity The method 'doSave(HttpServletRequest)' has a cyclomatic complexity of 17. 3 165247
NPathComplexity The method 'doSave(HttpServletRequest)' has an NPath complexity of 2016 3 165247
CommentRequired Protected method constructor comments are required 3 165247
CommentRequired Protected method constructor comments are required 3 249316
CommentRequired Protected method constructor comments are required 3 327352
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 329
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 329
ConfusingTernary Avoid if (x != y) ..; else ..; 3 339347
CommentRequired Protected method constructor comments are required 3 367383
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 413
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.ROC_PV' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.ROC_PV' 4 428
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.ROC_PV' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.ROC_PV' 4 428
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.ROC_AUTRES' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.ROC_AUTRES' 4 429
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.ROC_AUTRES' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.ROC_AUTRES' 4 429
CommentRequired Protected method constructor comments are required 3 443477
ConfusingTernary Avoid if (x != y) ..; else ..; 3 458474
CommentRequired Protected method constructor comments are required 3 479487
CommentRequired Protected method constructor comments are required 3 489510
MissingBreakInSwitch A switch statement does not contain a break 3 497507
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 506
CommentRequired Protected method constructor comments are required 3 512535

fr/paris/lutece/plugins/ctv/bo/controller/demandes/ConsultationCRVisiteJspBean.java

Rule Violation Priority Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 60128
CommentRequired Header comments are required 3 60128
FieldNamingConventions The field name 'ctvUploadCompteRenduVisiteHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 67
ImmutableField Private field 'ctvUploadCompteRenduVisiteHandler' could be made final; it is only initialized in the declaration or constructor. 3 67
FieldNamingConventions The static field name 'compteRenduVisiteService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 70
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 72
FieldNamingConventions The static field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 74
CommentRequired Public method and constructor comments are required 3 7798
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 79
ShortVariable Avoid variables with short names like id 3 85
CommentRequired Public method and constructor comments are required 3 101112
ShortVariable Avoid variables with short names like id 3 103
CommentRequired Public method and constructor comments are required 3 115119

fr/paris/lutece/plugins/ctv/bo/controller/demandes/CreationCRVisiteJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 62160
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 62160
ImmutableField Private field 'ctvUploadCompteRenduVisiteHandler' could be made final; it is only initialized in the declaration or constructor. 3 67
FieldNamingConventions The field name 'ctvUploadCompteRenduVisiteHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 67
FieldNamingConventions The static field name 'compteRenduVisiteService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 70
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 72
FieldNamingConventions The static field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 74
FieldNamingConventions The static field name 'alerteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 76
CommentRequired Public method and constructor comments are required 3 7982
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 3 84
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 86
AvoidDuplicateLiterals The String literal "num_demande" appears 4 times in this file; the first occurrence is on line 94 3 94
CommentRequired Public method and constructor comments are required 3 119123
CommentRequired Public method and constructor comments are required 3 126151

fr/paris/lutece/plugins/ctv/bo/controller/demandes/ProjetJspBean.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34986
TooManyStaticImports Too many static imports may lead to messy code 3 34986
CommentRequired Header comments are required 3 106986
TooManyMethods This class has too many methods, consider refactoring it. 3 107986
FieldNamingConventions The static field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 113
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 114
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 3 114
FieldNamingConventions The field name 'demandeFinTravauxService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 115
ImmutableField Private field 'demandeFinTravauxService' could be made final; it is only initialized in the declaration or constructor. 3 115
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 116
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 116
FieldNamingConventions The field name 'prolongationService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 118
ImmutableField Private field 'prolongationService' could be made final; it is only initialized in the declaration or constructor. 3 118
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 119
FieldNamingConventions The static field name 'alerteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 120
FieldNamingConventions The constant name 'ctvDemandeUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 1 121
FieldNamingConventions The constant name 'ctvEmpriseUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 1 123
FieldNamingConventions The constant name 'emailTask' doesn't match '[A-Z][A-Z_0-9]*' 1 126
CommentRequired Public method and constructor comments are required 3 129134
CyclomaticComplexity The method 'initModel(HttpServletRequest)' has a cyclomatic complexity of 15. 3 136242
ExcessiveMethodLength Avoid really long methods. 3 136242
NPathComplexity The method 'initModel(HttpServletRequest)' has an NPath complexity of 273 3 136242
NcssCount The method 'initModel(HttpServletRequest)' has a NCSS line count of 66. 3 136242
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 138
ConfusingTernary Avoid if (x != y) ..; else ..; 3 163
ConfusingTernary Avoid if (x != y) ..; else ..; 3 166228
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 198
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 198
CommentRequired Public method and constructor comments are required 3 251279
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 259
CommentRequired Public method and constructor comments are required 3 282286
AvoidDuplicateLiterals The String literal "empriseId" appears 14 times in this file; the first occurrence is on line 284 3 284
CommentRequired Public method and constructor comments are required 3 289326
CommentRequired Public method and constructor comments are required 3 329339
CommentRequired Public method and constructor comments are required 3 342352
CommentRequired Public method and constructor comments are required 3 355392
CommentRequired Public method and constructor comments are required 3 395408
CommentRequired Public method and constructor comments are required 3 411424
CommentRequired Public method and constructor comments are required 3 427433
CommentRequired Public method and constructor comments are required 3 436488
ConfusingTernary Avoid if (x != y) ..; else ..; 3 452459
ConfusingTernary Avoid if (x != y) ..; else ..; 3 461468
ConfusingTernary Avoid if (x != y) ..; else ..; 3 470477
CommentRequired Public method and constructor comments are required 3 491534
ConfusingTernary Avoid if (x != y) ..; else ..; 3 504511
ConfusingTernary Avoid if (x != y) ..; else ..; 3 513520
CommentRequired Public method and constructor comments are required 3 537569
ConfusingTernary Avoid if (x != y) ..; else ..; 3 549556
CommentRequired Public method and constructor comments are required 3 572665
CyclomaticComplexity The method 'refusProlongation(HttpServletRequest)' has a cyclomatic complexity of 19. 3 572665
NPathComplexity The method 'refusProlongation(HttpServletRequest)' has an NPath complexity of 604 3 572665
CyclomaticComplexity The method 'acceptationProlongation(HttpServletRequest)' has a cyclomatic complexity of 20. 3 668763
CommentRequired Public method and constructor comments are required 3 668763
NPathComplexity The method 'acceptationProlongation(HttpServletRequest)' has an NPath complexity of 1190 3 668763
CyclomaticComplexity The method 'acceptationFinDeTravaux(HttpServletRequest, TypeFinTravaux, StatutEmprise, int)' has a cyclomatic complexity of 18. 3 805892
NPathComplexity The method 'acceptationFinDeTravaux(HttpServletRequest, TypeFinTravaux, StatutEmprise, int)' has an NPath complexity of 1057 3 805892
CollapsibleIfStatements These nested if statements could be combined 3 865878
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 868
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 868
CommentRequired Public method and constructor comments are required 3 920931
CommentRequired Public method and constructor comments are required 3 934948
CommentRequired Public method and constructor comments are required 3 951970
CommentRequired Public method and constructor comments are required 3 979984

fr/paris/lutece/plugins/ctv/bo/controller/demandes/generic/DemandeGenericJspBean.java

Rule Violation Priority Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 4796
CommentRequired Header comments are required 3 4796
CommentRequired Public method and constructor comments are required 3 5394
ConfusingTernary Avoid if (x != y) ..; else ..; 3 5992

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/ChantierJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 60150
ImmutableField Private field 'chantierIntentionControllerService' could be made final; it is only initialized in the declaration or constructor. 3 65
FieldNamingConventions The field name 'chantierIntentionControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 65
FieldNamingConventions The field name 'formulaireIntentionService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 66
FieldNamingConventions The field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 67
CommentRequired Public method and constructor comments are required 3 7086
CommentRequired Public method and constructor comments are required 3 98111
CommentRequired Public method and constructor comments are required 3 114118
CommentRequired Public method and constructor comments are required 3 121128
CommentRequired Public method and constructor comments are required 3 131140

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/DemandeIntentionJspBean.java

Rule Violation Priority Ligne
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 101257
FieldNamingConventions The field name 'formulaireIntentionService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 121
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 123
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 123
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 125
CyclomaticComplexity The method 'getInstructionDemandeIntention(HttpServletRequest)' has a cyclomatic complexity of 12. 3 136200
NPathComplexity The method 'getInstructionDemandeIntention(HttpServletRequest)' has an NPath complexity of 432 3 136200
ConfusingTernary Avoid if (x != y) ..; else ..; 3 144152
ConfusingTernary Avoid if (x != y) ..; else ..; 3 173
ConfusingTernary Avoid if (x != y) ..; else ..; 3 189197
CommentRequired Public method and constructor comments are required 3 203208
CommentRequired Public method and constructor comments are required 3 211216
CommentRequired Public method and constructor comments are required 3 219228
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 239

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/DemandeurJspBean.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34240
CommentRequired Header comments are required 3 71240
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 3 76
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 76
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 3 77
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 77
FieldNamingConventions The field name 'formulaireIntentionService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 78
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 81
CommentRequired Public method and constructor comments are required 3 85102
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 88
CommentRequired Public method and constructor comments are required 3 105119
CommentRequired Public method and constructor comments are required 3 122128
CommentRequired Public method and constructor comments are required 3 131137
CommentRequired Public method and constructor comments are required 3 140149

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intention/EmpriseJspBean.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34637
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34637
CommentRequired Header comments are required 3 108637
TooManyMethods This class has too many methods, consider refactoring it. 3 109637
FieldNamingConventions The constant name 'uploadBaseDirectory' doesn't match '[A-Z][A-Z_0-9]*' 1 111
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 3 118
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 118
ImmutableField Private field 'empriseCtrlService' could be made final; it is only initialized in the declaration or constructor. 3 119
FieldNamingConventions The field name 'empriseCtrlService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 119
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 120
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 3 120
FieldNamingConventions The field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 121
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 122
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 3 124
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 124
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 126
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 127
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 127
FieldNamingConventions The static field name 'alerteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 128
CommentRequired Public method and constructor comments are required 3 133162
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 164
CommentRequired Public method and constructor comments are required 3 165177
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 179
CommentRequired Public method and constructor comments are required 3 180190
CommentRequired Public method and constructor comments are required 3 193213
ConfusingTernary Avoid if (x != y) ..; else ..; 3 197205
AvoidDuplicateLiterals The String literal "profile_default" appears 6 times in this file; the first occurrence is on line 207 3 207
CommentRequired Public method and constructor comments are required 3 216225
CommentRequired Public method and constructor comments are required 3 228237
CommentRequired Public method and constructor comments are required 3 240250
CommentRequired Public method and constructor comments are required 3 256273
CommentRequired Public method and constructor comments are required 3 276293
CommentRequired Public method and constructor comments are required 3 313319
CommentRequired Public method and constructor comments are required 3 322342
ConfusingTernary Avoid if (x != y) ..; else ..; 3 334341
CommentRequired Public method and constructor comments are required 3 348352
CommentRequired Public method and constructor comments are required 3 360369
CommentRequired Public method and constructor comments are required 3 372401
CommentRequired Public method and constructor comments are required 3 414447
CommentRequired Public method and constructor comments are required 3 450462
CommentRequired Public method and constructor comments are required 3 465490
CommentRequired Public method and constructor comments are required 3 493503
CommentRequired Public method and constructor comments are required 3 506516
CommentRequired Public method and constructor comments are required 3 519536
ShortVariable Avoid variables with short names like x 3 552
ShortVariable Avoid variables with short names like y 3 553
CommentRequired Public method and constructor comments are required 3 577607
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 596
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 596
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 611
CommentRequired Public method and constructor comments are required 3 622627

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/ChantierJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 56134
FieldNamingConventions The field name 'formulaireInterventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 61
FieldNamingConventions The field name 'chantierControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 62
ImmutableField Private field 'chantierControllerService' could be made final; it is only initialized in the declaration or constructor. 3 62
CommentRequired Public method and constructor comments are required 3 6574
CommentRequired Public method and constructor comments are required 3 7781
CommentRequired Public method and constructor comments are required 3 8489
CommentRequired Public method and constructor comments are required 3 92102
CommentRequired Public method and constructor comments are required 3 105114

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/DemandeInterventionJspBean.java

Rule Violation Priority Ligne
FieldNamingConventions The field name 'formulaireInterventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 85
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 87
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 87
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 89
CyclomaticComplexity The method 'getInstructionDemandeIntervention(HttpServletRequest)' has a cyclomatic complexity of 10. 3 100156
UnusedLocalVariable Avoid unused local variables such as 'canAccessDemande'. 3 125
ConfusingTernary Avoid if (x != y) ..; else ..; 3 128136
ConfusingTernary Avoid if (x != y) ..; else ..; 3 141
CommentRequired Public method and constructor comments are required 3 159164
CommentRequired Public method and constructor comments are required 3 167172
CommentRequired Public method and constructor comments are required 3 175184
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 194

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/DemandeurJspBean.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34203
CommentRequired Header comments are required 3 70203
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 3 75
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 75
FieldNamingConventions The field name 'formulaireInterventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 77
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 80
CommentRequired Public method and constructor comments are required 3 8497
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.JUSTIFICATIFS_CONVENTION' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.JUSTIFICATIFS_CONVENTION' 4 94
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.JUSTIFICATIFS_CONVENTION' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.JUSTIFICATIFS_CONVENTION' 4 94
CommentRequired Public method and constructor comments are required 3 100105
CommentRequired Public method and constructor comments are required 3 108119
CommentRequired Public method and constructor comments are required 3 122128
CommentRequired Public method and constructor comments are required 3 131140
ConfusingTernary Avoid if (x != y) ..; else ..; 3 173185
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 178
UseLocaleWithCaseConversions When doing a String.toLowerCase()/toUpperCase() call, use a Locale 3 199

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/DocumentsJspBean.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34189
CommentRequired Header comments are required 3 67189
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 82
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 85
CommentRequired Public method and constructor comments are required 3 8898
CommentRequired Public method and constructor comments are required 3 101113
CommentRequired Public method and constructor comments are required 3 116131
CommentRequired Public method and constructor comments are required 3 134143
CommentRequired Public method and constructor comments are required 3 146151

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/EmpriseJspBean.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34222
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34222
CommentRequired Header comments are required 3 73222
ImmutableField Private field 'empriseCtrlService' could be made final; it is only initialized in the declaration or constructor. 3 78
FieldNamingConventions The field name 'empriseCtrlService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 78
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 1 79
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 80
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 82
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 3 82
FieldNamingConventions The field name 'formulaireInterventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 84
CommentRequired Public method and constructor comments are required 3 8797
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 99
CommentRequired Public method and constructor comments are required 3 100108
CommentRequired Public method and constructor comments are required 3 111124
CommentRequired Public method and constructor comments are required 3 127148
CommentRequired Public method and constructor comments are required 3 163172
CommentRequired Public method and constructor comments are required 3 175181
CommentRequired Public method and constructor comments are required 3 184187
CommentRequired Public method and constructor comments are required 3 190193
CommentRequired Public method and constructor comments are required 3 196200
CommentRequired Public method and constructor comments are required 3 203220

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/ExecutantJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 55149
FieldNamingConventions The field name 'formulaireInterventionService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 63
FieldNamingConventions The field name 'executantControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 65
ImmutableField Private field 'executantControllerService' could be made final; it is only initialized in the declaration or constructor. 3 65
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 70
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 3 70
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 71
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 3 71
CommentRequired Public method and constructor comments are required 3 7684
CommentRequired Public method and constructor comments are required 3 8793
CommentRequired Public method and constructor comments are required 3 96105
CommentRequired Public method and constructor comments are required 3 108115
CommentRequired Public method and constructor comments are required 3 118130

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/GestionConflitsJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 54131
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 3 58
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 58
CommentRequired Public method and constructor comments are required 3 6468
CommentRequired Public method and constructor comments are required 3 7175
CommentRequired Public method and constructor comments are required 3 7898
CommentRequired Public method and constructor comments are required 3 101110

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/SuperviseurJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 67187
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 3 71
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 71
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 1 73
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 83
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 83
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 84
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 3 84
CommentRequired Public method and constructor comments are required 3 87103
CommentRequired Public method and constructor comments are required 3 106117
CommentRequired Public method and constructor comments are required 3 120123
CommentRequired Public method and constructor comments are required 3 126140
CommentRequired Public method and constructor comments are required 3 143154
CommentRequired Public method and constructor comments are required 3 169173
CommentRequired Public method and constructor comments are required 3 176185

fr/paris/lutece/plugins/ctv/bo/controller/demandes/intervention/ValidationJspBean.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34325
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34325
CommentRequired Header comments are required 3 98325
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 102
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 3 102
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 3 103
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 103
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 104
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 3 106
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 106
FieldNamingConventions The field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 107
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 108
FieldNamingConventions The field name 'saisineService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 109
ImmutableField Private field 'saisineService' could be made final; it is only initialized in the declaration or constructor. 3 109
ImmutableField Private field 'emailTask' could be made final; it is only initialized in the declaration or constructor. 3 110
FieldNamingConventions The field name 'emailTask' doesn't match '_[a-z][a-zA-Z0-9]*' 1 110
FieldNamingConventions The constant name 'infosSuperviseurService' doesn't match '[A-Z][A-Z_0-9]*' 1 112
CommentRequired Public method and constructor comments are required 3 115135
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 117
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 117
ConfusingTernary Avoid if (x != y) ..; else ..; 3 126133
CommentRequired Public method and constructor comments are required 3 138142
CommentRequired Public method and constructor comments are required 3 145185
CommentRequired Public method and constructor comments are required 3 188199
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 195
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 195
ShortVariable Avoid variables with short names like x 3 216
ShortVariable Avoid variables with short names like y 3 217
CommentRequired Public method and constructor comments are required 3 259311
CommentRequired Public method and constructor comments are required 3 314323

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/ChantierJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 52124
FieldNamingConventions The field name 'chantierControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 66
ImmutableField Private field 'chantierControllerService' could be made final; it is only initialized in the declaration or constructor. 3 66
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 67
CommentRequired Public method and constructor comments are required 3 7081
CommentRequired Public method and constructor comments are required 3 8498
CommentRequired Public method and constructor comments are required 3 101110

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DemandeOccupationJspBean.java

Rule Violation Priority Ligne
UnusedPrivateField Avoid unused private fields such as 'MARK_NUM_DEMANDE'. 3 85
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 91
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 91
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 93
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 95
ConfusingTernary Avoid if (x != y) ..; else ..; 3 145
CommentRequired Public method and constructor comments are required 3 162167
CommentRequired Public method and constructor comments are required 3 170175
CommentRequired Public method and constructor comments are required 3 178187
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 197

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DemandeurJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 66160
ImmutableField Private field 'entiteControllerService' could be made final; it is only initialized in the declaration or constructor. 3 72
FieldNamingConventions The field name 'entiteControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 72
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 73
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 74
FieldNamingConventions The constant name 'ctvDemandeUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 1 77
CommentRequired Public method and constructor comments are required 3 8899
CommentRequired Public method and constructor comments are required 3 102123
CommentRequired Public method and constructor comments are required 3 126135
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.JUSTIFICATIFS_CONVENTION' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.JUSTIFICATIFS_CONVENTION' 4 152
UnnecessaryFullyQualifiedName Unnecessary use of fully qualified name 'Document.JUSTIFICATIFS_CONVENTION' due to existing static import 'fr.paris.lutece.plugins.ctv.constant.Document.JUSTIFICATIFS_CONVENTION' 4 152

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DocumentsJspBean.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34144
CommentRequired Header comments are required 3 63144
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 76
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 78
CommentRequired Public method and constructor comments are required 3 8186
CommentRequired Public method and constructor comments are required 3 89104
CommentRequired Public method and constructor comments are required 3 107116

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/DownloadJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 5180
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 56
CommentRequired Public method and constructor comments are required 3 5966
CommentRequired Public method and constructor comments are required 3 6978

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/EmpriseJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 62284
TooManyMethods This class has too many methods, consider refactoring it. 3 63284
ImmutableField Private field 'empriseControllerService' could be made final; it is only initialized in the declaration or constructor. 3 68
FieldNamingConventions The field name 'empriseControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 68
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 88
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 90
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 3 92
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 92
CommentRequired Public method and constructor comments are required 3 96107
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 109
CommentRequired Public method and constructor comments are required 3 110121
CommentRequired Public method and constructor comments are required 3 124145
CommentRequired Public method and constructor comments are required 3 148158
CommentRequired Public method and constructor comments are required 3 161171
CommentRequired Public method and constructor comments are required 3 174181
CommentRequired Public method and constructor comments are required 3 194200
CommentRequired Public method and constructor comments are required 3 203206
CommentRequired Public method and constructor comments are required 3 209226
CommentRequired Public method and constructor comments are required 3 229236
CommentRequired Public method and constructor comments are required 3 239246

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/ExecutantJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 52119
FieldNamingConventions The field name 'executantControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 58
ImmutableField Private field 'executantControllerService' could be made final; it is only initialized in the declaration or constructor. 3 58
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 59
CommentRequired Public method and constructor comments are required 3 7077
CommentRequired Public method and constructor comments are required 3 8093
CommentRequired Public method and constructor comments are required 3 96105

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/GestionConflitsJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 54132
FieldNamingConventions The field name 'formulaireServiceBo' doesn't match '_[a-z][a-zA-Z0-9]*' 1 59
ImmutableField Private field 'formulaireServiceBo' could be made final; it is only initialized in the declaration or constructor. 3 59
CommentRequired Public method and constructor comments are required 3 6569
CommentRequired Public method and constructor comments are required 3 7276
CommentRequired Public method and constructor comments are required 3 7999
CommentRequired Public method and constructor comments are required 3 102111

fr/paris/lutece/plugins/ctv/bo/controller/demandes/occupation/SuperviseurJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 66180
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 3 69
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 69
ImmutableField Private field 'formulaireServiceBo' could be made final; it is only initialized in the declaration or constructor. 3 70
FieldNamingConventions The field name 'formulaireServiceBo' doesn't match '_[a-z][a-zA-Z0-9]*' 1 70
FieldNamingConventions The constant name 'chantierService' doesn't match '[A-Z][A-Z_0-9]*' 1 71
FieldNamingConventions The field name 'unitService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 76
ImmutableField Private field 'unitService' could be made final; it is only initialized in the declaration or constructor. 3 76
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 3 77
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 77
CommentRequired Public method and constructor comments are required 3 8395
CommentRequired Public method and constructor comments are required 3 98109
CommentRequired Public method and constructor comments are required 3 112115
CommentRequired Public method and constructor comments are required 3 118133
CommentRequired Public method and constructor comments are required 3 136148
CommentRequired Public method and constructor comments are required 3 151160
CommentRequired Public method and constructor comments are required 3 175178

fr/paris/lutece/plugins/ctv/bo/controller/emails/GestionEmailJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 5299
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the emailConfigService field if you want a default access modifier 3 56
FieldNamingConventions The field name 'emailConfigService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 56
DefaultPackage Use explicit scoping instead of the default package private level 3 56
CommentRequired Public method and constructor comments are required 3 5965
CommentRequired Public method and constructor comments are required 3 6878
ShortVariable Avoid variables with short names like id 3 71
CommentRequired Public method and constructor comments are required 3 8191
CommentRequired Public method and constructor comments are required 3 9397

fr/paris/lutece/plugins/ctv/bo/controller/referentiels/ManageNumericParametersJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 4580
DefaultPackage Use explicit scoping instead of the default package private level 3 49
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the MANAGE_PARAMETERS_JSP field if you want a default access modifier 3 49
DefaultPackage Use explicit scoping instead of the default package private level 3 50
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the CTV_GESTION_PARAMETRES_NUMERIC_PATH field if you want a default access modifier 3 50
CommentRequired Public method and constructor comments are required 3 5356
CommentRequired Public method and constructor comments are required 3 5973
LocalVariableNamingConventions The local variable name 'entity_key' doesn't match '[a-z][a-zA-Z0-9]*' 1 62
LocalVariableNamingConventions The local variable name 'entity_value' doesn't match '[a-z][a-zA-Z0-9]*' 1 63

fr/paris/lutece/plugins/ctv/bo/controller/referentiels/ManageReferentielsJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 52152
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the MANAGE_REFERENTIELS_JSP field if you want a default access modifier 3 55
DefaultPackage Use explicit scoping instead of the default package private level 3 55
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the CTV_GESTION_REFERENTIELS_PATH field if you want a default access modifier 3 57
DefaultPackage Use explicit scoping instead of the default package private level 3 57
FieldNamingConventions The field name 'referentielsService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 61
CommentRequired Public method and constructor comments are required 3 6473
CommentRequired Public method and constructor comments are required 3 8489
ShortVariable Avoid variables with short names like id 3 86
AvoidDuplicateLiterals The String literal "itemId" appears 5 times in this file; the first occurrence is on line 86 3 86
CommentRequired Public method and constructor comments are required 3 9196
ShortVariable Avoid variables with short names like id 3 93
CommentRequired Public method and constructor comments are required 3 98106
ShortVariable Avoid variables with short names like id 3 100
CommentRequired Public method and constructor comments are required 3 108116
ShortVariable Avoid variables with short names like id 3 110
CommentRequired Public method and constructor comments are required 3 118125
CommentRequired Public method and constructor comments are required 3 127134
ShortVariable Avoid variables with short names like id 3 130
CommentRequired Public method and constructor comments are required 3 136144

fr/paris/lutece/plugins/ctv/bo/controller/tdb/TableauDeBordJspBean.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 57182
FieldNamingConventions The static field name 'tableauDeBordInstructionControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 62
FieldNamingConventions The static field name 'tableauDeBordConcessionnaireControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 64
CommentRequired Public method and constructor comments are required 3 6797
AvoidDuplicateLiterals The String literal "profile_default" appears 8 times in this file; the first occurrence is on line 69 3 69
CommentRequired Public method and constructor comments are required 3 100129
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 110
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 111
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 116
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 116
CyclomaticComplexity The method 'selectDefaultProfileByPriority(HttpServletRequest, List)' has a cyclomatic complexity of 11. 3 131180
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 134

fr/paris/lutece/plugins/ctv/cas/web/CustomLuteceCASFilter.java

Rule Violation Priority Ligne
FieldNamingConventions The field name 'noCookieMessageKey' doesn't match '_[a-z][a-zA-Z0-9]*' 1 41
RedundantFieldInitializer Avoid using redundant field initializer for 'noCookieMessageKey' 3 41
FieldNamingConventions The field name 'redirectAfterGateway' doesn't match '_[a-z][a-zA-Z0-9]*' 1 53
RedundantFieldInitializer Avoid using redundant field initializer for 'redirectAfterGateway' 3 53
CyclomaticComplexity The method 'doFilter(ServletRequest, ServletResponse, FilterChain)' has a cyclomatic complexity of 13. 3 70135
NPathComplexity The method 'doFilter(ServletRequest, ServletResponse, FilterChain)' has an NPath complexity of 240 3 70135
IdenticalCatchBranches 'catch' branch identical to 'LoginException' branch 3 121124

fr/paris/lutece/plugins/ctv/constants/CtvConstants.java

Rule Violation Priority Ligne
ClassNamingConventions The utility class name 'CtvConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 36297
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 36297
CommentRequired Header comments are required 3 36297

fr/paris/lutece/plugins/ctv/controller/AccueilApp.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 4757
CommentRequired Public method and constructor comments are required 3 5356

fr/paris/lutece/plugins/ctv/controller/front/ChantierApp.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34175
TooManyStaticImports Too many static imports may lead to messy code 3 34175
CommentRequired Header comments are required 3 73175
FieldNamingConventions The field name 'chantierControllerService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 80
ImmutableField Private field 'chantierControllerService' could be made final; it is only initialized in the declaration or constructor. 3 80
ImmutableField Private field 'formulaireDemandeOccupationServiceImpl' could be made final; it is only initialized in the declaration or constructor. 3 81
FieldNamingConventions The field name 'formulaireDemandeOccupationServiceImpl' doesn't match '_[a-z][a-zA-Z0-9]*' 1 81
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 82
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 3 82
CommentRequired Public method and constructor comments are required 3 8693
CommentRequired Public method and constructor comments are required 3 96101
ConfusingTernary Avoid if (x != y) ..; else ..; 3 106113
CommentRequired Public method and constructor comments are required 3 117121
CommentRequired Public method and constructor comments are required 3 124135
CommentRequired Public method and constructor comments are required 3 138148
CommentRequired Public method and constructor comments are required 3 151159
ConfusingTernary Avoid if (x != y) ..; else ..; 3 165172

fr/paris/lutece/plugins/ctv/controller/front/ConfirmationApp.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34104
CommentRequired Header comments are required 3 62104
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 3 70
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 70
FieldNamingConventions The constant name 'demandeService' doesn't match '[A-Z][A-Z_0-9]*' 1 72
CommentRequired Public method and constructor comments are required 3 7581
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 85
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 85
CommentRequired Public method and constructor comments are required 3 99102

fr/paris/lutece/plugins/ctv/controller/front/DemandeurApp.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34309
TooManyStaticImports Too many static imports may lead to messy code 3 34309
CommentRequired Header comments are required 3 92309
TooManyMethods This class has too many methods, consider refactoring it. 3 93309
FieldNamingConventions The field name 'entiteDemandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 99
ImmutableField Private field 'entiteDemandeService' could be made final; it is only initialized in the declaration or constructor. 3 99
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 100
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 3 100
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 101
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 3 101
FieldNamingConventions The static field name 'chantierService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 102
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 104
ImmutableField Private field 'ctvDemandeUploadHandler' could be made final; it is only initialized in the declaration or constructor. 3 104
UnusedPrivateField Avoid unused private fields such as 'LOGGER'. 3 105
ProperLogger Logger should be defined private static final and have the correct class 3 105
CommentRequired Public method and constructor comments are required 3 108133
CommentRequired Public method and constructor comments are required 3 136142
ConfusingTernary Avoid if (x != y) ..; else ..; 3 146152
CommentRequired Public method and constructor comments are required 3 156162
CommentRequired Public method and constructor comments are required 3 165177
CommentRequired Public method and constructor comments are required 3 180190
CommentRequired Public method and constructor comments are required 3 193201
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 3 203
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 266
ConfusingTernary Avoid if (x != y) ..; else ..; 3 295301

fr/paris/lutece/plugins/ctv/controller/front/DocumentApp.java

Rule Violation Priority Ligne
TooManyStaticImports Too many static imports may lead to messy code 3 34300
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34300
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 91300
CommentRequired Header comments are required 3 91300
ImmutableField Private field 'formulaireService' could be made final; it is only initialized in the declaration or constructor. 3 94
FieldNamingConventions The field name 'formulaireService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 94
ImmutableField Private field 'ctvDemandeUploadHandler' could be made final; it is only initialized in the declaration or constructor. 3 95
FieldNamingConventions The field name 'ctvDemandeUploadHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 95
ImmutableField Private field 'ctvUploadEmpriseDocumentHandler' could be made final; it is only initialized in the declaration or constructor. 3 97
FieldNamingConventions The field name 'ctvUploadEmpriseDocumentHandler' doesn't match '_[a-z][a-zA-Z0-9]*' 1 97
ImmutableField Private field 'demandeService' could be made final; it is only initialized in the declaration or constructor. 3 99
FieldNamingConventions The field name 'demandeService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 99
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 3 100
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 100
CommentRequired Public method and constructor comments are required 3 103122
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 106
UnusedLocalVariable Avoid unused local variables such as 'guid'. 3 106
CommentRequired Public method and constructor comments are required 3 185201
CommentRequired Public method and constructor comments are required 3 204208
CommentRequired Public method and constructor comments are required 3 211215
CommentRequired Public method and constructor comments are required 3 218228
CommentRequired Public method and constructor comments are required 3 231242
CommentRequired Public method and constructor comments are required 3 245269
ConfusingTernary Avoid if (x != y) ..; else ..; 3 249267
ShortVariable Avoid variables with short names like id 3 256
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 3 271

fr/paris/lutece/plugins/ctv/controller/front/EditionApp.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 71240
FieldNamingConventions The constant name 'editionService' doesn't match '[A-Z][A-Z_0-9]*' 1 84
FieldNamingConventions The field name 'empriseService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 85
ImmutableField Private field 'empriseService' could be made final; it is only initialized in the declaration or constructor. 3 85
ProperLogger Logger should be defined private static final and have the correct class 3 86
FieldNamingConventions The constant name 'ctvEmpriseUploadHandler' doesn't match '[A-Z][A-Z_0-9]*' 1 90
CommentRequired Public method and constructor comments are required 3 94102
LinguisticNaming Linguistics Antipattern - The getter 'getPermisStationnement' should not return void linguistically 3 94102
CloseResource Ensure that resources like this Connection object are closed after use 3 96
LinguisticNaming Linguistics Antipattern - The getter 'getPermissionVoirie' should not return void linguistically 3 105113
CommentRequired Public method and constructor comments are required 3 105113
CloseResource Ensure that resources like this Connection object are closed after use 3 107
LinguisticNaming Linguistics Antipattern - The getter 'getPermissionAutorisationProlongationOccupation' should not return void linguistically 3 116125
CommentRequired Public method and constructor comments are required 3 116125
CloseResource Ensure that resources like this Connection object are closed after use 3 118
CommentRequired Public method and constructor comments are required 3 128137
LinguisticNaming Linguistics Antipattern - The getter 'getPermissionAutorisationProlongationIntervention' should not return void linguistically 3 128137
CloseResource Ensure that resources like this Connection object are closed after use 3 130
CommentRequired Public method and constructor comments are required 3 140148
LinguisticNaming Linguistics Antipattern - The getter 'getAutorisationIntervention' should not return void linguistically 3 140148
CloseResource Ensure that resources like this Connection object are closed after use 3 142
CommentRequired Public method and constructor comments are required 3 151159
LinguisticNaming Linguistics Antipattern - The getter 'getAnnulationEmprise' should not return void linguistically 3 151159
CloseResource Ensure that resources like this Connection object are closed after use 3 153
LinguisticNaming Linguistics Antipattern - The getter 'getEmprisesPdf' should not return void linguistically 3 162204
CommentRequired Public method and constructor comments are required 3 162204
CloseResource Ensure that resources like this Connection object are closed after use 3 164
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 172
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 187
LinguisticNaming Linguistics Antipattern - The getter 'getDocumentsAutorisations' should not return void linguistically 3 207216
CommentRequired Public method and constructor comments are required 3 207216
AvoidFileStream Avoid instantiating FileInputStream, FileOutputStream, FileReader, or FileWriter 1 222
AssignmentInOperand Avoid assignments in operands