Fork me on GitHub

PMD Results

The following document contains the results of PMD 5.3.5.

Files

fr/paris/lutece/plugins/directory/modules/pdfproducer/business/listener/DirectoryPDFProducerRemovalListener.java

Violation Line
Avoid variables with short names like id 64

fr/paris/lutece/plugins/directory/modules/pdfproducer/business/listener/EntryPDFProducerRemovalListener.java

Violation Line
Avoid variables with short names like id 72

fr/paris/lutece/plugins/directory/modules/pdfproducer/business/producerconfig/DefaultConfigProducer.java

Violation Line
Document empty method 6364
Document empty method 8788
Document empty method 110111
Document empty method 125126
Document empty method 156157
Document empty method 171172
Document empty method 186187

fr/paris/lutece/plugins/directory/modules/pdfproducer/service/DirectoryPDFProducerResourceIdService.java

Violation Line
Avoid variables with short names like rt 88
Avoid variables with short names like p 99

fr/paris/lutece/plugins/directory/modules/pdfproducer/utils/PDFUtils.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 34697
The class 'PDFUtils' has a Modified Cyclomatic Complexity of 5 (Highest = 13). 95697
The class 'PDFUtils' has a Cyclomatic Complexity of 7 (Highest = 23). 95697
The class 'PDFUtils' has a Standard Cyclomatic Complexity of 5 (Highest = 13). 95697
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. 185317
The method 'doCreateDocumentPDF' has a Cyclomatic Complexity of 10. 185317
The method 'doCreateDocumentPDF' has a Modified Cyclomatic Complexity of 10. 185317
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. 185317
When instantiating a SimpleDateFormat object, specify a Locale 218219
The method 'builderPDFWithEntry' has a Standard Cyclomatic Complexity of 10. 329402
The method 'builderPDFWithEntry' has a Cyclomatic Complexity of 14. 329402
The method 'builderPDFWithEntry' has a Modified Cyclomatic Complexity of 10. 329402
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. 464513
The method 'builFieldsInSinglePhrase' has a Cyclomatic Complexity of 23. 525604
The method 'builFieldsInSinglePhrase' has a Standard Cyclomatic Complexity of 13. 525604
The method 'builFieldsInSinglePhrase' has a Modified Cyclomatic Complexity of 13. 525604
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 573578
These nested if statements could be combined 597602
Avoid variables with short names like os 632

fr/paris/lutece/plugins/directory/modules/pdfproducer/web/PDFProducerJspBean.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 34967
The class 'PDFProducerJspBean' has a Standard Cyclomatic Complexity of 5 (Highest = 13). 77967
The class 'PDFProducerJspBean' has a Cyclomatic Complexity of 6 (Highest = 15). 77967
The class 'PDFProducerJspBean' has a Modified Cyclomatic Complexity of 5 (Highest = 13). 77967
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. 243350
The method 'doCreateConfigProducer' has a Modified Cyclomatic Complexity of 12. 243350
The method 'doCreateConfigProducer' has a Cyclomatic Complexity of 15. 243350
The method doCreateConfigProducer() has an NPath complexity of 200 243350
Avoid really long methods. 243350
The method modifyConfigProducer() has an NPath complexity of 320 407504
The method 'modifyConfigProducer' has a Modified Cyclomatic Complexity of 10. 407504
The method 'modifyConfigProducer' has a Cyclomatic Complexity of 11. 407504
The method 'modifyConfigProducer' has a Standard Cyclomatic Complexity of 10. 407504
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 511629
Avoid really long methods. 511629
The method 'doModifyConfigProducer' has a Standard Cyclomatic Complexity of 13. 511629
The method 'doModifyConfigProducer' has a Modified Cyclomatic Complexity of 13. 511629
The method 'doModifyConfigProducer' has a Cyclomatic Complexity of 15. 511629
Rather than using a lot of String arguments, consider using a container object for those values. 701702
The method 'doCheckAll' has a Cyclomatic Complexity of 10. 701746
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

fr/paris/lutece/plugins/directory/modules/pdfproducer/web/action/PDFConfigAction.java

Violation Line
Avoid unnecessary if..then..else statements when returning booleans 116119