From 4073cefc74893d052ffb1c25507ba2ef238a9abd Mon Sep 17 00:00:00 2001 From: rlaphoenix Date: Thu, 29 Feb 2024 22:06:02 +0000 Subject: [PATCH] Remove Subtitle.remove_multi_lang_srt_header() The root cause of the error which required calling this function was identified and fixed in this release. --- devine/core/tracks/subtitle.py | 21 --------------------- 1 file changed, 21 deletions(-) diff --git a/devine/core/tracks/subtitle.py b/devine/core/tracks/subtitle.py index bbdf28b..6ab4d02 100644 --- a/devine/core/tracks/subtitle.py +++ b/devine/core/tracks/subtitle.py @@ -507,27 +507,6 @@ class Subtitle(Track): stdout=subprocess.DEVNULL ) - def remove_multi_lang_srt_header(self) -> None: - """ - Remove Multi-Language SRT Header from Subtitle. - - Sometimes a SubRip (SRT) format Subtitle has a "MULTI-LANGUAGE SRT" line, - when it shouldn't. This can cause Subtitle format/syntax errors in some - programs including mkvmerge/MKVToolNix. - - This should only be used if it truly is a normal SubRip (SRT) subtitle - just with this line added by mistake. - """ - if not self.path or not self.path.exists(): - raise ValueError("You must download the subtitle track first.") - - if self.codec != Subtitle.Codec.SubRip: - raise ValueError("Only SubRip (SRT) format Subtitles have the 'MULTI-LANGUAGE SRT' header.") - - srt_text = self.path.read_text("utf8") - fixed_srt_text = srt_text.replace("MULTI-LANGUAGE SRT\n", "") - self.path.write_text(fixed_srt_text, "utf8") - def __str__(self) -> str: return " | ".join(filter(bool, [ "SUB",