summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobert Speicher <robert@gitlab.com>2016-06-27 17:49:29 +0000
committerRobert Speicher <robert@gitlab.com>2016-06-27 17:49:29 +0000
commit21842cf90f0d2545ed3d687ce864eb0d551b7059 (patch)
tree1574a518385c6f8bfeba429997843ad71adcac84
parent121c5c837fdf9b001fa80f9395cd327af17e5a9f (diff)
parentd17ab054cfa0d44fa36a20b249b2cab7ef99e9d4 (diff)
downloadgitlab-ce-21842cf90f0d2545ed3d687ce864eb0d551b7059.tar.gz
Merge branch 'mark-done-todo-id' into 'master'
Correctly return todo ID after creating todo See merge request !4941
-rw-r--r--app/services/todo_service.rb3
1 files changed, 2 insertions, 1 deletions
diff --git a/app/services/todo_service.rb b/app/services/todo_service.rb
index 540bf54b920..239bd17a035 100644
--- a/app/services/todo_service.rb
+++ b/app/services/todo_service.rb
@@ -159,8 +159,9 @@ class TodoService
def create_todos(users, attributes)
Array(users).map do |user|
next if pending_todos(user, attributes).exists?
- Todo.create(attributes.merge(user_id: user.id))
+ todo = Todo.create(attributes.merge(user_id: user.id))
user.update_todos_count_cache
+ todo
end
end