Make WordPress Core

Opened 9 years ago

Last modified 16 months ago

#2877 reopened defect (bug)

A Slash too much @ get_pagenum_link()

Reported by: dave@… Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 2.0.3
Component: Posts, Post Types Keywords: has-patch needs-unit-tests needs-refresh
Focuses: template Cc:


  • File: template-functions-links.php
  • Function: get_pagenum_link()

The following code is at the end of the function:

	// showing /page/1/ or ?paged=1 is redundant
	if ( 1 === $pagenum ) {
		$qstr = str_replace('page/1/', '', $qstr); // for mod_rewrite style
		$qstr = remove_query_arg('paged', $qstr); // for query style

If you work with the mod_rewrite style, you will get this URL for the first page:

This wont work (/ at the end).


	// showing /page/1/ or ?paged=1 is redundant
	if ( 1 === $pagenum ) {
		$qstr = str_replace('/page/1/', '', $qstr); // for mod_rewrite style
		$qstr = remove_query_arg('paged', $qstr); // for query style


  • sry for my bad english...
  • i've downloaded a german translation of wordpress 2.0.3, so maybe its just in this version... ?!

Attachments (1)

2877.diff (1.4 KB) - added by scribu 4 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 @darkfate9 years ago

No, it's that way in the english version. I don't think that would fix it and there would be a problem with that. Putting the slash in front like you did would make the server redirect to root. So if you have wordpress in a directory it would screw up like so:
You would want this:
But you would get this:

comment:2 @Nazgul8 years ago

  • Resolution set to wontfix
  • Status changed from new to closed

No traction in almost a year, so closing as wontfix.

Feel free to re-open it if you have additional information/suggestions/patches/...

comment:3 @scribu4 years ago

  • Keywords needs-patch added
  • Milestone set to Future Release
  • Resolution wontfix deleted
  • Status changed from closed to reopened

Only enhancements or feature requests can be closed when there's no traction.

@scribu4 years ago

comment:4 @scribu4 years ago

  • Keywords has-patch added; needs-patch removed

comment:5 @scribu4 years ago

Note that the problem is with first-page links, not with /page/2, /page/3/ etc.

Original thread: http://wordpress.org/support/topic/plugin-wp-pagenavi-trailing-slash-at-first-page-url-regardless-of-defined-permalink-structure

Last edited 4 years ago by scribu (previous) (diff)

comment:6 @nacin20 months ago

  • Component changed from General to Template

Interesting patch.

comment:7 @nacin20 months ago

  • Keywords needs-unit-tests added

comment:8 @nacin19 months ago

  • Component changed from Template to Posts, Post Types
  • Focuses template added

comment:9 @obenland16 months ago

  • Keywords needs-refresh added
Note: See TracTickets for help on using tickets.