FindBugs Bug Detector Report
The following document contains the results of FindBugs
FindBugs Version is 3.0.1
Threshold is
Effort is max
Summary
| Classes |
Bugs |
Errors |
Missing Classes |
| 65 |
23 |
0 |
51 |
fr.paris.lutece.plugins.newsletter.business.NewsLetter
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.newsletter.business.NewsLetter.getDateLastSending() may expose internal representation by returning NewsLetter._dateLastSending |
MALICIOUS_CODE |
EI_EXPOSE_REP |
255 |
Medium |
| fr.paris.lutece.plugins.newsletter.business.NewsLetter.setDateLastSending(Timestamp) may expose internal representation by storing an externally mutable object into NewsLetter._dateLastSending |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
245 |
Medium |
fr.paris.lutece.plugins.newsletter.business.SendingNewsLetter
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.newsletter.business.SendingNewsLetter.getDate() may expose internal representation by returning SendingNewsLetter._date |
MALICIOUS_CODE |
EI_EXPOSE_REP |
154 |
Medium |
| fr.paris.lutece.plugins.newsletter.business.SendingNewsLetter.setDate(Timestamp) may expose internal representation by storing an externally mutable object into SendingNewsLetter._date |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
144 |
Medium |
fr.paris.lutece.plugins.newsletter.business.Subscriber
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.newsletter.business.Subscriber.getDateSubscription() may expose internal representation by returning Subscriber._dDateSubscription |
MALICIOUS_CODE |
EI_EXPOSE_REP |
99 |
Medium |
| fr.paris.lutece.plugins.newsletter.business.Subscriber.setDateSubscription(Timestamp) may expose internal representation by storing an externally mutable object into Subscriber._dDateSubscription |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
110 |
Medium |
fr.paris.lutece.plugins.newsletter.business.portlet.NewsLetterArchivePortletHome
| Bug |
Category |
Details |
Line |
Priority |
| Static initializer for fr.paris.lutece.plugins.newsletter.business.portlet.NewsLetterArchivePortletHome creates instance before all static final fields assigned |
BAD_PRACTICE |
SI_INSTANCE_BEFORE_FINALS_ASSIGNED |
50 |
Medium |
fr.paris.lutece.plugins.newsletter.service.NewsLetterArchiveService
| Bug |
Category |
Details |
Line |
Priority |
| Static initializer for fr.paris.lutece.plugins.newsletter.service.NewsLetterArchiveService creates instance before all static final fields assigned |
BAD_PRACTICE |
SI_INSTANCE_BEFORE_FINALS_ASSIGNED |
64 |
Medium |
fr.paris.lutece.plugins.newsletter.service.NewsLetterRegistrationService
| Bug |
Category |
Details |
Line |
Priority |
| Static initializer for fr.paris.lutece.plugins.newsletter.service.NewsLetterRegistrationService creates instance before all static final fields assigned |
BAD_PRACTICE |
SI_INSTANCE_BEFORE_FINALS_ASSIGNED |
93 |
Medium |
fr.paris.lutece.plugins.newsletter.service.NewsletterService
| Bug |
Category |
Details |
Line |
Priority |
| Found reliance on default encoding in fr.paris.lutece.plugins.newsletter.service.NewsletterService.generateNewsletterHtmlCode(NewsLetter, int, String, AdminUser, Locale): new String(byte[]) |
I18N |
DM_DEFAULT_ENCODING |
370 |
High |
fr.paris.lutece.plugins.newsletter.web.NewsletterJspBean
| Bug |
Category |
Details |
Line |
Priority |
| Found reliance on default encoding in fr.paris.lutece.plugins.newsletter.web.NewsletterJspBean.doImportSubscribers(HttpServletRequest): new java.io.InputStreamReader(InputStream) |
I18N |
DM_DEFAULT_ENCODING |
1901 |
High |
| Found reliance on default encoding in fr.paris.lutece.plugins.newsletter.web.NewsletterJspBean.getTextFileContent(String): new java.io.FileReader(String) |
I18N |
DM_DEFAULT_ENCODING |
2515 |
High |
fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean
| Bug |
Category |
Details |
Line |
Priority |
| Found reliance on default encoding in fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean.doModifyNewsletterTemplateFile(HttpServletRequest): String.getBytes() |
I18N |
DM_DEFAULT_ENCODING |
653 |
High |
| Found reliance on default encoding in fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean.getModifyNewsLetterTemplateFile(HttpServletRequest): new java.io.FileReader(File) |
I18N |
DM_DEFAULT_ENCODING |
537 |
High |
| Found reliance on default encoding in fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean.getModifyNewsLetterTemplateFile(HttpServletRequest): new java.io.FileReader(String) |
I18N |
DM_DEFAULT_ENCODING |
547 |
High |
| Found reliance on default encoding in fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean.getModifyNewsLetterTemplateFile(HttpServletRequest): new java.io.FileWriter(File) |
I18N |
DM_DEFAULT_ENCODING |
534 |
High |
| Found reliance on default encoding in fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean.getModifyNewsLetterTemplateFile(HttpServletRequest): new String(byte[]) |
I18N |
DM_DEFAULT_ENCODING |
535 |
High |
| Exceptional return value of java.io.File.delete() ignored in fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean.doModifyNewsletterTemplate(HttpServletRequest) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
447 |
Medium |
| Exceptional return value of java.io.File.delete() ignored in fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean.doModifyNewsletterTemplateFile(HttpServletRequest) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
644 |
Medium |
| Exceptional return value of java.io.File.delete() ignored in fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean.doRemoveNewsLetterTemplate(HttpServletRequest) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
769 |
Medium |
| Exceptional return value of java.io.File.delete() ignored in fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean.getModifyNewsLetterTemplateFile(HttpServletRequest) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
539 |
Medium |
fr.paris.lutece.plugins.newsletter.web.portlet.NewsLetterArchivePortletJspBean
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.newsletter.web.portlet.NewsLetterArchivePortletJspBean.modifySendings(HttpServletRequest, NewsLetterArchivePortlet) invokes inefficient new Integer(String) constructor; use Integer.valueOf(String) instead |
PERFORMANCE |
DM_NUMBER_CTOR |
267 |
Medium |
fr.paris.lutece.plugins.newsletter.web.portlet.NewsLetterSubscriptionPortletJspBean
| Bug |
Category |
Details |
Line |
Priority |
| fr.paris.lutece.plugins.newsletter.web.portlet.NewsLetterSubscriptionPortletJspBean.modifySubscriptions(HttpServletRequest, NewsLetterSubscriptionPortlet) invokes inefficient new Integer(String) constructor; use Integer.valueOf(String) instead |
PERFORMANCE |
DM_NUMBER_CTOR |
277 |
Medium |
