On Tue, 2017-08-22 at 12:47 +1000, Andrew Donnellan wrote: > On 21/08/17 00:40, Andrew Donnellan wrote: > > In Comment.save() and Comment.delete(), we always call > > Submission.refresh_tag_counts(), which is an empty stub, rather than > > calling Patch.refresh_tag_counts() if the Submission is a Patch. > > > > As such, tag counts are never updated on incoming comments. > > > > Delete Submission.refresh_tag_counts(), as it's useless, and in > > Comment.save()/delete(), invoke Patch.refresh_tag_counts() directly when > > the submission is a Patch. > > > > Reported-by: David Demelier <mark...@malikania.fr> > > Fixes: 86172ccc161b ("models: Split Patch into two models") > > Closes-bug: #111 ("A/R/T not updated on comments") > > Signed-off-by: Andrew Donnellan <andrew.donnel...@au1.ibm.com> > > Stephen - is this the last bug that's blocking the release? Happy to > look at any other remaining blockers. > > I've pushed this fix to py{2,3}.patchwork.dja.id.au.
Yup, this looks good to me. Reviewed-by: Stephen Finucane <step...@that.guru> Stephen _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork