FindBugs Bug Detector Report
The following document contains the results of FindBugs
FindBugs Version is 3.0.1
Threshold is low
Effort is min
Files
org.apache.myfaces.application.ApplicationImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase() in new org.apache.myfaces.application.ApplicationImpl(RuntimeConfig) | I18N | DM_CONVERT_CASE | 320 | Low |
converterClass could be null and is guaranteed to be dereferenced in org.apache.myfaces.application.ApplicationImpl.internalCreateConverter(Class) | CORRECTNESS | NP_GUARANTEED_DEREF | 1769 | High |
Load of known null value in org.apache.myfaces.application.ApplicationImpl.publishEvent(FacesContext, Class, Class, Object) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 618 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.application.ApplicationImpl.internalCreateConverter(Class) | STYLE | REC_CATCH_EXCEPTION | 1753 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.application.ApplicationImpl.createBehavior(String) | STYLE | REC_CATCH_EXCEPTION | 1281 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.application.ApplicationImpl.createConverter(String) | STYLE | REC_CATCH_EXCEPTION | 1640 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.application.ApplicationImpl.createValidator(String) | STYLE | REC_CATCH_EXCEPTION | 2166 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.application.ApplicationImpl.internalCreateConverter(Class) | STYLE | REC_CATCH_EXCEPTION | 1665-1820 | Low |
org.apache.myfaces.application.ApplicationImpl$SystemListenerEntry
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Inconsistent synchronization of org.apache.myfaces.application.ApplicationImpl$SystemListenerEntry._lstSystemEventListener; locked 42% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 2741 | Medium |
Inconsistent synchronization of org.apache.myfaces.application.ApplicationImpl$SystemListenerEntry._sourceClassMap; locked 40% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 2721 | Medium |
org.apache.myfaces.application.DefaultResourceHandlerSupport
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.application.DefaultResourceHandlerSupport.getContractResourceLoaders() may expose internal representation by returning DefaultResourceHandlerSupport._contractResourceLoaders | MALICIOUS_CODE | EI_EXPOSE_REP | 168 | Medium |
org.apache.myfaces.application.DefaultResourceHandlerSupport.getResourceLoaders() may expose internal representation by returning DefaultResourceHandlerSupport._resourceLoaders | MALICIOUS_CODE | EI_EXPOSE_REP | 132 | Medium |
org.apache.myfaces.application.DefaultResourceHandlerSupport.getViewResourceLoaders() may expose internal representation by returning DefaultResourceHandlerSupport._viewResourceLoaders | MALICIOUS_CODE | EI_EXPOSE_REP | 192 | Medium |
org.apache.myfaces.application.NavigationHandlerImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to sourceFlow in org.apache.myfaces.application.NavigationHandlerImpl.applyFlowTransition(FacesContext, NavigationHandlerImpl$NavigationContext) | STYLE | DLS_DEAD_LOCAL_STORE | 405 | Low |
Integral division result cast to double or float in org.apache.myfaces.application.NavigationHandlerImpl.getOutcomeNavigationCase(FacesContext, String, String) | STYLE | ICAST_IDIV_CAST_TO_DOUBLE | 1164 | Medium |
Inconsistent synchronization of org.apache.myfaces.application.NavigationHandlerImpl._wildcardPatterns; locked 50% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 1411 | Low |
Possible null pointer dereference of navigationContext in org.apache.myfaces.application.NavigationHandlerImpl.applyFlowTransition(FacesContext, NavigationHandlerImpl$NavigationContext) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 393 | Medium |
org.apache.myfaces.application.NavigationHandlerImpl$KeyComparator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.application.NavigationHandlerImpl$KeyComparator.compare(_WildcardPattern, _WildcardPattern) negates the return value of String.compareTo(String) | BAD_PRACTICE | RV_NEGATING_RESULT_OF_COMPARETO | 1557 | Medium |
org.apache.myfaces.application.NavigationHandlerImpl$KeyComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 1553-1557 | Medium |
org.apache.myfaces.application.ResourceHandlerImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of resourceName, which is known to be non-null in org.apache.myfaces.application.ResourceHandlerImpl.deriveResourceMeta(FacesContext, ContractResourceLoader, String, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 1479 | Medium |
Redundant nullcheck of resourceName, which is known to be non-null in org.apache.myfaces.application.ResourceHandlerImpl.deriveResourceMeta(FacesContext, ResourceLoader, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 1304 | Medium |
Redundant nullcheck of resourceBasePath, which is known to be non-null in org.apache.myfaces.application.ResourceHandlerImpl.handleResourceRequest(FacesContext) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 570 | Medium |
org.apache.myfaces.application.StateManagerImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of uiViewRoot, which is known to be non-null in org.apache.myfaces.application.StateManagerImpl.restoreView(FacesContext, String, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 149 | Low |
org.apache.myfaces.application.ViewResourceIterator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unread field: org.apache.myfaces.application.ViewResourceIterator.support | PERFORMANCE | URF_UNREAD_FIELD | 57 | Medium |
Unused field: org.apache.myfaces.application.ViewResourceIterator.pathSet | PERFORMANCE | UUF_UNUSED_FIELD | Not available | Medium |
org.apache.myfaces.application._ApplicationUtils
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.application._ApplicationUtils._createEvent(FacesContext, Class, Object, SystemEvent) | STYLE | REC_CATCH_EXCEPTION | 82 | Low |
org.apache.myfaces.application.jsp.ServletViewResponseWrapper
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.application.jsp.ServletViewResponseWrapper.toString() may return null | BAD_PRACTICE | NP_TOSTRING_COULD_RETURN_NULL | 178 | Medium |
org.apache.myfaces.application.viewstate.IntByteArraySerializedViewKey
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.application.viewstate.IntByteArraySerializedViewKey implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 34-68 | Low |
org.apache.myfaces.application.viewstate.IntByteArraySerializedViewKey is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 34-68 | Low |
org.apache.myfaces.application.viewstate.IntIntSerializedViewKey
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.application.viewstate.IntIntSerializedViewKey implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 34-68 | Low |
org.apache.myfaces.application.viewstate.RandomKeyFactory
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.application.viewstate.RandomKeyFactory.decode(String) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 106 | Low |
org.apache.myfaces.application.viewstate.SecureRandomKeyFactory
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.application.viewstate.SecureRandomKeyFactory.decode(String) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 128 | Low |
org.apache.myfaces.application.viewstate.SerializedViewCollection
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Inconsistent synchronization of org.apache.myfaces.application.viewstate.SerializedViewCollection._lastWindowKeys; locked 57% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 269 | Medium |
org.apache.myfaces.application.viewstate.ServerSideStateCacheImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.application.viewstate.ServerSideStateCacheImpl.isSerializeStateInSession(FacesContext) | I18N | DM_CONVERT_CASE | 506 | Low |
Repeated conditional test in org.apache.myfaces.application.viewstate.ServerSideStateCacheImpl.saveSerializedViewInServletSession(FacesContext, Object) | CORRECTNESS | RpC_REPEATED_CONDITIONAL_TEST | 287 | Low |
org.apache.myfaces.application.viewstate.ServerSideStateCacheImpl$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.application.viewstate.ServerSideStateCacheImpl$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 570-574 | Low |
org.apache.myfaces.cdi.JsfArtifactFlowMapProducer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.cdi.JsfArtifactFlowMapProducer is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 40-50 | Low |
org.apache.myfaces.cdi.JsfArtifactFlowMapProducer$FlowMapAnnotationLiteral
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.cdi.JsfArtifactFlowMapProducer$FlowMapAnnotationLiteral is serializable and an inner class | BAD_PRACTICE | SE_INNER_CLASS | 33 | Low |
Should org.apache.myfaces.cdi.JsfArtifactFlowMapProducer$FlowMapAnnotationLiteral be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 33 | Medium |
org.apache.myfaces.cdi.behavior.FacesBehaviorInfo
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.cdi.behavior.FacesBehaviorInfo defines non-transient non-serializable instance field type | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
org.apache.myfaces.cdi.behavior.FacesBehaviorInfo is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 36-96 | Low |
org.apache.myfaces.cdi.behavior.FacesBehaviorProducer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.cdi.behavior.FacesBehaviorProducer is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 41-72 | Low |
org.apache.myfaces.cdi.converter.FacesConverterInfo
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.cdi.converter.FacesConverterInfo defines non-transient non-serializable instance field type | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
org.apache.myfaces.cdi.converter.FacesConverterInfo is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 38-114 | Low |
org.apache.myfaces.cdi.converter.FacesConverterProducer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.cdi.converter.FacesConverterProducer is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 41-75 | Low |
org.apache.myfaces.cdi.managedproperty.ManagedPropertyInfo
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.cdi.managedproperty.ManagedPropertyInfo defines non-transient non-serializable instance field type | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
org.apache.myfaces.cdi.managedproperty.ManagedPropertyInfo is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 36-86 | Low |
org.apache.myfaces.cdi.managedproperty.ManagedPropertyProducer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.cdi.managedproperty.ManagedPropertyProducer is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 58-91 | Low |
org.apache.myfaces.cdi.managedproperty.ManagedPropertyProducer$ManagedPropertyAnnotationLiteral
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.cdi.managedproperty.ManagedPropertyProducer$ManagedPropertyAnnotationLiteral is serializable and an inner class | BAD_PRACTICE | SE_INNER_CLASS | 45-52 | Low |
Should org.apache.myfaces.cdi.managedproperty.ManagedPropertyProducer$ManagedPropertyAnnotationLiteral be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 45-52 | Medium |
org.apache.myfaces.cdi.model.DataModelInfo
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.cdi.model.DataModelInfo defines non-transient non-serializable instance field type | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
org.apache.myfaces.cdi.model.DataModelInfo is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 36-90 | Low |
org.apache.myfaces.cdi.model.DynamicDataModelProducer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.cdi.model.DynamicDataModelProducer is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 39-55 | Low |
org.apache.myfaces.cdi.model.FacesDataModelExtension
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to map in org.apache.myfaces.cdi.model.FacesDataModelExtension.afterDeploymentValidation(AfterDeploymentValidation, BeanManager) | STYLE | DLS_DEAD_LOCAL_STORE | 89 | Medium |
org.apache.myfaces.cdi.scope.FacesScopedContextImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Return value of getFacesScopeBeanHolder(FacesContext) ignored, but method has no side effect | STYLE | RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT | 73 | Medium |
org.apache.myfaces.cdi.scope.ViewTransientScopedContextImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Return value of getViewTransientScopeBeanHolder(FacesContext) ignored, but method has no side effect | STYLE | RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT | 73 | Medium |
org.apache.myfaces.cdi.util.ContextualInstanceInfo
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.cdi.util.ContextualInstanceInfo defines non-transient non-serializable instance field creationalContext | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
org.apache.myfaces.cdi.util.ContextualInstanceInfo is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 31-75 | Low |
org.apache.myfaces.cdi.util.ContextualStorage
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.cdi.util.ContextualStorage defines non-transient non-serializable instance field beanManager | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
org.apache.myfaces.cdi.util.ParameterizedTypeImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.cdi.util.ParameterizedTypeImpl.getActualTypeArguments() may expose internal representation by returning ParameterizedTypeImpl.actualTypeArguments | MALICIOUS_CODE | EI_EXPOSE_REP | 59 | Medium |
new org.apache.myfaces.cdi.util.ParameterizedTypeImpl(Type, Class, Type[]) may expose internal representation by storing an externally mutable object into ParameterizedTypeImpl.actualTypeArguments | MALICIOUS_CODE | EI_EXPOSE_REP2 | 41 | Medium |
org.apache.myfaces.cdi.validator.FacesValidatorInfo
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.cdi.validator.FacesValidatorInfo defines non-transient non-serializable instance field type | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
org.apache.myfaces.cdi.validator.FacesValidatorInfo is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 36-96 | Low |
org.apache.myfaces.cdi.validator.FacesValidatorProducer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.cdi.validator.FacesValidatorProducer is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 41-72 | Low |
org.apache.myfaces.cdi.view.ViewScopeBeanHolder
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
facesContext could be null and is guaranteed to be dereferenced in org.apache.myfaces.cdi.view.ViewScopeBeanHolder.destroyBeans(String) | CORRECTNESS | NP_GUARANTEED_DEREF | 168 | Medium |
Possible null pointer dereference of facesContext in org.apache.myfaces.cdi.view.ViewScopeBeanHolder.destroyBeansOnPreDestroy() on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 217 | Medium |
Class org.apache.myfaces.cdi.view.ViewScopeBeanHolder defines non-transient non-serializable instance field applicationContextBean | BAD_PRACTICE | SE_BAD_FIELD | Not available | High |
org.apache.myfaces.cdi.view.ViewScopeBeanHolder is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 54-239 | Low |
org.apache.myfaces.cdi.view.ViewScopeCDIMap
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of st, which is known to be non-null in org.apache.myfaces.cdi.view.ViewScopeCDIMap.clear() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 190 | Low |
org.apache.myfaces.cdi.view.ViewScopeContextualKey
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.cdi.view.ViewScopeContextualKey is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 32-71 | Low |
org.apache.myfaces.cdi.view.ViewScopeContextualStorage
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The field org.apache.myfaces.cdi.view.ViewScopeContextualStorage.beanManager is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
org.apache.myfaces.component.search.IdSearchKeywordResolver$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.component.search.IdSearchKeywordResolver$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 66-85 | Low |
org.apache.myfaces.component.search.SearchExpressionHandlerImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.apache.myfaces.component.search.SearchExpressionHandlerImpl.invokeOnComponent(SearchExpressionContext, UIComponent, String, ContextCallback) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 537 | Medium |
Should org.apache.myfaces.component.search.SearchExpressionHandlerImpl.split(FacesContext, String, char[]) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 799 | Low |
Redundant nullcheck of target which is known to be null in org.apache.myfaces.component.search.SearchExpressionHandlerImpl.invokeOnComponent(SearchExpressionContext, UIComponent, String, ContextCallback) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE | 537 | Low |
org.apache.myfaces.component.search.SearchExpressionHandlerImpl.invokeOnComponent(SearchExpressionContext, UIComponent, String, ContextCallback) checks to see if result of String.indexOf is positive | STYLE | RV_CHECK_FOR_POSITIVE_INDEXOF | 440 | Low |
org.apache.myfaces.component.search.SearchExpressionHandlerImpl.isValidExpression(SearchExpressionContext, String) checks to see if result of String.indexOf is positive | STYLE | RV_CHECK_FOR_POSITIVE_INDEXOF | 709 | Low |
org.apache.myfaces.component.search.SearchExpressionHandlerImpl$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.component.search.SearchExpressionHandlerImpl$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 418-424 | Low |
org.apache.myfaces.component.search.SearchExpressionHandlerImpl$2
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.component.search.SearchExpressionHandlerImpl$2 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 502-508 | Low |
org.apache.myfaces.component.validate.WholeBeanValidator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.component.validate.ValidateWholeBeanComponent in org.apache.myfaces.component.validate.WholeBeanValidator.validate(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 115 | Low |
Redundant nullcheck of valueBaseClass, which is known to be non-null in org.apache.myfaces.component.validate.WholeBeanValidator.validate(FacesContext, UIComponent, Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 108 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.component.validate.WholeBeanValidator.createBeanCopy(Object) | STYLE | REC_CATCH_EXCEPTION | 207 | Low |
org.apache.myfaces.component.validate.WholeBeanValidator$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.component.validate.WholeBeanValidator$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 349-352 | Low |
org.apache.myfaces.component.visit.PartialVisitContext
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.component.visit.PartialVisitContext._removeSubtreeClientId(String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 371 | Medium |
org.apache.myfaces.config.DefaultFacesConfigurationMerger$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.config.DefaultFacesConfigurationMerger$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 831-874 | Low |
org.apache.myfaces.config.DefaultFacesConfigurationProvider
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.apache.myfaces.config.DefaultFacesConfigurationProvider.getWebAppFacesConfig(ExternalContext): new String(byte[]) | I18N | DM_DEFAULT_ENCODING | 412 | High |
Redundant nullcheck of instances, which is known to be non-null in org.apache.myfaces.config.DefaultFacesConfigurationProvider.getApplicationConfigurationResourceDocumentPopulatorFacesConfig(ExternalContext) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 508 | Low |
org.apache.myfaces.config.FacesConfigValidator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.apache.myfaces.config.FacesConfigValidator.validateNavRule(NavigationRule, List, ExternalContext) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 110 | Medium |
Load of known null value in org.apache.myfaces.config.FacesConfigValidator.validateNavRule(NavigationRule, List, ExternalContext) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 126 | Medium |
Redundant nullcheck of managedBeansMap, which is known to be non-null in org.apache.myfaces.config.FacesConfigValidator.validate(ExternalContext) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 61 | Low |
org.apache.myfaces.config.FacesConfigurator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.FacesConfigurator.getResourceLastModified(URLConnection) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 363 | Low |
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.config.FacesConfigurator.configureApplication() | I18N | DM_CONVERT_CASE | 851 | Low |
Redundant nullcheck of parms, which is known to be non-null in org.apache.myfaces.config.FacesConfigurator.getDefaultSourcClassForSystemEvent(Class) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 990 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.config.FacesConfigurator.configureRuntimeConfig() | STYLE | REC_CATCH_EXCEPTION | 1157 | Low |
Write to static field org.apache.myfaces.config.FacesConfigurator.lastUpdate from instance method org.apache.myfaces.config.FacesConfigurator.configure() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 624 | Medium |
Write to static field org.apache.myfaces.config.FacesConfigurator.lastUpdate from instance method org.apache.myfaces.config.FacesConfigurator.update() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 494 | Medium |
org.apache.myfaces.config.ManagedBeanBuilder
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of value, which is known to be non-null in org.apache.myfaces.config.ManagedBeanBuilder.initializeProperties(FacesContext, ManagedBean, Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 272 | Low |
org.apache.myfaces.config.NamedEventManager
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.config.NamedEventManager.getFixedName(Class) | I18N | DM_CONVERT_CASE | 117 | Low |
org.apache.myfaces.config.annotation.AnnotationConfigurator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of value, which is known to be non-null in org.apache.myfaces.config.annotation.AnnotationConfigurator.createFacesConfig(Map) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 123 | Medium |
Redundant nullcheck of value, which is known to be non-null in org.apache.myfaces.config.annotation.AnnotationConfigurator.createFacesConfig(Map) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 208 | Medium |
Repeated conditional test in org.apache.myfaces.config.annotation.AnnotationConfigurator.createFacesConfig(Map) | CORRECTNESS | RpC_REPEATED_CONDITIONAL_TEST | 123 | Low |
Repeated conditional test in org.apache.myfaces.config.annotation.AnnotationConfigurator.createFacesConfig(Map) | CORRECTNESS | RpC_REPEATED_CONDITIONAL_TEST | 208 | Low |
Useless object stored in variable app of method org.apache.myfaces.config.annotation.AnnotationConfigurator.createFacesConfig(Map) | STYLE | UC_USELESS_OBJECT | 220 | Medium |
org.apache.myfaces.config.annotation.DefaultAnnotationProvider
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.annotation.DefaultAnnotationProvider.getAnnotatedMetaInfClasses(ExternalContext, Set) is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 290-310 | High |
org.apache.myfaces.config.annotation.DefaultAnnotationProvider.getBaseUrls() is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 238-252 | High |
org.apache.myfaces.config.annotation.DefaultAnnotationProvider.getBaseUrls(ExternalContext) is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 258-284 | High |
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.config.annotation.DefaultAnnotationProvider.packageClasses(ExternalContext, String) | I18N | DM_CONVERT_CASE | 417 | Low |
org.apache.myfaces.config.annotation.NoInjectionAnnotationLifecycleProvider
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The parameter metadata to org.apache.myfaces.config.annotation.NoInjectionAnnotationLifecycleProvider.createDeclaredMethodBeansMap(ClassLoader, Map) is dead upon entry but overwritten | CORRECTNESS | IP_PARAMETER_IS_DEAD_BUT_OVERWRITTEN | 76 | Low |
Load of known null value in org.apache.myfaces.config.annotation.NoInjectionAnnotationLifecycleProvider.getDeclaredMethodBeansMap() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 66 | Medium |
Redundant nullcheck of methods, which is known to be non-null in org.apache.myfaces.config.annotation.NoInjectionAnnotationLifecycleProvider.postConstruct(Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 121 | Medium |
org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Invocation of reflect.Field.setAccessible(boolean), which should be invoked from within a doPrivileged block, in org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider.lookupFieldResource(Context, Object, Field, String) | MALICIOUS_CODE | DP_DO_INSIDE_DO_PRIVILEGED | 193 | Low |
Invocation of reflect.Method.setAccessible(boolean), which should be invoked from within a doPrivileged block, in org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider.lookupMethodResource(Context, Object, Method, String) | MALICIOUS_CODE | DP_DO_INSIDE_DO_PRIVILEGED | 229 | Low |
The parameter metadata to org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider.createDeclaredFieldBeansMap(ClassLoader, Map) is dead upon entry but overwritten | CORRECTNESS | IP_PARAMETER_IS_DEAD_BUT_OVERWRITTEN | 77 | Low |
Load of known null value in org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider.getDeclaredFieldBeansMap() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 67 | Medium |
org.apache.myfaces.config.annotation._ClassByteCodeAnnotationFilter
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.annotation._ClassByteCodeAnnotationFilter.couldContainAnnotationsOnClassDef(DataInput, Set) uses the same code for two switch clauses | STYLE | DB_DUPLICATE_SWITCH_CLAUSES | 144-145 | Low |
org.apache.myfaces.config.annotation._ClassByteCodeAnnotationFilter.couldContainAnnotationsOnClassDef(DataInput, Set) uses the same code for two switch clauses | STYLE | DB_DUPLICATE_SWITCH_CLAUSES | 151-153 | Low |
Dead store to minorVersion in org.apache.myfaces.config.annotation._ClassByteCodeAnnotationFilter.couldContainAnnotationsOnClassDef(DataInput, Set) | STYLE | DLS_DEAD_LOCAL_STORE | 107 | Medium |
org.apache.myfaces.config.annotation._PackageInfo
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.config.annotation._PackageInfo.getClasses(List, String) | I18N | DM_CONVERT_CASE | 160 | Low |
Redundant nullcheck of resources, which is known to be non-null in org.apache.myfaces.config.annotation._PackageInfo.getClasses(List, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 100 | Low |
org.apache.myfaces.config.annotation._PackageInfo$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.config.annotation._PackageInfo$1.accept(File) | I18N | DM_CONVERT_CASE | 261 | Low |
org.apache.myfaces.config.element.ConfigOthersSlot
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.element.ConfigOthersSlot implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 28 | Low |
org.apache.myfaces.config.element.FacesConfigNameSlot
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.element.FacesConfigNameSlot implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 23 | Low |
org.apache.myfaces.config.element.facelets.FaceletBehaviorTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.element.facelets.FaceletBehaviorTag implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 26 | Low |
org.apache.myfaces.config.element.facelets.FaceletComponentTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.element.facelets.FaceletComponentTag implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 26 | Low |
org.apache.myfaces.config.element.facelets.FaceletConverterTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.element.facelets.FaceletConverterTag implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 26 | Low |
org.apache.myfaces.config.element.facelets.FaceletHandlerTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.element.facelets.FaceletHandlerTag implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 26 | Low |
org.apache.myfaces.config.element.facelets.FaceletSourceTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.element.facelets.FaceletSourceTag implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 26 | Low |
org.apache.myfaces.config.element.facelets.FaceletValidatorTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.element.facelets.FaceletValidatorTag implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 26 | Low |
org.apache.myfaces.config.impl.FacesConfigEntityResolver
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Null passed for non-null parameter of java.util.jar.JarFile.getInputStream(ZipEntry) in org.apache.myfaces.config.impl.FacesConfigEntityResolver.resolveEntity(String, String) | CORRECTNESS | NP_NULL_PARAM_DEREF | 90 | High |
org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umactionListeners is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umapplicationFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umbehaviors is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umclientWindowFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umcomponentTagDeclarations is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umelResolvers is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umexceptionHandlerFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umexternalContextFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umfaceletCacheFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umfaceletTagLibraries is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umfaceletsTemplateMappings is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umfacesContextFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umfacesFlowDefinitions is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umflashFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umflowHandlerFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umlifecycleFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umlifecyclePhaseListeners is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.ummanagedBeans is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umnamedEvents is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umnavigationHandlers is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umnavigationRules is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umpartialViewContextFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umpropertyResolver is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umprotectedViewUrlPatterns is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umrenderKitFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umresourceBundles is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umresourceHandlers is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umresourceLibraryContractMappings is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umresourceResolvers is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umsearchExpressionContextFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umsearchExpressionHandlers is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umsearchKeywordResolvers is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umstateManagers is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umsystemEventListeners is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umtagHandlerDelegateFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umvariableResolver is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umviewDeclarationLanguageFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umviewHandlers is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umviewPoolMappings is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.umvisitContextFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
org.apache.myfaces.config.impl.digester.elements.AbsoluteOrderingImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.AbsoluteOrderingImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 25-37 | Low |
org.apache.myfaces.config.impl.digester.elements.AbsoluteOrderingImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 25-37 | Low |
org.apache.myfaces.config.impl.digester.elements.ApplicationImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ApplicationImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 29-248 | Low |
org.apache.myfaces.config.impl.digester.elements.ApplicationImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 29-248 | Low |
org.apache.myfaces.config.impl.digester.elements.AttributeImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.AttributeImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 31-160 | Low |
org.apache.myfaces.config.impl.digester.elements.AttributeImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 31-160 | Low |
org.apache.myfaces.config.impl.digester.elements.BehaviorImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.BehaviorImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 29-77 | Low |
org.apache.myfaces.config.impl.digester.elements.BehaviorImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 29-77 | Low |
org.apache.myfaces.config.impl.digester.elements.ClientBehaviorRendererImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ClientBehaviorRendererImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 27-52 | Low |
org.apache.myfaces.config.impl.digester.elements.ClientBehaviorRendererImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 27-52 | Low |
org.apache.myfaces.config.impl.digester.elements.ComponentImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ComponentImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 27-54 | Low |
org.apache.myfaces.config.impl.digester.elements.ComponentImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 27-54 | Low |
org.apache.myfaces.config.impl.digester.elements.ComponentTagDeclarationImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ComponentTagDeclarationImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 35-87 | Low |
org.apache.myfaces.config.impl.digester.elements.ComponentTagDeclarationImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 35-87 | Low |
org.apache.myfaces.config.impl.digester.elements.ConfigOthersSlotImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ConfigOthersSlotImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 23 | Low |
org.apache.myfaces.config.impl.digester.elements.ConfigOthersSlotImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 23 | Low |
org.apache.myfaces.config.impl.digester.elements.ContractMappingImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.ContractMappingImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 30-99 | Medium |
The field org.apache.myfaces.config.impl.digester.elements.ContractMappingImpl.unmodifiableContractsList is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.ContractMappingImpl.unmodifiableUrlPatternList is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
org.apache.myfaces.config.impl.digester.elements.ConverterImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ConverterImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 31-113 | Low |
org.apache.myfaces.config.impl.digester.elements.ConverterImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 31-113 | Low |
org.apache.myfaces.config.impl.digester.elements.FaceletsTemplateMappingImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.FaceletsTemplateMappingImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 33-50 | Low |
org.apache.myfaces.config.impl.digester.elements.FaceletsTemplateMappingImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 33-50 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesConfig
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.FacesConfig is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 25 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesConfigExtensionImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigExtensionImpl.unmodifiableFaceletsTemplateMappings is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigExtensionImpl.unmodifiableViewPoolMappings is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 45-551 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 45-551 | Medium |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableApplications is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableBehaviors is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableComponentTagDeclarations is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableComponents is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableConverters is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableFaceletTagLibraryList is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableFacesConfigExtensions is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableFacesFlowDefinitions is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableFactories is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableLifecyclePhaseListener is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableManagedBeans is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableNamedEvents is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableNavigationRules is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableProtectedViewsUrlPatternList is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableRenderKits is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableResourceResolvers is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
The field org.apache.myfaces.config.impl.digester.elements.FacesConfigImpl.unmodifiableValidators is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
org.apache.myfaces.config.impl.digester.elements.FacesConfigNameSlotImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.FacesConfigNameSlotImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 23-36 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesConfigNameSlotImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 23-36 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesFlowCallImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.FacesFlowCallImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 38-73 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesFlowDefinitionImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.FacesFlowDefinitionImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 52-206 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesFlowMethodCallImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.FacesFlowMethodCallImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 38-90 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesFlowMethodParameterImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.FacesFlowMethodParameterImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 27-62 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesFlowParameterImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.FacesFlowParameterImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 25-56 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesFlowReferenceImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.FacesFlowReferenceImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 35-59 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesFlowReturnImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.FacesFlowReturnImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 27-55 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesFlowSwitchImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.FacesFlowSwitchImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 36-74 | Low |
org.apache.myfaces.config.impl.digester.elements.FacesFlowViewImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.FacesFlowViewImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 31-57 | Low |
org.apache.myfaces.config.impl.digester.elements.FactoryImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The field name org.apache.myfaces.config.impl.digester.elements.FactoryImpl.ViewDeclarationLanguageFactories doesn't start with a lower case letter | BAD_PRACTICE | NM_FIELD_NAMING_CONVENTION | Not available | Low |
Class org.apache.myfaces.config.impl.digester.elements.FactoryImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 28-198 | Low |
org.apache.myfaces.config.impl.digester.elements.FactoryImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 28-198 | Low |
org.apache.myfaces.config.impl.digester.elements.ListEntriesImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ListEntriesImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 30-59 | Low |
org.apache.myfaces.config.impl.digester.elements.ListEntriesImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 30-59 | Low |
org.apache.myfaces.config.impl.digester.elements.ListEntriesImpl$Entry
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.ListEntriesImpl$Entry is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 63-90 | Low |
org.apache.myfaces.config.impl.digester.elements.LocaleConfigImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.LocaleConfigImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 29-56 | Low |
org.apache.myfaces.config.impl.digester.elements.LocaleConfigImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 29-56 | Low |
org.apache.myfaces.config.impl.digester.elements.ManagedBeanImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ManagedBeanImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 36-197 | Low |
org.apache.myfaces.config.impl.digester.elements.ManagedBeanImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 36-197 | Low |
org.apache.myfaces.config.impl.digester.elements.ManagedPropertyImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ManagedPropertyImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 35-137 | Low |
org.apache.myfaces.config.impl.digester.elements.ManagedPropertyImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 35-137 | Medium |
org.apache.myfaces.config.impl.digester.elements.MapEntriesImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.MapEntriesImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 30-70 | Low |
org.apache.myfaces.config.impl.digester.elements.MapEntriesImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 30-70 | Low |
org.apache.myfaces.config.impl.digester.elements.MapEntriesImpl$Entry
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.MapEntriesImpl$Entry is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 74-114 | Low |
org.apache.myfaces.config.impl.digester.elements.NamedEventImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.NamedEventImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 30-51 | Low |
org.apache.myfaces.config.impl.digester.elements.NamedEventImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 30-51 | Low |
org.apache.myfaces.config.impl.digester.elements.NavigationCaseImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.NavigationCaseImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 30-93 | Low |
org.apache.myfaces.config.impl.digester.elements.NavigationCaseImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 30-93 | Low |
org.apache.myfaces.config.impl.digester.elements.NavigationRuleImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.NavigationRuleImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 29-57 | Low |
org.apache.myfaces.config.impl.digester.elements.NavigationRuleImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 29-57 | Low |
org.apache.myfaces.config.impl.digester.elements.OrderSlotImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.OrderSlotImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 23 | Low |
org.apache.myfaces.config.impl.digester.elements.OrderSlotImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 23 | Low |
org.apache.myfaces.config.impl.digester.elements.OrderingImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.OrderingImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 25-50 | Low |
org.apache.myfaces.config.impl.digester.elements.OrderingImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 25-50 | Low |
org.apache.myfaces.config.impl.digester.elements.PropertyImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.PropertyImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 40-169 | Low |
org.apache.myfaces.config.impl.digester.elements.PropertyImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 40-169 | Low |
org.apache.myfaces.config.impl.digester.elements.RedirectImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.RedirectImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 29-59 | Low |
org.apache.myfaces.config.impl.digester.elements.RedirectImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 29-59 | Low |
org.apache.myfaces.config.impl.digester.elements.RenderKitImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.RenderKitImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 28-83 | Low |
org.apache.myfaces.config.impl.digester.elements.RenderKitImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 28-83 | Low |
org.apache.myfaces.config.impl.digester.elements.RendererImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.RendererImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 27-68 | Low |
org.apache.myfaces.config.impl.digester.elements.RendererImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 27-68 | Low |
org.apache.myfaces.config.impl.digester.elements.ResourceBundleImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ResourceBundleImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 27-75 | Low |
org.apache.myfaces.config.impl.digester.elements.ResourceBundleImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 27-75 | Low |
org.apache.myfaces.config.impl.digester.elements.SystemEventListenerImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.SystemEventListenerImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 24-58 | Low |
org.apache.myfaces.config.impl.digester.elements.SystemEventListenerImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 24-58 | Low |
org.apache.myfaces.config.impl.digester.elements.ViewParamImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ViewParamImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 24-47 | Low |
org.apache.myfaces.config.impl.digester.elements.ViewParamImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 24-47 | Low |
org.apache.myfaces.config.impl.digester.elements.ViewPoolMappingImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.ViewPoolMappingImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 38-67 | Low |
org.apache.myfaces.config.impl.digester.elements.ViewPoolMappingImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 38-67 | Low |
org.apache.myfaces.config.impl.digester.elements.ViewPoolParameterImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.impl.digester.elements.ViewPoolParameterImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 26-57 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletBehaviorTagImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.facelets.FaceletBehaviorTagImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 33-65 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletBehaviorTagImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 33-65 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletComponentTagImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.facelets.FaceletComponentTagImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 35-84 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletComponentTagImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 35-84 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletConverterTagImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.facelets.FaceletConverterTagImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 33-65 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletConverterTagImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 33-65 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletFunctionImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.facelets.FaceletFunctionImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 34-72 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletFunctionImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 34-72 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletHandlerTagImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.facelets.FaceletHandlerTagImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 32-48 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletHandlerTagImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 32-48 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletSourceTagImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.facelets.FaceletSourceTagImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 32-48 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletSourceTagImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 32-48 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletTagImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.facelets.FaceletTagImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 35-67 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletTagImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 35-67 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletTagLibraryImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.facelets.FaceletTagLibraryImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 37-103 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletTagLibraryImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 37-103 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletValidatorTagImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.config.impl.digester.elements.facelets.FaceletValidatorTagImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 33-65 | Low |
org.apache.myfaces.config.impl.digester.elements.facelets.FaceletValidatorTagImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 33-65 | Low |
org.apache.myfaces.config.util.Vertex
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.config.util.Vertex defines compareTo(Vertex) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 188-201 | Medium |
org.apache.myfaces.context.FacesContextFactoryImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Invocation of reflect.Field.setAccessible(boolean), which should be invoked from within a doPrivileged block, in new org.apache.myfaces.context.FacesContextFactoryImpl() | MALICIOUS_CODE | DP_DO_INSIDE_DO_PRIVILEGED | 83 | Low |
Load of known null value in new org.apache.myfaces.context.FacesContextFactoryImpl() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 87 | Low |
Redundant nullcheck of externalContextFirstInstance, which is known to be non-null in new org.apache.myfaces.context.FacesContextFactoryImpl() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 85 | Low |
Redundant nullcheck of firstExternalContextInstance which is known to be null in new org.apache.myfaces.context.FacesContextFactoryImpl() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE | 87 | Low |
org.apache.myfaces.context.MyFacesExceptionHandlerWrapperImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.event.SystemEvent to javax.faces.event.ExceptionQueuedEvent in org.apache.myfaces.context.MyFacesExceptionHandlerWrapperImpl.processEvent(SystemEvent) | STYLE | BC_UNCONFIRMED_CAST | 331 | Low |
org.apache.myfaces.context.PartialResponseWriterImpl$StackEntry
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.context.PartialResponseWriterImpl$StackEntry be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 82-105 | Medium |
org.apache.myfaces.context.RequestViewMetadata
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.context.RequestViewMetadata defines non-transient non-serializable instance field addedResources | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Class org.apache.myfaces.context.RequestViewMetadata defines non-transient non-serializable instance field initialAddedResources | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
org.apache.myfaces.context.RequestViewMetadata is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 40-174 | Low |
org.apache.myfaces.context.servlet.PartialViewContextImpl$PhaseAwareVisitCallback
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.context.servlet.PartialViewContextImpl$PhaseAwareVisitCallback be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 756-854 | Medium |
org.apache.myfaces.context.servlet.ServletExternalContextImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.servlet.ServletRequest to javax.servlet.http.HttpServletRequest in new org.apache.myfaces.context.servlet.ServletExternalContextImpl(ServletContext, ServletRequest, ServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 122 | Low |
Unchecked/unconfirmed cast from javax.servlet.ServletResponse to javax.servlet.http.HttpServletResponse in new org.apache.myfaces.context.servlet.ServletExternalContextImpl(ServletContext, ServletRequest, ServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 123 | Low |
Unchecked/unconfirmed cast from javax.servlet.ServletRequest to javax.servlet.http.HttpServletRequest in org.apache.myfaces.context.servlet.ServletExternalContextImpl.setRequest(Object) | STYLE | BC_UNCONFIRMED_CAST | 666 | Low |
org.apache.myfaces.ee.MyFacesContainerInitializer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.ee.MyFacesContainerInitializer.isFacesConfigPresent(ServletContext) | STYLE | REC_CATCH_EXCEPTION | 343 | Low |
org.apache.myfaces.el.CompositeELResolver
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Inconsistent synchronization of org.apache.myfaces.el.CompositeELResolver._elResolvers; locked 75% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 42 | Low |
The class name org.apache.myfaces.el.CompositeELResolver shadows the simple name of the superclass javax.el.CompositeELResolver | BAD_PRACTICE | NM_SAME_SIMPLE_NAME_AS_SUPERCLASS | 35-65 | High |
org.apache.myfaces.el.ResolverForJSPInitializer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.el.ResolverForJSPInitializer defines non-transient non-serializable instance field _resolverForJSP | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
org.apache.myfaces.el.convert.ValueBindingToValueExpression
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.el.convert.ValueBindingToValueExpression defines non-transient non-serializable instance field _valueBinding | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
org.apache.myfaces.el.unified.CustomFirstELResolverComparator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.el.unified.CustomFirstELResolverComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 39-101 | Medium |
org.apache.myfaces.el.unified.CustomLastELResolverComparator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.el.unified.CustomLastELResolverComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 34-42 | Low |
org.apache.myfaces.el.unified.OpenWebBeansELResolverComparator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.el.unified.OpenWebBeansELResolverComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 33-50 | Medium |
org.apache.myfaces.el.unified.ResolverBuilderBase
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of facesContext, which is known to be non-null in org.apache.myfaces.el.unified.ResolverBuilderBase.addFromRuntimeConfig(List) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 117 | Medium |
org.apache.myfaces.el.unified.resolver.CompositeComponentELResolver$CompositeComponentAttributesMapWrapper
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.el.unified.resolver.CompositeComponentELResolver$CompositeComponentAttributesMapWrapper be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 271-441 | Medium |
org.apache.myfaces.el.unified.resolver.CompositeComponentELResolver$CompositeComponentAttributesMapWrapper.putAll(Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 425 | Medium |
org.apache.myfaces.el.unified.resolver.ImportConstantsELResolver
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of constantsMap, which is known to be non-null in org.apache.myfaces.el.unified.resolver.ImportConstantsELResolver.getValue(ELContext, Object, Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 129 | Low |
org.apache.myfaces.el.unified.resolver.ManagedBeanResolver
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.el.unified.resolver.ManagedBeanResolver.STANDARD_SCOPES is a mutable collection which should be package protected | MALICIOUS_CODE | MS_MUTABLE_COLLECTION_PKGPROTECT | 60 | Medium |
org.apache.myfaces.el.unified.resolver.ResourceBundleResolver
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.apache.myfaces.el.unified.resolver.ResourceBundleResolver.setValue(ELContext, Object, Object, Object) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 70 | Medium |
Redundant nullcheck of base which is known to be null in org.apache.myfaces.el.unified.resolver.ResourceBundleResolver.setValue(ELContext, Object, Object, Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE | 70 | Low |
org.apache.myfaces.event.PostClientWindowAndViewInitializedEvent
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.event.PostClientWindowAndViewInitializedEvent is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 31-32 | Low |
org.apache.myfaces.flow.FlowHandlerImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.apache.myfaces.flow.FlowHandlerImpl.transition(FacesContext, Flow, Flow, FlowCallNode, String) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 261 | Medium |
org.apache.myfaces.flow.FlowReference
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.flow.FlowReference is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 37-103 | Low |
org.apache.myfaces.flow.NavigationCaseImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.flow.NavigationCaseImpl.getCondition(FacesContext) has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 280 | Medium |
org.apache.myfaces.flow.cdi.ClientWindowFacesFlowLRUMap
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.flow.cdi.ClientWindowFacesFlowLRUMap implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 35-51 | Low |
org.apache.myfaces.flow.cdi.ClientWindowFacesFlowLRUMap is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 35-51 | Medium |
The field org.apache.myfaces.flow.cdi.ClientWindowFacesFlowLRUMap.holder is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Low |
org.apache.myfaces.flow.cdi.FacesFlowClientWindowCollection
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.flow.cdi.FacesFlowClientWindowCollection is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 31-62 | Low |
org.apache.myfaces.flow.cdi.FlowScopeBeanHolder
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Possible null pointer dereference of facesContext in org.apache.myfaces.flow.cdi.FlowScopeBeanHolder.destroyBeansOnPreDestroy() on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 251 | Medium |
Class org.apache.myfaces.flow.cdi.FlowScopeBeanHolder defines non-transient non-serializable instance field applicationContextBean | BAD_PRACTICE | SE_BAD_FIELD | Not available | High |
org.apache.myfaces.flow.cdi.FlowScopeBeanHolder is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 85-351 | Low |
org.apache.myfaces.lifecycle.CODIClientSideWindow
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to existingParameters in org.apache.myfaces.lifecycle.CODIClientSideWindow.addParameters(ExternalContext, String, boolean, boolean, boolean) | STYLE | DLS_DEAD_LOCAL_STORE | 334 | Medium |
Dead store to urlContainsWindowId in org.apache.myfaces.lifecycle.CODIClientSideWindow.addParameters(ExternalContext, String, boolean, boolean, boolean) | STYLE | DLS_DEAD_LOCAL_STORE | 335 | Medium |
Found reliance on default encoding in org.apache.myfaces.lifecycle.CODIClientSideWindow.convertStreamToString(InputStream): new java.io.InputStreamReader(InputStream) | I18N | DM_DEFAULT_ENCODING | 255 | High |
Found reliance on default encoding in org.apache.myfaces.lifecycle.CODIClientSideWindow.sendWindowHandlerHtml(FacesContext, String): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 237 | High |
org.apache.myfaces.lifecycle.DefaultRestoreViewSupport
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of viewId, which is known to be non-null in org.apache.myfaces.lifecycle.DefaultRestoreViewSupport.deriveViewId(FacesContext, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 290 | Medium |
Redundant nullcheck of viewId, which is known to be non-null in org.apache.myfaces.lifecycle.DefaultRestoreViewSupport.deriveViewId(FacesContext, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 284 | Low |
org.apache.myfaces.lifecycle.RenderResponseExecutor
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Comparison of String objects using == or != in org.apache.myfaces.lifecycle.RenderResponseExecutor.execute(FacesContext) | BAD_PRACTICE | ES_COMPARING_STRINGS_WITH_EQ | 102 | Low |
Load of known null value in org.apache.myfaces.lifecycle.RenderResponseExecutor.execute(FacesContext) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 102 | Medium |
org.apache.myfaces.lifecycle.RestoreViewExecutor
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of viewRoot, which is known to be non-null in org.apache.myfaces.lifecycle.RestoreViewExecutor.execute(FacesContext) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 266 | Medium |
Useless condition: it's known that valid == true at this point | STYLE | UC_USELESS_CONDITION | 379 | Low |
org.apache.myfaces.push.AbstractWebsocketComponent$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.push.AbstractWebsocketComponent$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 72-149 | Low |
org.apache.myfaces.push.EndpointImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.push.EndpointImpl.onClose(Session, CloseReason) | STYLE | REC_CATCH_EXCEPTION | 116 | Low |
org.apache.myfaces.push.WebsocketComponentRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIWebsocket in org.apache.myfaces.push.WebsocketComponentRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 116 | Low |
org.apache.myfaces.push.WebsocketInitRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.push.WebsocketInit in org.apache.myfaces.push.WebsocketInitRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 43 | Low |
org.apache.myfaces.push.cdi.PushContextImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of appTokenBean, which is known to be non-null in org.apache.myfaces.push.cdi.PushContextImpl.send(Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 106 | Medium |
Redundant nullcheck of appTokenBean, which is known to be non-null in org.apache.myfaces.push.cdi.PushContextImpl.send(Object, Collection) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 198 | Medium |
org.apache.myfaces.push.cdi.PushContextImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 46-235 | Low |
org.apache.myfaces.push.cdi.WebsocketApplicationSessionHolder
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to lruCache in org.apache.myfaces.push.cdi.WebsocketApplicationSessionHolder.initWebsocketSessionLRUCache(ExternalContext) | STYLE | DLS_DEAD_LOCAL_STORE | 103 | Medium |
The parameter metadata to org.apache.myfaces.push.cdi.WebsocketApplicationSessionHolder.createRestoredQueue(ClassLoader, Queue) is dead upon entry but overwritten | CORRECTNESS | IP_PARAMETER_IS_DEAD_BUT_OVERWRITTEN | 317 | Low |
The parameter metadata to org.apache.myfaces.push.cdi.WebsocketApplicationSessionHolder.createWebsocketSessionLRUCache(ClassLoader, ConcurrentLRUCache, int) is dead upon entry but overwritten | CORRECTNESS | IP_PARAMETER_IS_DEAD_BUT_OVERWRITTEN | 128 | Low |
Load of known null value in org.apache.myfaces.push.cdi.WebsocketApplicationSessionHolder.getRestoredQueue() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 308 | Medium |
Load of known null value in org.apache.myfaces.push.cdi.WebsocketApplicationSessionHolder.getWebsocketSessionLRUCache() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 74 | Medium |
Possible null pointer dereference in org.apache.myfaces.push.cdi.WebsocketApplicationSessionHolder.initWebsocketSessionLRUCache(ExternalContext) due to return value of called method | STYLE | NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE | 113 | Medium |
Possible null pointer dereference in org.apache.myfaces.push.cdi.WebsocketApplicationSessionHolder.send(String, Object) due to return value of called method | STYLE | NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE | 190 | Medium |
Return value of java.util.Queue.poll() ignored in org.apache.myfaces.push.cdi.WebsocketApplicationSessionHolder.synchronizeSessionInstances() | CORRECTNESS | RV_RETURN_VALUE_IGNORED | 288 | Low |
org.apache.myfaces.push.cdi.WebsocketChannel
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.push.cdi.WebsocketChannel is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 35-96 | Low |
org.apache.myfaces.push.cdi.WebsocketChannelMetadata
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.push.cdi.WebsocketChannelMetadata is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 39-153 | Low |
org.apache.myfaces.push.cdi.WebsocketChannelTokenBuilderBean
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Inconsistent synchronization of org.apache.myfaces.push.cdi.WebsocketChannelTokenBuilderBean.csrfSessionTokenFactory; locked 66% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 86 | Medium |
Inconsistent synchronization of org.apache.myfaces.push.cdi.WebsocketChannelTokenBuilderBean.initialized; locked 50% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 82 | Medium |
org.apache.myfaces.push.cdi.WebsocketSessionBean
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.push.cdi.WebsocketSessionBean is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 47-175 | Low |
org.apache.myfaces.push.cdi.WebsocketViewBean
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
String is incompatible with expected argument type WebsocketChannel in org.apache.myfaces.push.cdi.WebsocketViewBean.isSessionTokenValid(String) | CORRECTNESS | GC_UNRELATED_TYPES | 80 | High |
org.apache.myfaces.push.cdi.WebsocketViewBean is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 38-173 | Low |
org.apache.myfaces.renderkit.ErrorPageWriter
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.apache.myfaces.renderkit.ErrorPageWriter._splitTemplate(String): java.io.ByteArrayOutputStream.toString() | I18N | DM_DEFAULT_ENCODING | 660 | High |
Incorrect lazy initialization of static field org.apache.myfaces.renderkit.ErrorPageWriter.debugParts in org.apache.myfaces.renderkit.ErrorPageWriter._init(FacesContext) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 625-627 | Low |
Incorrect lazy initialization of static field org.apache.myfaces.renderkit.ErrorPageWriter.errorParts in org.apache.myfaces.renderkit.ErrorPageWriter._init(FacesContext) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 620-622 | Low |
Redundant nullcheck of Object.toString(), which is known to be non-null in org.apache.myfaces.renderkit.ErrorPageWriter._writeVariables(Writer, Map, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 806 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.renderkit.ErrorPageWriter._writeAttributes(Writer, UIComponent, boolean) | STYLE | REC_CATCH_EXCEPTION | 1360 | High |
Exception is caught when Exception is not thrown in org.apache.myfaces.renderkit.ErrorPageWriter._initCauseIfAvailable(Throwable, Throwable) | STYLE | REC_CATCH_EXCEPTION | 1454 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.renderkit.ErrorPageWriter._initCausePerReflection(Throwable, String) | STYLE | REC_CATCH_EXCEPTION | 1435 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.renderkit.ErrorPageWriter._writeAttributes(Writer, UIComponent, boolean) | STYLE | REC_CATCH_EXCEPTION | 1282-1364 | Low |
org.apache.myfaces.renderkit.ErrorPageWriter$ErrorPageBean
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.renderkit.ErrorPageWriter$ErrorPageBean.getErrorPageHtml() | STYLE | REC_CATCH_EXCEPTION | 158 | Low |
org.apache.myfaces.renderkit.ErrorPageWriter$ExtendedComponentTreeVisitCallback
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.renderkit.ErrorPageWriter$ExtendedComponentTreeVisitCallback.visit(VisitContext, UIComponent) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 1010 | Medium |
Invocation of reflect.Method.setAccessible(boolean), which should be invoked from within a doPrivileged block, in org.apache.myfaces.renderkit.ErrorPageWriter$ExtendedComponentTreeVisitCallback.visit(VisitContext, UIComponent) | MALICIOUS_CODE | DP_DO_INSIDE_DO_PRIVILEGED | 1007 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.renderkit.ErrorPageWriter$ExtendedComponentTreeVisitCallback.visit(VisitContext, UIComponent) | STYLE | REC_CATCH_EXCEPTION | 1010 | Medium |
org.apache.myfaces.renderkit.html.HtmlAjaxBehaviorRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.behavior.ClientBehavior to javax.faces.component.behavior.AjaxBehavior in org.apache.myfaces.renderkit.html.HtmlAjaxBehaviorRenderer.decode(FacesContext, UIComponent, ClientBehavior) | STYLE | BC_UNCONFIRMED_CAST | 81 | Low |
Unchecked/unconfirmed cast from javax.faces.component.behavior.ClientBehavior to javax.faces.component.behavior.AjaxBehavior in org.apache.myfaces.renderkit.html.HtmlAjaxBehaviorRenderer.getScript(ClientBehaviorContext, ClientBehavior) | STYLE | BC_UNCONFIRMED_CAST | 94 | Low |
org.apache.myfaces.renderkit.html.HtmlCommandScriptRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.html.HtmlCommandScript in org.apache.myfaces.renderkit.html.HtmlCommandScriptRenderer.decode(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 155 | Low |
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.html.HtmlCommandScript in org.apache.myfaces.renderkit.html.HtmlCommandScriptRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 80 | Low |
Load of known null value in org.apache.myfaces.renderkit.html.HtmlCommandScriptRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 123 | Low |
org.apache.myfaces.renderkit.html.HtmlFormatRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Check for sign of bitwise operation in org.apache.myfaces.renderkit.html.HtmlFormatRenderer.encodeEnd(FacesContext, UIComponent) | BAD_PRACTICE | BIT_SIGNED_CHECK | 104 | Low |
Redundant nullcheck of text, which is known to be non-null in org.apache.myfaces.renderkit.html.HtmlFormatRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 95 | Low |
org.apache.myfaces.renderkit.html.HtmlHiddenRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIOutput in org.apache.myfaces.renderkit.html.HtmlHiddenRenderer.getConvertedValue(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 74 | Low |
org.apache.myfaces.renderkit.html.HtmlScriptRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.apache.myfaces.renderkit.html.HtmlScriptRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 196 | Medium |
org.apache.myfaces.renderkit.html.HtmlStylesheetRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.apache.myfaces.renderkit.html.HtmlStylesheetRenderer.encodeEnd(FacesContext, UIComponent) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 174 | Medium |
org.apache.myfaces.resource.TempDirFileCacheContractResourceLoader
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Possible null pointer dereference in org.apache.myfaces.resource.TempDirFileCacheContractResourceLoader.deleteDir(File) due to return value of called method | STYLE | NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE | 107 | Medium |
Exceptional return value of java.io.File.delete() ignored in org.apache.myfaces.resource.TempDirFileCacheContractResourceLoader.createTemporalFileVersion(FacesContext, ResourceMeta, File) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 221 | Medium |
Exceptional return value of java.io.File.mkdirs() ignored in org.apache.myfaces.resource.TempDirFileCacheContractResourceLoader.createTemporalFileVersion(FacesContext, ResourceMeta, File) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 220 | Medium |
Exceptional return value of java.io.File.mkdirs() ignored in org.apache.myfaces.resource.TempDirFileCacheContractResourceLoader.initialize() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 87 | Medium |
org.apache.myfaces.resource.TempDirFileCacheResourceLoader
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Possible null pointer dereference in org.apache.myfaces.resource.TempDirFileCacheResourceLoader.deleteDir(File) due to return value of called method | STYLE | NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE | 147 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.resource.TempDirFileCacheResourceLoader.isValidCreateTemporalFiles(FacesContext) | STYLE | REC_CATCH_EXCEPTION | 105 | Low |
Exceptional return value of java.io.File.delete() ignored in org.apache.myfaces.resource.TempDirFileCacheResourceLoader.createTemporalFileVersion(FacesContext, ResourceMeta, File) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 266 | Medium |
Exceptional return value of java.io.File.mkdirs() ignored in org.apache.myfaces.resource.TempDirFileCacheResourceLoader.createTemporalFileVersion(FacesContext, ResourceMeta, File) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 265 | Medium |
Exceptional return value of java.io.File.mkdirs() ignored in org.apache.myfaces.resource.TempDirFileCacheResourceLoader.initialize() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 127 | Medium |
Exceptional return value of java.io.File.mkdirs() ignored in org.apache.myfaces.resource.TempDirFileCacheResourceLoader.isValidCreateTemporalFiles(FacesContext) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 101 | Medium |
org.apache.myfaces.shared_impl.util.serial.SerialFactory
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class name org.apache.myfaces.shared_impl.util.serial.SerialFactory shadows the simple name of implemented interface org.apache.myfaces.shared.util.serial.SerialFactory | BAD_PRACTICE | NM_SAME_SIMPLE_NAME_AS_INTERFACE | Not available | Medium |
org.apache.myfaces.shared_impl.webapp.webxml.WebXmlParser
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.shared_impl.webapp.webxml.WebXmlParser.parse() | STYLE | REC_CATCH_EXCEPTION | 113 | Low |
org.apache.myfaces.spi.AnnotationProvider
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.spi.AnnotationProvider.getBaseUrls() is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | Not available | High |
org.apache.myfaces.spi.AnnotationProvider.getBaseUrls(ExternalContext) is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 112 | High |
org.apache.myfaces.spi.AnnotationProviderWrapper
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.spi.AnnotationProviderWrapper.getBaseUrls() is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 58 | High |
org.apache.myfaces.spi.AnnotationProviderWrapper.getBaseUrls(ExternalContext) is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 63 | High |
Class org.apache.myfaces.spi.AnnotationProviderWrapper implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 37-69 | Low |
org.apache.myfaces.spi.FactoryFinderProviderFactory
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Invocation of reflect.Field.setAccessible(boolean), which should be invoked from within a doPrivileged block, in org.apache.myfaces.spi.FactoryFinderProviderFactory.setInstance(FactoryFinderProviderFactory) | MALICIOUS_CODE | DP_DO_INSIDE_DO_PRIVILEGED | 152 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.spi.FactoryFinderProviderFactory.setInstance(FactoryFinderProviderFactory) | STYLE | REC_CATCH_EXCEPTION | 174 | Low |
org.apache.myfaces.spi.ServiceProviderFinder
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.spi.ServiceProviderFinder.KNOWN_SERVICES is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 40 | High |
org.apache.myfaces.spi.StateCacheProviderWrapper
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.spi.StateCacheProviderWrapper implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 29-52 | Low |
org.apache.myfaces.spi.impl.CDIAnnotationDelegateInjectionProvider
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.apache.myfaces.spi.impl.CDIAnnotationDelegateInjectionProvider(ExternalContext) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 46 | Medium |
Exception is caught when Exception is not thrown in new org.apache.myfaces.spi.impl.CDIAnnotationDelegateInjectionProvider(ExternalContext) | STYLE | REC_CATCH_EXCEPTION | 46 | Medium |
org.apache.myfaces.spi.impl.DefaultServiceProviderFinder
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.spi.impl.DefaultServiceProviderFinder.getURLs(String) is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 55-75 | High |
Found reliance on default encoding in org.apache.myfaces.spi.impl.DefaultServiceProviderFinder.getServiceProviderList(String): new java.io.InputStreamReader(InputStream) | I18N | DM_DEFAULT_ENCODING | 117 | High |
Redundant nullcheck of is, which is known to be non-null in org.apache.myfaces.spi.impl.DefaultServiceProviderFinder.getServiceProviderList(String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 104 | Low |
Redundant nullcheck of rd, which is known to be non-null in org.apache.myfaces.spi.impl.DefaultServiceProviderFinder.getServiceProviderList(String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 141 | Low |
Redundant nullcheck of profiles, which is known to be non-null in org.apache.myfaces.spi.impl.DefaultServiceProviderFinder.getURLs(String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 65 | Low |
org.apache.myfaces.spi.impl.NoInjectionAnnotationInjectionProvider
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The parameter metadata to org.apache.myfaces.spi.impl.NoInjectionAnnotationInjectionProvider.createDeclaredMethodBeansMap(ClassLoader, Map) is dead upon entry but overwritten | CORRECTNESS | IP_PARAMETER_IS_DEAD_BUT_OVERWRITTEN | 78 | Low |
Load of known null value in org.apache.myfaces.spi.impl.NoInjectionAnnotationInjectionProvider.getDeclaredMethodBeansMap() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 68 | Medium |
Redundant nullcheck of methods, which is known to be non-null in org.apache.myfaces.spi.impl.NoInjectionAnnotationInjectionProvider.postConstruct(Object, Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 134 | Medium |
org.apache.myfaces.spi.impl.ResourceAnnotationInjectionProvider
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Invocation of reflect.Field.setAccessible(boolean), which should be invoked from within a doPrivileged block, in org.apache.myfaces.spi.impl.ResourceAnnotationInjectionProvider.lookupFieldResource(Context, Object, Field, String) | MALICIOUS_CODE | DP_DO_INSIDE_DO_PRIVILEGED | 193 | Low |
Invocation of reflect.Method.setAccessible(boolean), which should be invoked from within a doPrivileged block, in org.apache.myfaces.spi.impl.ResourceAnnotationInjectionProvider.lookupMethodResource(Context, Object, Method, String) | MALICIOUS_CODE | DP_DO_INSIDE_DO_PRIVILEGED | 229 | Low |
The parameter metadata to org.apache.myfaces.spi.impl.ResourceAnnotationInjectionProvider.createDeclaredFieldBeansMap(ClassLoader, Map) is dead upon entry but overwritten | CORRECTNESS | IP_PARAMETER_IS_DEAD_BUT_OVERWRITTEN | 77 | Low |
Load of known null value in org.apache.myfaces.spi.impl.ResourceAnnotationInjectionProvider.getDeclaredFieldBeansMap() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 67 | Medium |
org.apache.myfaces.spi.impl.Tomcat7AnnotationInjectionProvider
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.spi.impl.Tomcat7AnnotationInjectionProvider.isAvailable() | STYLE | REC_CATCH_EXCEPTION | 158 | Medium |
org.apache.myfaces.taglib.core.ConverterImplTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.taglib.core.ConverterImplTag.createClassicConverter() | STYLE | REC_CATCH_EXCEPTION | 153 | Low |
org.apache.myfaces.taglib.core.ConverterTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.taglib.core.ConverterTag.createConverter() | STYLE | REC_CATCH_EXCEPTION | 124 | Low |
org.apache.myfaces.taglib.core.DelegateConverter
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.taglib.core.DelegateConverter._createConverter() | STYLE | REC_CATCH_EXCEPTION | 157 | Low |
org.apache.myfaces.taglib.core.DelegateValidator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.taglib.core.DelegateValidator._createValidator() | STYLE | REC_CATCH_EXCEPTION | 138 | Low |
org.apache.myfaces.taglib.core.DelegateValueChangeListener
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.apache.myfaces.taglib.core.DelegateValueChangeListener._createValueChangeListener() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 137 | Low |
org.apache.myfaces.taglib.core.LoadBundleTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Possible null pointer dereference of basename in org.apache.myfaces.taglib.core.LoadBundleTag.doStartTag() | CORRECTNESS | NP_NULL_ON_SOME_PATH | 123 | Medium |
org.apache.myfaces.taglib.core.LoadBundleTag$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.taglib.core.LoadBundleTag$BundleMap.containsKey(Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 186 | Medium |
org.apache.myfaces.taglib.core.PhaseListenerTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.taglib.core.PhaseListenerTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 48-140 | Low |
org.apache.myfaces.taglib.core.SetPropertyActionListenerTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.taglib.core.SetPropertyActionListenerTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 41-138 | Low |
org.apache.myfaces.taglib.core.SubviewTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of wrappedOutput, which is known to be non-null in org.apache.myfaces.taglib.core.SubviewTag.createVerbatimComponentFromBodyContent() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 73 | Low |
org.apache.myfaces.taglib.core.ValidatorImplTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.taglib.core.ValidatorImplTag.createClassicValidator() | STYLE | REC_CATCH_EXCEPTION | 151 | Low |
org.apache.myfaces.taglib.core.ValidatorImplTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 55-164 | Low |
org.apache.myfaces.taglib.core.ValidatorTag
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.taglib.core.ValidatorTag.createValidator() | STYLE | REC_CATCH_EXCEPTION | 107 | Low |
org.apache.myfaces.taglib.core.ValidatorTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 36-120 | Low |
org.apache.myfaces.util.DebugUtils
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.apache.myfaces.util.DebugUtils.componentAsString(UIComponent): java.io.ByteArrayOutputStream.toString() | I18N | DM_DEFAULT_ENCODING | 398 | High |
Found reliance on default encoding in org.apache.myfaces.util.DebugUtils.componentAsString(UIComponent): new java.io.PrintStream(OutputStream) | I18N | DM_DEFAULT_ENCODING | 396 | High |
Found reliance on default encoding in org.apache.myfaces.util.DebugUtils.traceView(String, UIViewRoot): java.io.ByteArrayOutputStream.toString() | I18N | DM_DEFAULT_ENCODING | 142 | High |
Found reliance on default encoding in org.apache.myfaces.util.DebugUtils.traceView(String, UIViewRoot): new java.io.PrintStream(OutputStream) | I18N | DM_DEFAULT_ENCODING | 133 | High |
org.apache.myfaces.util.IllegalXmlCharacterFilterWriter
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.util.IllegalXmlCharacterFilterWriter.encodeCharArray(char[], int, int) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 129 | Low |
org.apache.myfaces.util.SkipMatchIterator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store of null to value in org.apache.myfaces.util.SkipMatchIterator.next() | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 84 | Low |
org.apache.myfaces.view.ViewDeclarationLanguageFactoryImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.view.ViewDeclarationLanguageFactoryImpl.isFacelets2Enabled(FacesContext) | I18N | DM_CONVERT_CASE | 160 | Low |
org.apache.myfaces.view.ViewDeclarationLanguageFactoryImpl._supportedLanguages is a volatile reference to an array; the array elements are non-volatile | MT_CORRECTNESS | VO_VOLATILE_REFERENCE_TO_ARRAY | Not available | Low |
org.apache.myfaces.view.facelets.DefaultFaceletsStateManagementStrategy
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of state, which is known to be non-null in org.apache.myfaces.view.facelets.DefaultFaceletsStateManagementStrategy.restoreView(FacesContext, String, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 262 | Medium |
Redundant nullcheck of view, which is known to be non-null in org.apache.myfaces.view.facelets.DefaultFaceletsStateManagementStrategy.restoreView(FacesContext, String, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 218 | Low |
org.apache.myfaces.view.facelets.DefaultFaceletsStateManagementStrategy$AddComponentCallback
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.apache.myfaces.view.facelets.DefaultFaceletsStateManagementStrategy$AddComponentCallback(Object[]) may expose internal representation by storing an externally mutable object into DefaultFaceletsStateManagementStrategy$AddComponentCallback.addedState | MALICIOUS_CODE | EI_EXPOSE_REP2 | 607 | Medium |
org.apache.myfaces.view.facelets.DynamicComponentRefreshTransientBuildEvent
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.DynamicComponentRefreshTransientBuildEvent is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 34-40 | Low |
org.apache.myfaces.view.facelets.FaceletDynamicComponentRefreshTransientBuildEvent
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.FaceletDynamicComponentRefreshTransientBuildEvent is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 34-40 | Low |
org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage.PARAMS_REFRESH_PERIOD should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 195 | Medium |
Load of known null value in org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage.buildView(FacesContext, UIViewRoot) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 383 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage.renderView(FacesContext, UIViewRoot) | STYLE | REC_CATCH_EXCEPTION | 2029 | Low |
org.apache.myfaces.view.facelets.FaceletsVDLUtils$KeyComparator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.FaceletsVDLUtils$KeyComparator.compare(String, String) negates the return value of String.compareTo(String) | BAD_PRACTICE | RV_NEGATING_RESULT_OF_COMPARETO | 97 | Medium |
org.apache.myfaces.view.facelets.FaceletsVDLUtils$KeyComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 93-97 | Medium |
org.apache.myfaces.view.facelets.PostBuildComponentTreeOnRestoreViewEvent
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.PostBuildComponentTreeOnRestoreViewEvent is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 44-53 | Low |
org.apache.myfaces.view.facelets.StateWriter
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
StateWriter.fast not initialized in constructor and dereferenced in org.apache.myfaces.view.facelets.StateWriter.getAndResetBuffer() | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | 202 | Low |
org.apache.myfaces.view.facelets.ViewPoolProcessor
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Possible null pointer dereference of child in org.apache.myfaces.view.facelets.ViewPoolProcessor.clearTransientAndNonFaceletComponents(FacesContext, UIComponent) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 730 | Medium |
Possible null pointer dereference of fc in org.apache.myfaces.view.facelets.ViewPoolProcessor.clearTransientAndNonFaceletComponents(FacesContext, UIComponent) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 752 | Medium |
Possible null pointer dereference of child in org.apache.myfaces.view.facelets.ViewPoolProcessor.clearTransientAndNonFaceletComponentsForDynamicView(FacesContext, UIViewRoot, ViewStructureMetadata) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 557 | Medium |
Nullcheck of elMode at line 171 of value previously dereferenced in org.apache.myfaces.view.facelets.ViewPoolProcessor.initialize(FacesContext) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 171 | High |
Nullcheck of child at line 627 of value previously dereferenced in org.apache.myfaces.view.facelets.ViewPoolProcessor.clearComponentResourceContainer(FacesContext, UIComponent, List) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 627 | Medium |
Nullcheck of child at line 856 of value previously dereferenced in org.apache.myfaces.view.facelets.ViewPoolProcessor.clearTransientAndRemoveNonResetableComponents(FacesContext, ViewPoolProcessor$ClearPartialTreeContext, UIComponent) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 856 | Medium |
Nullcheck of fc at line 922 of value previously dereferenced in org.apache.myfaces.view.facelets.ViewPoolProcessor.clearTransientAndRemoveNonResetableComponents(FacesContext, ViewPoolProcessor$ClearPartialTreeContext, UIComponent) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 922 | Medium |
Nullcheck of child at line 777 of value previously dereferenced in org.apache.myfaces.view.facelets.ViewPoolProcessor.clearTransientAndRemoveNonResetableComponents(FacesContext, ViewPoolProcessor$ClearPartialTreeContext, UIViewRoot, ViewStructureMetadata) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 777 | Medium |
org.apache.myfaces.view.facelets.compiler.AddFacesMessageInstruction
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.el.ELContext to org.apache.myfaces.view.facelets.AbstractFaceletContext in org.apache.myfaces.view.facelets.compiler.AddFacesMessageInstruction.apply(ExpressionFactory, ELContext) | STYLE | BC_UNCONFIRMED_CAST | 55 | Low |
org.apache.myfaces.view.facelets.compiler.FaceletsCompilerSupport
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of tl, which is known to be non-null in org.apache.myfaces.view.facelets.compiler.FaceletsCompilerSupport.loadLibraries(FacesContext, Compiler) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 158 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.view.facelets.compiler.FaceletsCompilerSupport.loadDecorators(FacesContext, Compiler) | STYLE | REC_CATCH_EXCEPTION | 188 | Low |
org.apache.myfaces.view.facelets.compiler.SAXCompiler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.apache.myfaces.view.facelets.compiler.SAXCompiler.getXmlDecl(InputStream, CompilationManager): new String(byte[]) | I18N | DM_DEFAULT_ENCODING | 1045 | High |
Found reliance on default encoding in org.apache.myfaces.view.facelets.compiler.SAXCompiler.writeXmlDecl(InputStream, CompilationManager): new String(byte[]) | I18N | DM_DEFAULT_ENCODING | 1014 | High |
Null passed for non-null parameter of new org.apache.myfaces.view.facelets.tag.TagAttributeImpl(Location, String, String, String, String) in org.apache.myfaces.view.facelets.compiler.SAXCompiler.doCompileComponent(String, String, Map) | CORRECTNESS | NP_NULL_PARAM_DEREF | 947 | Medium |
Null passed for non-null parameter of new org.apache.myfaces.view.facelets.tag.TagAttributeImpl(Location, String, String, String, String) in org.apache.myfaces.view.facelets.compiler.SAXCompiler.doCompileComponent(String, String, Map) | CORRECTNESS | NP_NULL_PARAM_DEREF | 972 | Medium |
org.apache.myfaces.view.facelets.compiler.SAXCompiler$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.view.facelets.compiler.SAXCompiler$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 821-825 | Low |
org.apache.myfaces.view.facelets.compiler.SAXCompiler$CompilationHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Repeated conditional test in org.apache.myfaces.view.facelets.compiler.SAXCompiler$CompilationHandler.characters(char[], int, int) | CORRECTNESS | RpC_REPEATED_CONDITIONAL_TEST | 108 | Low |
Useless condition: it's known that this.consumingCDATA == true at this point | STYLE | UC_USELESS_CONDITION | 108 | Low |
SAXCompiler$CompilationHandler.locator not initialized in constructor and dereferenced in org.apache.myfaces.view.facelets.compiler.SAXCompiler$CompilationHandler.createLocation() | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | 136 | Low |
org.apache.myfaces.view.facelets.compiler.SAXCompiler$CompositeComponentMetadataHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Repeated conditional test in org.apache.myfaces.view.facelets.compiler.SAXCompiler$CompositeComponentMetadataHandler.characters(char[], int, int) | CORRECTNESS | RpC_REPEATED_CONDITIONAL_TEST | 550 | Low |
Useless condition: it's known that this.consumingCDATA == true at this point | STYLE | UC_USELESS_CONDITION | 550 | Low |
SAXCompiler$CompositeComponentMetadataHandler.locator not initialized in constructor and dereferenced in org.apache.myfaces.view.facelets.compiler.SAXCompiler$CompositeComponentMetadataHandler.createLocation() | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | 580 | Low |
org.apache.myfaces.view.facelets.compiler.SAXCompiler$ViewMetadataHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Repeated conditional test in org.apache.myfaces.view.facelets.compiler.SAXCompiler$ViewMetadataHandler.characters(char[], int, int) | CORRECTNESS | RpC_REPEATED_CONDITIONAL_TEST | 316 | Low |
Useless condition: it's known that this.consumingCDATA == true at this point | STYLE | UC_USELESS_CONDITION | 316 | Low |
SAXCompiler$ViewMetadataHandler.locator not initialized in constructor and dereferenced in org.apache.myfaces.view.facelets.compiler.SAXCompiler$ViewMetadataHandler.createLocation() | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | 344 | Low |
org.apache.myfaces.view.facelets.compiler.TagLibraryConfig
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.compiler.TagLibraryConfig is final but declares protected field org.apache.myfaces.view.facelets.compiler.TagLibraryConfig.log | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
org.apache.myfaces.view.facelets.compiler.TagLibraryConfig$TagLibraryImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of compositeComponentResource, which is known to be non-null in org.apache.myfaces.view.facelets.compiler.TagLibraryConfig$TagLibraryImpl.createTagHandler(String, String, TagConfig) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 254 | Low |
org.apache.myfaces.view.facelets.compiler.TagLibraryConfigUnmarshallerImpl$LibraryHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
TagLibraryConfigUnmarshallerImpl$LibraryHandler.locator not initialized in constructor and dereferenced in org.apache.myfaces.view.facelets.compiler.TagLibraryConfigUnmarshallerImpl$LibraryHandler.endElement(String, String, String) | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | 393 | Low |
org.apache.myfaces.view.facelets.compiler.TextUnit
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Impossible cast from org.apache.myfaces.view.facelets.compiler.LiteralNonExcapedTextInstruction to org.apache.myfaces.view.facelets.compiler.LiteralTextInstruction in org.apache.myfaces.view.facelets.compiler.TextUnit.compressSpaces(List, int) | CORRECTNESS | BC_IMPOSSIBLE_CAST | 589 | High |
Redundant nullcheck of s, which is known to be non-null in org.apache.myfaces.view.facelets.compiler.TextUnit.flushTextBuffer(boolean) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 197 | Low |
Useless condition: it's known that s.length() > 0 at this point | STYLE | UC_USELESS_CONDITION | 197 | Low |
org.apache.myfaces.view.facelets.compiler.UILeaf
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.view.facelets.compiler.UILeaf.getFacesListeners(Class) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 503 | Low |
org.apache.myfaces.view.facelets.component.JsfElementRenderer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to org.apache.myfaces.view.facelets.component.JsfElement in org.apache.myfaces.view.facelets.component.JsfElementRenderer.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 77 | Low |
Possible null pointer dereference of behaviors in org.apache.myfaces.view.facelets.component.JsfElementRenderer.encodeBegin(FacesContext, UIComponent) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 87 | Medium |
org.apache.myfaces.view.facelets.component.UIRepeat
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Suspicious comparison of a Integer reference to constant in org.apache.myfaces.view.facelets.component.UIRepeat.saveState(FacesContext) | BAD_PRACTICE | RC_REF_COMPARISON_BAD_PRACTICE | 1707 | High |
Suspicious comparison of a Integer reference to constant in org.apache.myfaces.view.facelets.component.UIRepeat.saveState(FacesContext) | BAD_PRACTICE | RC_REF_COMPARISON_BAD_PRACTICE | 1701 | High |
org.apache.myfaces.view.facelets.component.UIRepeat$IndexedEvent
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.component.UIRepeat$IndexedEvent is serializable but also an inner class of a non-serializable class | BAD_PRACTICE | SE_BAD_FIELD_INNER_CLASS | 1513-1576 | Low |
org.apache.myfaces.view.facelets.component.UIRepeat$IndexedEvent is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 1513-1576 | Low |
org.apache.myfaces.view.facelets.el.DefaultContextAwareELException
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.el.DefaultContextAwareELException is not derived from an Exception, even though it is named as such | BAD_PRACTICE | NM_CLASS_NOT_EXCEPTION | 29-68 | Medium |
org.apache.myfaces.view.facelets.el.ELText
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Null passed for non-null parameter of parse(ExpressionFactory, ELContext, String) in org.apache.myfaces.view.facelets.el.ELText.parseAllowEmptyString(String) | CORRECTNESS | NP_NULL_PARAM_DEREF | 467 | Medium |
Should org.apache.myfaces.view.facelets.el.ELText.parseAsArray(ExpressionFactory, ELContext, String) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 637 | Low |
org.apache.myfaces.view.facelets.el.ELText$ELCacheableTextVariable
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.el.ELContext to org.apache.myfaces.view.facelets.AbstractFaceletContext in org.apache.myfaces.view.facelets.el.ELText$ELCacheableTextVariable.apply(ExpressionFactory, ELContext) | STYLE | BC_UNCONFIRMED_CAST | 256 | Low |
org.apache.myfaces.view.facelets.el.ELText$LiteralValueExpression
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.el.ELText$LiteralValueExpression.equals(Object) always returns false | CORRECTNESS | EQ_ALWAYS_FALSE | 79 | High |
org.apache.myfaces.view.facelets.el.ResourceELUtils
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class name org.apache.myfaces.view.facelets.el.ResourceELUtils shadows the simple name of the superclass org.apache.myfaces.shared.resource.ResourceELUtils | BAD_PRACTICE | NM_SAME_SIMPLE_NAME_AS_SUPERCLASS | 26 | High |
org.apache.myfaces.view.facelets.el.ResourceLocationValueExpression
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.el.ResourceLocationValueExpression implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 32-149 | Low |
org.apache.myfaces.view.facelets.impl.DefaultFacelet
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of url, which is known to be non-null in org.apache.myfaces.view.facelets.impl.DefaultFacelet.getRelativePath(FacesContext, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 469 | Low |
org.apache.myfaces.view.facelets.impl.DefaultFaceletContext
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Bad attempt to compute absolute value of signed 32-bit hashcode in org.apache.myfaces.view.facelets.impl.DefaultFaceletContext.initPrefix() | CORRECTNESS | RV_ABSOLUTE_VALUE_OF_HASHCODE | 299 | High |
org.apache.myfaces.view.facelets.impl.DefaultFaceletContext$CompositeComponentTemplateManager
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.impl.DefaultFaceletContext$CompositeComponentTemplateManager is final but declares protected field org.apache.myfaces.view.facelets.impl.DefaultFaceletContext$CompositeComponentTemplateManager._target | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
org.apache.myfaces.view.facelets.impl.DefaultFaceletContext$CompositeComponentTemplateManager.equals(Object) is unusual | STYLE | EQ_UNUSUAL | 724 | Medium |
Class org.apache.myfaces.view.facelets.impl.DefaultFaceletContext$CompositeComponentTemplateManager implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 676-732 | Low |
org.apache.myfaces.view.facelets.impl.DefaultFaceletFactory
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.impl.DefaultFaceletFactory is final but declares protected field org.apache.myfaces.view.facelets.impl.DefaultFaceletFactory.log | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
org.apache.myfaces.view.facelets.impl.DefaultFaceletFactory.needsToBeRefreshed(DefaultFacelet) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 356 | Low |
Redundant nullcheck of url, which is known to be non-null in org.apache.myfaces.view.facelets.impl.DefaultFaceletFactory.getCompositeComponentMetadataFacelet(FacesContext, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 535 | Medium |
Redundant nullcheck of url, which is known to be non-null in org.apache.myfaces.view.facelets.impl.DefaultFaceletFactory.getFacelet(FacesContext, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 217 | Medium |
Redundant nullcheck of url, which is known to be non-null in org.apache.myfaces.view.facelets.impl.DefaultFaceletFactory.getViewMetadataFacelet(FacesContext, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 481 | Medium |
org.apache.myfaces.view.facelets.impl.FaceletCacheImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of is, which is known to be non-null in org.apache.myfaces.view.facelets.impl.FaceletCacheImpl.needsToBeRefreshed(DefaultFacelet) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 178 | Low |
org.apache.myfaces.view.facelets.impl.FaceletCompositionContextImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.apache.myfaces.view.facelets.impl.FaceletCompositionContextImpl.removeComponentForDeletion(UIComponent) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 881 | Medium |
Redundant nullcheck of id which is known to be null in org.apache.myfaces.view.facelets.impl.FaceletCompositionContextImpl.removeComponentForDeletion(UIComponent) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE | 881 | Low |
org.apache.myfaces.view.facelets.impl.SectionUniqueIdCounter
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.apache.myfaces.view.facelets.impl.SectionUniqueIdCounter(String, String[]) may expose internal representation by storing an externally mutable object into SectionUniqueIdCounter._uniqueIdsCache | MALICIOUS_CODE | EI_EXPOSE_REP2 | 79 | Medium |
org.apache.myfaces.view.facelets.impl.TemplateContextImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Call to org.apache.myfaces.view.facelets.TemplateManager.equals(javax.faces.view.facelets.Facelet) in org.apache.myfaces.view.facelets.impl.TemplateContextImpl.includeDefinition(FaceletContext, Facelet, UIComponent, String) | CORRECTNESS | EC_UNRELATED_TYPES | 132 | High |
org.apache.myfaces.view.facelets.impl.TemplateContextImpl$TemplateClientKnownParameterMap
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.impl.TemplateContextImpl$TemplateClientKnownParameterMap.getAttribute(String) has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 480 | Medium |
org.apache.myfaces.view.facelets.impl.TemplateContextImpl$TemplateManagerImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.impl.TemplateContextImpl$TemplateManagerImpl.equals(Object) is unusual | STYLE | EQ_UNUSUAL | 237-243 | Medium |
Class org.apache.myfaces.view.facelets.impl.TemplateContextImpl$TemplateManagerImpl implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | 153-257 | Low |
org.apache.myfaces.view.facelets.pool.impl.DynamicViewKey
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.pool.impl.DynamicViewKey is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 33-67 | Low |
org.apache.myfaces.view.facelets.pool.impl.MetadataViewKeyImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.apache.myfaces.view.facelets.pool.impl.MetadataViewKeyImpl(String, String, Locale, String[]) may expose internal representation by storing an externally mutable object into MetadataViewKeyImpl.contracts | MALICIOUS_CODE | EI_EXPOSE_REP2 | 52 | Medium |
org.apache.myfaces.view.facelets.pool.impl.MetadataViewKeyImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 40-127 | Low |
org.apache.myfaces.view.facelets.tag.CompositeFaceletHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.tag.CompositeFaceletHandler.getHandlers() may expose internal representation by returning CompositeFaceletHandler.children | MALICIOUS_CODE | EI_EXPOSE_REP | 61 | Medium |
new org.apache.myfaces.view.facelets.tag.CompositeFaceletHandler(FaceletHandler[]) may expose internal representation by storing an externally mutable object into CompositeFaceletHandler.children | MALICIOUS_CODE | EI_EXPOSE_REP2 | 46 | Medium |
org.apache.myfaces.view.facelets.tag.CompositeTagDecorator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.apache.myfaces.view.facelets.tag.CompositeTagDecorator(TagDecorator[]) may expose internal representation by storing an externally mutable object into CompositeTagDecorator.decorators | MALICIOUS_CODE | EI_EXPOSE_REP2 | 41 | Medium |
org.apache.myfaces.view.facelets.tag.CompositeTagLibrary
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.apache.myfaces.view.facelets.tag.CompositeTagLibrary(TagLibrary[]) may expose internal representation by storing an externally mutable object into CompositeTagLibrary.libraries | MALICIOUS_CODE | EI_EXPOSE_REP2 | 44 | Medium |
org.apache.myfaces.view.facelets.tag.LegacyUserTagHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.tag.LegacyUserTagHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.LegacyUserTagHandler._handlers | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
Class org.apache.myfaces.view.facelets.tag.LegacyUserTagHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.LegacyUserTagHandler._location | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
Class org.apache.myfaces.view.facelets.tag.LegacyUserTagHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.LegacyUserTagHandler._vars | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
Dead store to actx in org.apache.myfaces.view.facelets.tag.LegacyUserTagHandler.apply(FaceletContext, UIComponent, String) | STYLE | DLS_DEAD_LOCAL_STORE | 174 | Medium |
org.apache.myfaces.view.facelets.tag.MetaRulesetImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The parameter metadata to org.apache.myfaces.view.facelets.tag.MetaRulesetImpl.createMetaData(ClassLoader, Map) is dead upon entry but overwritten | CORRECTNESS | IP_PARAMETER_IS_DEAD_BUT_OVERWRITTEN | 97 | Medium |
Load of known null value in org.apache.myfaces.view.facelets.tag.MetaRulesetImpl.getMetaData() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 88 | Medium |
org.apache.myfaces.view.facelets.tag.MetadataImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.apache.myfaces.view.facelets.tag.MetadataImpl(Metadata[]) may expose internal representation by storing an externally mutable object into MetadataImpl._mappers | MALICIOUS_CODE | EI_EXPOSE_REP2 | 36 | Medium |
org.apache.myfaces.view.facelets.tag.MethodRule
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.apache.myfaces.view.facelets.tag.MethodRule(String, Class, Class[]) may expose internal representation by storing an externally mutable object into MethodRule.params | MALICIOUS_CODE | EI_EXPOSE_REP2 | 51 | Medium |
org.apache.myfaces.view.facelets.tag.MethodRule$MethodExpressionMetadata
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.view.facelets.tag.MethodRule$MethodExpressionMetadata be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 85-108 | Medium |
org.apache.myfaces.view.facelets.tag.TagAttributeImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.myfaces.view.facelets.tag.TagAttributeImpl.getMethodExpression(FaceletContext, Class, Class[]) | STYLE | REC_CATCH_EXCEPTION | 340 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.view.facelets.tag.TagAttributeImpl.getValueExpression(FaceletContext, Class) | STYLE | REC_CATCH_EXCEPTION | 580 | Low |
org.apache.myfaces.view.facelets.tag.TagAttributeImpl.cachedExpression is a volatile reference to an array; the array elements are non-volatile | MT_CORRECTNESS | VO_VOLATILE_REFERENCE_TO_ARRAY | Not available | Low |
org.apache.myfaces.view.facelets.tag.TagAttributesImpl
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.tag.TagAttributesImpl.getAll() may expose internal representation by returning TagAttributesImpl.attributes | MALICIOUS_CODE | EI_EXPOSE_REP | 101 | Medium |
org.apache.myfaces.view.facelets.tag.TagAttributesImpl.getNamespaces() may expose internal representation by returning TagAttributesImpl.namespaces | MALICIOUS_CODE | EI_EXPOSE_REP | 157 | Medium |
new org.apache.myfaces.view.facelets.tag.TagAttributesImpl(TagAttribute[]) may expose internal representation by storing an externally mutable object into TagAttributesImpl.attributes | MALICIOUS_CODE | EI_EXPOSE_REP2 | 51 | Medium |
org.apache.myfaces.view.facelets.tag.UserTagHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.tag.UserTagHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.UserTagHandler._handlers | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
Class org.apache.myfaces.view.facelets.tag.UserTagHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.UserTagHandler._location | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
Class org.apache.myfaces.view.facelets.tag.UserTagHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.UserTagHandler._vars | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
org.apache.myfaces.view.facelets.tag.composite.ClientBehaviorRedirectEventComponentWrapper
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.view.facelets.tag.composite.ClientBehaviorRedirectEventComponentWrapper.getFacesListeners(Class) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 427 | Low |
org.apache.myfaces.view.facelets.tag.composite.CompositeComponentBeanInfo
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.tag.composite.CompositeComponentBeanInfo.getPropertyDescriptors() may expose internal representation by returning CompositeComponentBeanInfo._propertyDescriptorsArray | MALICIOUS_CODE | EI_EXPOSE_REP | 145 | Medium |
org.apache.myfaces.view.facelets.tag.composite.CompositeComponentRule$LiteralAttributeMetadata
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.view.facelets.tag.composite.CompositeComponentRule$LiteralAttributeMetadata be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 47-55 | Medium |
org.apache.myfaces.view.facelets.tag.composite.CompositeComponentRule$TypedLiteralAttributeMetadata
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.view.facelets.tag.composite.CompositeComponentRule$TypedLiteralAttributeMetadata be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 65-74 | Medium |
org.apache.myfaces.view.facelets.tag.jsf.ActionSourceRule
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.tag.jsf.ActionSourceRule.ACTION_LISTENER_SIG should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 44 | Medium |
org.apache.myfaces.view.facelets.tag.jsf.BehaviorTagHandlerDelegate
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.behavior.ClientBehaviorHolder in org.apache.myfaces.view.facelets.tag.jsf.BehaviorTagHandlerDelegate.applyAttachedObject(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 152 | Low |
org.apache.myfaces.view.facelets.tag.jsf.ComponentRule$LiteralAttributeMetadata
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.view.facelets.tag.jsf.ComponentRule$LiteralAttributeMetadata be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 45-53 | Medium |
org.apache.myfaces.view.facelets.tag.jsf.ComponentSupport
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.view.facelets.tag.jsf.ComponentSupport.getLocale(FaceletContext, TagAttribute) | I18N | DM_CONVERT_CASE | 422 | Low |
Invocation of reflect.Method.setAccessible(boolean), which should be invoked from within a doPrivileged block, in org.apache.myfaces.view.facelets.tag.jsf.ComponentSupport.<static initializer for ComponentSupport>() | MALICIOUS_CODE | DP_DO_INSIDE_DO_PRIVILEGED | 66 | Low |
org.apache.myfaces.view.facelets.tag.jsf.ComponentTagHandlerDelegate
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.tag.jsf.ComponentTagHandlerDelegate.createComponent(FaceletContext) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 530 | Low |
Redundant nullcheck of parent, which is known to be non-null in org.apache.myfaces.view.facelets.tag.jsf.ComponentTagHandlerDelegate.apply(FaceletContext, UIComponent) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 387 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.view.facelets.tag.jsf.ComponentTagHandlerDelegate.createComponent(FaceletContext) | STYLE | REC_CATCH_EXCEPTION | 530 | Low |
org.apache.myfaces.view.facelets.tag.jsf.ConverterTagHandlerDelegate
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.ValueHolder in org.apache.myfaces.view.facelets.tag.jsf.ConverterTagHandlerDelegate.applyAttachedObject(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 137 | Low |
org.apache.myfaces.view.facelets.tag.jsf.ValidatorTagHandlerDelegate
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.EditableValueHolder in org.apache.myfaces.view.facelets.tag.jsf.ValidatorTagHandlerDelegate.applyAttachedObject(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 273 | Low |
org.apache.myfaces.view.facelets.tag.jsf.core.ActionListenerHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.ActionSource in org.apache.myfaces.view.facelets.tag.jsf.core.ActionListenerHandler.applyAttachedObject(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 192 | Low |
org.apache.myfaces.view.facelets.tag.jsf.core.AjaxHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.behavior.ClientBehaviorHolder in org.apache.myfaces.view.facelets.tag.jsf.core.AjaxHandler.applyAttachedObject(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 347 | Low |
org.apache.myfaces.view.facelets.tag.jsf.core.AjaxHandler.AJAX_BEHAVIOR_LISTENER_SIG should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 90 | Medium |
org.apache.myfaces.view.facelets.tag.jsf.core.AttributeHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class name org.apache.myfaces.view.facelets.tag.jsf.core.AttributeHandler shadows the simple name of implemented interface javax.faces.view.facelets.AttributeHandler | BAD_PRACTICE | NM_SAME_SIMPLE_NAME_AS_INTERFACE | 60-99 | Medium |
org.apache.myfaces.view.facelets.tag.jsf.core.FacetHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.tag.jsf.core.FacetHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.jsf.core.FacetHandler.name | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
The class name org.apache.myfaces.view.facelets.tag.jsf.core.FacetHandler shadows the simple name of implemented interface javax.faces.view.facelets.FacetHandler | BAD_PRACTICE | NM_SAME_SIMPLE_NAME_AS_INTERFACE | 55-85 | Medium |
org.apache.myfaces.view.facelets.tag.jsf.core.LoadBundleHandler$ResourceBundleMap
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.tag.jsf.core.LoadBundleHandler$ResourceBundleMap is final but declares protected field org.apache.myfaces.view.facelets.tag.jsf.core.LoadBundleHandler$ResourceBundleMap.bundle | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
org.apache.myfaces.view.facelets.tag.jsf.core.LoadBundleHandler$ResourceBundleMap$ResourceEntry
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.tag.jsf.core.LoadBundleHandler$ResourceBundleMap$ResourceEntry is final but declares protected field org.apache.myfaces.view.facelets.tag.jsf.core.LoadBundleHandler$ResourceBundleMap$ResourceEntry.key | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
Class org.apache.myfaces.view.facelets.tag.jsf.core.LoadBundleHandler$ResourceBundleMap$ResourceEntry is final but declares protected field org.apache.myfaces.view.facelets.tag.jsf.core.LoadBundleHandler$ResourceBundleMap$ResourceEntry.value | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
org.apache.myfaces.view.facelets.tag.jsf.core.ResetValuesActionListenerHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.ActionSource in org.apache.myfaces.view.facelets.tag.jsf.core.ResetValuesActionListenerHandler.applyAttachedObject(FacesContext, UIComponent, boolean) | STYLE | BC_UNCONFIRMED_CAST | 162 | Low |
org.apache.myfaces.view.facelets.tag.jsf.core.SelectItemsHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Private method org.apache.myfaces.view.facelets.tag.jsf.core.SelectItemsHandler.getNoSelectionValue() is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 55 | Low |
org.apache.myfaces.view.facelets.tag.jsf.core.SetPropertyActionListenerHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.ActionSource in org.apache.myfaces.view.facelets.tag.jsf.core.SetPropertyActionListenerHandler.applyAttachedObject(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 171 | Low |
org.apache.myfaces.view.facelets.tag.jsf.core.ValueChangeListenerHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.EditableValueHolder in org.apache.myfaces.view.facelets.tag.jsf.core.ValueChangeListenerHandler.applyAttachedObject(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 182 | Low |
org.apache.myfaces.view.facelets.tag.jsf.core.WebsocketHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIWebsocket in org.apache.myfaces.view.facelets.tag.jsf.core.WebsocketHandler.onComponentCreated(FaceletContext, UIComponent, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 58 | Low |
org.apache.myfaces.view.facelets.tag.jsf.html.DefaultTagDecorator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Private method org.apache.myfaces.view.facelets.tag.jsf.html.DefaultTagDecorator.isReservedJSFAttribute(String) is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 399-410 | Low |
org.apache.myfaces.view.facelets.tag.jstl.core.ForEachHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.apache.myfaces.view.facelets.tag.jstl.core.ForEachHandler.removeVarAndVarStatus(PageContext, String, ValueExpression, String, ValueExpression) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 667 | Medium |
Load of known null value in org.apache.myfaces.view.facelets.tag.jstl.core.ForEachHandler.removeVarAndVarStatus(PageContext, String, ValueExpression, String, ValueExpression) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 675 | Medium |
org.apache.myfaces.view.facelets.tag.jstl.core.ForEachHandler$ArrayIterator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.tag.jstl.core.ForEachHandler$ArrayIterator.next() can't throw NoSuchElementException | BAD_PRACTICE | IT_NO_SUCH_ELEMENT | 90 | Medium |
org.apache.myfaces.view.facelets.tag.jstl.core.IterationState
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.tag.jstl.core.IterationState is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 41-79 | Low |
org.apache.myfaces.view.facelets.tag.jstl.core.IterationStatusExpression
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.tag.jstl.core.IterationStatusExpression.equals(Object) checks for operand being a IterationStatus | BAD_PRACTICE | EQ_CHECK_FOR_OPERAND_NOT_COMPATIBLE_WITH_THIS | 79 | High |
org.apache.myfaces.view.facelets.tag.jstl.core.LegacyForEachHandler$ArrayIterator
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.tag.jstl.core.LegacyForEachHandler$ArrayIterator.next() can't throw NoSuchElementException | BAD_PRACTICE | IT_NO_SUCH_ELEMENT | 88 | Medium |
org.apache.myfaces.view.facelets.tag.jstl.core.LegacySetHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Nullcheck of LegacySetHandler.value at line 147 of value previously dereferenced in org.apache.myfaces.view.facelets.tag.jstl.core.LegacySetHandler.apply(FaceletContext, UIComponent) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 103 | Medium |
org.apache.myfaces.view.facelets.tag.jstl.core.MappedValueExpression$Entry
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.tag.jstl.core.MappedValueExpression$Entry is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 39-59 | Low |
org.apache.myfaces.view.facelets.tag.jstl.core.SetHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Nullcheck of SetHandler.value at line 162 of value previously dereferenced in org.apache.myfaces.view.facelets.tag.jstl.core.SetHandler.apply(FaceletContext, UIComponent) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 103 | Medium |
org.apache.myfaces.view.facelets.tag.jstl.fn.JstlFunction
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.view.facelets.tag.jstl.fn.JstlFunction.containsIgnoreCase(String, String) | I18N | DM_CONVERT_CASE | 59 | Low |
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.view.facelets.tag.jstl.fn.JstlFunction.toLowerCase(String) | I18N | DM_CONVERT_CASE | 379 | Low |
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.view.facelets.tag.jstl.fn.JstlFunction.toUpperCase(String) | I18N | DM_CONVERT_CASE | 394 | Low |
org.apache.myfaces.view.facelets.tag.ui.CompositionHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.tag.ui.CompositionHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.ui.CompositionHandler._handlers | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
Class org.apache.myfaces.view.facelets.tag.ui.CompositionHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.ui.CompositionHandler._params | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
Class org.apache.myfaces.view.facelets.tag.ui.CompositionHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.ui.CompositionHandler._template | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
org.apache.myfaces.view.facelets.tag.ui.DebugPhaseListener
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.tag.ui.DebugPhaseListener defines non-transient non-serializable instance field _currentPhase | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
Class org.apache.myfaces.view.facelets.tag.ui.DebugPhaseListener defines non-transient non-serializable instance field _visitCallback | BAD_PRACTICE | SE_BAD_FIELD | Not available | Low |
org.apache.myfaces.view.facelets.tag.ui.InsertHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Comparison of String parameter using == or != in org.apache.myfaces.view.facelets.tag.ui.InsertHandler.apply(FaceletContext, UIComponent, String) | BAD_PRACTICE | ES_COMPARING_PARAMETER_STRING_WITH_EQ | 110 | Low |
org.apache.myfaces.view.facelets.tag.ui.LegacyCompositionHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.tag.ui.LegacyCompositionHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.ui.LegacyCompositionHandler._handlers | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
Class org.apache.myfaces.view.facelets.tag.ui.LegacyCompositionHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.ui.LegacyCompositionHandler._params | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
Class org.apache.myfaces.view.facelets.tag.ui.LegacyCompositionHandler is final but declares protected field org.apache.myfaces.view.facelets.tag.ui.LegacyCompositionHandler._template | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
org.apache.myfaces.view.facelets.tag.ui.RepeatHandler$TagMetaData
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in new org.apache.myfaces.view.facelets.tag.ui.RepeatHandler$TagMetaData(RepeatHandler, Class) | STYLE | REC_CATCH_EXCEPTION | 92 | Low |
org.apache.myfaces.view.facelets.tag.ui.UIDebug
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.view.facelets.tag.ui.UIDebug.setHotkey(String) | I18N | DM_CONVERT_CASE | 243 | Low |
org.apache.myfaces.view.facelets.tag.ui.UIDebug$1
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class org.apache.myfaces.view.facelets.tag.ui.UIDebug$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 85-93 | Low |
org.apache.myfaces.view.facelets.util.Classpath
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.util.Classpath._searchDir(Set, File, String) is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 114-151 | High |
org.apache.myfaces.view.facelets.util.Classpath._searchFromURL(Set, String, String, URL) is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 170-235 | High |
org.apache.myfaces.view.facelets.util.Classpath._searchJar(ClassLoader, Set, JarFile, String, String) is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 328-348 | High |
org.apache.myfaces.view.facelets.util.Classpath._searchResource(Set, ClassLoader, String, String, String) is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 79-110 | High |
Exception is caught when Exception is not thrown in org.apache.myfaces.view.facelets.util.Classpath._searchFromURL(Set, String, String, URL) | STYLE | REC_CATCH_EXCEPTION | 210 | Medium |
org.apache.myfaces.view.facelets.util.ReflectionUtil
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.facelets.util.ReflectionUtil.PRIMITIVES should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 33 | Medium |
org.apache.myfaces.view.facelets.util.ReflectionUtil.PRIMITIVE_NAMES should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 30 | Medium |
Should org.apache.myfaces.view.facelets.util.ReflectionUtil.toTypeArray(String[]) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 100 | Low |
Should org.apache.myfaces.view.facelets.util.ReflectionUtil.toTypeNameArray(Class[]) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 122 | Low |
org.apache.myfaces.view.facelets.util.Resource
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.apache.myfaces.view.facelets.util.Resource is final but declares protected field org.apache.myfaces.view.facelets.util.Resource.log | STYLE | CI_CONFUSED_INHERITANCE | Not available | Low |
org.apache.myfaces.view.impl.DefaultViewScopeHandler
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Boxed value is unboxed and then immediately reboxed in org.apache.myfaces.view.impl.DefaultViewScopeHandler.generateViewScopeId(FacesContext) | PERFORMANCE | BX_UNBOXING_IMMEDIATELY_REBOXED | 177 | Medium |
org.apache.myfaces.view.jsp.JspViewDeclarationLanguage
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.view.jsp.JspViewDeclarationLanguage.FACELETS_ONLY_F_TAGS should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 65 | Medium |
org.apache.myfaces.view.jsp.JspViewDeclarationLanguage.FACELETS_ONLY_H_TAGS should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 66 | Medium |
org.apache.myfaces.webapp.AbstractFacesInitializer
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.webapp.AbstractFacesInitializer.lookupBeanManagerFromJndi() might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 714 | Low |
Use of non-localized String.toUpperCase() or String.toLowerCase() in org.apache.myfaces.webapp.AbstractFacesInitializer.initFaces(ServletContext) | I18N | DM_CONVERT_CASE | 257 | Low |
Invocation of reflect.Method.setAccessible(boolean), which should be invoked from within a doPrivileged block, in org.apache.myfaces.webapp.AbstractFacesInitializer.destroyFaces(ServletContext) | MALICIOUS_CODE | DP_DO_INSIDE_DO_PRIVILEGED | 427 | Low |
Exception is caught when Exception is not thrown in org.apache.myfaces.webapp.AbstractFacesInitializer.lookupBeanManagerFromCDI() | STYLE | REC_CATCH_EXCEPTION | 662 | Medium |
Exception is caught when Exception is not thrown in org.apache.myfaces.webapp.AbstractFacesInitializer.initFaces(ServletContext) | STYLE | REC_CATCH_EXCEPTION | 282 | Low |
org.apache.myfaces.webapp.Jsp20FacesInitializer$ErrorExpressionFactory
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.apache.myfaces.webapp.Jsp20FacesInitializer$ErrorExpressionFactory be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 135-163 | Medium |
org.apache.myfaces.webapp.ManagedBeanDestroyerListener
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Possible null pointer dereference of facesContext in org.apache.myfaces.webapp.ManagedBeanDestroyerListener.sessionDestroyed(HttpSessionEvent) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 205 | Medium |
org.apache.myfaces.webapp.MyFacesServlet
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.apache.myfaces.webapp.MyFacesServlet._facesInitializer is a mutable servlet field | MT_CORRECTNESS | MSF_MUTABLE_SERVLET_FIELD | 54 | Low |