Move to helper, no instance variables

app/models/todo.rb

@@ -34,13 +34,6 @@ class Todo < ActiveRecord::Base
     action == BUILD_FAILED
   end
-  def action_name
-    case action
-    when Todo::ASSIGNED then 'assigned you'
-    when Todo::MENTIONED then 'mentioned you on'
-    end
-  end
-
   def body
     if note.present?
       note.note

app/views/dashboard/todos/_todo.html.haml

@@ -11,7 +11,7 @@
           - else
             (removed)
       %span.todo-label
-        = todo.action_name
+        = todo_action_name(todo)
         - if todo.target
           = todo_target_link(todo)
         - else

lib/api/todos.rb

@@ -14,11 +14,11 @@ module API
       #
       # Example Request:
       #  GET /todos
+      #
       get do
-        @todos = find_todos
-        @todos = paginate @todos
-        present @todos, with: Entities::Todo
+        todos = find_todos
+        present paginate(todos), with: Entities::Todo
       end
       # Mark todo as done
@@ -31,10 +31,10 @@ module API
       #  DELETE /todos/:id
       #
       delete ':id' do
-        @todo = current_user.todos.find(params[:id])
-        @todo.done
-        present @todo, with: Entities::Todo
+        todo = current_user.todos.find(params[:id])
+        todo.done
+        present todo, with: Entities::Todo
       end
       # Mark all todos as done
@@ -44,10 +44,10 @@ module API
       #  DELETE /todos
       #
       delete do
-        @todos = find_todos
-        @todos.each(&:done)
-        present @todos, with: Entities::Todo
+        todos = find_todos
+        todos.each(&:done)
+        present paginate(todos), with: Entities::Todo
       end
     end
   end