do not activate already completed todos, fix #1778

This commit is contained in:
Carsten Otto 2015-03-22 00:50:01 +01:00 committed by Reinier Balt
parent 32b48df78f
commit 6017637abc
2 changed files with 19 additions and 1 deletions

View file

@ -308,7 +308,7 @@ class Todo < ActiveRecord::Base
# activate todos that should be activated if the current todo is completed
def activate_pending_todos
pending_todos = successors.select {|t| t.uncompleted_predecessors.empty?}
pending_todos = successors.select { |t| t.uncompleted_predecessors.empty? and !t.completed? }
pending_todos.each {|t| t.activate! }
return pending_todos
end