From 61af75e34c6fb71af9431557caca2087c0e3e8ce Mon Sep 17 00:00:00 2001 From: naglepuff Date: Tue, 23 Dec 2025 10:04:40 -0500 Subject: [PATCH] Don't populate list with null tag --- bats_ai/core/views/recording.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/bats_ai/core/views/recording.py b/bats_ai/core/views/recording.py index d4b5552..0bd8f35 100644 --- a/bats_ai/core/views/recording.py +++ b/bats_ai/core/views/recording.py @@ -252,13 +252,13 @@ def get_recordings(request: HttpRequest, public: bool | None = None): recordings = ( Recording.objects.filter(public=True) .exclude(Q(owner=request.user) | Q(spectrogram__isnull=True)) - .annotate(tags_text=ArrayAgg('tags__text')) + .annotate(tags_text=ArrayAgg('tags__text', filter=Q(tags__text__isnull=False))) .values() ) else: recordings = ( Recording.objects.filter(owner=request.user) - .annotate(tags_text=ArrayAgg('tags__text')) + .annotate(tags_text=ArrayAgg('tags__text', filter=Q(tags__text__isnull=False))) .values() ) @@ -298,7 +298,9 @@ def get_recording(request: HttpRequest, id: int): # Filter recordings based on the owner's id or public=True try: recordings = ( - Recording.objects.filter(pk=id).annotate(tags_text=ArrayAgg('tags__text')).values() + Recording.objects.filter(pk=id) + .annotate(tags_text=ArrayAgg('tags__text', filter=Q(tags__text__isnull=False))) + .values() ) if len(recordings) > 0: recording = recordings[0]