From 2dc7acc53660efa9316b51b20b475ece76006344 Mon Sep 17 00:00:00 2001 From: Colin Powell Date: Wed, 11 Mar 2026 11:17:12 -0400 Subject: [PATCH] [scrobbles] Blacken views --- vrobbler/apps/scrobbles/views.py | 116 +++++++++++++++++++++++-------- 1 file changed, 87 insertions(+), 29 deletions(-) diff --git a/vrobbler/apps/scrobbles/views.py b/vrobbler/apps/scrobbles/views.py index 5574e0f..4d2d1fe 100644 --- a/vrobbler/apps/scrobbles/views.py +++ b/vrobbler/apps/scrobbles/views.py @@ -155,17 +155,25 @@ class RecentScrobbleList(ListView): next_date = date + timedelta(weeks=+2) prev_date = date + timedelta(weeks=-1) if date.isocalendar()[1] < today.isocalendar()[1]: - data["next_link"] = f"?date={next_date.strftime('%Y-W%W')}" + data["next_link"] = ( + f"?date={next_date.strftime('%Y-W%W')}" + ) data["prev_link"] = f"?date={prev_date.strftime('%Y-W%W')}" - data["title"] = f"Week {date.strftime('%-W')} of {date.year}" + data["title"] = ( + f"Week {date.strftime('%-W')} of {date.year}" + ) data = data | Scrobble.as_dict_by_type( - Scrobble.for_week(user_id, date.year, date.isocalendar()[1]) + Scrobble.for_week( + user_id, date.year, date.isocalendar()[1] + ) ) elif date_str == "this_month" or date_str.count("-") == 1: next_date = date + relativedelta(months=1) prev_date = date + relativedelta(months=-1) if date.month < today.month: - data["next_link"] = f"?date={next_date.strftime('%Y-%m')}" + data["next_link"] = ( + f"?date={next_date.strftime('%Y-%m')}" + ) data["title"] = f"{date.strftime('%B %Y')}" data["prev_link"] = f"?date={prev_date.strftime('%Y-%m')}" data = data | Scrobble.as_dict_by_type( @@ -174,16 +182,24 @@ class RecentScrobbleList(ListView): elif date_str == "today" or date_str.count("-") == 2: next_date = date + timedelta(days=1) prev_date = date - timedelta(days=1) - data["prev_link"] = f"?date={prev_date.strftime('%Y-%m-%d')}" + data["prev_link"] = ( + f"?date={prev_date.strftime('%Y-%m-%d')}" + ) if date < today: - data["next_link"] = f"?date={next_date.strftime('%Y-%m-%d')}" + data["next_link"] = ( + f"?date={next_date.strftime('%Y-%m-%d')}" + ) if date == today: data["title"] = "Today" else: data["title"] = f"{date.strftime('%Y-%m-%d')}" - data["today_link"] = f"?date={today.strftime('%Y-%m-%d')}" + data["today_link"] = ( + f"?date={today.strftime('%Y-%m-%d')}" + ) data = data | Scrobble.as_dict_by_type( - Scrobble.for_day(user_id, date.year, date.month, date.day) + Scrobble.for_day( + user_id, date.year, date.month, date.day + ) ) elif date_str == "this_year" or date_str.count("-") == 0: next_date = date + relativedelta(years=+1) @@ -206,7 +222,9 @@ class RecentScrobbleList(ListView): data = data | Scrobble.as_dict_by_type( Scrobble.for_day(user_id, date.year, date.month, date.day) ) - data["today_link"] = "" # f"?date={today.strftime('%Y-%m-%d')}" + data["today_link"] = ( + "" # f"?date={today.strftime('%Y-%m-%d')}" + ) data["active_imports"] = AudioScrobblerTSVImport.objects.filter( processing_started__isnull=False, @@ -235,7 +253,9 @@ class ScrobbleLongPlaysView(TemplateView): def get_context_data(self, **kwargs): context_data = super().get_context_data(**kwargs) context_data["view"] = self.request.GET.get("view", "grid") - context_data["in_progress"] = get_long_plays_in_progress(self.request.user) + context_data["in_progress"] = get_long_plays_in_progress( + self.request.user + ) context_data["completed"] = get_long_plays_completed(self.request.user) return context_data @@ -312,7 +332,9 @@ class ManualScrobbleView(FormView): scrobble_fn = MANUAL_SCROBBLE_FNS[key] scrobble = eval(scrobble_fn)(item_id, self.request.user.id) - return HttpResponseRedirect(scrobble.redirect_url(self.request.user.id)) + return HttpResponseRedirect( + scrobble.redirect_url(self.request.user.id) + ) class JsonableResponseMixin: @@ -358,7 +380,9 @@ class AudioScrobblerImportCreateView( return HttpResponseRedirect(self.request.META.get("HTTP_REFERER")) -class KoReaderImportCreateView(LoginRequiredMixin, JsonableResponseMixin, CreateView): +class KoReaderImportCreateView( + LoginRequiredMixin, JsonableResponseMixin, CreateView +): model = KoReaderImport fields = ["sqlite_file"] template_name = "scrobbles/upload_form.html" @@ -417,7 +441,9 @@ def web_scrobbler_webhook(request): playing_state = "started" if request.POST.get("isPlaying") == "false": playing_state = "stopped" - scrobble = web_scrobbler_scrobble_media(youtube_id, user_id, status=playing_state) + scrobble = web_scrobbler_scrobble_media( + youtube_id, user_id, status=playing_state + ) if not scrobble: logger.warning( "[web_scrobbler_webhook] failed", @@ -542,12 +568,18 @@ def import_audioscrobbler_file(request): scrobbles_created = [] # tsv_file = request.FILES[0] - file_serializer = serializers.AudioScrobblerTSVImportSerializer(data=request.data) + file_serializer = serializers.AudioScrobblerTSVImportSerializer( + data=request.data + ) if file_serializer.is_valid(): import_file = file_serializer.save() - return Response({"scrobble_ids": scrobbles_created}, status=status.HTTP_200_OK) + return Response( + {"scrobble_ids": scrobbles_created}, status=status.HTTP_200_OK + ) else: - return Response(file_serializer.errors, status=status.HTTP_400_BAD_REQUEST) + return Response( + file_serializer.errors, status=status.HTTP_400_BAD_REQUEST + ) @permission_classes([IsAuthenticated]) @@ -724,7 +756,9 @@ class ChartRecordView(TemplateView): ).order_by("rank") if charts.count() == 0: - ChartRecord.build(user=self.request.user, model_str=media_type, **kwargs) + ChartRecord.build( + user=self.request.user, model_str=media_type, **kwargs + ) charts = ChartRecord.objects.filter( media_filter, user=self.request.user, **kwargs ).order_by("rank") @@ -756,7 +790,9 @@ class ChartRecordView(TemplateView): media_type = self.request.GET.get("media", "Track") user = self.request.user params = {} - context_data["chart_type"] = self.request.GET.get("chart_type", "maloja") + context_data["chart_type"] = self.request.GET.get( + "chart_type", "maloja" + ) context_data["artist_charts"] = {} if not date: @@ -821,7 +857,9 @@ class ChartRecordView(TemplateView): params["day"] = day month_str = calendar.month_name[month] name = f"Chart for {month_str} {day}, {year}" - in_progress = now.month == month and now.year == year and now.day == day + in_progress = ( + now.month == month and now.year == year and now.day == day + ) media_filter = self.get_media_filter("Track") track_charts = ChartRecord.objects.filter( @@ -833,13 +871,17 @@ class ChartRecordView(TemplateView): ).order_by("rank") if track_charts.count() == 0 and not in_progress: - ChartRecord.build(user=self.request.user, model_str="Track", **params) + ChartRecord.build( + user=self.request.user, model_str="Track", **params + ) media_filter = self.get_media_filter("Track") track_charts = ChartRecord.objects.filter( media_filter, user=self.request.user, **params ).order_by("rank") if artist_charts.count() == 0 and not in_progress: - ChartRecord.build(user=self.request.user, model_str="Artist", **params) + ChartRecord.build( + user=self.request.user, model_str="Artist", **params + ) media_filter = self.get_media_filter("Artist") artist_charts = ChartRecord.objects.filter( media_filter, user=self.request.user, **params @@ -860,24 +902,38 @@ class ScrobbleStatusView(LoginRequiredMixin, TemplateView): def get_context_data(self, **kwargs): data = super().get_context_data(**kwargs) user_scrobble_qs = Scrobble.objects.filter().order_by("-timestamp") - progress_plays = user_scrobble_qs.filter(in_progress=True, is_paused=False) + progress_plays = user_scrobble_qs.filter( + in_progress=True, is_paused=False + ) data["listening"] = progress_plays.filter( Q(track__isnull=False) | Q(podcast_episode__isnull=False) ).first() data["watching"] = progress_plays.filter(video__isnull=False).first() - data["going"] = progress_plays.filter(geo_location__isnull=False).first() - data["playing"] = progress_plays.filter(board_game__isnull=False).first() - data["sporting"] = progress_plays.filter(sport_event__isnull=False).first() - data["browsing"] = progress_plays.filter(web_page__isnull=False).first() - data["participating"] = progress_plays.filter(life_event__isnull=False).first() + data["going"] = progress_plays.filter( + geo_location__isnull=False + ).first() + data["playing"] = progress_plays.filter( + board_game__isnull=False + ).first() + data["sporting"] = progress_plays.filter( + sport_event__isnull=False + ).first() + data["browsing"] = progress_plays.filter( + web_page__isnull=False + ).first() + data["participating"] = progress_plays.filter( + life_event__isnull=False + ).first() data["working"] = progress_plays.filter(task__isnull=False).first() long_plays = user_scrobble_qs.filter( long_play_complete=False, played_to_completion=True ) data["reading"] = long_plays.filter(book__isnull=False).first() - data["sessioning"] = long_plays.filter(video_game__isnull=False).first() + data["sessioning"] = long_plays.filter( + video_game__isnull=False + ).first() return data @@ -923,7 +979,9 @@ class ScrobbleDetailView(DetailView): data[field_name] = original_value if data.get("with_people_ids", False): - data["with_people_ids"] = [p.id for p in data["with_people_ids"]] + data["with_people_ids"] = [ + p.id for p in data["with_people_ids"] + ] if data.get("platform_id", False): data["platform_id"] = data["platform_id"].id