[jira] Commented: (TOMAHAWK-1194) SelectOneRow: The attribute GroupName can't contain EL

[Leonardo Uribe (JIRA)]

    [ 
https://issues.apache.org/jira/browse/TOMAHAWK-1194?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12604330#action_12604330
 ] 

Leonardo Uribe commented on TOMAHAWK-1194:
------------------------------------------


Checking the renderer I'm not think that allow EL in groupName could be valid.

groupName parameter is used to set the name on the input type:

<s:selectOneRow  groupName="selection" id="hugo" 
value="#{selectOneRowList.selectedRowIndex}"
         onchange="submit();" immediate="true"
          valueChangeListener="#{selectOneRowList.processRowSelection}"/>

One of the elements rendered looks like this:

<input type="radio" name="selection" id="_idJsp0:data:0:hugo" 
value="_idJsp0:data:0:hugo" onchange="submit();" />

the name must be a static String value, if this is not preserved, unwanted 
effects could occur (on decode method the parameter on request scope with the 
name contained in groupName is retrieved), so this issue should be marked as 
invalid.

if no objections I'll close this issue.

> SelectOneRow: The attribute GroupName can't contain EL
> ------------------------------------------------------
>
>                 Key: TOMAHAWK-1194
>                 URL: https://issues.apache.org/jira/browse/TOMAHAWK-1194
>             Project: MyFaces Tomahawk
>          Issue Type: Improvement
>    Affects Versions: 1.1.6
>         Environment: tomahawk-sandbox-1.1.6
>            Reporter: Jorge Rodríguez Pedrianes
>            Priority: Minor
>             Fix For: 1.1.7-SNAPSHOT
>
>         Attachments: patch.txt
>
>   Original Estimate: 0.08h
>  Remaining Estimate: 0.08h
>
> Hello!
>     I am using SelectOneRow but i see that wich groupName attribute a can't 
> use expressiona language.
>    I attach a path to allow this.
>    Thanks

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.