Skip to content

Commit

Permalink
[youtube] improve title and description extraction(closes #21934)
Browse files Browse the repository at this point in the history
  • Loading branch information
remitamine committed Jul 29, 2019
1 parent 90634ac commit 8dbf751
Showing 1 changed file with 3 additions and 12 deletions.
15 changes: 3 additions & 12 deletions youtube_dl/extractor/youtube.py
Original file line number Diff line number Diff line change
Expand Up @@ -1820,16 +1820,11 @@ def extract_unavailable_message():
video_details = try_get(
player_response, lambda x: x['videoDetails'], dict) or {}

# title
if 'title' in video_info:
video_title = video_info['title'][0]
elif 'title' in player_response:
video_title = video_details['title']
else:
video_title = video_info.get('title', [None])[0] or video_details.get('title')
if not video_title:
self._downloader.report_warning('Unable to extract video title')
video_title = '_'

# description
description_original = video_description = get_element_by_id("eow-description", video_webpage)
if video_description:

Expand All @@ -1854,11 +1849,7 @@ def replace_url(m):
''', replace_url, video_description)
video_description = clean_html(video_description)
else:
fd_mobj = re.search(r'<meta name="description" content="([^"]+)"', video_webpage)
if fd_mobj:
video_description = unescapeHTML(fd_mobj.group(1))
else:
video_description = ''
video_description = self._html_search_meta('description', video_webpage) or video_details.get('shortDescription')

if not smuggled_data.get('force_singlefeed', False):
if not self._downloader.params.get('noplaylist'):
Expand Down

1 comment on commit 8dbf751

@Awan
Copy link

@Awan Awan commented on 8dbf751 Sep 8, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit resolved the issue of title.

Please sign in to comment.