diff options
author | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2016-03-16 20:56:23 -0300 |
---|---|---|
committer | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2016-03-18 11:00:53 -0300 |
commit | fb72271e24663c74e6dd66e610fd1add98628648 (patch) | |
tree | 35f8efea08d81fde6bdf5f5e48c86ecbbe2e5ce4 /app | |
parent | c29da3f8ca1d759b8cbecb91b6e0529b18cc5c85 (diff) | |
download | gitlab-ce-fb72271e24663c74e6dd66e610fd1add98628648.tar.gz |
Use todo.done without ! in the controller to mark todo as done
Diffstat (limited to 'app')
-rw-r--r-- | app/controllers/dashboard/todos_controller.rb | 4 | ||||
-rw-r--r-- | app/models/todo.rb | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/app/controllers/dashboard/todos_controller.rb b/app/controllers/dashboard/todos_controller.rb index 7857af9c5de..be488483b09 100644 --- a/app/controllers/dashboard/todos_controller.rb +++ b/app/controllers/dashboard/todos_controller.rb @@ -6,7 +6,7 @@ class Dashboard::TodosController < Dashboard::ApplicationController end def destroy - todo.done! + todo.done todo_notice = 'Todo was successfully marked as done.' @@ -20,7 +20,7 @@ class Dashboard::TodosController < Dashboard::ApplicationController end def destroy_all - @todos.each(&:done!) + @todos.each(&:done) respond_to do |format| format.html { redirect_to dashboard_todos_path, notice: 'All todos were marked as done.' } diff --git a/app/models/todo.rb b/app/models/todo.rb index b00a1b3dc7d..68263a64d5a 100644 --- a/app/models/todo.rb +++ b/app/models/todo.rb @@ -38,7 +38,7 @@ class Todo < ActiveRecord::Base state_machine :state, initial: :pending do event :done do - transition [:pending, :done] => :done + transition [:pending] => :done end state :pending |