From b3b271a0b126c973886fa1d8677d2f247add82b3 Mon Sep 17 00:00:00 2001 From: Leonardo Taccari Date: Thu, 30 Aug 2018 21:08:30 +0200 Subject: [PATCH] [internazionale] Avoid bool(int(...)) casting and code duplication (Hopefully) based on what @dstftw suggested. It is better to avoid bool(int(...)) because it is fragile. Avoid code duplication in video_base initialization. --- youtube_dl/extractor/internazionale.py | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/youtube_dl/extractor/internazionale.py b/youtube_dl/extractor/internazionale.py index ea1dfc960..676e8e269 100644 --- a/youtube_dl/extractor/internazionale.py +++ b/youtube_dl/extractor/internazionale.py @@ -56,15 +56,13 @@ class InternazionaleIE(InfoExtractor): DATA_RE % 'job-id', webpage, 'video id', group='value') video_path = self._search_regex( DATA_RE % 'video-path', webpage, 'video path', group='value') - video_available_abroad = bool(int(self._search_regex( + video_available_abroad = self._search_regex( DATA_RE % 'video-available_abroad', webpage, - 'video available aboard', default='1', group='value'))) + 'video available aboard', default='1', group='value') + video_available_abroad = video_available_abroad == '1' - if video_available_abroad: - video_base = 'https://video.internazionale.it/' - else: - video_base = 'https://video-ita.internazionale.it/' - video_base = video_base + '%s/%s.' % (video_path, video_id) + video_base = 'https://video%s.internazionale.it/%s/%s.' % \ + ('' if video_available_abroad else '-ita', video_path, video_id) formats = self._extract_m3u8_formats( video_base + 'm3u8', display_id, 'mp4',