Make WordPress Core

Opened 3 years ago

Closed 2 months ago

#21167 closed enhancement (wontfix)

Problem with custom permalinks

Reported by: 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 (http://kaffeeringe.de/) 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 3 years ago.
21167.2.patch (502 bytes) - added by SergeyBiryukov 3 years ago.
21167-unit-test.patch (3.1 KB) - added by kurtpayne 3 years ago.
Unit test
21167-tests.patch (3.1 KB) - added by boonebgorges 13 months ago.
Removing from trunk as per #30284

Download all attachments as: .zip

Change History (31)

#1 @Ipstenu
3 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)

http://wordpress.org/support if you need more help.

#2 @helenyhou
3 years ago

  • Milestone Awaiting Review deleted

#3 @nacin
3 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
3 years ago

  • Keywords close removed
  • Milestone set to Awaiting Review

#5 @nacin
3 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
3 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 3 years ago by Ipstenu (previous) (diff)

#7 @sirzooro
3 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
3 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
3 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
3 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
3 years ago

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

3 years ago

Unit test

#12 @kurtpayne
3 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
3 years ago

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

#14 @tlovett1
3 years ago

  • Owner tlovett1 deleted
  • Status changed from accepted to assigned

#15 @nacin
3 years ago

  • Keywords commit added

#16 @wonderboymusic
3 years ago

@kurtpayne - was that unit test patch committed?

#17 @SergeyBiryukov
3 years ago

  • Keywords needs-unit-tests removed

#19 @ryan
3 years ago

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

#20 @xknown
3 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
3 years ago

  • Keywords has-patch commit removed

#22 @nacin
3 years ago

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

#23 @jeremyfelt
3 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
3 years ago

In 22403:

Revert [22365]. see #21167

#25 @ryan
3 years ago

  • Milestone changed from 3.5 to Future Release

13 months ago

Removing from trunk as per #30284

#26 @chriscct7
3 months ago

  • Keywords needs-patch dev-feedback added

#27 @johnbillion
2 months 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.