ForumMagnum / ForumMagnum / 25463845060

48%

master: 47%

Build Type

Pull #12497

github

Committed by claude

Commit Message

Document the ES indexer's index-everything-filter-at-query-time convention

Add a paragraph to the IndexConfig.filters doc comment so the next person
(or AI agent) touching the search code knows that ForumMagnum indexes
every row regardless of visibility flags and does the filtering here at
query time, not in the SQL export methods.

I half-implemented the alternative pattern earlier on this branch (added
WHERE rejected IS NOT TRUE to the bulk-export queries) and reverted it
in fb9c12f because the per-document update path always re-indexes, so
the filter just creates drift between recreateIndex and steady-state
behavior. Captured the rationale here so the same trap doesn't catch
the next contributor.

Pull Request Pull Request #12497: Filter rejected content from shared queries and clean up notifications

Coverage Stats

7921 of 10714 branches covered (73.93%)

Branch coverage included in aggregate %.

29 of 42 new or added lines in 9 files covered. (69.05%)

94 existing lines in 7 files now uncovered.

177816 of 373127 relevant lines covered (47.66%)

36.53 hits per line