add parameter to REST api for todos to limit result to active todos. Fixes #1388

app/controllers/todos_controller.rb

@@ -47,7 +47,10 @@ class TodosController < ApplicationController
         headers['Content-Type']=Mime::TEXT.to_s
         render :content_type => Mime::TEXT
       end
-      format.xml   { render :xml => @todos.to_xml( *todo_xml_params ) }
+      format.xml do
+        @xml_todos = params[:limit_to_active_todos] ? @not_done_todos : @todos
+        render :xml => @xml_todos.to_xml( *todo_xml_params ) 
+      end
       format.any(:rss, :atom) { @feed_title, @feed_description = 'Tracks Actions', "Actions for #{current_user.display_name}" }
       format.ics
     end

app/views/integrations/rest_api.html.erb

@@ -79,6 +79,15 @@ field to <code>ID, created_at, modified_at, completed_at</code> by adding the pa
 </code>
 </pre>
 
+If you only want to get the active todos, you add the parameter <code>limit_to_active_todos</code> and set it to some value like this:
+
+<pre>
+<code>
+  $ curl -u username:p4ssw0rd -H "Content-Type: text/xml" \
+  <%= root_url %>todos.xml?limit_to_active_todos=1
+</code>
+</pre>
+
 <h2>Writing to the API</h2>
 
 <p>The API provides mechanisms for adding, updating and deleting resources using the HTTP methods <code>PUT</code>, <code>POST</code> and <code>DELETE</code> in combination with the content.</p>

test/integration/todo_xml_api_test.rb

@@ -17,7 +17,7 @@ class TodoXmlApiTest < ActionDispatch::IntegrationTest
     get '/tickler.xml', {}, {}
     assert_response 401
 
-    get "/tickler.xml", {}, {'HTT_AUTHORIZATION' => "Basic " + Base64.encode64("wrong:wrong"),'ACCEPT' => 'application/xml'}
+    get "/tickler.xml", {}, {'HTTP_AUTHORIZATION' => "Basic " + Base64.encode64("wrong:wrong"),'ACCEPT' => 'application/xml'}
     assert_response 401
   end
 
@@ -32,6 +32,19 @@ class TodoXmlApiTest < ActionDispatch::IntegrationTest
     assert_no_tag :tag => "user_id"
   end
 
+  def test_get_index_with_only_active_todos
+    authenticated_get_xml "/todos.xml", @user.login, @password, {}
+    assert_response 200
+
+    all_todo_count = assigns['xml_todos']
+
+    authenticated_get_xml "/todos.xml?limit_to_active_todos=1", @user.login, @password, {}
+    assert_response 200
+
+    active_todo_count = assigns['xml_todos']
+    assert all_todo_count != active_todo_count, "active should be less than all todos"
+  end
+
   def test_create_todo_with_show_from
     old_count = @user.todos.count
     authenticated_post_xml_to_todo_create "