Opened 16 years ago
Last modified 3 months ago
#4463 reviewing defect (bug)
Pretty permalink on search
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Future Release | Priority: | normal |
Severity: | normal | Version: | 2.3 |
Component: | Permalinks | Keywords: | has-patch needs-unit-tests |
Focuses: | Cc: |
Attachments (2)
Change History (24)
#3
@
14 years ago
- Component changed from General to Template
- Priority changed from high to normal
What would be a suggested fix for this ticket? /search/search-term/page/2 ? /?s=search-term&paged=2 ?
#5
@
14 years ago
- Component changed from Template to Permalinks
- Owner changed from anonymous to ryan
#6
@
14 years ago
/search/search-term/page/2 with a configureable option in backend for wording of "search" would be pretty nice to fillup serps.
anyway isn't this pretty uninteresting anyway as long as it doesn't lead to a 404?
#9
@
11 years ago
- Keywords needs-patch added; reporter-feedback removed
Paged search links still look as described: http://trunk.wordpress/page/2/?s=test
.
Before [5454], they used to be http://trunk.wordpress/?s=test&paged=2
.
#10
@
11 years ago
- Keywords has-patch added; needs-patch removed
Makes search links with pretty permalinks on used paged in the query string
#11
@
10 years ago
- Milestone changed from Future Release to 3.6
Refreshed patch against trunk - this ticket is like 19,000 tickets old, but the patch works
#14
@
10 years ago
- Cc mdhansen@… added
Both 4463.diff and 4463_1.diff apply and work as intended. In 4463_1.diff there is a couple addition lines for is_single ! is_home and ! is_front_page. I am not sure if that is necessary as 4463.diff seems to work without it.
#15
@
10 years ago
- Keywords 3.7-early added
- Milestone changed from 3.6 to Future Release
Longstanding bug.
#20
@
8 years ago
- Keywords 3.7-early removed
- Milestone changed from Future Release to 4.4
- Owner set to chriscct7
- Status changed from assigned to reviewing
#22
@
3 months ago
- Summary changed from Strange paging links to Pretty permalink on search
Just adding a more context focused title to avoid reading all the history in this 16 years ticket.
I don't know if it is something that we want, I mean now it is a standard that on wordpress the only parameter not pretty
is s
for the search.
The topic was opened because at the time there were some changes on adding as pretty the paged
value.
As today, all the search engine doesn't have pretty permalinks for the search string value like Google.
Tested with 7236.
Whenever performing searches, the search results do have URLs in the form of /?s=test.