[jboss-cvs] jboss-seam/src/ui/org/jboss/seam/ui/facelet ...
Gavin King
gavin.king at jboss.com
Fri Oct 27 17:32:35 EDT 2006
User: gavin
Date: 06/10/27 17:32:35
Modified: src/ui/org/jboss/seam/ui/facelet SeamExpressionFactory.java
Log:
event parameters
Revision Changes Path
1.3 +5 -5 jboss-seam/src/ui/org/jboss/seam/ui/facelet/SeamExpressionFactory.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: SeamExpressionFactory.java
===================================================================
RCS file: /cvsroot/jboss/jboss-seam/src/ui/org/jboss/seam/ui/facelet/SeamExpressionFactory.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -b -r1.2 -r1.3
--- SeamExpressionFactory.java 12 Oct 2006 22:34:01 -0000 1.2
+++ SeamExpressionFactory.java 27 Oct 2006 21:32:35 -0000 1.3
@@ -49,9 +49,7 @@
faceletsExpressionFactory = compiler.createExpressionFactory();
}
- public SeamExpressionFactory()
- {
- }
+ public SeamExpressionFactory() {}
/**
* Note default access. Other classes in this package use this factory
@@ -75,13 +73,15 @@
Class[] paramTypes)
{
MethodExpressionParser parser = new MethodExpressionParser(expression);
- if (parser.isParamExpression())
+ if ( parser.isParamExpression() )
{
return new ParamMethodExpression(parser, elContext);
}
- if (paramTypes.length==1 && ActionEvent.class.equals( paramTypes[0] ) )
+ if ( paramTypes.length==1 && paramTypes[0].equals(ActionEvent.class) )
{
+ //so that JSF action listeners don't have to declare
+ //the totally frickin useless ActionEvent parameter
return new OptionalParameterMethodExpression(
faceletsExpressionFactory.createMethodExpression(elContext, expression, returnType, paramTypes),
faceletsExpressionFactory.createMethodExpression(elContext, expression, returnType, new Class[0])
More information about the jboss-cvs-commits
mailing list