ChiliProject is not maintained anymore. Please be advised that there will be no more updates.
We do not recommend that you setup new ChiliProject instances and we urge all existing users to migrate their data to a maintained system, e.g. Redmine. We will provide a migration script later. In the meantime, you can use the instructions by Christian Daehn.
Remove dead code in IssueHelper (Feature #539)
With the aaj-merge, the rather large
IssueHelper#show_detail was replaced by
JournalFormatter#render_detail. However, the original method was kept in place, possibly to keep backwards compatibility. But I guess, it is only there because somebody forgot about it. As far as I can tell, it also uses APIs that where removed with the merge so it is probably not working at the moment.
I propose to remove the method.
Attached is a pull request, which does that.
It also includes a TODO to refactor the test code. I'm currently not aware, where aaj-test-code is kept and therefore was not brave enough to move the code to the right places.
The pull request is at https://github.com/chiliproject/chiliproject/pull/85.
- Assignee deleted (
- Status changed from Open to Ready for review
This change breaks compatibility with redmine_code_review which uses show_detail in their app/views/code_review/_reply.html.erb. I can't replace it with render_detail, @journal.render_detail, or @reply.render_detail. I'm creating a related issue in redmine_code_review to have them update, but in case they need help I'm letting you know ahead of time.
Simon Stearn wrote:
Tactical hack: replace show_detail with puts until either a compatibility layer in chiliproject or a change to the plugin fixes this..
Such fix of the plugin is available at https://bitbucket.org/schuhschuh/redmine_code_review.