api: Update Patch related serializers to accomodate for the new 'interested_users'field

Signed-off-by: andrepapoti <andrepapoti@gmail.com>

Author: andrepapoti

patchwork/api/patch.py

@@ -8,6 +8,7 @@
 import email.parser
 
 from django.core.exceptions import ValidationError
+from django.utils import timezone
 from django.utils.text import slugify
 from django.utils.translation import gettext_lazy as _
 from rest_framework.exceptions import APIException
@@ -29,6 +30,7 @@
 from patchwork.api.embedded import UserSerializer
 from patchwork.api.filters import PatchFilterSet
 from patchwork.models import Patch
+from patchwork.models import PatchInterest
 from patchwork.models import PatchRelation
 from patchwork.models import State
 from patchwork.models import User
@@ -78,12 +80,41 @@ class PatchConflict(APIException):
     )
 
 
+class PatchInterestSerializer(BaseHyperlinkedModelSerializer):
+    user = UserSerializer()
+    patch = PatchSerializer()
+    is_stale = SerializerMethodField()
+
+    def get_is_stale(self, patch_interest):
+        expiration_time = (
+            patch_interest.patch.state.review_interest_expiration_time
+        )
+        valid_until = (
+            patch_interest.last_time_marked_for_review + expiration_time
+        )
+
+        return timezone.now() > valid_until
+
+    class Meta:
+        model = PatchInterest
+        fields = [
+            'id',
+            'user',
+            'patch',
+            'last_time_marked_for_review',
+            'is_stale',
+        ]
+
+
 class PatchListSerializer(BaseHyperlinkedModelSerializer):
     web_url = SerializerMethodField()
     project = ProjectSerializer(read_only=True)
     state = StateField()
     submitter = PersonSerializer(read_only=True)
     delegate = UserSerializer(allow_null=True)
+    interested_users = PatchInterestSerializer(
+        source='patchinterest_set', many=True
+    )
     mbox = SerializerMethodField()
     series = SeriesSerializer(read_only=True)
     comments = SerializerMethodField()
@@ -172,6 +203,8 @@ class Meta:
             'hash',
             'submitter',
             'delegate',
+            'interested_users',
+            'pending_review',
             'mbox',
             'series',
             'comments',
@@ -190,6 +223,7 @@ class Meta:
             'name',
             'hash',
             'submitter',
+            'pending_review',
             'mbox',
             'series',
             'comments',
@@ -230,16 +264,11 @@ def get_headers(self, patch):
     def get_prefixes(self, instance):
         return clean_subject(instance.name)[1]
 
-    def update(self, instance, validated_data):
-        # d-r-f cannot handle writable nested models, so we handle that
-        # specifically ourselves and let d-r-f handle the rest
-        if 'related' not in validated_data:
-            return super(PatchDetailSerializer, self).update(
-                instance, validated_data
-            )
-
-        related = validated_data.pop('related')
+    def update_interested_users(self, instance, patchinterest_set):
+        intereted_user = patchinterest_set.pop()['user']
+        instance.interested_users.add(intereted_user.id)
 
+    def update_related(self, instance, related):
         # Validation rules
         # ----------------
         #
@@ -280,9 +309,7 @@ def update(self, instance, validated_data):
             if instance.related and instance.related.patches.count() == 2:
                 instance.related.delete()
             instance.related = None
-            return super(PatchDetailSerializer, self).update(
-                instance, validated_data
-            )
+            return
 
         # break before make
         relations = {patch.related for patch in patches if patch.related}
@@ -306,6 +333,18 @@ def update(self, instance, validated_data):
         instance.related = relation
         instance.save()
 
+    def update(self, instance, validated_data):
+        # d-r-f cannot handle writable nested models, so we handle that
+        # specifically ourselves and let d-r-f handle the rest
+
+        if 'related' in validated_data:
+            related = validated_data.pop('related')
+            self.update_related(instance, related)
+
+        if 'patchinterest_set' in validated_data:
+            patchinterest_set = validated_data.pop('patchinterest_set')
+            self.update_interested_users(instance, patchinterest_set)
+
         return super(PatchDetailSerializer, self).update(
             instance, validated_data
         )