Skip to content

Fix admin - #2811

Merged
acasajus merged 2 commits into
masterfrom
fix-admin
Sep 14, 2026
Merged

acasajus merged 2 commits into
masterfrom
fix-admin

Conversation

@acasajus

Copy link
Copy Markdown
Collaborator

Show paddle link in the admin panel for a given user.
When searching by user_id filter the query by that field and do not try to loop on emails.

flask-admin casts every searchable column to text and matches it with
ILIKE '%term%', so searching a user id ran
CAST(users.id AS VARCHAR) ILIKE '%123%' on both the list and the count
query, sequentially scanning the whole users table.

Override _apply_search on SLModelView so a plain integer term compares
the integer columns directly and skips the text ones, and a non-integer
term skips the integer columns, which can never match by equality. A
term is now either an id lookup or a text search, never both, so bare
numbers no longer substring-match email columns.
@acasajus
acasajus enabled auto-merge (rebase) September 14, 2026 14:23
@acasajus
acasajus merged commit e6c312b into master Sep 14, 2026
3 checks passed
@acasajus
acasajus deleted the fix-admin branch September 14, 2026 14:30
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