X-Git-Url: https://git.openstreetmap.org./osqa.git/blobdiff_plain/ee36e3673f75b20e4b37f0a18ab471ce16b0f389..ecd9fa68ea62902f79744b458e4ea2ed95ce00ac:/forum/forms.py diff --git a/forum/forms.py b/forum/forms.py index 245372a..34fa271 100644 --- a/forum/forms.py +++ b/forum/forms.py @@ -23,25 +23,43 @@ class TitleField(forms.CharField): def clean(self, value): if len(value) < settings.FORM_MIN_QUESTION_TITLE: - raise forms.ValidationError(_('title must be must be at least %s characters' % settings.FORM_MIN_QUESTION_TITLE)) + raise forms.ValidationError(_('title must be must be at least %s characters') % settings.FORM_MIN_QUESTION_TITLE) return value class EditorField(forms.CharField): def __init__(self, *args, **kwargs): super(EditorField, self).__init__(*args, **kwargs) - self.required = True self.widget = forms.Textarea(attrs={'id':'editor'}) self.label = _('content') self.help_text = u'' self.initial = '' + +class QuestionEditorField(EditorField): + def __init__(self, *args, **kwargs): + super(QuestionEditorField, self).__init__(*args, **kwargs) + self.required = not bool(settings.FORM_EMPTY_QUESTION_BODY) + + def clean(self, value): - if len(value) < settings.FORM_MIN_QUESTION_BODY and not settings.FORM_EMPTY_QUESTION_BODY: - raise forms.ValidationError(_('question content must be must be at least %s characters' % settings.FORM_MIN_QUESTION_BODY)) + if self.required and (len(value) < settings.FORM_MIN_QUESTION_BODY): + raise forms.ValidationError(_('question content must be at least %s characters') % settings.FORM_MIN_QUESTION_BODY) return value +class AnswerEditorField(EditorField): + def __init__(self, *args, **kwargs): + super(AnswerEditorField, self).__init__(*args, **kwargs) + self.required = True + + def clean(self, value): + if len(value) < settings.FORM_MIN_QUESTION_BODY: + raise forms.ValidationError(_('answer content must be at least %s characters') % settings.FORM_MIN_QUESTION_BODY) + + return value + + class TagNamesField(forms.CharField): def __init__(self, *args, **kwargs): super(TagNamesField, self).__init__(*args, **kwargs) @@ -66,12 +84,12 @@ class TagNamesField(forms.CharField): raise forms.ValidationError(_('please use between %(min)s and %(max)s tags') % { 'min': settings.FORM_MIN_NUMBER_OF_TAGS, 'max': settings.FORM_MAX_NUMBER_OF_TAGS}) list_temp = [] - tagname_re = re.compile(r'^[\w+\.-]$', re.UNICODE) + tagname_re = re.compile(r'^[\w+\.-]+$', re.UNICODE) for tag in list: if len(tag) > settings.FORM_MAX_LENGTH_OF_TAG or len(tag) < settings.FORM_MIN_LENGTH_OF_TAG: raise forms.ValidationError(_('please use between %(min)s and %(max)s characters in you tags') % { 'min': settings.FORM_MIN_LENGTH_OF_TAG, 'max': settings.FORM_MAX_LENGTH_OF_TAG}) if not tagname_re.match(tag): - raise forms.ValidationError(_('please use following characters in tags: letters , numbers, and the following characters \'.-_#\'')) + raise forms.ValidationError(_('please use following characters in tags: letters , numbers, and characters \'.-_\'')) # only keep one same tag if tag not in list_temp and len(tag.strip()) > 0: list_temp.append(tag) @@ -111,19 +129,19 @@ class FeedbackForm(forms.Form): class AskForm(forms.Form): title = TitleField() - text = EditorField() + text = QuestionEditorField() tags = TagNamesField() wiki = WikiField() class AnswerForm(forms.Form): - text = EditorField() + text = AnswerEditorField() wiki = WikiField() def __init__(self, question, *args, **kwargs): super(AnswerForm, self).__init__(*args, **kwargs) - if question.wiki and settings.WIKI_ON: + if question.nis.wiki and settings.WIKI_ON: self.fields['wiki'].initial = True @@ -154,7 +172,7 @@ class RevisionForm(forms.Form): class EditQuestionForm(forms.Form): title = TitleField() - text = EditorField() + text = QuestionEditorField() tags = TagNamesField() summary = SummaryField() @@ -169,11 +187,11 @@ class EditQuestionForm(forms.Form): self.fields['tags'].initial = revision.tagnames # Once wiki mode is enabled, it can't be disabled - if not question.wiki: + if settings.WIKI_ON and not question.nis.wiki: self.fields['wiki'] = WikiField() class EditAnswerForm(forms.Form): - text = EditorField() + text = AnswerEditorField() summary = SummaryField() def __init__(self, answer, revision=None, *args, **kwargs):