Media: Only add `aria-describedby` in `img_caption_shortcode()` if a non-empty attachment ID attribute was provided.

See #34595.

git-svn-id: https://develop.svn.wordpress.org/trunk@42693 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Sergey Biryukov 2018-02-11 15:19:58 +00:00
parent b66da520b3
commit bff366759c
1 changed files with 24 additions and 15 deletions

View File

@ -1543,11 +1543,12 @@ function img_caption_shortcode( $attr, $content = null ) {
$atts = shortcode_atts(
array(
'id' => '',
'align' => 'alignnone',
'width' => '',
'caption' => '',
'class' => '',
'id' => '',
'caption_id' => '',
'align' => 'alignnone',
'width' => '',
'caption' => '',
'class' => '',
), $attr, 'caption'
);
@ -1556,12 +1557,20 @@ function img_caption_shortcode( $attr, $content = null ) {
return $content;
}
$caption_id = '';
if ( ! empty( $atts['id'] ) ) {
if ( $atts['id'] ) {
$att_id = esc_attr( sanitize_html_class( $atts['id'] ) );
$atts['id'] = 'id="' . $att_id . '" ';
$caption_id = 'caption-' . str_replace( '_', '-', $att_id );
if ( ! $atts['caption_id'] ) {
$atts['caption_id'] = 'caption-' . str_replace( '_', '-', $att_id );
}
}
$describedby = '';
if ( $atts['caption_id'] ) {
$describedby = 'aria-describedby="' . $atts['caption_id'] . '" ';
$atts['caption_id'] = 'id="' . $atts['caption_id'] . '" ';
}
$class = trim( 'wp-caption ' . $atts['align'] . ' ' . $atts['class'] );
@ -1594,15 +1603,15 @@ function img_caption_shortcode( $attr, $content = null ) {
if ( $html5 ) {
$html = sprintf(
'<figure %s%sclass="%s" aria-describedby="%s">%s%s</figure>',
'<figure %s%s%sclass="%s">%s%s</figure>',
$atts['id'],
$describedby,
$style,
esc_attr( $class ),
$caption_id,
do_shortcode( $content ),
sprintf(
'<figcaption id="%s" class="wp-caption-text">%s</figcaption>',
$caption_id,
'<figcaption %sclass="wp-caption-text">%s</figcaption>',
$atts['caption_id'],
$atts['caption']
)
);
@ -1612,10 +1621,10 @@ function img_caption_shortcode( $attr, $content = null ) {
$atts['id'],
$style,
esc_attr( $class ),
str_replace( '<img ', '<img aria-describedby="' . $caption_id . '" ', do_shortcode( $content ) ),
str_replace( '<img ', '<img ' . $describedby, do_shortcode( $content ) ),
sprintf(
'<p id="%s" class="wp-caption-text">%s</p>',
$caption_id,
$atts['caption_id'],
$atts['caption']
)
);