incubator-adffaces-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From awi...@apache.org
Subject 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
Date Fri, 09 Feb 2007 02:32:47 GMT
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");



Mime
View raw message