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:29 UTC

[jira] Created: (ADFFACES-22) Allow parameter values to take a wider range of types

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 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);
           }
         }

-- 
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


[jira] Closed: (ADFFACES-22) Allow parameter values to take a wider range of types

Posted by "Adam Winer (JIRA)" <ad...@incubator.apache.org>.
     [ http://issues.apache.org/jira/browse/ADFFACES-22?page=all ]
     
Adam Winer closed ADFFACES-22:
------------------------------

    Resolution: Fixed

Fixed, thanks (tweaked the patch to also handle null values).

> 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
>     Assignee: Adam Winer
>     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


[jira] Updated: (ADFFACES-22) Allow parameter values to take a wider range of types

Posted by "Matthias Weßendorf (JIRA)" <ad...@incubator.apache.org>.
     [ https://issues.apache.org/jira/browse/ADFFACES-22?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matthias Weßendorf updated ADFFACES-22:
---------------------------------------

        Fix Version/s: 1.0.0-incubating-core
    Affects Version/s: 1.0.0-incubating-core

> Allow parameter values to take a wider range of types
> -----------------------------------------------------
>
>                 Key: ADFFACES-22
>                 URL: https://issues.apache.org/jira/browse/ADFFACES-22
>             Project: MyFaces ADF-Faces
>          Issue Type: Bug
>    Affects Versions: 1.0.0-incubating-core
>         Environment: 3.2-SNAPSHOT version of the andromda.org jsf cartridge.
>            Reporter: Leif Johansson
>         Assigned To: Adam Winer
>            Priority: Minor
>             Fix For: 1.0.0-incubating-core
>
>
> 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.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (ADFFACES-22) Allow parameter values to take a wider range of types

Posted by "Adam Winer (JIRA)" <ad...@incubator.apache.org>.
     [ http://issues.apache.org/jira/browse/ADFFACES-22?page=all ]

Adam Winer reassigned ADFFACES-22:
----------------------------------

    Assign To: Adam Winer

> 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
>     Assignee: Adam Winer
>     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


[jira] Updated: (ADFFACES-22) Allow parameter values to take a wider range of types

Posted by "Leif Johansson (JIRA)" <ad...@incubator.apache.org>.
     [ 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