Code Review Plugin
Added by Haruyuki Iida over 15 years ago
Hi. Let me announce to you My "Code Review Plugin" .It makes you can write code reviews of committed source code on Redmine's "Diff" page of repository browser.
like this...
Adding review.¶
Showing reviews.¶
I tested this plugin on Redmine 0.8.3 and current trunk.
I hope you will like this plugin. Thanks.
add_review.png (21.4 KB) add_review.png | Adding rview | ||
show_reviews.png (45 KB) show_reviews.png | Showing reviews |
Replies (118)
RE: Code Review Plugin
-
Added by Haruyuki Iida over 15 years ago
plugin is here.
redmine_code_review-0.1.0.zip (33.1 KB) redmine_code_review-0.1.0.zip | Code Review Plugin 0.1.0 |
RE: Code Review Plugin
-
Added by Andrew Chaika over 15 years ago
Nice plugin, but I have catch an error:
ActionView::TemplateError (Called id for nil, which would mistakenly be 4 -- if you really wanted the id of nil, use object_id) on line #50 of vendor/plugins/redmine_code_review/app/views/code_review/_new_form.html.erb: 47: </p> 48: <p> 49: <%= submit_to_remote 'apply', l(:button_apply), :update => 'review-form', 50: :url => {:controller => 'code_review', :action => 'new', :id=>@project, :line => @line}, :method => :post, :change_id => @change.id, 220 => "$('review-form-frame').hide();" %> 51: <input type="button" value="<%=h l(:button_cancel) %> " onclick="$('review-form-frame').hide();"/> 52: </p> 53: <%= wikitoolbar_for 'review_comment' %> vendor/plugins/redmine_code_review/app/views/code_review/_new_form.html.erb:50 D:/Work/New/Redmine/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_view/helpers/form_helper.rb:313:in `fields_for' D:/Work/New/Redmine/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_view/helpers/form_helper.rb:253:in `form_for'
after changing the 50-th line to:
:url => {:controller => 'code_review', :action => 'new', :id=>@project, :line => @line}, :method => :post, :change_id => @review.change_id, 220 => "$('review-form-frame').hide();" %>
all work okay.
I think that "My page" block with code reviews will be useful or maybe mail notification.
RE: Code Review Plugin
-
Added by Andrew Chaika over 15 years ago
PS It's good idea with popup windows, I think that It's need to use they in other places of Redmine, for example when there is used Web 1.0 legacy "input" JavaScript function.
RE: Code Review Plugin
-
Added by Thomas M over 15 years ago
Very interesting,
by the way I have also catched an error.
When clicking on a review from the Code Review list (project wise) I got a Redmine "Revision not found" error.
The trace is :
Processing CodeReviewController#update_diff_view (for 152.xx.xx.xx at 2009-05-11 11:18:56) [POST] Session ID: b1673107f867f1f24ebb1d59cc4bf3e7 Parameters: {"rev"=>"847", "action"=>"update_diff_view", "id"=>"tools-workshop", "controller"=>"code_review", "review_id"=>"1", "path"=>"/io/bla/BLA/dev_verif/sources/com.c"} NoMethodError (undefined method `changes' for nil:NilClass): /vendor/plugins/redmine_code_review/app/controllers/code_review_controller.rb:78:in `update_diff_view' /vendor/rails/actionpack/lib/action_controller/base.rb:1166:in `send' /vendor/rails/actionpack/lib/action_controller/base.rb:1166:in `perform_action_without_filters' /vendor/rails/actionpack/lib/action_controller/filters.rb:579:in `call_filters' /vendor/rails/actionpack/lib/action_controller/filters.rb:572:in `perform_action_without_benchmark' /vendor/rails/actionpack/lib/action_controller/benchmarking.rb:68:in `perform_action_without_rescue' C:/Work/Ruby/Ruby/lib/ruby/1.8/benchmark.rb:293:in `measure' /vendor/rails/actionpack/lib/action_controller/benchmarking.rb:68:in `perform_action_without_rescue' /vendor/rails/actionpack/lib/action_controller/rescue.rb:201:in `perform_action_without_caching' /vendor/rails/actionpack/lib/action_controller/caching/sql_cache.rb:13:in `perform_action' /vendor/rails/activerecord/lib/active_record/connection_adapters/abstract/query_cache.rb:33:in `cache' /vendor/rails/activerecord/lib/active_record/query_cache.rb:8:in `cache' /vendor/rails/actionpack/lib/action_controller/caching/sql_cache.rb:12:in `perform_action' /vendor/rails/actionpack/lib/action_controller/base.rb:529:in `send' /vendor/rails/actionpack/lib/action_controller/base.rb:529:in `process_without_filters' /vendor/rails/actionpack/lib/action_controller/filters.rb:568:in `process_without_session_management_support' /vendor/rails/actionpack/lib/action_controller/session_management.rb:130:in `process' /vendor/rails/actionpack/lib/action_controller/base.rb:389:in `process'Your work is very promising, but it does not really fit my needs as is.
Here are some points :
- I think that having one "Review" (in the list) per comment is far too much
- I think that commenting right from the diff view is simple but to systematic (even if it may be solved with rights)
- a list of file diffs (why not having a creation process with start/end path and revisions ?)
- reviewed by several guys (not only the author)
- finally acknowledged by someone (the project manager ?)
While waiting for your point of view please accept my best greatings for your work !
RE: Code Review Plugin
-
Added by Haruyuki Iida over 15 years ago
Hi, Andrew.
Thank you for your bug reporting. I fixed my plugin.
And your "My page" block idea seems very usefull. I will try it.
redmine_code_review-0.1.1.zip (33.1 KB) redmine_code_review-0.1.1.zip | Code Review Plugin 0.1.1 |
RE: Code Review Plugin
-
Added by Haruyuki Iida over 15 years ago
Hi. Thomas
Thank you for your error reporting.
But unfortunately I can't fix it yet. Please wait.
And I understand your thought for my plugin. It seems quite difficult. I will examine how to implement it.
RE: Code Review Plugin
-
Added by Thomas M over 15 years ago
Hi Haru,
thanks for your answer, if you need more info feel free to ask.
You will find enclosed a French translation, I was not able to use Latin characters (rake error during db:migrate_plugins) : any idea ?
RE: Code Review Plugin
-
Added by Adrian Tarau over 15 years ago
Very nice,
Does the committer receives a notification about the review? Actually it should be sent to every user...if the per user notification is enabled.
RE: Code Review Plugin
-
Added by Haruyuki Iida over 15 years ago
Hi Thomas,
Thanks for Frech translation. I made a new package which contains your file and a few bug fixes.
You can download it form Google Code.
http://code.google.com/p/r-labs/downloads/list
Well, I have a few questions about a error you reported.if you need more info feel free to ask.
- What kind of SCM do you use? Subversion?
- Does the error occurs on every review you created?
You will find enclosed a French translation, I was not able to use Latin characters (rake error during db:migrate_plugins) : any idea ?
Sorry, I have no idea about it.
RE: Code Review Plugin
-
Added by Haruyuki Iida over 15 years ago
Adrian Tarau wrote:
Does the committer receives a notification about the review? Actually it should be sent to every user...if the per user notification is enabled.
Thank you for a comment. Maybe in a few weeks.
RE: Code Review Plugin
-
Added by Thomas M over 15 years ago
Haru Iida wrote:
You can download it form Google Code.
I will test it tomorrow and let you know.
Well, I have a few questions about a error you reported.if you need more info feel free to ask.
- What kind of SCM do you use? Subversion?
- Does the error occurs on every review you created?
Yes I'm using SVN and the error occurs on every review.
You will find enclosed a French translation, I was not able to use Latin characters (rake error during db:migrate_plugins) : any idea ?
Sorry, I have no idea about it.
I will have a look into the other plugins (who have worked ;))
RE: Code Review Plugin
-
Added by Thomas M over 15 years ago
- in the list, directory and file name are not the correct ones (it is a real file but from another repository at another revision)
- when opening the correct file back from the repository viewer the reviews are well placed !
RE: Code Review Plugin
-
Added by Muhamed niyas over 15 years ago
Haru Iida wrote:
plugin is here.
Good work. I could do the code commenting very well.
The path showing in the 'code review' tab is wrong for each code comment.
If we click on the path redmine shows an error message like
"The entry or revision was not found in the repository."
Can you help me to fix this issue.?
pls find the screen shot attached...
I have another doubt..
How can v upload a photo of the user in redmine?
code_coment.jpg (41.7 KB) code_coment.jpg | code review details |
RE: Code Review Plugin
-
Added by Haruyuki Iida over 15 years ago
Thank you.
But I also have several projects and repositories too. 2 SVNs, git, Mercurial and CVS. Each project works well.
What version of your redmine? and Is it running on development mode or production mode?
RE: Code Review Plugin
-
Added by Haruyuki Iida over 15 years ago
Hi Muhamed
Thanks. Your problem is maybe same as Thomas M's one. and I'm trying to fix it. Please wait.
I have another doubt..
How can v upload a photo of the user in redmine?
Use Gravater or Local avatars Plugin.
http://www.redmine.org/boards/3/topics/5365
RE: Code Review Plugin
-
Added by Thomas M over 15 years ago
Haru Iida wrote:
Thank you.
But I also have several projects and repositories too. 2 SVNs, git, Mercurial and CVS. Each project works well.
What version of your redmine? and Is it running on development mode or production mode?
I'm using Redmine 0.8.3 in production mode (with recommended ruby/rails settings and MySQL).
RE: Code Review Plugin
-
Added by Muhamed niyas over 15 years ago
Haru Iida wrote:
Thank you.
But I also have several projects and repositories too. 2 SVNs, git, Mercurial and CVS. Each project works well.
What version of your redmine? and Is it running on development mode or production mode?
Thanks for your immediate response.
I am using redmine 0.8.3 version. and redmine is running in production mode.
RE: Code Review Plugin
-
Added by Denis Tomashenko over 15 years ago
Great plugin. Thank you!
Some questions and bugs:
1. If all reviews are closed - there is no checkbox "Show closed reviews" and I can't view closed reviews.
2. Can I make link to review like "r100" or "#100"?
RE: Code Review Plugin
-
Added by Denis Tomashenko over 15 years ago
3. If I close review - initial review on activity page is removed. Stay only closed entry.
4. Maybe use different image for closed reviews?
And once again - thank you.
RE: Code Review Plugin
-
Added by Haruyuki Iida over 15 years ago
Hi Denis.
Thanks for your report.
I released redmine_code_review-0.1.3.zip on http://code.google.com/p/r-labs/downloads/list
Denis Tomashenko wrote:
1. If all reviews are closed - there is no checkbox "Show closed reviews" and I can't view closed reviews.
I fixed.
2. Can I make link to review like "r100" or "#100"?
I wrote wiki macro. Write "{{review(review_no)}}" in wiki format.
ex. linkng to review no. 8.
{{review(8)}}
Try it.
Thank you.
RE: Code Review Plugin
-
Added by Thomas M over 15 years ago
On my side all created review are always linking to the same file that is from another repository.
Nevertheless they seem to be recorded fine : if I open the diff back the comment is placed correctly.
RE: Code Review Plugin
-
Added by Haruyuki Iida over 15 years ago
Thomas M wrote:
On my side all created review are always linking to the same file that is from another repository.
Nevertheless they seem to be recorded fine : if I open the diff back the comment is placed correctly.
Thanks.
I changed the way reading repository data with redmine_code_review-0.1.3.1.
You can find it on http://code.google.com/p/r-labs/downloads/list
But I'm not sure it solve your problem.
Could you try it and inform me does it works or not?
RE: Code Review Plugin
-
Added by Thomas M about 15 years ago
Hi Haru,
I'm still having the same behaviour, sorry.
RE: Code Review Plugin
-
Added by Muhamed niyas about 15 years ago
Haru Iida wrote:
Hi. Let me announce to you My "Code Review Plugin" .It makes you can write code reviews of committed source code on Redmine's "Diff" page of repository browser.
like this...
Adding review.¶
Showing reviews.¶
I tested this plugin on Redmine 0.8.3 and current trunk.
I hope you will like this plugin. Thanks.
I got this error.
ActiveRecord::RecordNotFound in Code_review#index Showing code_review/index.html.erb where line #79 raised: Couldn't find Change with ID=4 Extracted source (around line #79): 76: </td> 77: <td colspan="2" class="path"> 78: <% 79: codepath = review.path 80: if (review.path.length > 55) 81: codepath = review.path[0, 15] + '...' + review.path[review.path.length - 35, 35] 82: end
can you help me to fix this issue?