[jira] Commented: (WINK-167) The sort algorithm in ProvidersRegistry.getProvidersByWildcardMediaType ignores priority

["Mike Rheinheimer (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/WINK-167?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12758302#action_12758302 ] 

Mike Rheinheimer commented on WINK-167:
---------------------------------------

I think perhaps the easy and fast way to do this is to just sort the result list at the end of the getProvidersByMediaType method.  I think trying to sort inline would be too expensive since the sort currently implemented there is sorting Entry<MediaType, Set<ObjectFactory<T>>> and the list we need sorted is the value of that Entry object.  It's probably easier and faster to go ahead and sort based on MediaTypes then sort the result list, which will hopefully be short.

Here is a proposed fix;  insert this code just before the 'return list;' statement at the bottom of getProvidersByMediaType.  I'm posting the new code inline here because a patch would overlap with WINK-193

            // now we should sort by declared priority on the ObjectFactory
            Collections.sort(list, new Comparator<ObjectFactory<T>>() {

                public int compare(ObjectFactory<T> object1, ObjectFactory<T> object2) {
                    PriorityObjectFactory<T> factory1 = (PriorityObjectFactory<T>)object1;
                    PriorityObjectFactory<T> factory2 = (PriorityObjectFactory<T>)object2;
                    if (factory1.priority > factory2.priority) {
                        return -1;
                    } else if (factory1.priority < factory2.priority) {
                        return 1;
                    }
                    return 0;
                }

            });


Also add the following test to org.apache.wink.common.internal.providers.ProvidersContextResolverTest

    public void testContextResolverPrioritySort() {
        ProvidersRegistry providers = createProvidersRegistryImpl();
        // note: the order these are added is important to the test
        assertTrue(providers.addProvider(new StringContextResolver4(), 0));
        assertTrue(providers.addProvider(new StringContextResolver3(), 0.2));
        assertTrue(providers.addProvider(new StringContextResolver2(), 0.1));

        // StringContextResolver3 has the highest priority (0.2) even though StringContextResolver2
        // more closely matches based on the media type in @Produces
        
        assertSame(STRING3, providers.getContextResolver(String.class,
                                                         null,
                                                         null).getContext(String.class));
    }

> The sort algorithm in ProvidersRegistry.getProvidersByWildcardMediaType ignores priority
> ----------------------------------------------------------------------------------------
>
>                 Key: WINK-167
>                 URL: https://issues.apache.org/jira/browse/WINK-167
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 0.1
>            Reporter: Michael Elman
>
> The sort algorithm in ProvidersRegistry.getProvidersByWildcardMediaType ignores priority property of the provider.
> It should be latest key in the sort algorithm as it is used in getProvidersByMediaType method.

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