[ 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

Reply via email to