[wp-trac] [WordPress Trac] #40179: 404 for two emojis

WordPress Trac noreply at wordpress.org
Fri Mar 17 10:55:24 UTC 2017


#40179: 404 for two emojis
-------------------------------+--------------------------
 Reporter:  Grzegorz.Janoszka  |       Owner:
     Type:  defect (bug)       |      Status:  closed
 Priority:  normal             |   Milestone:
Component:  Emoji              |     Version:  4.7.3
 Severity:  normal             |  Resolution:  invalid
 Keywords:                     |     Focuses:  performance
-------------------------------+--------------------------

Comment (by SergeyBiryukov):

 Replying to [comment:2 Grzegorz.Janoszka]:
 > Please note that in /wp-includes/js/wp-emoji-release.min.js I see the
 following:
 > {{{
 > var
 p={base:"https://twemoji.maxcdn.com/2/",ext:".png",size:"72x72",className:"emoji",
 > }}}

 Right, that's the default value, but it's only used if no other value is
 specified via [source:tags/4.7.3/src/wp-includes/js/wp-
 emoji.js?marks=166#L155 settings.svgUrl]. WordPress does specify its own
 [source:tags/4.7.3/src/wp-includes/formatting.php?marks=5032#L5025 svgUrl
 value].

 > I actually dug into that and I found one plugin that was meant to speed
 up the website had a code:
 > {{{
 >   # remove <link rel='dns-prefetch' href='//s.w.org' /> from the head:
 >   add_filter('emoji_svg_url','__return_false');
 > }}}
 > Could it be that it actually changes the CDN from s.w.org to maxcdn?

 Yes, that's exactly what it does. If the goal is just to remove the `dns-
 prefetch` resource hint,
 [https://developer.wordpress.org/reference/hooks/wp_resource_hints/
 wp_resource_hints] filter should have been used for that.

 > And if yes - there seems to be some replication issues as some files are
 missing.

 For reference, the missing files are:
 * `slightly frowning face`:
 http://www.fileformat.info/info/unicode/char/1f641/index.htm
 * `slightly smiling face`:
 http://www.fileformat.info/info/unicode/char/1f642/index.htm

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


More information about the wp-trac mailing list