Skip to content

Fix: avoid per-tag DB lookup when rendering tag columns - #368

Merged
stonebuzz merged 2 commits into
mainfrom
fix_speed
Aug 11, 2026
Merged

Fix: avoid per-tag DB lookup when rendering tag columns#368
stonebuzz merged 2 commits into
mainfrom
fix_speed

Conversation

@Rom1-B

@Rom1-B Rom1-B commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes #N/A
  • The "Tags" column shown in item lists (tickets, etc.) issued one extra database query per tag displayed on every row.
    On lists with many tagged items this caused severe slowdowns, up to a reported 1230 SQL queries on a single ticket list page.
    The tag color is now read directly from the data already fetched by the search query, removing the per-row lookup entirely.

Screenshots (if appropriate):

Comment thread hook.php
Co-authored-by: Stanislas <skita@teclib.com>
@Rom1-B
Rom1-B requested a review from stonebuzz August 11, 2026 12:54
@stonebuzz
stonebuzz merged commit 9009909 into main Aug 11, 2026
3 checks passed
@stonebuzz
stonebuzz deleted the fix_speed branch August 11, 2026 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants