django-simple-history,没有历史记录的保存不起作用

时间:2018-12-02 15:06:57

标签: django-simple-history

我是django-simple-history的忠实拥护者,但是在模型的默认save()方法中使用“ save_without_historical_record”时,似乎无法正常工作。

我有这样的模型

class NzPlasmid (models.Model):
    ...
    plasmid_map = models.FileField("Plasmid Map (max. 2 MB)", upload_to="temp/", blank=True)
    history = HistoricalRecords()
    ...

它具有一个自定义的save()方法,该方法使用新创建的对象的ID重命名Plasmid_map。为此,我第一次保存该对象以获取其ID,然后使用该对象重命名质粒_映射。我不想为第一次保存保存历史记录,而只为第二次保存。我的自定义save()方法如下所示

def save(self, force_insert=False, force_update=False):

    self.skip_history_when_saving = True
    super(NzPlasmid, self).save(force_insert, force_update)

    ... some rename magic here ...

    del self.skip_history_when_saving
    super(NzPlasmid, self).save(force_insert, force_update)

这不起作用,因为每次创建质粒时我仍然会获得“重复的”历史记录。

非常感谢。

2 个答案:

答案 0 :(得分:0)

第一次保存时,您正在创建对象。但是,根据this line,如果对象正在更新而不是创建,则只能保存而没有历史记录。您可以尝试的一种解决方法是使用描述为herepre_create_historical_record信号。这有点麻烦,但是您可以在下面的apps.py文件中包含信号处理代码:

def update_plasmid_map_and_save_instance(sender, instance, history_instance):
    # instance should have id here

    ... some rename magic on your instance model...

    history_instance.plasmid_map = instance.plasmid_map

    instance.save_without_historical_record()


# in apps.py
class TestsConfig(AppConfig):
     def ready(self):
         from ..models import HistoricalNzPlasmid

         pre_create_historical_record.connect(
             update_plasmid_map_and_save_instance,
             sender=HistoricalNzPlasmid,
             dispatch_uid='update_plasmid_map_on_history'
         )

然后,您将不必覆盖save上的NzPlasmid。有点hacky,但是应该可以。

答案 1 :(得分:0)

我通过修改save_model中的admin.py方法解决了该问题。创建具有映射的新质粒对象时,由于plasmid_map重命名而生成了两个历史记录,因此我删除了第一个包含“错误的”质粒图谱名称的历史记录,并更改了第二个的history_type,从更改(〜)到创建(+):

def save_model(self, request, obj, form, change):

    rename_and_preview = False
    new_obj = False

    if obj.pk == None:
        if obj.plasmid_map:
            rename_and_preview = True
            new_obj = True
        obj.save()

        ... some rename magic here ...

        if new_obj:
            obj.history.last().delete()
            history_obj = obj.history.first()
            history_obj.history_type = "+"
            history_obj.save()