Rapport FindBugs de détection des bugs
Le document suivant contient les résultats du FindBugs
La version de FindBugs est 3.0.1
La limite est
L'effort est max
Résumé
Classes |
Bugs |
Erreurs |
Classes Manquantes |
64 |
21 |
0 |
0 |
fr.paris.lutece.plugins.newsletter.business.NewsLetter
Bug |
Catégorie |
Détails |
Ligne |
Priorité |
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 |
Catégorie |
Détails |
Ligne |
Priorité |
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 |
Catégorie |
Détails |
Ligne |
Priorité |
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 |
Catégorie |
Détails |
Ligne |
Priorité |
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 |
Catégorie |
Détails |
Ligne |
Priorité |
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 |
Catégorie |
Détails |
Ligne |
Priorité |
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 |
Catégorie |
Détails |
Ligne |
Priorité |
Class fr.paris.lutece.plugins.newsletter.web.NewsLetterApp defines non-transient non-serializable instance field _captchaService |
BAD_PRACTICE |
SE_BAD_FIELD |
Non disponible |
Medium |
fr.paris.lutece.plugins.newsletter.web.NewsletterJspBean
Bug |
Catégorie |
Détails |
Ligne |
Priorité |
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 |
Catégorie |
Détails |
Ligne |
Priorité |
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 |
Catégorie |
Détails |
Ligne |
Priorité |
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 |
Catégorie |
Détails |
Ligne |
Priorité |
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 |