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

Allow String converters #131

Closed
eclipse-faces-bot opened this issue Nov 7, 2005 · 14 comments
Closed

Allow String converters #131

eclipse-faces-bot opened this issue Nov 7, 2005 · 14 comments

Comments

@eclipse-faces-bot
Copy link

The JSF-RI disallows String converters because of performance reasons, according
to Adam Winer.

The specification and javadoc APIs are silent on this limitation. The
"documentation" of the behavior is a comment in a private method.

// if converterType is null, String, or Object, assume no
conversion is needed

My use case is that I would like to create a java.lang.String
converter that automatically converts all empty strings to null.

I am also appending another user's request for String converter functionality,
since he's also not heard back on this issue, and it's essentially the same request.

==============================================================

To: jsr-252-comments@jcp.org
Cc: mkienenb@gmail.com
Subject: String Converter
Message-ID: OF2A560BB1.750CF5CB-ON89257098.005FD316-89257098.0063D709@blm.gov
From: Dennis_Byrne@ak.blm.gov
Date: Wed, 12 Oct 2005 10:10:31 -0800

I would like to register a Converter for Strings, allowing me to trim
everything in the RR [RenderResponse] phase. This would save me a lot of time,
as there are several hundred Strings in my domain model and the database yields
padding for text/char fields. Trimming Strings is not a performance hit
for this usecase, since an intern will have to write code to manually trim
everything otherwise AFAIK, the 1.1 spec permits this but I am alone
w/ this interpretation. What are the chances of this being explicitly
defined in the 1.2 spec?

Environment

Operating System: All
Platform: All

Affected Versions

[2.0]

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Reported by mkienenb

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
This is worth trying for 1.2.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Created an attachment (id=97)
complete files for this change-bundle.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
File: jsf-spec-131.tar.gz
Attached By: @edburns

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Created an attachment (id=98)
Fix for this bug, first iteration.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
File: message.txt
Attached By: @edburns

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@rlubke said:
r=rlubke

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Fix checked in, you can now use String Converters.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Prepare to delete api subcomponent

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Move all to 1.2

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Marked as fixed on Thursday, March 4th 2010, 6:09:28 am

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@manfredriem said:
Closing resolved issue out

@eclipse-faces-bot
Copy link
Author

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

@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