diff --git a/app/assets/javascripts/protocols/steps.js.erb b/app/assets/javascripts/protocols/steps.js.erb index f32927663..6b2b62f58 100644 --- a/app/assets/javascripts/protocols/steps.js.erb +++ b/app/assets/javascripts/protocols/steps.js.erb @@ -188,11 +188,9 @@ function formCallback($form) { $form.submit(function(){ $(this).find("[data-role='editable-table']").each(function() { var hot = $(this).find(".hot").handsontable('getInstance'); - if (hot && hot.countEmptyRows() != hot.countRows()) { - var contents = $(this).find('.hot-contents'); - var data = JSON.stringify({data: hot.getData()}); - contents.attr("value", data); - } + var contents = $(this).find('.hot-contents'); + var data = JSON.stringify({data: hot.getData()}); + contents.attr("value", data); }); setTimeout(function() { @@ -531,7 +529,6 @@ $("[data-action='new-step']").on("ajax:success", function(e, data) { function processStep(ev, editMode) { var $form = $(ev.target.form); $form.clearFormErrors(); - $form.removeBlankExcelTables(editMode); $form.removeBlankFileForms(); var $fileInputs = $form.find("input[type=file]"); diff --git a/app/assets/javascripts/sitewide/form_cleanups.js b/app/assets/javascripts/sitewide/form_cleanups.js index 33e6e25c3..7f2a42d92 100644 --- a/app/assets/javascripts/sitewide/form_cleanups.js +++ b/app/assets/javascripts/sitewide/form_cleanups.js @@ -21,18 +21,3 @@ $.fn.removeBlankFileForms = function () { } }); } - -/* - * Not the actual Excel tables, but are similar. - */ -$.fn.removeBlankExcelTables = function (editMode) { - if(editMode) { - $tables = $(this).find("[data-role='editable-table']"); - // In edit mode, tables can't be blank - $tables.each(function () { - if (!$(this).find("td:not(:empty)").length) { - $(this).closest("fieldset").remove(); - } - }); - } -}