w/youtube-dl
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

[brightcove:new] Clarify ref: prefix

Browse source
This commit is contained in:
Sergey M․ 2015-12-18 22:22:41 +06:00
parent f81ccbb3df
commit 5b72fda140
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
youtube_dl/extractor/brightcove.py
View file

@ -400,6 +400,7 @@ class BrightcoveNewIE(InfoExtractor):
# 1. http://docs.brightcove.com/en/video-cloud/brightcove-player/guides/publish-video.html#setvideoiniframe # 1. http://docs.brightcove.com/en/video-cloud/brightcove-player/guides/publish-video.html#setvideoiniframe
# 2. http://docs.brightcove.com/en/video-cloud/brightcove-player/guides/publish-video.html#setvideousingjavascript # 2. http://docs.brightcove.com/en/video-cloud/brightcove-player/guides/publish-video.html#setvideousingjavascript
# 3. http://docs.brightcove.com/en/video-cloud/brightcove-player/guides/embed-in-page.html # 3. http://docs.brightcove.com/en/video-cloud/brightcove-player/guides/embed-in-page.html
# 4. https://support.brightcove.com/en/video-cloud/docs/dynamically-assigning-videos-player
entries = [] entries = []
@ -412,6 +413,7 @@ class BrightcoveNewIE(InfoExtractor):
for video_id, account_id, player_id, embed in re.findall( for video_id, account_id, player_id, embed in re.findall(
# According to examples from [3] it's unclear whether video id # According to examples from [3] it's unclear whether video id
# may be optional and what to do when it is # may be optional and what to do when it is
# According to [4] data-video-id may be prefixed with ref:
r'''(?sx) r'''(?sx)
<video[^>]+ <video[^>]+
data-video-id=["\']((?:ref:)?\d+)["\'][^>]*>.*? data-video-id=["\']((?:ref:)?\d+)["\'][^>]*>.*?