From 303504df4788fead8ab200dd5490658489ba5385 Mon Sep 17 00:00:00 2001 From: Stan Hu Date: Sat, 29 Apr 2017 12:29:59 +0000 Subject: Revert "Merge branch 'tc-no-todo-service-select' into 'master'" This reverts merge request !10845 --- lib/api/todos.rb | 4 ++-- lib/api/v3/todos.rb | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'lib/api') diff --git a/lib/api/todos.rb b/lib/api/todos.rb index 10aa64947fe..d1f7e364029 100644 --- a/lib/api/todos.rb +++ b/lib/api/todos.rb @@ -59,10 +59,10 @@ module API requires :id, type: Integer, desc: 'The ID of the todo being marked as done' end post ':id/mark_as_done' do - TodoService.new.mark_todos_as_done_by_ids(params[:id], current_user) todo = current_user.todos.find(params[:id]) + TodoService.new.mark_todos_as_done([todo], current_user) - present todo, with: ::API::Entities::Todo, current_user: current_user + present todo.reload, with: Entities::Todo, current_user: current_user end desc 'Mark all todos as done' diff --git a/lib/api/v3/todos.rb b/lib/api/v3/todos.rb index 3e2c61f6dbd..e3b311d61cd 100644 --- a/lib/api/v3/todos.rb +++ b/lib/api/v3/todos.rb @@ -11,10 +11,10 @@ module API requires :id, type: Integer, desc: 'The ID of the todo being marked as done' end delete ':id' do - TodoService.new.mark_todos_as_done_by_ids(params[:id], current_user) todo = current_user.todos.find(params[:id]) + TodoService.new.mark_todos_as_done([todo], current_user) - present todo, with: ::API::Entities::Todo, current_user: current_user + present todo.reload, with: ::API::Entities::Todo, current_user: current_user end desc 'Mark all todos as done' -- cgit v1.2.1