You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ma...@apache.org on 2008/08/29 14:20:41 UTC

svn commit: r690201 - /myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/menu/ItemNode.java

Author: matzew
Date: Fri Aug 29 05:20:40 2008
New Revision: 690201

URL: http://svn.apache.org/viewvc?rev=690201&view=rev
Log:
TRINIDAD-1134 - XmlMenuModel: action is interpreted as ValueExpression
Thanks to Abhijit S Ghosh for his patch

Modified:
    myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/menu/ItemNode.java

Modified: myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/menu/ItemNode.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/menu/ItemNode.java?rev=690201&r1=690200&r2=690201&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/menu/ItemNode.java (original)
+++ myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/menu/ItemNode.java Fri Aug 29 05:20:40 2008
@@ -20,6 +20,8 @@
 
 import java.util.Map;
 
+import javax.faces.context.FacesContext;
+import javax.faces.el.MethodBinding;
 import javax.faces.webapp.UIComponentTag;
 
 /**
@@ -71,9 +73,10 @@
         && UIComponentTag.isValueReference(value)
        ) 
     {
-      // Value of action is EL method binding, so we 
-      // need to evaluate it
-      value = (String)MenuUtils.getBoundValue(value);
+      FacesContext facesContext = FacesContext.getCurrentInstance();
+      MethodBinding methodBinding = facesContext.getApplication().createMethodBinding(value,null);
+      
+      value = (String) methodBinding.invoke(facesContext, null);
     }
 
     // Post me as the selected Node for the request