malyshevadv/api_yamdb

api_yamdb

api_yamdb's Issues

api/urls.py - поправить url

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/api/urls.py

router.register(r'titles', TitleViewSet, basename='titles')
r нужны для регулярок, давай уберём их оттуда, где их нет

reviews/admin.py - добавить возможностей в админку

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/reviews/admin.py

Давайте побольше в админке возможностей сделаем)

api/serializers.py - добавить f-строку

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/api/serializers.py

def validate_username(self, value):
if value in self.BANED_USERNAMES:
raise serializers.ValidationError(
'Использовать имя "{}" в качестве username запрещено.'.format(

А почему не f-строкой? :(

api/views.py - поправить if serializer.is_valid() в get_token

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/api/views.py

@api_view(['POST']) 
@permission_classes([AllowAny]) 
def get_token(request): 
    """Получение JWT-токена. 
 
    Если полученный код подтверждения валиден - отправляет юзеру JWT-токен. 
    """ 
    serializer = TokenSerializer(data=request.data) 
 
    if serializer.is_valid():

Тут тоже стоит добавить исключение и проверять обратное условие

api/views.py - перенести email в настройки

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/api/views.py

def send_confirmation(user):
"""Отправляет письмо пользователю."""
confirmation_code = default_token_generator.make_token(user)

send_mail( 
    'Email confirmation', 
    f'Ваш код для подтверждения почты: {confirmation_code}', 
    '[email protected]', 

Стоит этот email ([email protected]) задать в настройках проекта

api/serializers.py - Поправить вложенный блок

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/api/serializers.py

def validate(self, attrs): 
    if self.context['request']._request.method == 'POST': 

Давай не будем создавать большой вложенный блок и проверим обратное условие

api/views.py - убрать if

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/api/views.py

@api_view(['POST']) 
@permission_classes([AllowAny]) 
def get_token(request): 
    """Получение JWT-токена. 
 
    Если полученный код подтверждения валиден - отправляет юзеру JWT-токен. 
    """ 
    serializer = TokenSerializer(data=request.data) 
 
    if serializer.is_valid(raise_exception=True): 

К строке if serializer.is_valid(raise_exception=True):
Можно не использовать if, просто пишем эту строку и будем знать, что если что-то не так - то выбросится исключение, если нет - идём дальше, всё норм
https://www.django-rest-framework.org/api-guide/exceptions/#validationerror

api/views.py - поправить serializer.is_valid в def signup(request):

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/api/views.py

api_view(['POST']) 
@permission_classes([AllowAny]) 
def signup(request): 
    """Регистрация пользователя. 
 
    Создает нового пользователя, отправляет на указанный email код 
    подтверждения. 
    Если пользователь с полученными username и email уже существует - 
    отправляет ему код подтверждения. 
    """ 
    user = User.objects.filter( 
        username=request.data.get('username'), 
        email=request.data.get('email') 
    ).first() 
    if user: 
        send_confirmation(user) 
        return Response('Мы отправили код подтверждения на вашу почту.', 
                        status=status.HTTP_200_OK) 
    else: 
       serializer = SignUpSerializer(data=request.data) 
 
        if serializer.is_valid(): 

для if serializer.is_valid():
Вот тут лучше сделать так:
https://www.django-rest-framework.org/api-guide/exceptions/#validationerror

api/filters.py - Поменять на tuple

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/api/filters.py

class Meta: 
    model = Title 
    fields = ['name', 'year', 'genre', 'category', ]

Комментарий
fields = ['name', 'year', 'genre', 'category', ] - Давай такие значения в tuple заносить, они неизменяемые, меньше памяти занимают, со всех сторон подходят в общем)

api/views.py - изменить метод запрета ненужных методов для Categry и Genre

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/api/views.py

class CategoryViewSet(ModelViewSet): 
    serializer_class = CategorySerializer 
    queryset = Category.objects.all() 
    permission_classes = [ 
        (IsAuthenticated & IsAdmin) | ReadOnly 
    ] 
    pagination_class = PageNumberPagination 
    filter_backends = [SearchFilter] 
    search_fields = ['name', ] 
 
    def retrieve(self, request, **kwargs): 

А может другим способом запретим ненужные методы? Миксинами, например :)

class GenreViewSet(ModelViewSet): 
    serializer_class = GenreSerializer 
    queryset = Genre.objects.all() 
    permission_classes = [ 
        (IsAuthenticated & IsAdmin) | ReadOnly 
    ] 
    pagination_class = PageNumberPagination 
    filter_backends = [SearchFilter] 
    search_fields = ['name', ] 
 
    def retrieve(self, request, **kwargs): 

И тут)

class TitleViewSet(ModelViewSet): 
    queryset = Title.objects.all().annotate(rating=Avg('reviews__score')) 

Красота!
Отлично

api/views.py - убрать else

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/api/views.py

def signup(request): 
    """Регистрация пользователя. 
 
    Создает нового пользователя, отправляет на указанный email код 
    подтверждения. 
    Если пользователь с полученными username и email уже существует - 
    отправляет ему код подтверждения. 
    """ 
    user = User.objects.filter( 
        username=request.data.get('username'), 
        email=request.data.get('email') 
    ).first() 
 
    if user: 
        send_confirmation(user) 
        return Response('Мы отправили код подтверждения на вашу почту.', 
                        status=status.HTTP_200_OK) 
    else: 

После return не нужен else

reviews/models.py - валидация года издания

https://github.com/malyshevadv/api_yamdb/blob/master/api_yamdb/reviews/models.py

class Title(models.Model): 
    name = models.CharField(max_length=256) 
    year = models.PositiveIntegerField('Год издания') 

Нужно валидировать, что не больше текущего :)