rnhmjoj/qutebrowser
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Make sure we only show dictionary warnings once

Browse Source 
After 3956f81e73 where this was made a function,
the warning was shown twice, causing AppVeyor to fail.
This commit is contained in:
Florian Bruhin 2018-02-28 08:01:11 +01:00
parent 889b03169a
commit 824825e67d
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
qutebrowser/browser/webengine/webenginesettings.py
View File

@ -255,14 +255,15 @@ def _set_persistent_cookie_policy(profile):
profile.setPersistentCookiesPolicy(value)
def _set_dictionary_language(profile):
def _set_dictionary_language(profile, warn=True):
filenames = []
for code in config.val.spellcheck.languages or []:
local_filename = spell.local_filename(code)
if not local_filename:
message.warning(
"Language {} is not installed - see scripts/dictcli.py "
"in qutebrowser's sources".format(code))
if warn:
message.warning(
"Language {} is not installed - see scripts/dictcli.py "
"in qutebrowser's sources".format(code))
continue
filenames.append(local_filename)
@ -293,7 +294,7 @@ def _update_settings(option):
# We're not touching the private profile's cookie policy.
elif option == 'spellcheck.languages' and qtutils.version_check('5.8'):
_set_dictionary_language(default_profile)
_set_dictionary_language(private_profile)
_set_dictionary_language(private_profile, warn=False)
def _init_profile(profile):