You are viewing a plain text version of this content. The canonical link for it is here.
Posted to adffaces-commits@incubator.apache.org by aw...@apache.org on 2007/02/09 02:32:47 UTC
svn commit: r505144 -
/incubator/adffaces/branches/faces-1_2-070102/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/GenerateJspTaglibsMojo.java
Author: awiner
Date: Thu Feb 8 18:32:46 2007
New Revision: 505144
URL: http://svn.apache.org/viewvc?view=rev&rev=505144
Log:
ADFFACES-377: JSF 1.2: EL binding to null Strings show up as empty strings
Modified:
incubator/adffaces/branches/faces-1_2-070102/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/GenerateJspTaglibsMojo.java
Modified: incubator/adffaces/branches/faces-1_2-070102/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/GenerateJspTaglibsMojo.java
URL: http://svn.apache.org/viewvc/incubator/adffaces/branches/faces-1_2-070102/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/GenerateJspTaglibsMojo.java?view=diff&rev=505144&r1=505143&r2=505144
==============================================================================
--- incubator/adffaces/branches/faces-1_2-070102/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/GenerateJspTaglibsMojo.java (original)
+++ incubator/adffaces/branches/faces-1_2-070102/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/GenerateJspTaglibsMojo.java Thu Feb 8 18:32:46 2007
@@ -1787,7 +1787,14 @@
static final private Set _CAN_COERCE = new HashSet();
static
{
- _CAN_COERCE.add("java.lang.String");
+ // What? Can't coerce Strings? How could that be? Well, take a look at:
+ // http://issues.apache.org/jira/browse/ADFFACES-377
+ // The silly coercion rules in JSP convert null to the
+ // empty string. So it's not that we can't coerce to
+ // String, we just really, really don't want to.
+ // _CAN_COERCE.add("java.lang.String");
+ // TODO: consider getting rid of coercion rules for
+ // all non-primitives
_CAN_COERCE.add("java.lang.Integer");
_CAN_COERCE.add("java.lang.Long");
_CAN_COERCE.add("java.lang.Boolean");