PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/reportlauncher/business/Parameter.java
| Violation |
Line |
| Avoid unused imports such as 'org.hibernate.validator.constraints' |
36 |
| Avoid unused imports such as 'javax.validation.constraints' |
40 |
fr/paris/lutece/plugins/reportlauncher/business/ParameterDAO.java
| Violation |
Line |
| Avoid unused imports such as 'java.util.List' |
41 |
| Avoid unused imports such as 'java.util.List' |
41 |
fr/paris/lutece/plugins/reportlauncher/business/Report.java
| Violation |
Line |
| Avoid unused imports such as 'org.hibernate.validator.constraints' |
36 |
| Avoid unused imports such as 'javax.validation.constraints' |
42 |
fr/paris/lutece/plugins/reportlauncher/business/ReportHome.java
| Violation |
Line |
| Avoid reassigning parameters such as 'report' |
67 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
83 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
126 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
150 |
fr/paris/lutece/plugins/reportlauncher/business/dto/MockReportLauncher.java
| Violation |
Line |
| No need to import a type that lives in the same package |
36 |
fr/paris/lutece/plugins/reportlauncher/web/ManageReportlauncherJspBean.java
| Violation |
Line |
| Abstract classes should be named AbstractXXX |
51–96 |
| Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. |
61 |
| Do not add empty strings |
90 |
fr/paris/lutece/plugins/reportlauncher/web/ParameterJspBean.java
| Violation |
Line |
| Avoid unused private fields such as 'PROPERTY_DEFAULT_LIST_PARAMETER_PER_PAGE'. |
80 |
| Assigning an Object to null is a code smell. Consider refactoring. |
110 |
| Avoid empty if statements |
207–209 |
fr/paris/lutece/plugins/reportlauncher/web/ReportDashboardComponent.java
| Violation |
Line |
| Avoid unused private fields such as 'VIEW_PERMISSIONS_FORM'. |
57 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
67 |
fr/paris/lutece/plugins/reportlauncher/web/ReportJspBean.java
| Violation |
Line |
| Avoid unused private fields such as 'PROPERTY_DEFAULT_LIST_REPORT_PER_PAGE'. |
94 |
| Assigning an Object to null is a code smell. Consider refactoring. |
129 |
fr/paris/lutece/plugns/reportlauncher/service/IReportLauncherService.java
| Violation |
Line |
| Avoid modifiers which are implied by the context |
44 |
| Avoid modifiers which are implied by the context |
46 |
| Avoid modifiers which are implied by the context |
48 |
fr/paris/lutece/plugns/reportlauncher/service/ReportLauncherService.java
| Violation |
Line |
| A class which only has private constructors should be final |
43–115 |
| Singleton is not thread safe |
60–64 |
