diff --git a/app/models/todo.rb b/app/models/todo.rb index 974616b3..986a4f12 100644 --- a/app/models/todo.rb +++ b/app/models/todo.rb @@ -287,7 +287,7 @@ class Todo < ActiveRecord::Base def add_predecessor(t) @predecessor_array = predecessors - @predecessor_array << t.specification + @predecessor_array << t end # Return todos that should be activated if the current todo is completed diff --git a/public/javascripts/application.js b/public/javascripts/application.js index e74da3b7..ee38999d 100644 --- a/public/javascripts/application.js +++ b/public/javascripts/application.js @@ -335,7 +335,7 @@ var TodoItems = { $('.item-container a.icon_delete_item').live('click', function(evt){ var confirm_message = $(this).attr("x_confirm_message") if(confirm(confirm_message)){ - delete_with_ajax_and_block_element(this.href, $(this).parents('.project')); + delete_with_ajax_and_block_element(this.href, $(this).parents('.item-container')); } return false; }); @@ -1074,15 +1074,10 @@ function enable_rich_interaction(){ var dropped_todo = this.id.split('_')[2]; ui.draggable.remove(); $('.drop_target').hide(); // IE8 doesn't call stop() in this situation - $(this).block({ - message: null - }); - $.post(relative_to_root('todos/add_predecessor'), - { - successor: dragged_todo, - predecessor: dropped_todo - }, - null, 'script'); + + ajax_options = default_ajax_options_for_scripts('POST', relative_to_root('todos/add_predecessor'), $(this)); + ajax_options.data += "&predecessor="+dropped_todo + "&successor="+dragged_todo + $.ajax(ajax_options); } function drag_todo(){