akarpov/music/api/serializers.py

@@ -239,26 +239,10 @@ class Meta:
 
 class FullAlbumSerializer(serializers.ModelSerializer):
     songs = ListSongSerializer(many=True, read_only=True)
-    artists = serializers.SerializerMethodField("get_artists")
-
-    @extend_schema_field(AuthorSerializer(many=True))
-    def get_artists(self, obj):
-        artists = []
-        qs = Author.objects.cache().filter(
-            songs__id__in=obj.songs.cache().all().values("id").distinct()
-        )
-        for artist in qs:
-            if artist not in artists:
-                artists.append(artist)
-
-        return AuthorSerializer(
-            artists,
-            many=True,
-        ).data
 
     class Meta:
         model = Album
-        fields = ["name", "link", "image", "songs", "artists"]
+        fields = ["name", "link", "image", "songs"]
         extra_kwargs = {
             "link": {"read_only": True},
             "image": {"read_only": True},
@@ -267,27 +251,10 @@ class Meta:
 
 class FullAuthorSerializer(serializers.ModelSerializer):
     songs = ListSongSerializer(many=True, read_only=True)
-    albums = serializers.SerializerMethodField(method_name="get_albums")
-
-    @extend_schema_field(AlbumSerializer(many=True))
-    def get_albums(self, obj):
-        qs = Album.objects.cache().filter(
-            songs__id__in=obj.songs.cache().all().values("id").distinct()
-        )
-        albums = []
-        for album in qs:
-            # TODO: rewrite to filter
-            if album not in albums:
-                albums.append(album)
-
-        return AlbumSerializer(
-            albums,
-            many=True,
-        ).data
 
     class Meta:
         model = Author
-        fields = ["name", "link", "image", "songs", "albums"]
+        fields = ["name", "link", "image", "songs"]
         extra_kwargs = {
             "link": {"read_only": True},
             "image": {"read_only": True},

akarpov/music/api/views.py

@@ -272,7 +272,9 @@ class ListAlbumsAPIView(generics.ListAPIView):
     serializer_class = AlbumSerializer
     pagination_class = StandardResultsSetPagination
     permission_classes = [permissions.AllowAny]
-    queryset = Album.objects.cache().all()
+
+    def get_queryset(self):
+        return Album.objects.all()
 
 
 class RetrieveUpdateDestroyAlbumAPIView(
@@ -282,14 +284,15 @@ class RetrieveUpdateDestroyAlbumAPIView(
     lookup_url_kwarg = "slug"
     permission_classes = [IsAdminOrReadOnly]
     serializer_class = FullAlbumSerializer
-    queryset = Album.objects.cache().all()
 
 
 class ListAuthorsAPIView(generics.ListAPIView):
     serializer_class = AuthorSerializer
     pagination_class = StandardResultsSetPagination
     permission_classes = [permissions.AllowAny]
-    queryset = Author.objects.cache().all()
+
+    def get_queryset(self):
+        return Author.objects.all()
 
 
 class RetrieveUpdateDestroyAuthorAPIView(
@@ -299,7 +302,6 @@ class RetrieveUpdateDestroyAuthorAPIView(
     lookup_url_kwarg = "slug"
     permission_classes = [IsAdminOrReadOnly]
     serializer_class = FullAuthorSerializer
-    queryset = Author.objects.cache().all()
 
 
 class ListenSongAPIView(generics.GenericAPIView):

config/settings/base.py

@@ -74,7 +74,7 @@
     "blog.post": {"ops": ("fetch", "get"), "timeout": 20 * 15},
     "themes.theme": {"ops": ("fetch", "get"), "timeout": 60 * 60},
     "gallery.*": {"ops": ("fetch", "get", "list"), "timeout": 60 * 15},
-    "files.*": {"ops": ("fetch", "get", "list"), "timeout": 60},
+    "files.*": {"ops": ("fetch", "get"), "timeout": 60},
     "auth.permission": {"ops": "all", "timeout": 60 * 15},
     "music.*": {"ops": ("fetch", "get", "list"), "timeout": 60 * 15},
 }