previous_posts_link() mangles query string

[markjaquith]

Using previous_posts_link() from http://example.com/?s=test&paged=2 will give you a link of:

http://example.com/?s=test#038;paged=1

next_posts_link() is not afflicted.

We'll want this in 2.2.1 as well.

In trunk, with cruft free links, I get stuff like this:

 http://foo.blog/page/3/?s=test

That's not right. Maybe we should revert back to pre [5454] to fix the trunk problems. Anyhow, that's a different issue.

(In [5702]) remove &paged=1 *before* we HTML entitize the link. fixes #4462

Yeah, my mistake, this only affected 2.2.x

Created #4463 for the bug Ryan noticed.

Still not right... missing the question mark.

Bugs are like onions, they have layers. This was fixed after all, it just uncovered another issue.

See #4464

(In [5703]) only strip question marks from the RIGHT side of the query string. fixes #4464. see #4462