представление класса django с декоратором и сессиями - PullRequest
Новая
       13

представление класса django с декоратором и сессиями

3 голосов
/ 15 сентября 2009

Я пытаюсь преобразовать некоторые из моих представлений django из представлений на основе функций в представления на основе классов, и я столкнулся с небольшой проблемой.

Мой ОО немного слаб, и я думаю, что проблема в том, что я потерял счет того, что происходит.

У меня есть собственный декоратор входа в систему, который мне нужен для просмотра, поэтому у меня есть ...

Сначала у меня есть класс View из этого примера. http://www.djangosnippets.org/snippets/760/

Тогда мой класс просмотра выглядит так ... 

class TopSecretPage(View):
    @custom_login
    def __call__(self, request, **kwargs):
        #bla bla view stuff...
        pass

Проблема в том, что мой декоратор не может получить доступ к request.session по какой-то причине ...

Мой декоратор выглядит так ... 

def myuser_login_required(f):
    def wrap(request, *args, **kwargs):

        # this check the session if userid key exist,
        # if not it will redirect to login page

        if 'field' not in request.session.keys():
        return wrap

Я думаю, что мне чего-то не хватает, поэтому спасибо всем за терпение!

UPDATE: Итак, вот ошибка, которую я получаю ...

"ViewDoesNotExist: пробовал TopSecretPage в модуле имя_проекта.application.views. Ошибка: объект типа" TopSecretPage "не имеет атрибута" сеанс ""

Я также упростил оформитель, чтобы он выглядел следующим образом ... 

def myuser_login_required(request, *args, **kwargs):


    # this check the session if userid key exist,
    # if not it will redirect to login page

    if 'username' not in request.session.keys():
        return  HttpResponseRedirect(reverse("login-page"))

    return True

Ответы [ 5 ]

6 голосов
/ 03 июля 2011

Правильный способ сделать это для любого декоратора, примененного к любому методу представления на основе классов, - использовать django.utils.decorators.method_decorator(). Я не уверен, когда был введен method_decorator (), но вот пример / обновление в заметках о выпуске Django 1.2 . Используйте это так: 

from django.utils.decorators import method_decorator

class TopSecretPage(View):
    @method_decorator(custom_login)
    def __call__(self, request, **kwargs):
        #bla bla view stuff...
        pass
2 голосов
/ 16 сентября 2009

Проблема в том, что ваша оболочка ожидает «запрос» в качестве первого аргумента, но метод класса всегда принимает «себя» в качестве первого аргумента. Таким образом, в вашем декораторе объект запроса фактически является самим TopSecretPage.

Либо решения Vinay, либо artran должны работать, поэтому я не буду их повторять. Просто подумал, что более четкое описание проблемы может быть полезным.

2 голосов
/ 16 сентября 2009

Эта проблема возникла до .Включено решение, которое может работать для вас.

Обновление: Пример метода с декоратором: 

class ContentView(View):

    # the thing in on_method() is the actual Django decorator
    #here are two examples
    @on_method(cache_page(60*5))
    @on_method(cache_control(max_age=60*5))
    def get(self, request, slug): # this is the decorated method
        pass #in here, you access request normally
1 голос
/ 06 января 2016

Это фактически дубликат Django - правильный способ передачи аргументов декораторам CBV? который описывает правильный способ решения этой проблемы. Правильный способ сделать это для django 1.9 следующий: 

@method_decorator(myuser_login_required(), name='dispatch')
class TopSecretPage(View):
    ..
1 голос
/ 16 сентября 2009

Вместо использования декоратора в представлении вы можете украсить URL.

Например, в urls.py:

from my_decorators import myuser_login_required
from my_views import TopSecretPage

urlpatterns = patterns('', 
    (r'^whatever-the-url-is/$', myuser_login_required(TopSecretPage), {}),
)

Возможно, вам придется немного поиграть с этим, но это правильно.

...