Accept filters from API calls without set_filter parameter.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3335 e93f8b46-1217-0410-a6f0-8f06a7374b81

Accept filters from API calls without set_filter parameter.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3335 e93f8b46-1217-0410-a6f0-8f06a7374b81
1bc00ac0 · Jean-Philippe Lang · 320c191f · 1bc00ac0 · 1bc00ac0
Commit 1bc00ac0 authored Jan 18, 2010 by Jean-Philippe Lang
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 1 deletion

issues_controller.rb app/controllers/issues_controller.rb +1 -1

issues_api_test.rb test/integration/issues_api_test.rb +9 -0

No files found.
--- a/app/controllers/issues_controller.rb
+++ b/app/controllers/issues_controller.rb
@@ -538,7 +538,7 @@ private
      session[:query] = {:id => @query.id, :project_id => @query.project_id}
      sort_clear
    else
-      if params[:set_filter] || session[:query].nil? || session[:query][:project_id] != (@project ? @project.id : nil)
+      if api_request? || params[:set_filter] || session[:query].nil? || session[:query][:project_id] != (@project ? @project.id : nil)
        # Give it a name, required to be valid
        @query = Query.new(:name => "_")
        @query.project = @project

--- a/test/integration/issues_api_test.rb
+++ b/test/integration/issues_api_test.rb
@@ -58,6 +58,15 @@ class IssuesApiTest < ActionController::IntegrationTest
    assert_response :success
    assert_equal 'application/xml', @response.content_type
  end
+  
+  def test_index_with_filter
+    get '/issues.xml?status_id=5'
+    assert_response :success
+    assert_equal 'application/xml', @response.content_type
+    assert_tag :tag => 'issues',
+               :children => { :count => Issue.visible.count(:conditions => {:status_id => 5}), 
+                              :only => { :tag => 'issue' } }
+  end
    
  def test_show_routing
    assert_routing(