Redmine

1. The side-by-side diffs do not get syntax highlighting and appear normal.
2. I discovered last week that the line spacing is screwy on projects that do not have the code review plugin enabled.

I will have a look at the pygments plug.

Hi,
I´m having problems installing the code review plugin. I copied the plugin to the vendor directory
and when I run the rakefile I get this:
@
C:\Program Files\BitNami Redmine Stack\ruby\bin>rake -f "C:\Program Files\BitNam
i Redmine Stack\apps\redmine\vendor\plugins\redmine_code_review\rakefile" rake
db:migrate_plugins RAILS_ENV=production
(in C:/Program Files/BitNami Redmine Stack/ruby/bin)
rake aborted!
No such file or directory - init.rb

(See full trace by running task with --trace)

C:\Program Files\BitNami Redmine Stack\ruby\bin>@

Hi Ezequiel.

You specified a wrong rakefile for database migration.
You must specify the Redmine's one. Not a plugin's.

try

rake -f "C:\Program Files\BitNami Redmine Stack\apps\redmine\Rakefile" db:migrate_plugins RAILS_ENV=production --trace

Thanks Haru, its working now!

Steven Risner wrote:

I will have a look at the pygments plug.

Hi Steven.

I found how to fix the pygments plugin.

Change redmine_repo_pygments/lib/repo_pygments_application_helper_patch.rb: line 24 from

@code_css = "highlight"

to

@code_css = "line-code highlight"

and change redmine_repo_pygments/lib/python_code/pyscm.py: line 50 from

pattern_str = '^<div\sclass=\"highlight\"><pre>(.*</span>)'

to

pattern_str = '^<div\sclass=\"line-code highlight\"><pre>(.*</span>)'

Then it works. Try this.

I'm only getting this to work for the "admin" user. No other user can see the pen icons. Can't find anyone else mentioning this so I must be doing something wrong... Any ideas?

André Jonsson wrote:

I'm only getting this to work for the "admin" user. No other user can see the pen icons. Can't find anyone else mentioning this so I must be doing something wrong... Any ideas?

Scratch that. N00b redmine admin at work :-/ I hadn't checked the code review permission boxes. Never mind.

Thanks Haru

You saved me a lot of time.

Haru Iida wrote:

Change redmine_repo_pygments/lib/repo_pygments_application_helper_patch.rb: line 24 to
@code_css = "line-code highlight" 

change redmine_repo_pygments/lib/python_code/pyscm.py: line 50 to
pattern_str = '^<div\sclass=\"line-code highlight\"><pre>(.*</span>)'

</pre>

You should put this fix in a readme or on your plugin page.

I successfully managed to install the latest version of the plugin. I added the code review module to my project, I changed roles and permissions for my role to be able to do anything with code reviews. I can see and enjoy nice review tabs in the project page and in the project settings tabs. I can see and use “Review assignments: Add” button. BUT!... there is no edit icon whatsoever anywhere in the repository browser as it is described here. And therefore I cannot right reviews. What is wrong?

I use Redmine 0.9.3.stable with PostgreSQL 8.4.

Nox Metus wrote:

I successfully managed to install the latest version of the plugin. I added the code review module to my project, I changed roles and permissions for my role to be able to do anything with code reviews. I can see and enjoy nice review tabs in the project page and in the project settings tabs. I can see and use “Review assignments: Add” button. BUT!... there is no edit icon whatsoever anywhere in the repository browser as it is described here. And therefore I cannot right reviews. What is wrong?

Hi, Nox.

I think it's a same case with this one. http://www.redmine.org/boards/3/topics/9627?r=15657#message-15657

Haru Iida wrote:

Hi, Nox.

I think it's a same case with this one. http://www.redmine.org/boards/3/topics/9627?r=15657#message-15657

Unfortunately, it is not. I've seen that message. And I set the full permission for everyone to this folder just in case:

ave@arcanum-squid:~$ ls -lA /usr/share/redmine/public/
total 40
-rw-r--r-- 1 root root  484 2010-02-28 16:28 404.html
-rw-r--r-- 1 root root  576 2010-02-28 16:28 500.html
-rwxr-xr-x 1 root root  971 2010-03-23 10:23 dispatch.fcgi
-rwxr-xr-x 1 root root  477 2010-02-28 16:28 dispatch.rb.example
-rw-r--r-- 1 root root    0 2010-02-28 16:28 favicon.ico
drwxr-xr-x 2 root root 4096 2010-09-05 03:20 help
drwxr-xr-x 4 root root 4096 2010-09-05 03:20 images
drwxr-xr-x 4 root root 4096 2010-09-05 03:20 javascripts
drwxrwxrwx 2 root root 4096 2010-09-06 16:19 plugin_assets
drwxr-xr-x 2 root root 4096 2010-09-05 03:20 stylesheets
drwxr-xr-x 5 root root 4096 2010-09-05 18:42 themes

I reinstalled everything from scratch, beginning from db server, then redmine, code review plugin, restored database, added code review module to the project, tuned permissions for code reviewing. The result is still the same: there is no edit icon whatsoever anywhere in the repository browser. What is wrong?

