Make WordPress Core

Changeset 54920


Ignore:
Timestamp:
12/01/2022 03:41:12 PM (18 months ago)
Author:
SergeyBiryukov
Message:

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.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/pluggable.php

    r54895 r54920  
    28682868        }
    28692869
    2870         // Add `loading` attribute.
     2870        // Add `loading` and `decoding` attributes.
    28712871        $extra_attr = $args['extra_attr'];
    2872         $loading    = $args['loading'];
    2873 
    2874         if ( in_array( $loading, array( 'lazy', 'eager' ), true ) && ! preg_match( '/\bloading\s*=/', $extra_attr ) ) {
     2872
     2873        if ( in_array( $args['loading'], array( 'lazy', 'eager' ), true )
     2874            && ! preg_match( '/\bloading\s*=/', $extra_attr )
     2875        ) {
    28752876            if ( ! empty( $extra_attr ) ) {
    28762877                $extra_attr .= ' ';
    28772878            }
    28782879
    2879             $extra_attr .= "loading='{$loading}'";
    2880         }
    2881 
    2882         if ( in_array( $args['decoding'], array( 'async', 'sync', 'auto' ), true ) && ! preg_match( '/\bdecoding\s*=/', $extra_attr ) ) {
     2880            $extra_attr .= "loading='{$args['loading']}'";
     2881        }
     2882
     2883        if ( in_array( $args['decoding'], array( 'async', 'sync', 'auto' ), true )
     2884            && ! preg_match( '/\bdecoding\s*=/', $extra_attr )
     2885        ) {
    28832886            if ( ! empty( $extra_attr ) ) {
    28842887                $extra_attr .= ' ';
    28852888            }
     2889
    28862890            $extra_attr .= "decoding='{$args['decoding']}'";
    28872891        }
Note: See TracChangeset for help on using the changeset viewer.