Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve UISelectMany to automatically convert based on select items when type is java.util.Collection #1422

Closed
eclipse-faces-bot opened this issue Jun 11, 2016 · 6 comments

Comments

@eclipse-faces-bot
Copy link

Currently, http://docs.oracle.com/javaee/7/api/javax/faces/component/UISelectMany.html states that it won't make any attempt to obtain the converter when the value is of type java.util.Collection. It is technically however possible to automatically convert based on select items.

* <li class="changed_added_2_0"><p>A <code>java.util.Collection</code>.
 * Do not convert the values. <span class="changed_modified_2_3">Instead,
 * convert the provided set of available options to string, exactly as done
 * during render response, and for any match with the submitted values, add
 * the available option as object to the collection.</span></p></li>

Affected Versions

[2.3]

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Reported by @BalusC

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@BalusC said:
Scratch previous commit which was taken over from MyFaces. It didn't properly cover cases when a collection contains items of different types (bad practice, but technically not impossible). Here are the improvements: https://java.net/projects/mojarra/sources/git/revision/bbf7f0e0546b376d6fb8f927f6b2703f5152b184

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Marked as fixed on Friday, June 17th 2016, 5:17:47 am

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
This issue was imported from java.net JIRA JAVASERVERFACES_SPEC_PUBLIC-1422

@eclipse-faces-bot
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant