diff --git a/earwigbot/tasks/drn_clerkbot.py b/earwigbot/tasks/drn_clerkbot.py index f41964d..d4d262a 100644 --- a/earwigbot/tasks/drn_clerkbot.py +++ b/earwigbot/tasks/drn_clerkbot.py @@ -293,7 +293,8 @@ class DRNClerkBot(Task): return notices def clerk_open_case(self, case, signatures): - self.check_for_review(case) + if self.check_for_review(case): + return [] if len(case.body) - case.last_volunteer_size > 15000: if case.last_action != self.STATUS_NEEDASSIST: case.status = self.STATUS_NEEDASSIST @@ -305,14 +306,16 @@ class DRNClerkBot(Task): return [] def clerk_needassist_case(self, case, volunteers, newsigs): - self.check_for_review(case) + if self.check_for_review(case): + return [] if any([editor in volunteers for (editor, timestamp) in newsigs]): if case.last_action != self.STATUS_OPEN: case.status = self.STATUS_OPEN return [] def clerk_stale_case(self, case, newsigs): - self.check_for_review(case) + if self.check_for_review(case): + return [] if newsigs: if case.last_action != self.STATUS_OPEN: case.status = self.STATUS_OPEN @@ -351,6 +354,8 @@ class DRNClerkBot(Task): if age > 60 * 60 * 24 * 4: if case.last_action != self.STATUS_REVIEW: case.status = self.STATUS_REVIEW + return True + return False def read_signatures(self, text): regex = r"\[\[(?:User(?:\stalk)?\:|Special\:Contributions\/)"