[PATCH 1/2] models: Fix invocation of refresh_tag_counts() for Comments

Andrew Donnellan andrew.donnellan at au1.ibm.com
Tue Aug 22 12:47:42 AEST 2017


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 <markand at 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.donnellan at 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.

-- 
Andrew Donnellan              OzLabs, ADL Canberra
andrew.donnellan at au1.ibm.com  IBM Australia Limited



More information about the Patchwork mailing list