Re: ResponseBuilder

From: Bill Burke <>
Date: Fri, 13 Jun 2008 09:19:17 -0400

I'm saying isn't it easier to do:

builder.type("text/xml") rather than
builder.type(MediaType.valueOf("text/xml")) // this requires an import too.


builder.language("fr") rather than whatever the Locale equivalent is?

Why can't we have both methods?


Stephan Koops wrote:
> The javadoc of method ResponseBuilder.type(String) says, that the given
> value must be parseable. This requires to test it, that means typically
> try to create an instance and see, if no IllegalArgumentException is
> thrown (the way is not required, but typically). For other objects, e.g.
> CacheControl or NewCookie it seems useful to me to not serialize them to
> a String until it is necessary. For consistency it seems good to me to
> not convert everything to a String, but to leave it as it is, until it
> is possible.
> Stephan
> Bill Burke schrieb:
>> You don't think using strings is easier? I don't know...
>> Marc Hadley wrote:
>>> On Jun 12, 2008, at 4:34 PM, Stephan Koops wrote:
>>>> If calling ResponseBuilder.type(String), must the String be
>>>> converted to a MediaType? Or ResponseBuilder.type(MediaType) be
>>>> converted to a String? Or should it not changed in both cases? This
>>>> should be clear if a message body writer would reads data from the
>>>> Response for necessary casts. A javadoc clarification on both
>>>> ResponseBuilder.type(.) methods and Response.getMetadata() would be
>>>> good.
>>>> The same issue exists for both ResponseBuilder.tag(.)
>>> I propose we convert to instances of MediaType and EntityTag.
>>>> If HttpHeaders.getAcceptedLanguages() returns List<Locale>, than we
>>>> should also add ResponseBuilder.language(Locale) and the conversion
>>>> be documented.
>>> Yes.
>>> While we are thinking about Locale, anyone have an objection to
>>> closing issue 42 by switching from String to Locale:
>>> Marc.
>>> ---
>>> Marc Hadley <marc.hadley at>
>>> CTO Office, Sun Microsystems.
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail:
>>> For additional commands, e-mail:
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

Bill Burke
JBoss, a division of Red Hat