Commit 5605950659bfb5fbd1705ef00cb055d2ee94c783
1 parent
6ae0945a
Exists in
master
and in
1 other branch
Fix bug in nested attributes form
Showing
1 changed file
with
2 additions
and
4 deletions
Show diff stats
public/javascripts/form.js
| @@ -26,7 +26,7 @@ function makeNestedItemsDestroyable(wrapper) { | @@ -26,7 +26,7 @@ function makeNestedItemsDestroyable(wrapper) { | ||
| 26 | 26 | ||
| 27 | function appendNestedItem() { | 27 | function appendNestedItem() { |
| 28 | var addLink = $(this); | 28 | var addLink = $(this); |
| 29 | - var nestedItem = addLink.parent().find('.nested').first().clone(); | 29 | + var nestedItem = addLink.parent().find('.nested').first().clone().show(); |
| 30 | nestedItem.find('input, select').each(function(){ | 30 | nestedItem.find('input, select').each(function(){ |
| 31 | var input = $(this); | 31 | var input = $(this); |
| 32 | var timestamp = new Date(); | 32 | var timestamp = new Date(); |
| @@ -47,8 +47,6 @@ function removeNestedItem() { | @@ -47,8 +47,6 @@ function removeNestedItem() { | ||
| 47 | var destroyFlagName = inputNameExample.replace(/\[[^\]]*\]$/,'[_destroy]') | 47 | var destroyFlagName = inputNameExample.replace(/\[[^\]]*\]$/,'[_destroy]') |
| 48 | var destroyFlag = $('<input/>').attr('name',destroyFlagName).attr('type','hidden').val('true'); | 48 | var destroyFlag = $('<input/>').attr('name',destroyFlagName).attr('type','hidden').val('true'); |
| 49 | $("input[name='"+idFieldName+"']").after(destroyFlag); | 49 | $("input[name='"+idFieldName+"']").after(destroyFlag); |
| 50 | - nestedItem.hide(); | ||
| 51 | - } else { | ||
| 52 | - nestedItem.remove(); | ||
| 53 | } | 50 | } |
| 51 | + nestedItem.hide(); | ||
| 54 | } | 52 | } |
| 55 | \ No newline at end of file | 53 | \ No newline at end of file |