The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_formResponse' doesn't match '[a-z][a-zA-Z0-9]*' | 62 |
| FieldNamingConventions | The final field name '_fileName' doesn't match '[a-z][a-zA-Z0-9]*' | 63 |
| FieldNamingConventions | The final field name '_fileDescription' doesn't match '[a-z][a-zA-Z0-9]*' | 64 |
| FieldNamingConventions | The final field name '_formsPDFTaskTemplate' doesn't match '[a-z][a-zA-Z0-9]*' | 65 |
| Rule | Violation | Line |
|---|---|---|
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 174 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 180 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The constant name '_formsPDFTaskConfigService' doesn't match '[A-Z][A-Z_0-9]*' | 82 |
| FieldNamingConventions | The constant name '_resourceHistoryService' doesn't match '[A-Z][A-Z_0-9]*' | 87 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 144 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'strtemplate' | 220 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'strtemplate' | 231 |
| Rule | Violation | Line |
|---|---|---|
| AvoidDuplicateLiterals | The String literal "resource" appears 4 times in this file; the first occurrence is on line 73 | 73 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–96 |
| CommentRequired | Public method and constructor comments are required | 50–52 |
| CommentRequired | Public method and constructor comments are required | 54–56 |
| CommentRequired | Public method and constructor comments are required | 58–60 |
| CommentRequired | Public method and constructor comments are required | 62–64 |
| CommentRequired | Public method and constructor comments are required | 66–68 |
| CommentRequired | Public method and constructor comments are required | 70–72 |
| CommentRequired | Public method and constructor comments are required | 74–76 |
| CommentRequired | Public method and constructor comments are required | 78–80 |
| CommentRequired | Public method and constructor comments are required | 82–84 |
| CommentRequired | Public method and constructor comments are required | 86–88 |
| CommentRequired | Public method and constructor comments are required | 89–91 |
| CommentRequired | Public method and constructor comments are required | 92–94 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 44–207 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–52 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 40–42 |
| CommentRequired | Public method and constructor comments are required | 40–42 |
| CommentRequired | Public method and constructor comments are required | 44–46 |
| CommentRequired | Public method and constructor comments are required | 48–50 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 40–86 |
| CommentRequired | Public method and constructor comments are required | 49–53 |
| CommentRequired | Public method and constructor comments are required | 55–59 |
| CommentRequired | Public method and constructor comments are required | 61–64 |
| CommentRequired | Public method and constructor comments are required | 66–69 |
| CommentRequired | Public method and constructor comments are required | 71–74 |
| CommentRequired | Public method and constructor comments are required | 76–79 |
| CommentRequired | Public method and constructor comments are required | 81–84 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–54 |
| CommentRequired | Public method and constructor comments are required | 40 |
| 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 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 54–117 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_configList' | 67 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 55–58 |
| Rule | Violation | Line |
|---|---|---|
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 77 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 78 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 79 |
| IdenticalCatchBranches | 'catch' branch identical to 'PdfConverterServiceException' branch | 176–181 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 119 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 138 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 102 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 112 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 128–135 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 54–237 |
| CommentRequired | Public method and constructor comments are required | 98–111 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 113 |
| CommentRequired | Public method and constructor comments are required | 114–155 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 157 |
| CommentRequired | Public method and constructor comments are required | 158–174 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 176 |
| CommentRequired | Public method and constructor comments are required | 177–186 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.workflowcore.service.provider.InfoMarker' | 60 |
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.forms.business.FormQuestionResponse' | 61 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.Locale' | 36 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidDuplicateLiterals | The String literal "resource" appears 4 times in this file; the first occurrence is on line 73 | 3 | 73 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–96 |
| CommentRequired | Public method and constructor comments are required | 3 | 50–52 |
| CommentRequired | Public method and constructor comments are required | 3 | 54–56 |
| CommentRequired | Public method and constructor comments are required | 3 | 58–60 |
| CommentRequired | Public method and constructor comments are required | 3 | 62–64 |
| CommentRequired | Public method and constructor comments are required | 3 | 66–68 |
| CommentRequired | Public method and constructor comments are required | 3 | 70–72 |
| CommentRequired | Public method and constructor comments are required | 3 | 74–76 |
| CommentRequired | Public method and constructor comments are required | 3 | 78–80 |
| CommentRequired | Public method and constructor comments are required | 3 | 82–84 |
| CommentRequired | Public method and constructor comments are required | 3 | 86–88 |
| CommentRequired | Public method and constructor comments are required | 3 | 89–91 |
| CommentRequired | Public method and constructor comments are required | 3 | 92–94 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 44–207 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–52 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 40–42 |
| CommentRequired | Public method and constructor comments are required | 3 | 40–42 |
| CommentRequired | Public method and constructor comments are required | 3 | 44–46 |
| CommentRequired | Public method and constructor comments are required | 3 | 48–50 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 40–86 |
| CommentRequired | Public method and constructor comments are required | 3 | 49–53 |
| CommentRequired | Public method and constructor comments are required | 3 | 55–59 |
| CommentRequired | Public method and constructor comments are required | 3 | 61–64 |
| CommentRequired | Public method and constructor comments are required | 3 | 66–69 |
| CommentRequired | Public method and constructor comments are required | 3 | 71–74 |
| CommentRequired | Public method and constructor comments are required | 3 | 76–79 |
| CommentRequired | Public method and constructor comments are required | 3 | 81–84 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–54 |
| CommentRequired | Public method and constructor comments are required | 3 | 40 |
| CommentRequired | Public method and constructor comments are required | 3 | 42 |
| CommentRequired | Public method and constructor comments are required | 3 | 44 |
| CommentRequired | Public method and constructor comments are required | 3 | 46 |
| CommentRequired | Public method and constructor comments are required | 3 | 48 |
| CommentRequired | Public method and constructor comments are required | 3 | 50 |
| CommentRequired | Public method and constructor comments are required | 3 | 52 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 54–117 |
| FieldNamingConventions | The final field name '_formResponse' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 62 |
| FieldNamingConventions | The final field name '_fileName' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 63 |
| FieldNamingConventions | The final field name '_fileDescription' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 64 |
| FieldNamingConventions | The final field name '_formsPDFTaskTemplate' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 65 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_configList' | 3 | 67 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 55–58 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 77 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 78 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 79 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 174 |
| IdenticalCatchBranches | 'catch' branch identical to 'PdfConverterServiceException' branch | 3 | 176–181 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 180 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.workflowcore.service.provider.InfoMarker' | 4 | 60 |
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.forms.business.FormQuestionResponse' | 4 | 61 |
| FieldNamingConventions | The constant name '_formsPDFTaskConfigService' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 82 |
| FieldNamingConventions | The constant name '_resourceHistoryService' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 87 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 119 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 138 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 144 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 102 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 112 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 128–135 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.Locale' | 4 | 36 |
| CommentRequired | Header comments are required | 3 | 54–237 |
| CommentRequired | Public method and constructor comments are required | 3 | 98–111 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 113 |
| CommentRequired | Public method and constructor comments are required | 3 | 114–155 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 157 |
| CommentRequired | Public method and constructor comments are required | 3 | 158–174 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 176 |
| CommentRequired | Public method and constructor comments are required | 3 | 177–186 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'strtemplate' | 2 | 220 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'strtemplate' | 2 | 231 |
