From 299c3de64e43eaf4451857a6623cc446130d9005 Mon Sep 17 00:00:00 2001 From: Jean-Philippe Lang Date: Fri, 3 Mar 2017 17:21:36 +0000 Subject: [PATCH] The cancel operation in the issue edit mode doesn't work (#21579). Patch by Marius BALTEANU. git-svn-id: http://svn.redmine.org/redmine/trunk@16361 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- app/views/issues/_edit.html.erb | 2 +- test/functional/issues_controller_test.rb | 16 ++++++++++++++++ 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/app/views/issues/_edit.html.erb b/app/views/issues/_edit.html.erb index fff258db5..fe2119a07 100644 --- a/app/views/issues/_edit.html.erb +++ b/app/views/issues/_edit.html.erb @@ -69,7 +69,7 @@ <%= hidden_field_tag 'last_journal_id', params[:last_journal_id] || @issue.last_journal_id %> <%= submit_tag l(:button_submit) %> <%= preview_link preview_edit_issue_path(:project_id => @project, :id => @issue), 'issue-form' %> - | <%= link_to l(:button_cancel), {}, :onclick => "$('#update').hide(); return false;" %> + | <%= link_to l(:button_cancel), issue_path(id: @issue.id), :onclick => params[:action] == 'show' ? "$('#update').hide(); return false;" : '' %> <%= hidden_field_tag 'prev_issue_id', @prev_issue_id if @prev_issue_id %> <%= hidden_field_tag 'next_issue_id', @next_issue_id if @next_issue_id %> diff --git a/test/functional/issues_controller_test.rb b/test/functional/issues_controller_test.rb index 455bf834a..b47ff53fb 100644 --- a/test/functional/issues_controller_test.rb +++ b/test/functional/issues_controller_test.rb @@ -4852,4 +4852,20 @@ class IssuesControllerTest < Redmine::ControllerTest User.add_to_project(user, Project.find(2), Role.find_by_name('Manager')) user end + + def test_cancel_edit_link_for_issue_show_action_should_have_onclick_action + @request.session[:user_id] = 1 + + get :show, :id => 1 + assert_response :success + assert_select 'a[href=?][onclick=?]', "/issues/1", "$('#update').hide(); return false;", :text => 'Cancel' + end + + def test_cancel_edit_link_for_issue_edit_action_should_not_have_onclick_action + @request.session[:user_id] = 1 + + get :edit, :id => 1 + assert_response :success + assert_select 'a[href=?][onclick=?]', "/issues/1", "", :text => 'Cancel' + end end