Fix for provider subtitle language being unintentionally modified

This prevents unwanted changes to the provider copy of the language.
Thanks to @vitiko98 for suggesting a better place for this fix.
pull/2610/head v1.4.4-beta.29
JayZed 5 months ago committed by GitHub
parent 866b1d5894
commit 5582cc076d
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

@ -12,6 +12,7 @@ import chardet
import pysrt import pysrt
import pysubs2 import pysubs2
from bs4 import UnicodeDammit from bs4 import UnicodeDammit
from copy import deepcopy
from pysubs2 import SSAStyle from pysubs2 import SSAStyle
from pysubs2.formats.subrip import parse_tags, MAX_REPRESENTABLE_TIME from pysubs2.formats.subrip import parse_tags, MAX_REPRESENTABLE_TIME
from pysubs2.time import ms_to_times from pysubs2.time import ms_to_times
@ -65,6 +66,11 @@ class Subtitle(Subtitle_):
# format = "srt" # default format is srt # format = "srt" # default format is srt
def __init__(self, language, hearing_impaired=False, page_link=None, encoding=None, mods=None, original_format=False): def __init__(self, language, hearing_impaired=False, page_link=None, encoding=None, mods=None, original_format=False):
# language needs to be cloned because it is actually a reference to the provider language object
# if a new copy is not created then all subsequent subtitles for this provider will incorrectly be modified
# at least until Bazarr is restarted or the provider language object is recreated somehow
language = deepcopy(language)
# set subtitle language to hi if it's hearing_impaired # set subtitle language to hi if it's hearing_impaired
if hearing_impaired: if hearing_impaired:
language = Language.rebuild(language, hi=True) language = Language.rebuild(language, hi=True)

Loading…
Cancel
Save