Как сделать форму регистрации и форму вход на сайт на одной странице?

Question

[Isaac Clark]

Здравствуйте.

Подскажите пожалуйтса.

Есть форма регистрации и входа на сайт, которые расположены на одной странице и имеют одно urls.

#urls.py
urlpatterns = patterns('users.views',   
    url(r'^$', 'PersonRegistration', name = 'Home Page'),   
    url(r'^$', 'LoginRequest', name = 'Home Page'),    
)

Обе они находятся в одном views

#views.py
def PersonRegistration(request):
    ...

def LoginRequest(request):
    ...

Регистрация работает, все супер! Но… Как только я подключаю LoginRequest и пытаюсь зайти через него зарегестрированным пользователем, то начинает обрабатываться и форма регистрации PersonRegistration…

Как мне отделить формы и сделать так чтоб обрабатывалась только та, которая нужна.

action=""

Очень нужна ваша помощь, уважаемые знатоки…

Спасибо

Answer 1

[Андрей Буров]

<form>
<input type="hidden" name="form" value="registration" />
</form>

<form>
<input type="hidden" name="form" value="auth" />
</form>

Comments

[Isaac Clark]

и что? а запрос как будет обрабатываться? как джанго узнает какую именно форму ему обрабатывать?

[Дамир Абдуллин]

Чтобы не «размножать» лишние поля, можно добавить атрибут value к кнопке.

<input type="submit" name="auth" value="Войти" />

<input type="submit" name="register" value="Регистрировать" />

В коде проверять просто:

if 'auth' in request.POST:
  action

[Дамир Абдуллин]

Прошу прощения, к кнопке добавить атрибут name, value там обычно и так уже есть :)

[Isaac Clark]

а если у меня не кнопка, а ссылка <a data-toggle="modal" href="#registration-form" name="registration">

[Дамир Абдуллин]

Вы хотите использовать для обработки django forms или напишете свой обработчик?
В первом случае вам нужно использовать формсеты, во втором предложенные выше варианты. Если вы отправляете форму с помощью ссылки и onsubmit, то можно добавить скрытое поле, как посоветовал товарищ выше и затем проверять наличие нужной переменной в request.POST, как написано в моем примере.

[Анатолий]

А если я просто ентер нажму на поле?

[Isaac Clark]

Выдает ошибку…
Сделал, как вы написали: if 'auth' in request.POST: action
Выдает: Exception Type: ValueError
Exception Value:

The view users.views.PersonRegistration didn't return an HttpResponse object.

[Isaac Clark]

вот весь мой код со строчкой, которую вы посоветовали добавить
def PersonRegistration(request): if 'auth' in request.POST: if request.user.is_authenticated(): return HttpResponseRedirect('/profile/') if request.method == 'POST': form = RegistrationForm(request.POST) if form.is_valid(): user = User.objects.create_user(username = form.cleaned_data['username'], email = form.cleaned_data['email'], password = form.cleaned_data['password']) user.save() person = Person(user = user, first_name = form.cleaned_data['first_name'], last_name = form.cleaned_data['last_name'], email = form.cleaned_data['email'], password = form.cleaned_data['password']) person.save() return HttpResponseRedirect('/profile/') else: return render_to_response('home.html', { 'form' : form }, context_instance = RequestContext(request)) else: ''' user not submitting the form, show them a blank registration form.''' form = RegistrationForm() context = { 'form' : form } return render_to_response('home.html', context, context_instance = RequestContext(request))

[Isaac Clark]

def PersonRegistration(request): if 'auth' in request.POST: if request.user.is_authenticated(): return HttpResponseRedirect('/profile/') if request.method == 'POST': form = RegistrationForm(request.POST) if form.is_valid(): user = User.objects.create_user(username = form.cleaned_data['username'], email = form.cleaned_data['email'], password = form.cleaned_data['password']) user.save() person = Person(user = user, first_name = form.cleaned_data['first_name'], last_name = form.cleaned_data['last_name'], email = form.cleaned_data['email'], password = form.cleaned_data['password']) person.save() return HttpResponseRedirect('/profile/') else: return render_to_response('home.html', { 'form' : form }, context_instance = RequestContext(request)) else: ''' user not submitting the form, show them a blank registration form.''' form = RegistrationForm() context = { 'form' : form } return render_to_response('home.html', context, context_instance = RequestContext(request))

[Isaac Clark]

ИЗВИНИТЕ, не могу табы поставить!!! ИЗВИНИТЕ!!!

[Анатолий]

<pre>

[Анатолий]

<source lang="python">
// code goes here
</source>

Answer 2

[Анатолий]

А я бы советовал разделить эти две страницы =) Разные ведь вещи делают, зачем лепить все в одно место?

Comments

[Isaac Clark]

По ТЗ — формы входа и регистрации должны быть на одной странице.

[Анатолий]

Иногда стоит предлагать свои варианты а не слепо следовать тз. Вы как разработчик имеете большее представление о сайтах нежели люди заказывающие их.

Answer 3

[marazmiki]

Дикость какая-то.

Зачем обрабатывать регистрацию и вход в одной вьюхе-то? Ладно, можно на одной странице обе формы вывести, если совсем неймётся. Но обрабатывать!..

Comments

[Isaac Clark]

Ок, допустим обработка у меня в разных вьюхах, но вопрос в том, что формы в любом случае находятся на одной странице и соостветственно имеют лдин урл, как в таком случае мне объяснить вьюхе, какую нужно подключать функцию для кажной формы, или в вашем случае, какую нужно подключать вьюху для обработки определенной формы?

[larikov]

На одной странице формы могут иметь разные URL.

Answer 4

[marazmiki]

Проще всего сделать отдельную страничку, на которой будут обе формы. Только вывод, никакой обработки.

Сперва написать вьюху:

from django.shortcuts import render
def all_in_one(request):
    return render('reg_and_login.html', {
        'login_form': LoginForm(),
        'register_form': RegistrationForm()
    })

Затем добавить её к тем, что уже объявлены. Особое внимание уделить тому, чтобы регистрация и вход ввели на разные урлы:

url(r'^$', 'all_in_one'), # Добавили
url(r'^registration/$', 'PersonRegistration', name = 'register-form'), # Изменили
url(r'^login$', 'LoginRequest', name = 'login-form'),  # Изменили

И сделать шаблон (этот код взят из ответа larikov):

<!-- reg_and_login.html -->
<form action="{% url login-form %}" method="post">
    {{ login_form }}
</form>

<form action="{% url register-form %}" method="post">
     {{ register_form }}
</form>

Comments

[marazmiki]

Хм, вроде в комментариях писал, а после выпавшей 500-й ошибки сообщение таки отправилось, но не в комментарий, а отдельно

[Isaac Clark]

Спасибо, сейчас буду делать!

[larikov]

Только не бездумно копируйте конечно же. В коде нет нужных импортов. PEP-8 кстати не позволяет называть методы в CamelCase, только классы.

def register_login_user(request):
    template_dict = {
        'register_form': PersonRegistration(),
        'login_form': LoginRequest(),
    }

    render(request, 'register_login.html', template_dict)


def register_user(request):
    form = PersonRegistration(request.POST)
    if form.is_valid():
        return HttpResponseRedirect(reverse('url_one'))


def login_user(request):
    form = LoginRequest(request.POST)
    if form.is_valid():
        return HttpResponseRedirect(reverse('url_two'))

url(r'^$', 'views.register_login_user', name='register_login_user')
url(r'^register/$', 'views.register_user', name='register-form'), 
url(r'^login/$', 'views.login_user', name='login-form'),

[larikov]

не render(...) конечно же, а return render(...)

[Isaac Clark]

