[wp-trac] [WordPress Trac] #17571: bad double quote replacement in wp_texturize

WordPress Trac wp-trac at lists.automattic.com
Fri Feb 3 22:28:37 UTC 2012


#17571: bad double quote replacement  in wp_texturize
----------------------------------------+------------------------------
 Reporter:  csanquer                    |       Owner:
     Type:  defect (bug)                |      Status:  new
 Priority:  normal                      |   Milestone:  Awaiting Review
Component:  General                     |     Version:
 Severity:  minor                       |  Resolution:
 Keywords:  needs-unit-tests has-patch  |
----------------------------------------+------------------------------
Changes (by kurtpayne):

 * keywords:  needs-patch needs-unit-tests => needs-unit-tests has-patch


Comment:

 Replying to [comment:6 kurtpayne]:
 > This was invalidated by #19602.

 I was wrong about this.  This ticket has 2 example strings given that
 produce undesired output.  One is HTML (`"<a href="...">link</a>"`) and
 one is escaped quotes (`double \"quotes\"`).

 This patch will not work with the HTML input.  There is a separate ticket
 for this at #4539.

 This patch will work with the escaped input.  The patch has been refreshed
 for 3.4 and the unit tests pass, too.

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


More information about the wp-trac mailing list