Merge pull request #5105 from Ftornik/Lynda-subtitle-hotfix-2
[lynda] Check for the empty subtitles
This commit is contained in:
commit
549e58069c
|
@ -144,6 +144,7 @@ class LyndaIE(InfoExtractor):
|
||||||
|
|
||||||
def _fix_subtitles(self, subs):
|
def _fix_subtitles(self, subs):
|
||||||
srt = ''
|
srt = ''
|
||||||
|
seq_counter = 0
|
||||||
for pos in range(0, len(subs) - 1):
|
for pos in range(0, len(subs) - 1):
|
||||||
seq_current = subs[pos]
|
seq_current = subs[pos]
|
||||||
m_current = re.match(self._TIMECODE_REGEX, seq_current['Timecode'])
|
m_current = re.match(self._TIMECODE_REGEX, seq_current['Timecode'])
|
||||||
|
@ -155,8 +156,10 @@ class LyndaIE(InfoExtractor):
|
||||||
continue
|
continue
|
||||||
appear_time = m_current.group('timecode')
|
appear_time = m_current.group('timecode')
|
||||||
disappear_time = m_next.group('timecode')
|
disappear_time = m_next.group('timecode')
|
||||||
text = seq_current['Caption'].lstrip()
|
text = seq_current['Caption'].strip()
|
||||||
srt += '%s\r\n%s --> %s\r\n%s' % (str(pos), appear_time, disappear_time, text)
|
if text:
|
||||||
|
seq_counter += 1
|
||||||
|
srt += '%s\r\n%s --> %s\r\n%s\r\n\r\n' % (seq_counter, appear_time, disappear_time, text)
|
||||||
if srt:
|
if srt:
|
||||||
return srt
|
return srt
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user