topics: name unicity check

proposals/forms.py

@@ -1,15 +1,32 @@
-from django.forms import CheckboxSelectMultiple
+from django.forms import CheckboxSelectMultiple, ModelForm
 from django.forms.models import modelform_factory
+from django.core.exceptions import ValidationError
 
 from django_select2.forms import Select2TagWidget
 
 from proposals.models import Talk, Topic
 
+
 TalkForm = modelform_factory(Talk, fields=['title', 'description', 'topics', 'event', 'speakers'],
                              widgets={'topics': CheckboxSelectMultiple(), 'speakers': Select2TagWidget()})
 
-TopicCreateForm = modelform_factory(Topic, fields=['name', 'reviewers'],
-                                    widgets={'reviewers': Select2TagWidget()})
+
+class TopicCreateForm(ModelForm):
+    def __init__(self, *args, **kwargs):
+        self.site_id = kwargs.pop('site_id')
+        super(TopicCreateForm, self).__init__(*args, **kwargs)
+
+    class Meta:
+        model = Topic
+        fields = ['name', 'reviewers']
+        widgets = {'reviewers': Select2TagWidget()}
+
+    def clean_name(self):
+        name = self.cleaned_data['name']
+        if Topic.objects.filter(site__id=self.site_id, name=name).exists():
+            raise self.instance.unique_error_message(self._meta.model, ['name'])
+        return name
+
 
 TopicUpdateForm = modelform_factory(Topic, fields=['reviewers'],
                                     widgets={'reviewers': Select2TagWidget()})

proposals/views.py

@@ -93,6 +93,11 @@ class TopicCreate(OrgaRequiredMixin, TopicMixin, CreateView):
     model = Topic
     form_class = TopicCreateForm
 
+    def get_form_kwargs(self):
+        kwargs = super(TopicCreate, self).get_form_kwargs()
+        kwargs.update({'site_id': get_current_site(self.request).id})
+        return kwargs
+
     def form_valid(self, form):
         form.instance.site = get_current_site(self.request)
         return super().form_valid(form)