#1357 closed defect (bug) (fixed)
/page/1/ is redundant information
| Reported by: | Denis de Bernardy | Owned by: | markjaquith |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Template | Version: | 1.5.1 |
| Severity: | trivial | Keywords: | bg|has-patch bg|commit |
| Cc: | Focuses: |
Description
in the next/prev posts and post page links, paged=1 is redudant information.
Attachments (1)
Change History (8)
#3
@
21 years ago
i think it could cause damage, too: odds are googlebot might consider this a page with two addresses, i.e. spam
#4
@
21 years ago
- Patch No → Yes
The patch I uploaded is dependent on this patch:
http://mosquito.wordpress.org/view.php?id=1362
Otherwise, use $qstr = add_query_arg('paged', , $qstr); instead of remove_query_arg(), which is useless without the patch on 1362.
I did basic testing with mod_rewrite/query style setups and it seemed to work, but I'd appreciate testing. Basically, if $pagenum === 1, it does str_replace('page/1/', , $qstr); and remove_query_ary('paged', $qstr); to remove both the mod_rewrite and query style versions of the "first page."
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Agreed.