Opened 14 years ago

#6500 new defect

Resubmit for Review button throws Unicode coersion error

Reported by: james.tait@… Owned by: Jörg Viola
Priority: normal Component: CodeReviewPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10


I'm trialling this hack with a small team and have come across a problem. I have assigned myself CODE_REVIEW_DEV and CODE_REVIEW_MGR permissions, and my devs CODE_REVIEW_DEV permission. I created a new code review on r4097 of our repo, added a bunch of files that were newly added to the branch in that revision. I then went ahead with the review, adding comments to several files and ultimately rejecting the review. So far so good.

My developer then re-worked the files in question and committed the changes to the branch. At this point I'm not quite sure how it's supposed to work, but my assumption is that I click the Resubmit for Review button, select a new revision for the review and go through the process again. Unfortunately when I click the Resubmit for Review button I get the following error:

Traceback (most recent call last):
  File "C:\Python24\lib\site-packages\trac\web\", line 387, in dispatch_request
  File "C:\Python24\lib\site-packages\trac\web\", line 237, in dispatch
    resp = chosen_handler.process_request(req)
  File "build\bdist.win32\egg\codereview\", line 90, in process_request
TypeError: coercing to Unicode: need string or buffer, int found

I haven't attempted to investigate in the code.

We're using Trac on Python 2.4 with a SQLite backend on Windows 2003.

Attachments (1)

CodeReviewPlugin-6500.diff (1.7 KB) - added by james.tait@… 14 years ago.
Patch to fix - tested on 0.10.

Download all attachments as: .zip

Change History (1)

Changed 14 years ago by james.tait@…

Attachment: CodeReviewPlugin-6500.diff added

Patch to fix - tested on 0.10.

Modify Ticket

Change Properties
Set your email in Preferences
as new The owner will remain Jörg Viola.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.