Make WordPress Core

Opened 17 years ago

Closed 15 years ago

#9492 closed defect (bug) (wontfix)

IRI Encoding of slugs is broken

Reported by: hakre Owned by: nbachiyski
Priority: normal Milestone:
Component: I18N Version: 2.7.1
Severity: normal Keywords: needs-patch
Cc: Focuses:

Description

According to ryan, WordPress does "standard IRI-to-URI conversion for slugs." http://core.trac.wordpress.org/ticket/9480#comment:2

If that is true (what I assume), this feature is broken. Just tested the Post Title "Äpfel wachsen überirdisch" and it is converted to "apfel-wachsen-uberirdisch".

That is not the correct IRI encoding. Properly IRI encoded, the string must be converted to: "%c3%84pfel%20wachsen%20%c3%bcberirdisch".

Change History (9)

#1 @ryan
17 years ago

We decompose accented Latin characters to their unaccented forms. See remove_accents().

#2 @hakre
17 years ago

Well, that's irritating. So infact the the slug is partly IRI encoded? Is next to remove_accents() other stuff made? Why is that done anyway, the IRI stuff looks very good.

#3 @hakre
17 years ago

Well obviously spaces are replaced with minus-signs as well.

#4 @ryan
17 years ago

  • Milestone 2.7.2
  • Resolutionwontfix
  • Status newclosed

It's done because people like it. Regardless, changing how slugs are encoded breaks resolution of old slugs. The current method won't change until we have a means of resolving slugs made with different versions of the sanitizer.

#5 @hakre
17 years ago

sanitizer = sanitize_title();

#6 @hakre
17 years ago

Related: #12361.

#7 @texttheater
15 years ago

One can live with this as long as it's possible to fix it via the plugin API. Which it isn't anymore in 3.1, see #16905.

#8 @texttheater
15 years ago

  • Cc poststelle@… added
  • Resolution wontfix
  • Status closedreopened

#9 @texttheater
15 years ago

  • Resolutionwontfix
  • Status reopenedclosed

Sorry, I just wanted to add myself to Cc.

Note: See TracTickets for help on using tickets.