diff options
| author | Stephen Finucane <stephen@that.guru> | 2016-10-20 08:28:56 +0100 |
|---|---|---|
| committer | Stephen Finucane <stephen@that.guru> | 2016-10-22 11:22:58 +0100 |
| commit | 1be8980720d03d0fa81057d5c554453689de561d (patch) | |
| tree | 056a0ac2ed7c526bd35ac917aaf371836566ca03 | |
| parent | d497c2b234756a8846d86cbf704cec4bd91bf915 (diff) | |
| download | patchwork-1be8980720d03d0fa81057d5c554453689de561d.tar patchwork-1be8980720d03d0fa81057d5c554453689de561d.tar.gz | |
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>
| -rw-r--r-- | patchwork/models.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/patchwork/models.py b/patchwork/models.py index f6a2f26..b18213d 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 |