Все формы находятся в шаблоне register_login.html?

[larikov]

Чувствую, пока вы не покажете свой код — ничего не получится. Поэтому вот я для вас набросал — github.com/larikov/for_habr/

[Isaac Clark]

Спасибо огромное, сейчас буду разбираться с вашим кодом. Если не получится, то тогда покажу весь свой. Еще раз спасибо1

[Isaac Clark]

Выдает ошибку.
Reverse for 'habr_register' with arguments '()' and keyword arguments '{}' not found.
Если для главной страницы меняю в urls.py
url(r'^$', 'habr.views.index', name='habr_home'),
на
url(r'^$', 'habr.views.index', name='habr_register'),
То єту ошибку пропустакет, но тогда ругается на {% url habr_login %}
Возможно это мотому что эти страницы ссылаются на другие урл?

[larikov]

имена должны быть уникальными. habr_register уже есть url(r'^register/$', 'habr.views.register', name='habr_register'),
Без вашего полного кода трудно что-то сказать

Answer 5

[larikov]

Меня больше интересует, как в вашем случае django знает, какой view отображать?

Comments

[larikov]

В смысле, вот сделаете вы reverse('Home Page') и какой URL получите?

Сделайте себе нормальную структуру приложения и не мучайтесь

class PersonRegistration(FormView):
    form_class = RegistrationForm
    
    def form_valid(self, form):
        # bla-bla-bla

    def form_invalid(self, form):
        # bla-bla


class LoginRequest(FormView):
    form_class = LoginForm
    def form_valid(self, form):
        # bla-bla-bla

    def form_invalid(self, form):
        # bla-bla

urlpatterns = patterns('users.views',
    url(r'^register/$', PersonRegistration.as_view(), name='register'),
    url(r'^login/$', LoginRequest.as_view(), name='login'),
)

[marazmiki]

Вопрос, собственно, примерно в этом и состоял :-)

Когда страница запрашивается по GET, вьюха просто рендерит шаблон с двумя формами. Все рады и счастливы. А вот когда нужно формы не показать а обработать, начинается свистопляска. Ребята там про имя сабмита подсказывали. В принципе правильно настолько, насколько это слово применимо к самой задаче.

[Isaac Clark]

larikov, если не ошибаюсь, то ваши урл указывают на другие страницы. А у меня все формы находятся на одной странице и там они должны находится

[larikov]

Почему бы для каждой формы не прописать свой URL в action, и уже после сабмита, редиректить куда надо.

<form action="{% url login-form %}" method="post">
    {{ login_form }}
</form>

<form action="{% url register-form %}" method="post">
     {{ register_form }}
</form>

Вобщем, ничего сложного

[Isaac Clark]

marazmiki, сейчас попробую этот способ реализовать…

[Isaac Clark]

larikov, как только добавил в action новый урл, то сразу выдал ошибку джанго
Exception Value: Reverse for 'register-form' with arguments '()' and keyword arguments '{}' not found.

[larikov]

В urls.py прописалиname='register-form'?

[Isaac Clark]

larikov, все ок, прописал, но с опечаткой, исправил теперь работает, спасибо, а теперь для в urls.py нужно прописать
url(r'^$', 'PersonRegistration', name = 'register-form'), — для регистрации
url(r'^$', 'LoginRequest', name = 'login-form'), — для входа на сайт
И теперь формі будут обрабатіваться каждая отдельно, я правильно понял?

[marazmiki]

Ну как по одному урлу могут быть две разные акции-то?

[Isaac Clark]

marazmiki, тогда подскажите пожалуйста, как правильно сделать?

[larikov]

Да, при сабмите формы пользователь будет перенаправляться на url, указанный в action

только ваш urls.py неверный. нужно указать разные url'ы

url(r'^register/$', 'PersonRegistration', name = 'register-form'), 
url(r'^login/$', 'LoginRequest', name = 'login-form'),