Commit 2bc2dd94 authored by Michal Čihař's avatar Michal Čihař

Perform flags validation on save

Issue #540
Signed-off-by: default avatarMichal Čihař <michal@cihar.com>
parent 9ab7a6a5
...@@ -32,6 +32,7 @@ from weblate.trans.models import Unit ...@@ -32,6 +32,7 @@ from weblate.trans.models import Unit
from weblate.trans.models.source import PRIORITY_CHOICES from weblate.trans.models.source import PRIORITY_CHOICES
from weblate.trans.checks import CHECKS from weblate.trans.checks import CHECKS
from weblate.trans.specialchars import get_special_chars from weblate.trans.specialchars import get_special_chars
from weblate.trans.validators import validate_check_flags
from weblate.accounts.forms import sort_choices from weblate.accounts.forms import sort_choices
from urllib import urlencode from urllib import urlencode
import weblate import weblate
...@@ -742,6 +743,17 @@ class CheckFlagsForm(forms.Form): ...@@ -742,6 +743,17 @@ class CheckFlagsForm(forms.Form):
) )
) )
def clean_flags(self):
"""
Be a little bit more tolerant on whitespaces.
"""
flags = [
x.strip() for x in self.cleaned_data['flags'].strip().split(',')
]
flags = ','.join([x for x in flags if x])
validate_check_flags(flags)
return flags
class AddUserForm(forms.Form): class AddUserForm(forms.Form):
name = forms.CharField( name = forms.CharField(
......
...@@ -96,6 +96,8 @@ def validate_check_flags(val): ...@@ -96,6 +96,8 @@ def validate_check_flags(val):
''' '''
Validates check influencing flags. Validates check influencing flags.
''' '''
if not val:
return
for flag in val.split(','): for flag in val.split(','):
if flag in VALID_FLAGS: if flag in VALID_FLAGS:
continue continue
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment