You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wink.apache.org by "Mike Rheinheimer (JIRA)" <ji...@apache.org> on 2009/07/10 18:17:14 UTC

[jira] Created: (WINK-71) getContextResolver search algorithm does not expand for text/*

getContextResolver search algorithm does not expand for text/*
--------------------------------------------------------------

                 Key: WINK-71
                 URL: https://issues.apache.org/jira/browse/WINK-71
             Project: Wink
          Issue Type: Bug
          Components: Common
    Affects Versions: 0.1
            Reporter: Mike Rheinheimer
             Fix For: 0.1
         Attachments: patch.txt

I'm hoping this patch gets overwritten by WINK-47, but they are not related.  There is a definite existing bug in that when we have a provider:

    @Provider
    @Produces( "text/plain", "*/*" )
    public static class StringContextResolver implements ContextResolver<String> {
        ...

And search by:

     ProvidersRegistry.getContextResolver(String.class, new MediaType("text", "*"), null)

We'll get null back.  The algorithm in ProvidersRegistry.MediaTypeMap.getProvidersByMediaType(MediaType, Class<?>)  needs fixed.  Please see patch for test that demonstrates the problem, and the fix.

This Jira can be closed if WINK-47 is integrated prior to integrating the attached patch.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (WINK-71) getContextResolver search algorithm does not expand for text/*

Posted by "Mike Rheinheimer (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WINK-71?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mike Rheinheimer closed WINK-71.
--------------------------------

    Resolution: Duplicate

This Jira is already fixed by WINK-47.  Hence, I'm closing this one.

> getContextResolver search algorithm does not expand for text/*
> --------------------------------------------------------------
>
>                 Key: WINK-71
>                 URL: https://issues.apache.org/jira/browse/WINK-71
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 0.1
>            Reporter: Mike Rheinheimer
>             Fix For: 0.1
>
>         Attachments: patch.txt
>
>
> I'm hoping this patch gets overwritten by WINK-47, but they are not related.  There is a definite existing bug in that when we have a provider:
>     @Provider
>     @Produces( "text/plain", "*/*" )
>     public static class StringContextResolver implements ContextResolver<String> {
>         ...
> And search by:
>      ProvidersRegistry.getContextResolver(String.class, new MediaType("text", "*"), null)
> We'll get null back.  The algorithm in ProvidersRegistry.MediaTypeMap.getProvidersByMediaType(MediaType, Class<?>)  needs fixed.  Please see patch for test that demonstrates the problem, and the fix.
> This Jira can be closed if WINK-47 is integrated prior to integrating the attached patch.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (WINK-71) getContextResolver search algorithm does not expand for text/*

Posted by "Mike Rheinheimer (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WINK-71?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mike Rheinheimer updated WINK-71:
---------------------------------

    Attachment: patch.txt

> getContextResolver search algorithm does not expand for text/*
> --------------------------------------------------------------
>
>                 Key: WINK-71
>                 URL: https://issues.apache.org/jira/browse/WINK-71
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 0.1
>            Reporter: Mike Rheinheimer
>             Fix For: 0.1
>
>         Attachments: patch.txt
>
>
> I'm hoping this patch gets overwritten by WINK-47, but they are not related.  There is a definite existing bug in that when we have a provider:
>     @Provider
>     @Produces( "text/plain", "*/*" )
>     public static class StringContextResolver implements ContextResolver<String> {
>         ...
> And search by:
>      ProvidersRegistry.getContextResolver(String.class, new MediaType("text", "*"), null)
> We'll get null back.  The algorithm in ProvidersRegistry.MediaTypeMap.getProvidersByMediaType(MediaType, Class<?>)  needs fixed.  Please see patch for test that demonstrates the problem, and the fix.
> This Jira can be closed if WINK-47 is integrated prior to integrating the attached patch.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.