0

デコレータに問題があります。オプションの引数を使用して独自のデコレータを作成しようとしています。

これが今のやり方です：

def CheckPremissions(manager=1):
    def wrap(func):
        def wrapper(request, *args, **kwargs):
            if request.user.is_anonymous():
                return HttpResponseRedirect(reverse('login'))

            logged_user = getRelatedWorker(request.user)

            if (logged_user == None):
                return HttpResponseRedirect('accounts/no_worker_error.html')

            if self.manager != 0:
                try:
                    dzial = Dzial.objects.get(kierownik=logged_user)
                except Dzial.DoesNotExist:
                    isManager = False
                else:
                    isManager = True

                if not isManager:
                    return HttpResponseRedirect('accounts/denied_logged.html')

            return func(request, *args, **kwargs)
        return wrapper
    return wrap

コードは（私にとっては）良さそうに見えますが、デコレーターを使用すると、次のエラーが発生します。

Environment:

    Request Method: GET
    Request URL: http://127.0.0.1:8080/applications/show

    Django Version: 1.4.1
    Python Version: 2.7.3


Traceback:
    File "/home/marcin/projekt/lib/python2.7/site-packages/django/core/handlers/base.py" in get_response
      188.                 response = middleware_method(request, response)
    File "/home/marcin/projekt/lib/python2.7/site-packages/django/middleware/common.py" in process_response
      94.         if response.status_code == 404:

    Exception Type: AttributeError at /applications/show
    Exception Value: 'function' object has no attribute 'status_code'

私は何を間違っていますか？

4

1 に答える 1

7

デコレータを間違って適用していると思われます。managerパラメータを指定するには、それを呼び出す必要があります。

@CheckPremissions()
def someview(request):
    pass

または明示的に指定するには:

@CheckPremissions(manager=0)
def someview(request):
    pass

デコレータにも別の問題があります。あなたはself.managerコードで参照します：

if self.manager != 0:

ただし、これはインスタンスではなく、selfパラメーターはありません。私はあなたが意味したと思います：

if manager:

（変数をブール値として扱うことで、変数がゼロでないことをテストできます）。ああ、デコレータのスペルを修正したいかもしれません。あなたはおそらくCheckPermissions代わりに意味しました。:-)

于 2012-09-12T10:57:36.893 に答える