fix bug in subscription_id

Author: earthcomfy

Diff for: apps/forum/serializers/post_serializers.py

@@ -135,7 +135,14 @@ class Meta:
             "created_at",
             "updated_at",
         )
-        read_only_fields = ("view_count", "answer_count", "is_answered", "is_closed", "asked_by", "slug")
+        read_only_fields = (
+            "view_count",
+            "answer_count",
+            "is_answered",
+            "is_closed",
+            "asked_by",
+            "slug",
+        )
 
     def get_asked_by(self, obj) -> str:
         return obj.post.user.username
@@ -201,12 +208,15 @@ class Meta(BaseQuestionSerializer.Meta):
         fields = "__all__"
 
     def get_subscription_id(self, obj) -> str:
-        user = self.context["request"].user
-        content_type = ContentType.objects.get(model="question")
-        subscription = Subscription.objects.filter(
-            user=user, target_content_type=content_type, target_object_id=obj.id
-        ).first()
-        return subscription.id if subscription else ""
+        request = self.context.get("request")
+        if request and hasattr(request, "user"):
+            user = self.context["request"].user
+            content_type = ContentType.objects.get(model="question")
+            subscription = Subscription.objects.filter(
+                user=user, target_content_type=content_type, target_object_id=obj.id
+            ).first()
+            return subscription.id if subscription else ""
+        return ""
 
 
 class BookmarkedPostSerializer(BaseQuestionSerializer):

Diff for: apps/notifications/serializers/notification_serializers.py

@@ -13,7 +13,8 @@ class SubscriptionSerializer(serializers.ModelSerializer):
     """
 
     target_content_type = serializers.SlugRelatedField(
-        slug_field="model", queryset=ContentType.objects.filter(model__in=("question", "answer", "resource"))
+        slug_field="model",
+        queryset=ContentType.objects.filter(model__in=("question", "answer", "resource")),
     )
     target_object_id = serializers.UUIDField()
     target = serializers.SerializerMethodField()
@@ -99,9 +100,14 @@ def get_subscription_id(self, obj) -> str:
         """
         Returns the subscription ID for the notification.
         """
-        user = self.context["request"].user
-        target_content_type = ContentType.objects.get_for_model(obj.target)
-        subscription = Subscription.objects.filter(
-            user=user, target_content_type=target_content_type, target_object_id=obj.target_object_id
-        ).first()
-        return subscription.id if subscription else ""
+        request = self.context.get("request")
+        if request and hasattr(request, "user"):
+            user = self.context["request"].user
+            target_content_type = ContentType.objects.get_for_model(obj.target)
+            subscription = Subscription.objects.filter(
+                user=user,
+                target_content_type=target_content_type,
+                target_object_id=obj.target_object_id,
+            ).first()
+            return subscription.id if subscription else ""
+        return ""

Diff for: apps/resources/serializers/resource_serializers.py

@@ -7,7 +7,11 @@
 from apps.forum.models import Tag
 from apps.notifications.models.notification_models import Subscription
 from apps.resources.constants import ResourceConstants
-from apps.resources.models.resource_models import Resource, ResourceCategory, ResourceFile
+from apps.resources.models.resource_models import (
+    Resource,
+    ResourceCategory,
+    ResourceFile,
+)
 
 
 class ResourceFileSerializer(serializers.ModelSerializer):
@@ -79,12 +83,15 @@ def get_is_bookmarked(self, obj) -> bool:
         return False
 
     def get_subscription_id(self, obj) -> str:
-        user = self.context["request"].user
-        content_type = ContentType.objects.get(model="resource")
-        subscription = Subscription.objects.filter(
-            user=user, target_content_type=content_type, target_object_id=obj.id
-        ).first()
-        return subscription.id if subscription else ""
+        request = self.context.get("request")
+        if request and hasattr(request, "user"):
+            user = self.context["request"].user
+            content_type = ContentType.objects.get(model="resource")
+            subscription = Subscription.objects.filter(
+                user=user, target_content_type=content_type, target_object_id=obj.id
+            ).first()
+            return subscription.id if subscription else ""
+        return ""
 
     def create(self, validated_data):
         tags_data = validated_data.pop("tags")