The following document contains the results of PMD 5.3.5.
| Violation | Line |
|---|---|
| Avoid variables with short names like id | 64 |
| Violation | Line |
|---|---|
| Avoid variables with short names like id | 72 |
| Violation | Line |
|---|---|
| Document empty method | 63–64 |
| Document empty method | 87–88 |
| Document empty method | 110–111 |
| Document empty method | 125–126 |
| Document empty method | 156–157 |
| Document empty method | 171–172 |
| Document empty method | 186–187 |
| Violation | Line |
|---|---|
| Avoid variables with short names like rt | 88 |
| Avoid variables with short names like p | 99 |
| Violation | Line |
|---|---|
| A high number of imports can indicate a high degree of coupling within an object. | 34–697 |
| The class 'PDFUtils' has a Modified Cyclomatic Complexity of 5 (Highest = 13). | 95–697 |
| The class 'PDFUtils' has a Cyclomatic Complexity of 7 (Highest = 23). | 95–697 |
| The class 'PDFUtils' has a Standard Cyclomatic Complexity of 5 (Highest = 13). | 95–697 |
| Avoid unused private fields such as 'PARAMETER_ID_DIRECTORY_RECORD'. | 123 |
| Avoid unused private fields such as 'PARAMETER_ID_RECORD'. | 124 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 150 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 167 |
| Avoid really long methods. | 185–317 |
| The method 'doCreateDocumentPDF' has a Cyclomatic Complexity of 10. | 185–317 |
| The method 'doCreateDocumentPDF' has a Modified Cyclomatic Complexity of 10. | 185–317 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 185 |
| The method 'doCreateDocumentPDF' has a Standard Cyclomatic Complexity of 10. | 185–317 |
| When instantiating a SimpleDateFormat object, specify a Locale | 218–219 |
| The method 'builderPDFWithEntry' has a Standard Cyclomatic Complexity of 10. | 329–402 |
| The method 'builderPDFWithEntry' has a Cyclomatic Complexity of 14. | 329–402 |
| The method 'builderPDFWithEntry' has a Modified Cyclomatic Complexity of 10. | 329–402 |
| Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 435 |
| Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' due to existing import 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' | 439 |
| The method 'builFieldsInParagraph' has a Cyclomatic Complexity of 16. | 464–513 |
| The method 'builFieldsInSinglePhrase' has a Cyclomatic Complexity of 23. | 525–604 |
| The method 'builFieldsInSinglePhrase' has a Standard Cyclomatic Complexity of 13. | 525–604 |
| The method 'builFieldsInSinglePhrase' has a Modified Cyclomatic Complexity of 13. | 525–604 |
| Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.directory.business.EntryTypeDownloadUrl' due to existing import 'fr.paris.lutece.plugins.directory.business.EntryTypeDownloadUrl' | 535 |
| Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' due to existing import 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' | 546 |
| These nested if statements could be combined | 573–578 |
| These nested if statements could be combined | 597–602 |
| Avoid variables with short names like os | 632 |
| Violation | Line |
|---|---|
| A high number of imports can indicate a high degree of coupling within an object. | 34–967 |
| The class 'PDFProducerJspBean' has a Standard Cyclomatic Complexity of 5 (Highest = 13). | 77–967 |
| The class 'PDFProducerJspBean' has a Cyclomatic Complexity of 6 (Highest = 15). | 77–967 |
| The class 'PDFProducerJspBean' has a Modified Cyclomatic Complexity of 5 (Highest = 13). | 77–967 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 161 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 188 |
| The method 'doCreateConfigProducer' has a Standard Cyclomatic Complexity of 12. | 243–350 |
| The method 'doCreateConfigProducer' has a Modified Cyclomatic Complexity of 12. | 243–350 |
| The method 'doCreateConfigProducer' has a Cyclomatic Complexity of 15. | 243–350 |
| The method doCreateConfigProducer() has an NPath complexity of 200 | 243–350 |
| Avoid really long methods. | 243–350 |
| The method modifyConfigProducer() has an NPath complexity of 320 | 407–504 |
| The method 'modifyConfigProducer' has a Modified Cyclomatic Complexity of 10. | 407–504 |
| The method 'modifyConfigProducer' has a Cyclomatic Complexity of 11. | 407–504 |
| The method 'modifyConfigProducer' has a Standard Cyclomatic Complexity of 10. | 407–504 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 438 |
| The method doModifyConfigProducer() has an NPath complexity of 340 | 511–629 |
| Avoid really long methods. | 511–629 |
| The method 'doModifyConfigProducer' has a Standard Cyclomatic Complexity of 13. | 511–629 |
| The method 'doModifyConfigProducer' has a Modified Cyclomatic Complexity of 13. | 511–629 |
| The method 'doModifyConfigProducer' has a Cyclomatic Complexity of 15. | 511–629 |
| Rather than using a lot of String arguments, consider using a container object for those values. | 701–702 |
| The method 'doCheckAll' has a Cyclomatic Complexity of 10. | 701–746 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 817 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 916 |
