[wp-trac] [WordPress Trac] #63731: wp-emoji-release.min.js Requires URL Update

[WordPress Trac]

#63731: wp-emoji-release.min.js Requires URL Update
--------------------------+-------------------------
 Reporter:  isaumya       |       Owner:  (none)
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:
Component:  Emoji         |     Version:
 Severity:  normal        |  Resolution:  wontfix
 Keywords:                |     Focuses:  javascript
--------------------------+-------------------------
Changes (by peterwilsoncc):

 * status:  new => closed
 * focuses:  ui, javascript => javascript
 * resolution:   => wontfix
 * severity:  major => normal
 * milestone:  Awaiting Review =>


Comment:

 As mentioned above, by default WordPress serves images from s.w.org and
 the base setting in the twemoji script is ignored.

 Testing with `add_filter( 'emoji_svg_url', '__return_false' );` removes
 the full URL and sets the image source to `src="1f426-200d-1f525.svg"`, so
 patching the `base` url in the JavaScript file will make no difference for
 that use case.

 As the code is working without any plugins enabled, I don't think there's
 any value in further patching the version of twemoji included in WordPress
 Core as it's the responsibility of plugins modifying the code to ensure
 they do so in a bug free manner.

 Furthermore, I think attempting to defend against plugin bugs will
 introduce problems as valid urls could be in a wide range of formats.

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