Feature: list value localization #18
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
updating java-backed bean
Conflicts: core/src/site/resources/tags/checkboxlist.html
kusalk
added a commit
that referenced
this pull request
Dec 27, 2023
Merge in BAM/struts2-atlassian from stash-WW-5340-ognl-guard to fork/6.3 * commit 'ef7ef2f08f73e6a56860932aaf1105fa1dd3daeb': WW-5340 Correct optimisation WW-5340 Refactor OgnlGuard to do the parsing WW-5340 Add unit tests WW-5340 Add validation to excluded node configuration WW-5340 Cache OgnlGuard result WW-5340 Make OgnlGuard a configurable bean WW-5340 Fix tests WW-5340 Introducing OGNL Guard
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is continuation of @abrin work started in #7
Closes WW-4295
Closes #7