diff options
author | Rémy Coutable <remy@rymai.me> | 2017-03-30 11:29:40 +0000 |
---|---|---|
committer | James Lopez <james@jameslopez.es> | 2017-04-03 12:12:03 +0200 |
commit | 1d9e14a564ccb7b9cdb68ad483700a5c9c6d605a (patch) | |
tree | 7931ff75d77a446aaa4e299a87273afb03dfc4ff | |
parent | 54842848ec3bd520874c7d88ffc7d30e43e312d4 (diff) | |
download | gitlab-ce-1d9e14a564ccb7b9cdb68ad483700a5c9c6d605a.tar.gz |
Merge branch '26595-fix-issue-preselected-template' into 'master'
Fix linking to new issue with selected template
Closes #26595
See merge request !10155
-rw-r--r-- | app/helpers/issuables_helper.rb | 2 | ||||
-rw-r--r-- | changelogs/unreleased/26595-fix-issue-preselected-template.yml | 4 | ||||
-rw-r--r-- | spec/features/issues_spec.rb | 17 |
3 files changed, 22 insertions, 1 deletions
diff --git a/app/helpers/issuables_helper.rb b/app/helpers/issuables_helper.rb index fae10668a20..3704ec0d9e8 100644 --- a/app/helpers/issuables_helper.rb +++ b/app/helpers/issuables_helper.rb @@ -234,6 +234,6 @@ module IssuablesHelper end def selected_template(issuable) - params[:issuable_template] if issuable_templates(issuable).include?(params[:issuable_template]) + params[:issuable_template] if issuable_templates(issuable).any?{ |template| template[:name] == params[:issuable_template] } end end diff --git a/changelogs/unreleased/26595-fix-issue-preselected-template.yml b/changelogs/unreleased/26595-fix-issue-preselected-template.yml new file mode 100644 index 00000000000..a94765f8f2a --- /dev/null +++ b/changelogs/unreleased/26595-fix-issue-preselected-template.yml @@ -0,0 +1,4 @@ +--- +title: Fix linking to new issue with selected template via url parameter +merge_request: +author: diff --git a/spec/features/issues_spec.rb b/spec/features/issues_spec.rb index 1c8267b1593..afdc5e4ac0a 100644 --- a/spec/features/issues_spec.rb +++ b/spec/features/issues_spec.rb @@ -13,6 +13,13 @@ describe 'Issues', feature: true do user2 = create(:user) project.team << [[@user, user2], :developer] + + project.repository.create_file( + @user, + '.gitlab/issue_templates/bug.md', + 'this is a test "bug" template', + message: 'added issue template', + branch_name: 'master') end describe 'Edit issue' do @@ -582,6 +589,16 @@ describe 'Issues', feature: true do expect(page.find_field("issue_description").value).to match /\n\n$/ end end + + context 'form filled by URL parameters' do + before do + visit new_namespace_project_issue_path(project.namespace, project, issuable_template: 'bug') + end + + it 'fills in template' do + expect(find('.js-issuable-selector .dropdown-toggle-text')).to have_content('bug') + end + end end describe 'new issue by email' do |