change forms,models,views with bs5

Author: gordienkoas

db.sqlite3

@@ -1,8 +1,11 @@
 from django import forms
 from .models import Label
+from django.utils.translation import gettext_lazy as _
 
 
-class LabelForm(forms.ModelForm):
+class CreateLabelForm(forms.ModelForm):
+    name = forms.CharField(max_length=40, label=_("Name"))
+
     class Meta:
         model = Label
-        fields = ['name']
+        fields = ("name",)

task_manager/labels/forms.py

@@ -1,10 +1,11 @@
 from django.urls import path
-from task_manager.labels.views import LabelListView, LabelCreateView, LabelUpdateView, LabelDeleteView
+from .views import LabelView, CreateLabelView, UpdateLabelView, DeleteLabelView
 
+app_name = "labels"
 
 urlpatterns = [
-    path('labels/', LabelListView.as_view(), name='label-list'),
-    path('labels/create/', LabelCreateView.as_view(), name='label-create'),
-    path('labels/<int:pk>/update/', LabelUpdateView.as_view(), name='label-update'),
-    path('labels/<int:pk>/delete/', LabelDeleteView.as_view(), name='label-delete'),
+    path("", LabelView.as_view(), name="label_list"),
+    path("create/", CreateLabelView.as_view(), name="create_label"),
+    path("<int:pk>/delete/", DeleteLabelView.as_view(), name="delete_label"),
+    path("<int:pk>/update/", UpdateLabelView.as_view(), name="update_label"),
 ]

task_manager/labels/urls.py

@@ -1,59 +1,41 @@
-from django.contrib.auth.mixins import LoginRequiredMixin
-from django.views.generic import ListView, CreateView, UpdateView, DeleteView
-from django.urls import reverse_lazy, reverse
-from django.contrib import messages
+# from django.shortcuts import render
 from .models import Label
-from .forms import LabelForm
-from django.shortcuts import redirect
-from django.contrib.messages.views import SuccessMessageMixin
+from .forms import CreateLabelForm
 
+from django.views.generic import ListView, CreateView, UpdateView, DeleteView
+from django.contrib.messages.views import SuccessMessageMixin
+from django.contrib.auth.mixins import LoginRequiredMixin
+from django.urls import reverse_lazy
+from django.utils.translation import gettext_lazy as _
 
-class LabelListView(LoginRequiredMixin, ListView):
-    model = Label
-    template_name = 'labels/label_list.html'
-    context_object_name = 'labels'
 
-class LabelCreateView(LoginRequiredMixin, CreateView):
+class LabelView(ListView):
     model = Label
-    form_class = LabelForm
-    template_name = 'labels/label_form.html'
-    success_url = reverse_lazy('label-list')
-
-    def form_valid(self, form):
-        messages.success(self.request, "Метка успешно создана")
-        return super().form_valid(form)
-
-
-# class LabelListView(LoginRequiredMixin, ListView):
-#     model = Label
-#     template_name = 'labels/label_list.html'
-#     context_object_name = 'labels'
+    context_object_name = "label_list"
+    template_name = "labels/label_list.html"
 
 
-# class LabelCreateView(LoginRequiredMixin, SuccessMessageMixin, CreateView):
-#     model = Label
-#     form_class = LabelForm
-#     template_name = 'labels/label_form.html'
-#     success_url = reverse_lazy('label-list')
-#     success_message = "Метка успешно создана"
+class CreateLabelView(SuccessMessageMixin, CreateView):
+    template_name = "labels/create.html"
+    success_url = reverse_lazy("labels:label_list")
+    form_class = CreateLabelForm
+    success_message = _("Label created successfully")
 
 
-class LabelUpdateView(LoginRequiredMixin, SuccessMessageMixin, UpdateView):
+class UpdateLabelView(LoginRequiredMixin, SuccessMessageMixin, UpdateView):
     model = Label
-    form_class = LabelForm
-    template_name = 'labels/label_form.html'
-    success_url = reverse_lazy('label-list')
-    success_message = "Метка успешно изменена"
+    success_url = reverse_lazy("labels:label_list")
+    template_name = "labels/update.html"
+    form_class = CreateLabelForm
+    success_message = _("Label updated successfully")
+    login_url = reverse_lazy("labels:label_list")
+    redirect_field_name = None
 
 
-class LabelDeleteView(LoginRequiredMixin, DeleteView):
+class DeleteLabelView(LoginRequiredMixin, SuccessMessageMixin, DeleteView):
     model = Label
