Lutece calendar document module

PMD Results

The following document contains the results of PMD 5.3.5.

Files

fr/paris/lutece/plugins/calendar/modules/document/service/search/DocumentCalendarIndexer.java

Violation	Line
A high number of imports can indicate a high degree of coupling within an object.	34–422
Avoid unused private fields such as 'JSP_SEARCH_CALENDAR'.	95
Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document'	155
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block	160
Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document'	183
Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document'	183
Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document'	212
An empty statement (semicolon) not part of a loop	215
Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document'	284
Avoid variables with short names like ft	288
Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document'	300
Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document'	300
New exception is thrown in catch block, original stack trace may be lost	330
New exception is thrown in catch block, original stack trace may be lost	334
Avoid variables with short names like sb	339
Avoid appending characters as strings in StringBuffer.append.	375
Avoid appending characters as strings in StringBuffer.append.	389

fr/paris/lutece/plugins/calendar/modules/document/web/CalendarDocInsertServiceJspBean.java

Violation	Line
Avoid unused imports such as 'fr.paris.lutece.plugins.document.business.portlet.DocumentListPortletHome'	39
Avoid unused imports such as 'fr.paris.lutece.plugins.document.business.portlet.DocumentListPortletHome'	39
Avoid unused imports such as 'fr.paris.lutece.plugins.document.service.publishing.PublishingService'	40
Avoid unused imports such as 'fr.paris.lutece.plugins.document.service.publishing.PublishingService'	40
Avoid unused imports such as 'fr.paris.lutece.portal.business.page.Page'	41
Avoid unused imports such as 'fr.paris.lutece.portal.business.page.Page'	41
Avoid unused imports such as 'fr.paris.lutece.portal.business.page.PageHome'	42
Avoid unused imports such as 'fr.paris.lutece.portal.business.page.PageHome'	42
Avoid unused imports such as 'fr.paris.lutece.portal.business.portlet.Portlet'	43
Avoid unused imports such as 'fr.paris.lutece.portal.business.portlet.Portlet'	43
Avoid unused imports such as 'fr.paris.lutece.portal.business.portlet.PortletHome'	44
Avoid unused imports such as 'fr.paris.lutece.portal.business.portlet.PortletHome'	44
Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppPathService'	51
Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppPathService'	51
Avoid unused imports such as 'fr.paris.lutece.portal.service.workgroup.AdminWorkgroupService'	53
Avoid unused imports such as 'fr.paris.lutece.portal.service.workgroup.AdminWorkgroupService'	53
Avoid unused imports such as 'fr.paris.lutece.portal.web.insert.InsertServiceJspBean'	55
Avoid unused imports such as 'fr.paris.lutece.portal.web.insert.InsertServiceJspBean'	55
Avoid unused imports such as 'fr.paris.lutece.portal.web.insert.InsertServiceSelectionBean'	56
Avoid unused imports such as 'fr.paris.lutece.portal.web.insert.InsertServiceSelectionBean'	56
Avoid unused imports such as 'fr.paris.lutece.util.url.UrlItem'	58
Avoid unused imports such as 'fr.paris.lutece.util.url.UrlItem'	58
Avoid unused imports such as 'org.apache.commons.lang.StringEscapeUtils'	60
Avoid unused imports such as 'org.apache.commons.lang.StringEscapeUtils'	60
Avoid unused imports such as 'java.util.ArrayList'	62
Avoid unused imports such as 'java.util.ArrayList'	62
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation	155
Avoid using implementation types like 'HashMap'; use the interface instead	171
Avoid using implementation types like 'HashMap'; use the interface instead	171

fr/paris/lutece/plugins/calendar/modules/document/web/DocumentCalendarUtils.java

Violation	Line
Avoid unnecessary constructors - the compiler will generate these for you	51–53
Document empty constructor	51–53
Avoid catching NullPointerException; consider removing the cause of the NPE.	72
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block	72
Avoid empty catch blocks	72–74