Display new issue link in version page (#28138).

Patch by Marius BALTEANU.

git-svn-id: http://svn.redmine.org/redmine/trunk@18755 e93f8b46-1217-0410-a6f0-8f06a7374b81

2 files changed, 64 insertions, 0 deletions

diff --git a/test/functional/versions_controller_test.rb b/test/functional/versions_controller_test.rb
index 8981c6932..44c683d7a 100644
--- a/test/functional/versions_controller_test.rb
+++ b/test/functional/versions_controller_test.rb
@@ -208,6 +208,14 @@ class VersionsControllerTest < Redmine::ControllerTest
     end
   end
 
+  def test_show_should_display_link_to_new_issue
+    @request.session[:user_id] = 1
+    get :show, :params => {:id => 3}
+
+    assert_response :success
+    assert_select 'a.icon.icon-add', :text => 'New issue'
+  end
+
   def test_new
     @request.session[:user_id] = 2
     get :new, :params => {:project_id => '1'}
diff --git a/test/helpers/version_helper_test.rb b/test/helpers/version_helper_test.rb
index e1c83e225..9bdb49749 100644
--- a/test/helpers/version_helper_test.rb
+++ b/test/helpers/version_helper_test.rb
@@ -53,4 +53,60 @@ class VersionsHelperTest < Redmine::HelperTest
     version.project = Project.find(5)
     assert_match /^\/issues\?/, version_filtered_issues_path(version)
   end
+
+  def test_link_to_new_issue_should_return_link_to_add_issue
+    version = Version.find(3)
+    project = Project.find(1)
+    User.current = User.find(1)
+
+    # href should contain the following params:
+    # fixed_version_id=3
+    # tracker_id=1
+    assert_select_in link_to_new_issue(version, project), '[href=?]', '/projects/ecookbook/issues/new?issue%5Bfixed_version_id%5D=3&issue%5Btracker_id%5D=1', :text => 'New issue'
+  end
+
+  def test_link_to_new_issue_should_return_nil_if_version_status_is_not_open
+    # locked version
+    version = Version.find(2)
+    project = Project.find(1)
+    User.current = User.find(1)
+
+    assert_nil link_to_new_issue(version, project)
+  end
+
+  def test_link_to_new_issue_should_return_nil_if_user_does_not_have_permission_to_add_issue
+    Role.find(1).remove_permission! :add_issues
+    version = Version.find(3)
+    project = Project.find(1)
+    User.current = User.find(2)
+
+    assert_nil link_to_new_issue(version, project)
+  end
+
+  def test_link_to_new_issue_should_return_nil_if_no_tracker_is_available_for_project
+    trackers = Tracker::CORE_FIELDS - %w(fixed_version_id)
+    # disable fixed_version_id field for all trackers
+    Tracker.all.each do |tracker|
+      tracker.core_fields = trackers
+      tracker.save!
+    end
+
+    version = Version.find(3)
+    project = Project.find(1)
+    User.current = User.find(2)
+
+    assert_nil link_to_new_issue(version, project)
+  end
+
+  def test_link_to_new_issue_should_take_into_account_user_permissions_on_fixed_version_id_field
+    WorkflowPermission.delete_all
+    WorkflowPermission.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :field_name => 'fixed_version_id', :rule => 'readonly')
+
+    version = Version.find(3)
+    project = Project.find(1)
+    User.current = User.find(2)
+
+    # href should contain param tracker_id=2 because for tracker_id 1, user has only readonly permissions on fixed_version_id
+    assert_select_in link_to_new_issue(version, project), '[href=?]', '/projects/ecookbook/issues/new?issue%5Bfixed_version_id%5D=3&issue%5Btracker_id%5D=2'
+  end
 end