WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 15 years ago

Last modified 15 years ago

#16999 closed defect (bug) (fixed)

BUG: child pages slug with non-english charecters

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

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 16 years ago.
wrong parent slug
16999.diff​ (510 bytes ) - added by solarissmoke 16 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 15 years ago.
Better visual example using bbPress plugin

Download all attachments as: .zip

Change History (13)

@ramiy
16 years ago

wrong parent slug

#1 @ramiy
16 years ago

Related #6915

#2 @solarissmoke
16 years ago

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

@solarissmoke
16 years ago

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

#3 @ramiy
16 years ago

  • Owner set to ramiy
  • Status new → accepted

Patch tested - it's working.
Thanks!

#4 @yoavf
16 years ago

  • Cc yoavf added

@ramiy
15 years ago

Better visual example using bbPress plugin

#5 @ramiy
15 years ago

  • Version 3.1 → 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
15 years ago

  • Description modified (diff)

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

#7 @nacin
15 years ago

  • Milestone Awaiting Review → 3.2

#8 @ryan
15 years ago

  • Resolution → fixed
  • Status accepted → closed

In [18105]:

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

#9 @ramiy
15 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
15 years ago

Will investigate. Thanks for cross posting.

Note: See TracTickets for help on using tickets.