#4462 closed defect (bug) (fixed)
previous_posts_link() mangles query string
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | high | Milestone: | 2.2.1 |
| Component: | Template | Version: | 2.2 |
| Severity: | major | Keywords: | |
| Cc: |
Description
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.
Change History (7)
comment:3
markjaquith — 6 years ago
- Resolution set to fixed
- Status changed from new to closed
comment:4
markjaquith — 6 years ago
Yeah, my mistake, this only affected 2.2.x
Created #4463 for the bug Ryan noticed.
comment:5
markjaquith — 6 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Still not right... missing the question mark.
comment:6
markjaquith — 6 years ago
- Resolution set to fixed
- Status changed from reopened to closed
Bugs are like onions, they have layers. This was fixed after all, it just uncovered another issue.
See #4464
Note: See
TracTickets for help on using
tickets.

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.