web-dev-qa-db-ja.com

Django：post_saveシグナルで元の（変更されていない）インスタンスにアクセスする方法

パフォーマンスを向上させるためにデータの非正規化を行い、ブログ投稿が受け取った投票の合計をPostモデル内に配置します。

class Post(models.Model):
    """ Blog entry """
    author          = models.ForeignKey(User)
    title           = models.CharField(max_length=255)
    text            = models.TextField()
    rating          = models.IntegerField(default=0) # here is the sum of votes!

class Vote(models.Model):
    """ Vote for blog entry """
    post            = models.ForeignKey(Post)
    voter           = models.ForeignKey(User)
    value           = models.IntegerField()

もちろん、私はPost.rating実際の値。通常はデータベーストリガーを使用しますが、今ではpost_saveシグナル（データベースの処理時間を短縮するため）：

# vote was saved
@receiver(post_save, sender=Vote)
def update_post_votes(sender, instance, created, **kwargs):
    """ Update post rating """
    if created:
        instance.post.rating += instance.value
        instance.post.save()
    else:
        # if vote was updated, we need to remove the old vote value and add the new one
        # but how...?

保存する前にインスタンス値にアクセスするにはどうすればよいですか？データベーストリガーでは、OLDおよびNEWを事前定義しておく必要がありますが、post_save信号にこのようなものはありますか？

[〜＃〜]更新[〜＃〜]

マークの答えは次のとおりです。

# vote was saved
@receiver(pre_save, sender=Vote)
def update_post_votes_on_save(sender, instance, **kwargs):
    """ Update post rating """
    # if vote is being updated, then we must remove previous value first
    if instance.id:
        old_vote = Vote.objects.get(pk=instance.id)
        instance.post.rating -= old_vote.value
    # now adding the new vote
    instance.post.rating += instance.value
    instance.post.save()
pythondjangopostgresqldenormalizationdjango-signals
38
Silver Light

_post_save_は変更されていないバージョンを取得するには遅すぎると思います。名前が示すように、データはその時点ですでにデータベースに書き込まれています。代わりに_pre_save_を使用してください。その場合、pk：old = Vote.objects.get(pk=instance.pk)を介してデータベースからモデルを取得し、現在のインスタンスと前のインスタンスの違いを確認できます。

55
Mark Lavin

これはnot an最適解ですが、機能します。

@receiver(pre_save, sender=SomeModel)
def model_pre_save(sender, instance, **kwargs):
    try:
        instance._pre_save_instance = SomeModel.objects.get(pk=instance.pk)
    except SomeModel.DoesNotExist:
        instance._pre_save_instance = instance


@receiver(signal=post_save, sender=SomeModel)
def model_post_save(sender, instance, created, **kwargs):
    pre_save_instance = instance._pre_save_instance
    post_save_instance = instance
6
Parag Tyagi -morpheus-