1510: Exception thrown during post-review -- 'rsp' is used before being defined

rich****@gmai***** (Google Code) (Is this you? Claim this profile.)
Feb. 26, 2010
1537, 1548
What version are you running?
RBTools 0.2RC1

What is the expected output? What do you see instead?

An exception is thrown in post-review.  'rsp' is used before being defined.

What operating system are you using? What browser?

Windows 7 64-bit

Please provide any additional information below.

post-review.py:427, the else clause in the "except APIError, e:" clause.  I 
believe rsp['review_request']['id'] should be 
e.rsp['review_request']['id'].
Making this change allowed post-review to work without issue.
chipx86
#1 chipx86
  • +PendingReview
  • -Priority-Medium
    +Priority-Critical
    +Milestone-RBTools-Release1.0
    +Component-RBTools
#2 lon***@gmai***** (Google Code) (Is this you? Claim this profile.)
Please see http://reviews.reviewboard.org/r/1427/
chipx86
#3 chipx86
This has been committed. The nightlies should have the fix.
  • -PendingReview
    +Fixed