[jira] Created: (WW-2809) doubleselect ignores preselection of a multiple subselect

[Markus Katz (JIRA)]

doubleselect ignores preselection of a multiple subselect
---------------------------------------------------------

                 Key: WW-2809
                 URL: https://issues.apache.org/struts/browse/WW-2809
             Project: Struts 2
          Issue Type: Bug
    Affects Versions: 2.1.2
            Reporter: Markus Katz
            Priority: Minor


template/simple/doubleselect.ftl contains the following snippet to preselect 
the subselection:

        for (i = 0; i < ${parameters.id}Group[x].length; i++) {
            ${parameters.id}Temp.options[i] = new 
Option(${parameters.id}Group[x][i].text, ${parameters.id}Group[x][i].value);
            <#if parameters.doubleNameValue?exists>
                if (${parameters.id}Temp.options[i].value == 
'${parameters.doubleNameValue}') {
                        ${parameters.id}Temp.options[i].selected = true;
                        selected = true;
                }
            </#if>
        }

In the case of a multiple subselect the doubleNameValue is a collection of ids. 
In this case the single ...Temp.options[i].value is compared to a collection 
object which fails for all items.

I´d suggest a patch like:

        for (i = 0; i < ${parameters.id}Group[x].length; i++) {
            ${parameters.id}Temp.options[i] = new 
Option(${parameters.id}Group[x][i].text, ${parameters.id}Group[x][i].value);
            <#if parameters.doubleNameValue?exists>
                   <#if parameters.doubleMultiple?exists>
                         for (j = 0; j < ${parameters.doubleNameValue}.length; 
j++) {
                             if (${parameters.id}Temp.options[i].value == 
${parameters.doubleNameValue}[j]) {
                               ${parameters.id}Temp.options[i].selected = true;
             selected = true;
           }
        }
                      <#else>
         if (${parameters.id}Temp.options[i].value == 
'${parameters.doubleNameValue}') {
           ${parameters.id}Temp.options[i].selected = true;
           selected = true;
         }
                      </#if>
                 </#if>
            }




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