you are viewing a single comment's thread.

view the rest of the comments →

[–]magnora7[S] 4 insightful - 2 fun4 insightful - 1 fun5 insightful - 2 fun -  (2 children)

Marking spam and removing are separate operations an admin can do to remove a comment. I am not sure the difference between the two, honestly. Maybe when it's removed it does mark it as item._spam. Does item._removed exist as a classifier, I wonder?

The code bit you pointed out looks very close to what we need though, you're definitely hot on the trail

[–]teelo 3 insightful - 2 fun3 insightful - 1 fun4 insightful - 2 fun -  (1 child)

Does item._removed exist as a classifier

I think this will answer that question for us! test_validator.py, 237:

    def test_removed_comment(self):
    comment = self._mock_comment(_spam=True)
    result = self.validator.run('fullname', None)

    self.assertEqual(result, comment)
    self.assertTrue(self.validator.has_errors)
    self.assertIn((errors.DELETED_COMMENT, None), c.errors)

(I searched "_remove" and no results outside of the jQuery libraries)

[–]magnora7[S] 5 insightful - 2 fun5 insightful - 1 fun6 insightful - 2 fun -  (0 children)

Alright looks like a pretty promising start! I think this is about half the info needed to make this thing happen, thanks for your help so far! I'll add this thread to our to-do list so we can reference it later when we try to build the feature out fully