プロジェクト

全般

プロフィール

Defect #708

完了

pencil icon does not appear in diff view

Haru Iida さんが約13年前に追加. ほぼ13年前に更新.

ステータス:
終了(Closed)
優先度:
通常(Normal)
担当者:
対象バージョン:
開始日:
2011/04/05
期日:
進捗率:

100%

予定工数:

説明

Hello,

I've just installed the code-review plugin on Redmine 1.1.2 running on Debian Wheezy Testing with Passenger/MySQL.
I'm using it with a git repository.
I can create code-review related bugs linked to the certain revision of the code.

But the most important part of the plugin does not work. When I'm viewing diffs, the small icons do not appear on the left side, so I cannot annotate the diffs with reviews.

Do yuo know the solution?

Best regards

SZABÓ Gergely
Budapest, Hungary


ファイル

diffs.png (15.7 KB) diffs.png Haru Iida, 2011/04/05 23:26
diff.png (10.2 KB) diff.png Haru Iida, 2011/04/05 23:26
no_pencil.png (8.62 KB) no_pencil.png vincent auroch, 2011/04/11 23:10

Haru Iida さんが約13年前に更新

But the most important part of the plugin does not work. When I'm viewing diffs, the small icons do not appear on the left side, so I cannot annotate the diffs with reviews.

If you click "View differences", pencil icon does not appears.

If you click "diff" at right side of file name, you will see pencil icons.

Thanks.

Gergely Szabó さんが約13年前に更新

No, I don't see pencil icons, even if I click (diff) at right side of file name.
Arigato
Gergely

vincent auroch さんが約13年前に更新

Hello

I can confirm the same behavior, running a freshiling installed redmine (v1.0.1)

  • Pencils does not appear when making diff
  • Selecting "Code review" tab leads to a white page : Internal error

An error occurred on the page you were trying to access.
If you continue to experience problems please contact your redMine administrator for assistance.

Let me know if you would like a special access to a test project to see what could be wrong.

Vincent

Haru Iida さんが約13年前に更新

vincent auroch は書きました:

  • Pencils does not appear when making diff
  • Selecting "Code review" tab leads to a white page : Internal error

I want to see a stack trace of internal error. You can find it in production.log.

And let me know what version of code review plugin you use.

Haru Iida さんが約13年前に更新

Hi Gergely Szabó

No, I don't see pencil icons, even if I click (diff) at right side of file name.
Arigato
Gergely

hmm. Can you try my new code?
https://bitbucket.org/haru_iida/redmine_code_review/get/tip.zip

vincent auroch さんが約13年前に更新

Hello,

  • About issue Selecting "Code review" tab leads to a white page : Internal error

This was dues to a wrong name given to the pluggin when extracting your package into /vendor/pluggin. Your redmine install tutorial shall be updated to make sure user will give the right plugin name (required : redmine_code_review)

  • Still investigating the second issue (missing pensil).

Vincent

vincent auroch さんが約13年前に更新

I just add a quick look inside the log file when making the "diff" for a revision. I'm not notice anything special.

Log file content :

Processing RepositoriesController#diff (for 88.182.42.119 at 2011-04-11 14:06:40) [GET]
Parameters: {"rev"=>"f0428cd5c84664b240d5b2c45426d061ad34fa0d", "action"=>"diff", "id"=>"safboot", "path"=>["what's new.txt"], "controller"=>"repositories"}
Rendering template within layouts/base
Rendering repositories/diff
Completed in 299ms (View: 231, DB: 5) | 200 OK [http://intranet.seasidetech.net/redmine/projects/safboot/repository/revisions/f0428cd5c84664b240d5b2c45426d061ad34fa0d/diff/what's new.txt]

Processing CodeReviewController#update_diff_view (for 88.182.42.119 at 2011-04-11 14:06:40) [GET]
Parameters: {"rev"=>"f0428cd5c84664b240d5b2c45426d061ad34fa0d", "rev_to"=>"", "action"=>"update_diff_view", "id"=>"safboot", "review_id"=>"", "path"=>"/what's new.txt", "action_type"=>"diff", "controller"=>"code_review"}
Completed in 38ms (View: 13, DB: 2) | 200 OK [http://intranet.seasidetech.net/redmine/code_review/update_diff_view/safboot?rev=f0428cd5c84664b240d5b2c45426d061ad34fa0d&path=%2Fwhat%27s%20new.txt&review_id=&action_type=diff&rev_to=]

What I can see : the diff window without pencil :

vincent auroch さんが約13年前に更新

One question : is there a way to re-start the redmine pluggin install to make sure everything is configured ok ?

vincent auroch さんが約13年前に更新

Juste tested with your new 0.4.1 : no change. Still missing pencils.

Vincent

Haru Iida さんが約13年前に更新

vincent auroch は書きました:

Juste tested with your new 0.4.1 : no change. Still missing pencils.

Vincent

Thanks.

I have reproduced it.
If your file path has some kind of special characters(ex. "'"), pencil icon does not appear.

I will try to fix it.

vincent auroch さんが約13年前に更新

I confirm : it works definitely fine with files without special chars.

Thanks for your support

Vincent

Gergely Szabó さんが約13年前に更新

Haru Iida は書きました:

Hi Gergely Szabó

No, I don't see pencil icons, even if I click (diff) at right side of file name.
Arigato
Gergely

hmm. Can you try my new code?
https://bitbucket.org/haru_iida/redmine_code_review/get/tip.zip

I've made a backup of the database, installed your new code, migrated the database. My Redmine is now dead. Error message:

Ruby on Rails application could not be started
A source file that the application requires, is missing.

    It is possible that you didn't upload your application files correctly. Please check whether all your application files are uploaded.
    A required library may not installed. Please install all libraries that this application requires.

Further information about the error may have been written to the application's log file. Please check it in order to analyse the problem.

Error message:
    no such file to load -- code_review_application_hooks (MissingSourceFile)
Exception class:
    PhusionPassenger::UnknownError
Application root:
    /usr/share/redmine 

Haru Iida さんが約13年前に更新

  • ステータス新規(New) から 解決(Resolved) に変更
  • 進捗率0 から 100 に変更

更新履歴 commit:c3363e70710e で適用されました。

Haru Iida さんがほぼ13年前に更新

  • ステータス解決(Resolved) から 終了(Closed) に変更

他の形式にエクスポート: Atom PDF