update lint

Author: pilgrim-nord

task_manager/users/forms.py

@@ -3,75 +3,98 @@
 from django.contrib.auth.models import User
 from django.core.exceptions import ValidationError
 
-# Константы — SonarCloud их не трогает
-USERNAME_FIELD = 'username'
-PASSWORD1_FIELD = 'password1'
-PASSWORD2_FIELD = 'password2'
-
 
 class UserRegistrationForm(UserCreationForm):
     first_name = forms.CharField(
         required=True,
         label='Имя',
-        widget=forms.TextInput(attrs={'class': 'form-control'}),
-        error_messages={'required': 'Обязательное поле.'}
+        widget=forms.TextInput(attrs={
+            'class': 'form-control',
+        }),
+        error_messages={
+            'required': 'Обязательное поле.',
+        }
     )
     last_name = forms.CharField(
         required=True,
         label='Фамилия',
-        widget=forms.TextInput(attrs={'class': 'form-control'}),
-        error_messages={'required': 'Обязательное поле.'}
+        widget=forms.TextInput(attrs={
+            'class': 'form-control',
+        }),
+        error_messages={
+            'required': 'Обязательное поле.',
+        }
     )
 
     class Meta:
         model = User
-        fields = ("first_name", "last_name", USERNAME_FIELD, PASSWORD1_FIELD, PASSWORD2_FIELD)
+        fields = (
+            "first_name", "last_name", "username", "password1", "password2"
+        )
         labels = {
-            USERNAME_FIELD: 'Имя пользователя',
-            PASSWORD1_FIELD: 'Пароль',
-            PASSWORD2_FIELD: 'Подтверждение пароля',
+            'username': 'Имя пользователя',
+            'password1': 'Пароль',
+            'password2': 'Подтверждение пароля',
         }
         help_texts = {
-            USERNAME_FIELD: 'Обязательное поле. Не более 150 символов. Только буквы, цифры и @/./+/-/_',
-            PASSWORD1_FIELD: 'Ваш пароль должен содержать как минимум 3 символа.',
-            PASSWORD2_FIELD: 'Для подтверждения введите пароль ещё раз.',
+            'username': (
+                'Обязательное поле. Не более 150 символов. '
+                'Только буквы, цифры и символы @/./+/-/_.'
+            ),
+            'password1': 'Ваш пароль должен содержать как минимум 3 символа.',
+            'password2': 'Для подтверждения введите, пожалуйста, '
+                           'пароль ещё раз.',
         }
         error_messages = {
-            USERNAME_FIELD: {
+            'username': {
                 'required': 'Обязательное поле.',
                 'unique': 'Пользователь с таким именем уже существует.',
                 'max_length': 'Не более 150 символов.',
             },
-            PASSWORD1_FIELD: {'required': 'Обязательное поле.'},
-            PASSWORD2_FIELD: {'required': 'Обязательное поле.'},
+            'password1': {
+                'required': 'Обязательное поле.',
+            },
+            'password2': {
+                'required': 'Обязательное поле.',
+            },
+        }
+        widgets = {
+            'username': forms.TextInput(attrs={
+                'class': 'form-control',
+            }),
+            'password1': forms.PasswordInput(attrs={
+                'placeholder': 'Пароль',
+                'class': 'form-control',
+                'autocomplete': 'new-password',
+            }),
+            'password2': forms.PasswordInput(attrs={
+                'class': 'form-control',
+                'autocomplete': 'new-password',
+            }),
         }
-        # УБРАЛИ widgets отсюда — они больше не нужны
-        # widgets = { ... }  ← удалить весь блок!
 
     def __init__(self, *args, **kwargs):
+        # 1. Сначала вызываем родительский конструктор
         super().__init__(*args, **kwargs)
 
-        # Теперь безопасно обращаемся по константам
-        self.fields[USERNAME_FIELD].help_text = (
+        # 2. Затем переопределяем help_text для нужных полей
+        self.fields['username'].help_text = (
             'Обязательное поле. Не более 150 символов. '
             'Только буквы, цифры и символы @/./+/-/_.'
         )
-        self.fields[PASSWORD1_FIELD].widget.attrs.update({
-            'placeholder': 'Пароль',
-            'class': 'form-control',
-            'autocomplete': 'new-password',
-        })
-        self.fields[PASSWORD2_FIELD].widget.attrs.update({
-            'class': 'form-control',
-            'autocomplete': 'new-password',
-        })
+        self.fields['password1'].help_text = (
+            'Ваш пароль должен содержать как минимум 3 символа.'
+        )
+        self.fields['password2'].help_text = (
+            'Для подтверждения введите, пожалуйста, пароль ещё раз.'
+        )
 
     def clean_password2(self):
-        p1 = self.cleaned_data.get(PASSWORD1_FIELD)
-        p2 = self.cleaned_data.get(PASSWORD2_FIELD)
-        if p1 and p2 and p1 != p2:
+        password1 = self.cleaned_data.get("password1")
+        password2 = self.cleaned_data.get("password2")
+        if password1 and password2 and password1 != password2:
             raise ValidationError("Два поля пароля не совпадают.")
-        return p2
+        return password2
 
     def save(self, commit=True):
         user = super().save(commit=False)
@@ -83,23 +106,24 @@ def save(self, commit=True):
 
 
 class UserUpdateForm(forms.ModelForm):
+    # Добавляем поля пароля (необязательные)
     password1 = forms.CharField(
         label='Пароль',
-        widget=forms.PasswordInput(attrs={
-            'placeholder': 'Пароль',
-            'autocomplete': 'new-password'
-        }),
-        required=False,
+        widget=forms.PasswordInput(
+            attrs={'placeholder': 'Пароль', 'autocomplete': 'new-password'}
+        ),
+        required=True,
+        error_messages={'required': 'Обязательное поле'},
         help_text='Оставьте пустым, если не хотите менять пароль.',
     )
     password2 = forms.CharField(
         label='Подтверждение пароля',
-        widget=forms.PasswordInput(attrs={
-            'class': 'form-control',
-            'autocomplete': 'new-password'
-        }),
-        required=False,
-        help_text='Введите пароль ещё раз',
+        widget=forms.PasswordInput(
+            attrs={'class': 'form-control', 'autocomplete': 'new-password'}
+        ),
+        required=True,
+        error_messages={'required': 'Обязательное поле'},
+        help_text='Подтверждение пароля',
     )
 
     class Meta:
@@ -119,17 +143,21 @@ class Meta:
         }
 
     def clean(self):
-        cleaned = super().clean()
-        p1 = cleaned.get('password1')
-        p2 = cleaned.get('password2')
-        if p1 and p1 != p2:
+        cleaned_data = super().clean()
+        password1 = cleaned_data.get('password1')
+        password2 = cleaned_data.get('password2')
+        if password1 != password2:
             raise ValidationError('Пароли не совпадают')
-        return cleaned
+        return cleaned_data
 
     def save(self, commit=True):
         user = super().save(commit=False)
-        if self.cleaned_data['password1']:
-            user.set_password(self.cleaned_data['password1'])
+
+        # Обновляем пароль, только если он указан
+        password1 = self.cleaned_data.get('password1')
+        if password1:
+            user.set_password(password1)
+
         if commit:
             user.save()
         return user