Hi Michal,

Can you post this on http://reviews.review-board.org/?

Thanks,

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.review-board.org
VMware, Inc. - http://www.vmware.com


On Mon, Jul 27, 2009 at 11:04 PM, Michal <mic...@mxg.id.au> wrote:

>
> Hello,
>
> In Review Board 1.0, there seems to be an issue which causes the "ship
> it" count on a review request to not update correctly.  I think it
> only manifests if e-mail notification is off.
>
> Bellow is a patch which fixes the issue.
>
>
> Michal
>
>
> Index: reviewboard/reviews/models.py
> ===================================================================
> --- reviewboard/reviews/models.py       (revision 2071)
> +++ reviewboard/reviews/models.py       (working copy)
> @@ -1064,11 +1064,11 @@
>
>         # Update the last_updated timestamp on the review request.
>         self.review_request.last_review_timestamp = self.timestamp
> -        self.review_request.save()
>
>         # Atomicly update the shipit_count
>         if self.ship_it:
>             self.review_request.increment_ship_it()
> +        self.review_request.save()
>
>     def delete(self):
>         """
>
> >
>

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To post to this group, send email to reviewboard@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to