You are viewing a plain text version of this content. The canonical link for it is here.
Posted to adffaces-issues@incubator.apache.org by "Leif Johansson (JIRA)" <ad...@incubator.apache.org> on 2006/06/18 08:06:31 UTC
[jira] Updated: (ADFFACES-22) Allow parameter values to take a
wider range of types
[ http://issues.apache.org/jira/browse/ADFFACES-22?page=all ]
Leif Johansson updated ADFFACES-22:
-----------------------------------
Description:
When modelling a Long parameter on an action I noticed that FireActionBoundValue could not
handle UIParameter values not of type String. By replacing the typecast (String) with a call
to toString() a wider range of types are permitted. Impact on existing code should be nil. This
is a patch against adf-faces-impl revision 414891.
Index: src/main/java/org/apache/myfaces/adfinternal/uinode/FireActionBoundValue.java
===================================================================
--- src/main/java/org/apache/myfaces/adfinternal/uinode/FireActionBoundValue.java (revision 414891)
+++ src/main/java/org/apache/myfaces/adfinternal/uinode/FireActionBoundValue.java (working copy)
@@ -100,7 +100,7 @@
{
Parameter p = new Parameter();
p.setKey(((UIParameter)kid).getName());
- p.setValue((String)((UIParameter)kid).getValue());
+ p.setValue(((UIParameter)kid).getValue().toString());
params.add(p);
}
}
was:
When modelling a Long parameter on an action I noticed that FireActionBoundValue could not
handle UIParameter values not of type String. By replacing the typecast (String) with a call
to toString() a wider range of types are permitted. Impact on existingcode should be nil. This
is a patch agains adf-faces-impl revision 414891.
Index: src/main/java/org/apache/myfaces/adfinternal/uinode/FireActionBoundValue.java
===================================================================
--- src/main/java/org/apache/myfaces/adfinternal/uinode/FireActionBoundValue.java (revision 414891)
+++ src/main/java/org/apache/myfaces/adfinternal/uinode/FireActionBoundValue.java (working copy)
@@ -100,7 +100,7 @@
{
Parameter p = new Parameter();
p.setKey(((UIParameter)kid).getName());
- p.setValue((String)((UIParameter)kid).getValue());
+ p.setValue(((UIParameter)kid).getValue().toString());
params.add(p);
}
}
> Allow parameter values to take a wider range of types
> -----------------------------------------------------
>
> Key: ADFFACES-22
> URL: http://issues.apache.org/jira/browse/ADFFACES-22
> Project: MyFaces ADF-Faces
> Type: Bug
> Environment: 3.2-SNAPSHOT version of the andromda.org jsf cartridge.
> Reporter: Leif Johansson
> Priority: Minor
>
> When modelling a Long parameter on an action I noticed that FireActionBoundValue could not
> handle UIParameter values not of type String. By replacing the typecast (String) with a call
> to toString() a wider range of types are permitted. Impact on existing code should be nil. This
> is a patch against adf-faces-impl revision 414891.
> Index: src/main/java/org/apache/myfaces/adfinternal/uinode/FireActionBoundValue.java
> ===================================================================
> --- src/main/java/org/apache/myfaces/adfinternal/uinode/FireActionBoundValue.java (revision 414891)
> +++ src/main/java/org/apache/myfaces/adfinternal/uinode/FireActionBoundValue.java (working copy)
> @@ -100,7 +100,7 @@
> {
> Parameter p = new Parameter();
> p.setKey(((UIParameter)kid).getName());
> - p.setValue((String)((UIParameter)kid).getValue());
> + p.setValue(((UIParameter)kid).getValue().toString());
> params.add(p);
> }
> }
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira