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

WordPress Trac wp-trac at lists.automattic.com
Mon Jan 24 02:48:02 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):

 > The correct solution would be to have two versions of get_term_by(): one
 for WP_Query, that uses sanitize_title_for_query() and one for
 get_term_link() which uses sanitize_title() nothing.

 After reading that ticket earlier, It just made me even more confused why
 you replaced it with sanitize_title_for_query() in that spot.

 In the case of Term slugs, µ is sanitized to u (as sanitize_title does on
 pre_term_save), not sanitized to '%c2%b5' like sanitize_title_for_query().

 Perhaps, it's just that sanitize_title_for_query() needs to use
 remove_accents() as well.

 What does that changeset actually allow? Looking at that ticket, that
 commit was after it was decided it was 3.2 material, and being that it's
 caused this regression..

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

More information about the wp-trac mailing list