Skip to content

Commit 2350489

Browse files
fix: Add service_id to search fields instead of custom
1 parent 076a546 commit 2350489

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

apps/codecov-api/core/admin.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ class RepositoryAdmin(AdminMixin, admin.ModelAdmin):
4444
search_fields = (
4545
"name",
4646
"author__username__exact",
47+
"service_id__exact",
4748
)
4849
show_full_result_count = False
4950
autocomplete_fields = ("bot",)
@@ -84,19 +85,19 @@ def get_search_results(
8485
Search for repositories by name, service_id, author username, or repoid.
8586
https://docs.djangoproject.com/en/5.2/ref/contrib/admin/#django.contrib.admin.ModelAdmin.get_search_results
8687
"""
87-
# Default search is by name and author username (defined in `search_fields`)
88+
# Default search is by name, author username, and service_id (defined in `search_fields`)
8889
queryset, may_have_duplicates = super().get_search_results(
8990
request,
9091
queryset,
9192
search_term,
9293
)
94+
# Also search by repoid if the search term is numeric
9395
try:
9496
search_term_as_int = int(search_term)
9597
except ValueError:
9698
pass
9799
else:
98100
queryset |= self.model.objects.filter(repoid=search_term_as_int)
99-
queryset |= self.model.objects.filter(service_id=search_term_as_int)
100101
return queryset, may_have_duplicates
101102

102103
def has_add_permission(self, _, obj=None):

0 commit comments

Comments
 (0)