Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Make /r/mod/about/spam also include filtered comments #397

Closed
wants to merge 1 commit into from

2 participants

@Deimos
Owner

Seems to have just been an oversight - /r/mod/about/spam doesn't include comments, but single-subreddit spam pages and /r/mod/about/modqueue both do.

@bsimpson63
Owner

Merged as 1fd9a08

@bsimpson63 bsimpson63 closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 30, 2012
  1. @Deimos
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 1 deletion.
  1. +3 −1 r2/r2/lib/db/queries.py
View
4 r2/r2/lib/db/queries.py
@@ -315,7 +315,9 @@ def get_spam_comments(sr):
def get_spam(sr):
if isinstance(sr, ModContribSR):
srs = Subreddit._byID(sr.sr_ids, return_dict=False)
- results = [ get_spam_links(sr) for sr in srs ]
+ results = []
+ results.extend(get_spam_links(sr) for sr in srs)
+ results.extend(get_spam_comments(sr) for sr in srs)
return merge_results(*results)
else:
return merge_results(get_spam_links(sr),
Something went wrong with that request. Please try again.