PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/myportal/modules/rss/business/Category.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/rss/business/RssConf.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/rss/service/RssAdvancedWidgetHandler.java
| Violation |
Line |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' |
51 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' |
51 |
| The field name indicates a constant but its modifiers do not |
58 |
| Use explicit scoping instead of the default package private level |
58 |
| Use explicit scoping instead of the default package private level |
59 |
| The field name indicates a constant but its modifiers do not |
59 |
| Avoid reassigning parameters such as 'user' |
81 |
fr/paris/lutece/plugins/myportal/modules/rss/service/RssWidgetHandler.java
| Violation |
Line |
| Use explicit scoping instead of the default package private level |
48 |
| The field name indicates a constant but its modifiers do not |
48 |
| Use explicit scoping instead of the default package private level |
49 |
| The field name indicates a constant but its modifiers do not |
49 |
fr/paris/lutece/plugins/myportal/modules/rss/service/RssWidgetService.java
| Violation |
Line |
| Avoid unused imports such as 'fr.paris.lutece.portal.service.cache.AbstractCacheableService' |
38 |
| Avoid unused imports such as 'fr.paris.lutece.portal.service.cache.AbstractCacheableService' |
38 |
| A class which only has private constructors should be final |
49–116 |
| Consider simply returning the value vs storing it in local variable 'strRss' |
85 |
| Avoid empty catch blocks |
101–103 |
| Use explicit scoping instead of the default package private level |
107–115 |
fr/paris/lutece/plugins/myportal/modules/rss/web/CategoryJspBean.java
| Violation |
Line |
| Assigning an Object to null is a code smell. Consider refactoring. |
107 |
fr/paris/lutece/plugins/myportal/modules/rss/web/ManageRssJspBean.java
| Violation |
Line |
| Abstract classes should be named AbstractXXX |
50–99 |
| Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. |
66 |
| Do not add empty strings |
93 |
fr/paris/lutece/plugins/myportal/modules/rss/web/RssConfJspBean.java
| Violation |
Line |
| Assigning an Object to null is a code smell. Consider refactoring. |
113 |
