You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wink.apache.org by "Luciano Resende (JIRA)" <ji...@apache.org> on 2011/04/04 05:11:06 UTC

[jira] [Resolved] (WINK-342) Some static MediaType instances on MediaTypeUtils are incomplete/inconsistent

     [ https://issues.apache.org/jira/browse/WINK-342?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Luciano Resende resolved WINK-342.
----------------------------------

    Resolution: Fixed

Refactored the existing types to add prefix to _TYPE, and also added missing entries

> Some static MediaType instances on MediaTypeUtils are incomplete/inconsistent
> -----------------------------------------------------------------------------
>
>                 Key: WINK-342
>                 URL: https://issues.apache.org/jira/browse/WINK-342
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 1.1.2
>            Reporter: Chris Francuz
>            Priority: Minor
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> On the org.apache.wink.common.internal.utils.MediaTypeUtils class, the convention is to have 2 constants, 1 for the string representation and 1 for the MediaType representation, eg
>     public static final String IMAGE_JPEG = "image/jpeg";
>     public static final MediaType IMAGE_JPEG_TYPE = new MediaType("image", "jpeg");
>     public static final String UNKNOWN = "x-internal/unknown";
>     public static final MediaType UNKNOWN_TYPE = new MediaType("x-internal", "unknown");
> But in some cases there is no string constant, and the MediaType representation is name as per the other string contents, without the _TYPE suffix, eg
>     public static final MediaType IMAGE_X_ICON = new MediaType("image", "x-icon");
>     public static final MediaType IMAGE_PNG = new MediaType("image", "png");
>     public static final MediaType IMAGE_VND = new MediaType("image", "vnd.microsoft.icon");

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira