diff options
| author | erdgeist <erdgeist@erdgeist.org> | 2026-07-10 21:57:58 +0200 |
|---|---|---|
| committer | erdgeist <erdgeist@erdgeist.org> | 2026-07-10 21:57:58 +0200 |
| commit | 7d6a665b896252537b8b8df2f15e204d04417231 (patch) | |
| tree | 37b8d5434426646a019c6d5c8c12f7bb8a4de647 /app/models | |
| parent | 48eed5f35ea798c91f1e0a77c13e751145acfc48 (diff) | |
Fix current_user/uniq inconsistencies in admin#index, remove dead query
@mynodes used the bare @current_user ivar instead of the current_user
method (works today only because login_required already forces that
memoization; every other query and controller in this app uses the
method), and .uniq, which is plain Enumerable#uniq here, not
Relation#distinct -- it materializes every joined row into an Array
before deduplicating, rather than deduplicating in SQL.
@mypages had the same ivar pattern, no .order/.limit unlike every
sibling query in this action, and -- confirmed via a full grep -- is
referenced nowhere else in the codebase. Removed rather than bounded;
nothing was ever rendering it.
Diffstat (limited to 'app/models')
0 files changed, 0 insertions, 0 deletions
