Django：DRY原則とUserPassesTestMixin

Question

私はPostというモデルを持ち、owner（Userへの外部キー）と呼ばれるフィールドを持っています。もちろん、所有者だけがupdateまたはdeleteの投稿を投稿することができます。言われて、私は、ユーザーがログインしているが、その後、私はまたupdate/deleteに質問をしようとするユーザーがownerであることを確認する必要があることを確認するビューでlogin_requiredデコレータを使用することを

。

私はドキュメントDjango: Generic Editing Viewsを使用していたように、私はDjango: UserPassesTestMixinを使用する必要があると言います。

この検証はupdateとdeleteビューのために行われます。ドライ、これについてどうやって行くの？私はTestUserOwnerOfPostという名前のクラスを作成し、test_func()を作成し、updateとdeleteビューはそれから継承させる必要がありますか？すべてのは、ログインユーザがシステムに上記のコードでは

from django.views.generic.edit import UpdateView 
from django.contrib.auth.decorators import login_required 
from django.contrib.auth.mixins import UserPassesTestMixin 

class TestUserOwnerOfPost(UserPassesTestMixin):       
    def test_func(self):              
     return self.request.user == self.post.owner 

class EditPost(UpdateView, TestUserOwnerOfPost):       
    model = Post                   
    @method_decorator(login_required)           
    def dispatch(self, *args, **kwargs):          
     return super(EditPost, self).dispatch(*args, **kwargs) 

、できるedit/delete任意のポスト：私が試してみましたし、うまくいきませんでした何

原因、以下のコード。私は間違って何をしていますか？私は何かを欠いている？ありがとう。

Answer 1

最初の問題は@rafalmpが言うように、あなたが継承するクラスの順序は、間違っているということです。

しかし、UserPassesTestミックスインビューを実行前にテストを実行するため、問題を解決しない定着。つまり、self.objectはまだ設定されていないため、self.objectのオーナーを確認するのは実際には適していません。アイム・ビューが今までself.postを設定していることはないと思うが、私はそれについて間違っているかもしれない - 私はself.object代わりのself.postを使用しています注意してください。

1つのオプションは、テスト関数の内部でself.get_object()を呼び出すことです。ビューはオブジェクトを2回フェッチするため、これは少し非効率ですが、実際は問題ではないでしょう。

def test_func(self): 
    self.object = self.get_object() 
    return self.request.user == self.object.owner 

別のアプローチは、ユーザが所有するオブジェクトにそれを制限するために、get_querysetをオーバーライドすることです。つまり、オブジェクトを所有していない場合、404エラーが発生します。この動作は、ログインページにリダイレクトされるUserPassesTestMixinとまったく同じ動作ではありませんが、問題ありません。

class OwnerQuerysetMixin(object):       
    def get_queryset(self): 
     queryset = super(OwnerQuerysetMixin, self).get_queryset()             
     # perhaps handle the case where user is not authenticated 
     queryset = queryset.filter(owner=self.request.user) 
     return queryset 

Answer 2

あなたが問題を継承するクラスの順。 UpdateViewが実行される前に、仕事へのアクセス制御のために、それが施行されている必要があります

class EditPost(TestUserOwnerOfPost, UpdateView):