wp_audio_shortcode() and wp_video_shortcode() ignoring input $attr['id']

[thorbrink]

wp_audio_shortcode() and wp_video_shortcode() both takes an id as a part of the first input parameter. Both of these functions ignore the id input and sets the value of the id-attribute to default value.

[wonderboymusic]

Hi thorbrink, this is unlikely to change ... the id is set dynamically so that it is guaranteed to be unique. If [audio|video] were to take an id param, it would probably represent an attachment's id in the database. Currently, the implementation is URL-based. This allows you to write the shortcodes by hand and easily point at an asset stored on a CDN. It is also difficult to associate sub-attachments through our current UI by linking attachment IDs in a parent-child relationship.

I'll leave this open in case you or others can come up with a clever idea to support custom HTML IDs.

[aaroncampbell]

I agree with @wonderboymusic here, but would add that we should fix the documentation on those functions so that we aren't telling people they can pass in an ID.

[wonderboymusic]

oh man, I missed that docs error - fixing now, good catch!

[wonderboymusic]

In 33025:

wp_audio|video_shortcode() doesn't allow you to pass id, the docs shall reflect that.

Props thorbrink.
See #32855.

[wonderboymusic]

The docs have been fixed - this should alleviate the confusion here. I would say wontfix for now on the id attribute, which is unused by JS anyways.

[aaroncampbell]

@wonderboymusic I was going to make a patch as soon as I got a coffee refill. No time to recaffeinate with you around!

[thorbrink]

Thanks @wonderboymusic. I reported, prepared a patch and went to the train, didn't even get to the station before you had it fixed. Cheers