PMD Results
The following document contains the results of PMD 5.5.1.
Files
fr/paris/lutece/plugins/appstore/business/Application.java
Violation |
Line |
Too many fields |
41–432 |
fr/paris/lutece/plugins/appstore/business/Icon.java
Violation |
Line |
Avoid short class names like Icon |
41–168 |
Returning '_byValue' may expose an internal array. |
100 |
The user-supplied array 'value' is stored directly. |
109 |
fr/paris/lutece/plugins/appstore/business/IconDAO.java
Violation |
Line |
Use block level rather than method level synchronization |
89–104 |
fr/paris/lutece/plugins/appstore/service/ApplicationExtendableResourceService.java
Violation |
Line |
Avoid unused imports such as 'fr.paris.lutece.portal.web.constants.Parameters' |
41 |
Avoid unused imports such as 'fr.paris.lutece.portal.web.constants.Parameters' |
41 |
fr/paris/lutece/plugins/appstore/service/ComponentInfoService.java
Violation |
Line |
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. |
54–119 |
Avoid variables with short names like tr |
104 |
fr/paris/lutece/plugins/appstore/service/IconService.java
Violation |
Line |
Document empty constructor |
59–61 |
fr/paris/lutece/plugins/appstore/service/UpdaterService.java
Violation |
Line |
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. |
43–71 |
fr/paris/lutece/plugins/appstore/web/AppStoreApp.java
Violation |
Line |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
107 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
136 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
152 |
fr/paris/lutece/plugins/appstore/web/AppStoreJspBean.java
Violation |
Line |
Abstract classes should be named AbstractXXX |
41–53 |
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. |
41–53 |
fr/paris/lutece/plugins/appstore/web/ApplicationJspBean.java
Violation |
Line |
Avoid unused private fields such as 'PARAMETER_APPLICATION_POM_URL'. |
74 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
139 |
Do not add empty strings |
141 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
162 |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
181 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
290 |
The method doModifyApplication() has an NPath complexity of 256 |
310–384 |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
313 |
fr/paris/lutece/plugins/appstore/web/CategoryJspBean.java
Violation |
Line |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
98 |
Do not add empty strings |
100 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
121 |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
137 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
204 |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
222 |
fr/paris/lutece/plugins/appstore/web/ComponentJspBean.java
Violation |
Line |
Avoid unused private fields such as 'PARAMETER_COMPONENT_VERSION'. |
68 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
103 |
Do not add empty strings |
105 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
126 |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
142 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
216 |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
234 |
fr/paris/lutece/plugins/appstore/web/IconJspBean.java
Violation |
Line |
The class 'IconJspBean' has a Cyclomatic Complexity of 4 (Highest = 18). |
65–412 |
Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. |
111 |
The method 'getIconData' has a Cyclomatic Complexity of 18. |
305–373 |
The method getIconData() has an NPath complexity of 784 |
305–373 |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
312 |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
313 |