Opened 19 months ago
Closed 16 months ago
#58436 closed defect (bug) (fixed)
Emoji: Extraneous sprintf() present in _print_emoji_detection_script()
Reported by: | westonruter | Owned by: | westonruter |
---|---|---|---|
Milestone: | 6.3 | Priority: | low |
Severity: | trivial | Version: | 5.9 |
Component: | Emoji | Keywords: | has-patch |
Focuses: | Cc: |
Description
In r52132 for #44632 I erroneously included a sprintf()
without any arguments:
// src/wp-includes/embed.php
$output = wp_get_inline_script_tag(
file_get_contents( sprintf( ABSPATH . WPINC . '/js/wp-embed' . wp_scripts_get_suffix() . '.js' ) )
);
And:
// src/wp-includes/formatting.php
wp_print_inline_script_tag(
sprintf( 'window._wpemojiSettings = %s;', wp_json_encode( $settings ) ) .
file_get_contents( sprintf( ABSPATH . WPINC . '/js/wp-emoji-loader' . wp_scripts_get_suffix() . '.js' ) )
);
I fixed the former case in r52151 but I missed the latter.
Change History (5)
This ticket was mentioned in PR #4529 on WordPress/wordpress-develop by @westonruter.
19 months ago
#1
- Keywords has-patch added
#2
@
19 months ago
- Owner set to westonruter
- Resolution set to fixed
- Status changed from new to closed
In 55875:
#3
@
19 months ago
- Keywords fixed-major added
- Resolution fixed deleted
- Status changed from closed to reopened
Reopening for 6.2.3 consideration.
@westonruter commented on PR #4529:
19 months ago
#4
Committed to trunk
in r55875.
Note: See
TracTickets for help on using
tickets.
Trac ticket: https://core.trac.wordpress.org/ticket/58436