I've solved the problem by creating manually a folder public/plugin_assets/redmine_code_review and then copying there the contents of vendor/plugins/redmine_code_review/assets.

Nox Metus wrote:

I reinstalled everything from scratch, beginning from db server, then redmine, code review plugin, restored database, added code review module to the project, tuned permissions for code reviewing. The result is still the same: there is no edit icon whatsoever anywhere in the repository browser. What is wrong?

Exactly same problem here. We are using Redmine 1.0.1.devel.4068 (MySQL) and we have no "redmine_repo_pygments" installed and i set "chmod 700 ../plugin_assets/ -R" and chwon www-data.
I also checked that ".../redmine/public/plugin_assets/redmine_code_review" exists and the content of "../redmine/vendor/plugins/redmine_code_review/assets" is in there.

I got the tab Code-Reviews but there is no (Diff) link behind the file in repo browser, but a button to compare two revisions where i het a line by line view of a file, but np pencil's.

Please help!

Running 0.3.1 with Redmine 1.0.1 stable and Postgres. Works fine.
When I add Pygments plugin, it crashes.
Tried Haru's fix but all I get is:

ActionView::TemplateError (uninitialized constant RepoPygmentsApplicationHelperPatch::InstanceMethods::CodeRay) on line #5 of vendor/plugins/redmine_repo_pygments/app/views/common/_file.rhtml:
2:   <table class="filecontent CodeRay">
3:     <tbody>
4:       <% line_num = 1 %>
5:       <% syntax_highlight(filename, to_utf8(content)).each_line do |line| %>
6:         <tr>
7:           <th class="line-num" id="L<%= line_num %>"><a href="#L<%= line_num %>"><%= line_num %></a></th>
8:           <td class="<%= @code_css %>"><pre><%= line %></pre></td>

    vendor/plugins/redmine_repo_pygments/app/views/common/_file.rhtml:5:in `_run_rhtml_vendor47plugins47redmine_repo_pygments47app47views47common47_file46rhtml_locals_content_file_filename_object'
    app/views/repositories/entry.rhtml:11:in `_run_rhtml_app47views47repositories47entry46rhtml'
    passenger (2.2.15) lib/phusion_passenger/rack/request_handler.rb:92:in `process_request'
    passenger (2.2.15) lib/phusion_passenger/abstract_request_handler.rb:207:in `main_loop'
    passenger (2.2.15) lib/phusion_passenger/railz/application_spawner.rb:441:in `start_request_handler'
    passenger (2.2.15) lib/phusion_passenger/railz/application_spawner.rb:381:in `handle_spawn_application'
    passenger (2.2.15) lib/phusion_passenger/utils.rb:252:in `safe_fork'
    passenger (2.2.15) lib/phusion_passenger/railz/application_spawner.rb:377:in `handle_spawn_application'
    passenger (2.2.15) lib/phusion_passenger/abstract_server.rb:352:in `__send__'
    passenger (2.2.15) lib/phusion_passenger/abstract_server.rb:352:in `main_loop'
    passenger (2.2.15) lib/phusion_passenger/abstract_server.rb:196:in `start_synchronously'
    passenger (2.2.15) lib/phusion_passenger/abstract_server.rb:163:in `start'
    passenger (2.2.15) lib/phusion_passenger/railz/application_spawner.rb:222:in `start'
    passenger (2.2.15) lib/phusion_passenger/spawn_manager.rb:253:in `spawn_rails_application'
    passenger (2.2.15) lib/phusion_passenger/abstract_server_collection.rb:126:in `lookup_or_add'
    passenger (2.2.15) lib/phusion_passenger/spawn_manager.rb:247:in `spawn_rails_application'
    passenger (2.2.15) lib/phusion_passenger/abstract_server_collection.rb:80:in `synchronize'
    passenger (2.2.15) lib/phusion_passenger/abstract_server_collection.rb:79:in `synchronize'
    passenger (2.2.15) lib/phusion_passenger/spawn_manager.rb:246:in `spawn_rails_application'
    passenger (2.2.15) lib/phusion_passenger/spawn_manager.rb:145:in `spawn_application'
    passenger (2.2.15) lib/phusion_passenger/spawn_manager.rb:278:in `handle_spawn_application'
    passenger (2.2.15) lib/phusion_passenger/abstract_server.rb:352:in `__send__'
    passenger (2.2.15) lib/phusion_passenger/abstract_server.rb:352:in `main_loop'
    passenger (2.2.15) lib/phusion_passenger/abstract_server.rb:196:in `start_synchronously'

Rendering /usr/local/www/redmine/public/500.html (500 Internal Server Error)

I realize this might be not related to Haru's plugin but wondered if someone else encountered the same problem.

Thanks!

Terence Mill wrote:

I got the tab Code-Reviews but there is no (Diff) link behind the file in repo browser, but a button to compare two revisions where i het a line by line view of a file, but np pencil's.

Yes. The page which shows diff between two revisions has no pencil icon.
I mean when URL is like "/projects/XXX/repository/diff?rev=XXX&rev_to=XXX".

Pencil icon appears if URL has specific file path with revision.
ex)

