diff --git a/src/postorius/tests/mailman_api_tests/test_domain_new.py b/src/postorius/tests/mailman_api_tests/test_domain_new.py index 20a6f45..9fcd045 100644 --- a/src/postorius/tests/mailman_api_tests/test_domain_new.py +++ b/src/postorius/tests/mailman_api_tests/test_domain_new.py @@ -68,7 +68,6 @@ 'site': '1', } response = self.client.post(reverse('domain_new'), post_data) - self.assertContains(response, 'Please check the errors below') self.assertContains(response, 'Please enter a valid domain name') # self.assertHasErrorMessage(response) self.assertEquals(response.status_code, 200) diff --git a/src/postorius/views/domain.py b/src/postorius/views/domain.py index 8e3dbf3..e655be6 100644 --- a/src/postorius/views/domain.py +++ b/src/postorius/views/domain.py @@ -69,15 +69,13 @@ except MailmanApiError: return utils.render_api_error(request) except HTTPError as e: - messages.error(request, e) + form.add_error('mail_host', e.reason) else: messages.success(request, _("New Domain registered")) - MailDomain.objects.get_or_create( - site=form.cleaned_data['site'], - mail_domain=form.cleaned_data['mail_host']) - return redirect("domain_index") - else: - messages.error(request, _('Please check the errors below')) + MailDomain.objects.get_or_create( + site=form.cleaned_data['site'], + mail_domain=form.cleaned_data['mail_host']) + return redirect("domain_index") else: form = DomainForm(initial=form_initial) return render(request, 'postorius/domain/new.html', {'form': form})