From 04e3aebabedf2d279af0887ae7903cb7664a939e Mon Sep 17 00:00:00 2001
From: Matt Rogers <codemattr@gmail.com>
Date: Wed, 18 Feb 2015 22:07:42 -0600
Subject: [PATCH] Use rails helpers for the promote to project menu item

---
 app/helpers/todos_helper.rb       | 2 +-
 test/helpers/todo_helpers_test.rb | 5 +++++
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/app/helpers/todos_helper.rb b/app/helpers/todos_helper.rb
index 2035f3fb..1c99fc9b 100644
--- a/app/helpers/todos_helper.rb
+++ b/app/helpers/todos_helper.rb
@@ -201,7 +201,7 @@ module TodosHelper
       :_source_view => (@source_view.underscore.gsub(/\s+/,'_') rescue "")}
     url[:_tag_name] = @tag_name if @source_view == 'tag'
 
-    return link_to(t('todos.convert_to_project'), url, {:class => "icon_item_to_project", :id => "to_project_#{dom_id(todo)}"})
+    link_to(t('todos.convert_to_project'), url, {:class => "icon_item_to_project", :id => dom_id(todo, "to_project")})
   end
 
   def collapsed_notes_image(todo)
diff --git a/test/helpers/todo_helpers_test.rb b/test/helpers/todo_helpers_test.rb
index a800aca4..0e952a9d 100644
--- a/test/helpers/todo_helpers_test.rb
+++ b/test/helpers/todo_helpers_test.rb
@@ -19,4 +19,9 @@ class TodosHelpersTest < ActionView::TestCase
     html = remote_delete_dependency(todo, predecessor)
     assert_equal "<a class=\"delete_dependency_button\" href=\"/todos/18/remove_predecessor\" x_predecessors_id=\"1\"><img align=\"absmiddle\" alt=\"Blank\" class=\"delete_item\" src=\"/images/blank.png\" title=\"Remove dependency (does not delete the action)\" /></a>", html
   end
+
+  test "remote_promote_to_project_menu_item" do
+    html = remote_promote_to_project_menu_item(todos(:call_bill))
+    assert_equal "<a class=\"icon_item_to_project\" href=\"/todos/1/convert_to_project?_source_view=\" id=\"to_project_todo_1\">Make project</a>", html
+  end
 end