From: Stephen Finucane <stephen@that.guru>
Date: Thu, 20 Oct 2016 07:28:56 +0000 (+0100)
Subject: models: Remove TODO from 'refresh_tag_counts'
X-Git-Tag: v2.0.0-rc1~184
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=1be8980720d03d0fa81057d5c554453689de561d;p=thirdparty%2Fpatchwork.git

models: Remove TODO from 'refresh_tag_counts'

There's actually no harm keeping this around as it makes the code.
Remove the TODO, replacing it with a note on why the function exists.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Reviewed-by: Daniel Axtens <dja@axtens.net>
---

diff --git a/patchwork/models.py b/patchwork/models.py
index f6a2f266..b18213d6 100644
--- a/patchwork/models.py
+++ b/patchwork/models.py
@@ -296,7 +296,8 @@ class Submission(EmailMixin, models.Model):
     # patchwork metadata
 
     def refresh_tag_counts(self):
-        pass  # TODO(sfinucan) Once this is only called for patches, remove
+        # This is subclassed on 'Patch' to do something useful
+        pass
 
     def is_editable(self, user):
         return False