You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by aw...@apache.org on 2007/05/09 07:39:23 UTC
svn commit: r536425 - in
/myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component:
AbstractComponentGenerator.java TrinidadComponentGenerator.java
Author: awiner
Date: Tue May 8 22:39:22 2007
New Revision: 536425
URL: http://svn.apache.org/viewvc?view=rev&rev=536425
Log:
ADFFACES-487: Change Component generator to us XXX.valueOf(x) instead of new XXX(x) when converting between primitives to Objects in components
Modified:
myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component/AbstractComponentGenerator.java
myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component/TrinidadComponentGenerator.java
Modified: myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component/AbstractComponentGenerator.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component/AbstractComponentGenerator.java?view=diff&rev=536425&r1=536424&r2=536425
==============================================================================
--- myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component/AbstractComponentGenerator.java (original)
+++ myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component/AbstractComponentGenerator.java Tue May 8 22:39:22 2007
@@ -405,30 +405,69 @@
}
else if ("char".equals(className))
{
- return "new Character('" + value.replaceAll("\'", "\\'") + "')";
+ return "Character.valueOf('" + value.replaceAll("\'", "\\'") + "')";
}
else if ("int".equals(className))
{
- return "new Integer(" + value + ")";
+ return "Integer.valueOf(" + value + ")";
}
else if ("double".equals(className))
{
- return "new Double(" + value + ")";
+ return "Double.valueOf(" + value + ")";
}
else if ("float".equals(className))
{
- return "new Float(" + value + ")";
+ return "Float.valueOf(" + value + ")";
}
else if ("Number".equals(className))
{
if(value.indexOf(".") == -1)
{
- return "new Integer(" + value + ")";
+ return "Integer.valueOf(" + value + ")";
}
else
{
- return "new Double(" + value + ")";
+ return "Double.valueOf(" + value + ")";
}
+ }
+ else
+ {
+ throw new IllegalStateException("property-class " + className + " not supported for auto-boxing");
+ }
+ }
+
+
+ protected String convertVariableToBoxedForm(
+ String className,
+ String varName)
+ {
+ if ("boolean".equals(className))
+ {
+ return varName + " ? Boolean.TRUE : Boolean.FALSE";
+ }
+ else if ("char".equals(className))
+ {
+ return "Character.valueOf(" + varName + ")";
+ }
+ else if ("int".equals(className))
+ {
+ return "Integer.valueOf(" + varName + ")";
+ }
+ else if ("short".equals(className))
+ {
+ return "Short.valueOf(" + varName + ")";
+ }
+ else if ("long".equals(className))
+ {
+ return "Long.valueOf(" + varName + ")";
+ }
+ else if ("double".equals(className))
+ {
+ return "Double.valueOf(" + varName + ")";
+ }
+ else if ("float".equals(className))
+ {
+ return "Float.valueOf(" + varName + ")";
}
else
{
Modified: myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component/TrinidadComponentGenerator.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component/TrinidadComponentGenerator.java?view=diff&rev=536425&r1=536424&r2=536425
==============================================================================
--- myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component/TrinidadComponentGenerator.java (original)
+++ myfaces/trinidad/branches/faces-1_2-070427/plugins/maven-faces-plugin/src/main/java/org/apache/myfaces/trinidadbuild/plugin/faces/generator/component/TrinidadComponentGenerator.java Tue May 8 22:39:22 2007
@@ -168,17 +168,9 @@
if (Util.isPrimitiveClass(propertyClass))
{
- // TODO: use UIXComponentBase setXXXProperty methods when possible
- if (propertyClass.equals("boolean"))
- {
- // TODO: add back space before ternary operator
- out.println("setProperty(" + propKey + ", " + propVar + "? Boolean.TRUE : Boolean.FALSE);");
- }
- else
- {
- String boxedClass = Util.getBoxedClass(propertyClass);
- out.println("setProperty(" + propKey + ", new " + boxedClass + "(" + propVar + "));");
- }
+ out.println("setProperty(" + propKey + ", " +
+ convertVariableToBoxedForm(propertyClass, propVar) +
+ ");");
}
else
{