• /projects/codereview/repository/revisions/555/diff/trunk/your/file/path
• /projects/codereview/repository/revisions/555/trunk/your/file/path

Paul paul wrote:

Running 0.3.1 with Redmine 1.0.1 stable and Postgres. Works fine.
When I add Pygments plugin, it crashes.

Is coderay-0.9.2 exists in vendor/plugins directory?
The error says that CodeRay not found in your system.

Yes, that directory is there. It contains lib/ and within it, README, coderay/ and coderay.rb

Haru Iida wrote:

Terence Mill wrote:

I got the tab Code-Reviews but there is no (Diff) link behind the file in repo browser, but a button to compare two revisions where i het a line by line view of a file, but np pencil's.

Yes. The page which shows diff between two revisions has no pencil icon.
I mean when URL is like "/projects/XXX/repository/diff?rev=XXX&rev_to=XXX".

Pencil icon appears if URL has specific file path with revision.
ex)

• /projects/codereview/repository/revisions/555/diff/trunk/your/file/path
• /projects/codereview/repository/revisions/555/trunk/your/file/path

• /myproject/repository/revisions/4
• /myproject/repository/revisions/4/entry/trunk/JiraImporter/src/jira_mapping.txt
• /myproject/repository/revisions/4/changes/trunk/JiraImporter/src/xml_parser.pl
• /myproject/repository/revisions/3/annotate/trunk/JiraImporter/.project

NONE OF THEM shows pencil!

Sorry, but please help me setp by setp to find that pencil.

Terence Mill wrote:

Sorry, but please help me setp by setp to find that pencil.

First make sure you have everything set up correctly. Under Administration/Roles, make sure the role you are assigned to has all of the Code Review roles checked. Also make sure you have code review module enabled on the Modules tab of the project settings. To confirm it's working, you should have a Code Reviews tab to the right of Repository.

Now click the Repository tab, and click on a revision number (do not click the "View Differences" button).
For example in my project, under the Latest Revisions heading, my latest revision is 12406. I click that link in the first column to get a list of the files changed in the revision.

Here, find a file you want to review. I find it most useful to do code reviews on diffs rather than entire files, so what I typically do is click the (diff) link next to the file. However, even if you click the file link (and not the diff link) you should still see the pencil icons.

Hope this helps!
Chris

Chris Lajoie wrote:

Terence Mill wrote:

Sorry, but please help me setp by setp to find that pencil.

First make sure you have everything set up correctly. Under Administration/Roles, make sure the role you are assigned to has all of the Code Review roles checked. Also make sure you have code review module enabled on the Modules tab of the project settings. To confirm it's working, you should have a Code Reviews tab to the right of Repository.

Now click the Repository tab, and click on a revision number (do not click the "View Differences" button).
For example in my project, under the Latest Revisions heading, my latest revision is 12406. I click that link in the first column to get a list of the files changed in the revision.

Here, find a file you want to review. I find it most useful to do code reviews on diffs rather than entire files, so what I typically do is click the (diff) link next to the file. However, even if you click the file link (and not the diff link) you should still see the pencil icons.

Hope this helps!
Chris

Ahh its working now .. i missed the right to set for code review in role setup. You can see review tab in project site, but cant see the pencil as admin. Btw. really coll Plugin, and works like a charm.
Good Job!

Chris Lajoie wrote:

Terence Mill wrote:

Sorry, but please help me setp by setp to find that pencil.

First make sure you have everything set up correctly. Under Administration/Roles, make sure the role you are assigned to has all of the Code Review roles checked. Also make sure you have code review module enabled on the Modules tab of the project settings. To confirm it's working, you should have a Code Reviews tab to the right of Repository.

Now click the Repository tab, and click on a revision number (do not click the "View Differences" button).
For example in my project, under the Latest Revisions heading, my latest revision is 12406. I click that link in the first column to get a list of the files changed in the revision.

Here, find a file you want to review. I find it most useful to do code reviews on diffs rather than entire files, so what I typically do is click the (diff) link next to the file. However, even if you click the file link (and not the diff link) you should still see the pencil icons.

Hope this helps!
Chris

Ahh its working now .. i missed the right to set for code review in role setup. You can see review tab in project site, but cant see the pencil as admin. Btw. really cool Plugin, and works like a charm.
Good Job!

I have a (relatively) clean install of Redmine 1.0.3 (one other plugin) and was able to install this plugin without issues (svn checkout, rake db:migrate_plugins, restart app servers). I used the r562 from the svn repository (on google code).

Works pretty slick. Nice job, guys!

Tom Kersten wrote:

I have a (relatively) clean install of Redmine 1.0.3 (one other plugin) and was able to install this plugin without issues (svn checkout, rake db:migrate_plugins, restart app servers). I used the r562 from the svn repository (on google code).

Works pretty slick. Nice job, guys!

• Reviewer doesn't see the entirety of the code in context. Can the plugin be changed to allow assignment of reviews not only based on a diff?
• Each and every comment becomes an issue. That's a lot of issue traffic. Can comments be grouped?
• If three different people ask me to review their code, there's no easy way to group a related bunch of requests other than search & sort.

Are any of these possible?
Thank you!