-    template_name = 'labels/label_confirm_delete.html'
-    success_url = reverse_lazy('label-list')
-
-    def form_valid(self, form):
-        if self.object.tasks.exists():
-            messages.error(self.request, "Нельзя удалить метку, связанную с задачами")
-            return redirect(reverse('label-list'))
-        messages.success(self.request, "Метка успешно удалена")
-        return super().form_valid(form)
+    success_url = reverse_lazy("labels:label_list")
+    template_name = "labels/delete.html"
+    success_message = _("Label deleted successfully")
+    login_url = reverse_lazy("login")
+    redirect_field_name = None

task_manager/labels/views.py

@@ -0,0 +1,37 @@
+from django.shortcuts import redirect
+from django.urls import reverse
+from django.contrib import messages
+from django.utils.translation import gettext_lazy as _
+from django.contrib.auth.mixins import LoginRequiredMixin
+
+
+class LoginRequiredRedirectMixin(LoginRequiredMixin):
+    message_not_authenticated = _(
+        'You are not login. Please, login')
+    redirect_url_name = 'login'
+
+    def handle_no_permission(self):
+        messages.warning(self.request, self.message_not_authenticated)
+        return redirect(reverse(self.redirect_url_name))
+
+
+class ObjectUnusedRequaredMixin:
+    '''
+    Check object's usage in some task.
+    If is used - redirect with message.
+    Attrs: message_used_object: str, url_name_object_used,
+    message_success: str, url_name_success: str
+    '''
+    message_used_object = _('Unable to delete because it is used in task!')
+    url_name_object_used = "main_page"
+    message_object_has_not_attr = _(
+        'Enternal error: Unable to delete object!')
+    report_message = _("Object has not attr is_object_in_use")
+
+    def form_valid(self, form):
+        if not hasattr(self.object, 'is_object_in_use'):
+            return super().form_valid(form)
+        if self.object.is_object_in_use():
+            messages.warning(self.request, self.message_used_object)
+            return redirect(reverse(self.url_name_object_used))
+        return super().form_valid(form)

task_manager/mixins.py

@@ -99,7 +99,8 @@
 
 WSGI_APPLICATION = 'task_manager.wsgi.application'
 
-LOGOUT_REDIRECT_URL = '/'
+LOGIN_REDIRECT_URL = "index"
+LOGOUT_REDIRECT_URL = "index"
 
 
 

task_manager/settings.py

@@ -1,10 +1,11 @@
 from django import forms
-from task_manager.statuses.models import Status
+from .models import Status
+from django.utils.translation import gettext_lazy as _
 
 
-class StatusForm(forms.ModelForm):
+class CreateStatusForm(forms.ModelForm):
+    name = forms.CharField(max_length=30, label=_("Name"))
+
     class Meta:
         model = Status
-        fields = ['name']
-
-
+        fields = ("name",)

task_manager/statuses/forms.py

@@ -1,11 +1,11 @@
 from django.urls import path
-from task_manager.statuses.views import StatusListView, StatusCreateView, StatusUpdateView, StatusDeleteView
+from .views import StatusView, CreateStatusView, UpdateStatusView, DeleteStatusView
 
+app_name = "statuses"
 
 urlpatterns = [
-    path('statuses/', StatusListView.as_view(), name='status-list'),
-    path('statuses/create/', StatusCreateView.as_view(), name='status-create'),
-    path('statuses/<int:pk>/update/', StatusUpdateView.as_view(), name='status-update'),
-    path('status/update/<int:pk>/', StatusUpdateView.as_view(), name='status-update'),
-    path('statuses/<int:pk>/delete/', StatusDeleteView.as_view(), name='status-delete'),
+    path("", StatusView.as_view(), name="status_list"),
+    path("create/", CreateStatusView.as_view(), name="create_status"),
+    path("<int:pk>/delete/", DeleteStatusView.as_view(), name="delete_status"),
+    path("<int:pk>/update/", UpdateStatusView.as_view(), name="update_status"),
 ]

task_manager/statuses/urls.py

@@ -1,53 +1,48 @@
-from django.contrib.auth.mixins import LoginRequiredMixin
-from django.views.generic import ListView, CreateView, UpdateView, DeleteView
+# from django.shortcuts import render
 from django.urls import reverse_lazy
-from django.contrib import messages
-from task_manager.statuses.models import Status
-from task_manager.statuses.forms import StatusForm
+from django.views.generic import ListView, CreateView, UpdateView, DeleteView
+from django.contrib.messages.views import SuccessMessageMixin
+from django.contrib.auth.mixins import LoginRequiredMixin
+from .models import Status
+from django.utils.translation import gettext_lazy as _
 
-class StatusListView(LoginRequiredMixin, ListView):
-    model = Status
-    template_name = 'statuses/status_list.html'
+from .forms import CreateStatusForm
 
