Skip to content
This repository was archived by the owner on Dec 12, 2021. It is now read-only.

Added posibility specify helper for link_to_add and link_to_remove #240

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added posibility specify helper for link_to_add and link_to_remove #240

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 8 additions & 4 deletions lib/nested_form/builder_mixin.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,11 +28,13 @@ def link_to_add(*args, &block)
reflection = object.class.reflect_on_association(association)
reflection.klass.new
end

helper = options.delete(:helper) || :link_to

options[:class] = [options[:class], "add_nested_fields"].compact.join(" ")
options["data-association"] = association
options["data-blueprint-id"] = fields_blueprint_id = fields_blueprint_id_for(association)
args << (options.delete(:href) || "javascript:void(0)")
args << (options.delete(:href) || "javascript:void(0)") if helper == :link_to
args << options

@fields ||= {}
Expand All @@ -43,7 +45,7 @@ def link_to_add(*args, &block)
blueprint[:"data-blueprint"] = fields_for(association, model_object, options, &block).to_str
@template.content_tag(:div, nil, blueprint)
end
@template.link_to(*args, &block)
@template.send helper, *args, &block
end

# Adds a link to remove the associated record. The first argment is the name of the link.
Expand All @@ -65,10 +67,12 @@ def link_to_remove(*args, &block)
md = object_name.to_s.match /(\w+)_attributes\]\[[\w\d]+\]$/
association = md && md[1]
options["data-association"] = association

helper = options.delete(:helper) || :link_to

args << (options.delete(:href) || "javascript:void(0)")
args << (options.delete(:href) || "javascript:void(0)") if helper == :link_to
args << options
hidden_field(:_destroy) << @template.link_to(*args, &block)
hidden_field(:_destroy) << @template.send(helper, *args, &block)
end

def fields_for_with_nested_attributes(association_name, *args)
Expand Down
4 changes: 2 additions & 2 deletions vendor/assets/javascripts/jquery_nested_form.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,8 @@

window.nestedFormEvents = new NestedFormEvents();
$(document)
.delegate('form a.add_nested_fields', 'click', nestedFormEvents.addFields)
.delegate('form a.remove_nested_fields', 'click', nestedFormEvents.removeFields);
.delegate('form .add_nested_fields', 'click', nestedFormEvents.addFields)
.delegate('form .remove_nested_fields', 'click', nestedFormEvents.removeFields);
})(jQuery);

// http://plugins.jquery.com/project/closestChild
Expand Down
4 changes: 2 additions & 2 deletions vendor/assets/javascripts/prototype_nested_form.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
document.observe('click', function(e, el) {
if (el = e.findElement('form a.add_nested_fields')) {
if (el = e.findElement('form .add_nested_fields')) {
// Setup
var assoc = el.readAttribute('data-association'); // Name of child
var blueprint = $(el.readAttribute('data-blueprint-id'));
Expand Down Expand Up @@ -43,7 +43,7 @@ document.observe('click', function(e, el) {
});

document.observe('click', function(e, el) {
if (el = e.findElement('form a.remove_nested_fields')) {
if (el = e.findElement('form .remove_nested_fields')) {
var hidden_field = el.previous(0),
assoc = el.readAttribute('data-association'); // Name of child to be removed
if(hidden_field) {
Expand Down