Как устранить проблему с непрекращающейся отправкой писем через форму на сайте?

Question

[Евгений]

Добрый вечер.
Есть сайт, на котором форма заявки есть на всех страницах. Для упрощения работы создал контекстный процессор для forms.py. Форма письма отправляет, но бесконечное количество, т.е. процесс отправки не завершается, пока не остановишь принудительно.

Forms.py

from django.core.context_processors import request
from django import forms
from django.shortcuts import render
from django.http import HttpResponse
from django.core.mail import send_mail, BadHeaderError
class ContactForm(forms.Form):
	subject = forms.CharField(max_length=100,widget=forms.TextInput(attrs = {'required': 'required'}))
	sender = forms.EmailField(widget=forms.TextInput(attrs = {'required': 'required'}))
	message = forms.CharField(widget = forms.Textarea(attrs = {'required': 'required'}))
	copy = forms.BooleanField(required = False)
def my_form(request):
    if request.method == 'POST':
        form = ContactForm(request.POST)
        if form.is_valid():
            subject = form.cleaned_data['subject']
            sender = form.cleaned_data['sender']
            message = form.cleaned_data['message']
            copy = form.cleaned_data['copy']
            plus = message + ' ' + sender

            recipients = ['[e-mail]']
            if copy:
                recipients.append(sender)
            try:
                send_mail(subject, plus, '[e-mail]', recipients)
            except BadHeaderError: 
                return HttpResponse('Invalid header found')
            return render(request, 'single_index.html')
    else:
        form = ContactForm()
    return {'form': form}

Comments

[Ростислав Григорьев]

> Для упрощения работы создал контекстный процессор

и где он?

[Евгений]

Ростислав Григорьев: собственно, я разбираюсь с джангой, когда есть время свободное, некоторые вещи упускаю из виду из-за этого. Собственно, что имелось ввиду в вашем вопросе? Если текущая первая строчка, то упистил при копировании, если что-то еще, то не могли бы вы, объясниться?

[Ростислав Григорьев]

Евгений: что вы имеете в виду под "контекстный процессор". Этот импорт следует удалить.

`from django.core.context_processors import request`

[Евгений]

Ростислав Григорьев: я подключил forms.py как контекстный процессор через setting.py

Answer 1

[Mr_Floppy]

При успешной обработке формы нужно делать не render, а redirect (т.к. при нажатии F5 форма отправится снова)

Еще на последней строчке Вы возвращаете форму, а нужно render(request, 'single_index.html', {'form': form})

request вообще не надо импортировать, это агрумент view-функции.

Также непонятно зачем оборачивать send_mail в try...except и ловить только BadHeaderError. Я бы ловил более широкий класс ошибок и говорил пользователю, что отправка не удалась.

Comments

[Евгений]

Да, с настройками формы все понятно, однако, на последней строчке стоял render(request, 'single_index.html', {'form': form}), пока не было решено подключить форму, как контекстный процессор. Если её подключать с таким рендером, то проект падает.

[Mr_Floppy]

Евгений: Если подключаете форму как context processor - то просто
def show_form() :
return {'form': ContactForm()}
и в шаблоне выводите форму. Но обработку должен делать view, не processor.

[Евгений]

Mr_Floppy: Мне удалось вывести форму следующим образом, по другому не выводилось.
context.py
from twave.forms import *
def show_form() :
return {'form': ContactForm()}

forms.py не изменен, за исключением удаления верхней строчки.

Ситуация не изменилась, на почту сыпятся десятки писем, пока не остановишь процесс в ручную

[Mr_Floppy]

Евгений: возможно зацикливание происходит потому что my_form(request) находится в контектном процессоре и, заполняя форму из POST, рендерит render(request, 'single_index.html'), который (возможно) в свою очередь рекурсивно вызывает сам себя. А может и нет, не знаю точно.

Сделайте тупо по доке:
Объявите в forms.py Вашу форму.
Во views.py сделайте обработчик формы, который 1) выводит пустую форму 2) отправляет письмо если форма заполнена через POST и валидна. Не забывайте про render, redirect.
Когда все заработает нормально - можете сделать контектный процессор, чтобы выводить форму на любой странице. Еще раз подчеркиваю, что context processor не должен ничего обрабатывать, он лишь должен вернуть экземпляр формы.

[Евгений]

Я шел этим путем, создал форму, поставил обработчик во вью и все работало. Ровно до того момента, пока я не решил размножить форму на все модули. 2 кейса. Либо копипастить обработчик во все вью, где нужен вывод формы, либо делать контекстный процессор. Выбрал процессор. Обработчик поместил в forms.py. Я что-то делаю не так?

[Mr_Floppy]

В forms.py у Вас должен быть только класс формы. Обработчики принято класть во views.py

Обработчик примерно такой:
views.py
def send_email(request):
if request.method == 'POST':
form = ContactForm(request.POST)
if form.is_valid():
...
redirect('success_url')
else:
form = ContactForm()
return render(request, 'template.html', {'form': form})

Процессор такой:
context_processor.py
def contact_form(request):
return {'contact_form': ContactForm()}

[Евгений]

Хз что делаю не так. Пожалуйста, посмотрите и пальцем ткните. https://docs.google.com/document/d/1B1Gn7QrK_-iQZB...

[Mr_Floppy]

Начните новый проект и делайте пошагово. Снова рендер вместо редикта... В конце-концов возмите отладчик и выполните программу пошагово.