Lighter list presenter for /api/subjects (performance, future) #11

New issue

Open

opened 2026-06-06 01:25:07 +02:00 by myrmidex · 0 comments

myrmidex commented

2026-06-06 01:25:07 +02:00

Owner

Low priority — note for later.

SubjectPresenter::present() currently expands all ratings and computes aggregates for both the list endpoint (GET /api/subjects) and the detail endpoint (GET /api/subjects/{id}). This means the map-load fetches every rating for every subject just to draw pins — fine at current scale, wasteful as data grows.

Scope: Add a lighter presentForList() method to SubjectPresenter that returns only the fields needed for map pins (id, name, lat, lng) without expanding ratings. Use it in the index action; keep the full present() for the show action.

Size: small

**Low priority — note for later.** `SubjectPresenter::present()` currently expands all ratings and computes aggregates for both the list endpoint (`GET /api/subjects`) and the detail endpoint (`GET /api/subjects/{id}`). This means the map-load fetches every rating for every subject just to draw pins — fine at current scale, wasteful as data grows. **Scope:** Add a lighter `presentForList()` method to `SubjectPresenter` that returns only the fields needed for map pins (id, name, lat, lng) without expanding ratings. Use it in the `index` action; keep the full `present()` for the `show` action. **Size: small**