[wp-trac] [WordPress Trac] #63583: Issue when having multiple Audio and/or Video playlists at the same page

[WordPress Trac]

#63583: Issue when having multiple Audio and/or Video playlists at the same page
-------------------------------------------------+-------------------------
 Reporter:  Guido07111975                        |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
                                                 |  Review
Component:  Embeds                               |     Version:  6.8
 Severity:  normal                               |  Resolution:
 Keywords:  has-test-info has-screenshots has-   |     Focuses:
  patch needs-testing                            |
-------------------------------------------------+-------------------------
Changes (by rollybueno):

 * keywords:  has-test-info has-screenshots has-patch => has-test-info has-
     screenshots has-patch needs-testing


Comment:

 Yes, correct. We have patch 1 minute after your comment, so adding proper
 keywords. I'll check if there's existing test unit.

 Replying to [comment:3 hbhalodia]:
 > Hi @rollybueno @Guido07111975, Looks like the we are only checking for
 the first instance to load the intended script, which is ideally not
 correct because, before loading the script based on first instance, we are
 checking for empty attachment, and returning early, so it never reaches
 the action which loads the script, now on subsequent playlist, since if we
 find the attachment, we proceed, but the chcek related to first instance
 would fail and hence it won't load the script. So we need to update it
 with proper check instead of first instance check.
 >
 > Thank You,

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/63583#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform