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 |
| 64 |
21 |
0 |
0 |
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.web.NewsLetterApp
| Bug |
Category |
Details |
Line |
Priority |
| Class fr.paris.lutece.plugins.newsletter.web.NewsLetterApp defines non-transient non-serializable instance field _captchaService |
BAD_PRACTICE |
SE_BAD_FIELD |
Not available |
Medium |
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 |
2501 |
High |
| Useless object stored in variable sending of method fr.paris.lutece.plugins.newsletter.web.NewsletterJspBean.doTestNewsLetter(HttpServletRequest) |
STYLE |
UC_USELESS_OBJECT |
1651 |
Medium |
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): new java.io.FileWriter(String) |
I18N |
DM_DEFAULT_ENCODING |
633 |
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 |
533 |
High |
| Exceptional return value of java.io.File.mkdir() ignored in fr.paris.lutece.plugins.newsletter.web.NewsletterTemplateJspBean.doCreateNewsletterTemplate(HttpServletRequest) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
285 |
Medium |
| 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 |
440 |
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 |
627 |
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 |
731 |
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 |
