mroderick / codebar-planner / 22314229045

96%

master: 95%

push

github

fix: eager load permissions and organisers in DashboardController

Motivation:
Bullet was reporting N+1 queries on the homepage (/). The view accesses
event.organisers which requires the permissions join table to be eager loaded.

Analysis:
- The all_events method loads Event and Workshop objects that need
  organisers and permissions preloaded
- Also fixed Meeting eager loading to include permissions
- Changed from includes() to eager_load() to handle nested through-associations
  (organisers goes through permissions)

Before: N+1 queries per event for organisers
After: Single query with JOIN on permissions table.

6 of 6 new or added lines in 1 file covered. (100.0%)

3315 of 3466 relevant lines covered (95.64%)

45.4 hits per line