You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2014/06/27 13:11:24 UTC
[jira] [Commented] (WW-4295) provide better documentation and / or
functionality for localization of struts2 tag elements (select, radio,
etc.)
[ https://issues.apache.org/jira/browse/WW-4295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14045814#comment-14045814 ]
ASF GitHub Bot commented on WW-4295:
------------------------------------
GitHub user lukaszlenart opened a pull request:
https://github.com/apache/struts/pull/18
Feature: list value localization
This is continuation of @abrin work started in #7
Closes [WW-4295](https://issues.apache.org/jira/browse/WW-4295)
Closes #7
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/struts feature/WW-4295-localization
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/struts/pull/18.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #18
----
commit 7f586d39b7f8411e22fd13de8025f1866477fc3c
Author: adam brin <ad...@asu.edu>
Date: 2014-03-18T14:36:28Z
[PATCH] First attempt at enabling value-labels to be pulled from
Locale Files. Adding listValueKey attribute to checkboxList, radioMap, and
select tags.
Issues:
This implementation uses the <@s.text> tag to handle the localization with an assignment. I presume this is less than idea, but I lack the knowledge of the Struts2 tag model that would suggest where to check for this on the valueStack or elsewhere.
Example:
public enum yesno {
YES, NO;
}
<@s.radio name="yes no" list="#{yesno.values()}" valueKey="name()" />
<!-- this will look for the name() in the locale file and emit the localized value for the label of the radio button.
commit 901893628b31ee3b38b2fc9196045a097a0f4217
Author: Lukasz Lenart <lu...@apache.org>
Date: 2014-03-20T07:08:03Z
Adds additional field to allow set label key used in I18N lookup
commit 69df87047ab2a72c339a0ee320459757ba439f40
Author: Lukasz Lenart <lu...@apache.org>
Date: 2014-03-20T07:08:27Z
Extends template to use the new field to localise value's label
commit 7e98fe7a731b44bb85375f9363465838803a0a16
Author: Lukasz Lenart <lu...@apache.org>
Date: 2014-03-20T07:08:50Z
Adds autogenerated files
commit b78349ad5dade8a47fb335f6ef18c13a36101842
Author: Lukasz Lenart <lu...@apache.org>
Date: 2014-03-20T07:12:13Z
Extends blank app to demonstrate the new possibilities
commit 6def514524e81c4af188b47b6694384eea3a46c6
Author: adam brin <ad...@asu.edu>
Date: 2014-03-24T14:31:54Z
updating ListUIBean
updating java-backed bean
commit 23498a332119486a043e852366dfb73495a510d9
Author: Lukasz Lenart <lu...@apache.org>
Date: 2014-06-27T06:40:49Z
Merge remote-tracking branch 'abrin/feature-WW-4295-localization' into feature/WW-4295-localization
commit ca230ab30fde249d8af17cc5f2ba83ebe142d364
Author: Lukasz Lenart <lu...@apache.org>
Date: 2014-06-27T10:54:26Z
Adds auto-generated tags' info
commit 13269fa15f02409732ca51545141a40ae2f4680c
Author: Lukasz Lenart <lu...@apache.org>
Date: 2014-06-27T10:54:44Z
Adds missing tag attribute field
commit a2791a8305666859d6484098fb597f93c94908bc
Author: Lukasz Lenart <lu...@apache.org>
Date: 2014-06-27T10:55:13Z
Removes optional valueKey to keep backward compatibility
commit ed8a0b87b29114ac5c99838e7d4cfc819ad71e43
Author: Lukasz Lenart <lu...@apache.org>
Date: 2014-06-27T10:55:43Z
Checks is ListValueKey was defined and returns value
commit d67fce1f2577229139938d4588bb970c3f869078
Author: Lukasz Lenart <lu...@apache.org>
Date: 2014-06-27T10:56:01Z
Extends test to use the new attribute
----
> provide better documentation and / or functionality for localization of struts2 tag elements (select, radio, etc.)
> ------------------------------------------------------------------------------------------------------------------
>
> Key: WW-4295
> URL: https://issues.apache.org/jira/browse/WW-4295
> Project: Struts 2
> Issue Type: Bug
> Affects Versions: 2.3.16
> Reporter: adam brin
> Fix For: 2.3.18
>
> Attachments: WW-4295.patch
>
>
> The <@s.text> macro is very useful for localization of text values. The <@s.textfield tag also takes a "key" > but, the select, radio, and checkbox options don't have good options for localization. One option raised on the list and identified in a few places is to construct a separate map of variables that have been localized in Freemarker and use that for localization (https://stackoverflow.com/questions/5548104/struts-select-tag-localization-implementation/17258640#17258640)... this does not seem sustainable or maintainable. Two possible methods for addressing this might be:
> # update the documentation to better identify how to localize these values in the "approved" way
> # implement a listKey parameter that like the textfield implementation or other implementations might call getText() on the key prior to rendering
> {code:title=Example Implementation for Radiobutton}
> <#if parameters.listKey??>
> <#assign itemValue><@s.text name="${stack.findString(parameters.listKey)}" /><#rt></#assign>
> <#elseif parameters.listValue??>
> <#assign itemValue = stack.findString(parameters.listValue)/>
> <#else>
> <#assign itemValue = stack.findString('top')/>
> </#if>
> {code}
--
This message was sent by Atlassian JIRA
(v6.2#6252)