NomadSept. 22, 2020, 8:39 p.m.

MyForm(forms.Form): - непонятка

Django, forms

привет

уважаемые знатоки, может кто объяснить в чем проблема:
есть представление с двумя формами:

class MySettingsChangeView(LoginRequiredMixin, TemplateView):
    template_name = 'accounts/registration/settings_pass_status_change.html'

    def get(self, request, *args, **kwargs):
        context = self.get_context_data(**kwargs)
        return render(request, self.template_name, context)

    def get_context_data(self, **kwargs):
        extra_context = None
        context = super().get_context_data(**kwargs)
        context.update({
            'pass_form': MyPasswordChangeForm,
            'status_form': MyStatusChangeForm,
            **(self.extra_context or {})
        })
        return context

    def post(self, request):
        post_data = request.POST or None
        if 'pass' in self.request.POST:
            print(" srabotal pass")
            cont = self.request.POST
        elif 'status' in self.request.POST:
            if not request.user.is_active or not request.user.phone_active:
                return redirect('/')
            else:
                cont = self.request.POST
                try:
                    new_status = cont['user_status']
                except MultiValueDictKeyError:
                    new_status = request.user.user_status
                curent_user = request.user.pk

                object = get_object_or_404(User, pk=curent_user)

                object.user_status = new_status
                object.save()
        return redirect('/dashboardc/home/')

вот формы:

class MyPasswordChangeForm(PasswordChangeForm):
    pass

class MyStatusChangeForm(forms.Form):
    pass

форма MyStatusChangeForm состоит из 2 радио + сабмит

все работает гуд НО вся логика находится в представлении
я пытался в форме MyStatusChangeForm переопределить пару методов и для того чтобы понять что к ним стучатся написал по одному принту в теле НО такое осушение что все они в игноре, вот что я добавлял в класс формы:

def clean(self):
    print("in clean(self)")
    cleaned_data = super().clean()
    return cleaned_data

def is_valid(self):
    print("in is_valid(self)")

def clean_recipients(self):
    print("in clean_recipients(self)")
    data = super(MyStatusChangeForm, self).clean()
    return data

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

но вопрос в том почему эти методы не срабатывают??

We recommend hosting TIMEWEB
We recommend hosting TIMEWEB
Stable hosting, on which the social network EVILEG is located. For projects on Django we recommend VDS hosting.
13

1) что в шаблоне?
2) в определении контекста:

    def get_context_data(self, **kwargs):
        extra_context = None
        context = super().get_context_data(**kwargs)
        context.update({
            'pass_form': MyPasswordChangeForm,  # формы надо инициализировать
            'status_form': MyStatusChangeForm,  # формы надо инициализировать
            **(self.extra_context or {})  # это тут не нужно - всегда None
        })
        return context

3) в методе post во вьюхе нет никакого использования форм, вот ничего и не происходит
4) зачем напрямую работать с данными из request.POST если можно в форме определить метод "save()" и дергать его или, на худой конец, работать с form.cleaned_data
5) я таки рекомендую разделить это на 2 вьюхи, чтобы каждая их них обрабатывала свою форму - меньше путаницы будет

  • Sept. 23, 2020, 3:20 p.m.

вот шаблон: https://dpaste.org/PhjO

1) Почитайте про формы - многое, что у вас в шалобне определено, можно решить с помощью определения класса формы:

class MyPasswordChangeForm(forms.Form):
    old_password = forms.CharField(widget=forms.PasswordInput())
    new_password1 = forms.CharField(widget=forms.PasswordInput(), help_text="
<ul>
<li>YourYour password can’t be too similar to your other personal information.</li>
<li>Your password must contain at least 8 characters.</li>
<li>Your password can’t be a commonly used password.</li>
<li>Your password can’t be entirely numeric.</li>
</ul>", title="New password")
    new_password2 = forms.CharField(widget=forms.PasswordInput(), title="Repeat new Password")

    def __init__(self, *args, **kwargs):
        self.user = kwargs.pop('user', None)
        super().__init__(*args, **kwargs)

    def clean(self):
        cleaned_data = super().clean()
        if cleaned_data['new_password1'] != cleaned_data['new_password2']:
            self.add_error('new_password2', 'New password confirmation is not same to new password')
        if self.user and not self.user.check_password(cleaned_data['old_password']:
            self.add_error('old_password', 'Current password is not valid')
        return cleaned_data

а работа с формой будет:

class MySettingsChangeView(views.View):
    def get_context_data(self, *args, **kwargs):
        context = super().get_context_data(*args, **kwargs)
        context.update({
            'pass_form': MyPasswordChangeForm(self.request.POST or None),
            ...
        })
        return context

2) В шаблоне не используются формы из контекста. как вариант, что-то такое. Если используете bootstap - можно посмотреть на django-fancy-forms (или как-то так)

...
<form>
    {% csrf_token %}    
    {{ pass_form.non_fields_errors }}
    {{ pass_form.old_password }}
    {{ pass_form.new_password1 }}
    {{ pass_form.new_password2 }}
    <input type="submit">
</form>
...

3) во вьюхе надо бы как-то с формой взаимодействовать:

