README Dev Note #24's StatsModel invalidation caller list is missing an 8th caller (documentation gap, behavior is correct) #92

New Issue

2026-09-01T00:08:20-04:00

jared commented

2026-09-01 00:08:20 -04:00

Severity: Low / docs

README.md Developer Notes #24 lists 7 callers of StatsModel::invalidateCache(). Confirmed via exhaustive trace that api/ticket_status_api.php (the Bearer API's status-change endpoint) also correctly calls invalidateCache(), but isn't in the README's list.

Behavior itself is correct — no actual stats-staleness bug, this is purely a documentation completeness gap. Worth updating the README's caller list so it stays an accurate reference for future maintainers deciding whether a new mutating path needs to add the same call.

Fix: Add api/ticket_status_api.php to the "Callers:" list in README.md Dev Note #24.

**Severity:** Low / docs README.md Developer Notes #24 lists 7 callers of `StatsModel::invalidateCache()`. Confirmed via exhaustive trace that `api/ticket_status_api.php` (the Bearer API's status-change endpoint) also correctly calls `invalidateCache()`, but isn't in the README's list. Behavior itself is correct — no actual stats-staleness bug, this is purely a documentation completeness gap. Worth updating the README's caller list so it stays an accurate reference for future maintainers deciding whether a new mutating path needs to add the same call. **Fix:** Add `api/ticket_status_api.php` to the "Callers:" list in README.md Dev Note #24.

jared added the documentation priority/low labels 2026-09-08 10:15:49 -04:00

jared referenced this issue from a commit

2026-09-08 11:59:38 -04:00

README: add missing StatsModel::invalidateCache() caller (#92)

jared referenced this issue from a commit

2026-09-08 15:39:42 -04:00

Merge development into main: 15-issue triage + fix batch

Sign in to join this conversation.