From 5697adcecf651ac2d10eef3ed01adc0efc05af22 Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Thu, 1 Dec 2022 15:41:12 +0000 Subject: [PATCH] Coding Standards: Remove a one-time `$loading` variable in `get_avatar()`. This aims to bring consistency between two similar code fragments. Follow-up to [47554], [53480], [54895]. See #56791. git-svn-id: https://develop.svn.wordpress.org/trunk@54920 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/pluggable.php | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/src/wp-includes/pluggable.php b/src/wp-includes/pluggable.php index 7ed917001b..cf041fe7da 100644 --- a/src/wp-includes/pluggable.php +++ b/src/wp-includes/pluggable.php @@ -2867,22 +2867,26 @@ if ( ! function_exists( 'get_avatar' ) ) : } } - // Add `loading` attribute. + // Add `loading` and `decoding` attributes. $extra_attr = $args['extra_attr']; - $loading = $args['loading']; - if ( in_array( $loading, array( 'lazy', 'eager' ), true ) && ! preg_match( '/\bloading\s*=/', $extra_attr ) ) { + if ( in_array( $args['loading'], array( 'lazy', 'eager' ), true ) + && ! preg_match( '/\bloading\s*=/', $extra_attr ) + ) { if ( ! empty( $extra_attr ) ) { $extra_attr .= ' '; } - $extra_attr .= "loading='{$loading}'"; + $extra_attr .= "loading='{$args['loading']}'"; } - if ( in_array( $args['decoding'], array( 'async', 'sync', 'auto' ), true ) && ! preg_match( '/\bdecoding\s*=/', $extra_attr ) ) { + if ( in_array( $args['decoding'], array( 'async', 'sync', 'auto' ), true ) + && ! preg_match( '/\bdecoding\s*=/', $extra_attr ) + ) { if ( ! empty( $extra_attr ) ) { $extra_attr .= ' '; } + $extra_attr .= "decoding='{$args['decoding']}'"; }