Make WordPress Core


Ignore:
Timestamp:
09/20/2017 07:44:13 PM (7 years ago)
Author:
westonruter
Message:

Widgets: Omit attributes from an Image widget's link when they are empty.

Props subrataemfluence, Nenad Obradovic, westonruter.
See #39993.
Fixes #41919.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/widgets/class-wp-widget-media-image.php

    r41252 r41549  
    241241
    242242        if ( $url ) {
    243             $image = sprintf(
    244                 '<a href="%1$s" class="%2$s" rel="%3$s" target="%4$s">%5$s</a>',
    245                 esc_url( $url ),
    246                 esc_attr( $instance['link_classes'] ),
    247                 esc_attr( $instance['link_rel'] ),
    248                 ! empty( $instance['link_target_blank'] ) ? '_blank' : '',
    249                 $image
    250             );
     243            $link = sprintf( '<a href="%s"', esc_url( $url ) );
     244            if ( ! empty( $instance['link_classes'] ) ) {
     245                $link .= sprintf( ' class="%s"', esc_attr( $instance['link_classes'] ) );
     246            }
     247            if ( ! empty( $instance['link_rel'] ) ) {
     248                $link .= sprintf( ' rel="%s"', esc_attr( $instance['link_rel'] ) );
     249            }
     250            if ( ! empty( $instance['link_target_blank'] ) ) {
     251                $link .= ' target="_blank"';
     252            }
     253            $link .= '>';
     254            $link .= $image;
     255            $link .= '</a>';
     256            $image = $link;
    251257        }
    252258
Note: See TracChangeset for help on using the changeset viewer.