-class StatusCreateView(LoginRequiredMixin, CreateView):
-    model = Status
-    form_class = StatusForm
-    template_name = 'statuses/status_form.html'
-
-    def form_valid(self, form):
-        messages.success(self.request, 'Статус успешно создан.')
-        return super().form_valid(form)
-
-# class StatusListView(LoginRequiredMixin, ListView):
-#      model = Status
-#      template_name = 'statuses/status_list.html'
-
-# class StatusCreateView(LoginRequiredMixin, CreateView):
-#      model = Status
-#      form_class = StatusForm
-#      template_name = 'statuses/status_form.html'
-
-     # def form_valid(self, form):
-     #     messages.success(self.request, 'Статус успешно создан.')
-     #     return super().form_valid(form)
-     #
-     # def get_success_url(self):
-     #     return reverse_lazy('status-list')
-
-class StatusUpdateView(LoginRequiredMixin, UpdateView):
+
+class StatusView(ListView):
     model = Status
-    form_class = StatusForm
-    template_name = 'statuses/status_form.html'
-    success_url = reverse_lazy('status-list')
+    context_object_name = "status_list"
+    template_name = "statuses/statuses_list.html"
 
-    def form_valid(self, form):
-         messages.success(self.request, 'Статус успешно обновлен.')
-         return super().form_valid(form)
 
-class StatusDeleteView(LoginRequiredMixin, DeleteView):
-     model = Status
-     template_name = 'statuses/status_confirm_delete.html'
+class CreateStatusView(SuccessMessageMixin, CreateView):
+    template_name = "statuses/create.html"
+    success_url = reverse_lazy("statuses:status_list")
+    form_class = CreateStatusForm
+    success_message = _("Status created successfully")
 
-     def delete(self, request, *args, **kwargs):
-         messages.success(self.request, 'Статус успешно удален.')
-         return super().delete(request, *args, **kwargs)
+    # change dublication in templates and add translate in views
+    # def get_context_data(self, **kwargs):
+    #     context = super().get_context_data(**kwargs)
+    #     context["to_do"] = "create"
+    #     context["status"] = "Status"
+    #     return context
+
+
+class UpdateStatusView(LoginRequiredMixin, SuccessMessageMixin, UpdateView):
+    model = Status
+    success_url = reverse_lazy("statuses:status_list")
+    template_name = "statuses/update.html"
+    form_class = CreateStatusForm
+    success_message = _("Status updated successfully")
+    login_url = reverse_lazy("statuses:status_list")
+    redirect_field_name = None
+
+
+class DeleteStatusView(LoginRequiredMixin, SuccessMessageMixin, DeleteView):
+    model = Status
+    success_url = reverse_lazy("statuses:status_list")
+    template_name = "statuses/delete.html"
+    success_message = _("Status deleted successfully")
+    login_url = reverse_lazy("login")
+    redirect_field_name = None

task_manager/statuses/views.py

@@ -1,39 +1,35 @@
 import django_filters
-from django import forms
-from django.contrib.auth import get_user_model
-from .models import Task, Label
+from django_filters import ModelChoiceFilter, BooleanFilter
+from django.forms import CheckboxInput
+
+from task_manager.labels.models import Label
+from task_manager.tasks.models import Task
+
+# from django.utils.translation import gettext_lazy as _
 
-User = get_user_model()
 
 class TaskFilter(django_filters.FilterSet):
-    status = django_filters.ChoiceFilter(label='Статус', empty_label='Любой')
-    executor = django_filters.ModelChoiceFilter(
-        queryset=User.objects.all(),
-        label='Исполнитель',
-        empty_label='Любой',
-    )
-    labels = django_filters.ModelMultipleChoiceFilter(
+    labels = ModelChoiceFilter(
+        field_name="labels",
         queryset=Label.objects.all(),
-        label='Метки',
-        widget=forms.CheckboxSelectMultiple,
+        label="Метка",
     )
-    author = django_filters.BooleanFilter(
-        method='filter_author',
-        label='Только мои задачи',
-        widget=forms.CheckboxInput,
+    self_tasks = BooleanFilter(
+        method="filter_tasks",
+        widget=CheckboxInput,
+        label="Только свои задачи",
     )
 
-    class Meta:
-        model = Task
-        fields = ['status', 'executor', 'labels']
-
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-        statuses = Task.objects.values_list('status', flat=True).distinct()
-        choices = [(status, status) for status in statuses if status]
-        self.filters['status'].extra['choices'] = choices
-
-    def filter_author(self, queryset, name, value):
+    def filter_tasks(self, queryset, name, value):
         if value:
             return queryset.filter(author=self.request.user)
         return queryset
+
+    class Meta:
+        model = Task
+        fields = (
+            "status",
+            "executor",
+            "labels",
+            "self_tasks",
+        )