[wp-trac] [WordPress Trac] #11520: print_scripts_l10n() should use json_encode()

WordPress Trac wp-trac at lists.automattic.com
Sat Aug 13 17:48:12 UTC 2011


#11520: print_scripts_l10n() should use json_encode()
------------------------------------+------------------------
 Reporter:  scribu                  |       Owner:  azaozz
     Type:  enhancement             |      Status:  reviewing
 Priority:  normal                  |   Milestone:  3.3
Component:  JavaScript              |     Version:
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+------------------------

Comment (by nacin):

 Daryl mentioned that wp_localize_script() escapes data differently than
 json_encode(). We may need to resurrect it, and create
 wp_add_script_data() as a new way to do this.

 I'm not sure I like [18496].

 Removing l10n.js is probably guaranteed to trigger fatal errors in a
 plugin that is trying to use it (such as on the frontend). Mind you, they
 might be calling convertEntities(). Also, we didn't update all of the
 references to it, such as in install.php.

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


More information about the wp-trac mailing list