class MySettingsChangeView(views.View):
    ...
    def post(self, *args, **kwargs):
        password_form = MyPasswordChangeForm(request.POST or None, user=user)
        if password_form.is_valid():
            self.request.user.set_password(password_form.cleaned_data['new_password1'])
            self.request.user.save()
            return redirect(...)
        return render(
            self.request,
            ...,
            self.get_context_data()
        )

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

скажите пожалуйста а в чем смысл конструктора из формы:

def __init__(self, *args, **kwargs):
        self.user = kwargs.pop('user', None)
        super().__init__(*args, **kwargs)

просто в конструкторе self.user = None

  • Sept. 24, 2020, 2:33 p.m.
  • (edited)

последний вопрос снимаю
выдавал ошибку НО надо было в методе пост представления получить данные из формы вот так:

password_form = MyPasswordChangeForm1(user=request.user, data=request.POST or None)

т.е явно передать user=request.user

и все заработало

единственный вопрос остался, вы комент сделали, что в методе get_context_data из представления при добавлении форм в контекст, надо "# формы надо инициализировать"
вы имеите в виду это:

context.update({
'pass_form': MyPasswordChangeForm1(self.request.POST or None),
'status_form': MyStatusChangeForm1(self.request.POST or None)
})

если так, то нужна ли инициализация форм в контексте если в методе post того же представления рабораю с формой вот так:

password_form = MyPasswordChangeForm1(user=request.user, data=request.POST or None)
......

  • Sept. 24, 2020, 4:42 p.m.
  • (edited)
  • The answer was marked as a solution.

лично я бы сделал так:
в get_context_data я бы генерил основной контекст, в а методах get и post я бы уже добавлял туда формы, правильно инициализированные.
Т.е.

class MySettingsChangeView(views.View):
    def get_context_data(self, *args, **kwargs):
        context = super().get_context_data(*args, **kwargs)
        context.update({
            'pass_form': None,
            'status_form': None
        })
        return context

    def get(self, *args, **kwargs):
        ...
        context = self.get_context_data()
        context['pass_form'] = MyPasswordChangeForm(user=self.request.user)
        context['status_form'] = MyStatusChangeForm()
        return render(self.request, '<template>', context)

    def post(self, *args, **kwargs):
        password_form = MyPasswordChangeForm(self.request.POST or None, user=self.request.user)
        status_form = MyStatusChangeForm(self.request.POST or None)
        if password_form.is_valid():
            ...
        elif status_form.is_valid():
            ...
        context = self.get_context_data()
        context['pass_form'] = password_form
        context['status_form'] = status_form
        return render(self.request, '<template>', context)

таким образом ошибки в форме правильно перенесутся в контектс и правильно можно будет их отобразить

решил я вроде мою задачу, только вот один баг есть и ни как не могу придумать как от него избавится.

вот мое решение:

# views.py

class MySettingsChangeView1(LoginRequiredMixin, TemplateView):
    template_name = 'accounts/registration/settings_pass_status_change.html'

    def get_context_data(self, *args, **kwargs):
        context = super().get_context_data(*args, **kwargs)
        context.update({
            'pass_form': None,
            'status_form': None
        })
        return context

    def get(self, request, *args, **kwargs):
        context = self.get_context_data(**kwargs)
        context['pass_form'] = MyPasswordChangeForm1(user=self.request.user)
        context['status_form'] = MyStatusChangeForm1(user=self.request.user)
        return render(request, self.template_name, context)

    def post(self, request, *args, **kwargs):
        if 'pass' in self.request.POST:
            password_form = MyPasswordChangeForm1(request.POST or None, user=request.user)
            if password_form.is_valid():
                print("Password change")
                self.request.user.set_password(password_form.data['new_password1'])
                self.request.user.save()
                update_session_auth_hash(request, password_form.user)
                return redirect('/')
            else:
                context = self.get_context_data(**kwargs)
                context['pass_form'] = password_form
                context['status_form'] = MyStatusChangeForm1(user=request.user)
                return render(self.request, self.template_name, context)

        if 'status' in self.request.POST:
            status_form = MyStatusChangeForm1(request.POST or None, user=request.user)
            if status_form and status_form.is_valid():
                print("Status change")
                if not request.user.is_active or not request.user.phone_active:
                    return redirect('/')
                else:
                    cont = self.request.POST
                    try:
                        new_status = cont['user_status']
                    except MultiValueDictKeyError:
                        new_status = request.user.user_status
                    curent_user = request.user.pk
                    object = get_object_or_404(User, pk=curent_user)
                    object.user_status = new_status
                    object.save()
                return redirect('/')
            else:
                context = self.get_context_data(**kwargs)
                context['status_form'] = status_form
                context['pass_form'] = MyPasswordChangeForm1(user=request.user)
                return render(self.request, self.template_name, context)

        context = self.get_context_data()
        return render(self.request, self.template_name, context)
