summaryrefslogtreecommitdiff
path: root/test/models/node_test.rb
diff options
context:
space:
mode:
authorerdgeist <erdgeist@erdgeist.org>2026-07-10 21:57:58 +0200
committererdgeist <erdgeist@erdgeist.org>2026-07-10 21:57:58 +0200
commit7d6a665b896252537b8b8df2f15e204d04417231 (patch)
tree37b8d5434426646a019c6d5c8c12f7bb8a4de647 /test/models/node_test.rb
parent48eed5f35ea798c91f1e0a77c13e751145acfc48 (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 'test/models/node_test.rb')
0 files changed, 0 insertions, 0 deletions