The following document contains the results of FindBugs Report
FindBugs Version is 1.3.9
Threshold is low
Effort is min
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.apache.myfaces.examples.autoupdatedatatable.AutoUpdateDataTableBean.getTestList() uses the nextDouble method of Random to generate a random integer; using nextInt is more efficient | PERFORMANCE | DM_NEXTINT_VIA_NEXTDOUBLE | 36 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.clientvalidation.Customer is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 27 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.clientvalidation.CustomerCreateBean is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 31 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to i in org.apache.myfaces.examples.data.SortableTableBean.getCars() | STYLE | DLS_DEAD_LOCAL_STORE | 101 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.dateTimeConverter.DateTimeConverterBean.getDate1() may expose internal representation by returning DateTimeConverterBean.date1 | MALICIOUS_CODE | EI_EXPOSE_REP | 42 | Medium |
| org.apache.myfaces.examples.dateTimeConverter.DateTimeConverterBean.getDate2() may expose internal representation by returning DateTimeConverterBean.date2 | MALICIOUS_CODE | EI_EXPOSE_REP | 52 | Medium |
| org.apache.myfaces.examples.dateTimeConverter.DateTimeConverterBean.setDate1(Date) may expose internal representation by storing an externally mutable object into DateTimeConverterBean.date1 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 47 | Medium |
| org.apache.myfaces.examples.dateTimeConverter.DateTimeConverterBean.setDate2(Date) may expose internal representation by storing an externally mutable object into DateTimeConverterBean.date2 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 57 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.exporter.ExporterBean is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 29 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.fisheye.FishEyeHandler is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 38 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Redundant nullcheck of java.util.ResourceBundle.getObject(String), which is known to be non-null in org.apache.myfaces.examples.fisheye.FishEyeHandler$BundleMap.containsKey(Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 126 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.imageloop.ImageLoopBean.getImageArray() may expose internal representation by returning ImageLoopBean._imageArray | MALICIOUS_CODE | EI_EXPOSE_REP | 51 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.EditableValueHolder in org.apache.myfaces.examples.inputAjax.InputAjaxBean.validateFormText1(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 101 | Low |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.EditableValueHolder in org.apache.myfaces.examples.inputAjax.InputAjaxBean.validateFormText2(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 114 | Low |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.EditableValueHolder in org.apache.myfaces.examples.inputAjax.InputAjaxBean.validateText1(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 87 | Low |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.EditableValueHolder in org.apache.myfaces.examples.inputAjax.InputAjaxBean.validateText2(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 73 | Low |
| new org.apache.myfaces.examples.inputAjax.InputAjaxBean() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 53 | Medium |
| org.apache.myfaces.examples.inputAjax.InputAjaxBean.getDate1() may expose internal representation by returning InputAjaxBean.date1 | MALICIOUS_CODE | EI_EXPOSE_REP | 196 | Medium |
| org.apache.myfaces.examples.inputAjax.InputAjaxBean.setDate1(Date) may expose internal representation by storing an externally mutable object into InputAjaxBean.date1 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 201 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.inputSuggestAjax.Address is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 42 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.inputSuggestAjax.InputSuggestAjaxBean.dummyDataBaseAddresses isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 40 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| UserHandler.choices not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Low | |
| UserHandler.user not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 184 | Low |
| Method org.apache.myfaces.examples.ppr.PPRExampleBean.getPeriodicalUpdatedValues() uses the nextDouble method of Random to generate a random integer; using nextInt is more efficient | PERFORMANCE | DM_NEXTINT_VIA_NEXTDOUBLE | 216 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.apache.myfaces.examples.ppr.PPRScrollerExampleBean defines non-transient non-serializable instance field carMap | BAD_PRACTICE | SE_BAD_FIELD | Low | |
| org.apache.myfaces.examples.ppr.PPRScrollerExampleBean is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 30 | Low |
| org.apache.myfaces.examples.ppr.PPRScrollerExampleBean is Serializable but its superclass doesn't define an accessible void constructor | BAD_PRACTICE | SE_NO_SUITABLE_CONSTRUCTOR | 30 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class org.apache.myfaces.examples.ppr.PPRScrollerExampleBean$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 71 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.renderone.RenderOneBean is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 27 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 198 | Low |
| org.apache.myfaces.examples.roundeddiv.RoundedDivBean is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 31 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.valueChangeNotifier.NotifierBean$Entry is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 64 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.apache.myfaces.examples.valueChangeNotifier.NotifierBean$EntryList is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 44 | Low |
