#1357 closed defect (bug) (fixed)
/page/1/ is redundant information
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | trivial | Version: | 1.5.1 |
Component: | Template | Keywords: | bg|has-patch bg|commit |
Focuses: | Cc: |
Description
in the next/prev posts and post page links, paged=1 is redudant information.
Attachments (1)
Change History (8)
#3
@
20 years ago
i think it could cause damage, too: odds are googlebot might consider this a page with two addresses, i.e. spam
#4
@
20 years ago
- Patch changed from No to 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."
#5
@
20 years ago
- Keywords bg|has-patch bg|commit added
- Milestone set to 1.6
- Owner changed from anonymous to markjaquith
- Status changed from new to assigned
Note: See
TracTickets for help on using
tickets.
Agreed.