Verified Commit 7aa299e5 authored by Agate's avatar Agate 💬

See #170: ensure fetch works properly on tracks with public upload

parent 9e68aea7
Pipeline #10035 passed with stages
in 5 minutes and 18 seconds
......@@ -33,8 +33,12 @@ def library_track(request, pk, redirect_to_ap):
except models.Track.DoesNotExist:
return []
playable_uploads = obj.uploads.playable_by(None).order_by("id")
upload = playable_uploads.first()
if redirect_to_ap:
raise middleware.ApiRedirect(obj.fid)
redirect_url = upload.fid if upload else obj.fid
raise middleware.ApiRedirect(redirect_url)
track_url = utils.join_url(
settings.FUNKWHALE_URL,
......@@ -84,8 +88,6 @@ def library_track(request, pk, redirect_to_ap):
"content": obj.album.attachment_cover.download_url_medium_square_crop,
}
)
playable_uploads = obj.uploads.playable_by(None).order_by("id")
upload = playable_uploads.first()
if upload:
metas.append(
{
......
......@@ -394,6 +394,8 @@ def test_get_request_head_tags_calls_view_with_proper_arg_when_accept_header_set
("music.Album", {}, "library_album", "pk", "pk",),
("music.Track", {}, "library_track", "pk", "pk",),
("music.Library", {}, "library_library", "uuid", "uuid",),
# when a track as a public upload, we should redirect to the upload instead
("music.Upload", {"playable": True}, "library_track", "pk", "track.pk"),
],
)
def test_spa_views_raise_api_redirect_when_accept_json_set(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment