[wp-trac] [WordPress Trac] #7041: Strip \r on js escape

WordPress Trac wp-trac at lists.automattic.com
Mon May 26 14:40:17 GMT 2008


#7041: Strip \r on js escape
------------------------+---------------------------------------------------
 Reporter:  nbachiyski  |       Owner:  anonymous        
     Type:  defect      |      Status:  new              
 Priority:  normal      |   Milestone:  2.6              
Component:  i18n        |     Version:                   
 Severity:  normal      |    Keywords:  js i18n has-patch
------------------------+---------------------------------------------------
 Now in the {{{js_escape}}} function, {{{\r}}} is stripped only if it is
 before {{{\n}}}.

 I have seen many PO editors to insert {{{\r}}} in the end of translations,
 which breaks javascript.

 I propose that we filter {{{\r}}} out of javascript strings. A patch is
 attached and tested.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/7041>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list