[wp-trac] [WordPress Trac] #16282: PHP catchable error with get_term_link and WP3.1RC2

WordPress Trac wp-trac at lists.automattic.com
Sun Jan 23 23:51:11 UTC 2011


#16282: PHP catchable error with get_term_link and WP3.1RC2
------------------------------+-----------------------
 Reporter:  illutic           |       Owner:
     Type:  defect (bug)      |      Status:  reopened
 Priority:  normal            |   Milestone:  3.1
Component:  Multisite         |     Version:  3.1
 Severity:  major             |  Resolution:
 Keywords:  has-patch commit  |
------------------------------+-----------------------

Comment (by dd32):

 My vote is striping out sanitize_title_for_query() from get_term_by(). In
 this case it's encoding the characters instead of converting them to what
 we use in slugs (ie. 'België' to  'belgi%c3%ab' )

 Ideally, `sanitize_title_for_query()` should be replaced with
 `sanitize_title()` in that spot, This is the same function which sanitizes
 Term names into Term Slugs, so it'll have the desired effect of converting
 'België' to 'belgie'.

 This doesn't change the fact the example code was 'broken' to start with,
 as if the slug differs from the sanitized-title it'll not work, But this
 is a regression with a simple and (IMO, more correct) solution than that
 is currently in core.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/16282#comment:29>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list