This repository was archived by the owner on Feb 26, 2023. It is now read-only.
Repository navigation
This repository was archived by the owner on Feb 26, 2023. It is now read-only.
@Accept: Add support to other MediaType #221
Description
Activity
For
@Accept, we can either :- Replace the
MediaTypeenum with a class holding String constants, and update@Acceptto take a Stringvalue()param - Keep the current
value()parameter that can have a default empty value, and add a newstringType()(we need to find a better name) that accepts Strings.
I think the first option would be more universal. And it should be retrocompatible, if we keep the same qualified name for
MediaType.In the generated code, instead of:
httpHeaders.setAccept(Collections.singletonList(MediaType.APPLICATION_JSON));
We would do:
httpHeaders.setAccept(Collections.singletonList(MediaType.parseMediaType("text/json")));
- Replace the
I think I did that because Spring dependencies are not available in androidannotations project. So, I have created our own
MediaTypeto use the "native" annotation validation.See:
@Retention(RetentionPolicy.SOURCE) @Target({ ElementType.METHOD, ElementType.TYPE }) public @interface Accept { MediaType value(); }
You're right :) .
- added a commit that references this issue
on Aug 2, 2012
I'm trying to use a JSON API. The API defines a specific mime type => application/vnd.kippt.YYYYMMDD+json (where YYYYMMDD is the date of the API you want to use). This way they introduce a versioned API.
I'm not saying that it's a good solution to version API, but I just try to use their API !
Width @accept annotation, I can only use MediaType which is an Enum. With class Spring MediaType, I can create my own Mediatype.
I think that adding two attributes to @accept annotations would be enough (type and subtype).