mirror of
https://github.com/yt-dlp/yt-dlp.git
synced 2024-12-03 03:57:26 +00:00
[mtv] Fix title for gametrailers (Fixes #2188)
We now prefer the title including the category, because that title is what is presented at the actual sites.
This commit is contained in:
parent
4a3b72771f
commit
90834c78fe
2 changed files with 17 additions and 2 deletions
|
@ -10,7 +10,7 @@ class GametrailersIE(MTVServicesInfoExtractor):
|
||||||
u'file': u'70e9a5d7-cf25-4a10-9104-6f3e7342ae0d.mp4',
|
u'file': u'70e9a5d7-cf25-4a10-9104-6f3e7342ae0d.mp4',
|
||||||
u'md5': u'4c8e67681a0ea7ec241e8c09b3ea8cf7',
|
u'md5': u'4c8e67681a0ea7ec241e8c09b3ea8cf7',
|
||||||
u'info_dict': {
|
u'info_dict': {
|
||||||
u'title': u'E3 2013: Debut Trailer',
|
u'title': u'Mirror\'s Edge 2|E3 2013: Debut Trailer',
|
||||||
u'description': u'Faith is back! Check out the World Premiere trailer for Mirror\'s Edge 2 straight from the EA Press Conference at E3 2013!',
|
u'description': u'Faith is back! Check out the World Premiere trailer for Mirror\'s Edge 2 straight from the EA Press Conference at E3 2013!',
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
|
@ -5,9 +5,11 @@
|
||||||
from ..utils import (
|
from ..utils import (
|
||||||
compat_urllib_parse,
|
compat_urllib_parse,
|
||||||
ExtractorError,
|
ExtractorError,
|
||||||
|
find_xpath_attr,
|
||||||
fix_xml_ampersands,
|
fix_xml_ampersands,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
def _media_xml_tag(tag):
|
def _media_xml_tag(tag):
|
||||||
return '{http://search.yahoo.com/mrss/}%s' % tag
|
return '{http://search.yahoo.com/mrss/}%s' % tag
|
||||||
|
|
||||||
|
@ -72,8 +74,21 @@ def _get_video_info(self, itemdoc):
|
||||||
else:
|
else:
|
||||||
description = None
|
description = None
|
||||||
|
|
||||||
|
title_el = None
|
||||||
|
if title_el is None:
|
||||||
|
title_el = find_xpath_attr(
|
||||||
|
itemdoc, './/{http://search.yahoo.com/mrss/}category',
|
||||||
|
'scheme', 'urn:mtvn:video_title')
|
||||||
|
if title_el is None:
|
||||||
|
title_el = itemdoc.find('.//{http://search.yahoo.com/mrss/}title')
|
||||||
|
if title_el is None:
|
||||||
|
title_el = itemdoc.find('.//title')
|
||||||
|
title = title_el.text
|
||||||
|
if title is None:
|
||||||
|
raise ExtractorError('Could not find video title')
|
||||||
|
|
||||||
return {
|
return {
|
||||||
'title': itemdoc.find('title').text,
|
'title': title,
|
||||||
'formats': self._extract_video_formats(mediagen_page),
|
'formats': self._extract_video_formats(mediagen_page),
|
||||||
'id': video_id,
|
'id': video_id,
|
||||||
'thumbnail': self._get_thumbnail_url(uri, itemdoc),
|
'thumbnail': self._get_thumbnail_url(uri, itemdoc),
|
||||||
|
|
Loading…
Reference in a new issue