[ie/afl] refactored AFCVideoIE

1. added tag option to 'get_element_html_by_attribute' function.
   this removes the need for using '_search_json'
2. swapped 'if not' statement with 'if' for simplification

feedback: pzhlkj6612
This commit is contained in:
subrat-lima 2024-09-24 15:25:06 +05:30
parent cb0aa20d4f
commit 090fb58de7

View file

@ -155,11 +155,11 @@ class AFCVideoIE(InfoExtractor):
player_id = video_attrs['data-player-id'] + '_default' player_id = video_attrs['data-player-id'] + '_default'
account_id = video_attrs['data-account-id'] account_id = video_attrs['data-account-id']
video_element_html = get_element_html_by_attribute('data-id', display_id, webpage) video_element_html = get_element_html_by_attribute('data-id', display_id, webpage, tag='a')
if not video_element_html: if video_element_html:
video_data = self._download_json(f'https://aflapi.afc.com.au/content/aflc-adel/video/en/{display_id}', display_id) video_data = self._parse_json(extract_attributes(video_element_html)['data-ui-args'], display_id)
else: else:
video_data = self._search_json(r'data-ui-args\s*=\s*["\']', video_element_html, 'video-id', display_id) video_data = self._download_json(f'https://aflapi.afc.com.au/content/aflc-adel/video/en/{display_id}', display_id)
video_id = video_data['mediaId'] video_id = video_data['mediaId']
video_url = f'https://players.brightcove.net/{account_id}/{player_id}/index.html?videoId={video_id}' video_url = f'https://players.brightcove.net/{account_id}/{player_id}/index.html?videoId={video_id}'