[wp-trac] Re: [WordPress Trac] #4739: Some icelandic/Norwegian/Danish letters do not work in page slugs

WordPress Trac wp-trac at lists.automattic.com
Thu Dec 11 03:13:13 GMT 2008


#4739: Some icelandic/Norwegian/Danish letters do not work in page slugs
--------------------------------------------------------+-------------------
 Reporter:  einare                                      |        Owner:  westi   
     Type:  defect                                      |       Status:  reopened
 Priority:  high                                        |    Milestone:  2.8     
Component:  Permalinks                                  |      Version:  2.2.1   
 Severity:  major                                       |   Resolution:          
 Keywords:  needs-patch early 2nd-opinion dev-feedback  |  
--------------------------------------------------------+-------------------
Changes (by janbrasna):

 * cc: janbrasna (added)
  * keywords:  needs-patch early => needs-patch early 2nd-opinion dev-
               feedback
  * component:  i18n => Permalinks
  * milestone:  2.9 => 2.8

Comment:

 It used to work in non–UTF processing at some point in the past (see
 http://trac.wordpress.org/browser/trunk/wp-
 includes/formatting.php?rev=10150#L401 for the old Latin 1 transliteration
 code) but was apparently omitted when the UTF transliteration segment was
 written.

 Anyway the main problem is the {{{sanitize_title}}} line
 http://trac.wordpress.org/browser/trunk/wp-
 includes/query.php?rev=10150#L1671
 that makes it effectively impossible to change the transliteration array
 at any point. I can't seem to find the point in comment:11 because the
 post_name field in the DB is always matched and regarding comment:5 I
 think the {{{sanitize_title}}} at that point is maybe overboard? Wouldn't
 just escaping it for SQL be enough? Otherwise, comment:7 sounds fine for
 that matter.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/4739#comment:19>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list