jQuery Branch remove_link Issue

[pimpmaster]

Everything works great with this branch, except when I click the
remove link, it deletes the expected dom element, and when I save the
change is not reflected in DB.. the actualy child model is still there
when I reload.

Any ideas?

[James Golick]

sounds like an issue with either removing all the elements from the form, or :discard_if...

could it be either of those?

[pimpmaster]

Hmm, i just switched back to good ol' prototype and this is still
buggin, so it rules out the Jquery issue.

Can you see where I went wrong?

class Person < ActiveRecord::Base
has_many :phones, :as => :phoneable, :attributes =>
true, :discard_if => :blank?, :dependent => :destroy
end

class Phone < ActiveRecord::Base
belongs_to :phoneable, :polymorphic => true
def blank?; number.blank?; end
end

[pimpmaster]

> sounds like an issue with either removing all the elements from the form

Bingo!

If I have two child objects, it will delete the second one no problem.
But if its just one, att_fu wont delete it.

Is this expected behavior?

[James Golick]

ya, you need this line in your controller

params[:parent][:child_attributes] ||= {}

[pimpmaster]

Weird.. google groups wont let me post.. posting again:

I tried this:

[new_action,create,edit,update].each do |action|
action.before :destroy_lone_submodels
end

def destroy_lone_submodels
params[:person][:phone_attributes] ||= {}
end

and got this:

You have a nil object when you didn't expect it!
You might have expected an instance of ActiveRecord::Base.
The error occurred while evaluating nil.[]

[pimpmaster]

[James Golick]

params[:person][:phone_attributes] ||= {} unless params[:person].nil?

there won't always be person params

[pimpmaster]

That did the trick!

You are pretty good with this code stuff, you should like do it for a
living or something

:p