antonksa

https://stackoverflow.com/questions/58640045/kafka...

from django.views.generic import View, ListView, DetailView


class AddUserPhoneToContextMixin(View):
    def get_context_data(self, **kwargs):
        '''
        Add phone number to context data
        '''
        ctx = super().get_context_data(**kwargs)
        try:
            user = User.objects.get(pk=1)
            ctx['phone_number'] = user.userprofile.phone_number
        finally:
            return ctx


class ItemListView(AddUserPhoneToContextMixin, ListView):
    model = Item
    paginate_by = 9


class ItemDetailView(AddUserPhoneToContextMixin, DetailView):
    model = Item

Только не очень понятно почему только pk=1, что будет если его нет, и почему в try-except вы обернули попытку доступа к userprofile. Еще считается, что except без указания конкретных классов обрабатываемых исключений это bad practice. В таком случае вы теряете контроль над пониманием, какое конкретно исключение произошло - то ли коннект в базе оборвался, то ли юзер не существует, то ли поля такого нет у его профиля.

Хорошая практика - это указывать конкретно что бы обрабатываете:

try:
    phone_number = user.userprofile.phonenumber
except UserProfile.DoesNotExists:
    raise NotFound('Warning! User profile did not created!')

den = 'ПОНЕДЕЛЬНИК'
    '#Алгебра:' + handle_admin_dz.Algebra

ёпта, это в мемориз.
Особенно учитывая

Algebra = bot.send_message(message.chat.id, "Запишите дз")

Даже если бы можно было выковырять переменную из функции, нарушив принципы инкапсуляции, то что вы пытаетесь сделать? Сложить строку с чем? С функцией? Результатом исполнения функции?

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

Напишите функцию получения текста сообщения. Вызовите ее. Сохраните результат в переменную. Используйте ее для отправки сообщения.

def handle_admin_dz(message):
    if message.text in {
        "Алгебра", "Геометрия", "Русский", ...
    }:
        response_text = "Запишите дз"
    else:
        response_text = "Неизвестный раздел!"
    return response_text

@bot.message_handler(content_types='text')
def handle_text(message):
    if message.text == "Понедельник":
        response_text = handle_admin_dz(message)
        den = 'ПОНЕДЕЛЬНИК\n#Алгебра: ' + response_text

Мне кстати непонятно, как у вас может быть одновременно message.text == "Понедельник" и == "Алгебра"

Кривое у доктора решение.
Нужно так:

class Model2(models.Model):
    m2m_field = models.ManyToManyField(to=Model1, related_name='second_models')

И потом:

m1 = Model1.object.get(...) #он просто у меня есть
second_model_queryset = m1.second_models.all()

def get(self, request, **kwargs):
        if not request.user.is_staff:
            raise PermissionDenied
        super().get(request, **kwargs)

https://www.django-rest-framework.org/api-guide/ex...

#  my_project/my_app/utils.py

from rest_framework.exceptions import APIException
from rest_framework.views import exception_handler

def custom_exception_handler(exc, context):
    # Call REST framework's default exception handler first,
    # to get the standard error response.
    response = exception_handler(exc, context)

    # Now add the HTTP status code to the response.
    if response is not None:
        if response.status_code == 500:
            #  My custom 500 error handler.
            pass
    return response

#  settings.py

REST_FRAMEWORK = {
    'EXCEPTION_HANDLER': 'my_project.my_app.utils.custom_exception_handler'
}

ЗЫ. аймсорри, неправильно понял вопрос. Вообще-то если у вас DRF обрабатывает урл, то там по умолчанию и DRF обрабатывает 500, если у вас не так, то вы что-то не так делаете.