FindBugs Bug Detector Report
The following document contains the results of FindBugs Report
FindBugs Version is 1.0.0
Threshold is Low
Effort is Default
Files
org.apache.myfaces.trinidad.resource.AggregatingResourceLoader$AggregatingURLConnection
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.resource.AggregatingResourceLoader$AggregatingURLConnection.getInputStream() may fail to close stream | CORRECTNESS | OS_OPEN_STREAM | 235 |
org.apache.myfaces.trinidad.validator.DateRestrictionValidator
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.trinidad.validator.DateRestrictionValidator._getInvalidMonths() invokes dubious String.toUpperCase() or String.toLowerCase; use the Locale parameterized version instead | I18N | DM_CONVERT_CASE | 554 |
| Method org.apache.myfaces.trinidad.validator.DateRestrictionValidator._getInvalidDaysOfWeek() invokes dubious String.toUpperCase() or String.toLowerCase; use the Locale parameterized version instead | I18N | DM_CONVERT_CASE | 569 |
org.apache.myfaces.trinidad.bean.util.StateUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.trinidad.bean.util.StateUtils.<clinit>() invokes dubious String.toUpperCase() or String.toLowerCase; use the Locale parameterized version instead | I18N | DM_CONVERT_CASE | 77 |
org.apache.myfaces.trinidad.bean.FacesBeanFactory
| Bug | Category | Details | Line |
|---|---|---|---|
| Possible null pointer dereference in org.apache.myfaces.trinidad.bean.FacesBeanFactory.createFacesBean(String,String) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 105 |
| Possible null pointer dereference in org.apache.myfaces.trinidad.bean.FacesBeanFactory.createFacesBean(String,String) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 109 |
| Possible null pointer dereference in org.apache.myfaces.trinidad.bean.FacesBeanFactory.createFacesBean(String,String) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 114 |
org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding$SerializableValueExpressionValueBinding
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding$SerializableValueExpressionValueBinding is Serializable but its superclass doesn't define an accessible void constructor | CORRECTNESS | SE_NO_SUITABLE_CONSTRUCTOR | Not available |
| org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding$SerializableValueExpressionValueBinding is Serializable; consider declaring a serialVersionUID | CORRECTNESS | SE_NO_SERIALVERSIONID | Not available |
org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding$StateHolderValueExpressionValueBinding
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.el.ValueExpression to class javax.faces.component.StateHolder in org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding$StateHolderValueExpressionValueBinding.<init>(javax.el.ValueExpression) | STYLE | BC_UNCONFIRMED_CAST | 170 |
org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding$SerializableStateHolderValueExpressionValueBinding
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding$SerializableStateHolderValueExpressionValueBinding is Serializable but its superclass doesn't define an accessible void constructor | CORRECTNESS | SE_NO_SUITABLE_CONSTRUCTOR | Not available |
| org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding$SerializableStateHolderValueExpressionValueBinding is Serializable; consider declaring a serialVersionUID | CORRECTNESS | SE_NO_SERIALVERSIONID | Not available |
org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding.getValueBinding(javax.el.ValueExpression) does an unnecessary type check using instanceof operator when it can be determined statically | CORRECTNESS | SIO_SUPERFLUOUS_INSTANCEOF | 48 |
| Method org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding.getValueBinding(javax.el.ValueExpression) does an unnecessary type check using instanceof operator when it can be determined statically | CORRECTNESS | SIO_SUPERFLUOUS_INSTANCEOF | 53 |
| instanceof will always return true in org.apache.myfaces.trinidad.bean.ValueExpressionValueBinding.getValueBinding(javax.el.ValueExpression), since all javax.el.ValueExpression are instances of java.io.Serializable | STYLE | BC_VACUOUS_INSTANCEOF | 53 |
org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$SerializableValueBindingValueExpression
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$SerializableValueBindingValueExpression is Serializable; consider declaring a serialVersionUID | CORRECTNESS | SE_NO_SERIALVERSIONID | Not available |
| Class org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$SerializableValueBindingValueExpression implements same interface as superclass. | STYLE | RI_REDUNDANT_INTERFACES | Not available |
org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$StateHolderValueBindingValueExpression
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$StateHolderValueBindingValueExpression is Serializable; consider declaring a serialVersionUID | CORRECTNESS | SE_NO_SERIALVERSIONID | Not available |
| Class org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$StateHolderValueBindingValueExpression defines non-transient non-serializable instance field org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$StateHolderValueBindingValueExpression._stateHolder | CORRECTNESS | SE_BAD_FIELD | Not available |
| Unchecked/unconfirmed cast from javax.faces.el.ValueBinding to class javax.faces.component.StateHolder in org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$StateHolderValueBindingValueExpression.<init>(javax.faces.el.ValueBinding) | STYLE | BC_UNCONFIRMED_CAST | 186 |
org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$SerializableStateHolderValueBindingValueExpression
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$SerializableStateHolderValueBindingValueExpression is Serializable; consider declaring a serialVersionUID | CORRECTNESS | SE_NO_SERIALVERSIONID | Not available |
| Class org.apache.myfaces.trinidad.bean.ValueBindingValueExpression$SerializableStateHolderValueBindingValueExpression implements same interface as superclass. | STYLE | RI_REDUNDANT_INTERFACES | Not available |
org.apache.myfaces.trinidad.bean.ValueBindingValueExpression
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.bean.ValueBindingValueExpression is Serializable; consider declaring a serialVersionUID | CORRECTNESS | SE_NO_SERIALVERSIONID | Not available |
| Class org.apache.myfaces.trinidad.bean.ValueBindingValueExpression defines non-transient non-serializable instance field org.apache.myfaces.trinidad.bean.ValueBindingValueExpression._binding | CORRECTNESS | SE_BAD_FIELD | Not available |
org.apache.myfaces.trinidad.component.UIXCollection
| Bug | Category | Details | Line |
|---|---|---|---|
| Useless control flow in org.apache.myfaces.trinidad.component.UIXCollection.encodeBegin(javax.faces.context.FacesContext) | CORRECTNESS | UCF_USELESS_CONTROL_FLOW | 516 |
| Useless control flow in org.apache.myfaces.trinidad.component.UIXCollection.encodeEnd(javax.faces.context.FacesContext) | CORRECTNESS | UCF_USELESS_CONTROL_FLOW | 526 |
| Dead store to childCount in method org.apache.myfaces.trinidad.component.UIXCollection.saveStampState(javax.faces.context.FacesContext,javax.faces.component.UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 803 |
org.apache.myfaces.trinidad.component.UIXComponentBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.trinidad.component.UIXComponentBase._createType() catches Exception, but Exception is not thrown in the try block and RuntimeException is not explicitly caught | STYLE | REC_CATCH_EXCEPTION | 1628 |
org.apache.myfaces.trinidad.component.StampState$SDState
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to class org.apache.myfaces.trinidad.component.UIXShowDetail in org.apache.myfaces.trinidad.component.StampState$SDState.saveRowState(javax.faces.component.UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 361 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to class org.apache.myfaces.trinidad.component.UIXShowDetail in org.apache.myfaces.trinidad.component.StampState$SDState.restoreRowState(javax.faces.component.UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 368 |
org.apache.myfaces.trinidad.component.StampState$TableState
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to class org.apache.myfaces.trinidad.component.UIXCollection in org.apache.myfaces.trinidad.component.StampState$TableState.saveRowState(javax.faces.component.UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 407 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to class org.apache.myfaces.trinidad.component.UIXCollection in org.apache.myfaces.trinidad.component.StampState$TableState.restoreRowState(javax.faces.component.UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 413 |
org.apache.myfaces.trinidad.component.StampState$EVHState
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to class javax.faces.component.EditableValueHolder in org.apache.myfaces.trinidad.component.StampState$EVHState.saveRowState(javax.faces.component.UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 439 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to class javax.faces.component.EditableValueHolder in org.apache.myfaces.trinidad.component.StampState$EVHState.restoreRowState(javax.faces.component.UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 451 |
org.apache.myfaces.trinidad.component.UIXShowOne$UndisclosureCallback
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to class org.apache.myfaces.trinidad.component.UIXShowDetail in org.apache.myfaces.trinidad.component.UIXShowOne$UndisclosureCallback.processComponent(javax.faces.context.FacesContext,ComponentProcessingContext,javax.faces.component.UIComponent,UIXShowOne$UndisclosureCallbackState) | STYLE | BC_UNCONFIRMED_CAST | 155 |
org.apache.myfaces.trinidad.component.ChildArrayList
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.component.ChildArrayList is Serializable; consider declaring a serialVersionUID | CORRECTNESS | SE_NO_SERIALVERSIONID | Not available |
| Class org.apache.myfaces.trinidad.component.ChildArrayList defines non-transient non-serializable instance field org.apache.myfaces.trinidad.component.ChildArrayList._parent | CORRECTNESS | SE_BAD_FIELD | Not available |
org.apache.myfaces.trinidad.component.UIXPage
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.trinidad.component.UIXPage.processFacetsAndChildren(javax.faces.context.FacesContext,javax.faces.event.PhaseId) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 160 |
org.apache.myfaces.trinidad.component.UIXTree
| Bug | Category | Details | Line |
|---|---|---|---|
| Class org.apache.myfaces.trinidad.component.UIXTree implements same interface as superclass. | STYLE | RI_REDUNDANT_INTERFACES | Not available |
org.apache.myfaces.trinidad.component.MethodBindingMethodExpression
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.component.MethodBindingMethodExpression is Serializable; consider declaring a serialVersionUID | CORRECTNESS | SE_NO_SERIALVERSIONID | Not available |
| Class org.apache.myfaces.trinidad.component.MethodBindingMethodExpression defines non-transient non-serializable instance field org.apache.myfaces.trinidad.component.MethodBindingMethodExpression._binding | CORRECTNESS | SE_BAD_FIELD | Not available |
| Class org.apache.myfaces.trinidad.component.MethodBindingMethodExpression implements same interface as superclass. | STYLE | RI_REDUNDANT_INTERFACES | Not available |
org.apache.myfaces.trinidad.component.UIXNavigationPath
| Bug | Category | Details | Line |
|---|---|---|---|
| Class org.apache.myfaces.trinidad.component.UIXNavigationPath implements same interface as superclass. | STYLE | RI_REDUNDANT_INTERFACES | Not available |
org.apache.myfaces.trinidad.component.UIXNavigationLevel
| Bug | Category | Details | Line |
|---|---|---|---|
| Class org.apache.myfaces.trinidad.component.UIXNavigationLevel implements same interface as superclass. | STYLE | RI_REDUNDANT_INTERFACES | Not available |
org.apache.myfaces.trinidad.model.XMLMenuModel
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.trinidad.model.XMLMenuModel._createModel() catches Exception, but Exception is not thrown in the try block and RuntimeException is not explicitly caught | STYLE | REC_CATCH_EXCEPTION | 648 |
org.apache.myfaces.trinidad.model.RowKeySetTreeImpl
| Bug | Category | Details | Line |
|---|---|---|---|
| Private method org.apache.myfaces.trinidad.model.RowKeySetTreeImpl._dumpFlags() is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 570-580 |
| Private method org.apache.myfaces.trinidad.model.RowKeySetTreeImpl._containsDefaultNodes() is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 552-562 |
| Unchecked/unconfirmed cast from org.apache.myfaces.trinidad.model.CollectionModel to class org.apache.myfaces.trinidad.model.TreeModel in org.apache.myfaces.trinidad.model.RowKeySetTreeImpl.setCollectionModel(CollectionModel) | STYLE | BC_UNCONFIRMED_CAST | 311 |
org.apache.myfaces.trinidad.model.SortableModel$Comp
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.model.SortableModel$Comp implements Comparator but not Serializable | STYLE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | Not available |
org.apache.myfaces.trinidad.model.SortableModel$Inverter
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.model.SortableModel$Inverter implements Comparator but not Serializable | STYLE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | Not available |
org.apache.myfaces.trinidad.event.AttributeChangeEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.AttributeChangeListener in org.apache.myfaces.trinidad.event.AttributeChangeEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 77 |
org.apache.myfaces.trinidad.event.ReturnEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.ReturnListener in org.apache.myfaces.trinidad.event.ReturnEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 73 |
org.apache.myfaces.trinidad.event.DisclosureEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.DisclosureListener in org.apache.myfaces.trinidad.event.DisclosureEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 48 |
org.apache.myfaces.trinidad.event.SortEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.SortListener in org.apache.myfaces.trinidad.event.SortEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 63 |
org.apache.myfaces.trinidad.event.PollEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.PollListener in org.apache.myfaces.trinidad.event.PollEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 40 |
org.apache.myfaces.trinidad.event.RowDisclosureEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.RowDisclosureListener in org.apache.myfaces.trinidad.event.RowDisclosureEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 62 |
org.apache.myfaces.trinidad.event.RangeChangeEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.RangeChangeListener in org.apache.myfaces.trinidad.event.RangeChangeEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 103 |
org.apache.myfaces.trinidad.event.SelectionEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.SelectionListener in org.apache.myfaces.trinidad.event.SelectionEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 61 |
org.apache.myfaces.trinidad.event.FocusEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.FocusListener in org.apache.myfaces.trinidad.event.FocusEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 46 |
org.apache.myfaces.trinidad.event.LaunchEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.LaunchListener in org.apache.myfaces.trinidad.event.LaunchEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 108 |
org.apache.myfaces.trinidad.event.ChartDrillDownEvent
| Bug | Category | Details | Line |
|---|---|---|---|
| Should org.apache.myfaces.trinidad.event.ChartDrillDownEvent.getSeriesIndices() return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 67 |
| Should org.apache.myfaces.trinidad.event.ChartDrillDownEvent.getYValueIndices() return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 84 |
| Should org.apache.myfaces.trinidad.event.ChartDrillDownEvent.getYValues() return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 100 |
| Should org.apache.myfaces.trinidad.event.ChartDrillDownEvent.getXValues() return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 117 |
| Unchecked/unconfirmed cast from javax.faces.event.FacesListener to class org.apache.myfaces.trinidad.event.ChartDrillDownListener in org.apache.myfaces.trinidad.event.ChartDrillDownEvent.processListener(javax.faces.event.FacesListener) | STYLE | BC_UNCONFIRMED_CAST | 54 |
org.apache.myfaces.trinidad.webapp.ResourceServlet$_ResourceLifecycle
| Bug | Category | Details | Line |
|---|---|---|---|
| Should org.apache.myfaces.trinidad.webapp.ResourceServlet$_ResourceLifecycle.getPhaseListeners() return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 505 |
org.apache.myfaces.trinidad.webapp.ResourceServlet
| Bug | Category | Details | Line |
|---|---|---|---|
| Class org.apache.myfaces.trinidad.webapp.ResourceServlet defines non-transient non-serializable instance field org.apache.myfaces.trinidad.webapp.ResourceServlet._facesContextFactory | CORRECTNESS | SE_BAD_FIELD | Not available |
| Class org.apache.myfaces.trinidad.webapp.ResourceServlet defines non-transient non-serializable instance field org.apache.myfaces.trinidad.webapp.ResourceServlet._lifecycle | CORRECTNESS | SE_BAD_FIELD | Not available |
| org.apache.myfaces.trinidad.webapp.ResourceServlet$_ResourceLifecycle stored into non-transient field org.apache.myfaces.trinidad.webapp.ResourceServlet._lifecycle | CORRECTNESS | SE_BAD_FIELD_STORE | 129 |
| Class org/apache/myfaces/trinidad/webapp/ResourceServlet._loaders extends Servlet class and uses instance variables. | STYLE | MTIA_SUSPECT_SERVLET_INSTANCE_FIELD | 91 |
| Class org/apache/myfaces/trinidad/webapp/ResourceServlet._facesContextFactory extends Servlet class and uses instance variables. | STYLE | MTIA_SUSPECT_SERVLET_INSTANCE_FIELD | 92 |
| Class org/apache/myfaces/trinidad/webapp/ResourceServlet._lifecycle extends Servlet class and uses instance variables. | STYLE | MTIA_SUSPECT_SERVLET_INSTANCE_FIELD | 93 |
| Class org/apache/myfaces/trinidad/webapp/ResourceServlet._debug extends Servlet class and uses instance variables. | STYLE | MTIA_SUSPECT_SERVLET_INSTANCE_FIELD | 404 |
org.apache.myfaces.trinidad.webapp.UIXComponentELTag
| Bug | Category | Details | Line |
|---|---|---|---|
| Should org.apache.myfaces.trinidad.webapp.UIXComponentELTag._parseNameTokens(Object) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 356 |
org.apache.myfaces.trinidad.webapp.UIXComponentTag
| Bug | Category | Details | Line |
|---|---|---|---|
| Should org.apache.myfaces.trinidad.webapp.UIXComponentTag._parseNameTokens(String) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 496 |
| Unchecked/unconfirmed cast from javax.servlet.jsp.tagext.Tag to class org.apache.myfaces.trinidad.webapp.ELContextTag in org.apache.myfaces.trinidad.webapp.UIXComponentTag.doStartTag() | STYLE | BC_UNCONFIRMED_CAST | 76 |
org.apache.myfaces.trinidad.util.RequestStateMap
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.util.RequestStateMap is Serializable; consider declaring a serialVersionUID | CORRECTNESS | SE_NO_SERIALVERSIONID | Not available |
org.apache.myfaces.trinidad.util.ClassLoaderUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.trinidad.util.ClassLoaderUtils.getServices(String) catches Exception, but Exception is not thrown in the try block and RuntimeException is not explicitly caught | STYLE | REC_CATCH_EXCEPTION | 275 |
org.apache.myfaces.trinidad.util.URLUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.util.URLUtils.getLastModified(java.net.URLConnection) might ignore java.lang.Exception | CORRECTNESS | DE_MIGHT_IGNORE | 78 |
org.apache.myfaces.trinidad.util.CollectionUtils$CheckedSerializationMap
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.trinidad.util.CollectionUtils$CheckedSerializationMap is Serializable; consider declaring a serialVersionUID | CORRECTNESS | SE_NO_SERIALVERSIONID | Not available |
org.apache.myfaces.trinidad.util.ComponentUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Nullcheck at ComponentUtils.java:[line 429] of value previously dereferenced at org.apache.myfaces.trinidad.util.ComponentUtils.getScopedIdForComponent(javax.faces.component.UIComponent,javax.faces.component.UIComponent) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 429 |
org.apache.myfaces.trinidad.convert.DateTimeConverter
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.trinidad.convert.DateTimeConverter._getSimpleDateFormat(String,java.util.Locale) invokes dubious String.toUpperCase() or String.toLowerCase; use the Locale parameterized version instead | I18N | DM_CONVERT_CASE | 1501 |
| Class org.apache.myfaces.trinidad.convert.DateTimeConverter implements same interface as superclass. | STYLE | RI_REDUNDANT_INTERFACES | Not available |
org.apache.myfaces.trinidad.convert.ColorConverter
| Bug | Category | Details | Line |
|---|---|---|---|
| Possible null pointer dereference in org.apache.myfaces.trinidad.convert.ColorConverter._isEqualPatterns(String[]) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 627 |
| Possible null pointer dereference in org.apache.myfaces.trinidad.convert.ColorConverter._isEqualPatterns(String[]) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 627 |
org.apache.myfaces.trinidad.convert.RGBColorFormat
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.trinidad.convert.RGBColorFormat._subFormatHex(int,int,StringBuffer) invokes dubious String.toUpperCase() or String.toLowerCase; use the Locale parameterized version instead | I18N | DM_CONVERT_CASE | 553 |
| org.apache.myfaces.trinidad.convert.ColorConverter.DEFAULT_COLOR_FORMAT_PATTERNS should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 149 |
| The class org.apache.myfaces.trinidad.component.UIXComponentRef$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Should org.apache.myfaces.trinidad.component.UIXShowOne$UndisclosureCallback be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
| Should org.apache.myfaces.trinidad.resource.AggregatingResourceLoader$AggregatingURLStreamHandler be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
| The class org.apache.myfaces.trinidad.webapp.ResourceServlet$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |