You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ta...@apache.org on 2017/09/04 13:08:19 UTC
svn commit: r1807221 -
/myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderForFaces.java
Author: tandraschko
Date: Mon Sep 4 13:08:18 2017
New Revision: 1807221
URL: http://svn.apache.org/viewvc?rev=1807221&view=rev
Log:
MYFACES-3250 disable ManagedBeanResolver
Modified:
myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderForFaces.java
Modified: myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderForFaces.java
URL: http://svn.apache.org/viewvc/myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderForFaces.java?rev=1807221&r1=1807220&r2=1807221&view=diff
==============================================================================
--- myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderForFaces.java (original)
+++ myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderForFaces.java Mon Sep 4 13:08:18 2017
@@ -98,6 +98,9 @@ public class ResolverBuilderForFaces ext
@Override
public void build(FacesContext facesContext, CompositeELResolver compositeElResolver)
{
+ MyfacesConfig config = MyfacesConfig.getCurrentInstance(
+ FacesContext.getCurrentInstance().getExternalContext());
+
// add the ELResolvers to a List first to be able to sort them
List<ELResolver> list = new ArrayList<>();
@@ -122,7 +125,10 @@ public class ResolverBuilderForFaces ext
//Flash object is instanceof Map, so it is necessary to resolve
//before MapELResolver. Better to put this one before
list.add(new FlashELResolver());
- list.add(new ManagedBeanResolver());
+ if (config.isSupportManagedBeans())
+ {
+ list.add(new ManagedBeanResolver());
+ }
list.add(new ResourceResolver());
list.add(new ResourceBundleELResolver());
list.add(new ResourceBundleResolver());
@@ -175,8 +181,7 @@ public class ResolverBuilderForFaces ext
}
// Only add this resolver if the user wants to use the EL ImportHandler
- if (MyfacesConfig.getCurrentInstance(
- FacesContext.getCurrentInstance().getExternalContext()).isSupportEL3ImportHandler())
+ if (config.isSupportEL3ImportHandler())
{
compositeElResolver.add(new ImportHandlerResolver());
}