jlengrand/bugsink
1
0
Fork 0
mirror of https://github.com/jlengrand/bugsink.git synced 2026-03-10 08:01:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

digest_tags: just use project_id to avoid lookups

Browse Source 
maybe maybe the passed-in event also avoids this, but the present
method will always do what you expect and this is obvious upon
reading, rather than have the reader think about something
This commit is contained in:
Klaas van Schelven
2025-02-27 16:21:29 +01:00
parent 0d4d95defd
commit 89d53a15c9
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
tags/models.py
View File

@@ -96,8 +96,7 @@ def digest_tags(event_data, event, issue):
value = value[:200]
# TODO just use bulk_create for each of the types of objects
# TODO check: event.project won't trigger a query, right? it's already loaded, right?
tag_key, _ = TagKey.objects.get_or_create(project=event.project, key=key)
tag_value, _ = TagValue.objects.get_or_create(project=event.project, key=tag_key, value=value)
EventTag.objects.get_or_create(project=event.project, value=tag_value, event=event)
IssueTag.objects.get_or_create(project=event.project, value=tag_value, issue=issue)
tag_key, _ = TagKey.objects.get_or_create(project_id=event.project_id, key=key)
tag_value, _ = TagValue.objects.get_or_create(project_id=event.project_id, key=tag_key, value=value)
EventTag.objects.get_or_create(project_id=event.project_id, value=tag_value, event=event)
IssueTag.objects.get_or_create(project_id=event.project_id, value=tag_value, issue=issue)