?author=-N forwards to wrong canonical URL

[markjaquith]

/feed/?author=-123 will redirect to the feed for that author.

Expected: No forwarding, stay on that URL and exclude that author.

I'm on it.

[markjaquith]

(In [12034]) Do not do a canonical redirect for negated author query strings. fixes #10960

[aaroncampbell]

Wouldn't it be far more efficient to use ctype_digit( $_GET['author'] ) rather than a regular expression?

[filosofo]

Actually a better optimization would probably be leaving the regular expression and taking out the ! empty() check (ctype_digit() returns true for empty strings in some PHP versions).

[markjaquith]

(In [12040]) Remove redundant !empty() checks. props filosofo. see #10960