You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by im...@apache.org on 2006/02/22 19:28:11 UTC
svn commit: r379833 -
/myfaces/commons/trunk/src/main/java/org/apache/myfaces/taglib/UIComponentTagUtils.java
Author: imario
Date: Wed Feb 22 10:28:08 2006
New Revision: 379833
URL: http://svn.apache.org/viewcvs?rev=379833&view=rev
Log:
rollback MYFACES-1127
current solution didnt work if you reference the *listener through the
var= attribute (e.g. within an datatable)
Modified:
myfaces/commons/trunk/src/main/java/org/apache/myfaces/taglib/UIComponentTagUtils.java
Modified: myfaces/commons/trunk/src/main/java/org/apache/myfaces/taglib/UIComponentTagUtils.java
URL: http://svn.apache.org/viewcvs/myfaces/commons/trunk/src/main/java/org/apache/myfaces/taglib/UIComponentTagUtils.java?rev=379833&r1=379832&r2=379833&view=diff
==============================================================================
--- myfaces/commons/trunk/src/main/java/org/apache/myfaces/taglib/UIComponentTagUtils.java (original)
+++ myfaces/commons/trunk/src/main/java/org/apache/myfaces/taglib/UIComponentTagUtils.java Wed Feb 22 10:28:08 2006
@@ -284,12 +284,14 @@
MethodBinding mb = context.getApplication().createMethodBinding(actionListener,
ACTION_LISTENER_ARGS);
+ /**
if(! Void.class.equals(mb.getType(context)))
{
throw new IllegalArgumentException(
actionListener +
" : Return types for action listeners must be void, see JSF spec. 3.2.1.1");
}
+ */
((ActionSource)component).setActionListener(mb);
}
@@ -314,13 +316,14 @@
{
MethodBinding mb = context.getApplication().createMethodBinding(valueChangedListener,
VALUE_LISTENER_ARGS);
-
+ /**
if(! Void.class.equals(mb.getType(context)))
{
throw new IllegalArgumentException(
valueChangedListener +
" : Return types for value change listeners must be void, see JSF spec. 3.2.5.1");
}
+ */
((EditableValueHolder)component).setValueChangeListener(mb);
}