PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/myportal/modules/myfavorites/business/MyFavorites.java
| Violation |
Line |
| Avoid unused imports such as 'javax.validation.constraints' |
36 |
| Avoid unused imports such as 'org.hibernate.validator.constraints' |
37 |
fr/paris/lutece/plugins/myportal/modules/myfavorites/services/MyFavoritesService.java
| Violation |
Line |
| Deeply nested if..then statements are hard to read |
156–164 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
255 |
fr/paris/lutece/plugins/myportal/modules/myfavorites/services/MyFavoritesWidgetHandler.java
| Violation |
Line |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
83 |
fr/paris/lutece/plugins/myportal/modules/myfavorites/web/MyFavoritesXPage.java
| Violation |
Line |
| Assigning an Object to null is a code smell. Consider refactoring. |
133 |
| The method getCreateMyFavorites() has an NPath complexity of 280 |
150–198 |
| The method doCreateMyFavorites() has an NPath complexity of 210 |
210–247 |
| The method doModifyMyFavorites() has an NPath complexity of 300 |
364–402 |
fr/paris/lutece/plugins/myportal/modules/myfavorites/web/rs/MyFavoritesRest.java
| Violation |
Line |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
109 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
152 |
