add profile edit

pavelprokhorenko · pavelprokhorenko · commit d1380ae5471d · 2022-01-04T13:06:08.000+05:00
diff --git a/post/forms.py b/post/forms.py
@@ -1,14 +1,12 @@
-import re
-
 from django import forms
 from django.contrib.auth.models import User
-from django.core.validators import EmailValidator
 
 from .validators import LetterUsernameValidator
 
 
 class LoginForm(forms.Form):
-    username = forms.CharField(widget=forms.TextInput)
+    """Форма авторизации"""
+    username = forms.CharField()
     password = forms.CharField(widget=forms.PasswordInput)
 
     def __init__(self, *args, **kwargs):
@@ -29,6 +27,7 @@ def clean(self):
 
 
 class RegistrationForm(forms.ModelForm):
+    """Форма регистрации"""
     username_validator = LetterUsernameValidator()
     username = forms.CharField(
         widget=forms.TextInput,
@@ -72,3 +71,30 @@ def clean(self):
     class Meta:
         model = User
         fields = ('username', 'password', 'confirm_password', 'email', 'first_name', 'last_name')
+
+
+class EditProfileForm(RegistrationForm):
+    """Форма для изменения данных в профиле"""
+    bio = forms.CharField(widget=forms.Textarea, required=False, max_length=300)
+
+    def __init__(self, *args, **kwargs):
+        self.user = kwargs.pop('user', None)
+        super().__init__(*args, **kwargs)
+        self.fields['bio'].label = 'Краткая информация'
+
+    def clean_username(self):
+        username = self.cleaned_data['username']
+        if User.objects.filter(username=username).exists():
+            if User.objects.get(username=username) != self.user:
+                raise forms.ValidationError(f'Пользователь с логином {username} уже существует.')
+        return username
+
+    def clean_email(self):
+        email = self.cleaned_data['email']
+        if User.objects.filter(email=email).exists():
+            if User.objects.get(email=email) != self.user:
+                raise forms.ValidationError(f'Email {email} уже используется в системе.')
+        return email
+
+    class Meta(RegistrationForm.Meta):
+        fields = ('username', 'password', 'confirm_password', 'email', 'first_name', 'last_name', 'bio')
diff --git a/post/templates/post/base.html b/post/templates/post/base.html
@@ -52,6 +52,7 @@
 
 
 
+
     </style>
     <link href="cover.css" rel="stylesheet">
 </head>
@@ -98,7 +99,7 @@ <h1 class="me-4 pe-7">
         {% if messages %}
         <br>
         {% for message in messages %}
-        <div class="alert alert-success alert-dismissible fade show" role="alert"
+        <div class="alert alert-{{ message.tags }} alert-dismissible fade show" role="alert"
              style="margin-left: 450px; width: 900px;">
             <strong>{{ message }}</strong>
         </div>
@@ -126,7 +127,8 @@ <h1 class="me-4 pe-7">
             {% for post in posts %}
             <div class="card text-center">
                 <div class="card-header">
-                    <a href="{{ post.author.get_absolute_url }}" class="text-decoration-none text-dark" style="font-weight: 600">
+                    <a href="{{ post.author.get_absolute_url }}" class="text-decoration-none text-dark"
+                       style="font-weight: 600">
                         <img src="{{ post.author.avatar_thumbnail.url }}" alt="" width="32" height="32"
                              class="rounded-circle me-2">{{ post.author.user.username }}
                     </a>
diff --git a/post/templates/post/profile.html b/post/templates/post/profile.html
@@ -1,17 +1,17 @@
 {% extends 'post/base.html' %}
 {% block title %}
-{{ profile.user.username }} | Ukiblog
+{{ main_profile.user.username }} | Ukiblog
 {% endblock %}
 {% block categories %}
 {% endblock %}
 {% block main %}
 <div class="card" style="width: 800px; margin: 100px 600px 300px 500px; padding: 30px 30px">
     <div class="card-body">
         <span class="text-start fs-1">
-            <img src="{{ profile.avatar_thumbnail.url }}" alt="" width="100" height="100" class="rounded-circle me-2">
-            <a class="ms-2 text-decoration-none">@{{ profile.user.username }}</a>
-            {% if user == profile.user %}
-            <a class="text-decoration-none">
+            <img src="{{ main_profile.avatar_thumbnail.url }}" alt="" width="100" height="100" class="rounded-circle me-2">
+            <a class="ms-2 text-decoration-none">@{{ main_profile.user.username }}</a>
+            {% if user == main_profile.user %}
+            <a href="{% url 'post:edit_profile' slug=main_profile.slug %}" class="text-decoration-none">
                 <button type="button" class="btn btn-outline-primary btn-lg" style="margin-left: 200px">
                     Настройки профиля
                 </button>
@@ -26,28 +26,28 @@
             <tbody>
             <tr>
                 <th scope="row">Настоящее имя</th>
-                {% if profile.user.first_name %}
-                <td>{{ profile.user.first_name }}</td>
+                {% if main_profile.user.first_name %}
+                <td>{{ main_profile.user.first_name }}</td>
                 {% else %}
                 <td>&mdash;</td>
                 {% endif %}
             </tr>
             <tr>
                 <th scope="row">Настоящая фамилия</th>
-                {% if profile.user.last_name %}
-                <td>{{ profile.user.last_name }}</td>
+                {% if main_profile.user.last_name %}
+                <td>{{ main_profile.user.last_name }}</td>
                 {% else %}
                 <td>&mdash;</td>
                 {% endif %}
             </tr>
             <tr>
                 <th scope="row">Email</th>
-                <td>{{ profile.user.email }}</td>
+                <td>{{ main_profile.user.email }}</td>
             </tr>
             <tr>
                 <th scope="row">Статус</th>
                 <td>
-                    {% if profile.author %}
+                    {% if main_profile.author %}
                     Автор
                     {% else %}
                     Читатель
@@ -57,8 +57,8 @@
             </tbody>
         </table>
         <p class="fs-5"><strong class="text-secondary">Информация о пользователе:</strong>
-        {% if profile.bio %}
-        {{ profile.bio }}
+        {% if main_profile.bio %}
+        {{ main_profile.bio }}
         {% else %}
         отсутствует
         {% endif %}
@@ -77,10 +77,10 @@
         </ul>
         {% endfor %}
         {% else %}
-        <p class="text-center fs-3 text-secondary">Вы не опубликовали ни одну статью :(</p>
+        <p class="text-center fs-3 text-secondary">Публикации отсутствуют :(</p>
         {% endif %}
-        {% if user == profile.user %}
-        {% if profile.author %}
+        {% if user == main_profile.user %}
+        {% if main_profile.author %}
         <button type="button" class="btn btn-outline-primary btn-lg mt-5" style="margin-left: 200px">
             <a href="#" class="text-center text-decoration-none text-dark fs-5 mt-3">
                 Опубликовать новую статью
diff --git a/post/templates/post/profile_settings.html b/post/templates/post/profile_settings.html
@@ -0,0 +1,30 @@
+{% extends 'post/base.html' %}
+{% load crispy_forms_tags %}
+{% block title %}
+{{ profile.user.username }} | Ukiblog
+{% endblock %}
+{% block categories %}
+{% endblock %}
+{% block main %}
+<div class="card" style="width: 800px; margin: 100px 600px 300px 500px; padding: 30px 30px">
+    <div class="card-body">
+        <span class="text-start fs-1">
+            <img src="{{ profile.avatar_thumbnail.url }}" alt="" width="100" height="100" class="rounded-circle me-2">
+            <a class="ms-2 text-decoration-none">@{{ profile.user.username }}</a>
+        </span>
+        <hr>
+        <p class="text-secondary fs-5 text-center"><strong>Настройки профиля</strong></p>
+        <hr>
+        <div style="width: 705px">
+            <form action="" method="post">
+            {% csrf_token %}
+            {{ form|crispy }}
+            <br>
+            <div class="d-grid gap-2 col-6 mx-auto">
+                <input type="submit" class="btn btn-success btn-lg" value="Обновить данные">
+            </div>
+        </form>
+        </div>
+    </div>
+</div>
+{% endblock %}
diff --git a/post/urls.py b/post/urls.py
@@ -8,7 +8,8 @@
     TagDetailView,
     LoginView,
     RegistrationView,
-    ProfileView
+    ProfileView,
+    EditProfileView,
 )
 
 app_name = 'post'
@@ -20,4 +21,5 @@
     path('logout', LogoutView.as_view(next_page=reverse_lazy('post:base')), name='logout'),
     path('registration', RegistrationView.as_view(), name='registration'),
     path('profile/<slug:slug>', ProfileView.as_view(), name='profile'),
+    path('profile/<slug:slug>/edit', EditProfileView.as_view(), name='edit_profile'),
 ]
diff --git a/post/views.py b/post/views.py
@@ -1,18 +1,18 @@
 from django.shortcuts import render, redirect
 from django.urls import reverse
 from django.utils.text import slugify
-from django.views import View
-from django.views.generic.list import ListView
-from django.views.generic.detail import DetailView
+from django.views import View, generic
 from django.contrib.auth import login, authenticate
+from django.contrib.auth.models import AnonymousUser, User
 from django.contrib import messages
+from django.forms.models import model_to_dict
 
 from .models import Post, Tag, Profile
 from .mixins import DefaultContextMixin
-from .forms import LoginForm, RegistrationForm
+from .forms import LoginForm, RegistrationForm, EditProfileForm
 
 
-class BaseView(DefaultContextMixin, ListView):
+class BaseView(DefaultContextMixin, generic.ListView):
     """Основная страница"""
     model = Post
     template_name = 'post/base.html'
@@ -22,15 +22,15 @@ def get_context_data(self, **kwargs):
         return super().get_context_data(self.request, **kwargs)
 
 
-class PostDetailView(DefaultContextMixin, DetailView):
+class PostDetailView(DefaultContextMixin, generic.DetailView):
     """Отдельная статья"""
     model = Post
 
     def get_context_data(self, **kwargs):
         return super().get_context_data(self.request, **kwargs)
 
 
-class TagDetailView(DefaultContextMixin, DetailView):
+class TagDetailView(DefaultContextMixin, generic.DetailView):
     """Отдельный tag"""
     model = Tag
     context_object_name = 'main_tag'
@@ -106,13 +106,62 @@ def post(self, *args, **kwargs):
         return render(self.request, 'post/registration.html', context)
 
 
-class ProfileView(DetailView):
+class ProfileView(DefaultContextMixin, generic.DetailView):
     """Подробная информация о пользователе"""
     model = Profile
     template_name_suffix = ''
+    context_object_name = 'main_profile'
 
     def get_context_data(self, **kwargs):
-        context = super().get_context_data(**kwargs)
+        context = super().get_context_data(self.request, **kwargs)
         context['user'] = self.request.user
-        context['articles'] = Profile.objects.get(user=context['profile'].user).post_set.order_by('pub_date')
+        context['articles'] = Profile.objects.get(user=context['main_profile'].user).post_set.order_by('pub_date')
         return context
+
+
+class EditProfileView(View):
+    """Настройки профиля"""
+    def get(self, *args, **kwargs):
+        if isinstance(self.request.user, AnonymousUser):
+            messages.warning(self.request, 'У вас недостаточно прав!')
+            return redirect(reverse('post:base'))
+        elif Profile.objects.get(user=self.request.user) != Profile.objects.get(slug=kwargs['slug']):
+            messages.warning(self.request, 'Вы можете редактировать только свой профиль!')
+            return redirect(reverse('post:base'))
+        profile = Profile.objects.get(slug=kwargs['slug'])
+        data = model_to_dict(profile.user)
+        data.update(bio=profile.bio)
+        form = EditProfileForm(initial=data)
+        context = {
+            'profile': profile,
+            'form': form,
+        }
+        return render(self.request, 'post/profile_settings.html', context)
+
+    def post(self, *args, **kwargs):
+        profile = Profile.objects.get(slug=kwargs['slug'])
+        user = profile.user
+        form = EditProfileForm(self.request.POST, user=user)
+        if form.is_valid():
+            User.objects.filter(pk=user.pk).update(
+                username=form.cleaned_data['username'],
+                email=form.cleaned_data['email'],
+                first_name=form.cleaned_data['first_name'].title(),
+                last_name=form.cleaned_data['last_name'].title(),
+            )
+            user.set_password(form.cleaned_data['password'])
+            user.save(update_fields=['password'])
+            Profile.objects.filter(user=user).update(bio=form.cleaned_data['bio'])
+            messages.success(
+                self.request,
+                f'Ваш профиль успешно обновлен с:'
+            )
+            user = authenticate(username=form.cleaned_data['username'], password=form.cleaned_data['password'])
+            login(self.request, user)
+            return redirect(reverse('post:base'))
+        context = {
+            'profile': profile,
+            'form': form,
+        }
+        return render(self.request, 'post/profile_settings.html', context)
+

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,8 @@`
`8`	`8`	`TagDetailView,`
`9`	`9`	`LoginView,`
`10`	`10`	`RegistrationView,`
`11`		`- ProfileView`
	`11`	`+ ProfileView,`
	`12`	`+ EditProfileView,`
`12`	`13`	`)`
`13`	`14`
`14`	`15`	`app_name = 'post'`
`@@ -20,4 +21,5 @@`
`20`	`21`	`path('logout', LogoutView.as_view(next_page=reverse_lazy('post:base')), name='logout'),`
`21`	`22`	`path('registration', RegistrationView.as_view(), name='registration'),`
`22`	`23`	`path('profile/<slug:slug>', ProfileView.as_view(), name='profile'),`
	`24`	`+ path('profile/<slug:slug>/edit', EditProfileView.as_view(), name='edit_profile'),`
`23`	`25`	`]`