PATCH: Bug Updating Ship It Count

2009-07-28 Thread Michal

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
-~--~~~~--~~--~--~---



Re: PATCH: Bug Updating Ship It Count

2009-07-28 Thread Christian Hammond
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
-~--~~~~--~~--~--~---