Make WordPress Core

Opened 12 years ago

Closed 8 years ago

#21167 closed enhancement (wontfix)

Problem with custom permalinks

Reported by: kaffeeringe's profile kaffeeringe Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.4
Component: Permalinks Keywords: needs-patch dev-feedback
Focuses: Cc:


I am currently moving my blog ( to Wordpress and I don't want to break my old URLs (Cool URLs don't change). The problem is, that these custom URLs don't work in Wordpress 3.4.1:

/blog/%postname%/%post_id%.html works
/blog/%postname%.%post_id%.html (which I need) doesn't work.

It doesn't work with the . in between these parameters…

I hope somebody is able to help me.

Attachments (4)

21167.patch (667 bytes) - added by SergeyBiryukov 12 years ago.
21167.2.patch (502 bytes) - added by SergeyBiryukov 12 years ago.
21167-unit-test.patch (3.1 KB) - added by kurtpayne 12 years ago.
Unit test
21167-tests.patch (3.1 KB) - added by boonebgorges 9 years ago.
Removing from trunk as per #30284

Download all attachments as: .zip

Change History (31)

#1 @Ipstenu
12 years ago

  • Keywords close added
  • Resolution set to invalid
  • Status changed from new to closed

This isn't a bug - Trac is for submitted bugs in WordPress :)

You should post in the WordPress forums instead (and I would in this case use a .htaccess to parse /blog/%postname%.%post_id%.html into /blog/%postname%/%post_id%/ (or anything else you wanted) if you need more help.

#2 @helenyhou
12 years ago

  • Milestone Awaiting Review deleted

#3 @nacin
12 years ago

  • Resolution invalid deleted
  • Status changed from closed to reopened

Sounds like a bug to me. We allow random characters in a URL permalink structure, like ".html". Would be interesting to see where and why this fails.

#4 @helenyhou
12 years ago

  • Keywords close removed
  • Milestone set to Awaiting Review

#5 @nacin
12 years ago

  • Type changed from defect (bug) to enhancement

Looks like our support for this is quite tenuous. Many combinations work, but mostly by happenstance.

#6 @Ipstenu
12 years ago

/%postname%.%post_id%/ fails

/%year%.%monthnum%.%postname%/ works

/%post_id%.%postname%/ works

/%postname%.%year%/ works

Well now that's just odd...

Last edited 12 years ago by Ipstenu (previous) (diff)

#7 @sirzooro
12 years ago

  • Cc sirzooro added

I checked this and found that dot is not escaped when generating rules, e.g.:

  '([^/]+).([0-9]+)(/[0-9]+)?/?$' => 'index.php?name=$1&p=$2&page=$3'

Other special RX chars may also cause problems here - need to check if patch will work with them too.

#8 @SergeyBiryukov
12 years ago

  • Keywords has-patch added

21167.patch is an attempt to escape the special characters which are currently allowed in permalink structure and are acceptable in terms of valid URLs.

Seems to work in the following cases:


#9 follow-up: @dd32
12 years ago

Is there any reason why we can't just run the permastructure through preg_quote()?

> preg_quote( '.+/()$[]%' );

#10 in reply to: ↑ 9 @SergeyBiryukov
12 years ago

Replying to dd32:

Is there any reason why we can't just run the permastructure through preg_quote()?

Thanks, that indeed makes more sense.

#11 @SergeyBiryukov
12 years ago

  • Keywords needs-unit-tests added
  • Milestone changed from Awaiting Review to 3.5

12 years ago

Unit test

#12 @kurtpayne
12 years ago

  • Cc kpayne@… added

Unit test added, 21167.2.patch looks good. Let me know if there are any test cases missing.

#13 @tlovett1
12 years ago

  • Owner set to tlovett1
  • Status changed from reopened to accepted

#14 @tlovett1
12 years ago

  • Owner tlovett1 deleted
  • Status changed from accepted to assigned

#15 @nacin
11 years ago

  • Keywords commit added

#16 @wonderboymusic
11 years ago

@kurtpayne - was that unit test patch committed?

#17 @SergeyBiryukov
11 years ago

  • Keywords needs-unit-tests removed

#19 @ryan
11 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

#20 @xknown
11 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

The fix is probably incomplete. We should be passing the second argument on preg_quote and use the same delimiter where rewrite rules are being used.

For example, having the following permalink structure "/%post_id%#foo" (yes, probably unrealistic) would produce an invalid regular expression in wp-includes/class-wp.php, where # is used as a delimiter.

#21 @ryan
11 years ago

  • Keywords has-patch commit removed

#22 @nacin
11 years ago

Seems to me like this should be preg_quote()'d before getting used in a regexp, rather than on save.

#23 @jeremyfelt
11 years ago

Using preg_quote() breaks rewrites for post types with hyphens. I'm not entirely sure how supported that is to begin with, but I know I need to convert a few now. :)

#24 @ryan
11 years ago

In 22403:

Revert [22365]. see #21167

#25 @ryan
11 years ago

  • Milestone changed from 3.5 to Future Release

9 years ago

Removing from trunk as per #30284

#26 @chriscct7
9 years ago

  • Keywords needs-patch dev-feedback added

#27 @johnbillion
8 years ago

  • Milestone Future Release deleted
  • Resolution set to wontfix
  • Status changed from reopened to closed

No interest in three years. Feel free to re-open with a patch/tests.

Note: See TracTickets for help on using tickets.