[wp-trac] [WordPress Trac] #41052: Need validation update in video widget

WordPress Trac noreply at wordpress.org
Sun Oct 15 15:34:11 UTC 2017


#41052: Need validation update in video widget
-------------------------------------+-----------------------------
 Reporter:  ravipatel                |       Owner:
     Type:  defect (bug)             |      Status:  reopened
 Priority:  high                     |   Milestone:  4.9
Component:  Widgets                  |     Version:  4.8
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:  ui, javascript
-------------------------------------+-----------------------------

Comment (by westonruter):

 @gk.loveweb I iterated on your patch a bit and I think I came to something
 that works better in [attachment:41052.4.diff]. Would you give it a try?
 By moving the logic to `updateoEmbed` we can short-circuit the `fetch`
 logic earlier, since `updateoEmbed` is what calls `fetch` to begin with.

 The audio widget needs some more work, I believe. For one it doesn't
 support oEmbeds at all currently.

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


More information about the wp-trac mailing list