WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#16999 closed defect (bug) (fixed)

BUG: child pages slug with non-english charecters

Reported by: ramiy's profile ramiy Owned by: ramiy's profile ramiy
Milestone: 3.2 Priority: normal
Severity: major Version: 3.1.3
Component: Administration Keywords: has-patch
Focuses: Cc:

Description (last modified by yoavf)

(Related #10966 - same problem in other admin pages)

I found one more bug when creating a child page using slug with non-english characters.

I added a screenshot from a client site using WordPress in hebrew (RTL language).

To recreate the problem in an english installation (LTR language), create a new page called "internationalizætiøn-parent", and then create a child page called "internationalizætiøn-child".

Attachments (3)

slug-bug.png​ (65.6 KB) - added by ramiy 14 years ago.
wrong parent slug
16999.diff​ (510 bytes) - added by solarissmoke 14 years ago.
Pass parent page slug(s) through editable_slug filter before inserting them into the permalink
bbPress-problem.PNG​ (42.6 KB) - added by ramiy 14 years ago.
Better visual example using bbPress plugin

Download all attachments as: .zip

Change History (13)

@ramiy
14 years ago

wrong parent slug

#1 @ramiy
14 years ago

Related #6915

#2 @solarissmoke
14 years ago

  • Keywords has-patch added; needs-patch needs-ui removed

@solarissmoke
14 years ago

Pass parent page slug(s) through editable_slug filter before inserting them into the permalink

#3 @ramiy
14 years ago

  • Owner set to ramiy
  • Status changed from new to accepted

Patch tested - it's working.
Thanks!

#4 @yoavf
14 years ago

  • Cc yoavf added

@ramiy
14 years ago

Better visual example using bbPress plugin

#5 @ramiy
14 years ago

  • Version changed from 3.1 to 3.1.3

On the bbPress context this ticket related to #17598. Both tickets need to be fixed before the final release of bbPress.

#6 @yoavf
14 years ago

  • Description modified (diff)

solarissmoke's patch tested, works (small offset when applying, but no problem other than that).

#7 @nacin
14 years ago

  • Milestone changed from Awaiting Review to 3.2

#8 @ryan
14 years ago

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

In [18105]:

Apply editable_slug filter to parent page slugs in get_sample_permalink(). Props solarissmoke, ramiy. fixes #16999

#9 @ramiy
14 years ago

UPDATE: tested on wp3.2-RC1

The problem fixed for parent/child pages, but not for the bbpress plugin when using non-english forum slug.

#10 @johnjamesjacoby
14 years ago

Will investigate. Thanks for cross posting.

Note: See TracTickets for help on using tickets.