620: SQL exception if perforce changelist description too long

ciao****@gmai***** (Google Code) (Is this you? Claim this profile.)
Aug. 28, 2008
What's the URL of the page containing the problem?
https://<server>/api/json/reviewrequests/10/update_from_changenum/

What steps will reproduce the problem?
1. Create a perforce changelist where the description is > 300 characters 
long without any line breaks. 
2. Use post-review to post the changelist.
3. Provided the perforce changelist description is > 300 characters and is 
one sentence (no line breaks) you'll get an exception thrown: "Data 
truncated for column &#39;summary&#39; at row 1". This is because 
reviewboard is trying to use the full description text in the 'summary' 
database field which is limited to 300 characters.
  
Using the --summary option in post-review does NOT work around this 
problem. It appears the summary option isn't taking affect for perforce 
(another bug). 

What is the expected output? What do you see instead?
To work around the issue you must shorten the first sentence in the change 
description to be < 300 characters. 

The ReviewBoard source code should automatically limit the summary field 
to < 300 characters so taht MySQL doesn't throw an exception.

What operating system are you using? What browser?


Please provide any additional information below.
david
#1 david
This was just fixed in SVN last night. Duplicate of bug 438.
  • +Duplicate