'collections.OrderedDict' object has no attribute 'pk' - django rest framework

msln picture msln · Oct 18, 2017 · Viewed 8.3k times · Source

I have a model and I want to write an update() method for it in order to update. The below snippet is my model:

class Klass(models.Model):
    title = models.CharField(max_length=50)
    description = models.CharField(max_length=500)
    university = models.CharField(max_length=50,blank=True, null=True)
    teacher = models.ForeignKey(Profile, related_name='teacher', on_delete=models.CASCADE)

and the below snippet is corresponding Serializer:

class KlassSerializer(ModelSerializer):
        teacher = ProfileSerializer()
        url = HyperlinkedIdentityField(view_name='mainp-api:detail', lookup_field='pk')
        klass_settings = KlassSettingsSerializer()

    class Meta:
        model = Klass
        fields = ('url', 'id', 'title', 'description', 'university','teacher')

    
    def update(self, instance, validated_data):
        instance.title = validated_data.get('title', instance.title)
        instance.description = validated_data.get('description', instance.description)
        instance.university = validated_data.get('university', instance.university)
        instance.save()

        return instance

And for update, I use below snippet:

class KlassAPIView(APIView):    
    def put(self, request, pk=None):
        if pk == None:
            return Response({'message': 'You must specify class ID'}, status=HTTP_400_BAD_REQUEST)

        klass = Klass.objects.get(pk=pk)
        if request.user.profile.type != 't':
            raise PermissionDenied(detail={'message': 'You aren't teacher of this class, so you can't edit information.'})

        serializer = KlassSerializer(data=request.data, context={'request': request})
        serializer.initial_data['teacher'] = request.user.profile.__dict__

        if serializer.is_valid():
            serializer.update(instance=klass, validated_data=serializer.data)  # Retrieve teacher and store
            return Response({'data': serializer.data}, status=HTTP_200_OK)
        else:
            return Response({'data': serializer.errors}, status=HTTP_400_BAD_REQUEST)

but when I send data with PUT method, it returns below error:

AttributeError at /api/class/49/

'collections.OrderedDict' object has no attribute 'pk'

and the error occurs in serializer.update(instance=klass, validated_data=serializer.data) line.

Answer

MKaras picture MKaras · Mar 15, 2019

Just ran into the same error.

In my case the problem was I accessed serializer.data before doing serializer.save().

Google dropped me here, so maybe someone else will also find this helpful.

Source: https://github.com/encode/django-rest-framework/issues/2964