# forms.py

class MyPasswordChangeForm1(forms.Form):

    old_password = forms.CharField(strip=False, widget=forms.PasswordInput())
    new_password1 = forms.CharField(strip=False, widget=forms.PasswordInput())
    new_password2 = forms.CharField(strip=False, widget=forms.PasswordInput())

    def __init__(self, *args, **kwargs):
        self.user = kwargs.pop('user', None)
        super().__init__(*args, **kwargs)

    def clean_old_password(self):
        cleaned_data = super().clean()
        if cleaned_data:
            if self.user and not self.user.check_password(cleaned_data['old_password']):
                self.add_error('old_password', 'Current password is not valid')
        return cleaned_data

    def clean(self):
        cleaned_data = super().clean()
        if cleaned_data:
            if ('new_password1' in cleaned_data) and ('new_password2' in cleaned_data):
                if cleaned_data['new_password1'] != cleaned_data['new_password2']:
                    self.add_error('new_password2', 'New password confirmation is not same to new password')
            else:
                print("nu-s valori pu parole")
                # self.add_error('new_password1', 'New password must be')
                raise ValidationError("Please enter data", code='invalid')
        else:
            raise ValidationError("Please enter data", code='invalid')
        return cleaned_data

    def is_valid(self):
        return self.is_bound and not self.errors


class MyStatusChangeForm1(forms.Form):

    user_status = forms.CharField()

    def __init__(self, *args, **kwargs):
        self.user = kwargs.pop('user', None)
        super().__init__(*args, **kwargs)

    def clean(self):
        cleaned_data = super().clean()
        if cleaned_data:
            if not ('user_status' in cleaned_data):
                self.add_error('user_status', 'need check status')
        else:
            raise ValidationError("Please enter data", code='invalid')

        return cleaned_data

    def is_valid(self):
        return self.is_bound and not self.errors

баг в следующем: на хтмл странице 2 формы MyPasswordChangeForm1 , MyStatusChangeForm1
каждая из этих форм имеит свою собственную кнопку сaбмит

если я нажимаю на кнопку сабмит у любой формы НО без того чтобы заполнить форму (то есть формы невалидны) то появляются сообшения об ошибках над обеими формами а по моему мнению сообшение об ошибках должно появлятся только у той формы чей сабмит был нажат

может кто подсказать или направить как решить?

ну вообще, все логично.

Но чтобы они обрабатывались по отдельности - как я уже говорил в первом комменте, лучше было бы сделать 2 отдельные вьюхи. А у каждой формы в аттрибуте action должна быть своя вьюха. так получится и меньше сложность кода и тестируемость и все вот это

тоесть, получилось бы 3 вьюхи:
1. просто рендерит страницу + обратаывает только GET запрос
2. обрабатывает первую форму(вторую форму тоже рендерит без данных) + обрабатывает только POST запрос
3. обрабатывает вторую форму(первую рендерит без данных) + обрабатывает только POST запрос

кстати, не стоит переопределять метод is_valid() в формах

ведь метод is_valid() это метод форм

или вы намекаете что is_valid() можно в представлении переопределить? или его вообше не переопределять?

class MyStatusChangeForm1(forms.Form):

    user_status = forms.CharField()

    def __init__(self, *args, **kwargs):
        self.user = kwargs.pop('user', None)
        super().__init__(*args, **kwargs)

    def clean(self):
        cleaned_data = super().clean()
        if cleaned_data:
            if not ('user_status' in cleaned_data):
                self.add_error('user_status', 'need check status')
        else:
            raise ValidationError("Please enter data", code='invalid')

        return cleaned_data

    def is_valid(self):  # <---- This one
        return self.is_bound and not self.errors

