[wp-trac] [WordPress Trac] #32118: SCRIPT_DEBUG check in print_emoji_detection_script() generating PHP Notices

[WordPress Trac]

#32118: SCRIPT_DEBUG check in  print_emoji_detection_script() generating PHP
Notices
----------------------------+----------------------------
 Reporter:  aut0poietic     |       Owner:  craig-ralston
     Type:  defect (bug)    |      Status:  assigned
 Priority:  normal          |   Milestone:  4.2.1
Component:  Formatting      |     Version:  4.2
 Severity:  normal          |  Resolution:
 Keywords:  good-first-bug  |     Focuses:
----------------------------+----------------------------
Changes (by azaozz):

 * keywords:  good-first-bug has-patch => good-first-bug


Comment:

 32118.diff is not going to work properly. See how it is defined now in
 script-loader.php: https://core.trac.wordpress.org/browser/tags/4.2/src
 /wp-includes/script-loader.php#L51 (defaults to `true` when running from
 /src).

 The question is: can this possibly conflict with something a plugin would
 do if we move it to wp_initial_constants()?

--
Ticket URL: <https://core.trac.wordpress.org/ticket/32118#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform