Re: About the integration of Reviewboard and CVS

2012-08-18 Thread cmuser
did u get answer on how to do this?

On Thursday, January 7, 2010 5:35:56 PM UTC+5:30, JohnHenry wrote:

 Hi, all.
I am doing the task of RB integration with CVS server. We want to
 use loginfo file of CVS server to add scripts, so that after each
 code commit, a review-request will be generated for the commitor ( the
 review requestor ) automatically. But I found that, a script
 referenced by loginfo file will executed many times after one
 commition (if this commition files in different subdirectories).But I
 want that even if I had commited files of different subdirectories,
 the script will execute only once, so that I can make a single
 ReviewRequest for this commit. Can any body give me some suggestion?

 Best Regards!


-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
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

Diff comments seem to be broken

2012-08-18 Thread Sami Paavilainen
This is on version 1.6.10 of Review Board.

So, the symptoms:
The submitter is unable to add comments to review notes placed on diffs, and is 
also unable to resolve/drop issues opened on these.
Adding comments to the review itself works without a problem.

I opened firebug and had a look at this, and I think I've found what's 
happening, altough I cannot say why it would be happening.
This piece of code from reviews.js is the culprit (at least for being unable to 
add comments):
http://pastebin.com/XkL9PCtT

(Or if you already have the source, jump to line 522 in reviews.js)

This is called when clicking OK of the comment window. The reason the code 
fails for us is that the value of context_type is diff_comment, which is 
unexpected by the following if/else tree, causing it to go through the branch 
with the comment Shouldn't be reached, and returning without taking any 
action.

Similar thing happens with the resolve/drop buttons, value of context_type is 
diff_comment causing the code for them to also fail.

This is from the page source:
script language=javascript
$(document).ready(function() {
$(#rc187-49).commentSection(
49,
187,
diff_comments);
});
/script 

So that's where the diff_comments is coming from, and this is where I'll stop 
for now.

Can anyone more familiar with the code confirm this?

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
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