FindBugs Bug Detector Report
The following document contains the results of FindBugs Report
FindBugs Version is 1.3.9
Threshold is low
Effort is min
Files
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.ACCORDION_LAYOUT isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 46 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.BORDER_COLOR isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 57 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.COLLAPSED_BACK_COLOR isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 52 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.COLLAPSED_FONT_WEIGHT isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 54 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.COLLAPSED_TEXT_COLOR isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 53 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.EXPANDED_BACK_COLOR isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 49 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.EXPANDED_FONT_WEIGHT isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 51 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.EXPANDED_TEXT_COLOR isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 50 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.EXPAND_STATEHOLDER_ID isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 59 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.HOVER_BACK_COLOR isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 55 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.HOVER_TEXT_COLOR isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 56 | High |
org.apache.myfaces.custom.accordion.AbstractHtmlAccordionPanel.TOGGLING_LAYOUT isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 47 | High |
org.apache.myfaces.custom.accordion.HtmlAccordionPanelRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.accordion.HtmlAccordionPanel in org.apache.myfaces.custom.accordion.HtmlAccordionPanelRenderer.decode(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 202 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.accordion.HtmlAccordionPanel in org.apache.myfaces.custom.accordion.HtmlAccordionPanelRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 67 | Low |
org.apache.myfaces.custom.ajax.api.AjaxDecodePhaseListener
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
instanceof will always return true in org.apache.myfaces.custom.ajax.api.AjaxDecodePhaseListener.encodeAjax(UIComponent, FacesContext), since all org.apache.myfaces.custom.ajax.api.DeprecatedAjaxComponent are instances of org.apache.myfaces.custom.ajax.api.AjaxComponent | STYLE | BC_VACUOUS_INSTANCEOF | 240 | Medium |
Dead store to ajaxComponent in org.apache.myfaces.custom.ajax.api.AjaxDecodePhaseListener.beforePhase(PhaseEvent) | STYLE | DLS_DEAD_LOCAL_STORE | 88 | Medium |
org.apache.myfaces.custom.ajax.util.AjaxRendererUtils
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Method org.apache.myfaces.custom.ajax.util.AjaxRendererUtils.encodeAjax(FacesContext, UIComponent, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 255 | Medium |
org.apache.myfaces.custom.ajaxchildcombobox.HtmlAjaxChildComboBoxRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.ajaxchildcombobox.AjaxChildComboBox in org.apache.myfaces.custom.ajaxchildcombobox.HtmlAjaxChildComboBoxRenderer.encodeAjax(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 141 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.ajaxchildcombobox.AjaxChildComboBox in org.apache.myfaces.custom.ajaxchildcombobox.HtmlAjaxChildComboBoxRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 89 | Low |
org.apache.myfaces.custom.autoupdatedatatable.AutoUpdateDataTableRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.autoupdatedatatable.AutoUpdateDataTable in org.apache.myfaces.custom.autoupdatedatatable.AutoUpdateDataTableRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 101 | Low |
org.apache.myfaces.custom.clientvalidation.common.CVCall
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.clientvalidation.common.CVCall.getValidatorScriptFunctions() may expose internal representation by returning CVCall._validatorScriptFunctions | MALICIOUS_CODE | EI_EXPOSE_REP | 84 | Medium |
org.apache.myfaces.custom.clientvalidation.common.CVCall.getValidatorScriptResources() may expose internal representation by returning CVCall._validatorScriptResources | MALICIOUS_CODE | EI_EXPOSE_REP | 91 | Medium |
org.apache.myfaces.custom.clientvalidation.common.CVCall.setValidatorScriptFunctions(String[]) may expose internal representation by storing an externally mutable object into CVCall._validatorScriptFunctions | MALICIOUS_CODE | EI_EXPOSE_REP2 | 87 | Medium |
org.apache.myfaces.custom.clientvalidation.common.CVCall.setValidatorScriptResources(String[]) may expose internal representation by storing an externally mutable object into CVCall._validatorScriptResources | MALICIOUS_CODE | EI_EXPOSE_REP2 | 94 | Medium |
org.apache.myfaces.custom.clientvalidation.common.CVCall is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 40-95 | Low |
org.apache.myfaces.custom.clientvalidation.common.CVCallsHolder
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.clientvalidation.common.CVCallsHolder is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 35-49 | Low |
org.apache.myfaces.custom.clientvalidation.validationscript.ValidationScriptTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to comp in org.apache.myfaces.custom.clientvalidation.validationscript.ValidationScriptTag.setProperties(UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 58 | Medium |
Dead store to context in org.apache.myfaces.custom.clientvalidation.validationscript.ValidationScriptTag.setProperties(UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 62 | Medium |
org.apache.myfaces.custom.comparetovalidator.AbstractCompareToValidator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Comparison of String parameter using == or != in org.apache.myfaces.custom.comparetovalidator.AbstractCompareToValidator.nameForOperator(String) | BAD_PRACTICE | ES_COMPARING_PARAMETER_STRING_WITH_EQ | 183 | High |
Comparison of String parameter using == or != in org.apache.myfaces.custom.comparetovalidator.AbstractCompareToValidator.validateOperatorOnComparisonResult(String, int) | BAD_PRACTICE | ES_COMPARING_PARAMETER_STRING_WITH_EQ | 201 | High |
Redundant nullcheck of foreignValue, which is known to be non-null in org.apache.myfaces.custom.comparetovalidator.AbstractCompareToValidator.validate(FacesContext, UIComponent, Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 280 | Medium |
org.apache.myfaces.custom.comparetovalidator.ValidateCompareToTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.comparetovalidator.ValidateCompareToTag.createValidator() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 81 | Medium |
org.apache.myfaces.custom.conversation.ConversationManager
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.apache.myfaces.custom.conversation.ConversationManager() invokes org.apache.myfaces.custom.conversation.ConversationManager$ContextWiperThread.start() | MT_CORRECTNESS | SC_START_IN_CTOR | 96 | Medium |
org.apache.myfaces.custom.conversation.ConversationManager is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 43-590 | Medium |
org.apache.myfaces.custom.conversation.ConversationRequestParameterProvider
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.conversation.ConversationRequestParameterProvider.getFields() may expose internal representation by returning ConversationRequestParameterProvider.REQUEST_PARAMETERS | MALICIOUS_CODE | EI_EXPOSE_REP | 72 | Medium |
Should org.apache.myfaces.custom.conversation.ConversationRequestParameterProvider.getFields() return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 59 | Low |
org.apache.myfaces.custom.conversation.ConversationTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to comp in org.apache.myfaces.custom.conversation.ConversationTag.setProperties(UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 59 | Medium |
Dead store to context in org.apache.myfaces.custom.conversation.ConversationTag.setProperties(UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 63 | Medium |
org.apache.myfaces.custom.conversation.SeparateConversationContextTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to comp in org.apache.myfaces.custom.conversation.SeparateConversationContextTag.setProperties(UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 58 | Medium |
Dead store to context in org.apache.myfaces.custom.conversation.SeparateConversationContextTag.setProperties(UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 62 | Medium |
org.apache.myfaces.custom.conversation.UIEndConversation
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.conversation.UIEndConversation.getRestart() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 197 | Medium |
org.apache.myfaces.custom.conversation.UIEnsureConversation
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.conversation.UIEnsureConversation.getPreCheck() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 189 | Medium |
org.apache.myfaces.custom.conversation.UIStartConversation
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.conversation.UIStartConversation.getPersistence() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 202 | Medium |
org.apache.myfaces.custom.conversation.ValueBindingKey
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.conversation.ValueBindingKey implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 30-37 | Medium |
org.apache.myfaces.custom.convertDateTime.ConvertDateTimeTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.convertDateTime.ConvertDateTimeTag.createConverter() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 102 | Medium |
org.apache.myfaces.custom.convertDateTime.ConvertDateTimeTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 42-204 | Low |
org.apache.myfaces.custom.convertNumber.TypedNumberConverter
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.apache.myfaces.custom.convertNumber.TypedNumberConverter.checkJavaVersion14() where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | 563-566 | Medium |
Switch statement found in org.apache.myfaces.custom.convertNumber.TypedNumberConverter.checkJavaVersion14() where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 568-572 | Low |
org.apache.myfaces.custom.convertNumber.TypedNumberConverterTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.convertNumber.TypedNumberConverterTag.createConverter() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 163 | Medium |
org.apache.myfaces.custom.convertNumber.TypedNumberConverterTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 41-320 | Low |
org.apache.myfaces.custom.convertStringUtils.StringUtilsConverter
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Method org.apache.myfaces.custom.convertStringUtils.StringUtilsConverter.getAsObject(FacesContext, UIComponent, String) invokes toString() method on a String | PERFORMANCE | DM_STRING_TOSTRING | 64 | Low |
org.apache.myfaces.custom.convertStringUtils.StringUtilsConverterTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.convertStringUtils.StringUtilsConverterTag.createConverter() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 94 | Medium |
org.apache.myfaces.custom.convertboolean.ConvertBooleanTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.convertboolean.ConvertBooleanTag.createConverter() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 73 | Medium |
org.apache.myfaces.custom.csvvalidator.ValidateCSVTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.csvvalidator.ValidateCSVTag.createValidator() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 67 | Medium |
org.apache.myfaces.custom.dialog.ModalDialog
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.dialog.ModalDialog.getCloseButton() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 360 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dialog.ModalDialog.getCloseButton() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 354 | Medium |
org.apache.myfaces.custom.dialog.ModalDialogRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.dialog.ModalDialog in org.apache.myfaces.custom.dialog.ModalDialogRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 88 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.dialog.ModalDialog in org.apache.myfaces.custom.dialog.ModalDialogRenderer.encodeChildren(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 380 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.dialog.ModalDialog in org.apache.myfaces.custom.dialog.ModalDialogRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 98 | Low |
org.apache.myfaces.custom.dojolayouts.DojoContentPane
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.dojolayouts.DojoContentPane.getAdjustPaths() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 71 | Medium |
org.apache.myfaces.custom.dojolayouts.DojoContentPane.getCacheContent() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 96 | Medium |
org.apache.myfaces.custom.dojolayouts.DojoContentPane.getExecuteScripts() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 121 | Medium |
org.apache.myfaces.custom.dojolayouts.DojoContentPane.getExtractContent() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 146 | Medium |
org.apache.myfaces.custom.dojolayouts.DojoContentPane.getParseContent() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 258 | Medium |
org.apache.myfaces.custom.dojolayouts.DojoContentPane.getPreload() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 283 | Medium |
org.apache.myfaces.custom.dojolayouts.DojoContentPane.getRefreshOnShow() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 308 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoContentPane.getAdjustPaths() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 65 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoContentPane.getCacheContent() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 90 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoContentPane.getExecuteScripts() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 115 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoContentPane.getExtractContent() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 140 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoContentPane.getParseContent() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 252 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoContentPane.getPreload() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 277 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoContentPane.getRefreshOnShow() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 302 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoContentPane.getSizeShare() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 327 | Medium |
org.apache.myfaces.custom.dojolayouts.DojoContentPaneRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.component.StyleAware in org.apache.myfaces.custom.dojolayouts.DojoContentPaneRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 89 | Low |
org.apache.myfaces.custom.dojolayouts.DojoSplitPane
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.dojolayouts.DojoSplitPane.getPersist() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 208 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoSplitPane.getActiveSizing() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 177 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoSplitPane.getLastPoint() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 227 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoSplitPane.getPersist() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 202 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoSplitPane.getSizeShare() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 335 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoSplitPane.getSizerWidth() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 152 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.DojoSplitPane.getStartPoint() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 252 | Medium |
org.apache.myfaces.custom.dojolayouts.DojoSplitPaneRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to writer in org.apache.myfaces.custom.dojolayouts.DojoSplitPaneRenderer.encodeJavascriptBegin(FacesContext, UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 50 | Medium |
org.apache.myfaces.custom.dojolayouts.FloatingPaneBase
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getConstrainToContainer() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 71 | Medium |
org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getDisplayCloseAction() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 96 | Medium |
org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getDisplayMinimizeAction() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 121 | Medium |
org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getHasShadow() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 146 | Medium |
org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getModal() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 200 | Medium |
org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getResizable() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 225 | Medium |
org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getTitleBarDisplay() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 308 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getConstrainToContainer() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 65 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getDisplayCloseAction() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 90 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getDisplayMinimizeAction() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 115 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getHasShadow() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 140 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getModal() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 194 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getResizable() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 219 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.dojolayouts.FloatingPaneBase.getTitleBarDisplay() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 302 | Medium |
org.apache.myfaces.custom.dojolayouts.FloatingPaneBaseRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.dojolayouts.FloatingPaneBase in org.apache.myfaces.custom.dojolayouts.FloatingPaneBaseRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 93 | Low |
org.apache.myfaces.custom.effect.Effect
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.effect.Effect.getDuration() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 94 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.effect.Effect.getFade() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 119 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.effect.Effect.getPuff() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 144 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.effect.Effect.getPulsate() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 244 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.effect.Effect.getScale() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 169 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.effect.Effect.getScaleSize() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 194 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.effect.Effect.getSquish() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 219 | Medium |
org.apache.myfaces.custom.effect.EffectRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.div.Div in org.apache.myfaces.custom.effect.EffectRenderer.renderEffectsBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 203 | Low |
org.apache.myfaces.custom.exporter.ExporterActionListener
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
instanceof will always return true in org.apache.myfaces.custom.exporter.ExporterActionListener.processAction(ActionEvent), since all org.apache.myfaces.component.html.ext.HtmlDataTable are instances of org.apache.myfaces.component.html.ext.HtmlDataTable | STYLE | BC_VACUOUS_INSTANCEOF | 72 | Medium |
Method org.apache.myfaces.custom.exporter.ExporterActionListener.processAction(ActionEvent) does an unnecessary type check using instanceof operator when it can be determined statically | CORRECTNESS | SIO_SUPERFLUOUS_INSTANCEOF | 72 | Low |
org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu.getConservativeTrigger() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 124 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu.getConservativeTrigger() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 118 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu.getEffectUnits() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 143 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu.getItemHeight() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 168 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu.getItemMaxHeight() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 193 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu.getItemMaxWidth() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 218 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu.getItemPadding() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 243 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu.getItemWidth() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 268 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu.getVisibleWindow() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 64 | Medium |
org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenuRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu in org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenuRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 106 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenu in org.apache.myfaces.custom.fisheye.HtmlFishEyeNavigationMenuRenderer.encodeChildren(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 237 | Low |
org.apache.myfaces.custom.focus.HtmlFocusRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.focus.HtmlFocus in org.apache.myfaces.custom.focus.HtmlFocusRenderer.decode(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 59 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.focus.HtmlFocus in org.apache.myfaces.custom.focus.HtmlFocusRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 73 | Low |
org.apache.myfaces.custom.form.HtmlForm
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.form.HtmlForm.getPort() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 64 | Medium |
org.apache.myfaces.custom.globalId.GlobalIdTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to comp in org.apache.myfaces.custom.globalId.GlobalIdTag.setProperties(UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 58 | Medium |
Dead store to context in org.apache.myfaces.custom.globalId.GlobalIdTag.setProperties(UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 62 | Medium |
org.apache.myfaces.custom.graphicimagedynamic.GraphicImageDynamicRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.graphicimagedynamic.GraphicImageDynamic in org.apache.myfaces.custom.graphicimagedynamic.GraphicImageDynamicRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 75 | Low |
org.apache.myfaces.custom.imageloop.HtmlImageLoop
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.imageloop.HtmlImageLoop.getDelay() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 64 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.imageloop.HtmlImageLoop.getHeight() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 189 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.imageloop.HtmlImageLoop.getMaxDelay() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 114 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.imageloop.HtmlImageLoop.getMinDelay() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 89 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.imageloop.HtmlImageLoop.getTransitionTime() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 139 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.imageloop.HtmlImageLoop.getWidth() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 164 | Medium |
org.apache.myfaces.custom.imageloop.HtmlImageLoopRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.imageloop.HtmlImageLoop in org.apache.myfaces.custom.imageloop.HtmlImageLoopRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 90 | Low |
org.apache.myfaces.custom.imageloop.HtmlImageLoopRenderer.getJavascriptImageArray(FacesContext, UIComponent) invokes inefficient new String() constructor | PERFORMANCE | DM_STRING_VOID_CTOR | 139 | Medium |
Method org.apache.myfaces.custom.imageloop.HtmlImageLoopRenderer.getJavascriptImageArray(FacesContext, UIComponent) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 151 | Medium |
org.apache.myfaces.custom.inputAjax.AbstractHtmlSelectBooleanCheckboxAjax
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
instanceof will always return true in org.apache.myfaces.custom.inputAjax.AbstractHtmlSelectBooleanCheckboxAjax.decodeUISelectBoolean(FacesContext, AbstractHtmlSelectBooleanCheckboxAjax), since all org.apache.myfaces.custom.inputAjax.AbstractHtmlSelectBooleanCheckboxAjax are instances of javax.faces.component.EditableValueHolder | STYLE | BC_VACUOUS_INSTANCEOF | 103 | Medium |
Method org.apache.myfaces.custom.inputAjax.AbstractHtmlSelectBooleanCheckboxAjax.decodeUISelectBoolean(FacesContext, AbstractHtmlSelectBooleanCheckboxAjax) does an unnecessary type check using instanceof operator when it can be determined statically | CORRECTNESS | SIO_SUPERFLUOUS_INSTANCEOF | 103 | Low |
org.apache.myfaces.custom.inputAjax.HtmlInputTextAjax
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.inputAjax.HtmlInputTextAjax.getShowCancelButton() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 207 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.inputAjax.HtmlInputTextAjax.getShowOkButton() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 153 | Medium |
org.apache.myfaces.custom.inputAjax.HtmlInputTextAjaxRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.inputAjax.HtmlInputTextAjax in org.apache.myfaces.custom.inputAjax.HtmlInputTextAjaxRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 94 | Low |
org.apache.myfaces.custom.inputAjax.HtmlSelectBooleanCheckboxAjaxRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.inputAjax.HtmlSelectBooleanCheckboxAjax in org.apache.myfaces.custom.inputAjax.HtmlSelectBooleanCheckboxAjaxRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 122 | Low |
Load of known null value in org.apache.myfaces.custom.inputAjax.HtmlSelectBooleanCheckboxAjaxRenderer.encodeJavascript(FacesContext, UIComponent) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 79 | Low |
org.apache.myfaces.custom.inputAjax.HtmlSelectManyCheckboxAjaxRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.inputAjax.HtmlSelectManyCheckboxAjax in org.apache.myfaces.custom.inputAjax.HtmlSelectManyCheckboxAjaxRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 98 | Low |
org.apache.myfaces.custom.inputAjax.HtmlSelectOneRadioAjaxRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.inputAjax.HtmlSelectOneRadioAjax in org.apache.myfaces.custom.inputAjax.HtmlSelectOneRadioAjaxRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 96 | Low |
org.apache.myfaces.custom.isbnvalidator.ValidateISBNTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to facesContext in org.apache.myfaces.custom.isbnvalidator.ValidateISBNTag.createValidator() | STYLE | DLS_DEAD_LOCAL_STORE | 45 | Medium |
org.apache.myfaces.custom.loadbundle.AbstractLoadBundle$BundleMap
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of java.util.ResourceBundle.getObject(String), which is known to be non-null in org.apache.myfaces.custom.loadbundle.AbstractLoadBundle$BundleMap.containsKey(Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 236 | Low |
org.apache.myfaces.custom.media.AbstractMediaComponent
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.media.AbstractMediaComponent.COMPONENT_FAMILY isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 34 | Low |
org.apache.myfaces.custom.media.AbstractMediaComponent.COMPONENT_TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 32 | Low |
org.apache.myfaces.custom.media.AbstractMediaComponent.DEFAULT_RENDERER_TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 33 | Low |
org.apache.myfaces.custom.media.MediaRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.media.MediaComponent in org.apache.myfaces.custom.media.MediaRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 49 | Low |
org.apache.myfaces.custom.outputlinkdynamic.OutputLinkDynamicRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.outputlinkdynamic.OutputLinkDynamic in org.apache.myfaces.custom.outputlinkdynamic.OutputLinkDynamicRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 113 | Low |
org.apache.myfaces.custom.passwordStrength.AbstractPasswordStrengthComponent
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.passwordStrength.AbstractPasswordStrengthComponent.COMPONENT_FAMILY isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 56 | Low |
org.apache.myfaces.custom.passwordStrength.AbstractPasswordStrengthComponent.COMPONENT_TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 52 | Low |
org.apache.myfaces.custom.passwordStrength.AbstractPasswordStrengthComponent.DEFAULT_RENDERER_TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 54 | Low |
org.apache.myfaces.custom.passwordStrength.PasswordStrengthRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIInput in org.apache.myfaces.custom.passwordStrength.PasswordStrengthRenderer.decode(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 365 | Low |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 193 | Low |
Unread field: org.apache.myfaces.custom.passwordStrength.PasswordStrengthRenderer.BUNDLE_BASE_NAME; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 448 | Medium |
Unread field: org.apache.myfaces.custom.passwordStrength.PasswordStrengthRenderer.DEFAULT_PROGRESSBAR_HEIGHT; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 450 | Medium |
Unread field: org.apache.myfaces.custom.passwordStrength.PasswordStrengthRenderer.DEFAULT_PROGRESSBAR_VALUE; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 453 | Medium |
Unread field: org.apache.myfaces.custom.passwordStrength.PasswordStrengthRenderer.DEFAULT_PROGRESSBAR_WIDTH; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 449 | Medium |
Unread field: org.apache.myfaces.custom.passwordStrength.PasswordStrengthRenderer.PROGRESSBAR_CONTAINER_SUFFIX; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 452 | Medium |
Unread field: org.apache.myfaces.custom.passwordStrength.PasswordStrengthRenderer.PROGRESSBAR_SUFFIX; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 451 | Medium |
org.apache.myfaces.custom.ppr.PPRLifecycleWrapper$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.custom.ppr.PPRLifecycleWrapper$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 51-60 | Low |
org.apache.myfaces.custom.ppr.PPRPanelGroup
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.ppr.PPRPanelGroup.getPeriodicalUpdate() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 93 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.ppr.PPRPanelGroup.getShowDebugMessages() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 288 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.ppr.PPRPanelGroup.getStateUpdate() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 313 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.ppr.PPRPanelGroup.getWaitBeforePeriodicalUpdate() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 234 | Medium |
org.apache.myfaces.custom.ppr.PPRPanelGroupRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unread field: org.apache.myfaces.custom.ppr.PPRPanelGroupRenderer.log | PERFORMANCE | URF_UNREAD_FIELD | 46 | Low |
org.apache.myfaces.custom.ppr.PPRPhaseListener
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to componentId in org.apache.myfaces.custom.ppr.PPRPhaseListener.getComponentsByCommaSeparatedList(FacesContext, UIComponent, String, Class) | STYLE | DLS_DEAD_LOCAL_STORE | 408 | Low |
org.apache.myfaces.custom.ppr.PPRPhaseListener.addTriggeredComponent(FacesContext, String) invokes inefficient new String() constructor | PERFORMANCE | DM_STRING_VOID_CTOR | 218 | Medium |
Private method org.apache.myfaces.custom.ppr.PPRPhaseListener.getComponentsByCommaSeparatedList(FacesContext, UIComponent, String, Class) is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 404-411 | Low |
org.apache.myfaces.custom.ppr.PPRPhaseListener$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.custom.ppr.PPRPhaseListener$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 281-284 | Low |
org.apache.myfaces.custom.ppr.PPRSubmitRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.ppr.PPRSubmit in org.apache.myfaces.custom.ppr.PPRSubmitRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 51 | Low |
org.apache.myfaces.custom.ppr.PPRViewRootWrapper$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.custom.ppr.PPRViewRootWrapper$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 92-96 | Low |
org.apache.myfaces.custom.ppr.PPRViewRootWrapper$2
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.custom.ppr.PPRViewRootWrapper$2 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 117-121 | Low |
org.apache.myfaces.custom.ppr.PPRViewRootWrapper$3
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.custom.ppr.PPRViewRootWrapper$3 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 140-144 | Low |
org.apache.myfaces.custom.redirectTracker.RedirectTrackerManager
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.custom.redirectTracker.RedirectTrackerManager defines non-transient non-serializable instance field redirectEntryMap | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
org.apache.myfaces.custom.redirectTracker.RedirectTrackerManager is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 49-453 | High |
The field org.apache.myfaces.custom.redirectTracker.RedirectTrackerManager.requestBeanMap is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
RedirectTrackerManager.redirectEntryList not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available | Low |
RedirectTrackerManager.redirectEntryMap not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available | Low |
org.apache.myfaces.custom.redirectTracker.RedirectTrackerManager$Entry
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.custom.redirectTracker.RedirectTrackerManager$Entry defines non-transient non-serializable instance field beanMap | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
org.apache.myfaces.custom.redirectTracker.RedirectTrackerManager$Entry is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 62-95 | Medium |
org.apache.myfaces.custom.redirectTracker.RedirectTrackerManager$MessageEntry
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.redirectTracker.RedirectTrackerManager$MessageEntry is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 101-110 | Medium |
org.apache.myfaces.custom.redirectTracker.RedirectTrackerNavigationHandler$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.custom.redirectTracker.RedirectTrackerNavigationHandler$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 41-44 | Low |
org.apache.myfaces.custom.renderOne.UIRenderOne
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.renderOne.UIRenderOne.getValue() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 91 | Medium |
org.apache.myfaces.custom.requestParameterProvider.RequestParameterProviderManager
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.requestParameterProvider.RequestParameterProviderManager is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 40-172 | Medium |
org.apache.myfaces.custom.roundeddiv.HtmlRoundedDiv
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.roundeddiv.HtmlRoundedDiv.getBorderWidth() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 235 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.roundeddiv.HtmlRoundedDiv.getInverse() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 210 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.roundeddiv.HtmlRoundedDiv.getRadius() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 318 | Medium |
org.apache.myfaces.custom.roundeddiv.HtmlRoundedDivRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.roundeddiv.HtmlRoundedDiv in org.apache.myfaces.custom.roundeddiv.HtmlRoundedDivRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 224 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.roundeddiv.HtmlRoundedDiv in org.apache.myfaces.custom.roundeddiv.HtmlRoundedDivRenderer.encodeChildren(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 329 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.roundeddiv.HtmlRoundedDiv in org.apache.myfaces.custom.roundeddiv.HtmlRoundedDivRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 347 | Low |
Possible doublecheck on org.apache.myfaces.custom.roundeddiv.HtmlRoundedDivRenderer.cacheSize in org.apache.myfaces.custom.roundeddiv.HtmlRoundedDivRenderer.initCache(ServletContext) | MT_CORRECTNESS | DC_DOUBLECHECK | 986-992 | Medium |
Dead store of null to baos in org.apache.myfaces.custom.roundeddiv.HtmlRoundedDivRenderer.serveResource(ServletContext, HttpServletRequest, HttpServletResponse, String) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 136 | Low |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 975 | Low |
Redundant nullcheck of org.apache.myfaces.custom.roundeddiv.HtmlRoundedDivRenderer.cacheSize which is known to be null in org.apache.myfaces.custom.roundeddiv.HtmlRoundedDivRenderer.initCache(ServletContext) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE | 998 | Medium |
org.apache.myfaces.custom.roundeddiv.RoundedBorderGenerator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to width in org.apache.myfaces.custom.roundeddiv.RoundedBorderGenerator.paint3DBorderTransition(Graphics2D, Color, Color, int, int, boolean) | STYLE | DLS_DEAD_LOCAL_STORE | 367 | Medium |
org.apache.myfaces.custom.roundeddiv.RoundedBorderGenerator.main(String[]) may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 474 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.custom.roundeddiv.RoundedBorderGenerator.main(String[]) | STYLE | REC_CATCH_EXCEPTION | 480 | Low |
org.apache.myfaces.custom.scope.ScopeHolder
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.custom.scope.ScopeHolder defines non-transient non-serializable instance field oldScopes | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
Class org.apache.myfaces.custom.scope.ScopeHolder defines non-transient non-serializable instance field scopeMap | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
org.apache.myfaces.custom.scope.ScopeTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to comp in org.apache.myfaces.custom.scope.ScopeTag.setProperties(UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 58 | Medium |
Dead store to context in org.apache.myfaces.custom.scope.ScopeTag.setProperties(UIComponent) | STYLE | DLS_DEAD_LOCAL_STORE | 62 | Medium |
org.apache.myfaces.custom.scope.ScopeUtils
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.custom.scope.ScopeUtils.getManagedBean(String) | STYLE | REC_CATCH_EXCEPTION | 85 | Low |
org.apache.myfaces.custom.script.ScriptRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.script.Script in org.apache.myfaces.custom.script.ScriptRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 46 | Low |
org.apache.myfaces.custom.security.SecurityContext
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
SecurityContext.roles not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available | Low |
org.apache.myfaces.custom.statechangednotifier.StateChangedNotifier
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.statechangednotifier.StateChangedNotifier.getDisabled() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 101 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.statechangednotifier.StateChangedNotifier.getDisabled() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 95 | Medium |
org.apache.myfaces.custom.statechangednotifier.StateChangedNotifierRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.statechangednotifier.StateChangedNotifier in org.apache.myfaces.custom.statechangednotifier.StateChangedNotifierRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 71 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIOutput in org.apache.myfaces.custom.statechangednotifier.StateChangedNotifierRenderer.getConvertedValue(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 90 | Low |
org.apache.myfaces.custom.submitOnEvent.SubmitOnEventRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.submitOnEvent.SubmitOnEvent in org.apache.myfaces.custom.submitOnEvent.SubmitOnEventRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 66 | Low |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 134 | Low |
org.apache.myfaces.custom.suggest.InputSuggestRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.EditableValueHolder in org.apache.myfaces.custom.suggest.InputSuggestRenderer.decode(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 77 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.EditableValueHolder in org.apache.myfaces.custom.suggest.InputSuggestRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 97 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.ValueHolder in org.apache.myfaces.custom.suggest.InputSuggestRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 101 | Low |
Method call in org.apache.myfaces.custom.suggest.InputSuggestRenderer.encodeBegin(FacesContext, UIComponent) passes null for nonnull parameter of encodeSuggestions(FacesContext, ResponseWriter, Map, String, UIComponent) | CORRECTNESS | NP_NULL_PARAM_DEREF | 123 | Medium |
Method org.apache.myfaces.custom.suggest.InputSuggestRenderer.encodeSuggestions(FacesContext, ResponseWriter, Map, String, UIComponent) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 175 | Medium |
org.apache.myfaces.custom.suggestajax.SuggestAjax
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.suggestajax.SuggestAjax.getMaxSuggestedItems() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 66 | Medium |
org.apache.myfaces.custom.suggestajax.SuggestAjaxRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.suggestajax.SuggestAjax in org.apache.myfaces.custom.suggestajax.SuggestAjaxRenderer.getSuggestedItems(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 46 | Low |
org.apache.myfaces.custom.suggestajax.SuggestAjaxRenderer.addQueryString(String, String) checks to see if result of String.indexOf is positive | STYLE | RV_CHECK_FOR_POSITIVE_INDEXOF | 94 | Low |
org.apache.myfaces.custom.suggestajax.inputsuggestajax.InputSuggestAjax
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.suggestajax.inputsuggestajax.InputSuggestAjax.getAutoComplete() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 66 | Medium |
org.apache.myfaces.custom.suggestajax.inputsuggestajax.InputSuggestAjaxRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.suggestajax.inputsuggestajax.InputSuggestAjax in org.apache.myfaces.custom.suggestajax.inputsuggestajax.InputSuggestAjaxRenderer.encodeAjax(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 255 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.suggestajax.inputsuggestajax.InputSuggestAjax in org.apache.myfaces.custom.suggestajax.inputsuggestajax.InputSuggestAjaxRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 79 | Low |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 286 | Low |
Method org.apache.myfaces.custom.suggestajax.inputsuggestajax.InputSuggestAjaxRenderer.encodeEnd(FacesContext, UIComponent) invokes toString() method on a String | PERFORMANCE | DM_STRING_TOSTRING | 186 | Low |
org.apache.myfaces.custom.suggestajax.tablesuggestajax.HtmlOutputTextTag
Bug | Category | Details | Line | Priority |
---|
org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjax
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjax.getBetweenKeyUp() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 66 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjax.getStartRequest() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 91 | Medium |
org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjaxRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjax in org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjaxRenderer.encodeAjax(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 253 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjax in org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjaxRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 84 | Low |
org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjaxRenderer$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjax in org.apache.myfaces.custom.suggestajax.tablesuggestajax.TableSuggestAjaxRenderer$1.execute(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 240 | Low |
org.apache.myfaces.custom.table.FilterTable
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.table.FilterTable.getAlternateRows() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 182 | Medium |
org.apache.myfaces.custom.table.FilterTable.getMultiple() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 157 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.table.FilterTable.getAlternateRows() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 176 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.table.FilterTable.getBorder() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 276 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.table.FilterTable.getCellpadding() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 226 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.table.FilterTable.getCellspacing() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 251 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.table.FilterTable.getMaxSortable() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 201 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.table.FilterTable.getMultiple() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 151 | Medium |
org.apache.myfaces.custom.table.FilterTableRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.table.FilterTable in org.apache.myfaces.custom.table.FilterTableRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 66 | Low |
org.apache.myfaces.custom.table.SortableColumn
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.table.SortableColumn.getEscape() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 271 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.table.SortableColumn.getEscape() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 265 | Medium |
org.apache.myfaces.custom.timednotifier.TimedNotifier
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.timednotifier.TimedNotifier.getDisabled() has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 100 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.timednotifier.TimedNotifier.getDisabled() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 94 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.timednotifier.TimedNotifier.getHideDelay() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 131 | Medium |
Redundant nullcheck of vb, which is known to be non-null in org.apache.myfaces.custom.timednotifier.TimedNotifier.getShowDelay() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 168 | Medium |
org.apache.myfaces.custom.timednotifier.TimedNotifierRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.custom.timednotifier.TimedNotifier in org.apache.myfaces.custom.timednotifier.TimedNotifierRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 68 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIOutput in org.apache.myfaces.custom.timednotifier.TimedNotifierRenderer.getConvertedValue(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 86 | Low |
org.apache.myfaces.custom.urlvalidator.AbstractUrlValidator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.custom.urlvalidator.AbstractUrlValidator.getSchemesList() return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 117 | Low |
org.apache.myfaces.custom.urlvalidator.ValidateUrlTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.urlvalidator.ValidateUrlTag.createValidator() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 74 | Medium |
org.apache.myfaces.custom.valueChangeNotifier.ValueChangeCollector$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.custom.valueChangeNotifier.ValueChangeCollector$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 84-100 | Low |
org.apache.myfaces.custom.valueChangeNotifier.ValueChangeManager
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.valueChangeNotifier.ValueChangeManager.SIGNATURE should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 41 | Medium |
org.apache.myfaces.custom.valueChangeNotifier.ValueChangeNotifierTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.custom.valueChangeNotifier.ValueChangeNotifierTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 53-121 | Medium |
org.apache.myfaces.renderkit.html.ext.HtmlButtonExRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.html.HtmlCommandButton in org.apache.myfaces.renderkit.html.ext.HtmlButtonExRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 44 | Low |
org.apache.myfaces.renderkit.template.TemplateRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Possible null pointer dereference of templateEncoder in org.apache.myfaces.renderkit.template.TemplateRenderer.encodeTemplate(FacesContext, UIComponent, String) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 91 | Medium |
org.apache.myfaces.tomahawk.util.ErrorPageWriter
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Incorrect lazy initialization of static field org.apache.myfaces.tomahawk.util.ErrorPageWriter.DEBUG_PARTS in org.apache.myfaces.tomahawk.util.ErrorPageWriter.init(FacesContext) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 112-113 | Low |
Incorrect lazy initialization of static field org.apache.myfaces.tomahawk.util.ErrorPageWriter.ERROR_PARTS in org.apache.myfaces.tomahawk.util.ErrorPageWriter.init(FacesContext) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 108-109 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.tomahawk.util.ErrorPageWriter.writeAttributes(Writer, UIComponent) | STYLE | REC_CATCH_EXCEPTION | 441 | High |
Exception is caught when Exception is not thrown in org.apache.myfaces.tomahawk.util.ErrorPageWriter.initCauseIfAvailable(Throwable, Throwable) | STYLE | REC_CATCH_EXCEPTION | 612 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.tomahawk.util.ErrorPageWriter.writeAttributes(Writer, UIComponent) | STYLE | REC_CATCH_EXCEPTION | 429 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.tomahawk.util.ErrorPageWriter.getCause(Throwable) | STYLE | REC_CATCH_EXCEPTION | 558 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.tomahawk.util.ErrorPageWriter.initCausePerReflection(Throwable, String) | STYLE | REC_CATCH_EXCEPTION | 568 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.tomahawk.util.ErrorPageWriter.writeAttributes(Writer, UIComponent) | STYLE | REC_CATCH_EXCEPTION | 429 | Low |
org.apache.myfaces.tomahawk.util.ErrorRedirectJSFPageHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.apache.myfaces.tomahawk.util.ErrorRedirectJSFPageHandler.handleException(FacesContext, Exception) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 132 | Medium |
Load of known null value in org.apache.myfaces.tomahawk.util.ErrorRedirectJSFPageHandler.handleThrowable(FacesContext, Throwable) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 198 | Medium |