Issue 2313 in reviewboard: Base64Field requires an AutoField key to operate

2011-10-02 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 2313 by dru...@gmail.com: Base64Field requires an AutoField key  
to operate

http://code.google.com/p/reviewboard/issues/detail?id=2313

* NOTE: Do not post confidential information in this bug report. *
*   If you need immediate support, please contact*
*   reviewbo...@googlegroups.com *

What version are you running?
Master branch of Djblets

What steps will reproduce the problem?
1. Create a model without an auto incrementing primary key id.
2. Add a Base64Field to the model.
3. Create an instance of the model.

What is the expected output? What do you see instead?
Expect the model to be created. Instead, a base64 conversion padding  
exception is thrown.


Please provide any additional information below.
This exists in djblets/util/fields.py

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



Re: Issue 2313 in reviewboard: Base64Field requires an AutoField key to operate

2011-10-02 Thread reviewboard


Comment #2 on issue 2313 by dru...@gmail.com: Base64Field requires an  
AutoField key to operate

http://code.google.com/p/reviewboard/issues/detail?id=2313

Turning off id completely. Specifically, I was using a CharField as the  
primary key.

I will build a test case and get the specific error.

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



Re: Issue 1730 in reviewboard: post-review with mercurial fails with obscure error if there are no outgoing changesets

2011-10-02 Thread reviewboard


Comment #1 on issue 1730 by m...@catley.net.nz: post-review with mercurial  
fails with obscure error if there are no outgoing changesets

http://code.google.com/p/reviewboard/issues/detail?id=1730

I have noticed this too, and have written a fix for our internal version of  
rbtools. Is there any appetite for getting this into the main repository?


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



Re: Issue 1730 in reviewboard: post-review with mercurial fails with obscure error if there are no outgoing changesets

2011-10-02 Thread reviewboard


Comment #2 on issue 1730 by trowb...@gmail.com: post-review with mercurial  
fails with obscure error if there are no outgoing changesets

http://code.google.com/p/reviewboard/issues/detail?id=1730

Yes please.

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



Re: Issue 1730 in reviewboard: post-review with mercurial fails with obscure error if there are no outgoing changesets

2011-10-02 Thread reviewboard


Comment #3 on issue 1730 by m...@catley.net.nz: post-review with mercurial  
fails with obscure error if there are no outgoing changesets

http://code.google.com/p/reviewboard/issues/detail?id=1730

See http://reviews.reviewboard.org/r/2632.

Also I have a bug/review out that fixes the calculation of diffs if a merge  
has occurred with post-review and mercurial. See:

http://code.google.com/p/reviewboard/issues/detail?id=2311
http://reviews.reviewboard.org/r/2628/

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