[jira] [Resolved] (WW-3919) Checkboxlist does not render label tag correctly

["Lukasz Lenart (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/WW-3919?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Lukasz Lenart resolved WW-3919.
-------------------------------

    Resolution: Fixed
      Assignee: Lukasz Lenart

Patch applied, please prepare diff file next time ;-) Thanks for reporting!
                
> Checkboxlist does not render label tag correctly
> ------------------------------------------------
>
>                 Key: WW-3919
>                 URL: https://issues.apache.org/jira/browse/WW-3919
>             Project: Struts 2
>          Issue Type: Bug
>          Components: Plugin - Java Templates
>            Reporter: Greg Huber
>            Assignee: Lukasz Lenart
>            Priority: Minor
>             Fix For: 2.3.9
>
>
> Hello,
> Here are the mods to get the label working!
> {code:java}
> //Checkbox button section
> Attributes a = new Attributes();
> a.add("type", "checkbox")
> .add("name", name)
> .add("value", itemKeyStr)
> .addIfTrue("checked", checked)
> .addIfTrue("readonly", params.get("readonly"))
> .addIfTrue("disabled", disabled)
> .addIfExists("tabindex", params.get("tabindex"))
> //.addIfExists("id", name + "-" + Integer.toString(cnt++));
> .addIfExists("id", id + "-" + Integer.toString(cnt));  << CHANGED
> start("input", a);
> end("input");
> //Label section
> a = new Attributes();
> //a.add("for",id);
> a.add("for",id + "-" + Integer.toString(cnt))  << CHANGED
> .addIfExists("class", params.get("cssClass"))
> .addIfExists("style", params.get("cssStyle"));
> super.start("label", a);
>    if (StringUtils.isNotEmpty(itemValueStr))
>                     characters(itemValueStr);
>                 super.end("label");
>                 //Hidden input section
>                 a = new Attributes();
>                 a.add("type", "hidden")
>                         .add("id", "__multiselect_" + StringUtils.defaultString(StringEscapeUtils.escapeHtml4(id)))
>                         .add("name", "__multiselect_" + StringUtils.defaultString(StringEscapeUtils.escapeHtml4(name)))
>                         .add("value", "")
>                         .addIfTrue("disabled", disabled);
>                 start("input", a);
>                 end("input");
>                 super.end("br");
>                 stack.pop();
>                 
>                 cnt++;  // Increment <<< ADDED
> {code}
> Cheers Greg

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira