From 7894bf204fb9a3d3fa842a2aec42665dfb1fc186 Mon Sep 17 00:00:00 2001 From: Eric Davis Date: Tue, 11 Oct 2011 13:50:40 -0700 Subject: [PATCH] [#6574] Store deleted finances in a hidden HTML table --- assets/javascripts/contracts.js | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/assets/javascripts/contracts.js b/assets/javascripts/contracts.js index f2c0523..5bf7afd 100644 --- a/assets/javascripts/contracts.js +++ b/assets/javascripts/contracts.js @@ -69,7 +69,7 @@ jQuery(function($) { }, showDeliverableAddButton = function() { - var addLinks = $('table .add-labor a.add') + var addLinks = $('table.deliverable_finance_table .add-labor a.add') if (addLinks.length == 0) { // No link, add a blank form addNewDeliverableFinance("labor"); @@ -88,16 +88,21 @@ jQuery(function($) { showDeliverableAddButton(); }, + // Set the deleted flag for Rails and move it out of the row into + // a hidden table deleteDeliverableFinance = function(deleteLink) { if (confirm(i18nAreYouSure)) { - // Set the deleted flag for Rails and move it out of the row $(deleteLink).parent().find('.delete-flag').val('1') - $(deleteLink).closest("form"). - append( - $(deleteLink). // - parent(). // - parent().hide() - ) // + if ($('#deleted-finances').length == 0) { + $(deleteLink). + closest("form"). + append($("")); + } + $('#deleted-finances').append( + $(deleteLink). // + parent(). // + parent().hide() + ); // showDeliverableAddButton(); } },