По документации, для дополнительных проверок надо использовать метод clean, is_valid - лучше не трогать

то есть вы советуете создать 2 пары:

1 пара => первая форма + отдельное представление (обработчик) для данной формы
2 пара => вторая форма + отдельное представление (обработчик) для данной формы

плюс к этому, третье представление с методом get куда я просто в context добавляю эти 2 формы

а уже в этих формах в атрибуте action указывать url путь из urls.py к соответствующему представлению которое отвечает за обработку данной формы

тогда я думаю что возникает вопрос-баг: если вдруг просто в браузере отрыть прямой url к одному из этих представлении что обрабатывают формы, что же будет происходить?

ну поставить заглушку на метод get или редирект=)
если очень упрощенно и почти псевдокодом:

class ChangeProfileView(TemplateView):
    template = 'accounts/registration/settings_pass_status_change.html'

    def get_context_data(self, *args, **kwargs):
        context = super().get_context_data(*args, **kwargs)
        context.update({
            'pass_form': MyPasswordChangeForm(),
            'status_form': MyStatusChangeForm(),
        })

class ProcessPasswordForm(ChangeProfileView):
    def get(self, *args, **kwargs):
        return redirect(reverse('<какой там урл у ChangeProfileView>'))

    def post(self, *args, **kwargs):
        form = MyPasswordChangeForm(self.request.POST, user=self.request.user)
        if form.is_valid():
            form.save()
            return redirect(reverse('<какой там урл у ChangeProfileView>'))
        context = self.get_context_data()
        context.update({'pass_form': form})
        return render(self.request, self.template, context)

# для MyStatusChangeForm все аналогично

понятно
спасибо

Comments

Only authorized users can post comments.
Please, Log in or Sign up
Timeweb

Let me recommend you the excellent hosting on which EVILEG is located.

For many years, Timeweb has been proving his stability.

For projects on Django I recommend VDS hosting

View Hosting
l
  • ldim
  • March 4, 2021, 10:57 p.m.

C ++ - Test 004. Pointers, Arrays and Loops

  • Result:100points,
  • Rating points10
l
  • ldim
  • March 4, 2021, 10:39 p.m.

C ++ - Test 004. Pointers, Arrays and Loops

  • Result:80points,
  • Rating points4
E

C ++ - Test 004. Pointers, Arrays and Loops

  • Result:80points,
  • Rating points4
Last comments
D

Django - Tutorial 001. Deploying a site on Django + PostgreSQL + Gunicorn + Nginx

А почему нельзя? Где можно об этом почитать? Киньте, пожалуйста, в меня ссылкой.
D

Django - Tutoral 049. Optimizing Django Performance with a Real Project

Огромное спасибо вам за статью! Для меня стали открытием select_related и prefetch_related
t
  • t1m4
  • Feb. 24, 2021, 1:56 p.m.

Django - Tutorial 052. Redefining a User Model

В данном случае я заходил под superuser но все равно не появлялись эти поля

Django - Tutorial 001. Deploying a site on Django + PostgreSQL + Gunicorn + Nginx

Поднял сервис с помощью systemd, вот по этому мануалу: https://habr.com/ru/post/501414/
t
  • t1m4
  • Feb. 23, 2021, 6:11 p.m.

Django - Tutorial 052. Redefining a User Model

А как дать ему эти права?
Now discuss on the forum
M

как добавить списки в список, которые созданы динамически

Привет всем! Смотрите новое: https://cazino.website/luchshie-kazino/ лучшие онлайн казино россии - За регистрацию предоставляют до 225 фриспинов бесплатно + 100% бонуса до 500$ на первый д…

Не отдаётся статика на виртуальной машине

В итоге выставил 775 на все файлы проекта и всё заработало. Но я не уверен, что так правильно.
P
  • Pisych
  • March 3, 2021, 12:06 p.m.

Вывод данных из связанных таблиц.

Не знаю, правильно ли... Но работает <body>{% for i in CatList %} <ul> <li><h3>{{i.title}}</h3></li>{% for j in ProdList %}<ul>{…
M

нужна помощь с проектом qt c++

дали задание, на несколько дней. собираюсь делать, но хотел бы получить помощь более опытных людей, чтобы подкорректировать себя и подстраховаться. каким образом это можно сделать? …

QScrollArea dynamically add QCheckBoxes

Всё правильно. Это просто спамер, который отправился в вечный бан.
About
Services
© EVILEG 2015-2020
Recommend hosting TIMEWEB