diff --git a/app/controllers/auto_completes_controller.rb b/app/controllers/auto_completes_controller.rb index 203dae4..1bbcd33 100644 --- a/app/controllers/auto_completes_controller.rb +++ b/app/controllers/auto_completes_controller.rb @@ -20,25 +20,23 @@ class AutoCompletesController < ApplicationController def issues @issues = [] - q = (params[:q] || params[:term]).to_s - query = (params[:scope] == "all" && Setting.cross_project_issue_relations?) ? Issue : @project.issues - if q.match(/^\d+$/) - @issues << query.visible.find_by_id(q.to_i) + q = (params[:q] || params[:term]).to_s.strip + if q.present? + scope = (params[:scope] == "all" ? Issue : @project.issues).visible + if q.match(/^\d+$/) + @issues << scope.find_by_id(q.to_i) + end + @issues += scope.where("LOWER(#{Issue.table_name}.subject) LIKE ?", "%#{q.downcase}%").order("#{Issue.table_name}.id DESC").limit(10).all + @issues.compact! end - unless q.blank? - @issues += query.visible.find(:all, :conditions => ["LOWER(#{Issue.table_name}.subject) LIKE ?", "%#{q.downcase}%"], :limit => 10) - end - @issues.compact! render :layout => false end private def find_project - project_id = (params[:issue] && params[:issue][:project_id]) || params[:project_id] - @project = Project.find(project_id) + @project = Project.find(params[:project_id]) rescue ActiveRecord::RecordNotFound render_404 end - end diff --git a/app/views/issue_relations/_form.html.erb b/app/views/issue_relations/_form.html.erb index efd58b3..a1db380 100644 --- a/app/views/issue_relations/_form.html.erb +++ b/app/views/issue_relations/_form.html.erb @@ -9,6 +9,6 @@ <%= toggle_link l(:button_cancel), 'new-relation-form'%>
-<%= javascript_tag "observeAutocompleteField('relation_issue_to_id', '#{escape_javascript auto_complete_issues_path(:id => @issue, :project_id => @project, :scope => 'all')}')" %> +<%= javascript_tag "observeAutocompleteField('relation_issue_to_id', '#{escape_javascript auto_complete_issues_path(:project_id => @project, :scope => (Setting.cross_project_issue_relations? ? 'all' : nil))}')" %> <%= javascript_tag "setPredecessorFieldsVisibility();" %> diff --git a/app/views/issues/_attributes.html.erb b/app/views/issues/_attributes.html.erb index a80fac9..8bf7dc0 100644 --- a/app/views/issues/_attributes.html.erb +++ b/app/views/issues/_attributes.html.erb @@ -43,7 +43,7 @@<%= f.text_field :parent_issue_id, :size => 10, :required => @issue.required_attribute?('parent_issue_id') %>
-<%= javascript_tag "observeAutocompleteField('issue_parent_issue_id', '#{escape_javascript auto_complete_issues_path(:id => @issue, :project_id => @issue.project)}')" %> +<%= javascript_tag "observeAutocompleteField('issue_parent_issue_id', '#{escape_javascript auto_complete_issues_path(:project_id => @issue.project)}')" %> <% end %> <% if @issue.safe_attribute? 'start_date' %> diff --git a/test/functional/auto_completes_controller_test.rb b/test/functional/auto_completes_controller_test.rb index 19d90fa..554d520 100644 --- a/test/functional/auto_completes_controller_test.rb +++ b/test/functional/auto_completes_controller_test.rb @@ -19,6 +19,13 @@ class AutoCompletesControllerTest < ActionController::TestCase assert assigns(:issues).detect {|issue| issue.subject.match /recipe/} end + def test_issues_should_accept_term_param + get :issues, :project_id => 'ecookbook', :term => 'ReCiPe' + assert_response :success + assert_not_nil assigns(:issues) + assert assigns(:issues).detect {|issue| issue.subject.match /recipe/} + end + def test_issues_should_return_issue_with_given_id get :issues, :project_id => 'subproject1', :q => '13' assert_response :success @@ -26,18 +33,28 @@ class AutoCompletesControllerTest < ActionController::TestCase assert assigns(:issues).include?(Issue.find(13)) end - def test_auto_complete_with_scope_all_and_cross_project_relations - Setting.cross_project_issue_relations = '1' + def test_auto_complete_with_scope_all_should_search_other_projects get :issues, :project_id => 'ecookbook', :q => '13', :scope => 'all' assert_response :success assert_not_nil assigns(:issues) assert assigns(:issues).include?(Issue.find(13)) end - def test_auto_complete_with_scope_all_without_cross_project_relations - Setting.cross_project_issue_relations = '0' - get :issues, :project_id => 'ecookbook', :q => '13', :scope => 'all' + def test_auto_complete_without_scope_all_should_not_search_other_projects + get :issues, :project_id => 'ecookbook', :q => '13' assert_response :success assert_equal [], assigns(:issues) end + + def test_issues_should_return_json + get :issues, :project_id => 'subproject1', :q => '13' + assert_response :success + json = ActiveSupport::JSON.decode(response.body) + assert_kind_of Array, json + issue = json.first + assert_kind_of Hash, issue + assert_equal 13, issue['id'] + assert_equal 13, issue['value'] + assert_equal 'Bug #13: Subproject issue two', issue['label'] + end end