ForumMagnum / ForumMagnum / 25464730841 / 2

48%

master: 47%

Pull #12497

github

Revert redundant rejected filters in getCommentsWithReacts and getRecentCommentsOnPosts

Two more places where the original "rejected" filter I added isn't doing
useful work:

- getCommentsWithReacts: not used on EA Forum at all. The query also
  doesn't filter deleted/retracted/spam, so a rejected-only filter was
  asymmetric anyway. Drop the change (and the inner-subquery JOIN to
  Comments that I added to make LIMIT honor it).

- getRecentCommentsOnPosts: the caller in lib/collections/posts/helpers.ts
  builds its filter starting with getDefaultViewSelector("Comments"), which
  already includes rejected/deleted/draft handling and is then compiled
  into the SQL via SelectQuery. The hardcoded "rejected" IS NOT TRUE clause
  I added was redundant with the compiled filter. Same flavor of mistake
  as the bulk-indexer fix: half-implementing a backstop pattern at a layer
  that doesn't need it. If a future caller passes a filter without the
  default view, the fix belongs at that call site.

4954 of 6289 branches covered (78.77%)

Branch coverage included in aggregate %.

157300 of 372530 relevant lines covered (42.22%)

1.93 hits per line