[
https://issues.apache.org/struts/browse/WW-2156?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=44617#action_44617
]
Dave Newton commented on WW-2156:
---------------------------------
IMO it's actually a little backwards, the naming. the "doubleName" is what ends
up being set on the action. Personally I think the "name" attribute should be
the required one, and it's the name of the action property that gets set.
doubleName should be optional, and if not supplied, built from the "name"
attribute, although I'm not sure how it'd be useful.
> double doubleselect without ids and name cause selection error
> --------------------------------------------------------------
>
> Key: WW-2156
> URL: https://issues.apache.org/struts/browse/WW-2156
> Project: Struts 2
> Issue Type: Bug
> Components: Plugin - Tags
> Affects Versions: 2.0.9
> Reporter: Yinghao LI
> Fix For: 2.1.3
>
>
> When there are two doubleselect without id attributes, the selection of the
> first's first select while change the options of the second's second select.
> The problem is :
> the name and the id of doubleselect aren't required attributes. When they are
> both absent, the value of name and id is the form_name concat with "_". When
> we use two or more doubleselect, struts will create same variables and
> function in javascript with the doubleselect.ftl in the simple template.
> Personnally, i think the id of doubleselect should use the doubleName's value
> when there is neither id attribute nor name attribute.
> <script type="text/javascript">
> <#assign itemCount = startCount/>
> var ${parameters.id}Group = new Array(${parameters.listSize} +
> ${startCount});
> for (i = 0; i < (${parameters.listSize} + ${startCount}); i++)
> ${parameters.id}Group[i] = new Array();
> <@s.iterator value="parameters.list">
> <#if parameters.listKey?exists>
> <#assign itemKey = stack.findValue(parameters.listKey)/>
> <#else>
> <#assign itemKey = stack.findValue('top')/>
> </#if>
> <#if parameters.listValue?exists>
> <#assign itemValue = stack.findString(parameters.listValue)/>
> <#else>
> <#assign itemValue = stack.findString('top')/>
> </#if>
> <#assign doubleItemCount = 0/>
> <@s.iterator value="${parameters.doubleList}">
> <#if parameters.doubleListKey?exists>
> <#assign doubleItemKey =
> stack.findValue(parameters.doubleListKey)/>
> <#else>
> <#assign doubleItemKey = stack.findValue('top')/>
> </#if>
> <#assign doubleItemKeyStr = doubleItemKey.toString() />
> <#if parameters.doubleListValue?exists>
> <#assign doubleItemValue =
> stack.findString(parameters.doubleListValue)/>
> <#else>
> <#assign doubleItemValue = stack.findString('top')/>
> </#if>
> ${parameters.id}Group[${itemCount}][${doubleItemCount}] = new
> Option("${doubleItemValue}", "${doubleItemKeyStr}");
> <#assign doubleItemCount = doubleItemCount + 1/>
> </@s.iterator>
> <#assign itemCount = itemCount + 1/>
> </@s.iterator>
> var ${parameters.id}Temp =
> document.${parameters.formName}.${parameters.doubleId};
> <#assign itemCount = startCount/>
> <#assign redirectTo = 0/>
> <@s.iterator value="parameters.list">
> <#if parameters.listKey?exists>
> <#assign itemKey = stack.findValue(parameters.listKey)/>
> <#else>
> <#assign itemKey = stack.findValue('top')/>
> </#if>
> <#if tag.contains(parameters.nameValue, itemKey)>
> <#assign redirectTo = itemCount/>
> </#if>
> <#assign itemCount = itemCount + 1/>
> </@s.iterator>
> ${parameters.id}Redirect(${redirectTo});
> function ${parameters.id}Redirect(x) {
> var selected = false;
> for (m = ${parameters.id}Temp.options.length - 1; m >= 0; m--) {
> ${parameters.id}Temp.options[m] = null;
> }
> 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>
> }
> if ((${parameters.id}Temp.options.length > 0) && (! selected)) {
> ${parameters.id}Temp.options[0].selected = true;
> }
> }
> </script>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
