Does not work with more than 2 query parameters

Nearly found the issue, its because in this function 'redirect_load_by_source' it creates an array of potential redirect urls based on the arguments and query params. In my case above, the urls destination, resort and hotel will all be potential redirect links because of the rules that state if a child page cannot be found, go to the nearest parent page.

So with this array of potential urls it then does a sort in function '_redirect_uasort', here it orders based on language and rid where rid relates to the row position of the csv import and therefore assumes that when importing that rows will be in the order

destination -> node/x
resort -> node/y
hotel -> node/z

but if the order is reversed or random then it will select which ever row comes last as the final url to redirect to.

Which in my case is the incorrect url.

What i think is needed is ordering based on specificity, the url with the most matching query parameters would be selected to redirect to.

Don't you agree?

Found and fixed the problem

in redirect.module at line 1419

replace:

elseif (!empty($a->source_options['query']) != !empty($b->source_options['query'])) {
// Then sort by redirects that do not have query strings over ones that do.
return empty($a->source_options['query']);
}
with:

elseif(count($a->source_options['query']) != count($b->source_options['query'])){
return count($a->source_options['query']) < count($b->source_options['query']);
}

Now everything works fine no matter what order the redirects were created.

Patch attached