Unwanted backslash in output

[donpaolo]

After upgrading to 3.3 I got the windows notificating me the new features.

the "Media uploader updated" notification says in italian translation:

Ora l\'icona unificata "media" lancia un uploader per tutti i tipi di file, e la nuova interfaccia "drag and drop" rende il caricamento un gioco da ragazzi.

You see there is a backslash which shouldn't be there. I think it's not a translation error, but a bug in the treating of those strings.

[ocean90]

The translation is:

Ora l'icona unificata \"media\" lancia un uploader per tutti i tipi di file, e la nuova interfaccia \"drag and drop\" rende il caricamento un gioco da ragazzi.

[ocean90]

The problem is in json_encode( esc_js( ) ):

$string = "Foo'bar \"Hello\"";
var_dump( ( esc_js( $string ) ); // string(26) "Foo\'bar "Hello""
var_dump( json_encode( esc_js( $string ) ) ); // string(29) ""Foo\\'bar "Hello"""
var_dump( json_encode( $string ); // string(19) ""Foo'bar \"Hello\"""
var_dump( esc_js( json_encode( $string ) ) ); // string(38) ""Foo\'bar "Hello"""

[ocean90]

19644.patch​ does remove the esc_js calls.

[nacin]

Works for me, I think esc_js() was there from before we started to take the arguments and pass them to json_encode().

[nacin]

In [19655]:

Remove esc_js() as this content now gets json encoded rather than a straight echo. props ocean90. see #19644, for trunk.

[ryan]

In [19659]:

Remove esc_js() as this content now gets json encoded rather than a straight echo. props ocean90. fixes #19644, for 3.3