Python Django:对象属性未保存/更新

Python Django:对象属性未保存/更新,python,django,crud,Python,Django,Crud,我有以下型号: class Device(models.Model): #more_code supplier = models.ForeignKey( Supplier, on_delete=SET_NULL, blank=True, null=True, related_name="devices" ) 我需要一个补丁端点来更改设备的供应商。视图中的一切看起来都正常工作,只有device.save()不更新对象的状态 我的看法是: def patch

我有以下型号:

class Device(models.Model):
    #more_code
    supplier = models.ForeignKey(
        Supplier, on_delete=SET_NULL, blank=True, null=True, related_name="devices"
    )
我需要一个补丁端点来更改设备的供应商。视图中的一切看起来都正常工作,只有device.save()不更新对象的状态

我的看法是:

def patch(self, request, site_pk):
    """
    PATCH the device's supplier info of the site.
    """
    all_device_token = False
    if request.data.get("device") == "ALL":
        all_device_token = True
        arguments = {}
    else:
        device_pk = request.data.get("device")
        arguments = {"id": device_pk}

    devices = Device.objects.filter(site_id=site_pk, **arguments).all()
    if not devices.exists():
        raise PermissionDenied("The device does not belong to the site.")

    if all_device_token:
        serializer_class = self.get_serializer_class()
        serializer = serializer_class(
            devices, many=True, data=request.data, partial=True,
        )
        serializer.is_valid(raise_exception=True)
        new_supplier = get_object_or_404(Supplier, pk=request.data.get("supplier"))
        for device in devices:
            device.supplier.id = new_supplier.pk
            device.supplier.name = new_supplier.name
            device.save()
    else:
        device = devices.first()

        serializer_class = self.get_serializer_class()
        serializer = serializer_class(device, data=request.data, partial=True,)
        serializer.is_valid(raise_exception=True)

        if "supplier" in self.request.data:
            new_supplier = serializer.validated_data["supplier"]
            device.supplier.id = new_supplier.pk
            device.supplier.name = new_supplier.name
            device.save()

        # serializer.save()
    return Response(serializer.data, status=status.HTTP_200_OK)
我的序列化程序正在将正确的信息返回给。
我看到开销了吗?

您在这里用一个新的主键和一个新的
名称更新
供应商
对象:

new_supplier = get_object_or_404(Supplier, pk=request.data.get('supplier'))
for device in devices:
    device.supplier.id = new_supplier.pk
    device.supplier.name = new_supplier.name
    device.save()
但是,我强烈建议不要更改旧供应商的主键(也不要重复数据)。如果这样做,您应该保存
设备.supplier.save()
,因为您更新了该对象

但这样做不是一种好的做法,因为这样您就可以更改
供应商
对象,而您不想更改旧供应商的名称,而是想将
设备
链接到新的
供应商
。您可以通过设置
supplier\u id
外键来完成此操作,并保存设备:

supplier_id = request.data.get('supplier')
for device in devices:
    device.supplier_id = supplier_id
    device.save()
supplier\u id=request.data.get('supplier'))
对于设备中的设备:
device.supplier\u id=供应商\u id

device.save()
您不应该
device.save()
,而应该
device.supplier.save()
,因为这是您正在更新的对象。也就是说,您不应该更改主键。只需创建一个新的
供应商
(具有新内容),并让您的
设备
引用该供应商。