Paths with question marks (question mark NOT at the front of the url)

Try without the trailing slash (wordpress?feed=rss2). Also, if these paths currently exist and are actually valid wordpress paths, we can't do anything about it.

Ah... I am having the same issue. I have a main page "something.asp" and it has some sub pages "something.asp?tab=1" and "something.asp?tab=2". I set up redirects for all three of them to different URLs but they all redirect to the URL for "something.asp". I'm not sure why it "fails". Perhaps it is matching the first one and quits looking for the one that matches the query string. Is this a bug?

I believe the logic in path_redirect_compare_array() is a little backwards. The function accepts two parameters: $match and $haystack where $match is the query string from the database and $haystack is the query string from the original URL.

function path_redirect_compare_array($match, $haystack) {
  foreach ($match as $key => $value) {
    if (!array_key_exists($key, $haystack)) {
      return FALSE;
    }
    elseif (is_array($value)) {
      if (!is_array($haystack[$key])) {
        return FALSE;
      }
      elseif (!path_redirect_compare_array($value, $haystack[$key])) {
        return FALSE;
      }
    }
    elseif ($value != $haystack[$key]) {
      return FALSE;
    }
  }
  return TRUE;
}

So path_redirect_compare_array() is called repeatedly with the results of $redirects = path_redirect_load_multiple(). But if $redirects contains a redirect without a query string then when path_redirect_compare_array() gets called, it gets passed an empty $match array which will never get looped through in the function so it will return a TRUE... even if another redirect had matching query parameter. I think if you switch the two arrays (loop through $haystack and compare it to $match) then the logic will work out.

@Sansui ~ Since the redirects are evaluated in the order they are entered, you can always enter the one without the "?" last. It will be at the bottom of the list so the others have a chance to get picked first. Not ideal but it works.

Ok looking into this not being able to handle this properly...

Alrighty, here's the backport of the improvements I've done to selecting a redirect by source from the Redirect module for Drupal 7 which should fix this bug. Even added tests that with the bug fix part reverted, confirms I get 1 error. Tested and patched against 6.x-1.x-dev.

Revised patch after testing changes were committed.

Ok I feel good about #10 so I committed it to CVS. http://drupal.org/cvs?commit=457704

Would be great if everyone having this problem could re-check the 6.x-1.x-dev version in about 12 hours and re-confirm that it's fixed for you as well.