thr3ads.net - Rails - assigning collection values and exceptions [Aug 2008]

If this information is useful, please help other people find it:
Share via:

Claudio Poli

2008-Aug-31 18:36 UTC

assigning collection values and exceptions

hi everyone,

I''ve this code in my Report model:

  has_many :report_reasons, :validate => true
  has_many :reasons, :through => :report_reasons #, :uniq => true
# :accessible => true

  def reason_attributes=(reason_attributes)
    reasons.clear
      reason_attributes.uniq.each do |reason|
        reasons << Reason.find_or_create_by_content(reason)
      end
  end

What I''m trying to accomplish (and what works) is to have a form which
can submit to a Report some Reason via fields_for (Reason :content
field).
So I just swipe existing associations with .clear and proceed to
rebuild new ones.
I like to keep things that way so I have my validation errors back,
this is true especially when saving a new Report.

What happens here is that, according to rails'' code, when the owner
object, in this case Report, doesn''t exists yet, I can assign to it
every kind of Reason I can think of, most importantly even those that
doesn''t pass validation.
Because validations happens at .save I''ve my errors and I''m
quite
happy.

However this is different when using update_attributes, for updating.
The owner object (Report) already exists, and every invalid Reason
assigned through << will simply result in a raised
ActiveRecord::InvalidRecord exception. This is best described directly
into rails'' code in association_collection.rb (<<) and
has_many_through_association.rb (insert_record).

Now I may trap this exception in controller and do something with it,
however I lose in some way my validation error messages on my form,
which is dynamic (add/remove reasons via js).

The reason why I''m not using :accessible or some other plugin to ease
multimodel forms is that they wants to create a record in Reason,
which is good, but they don''t try to reuse it if it already exists.
Instead they, of course, will duplicate rows, defeating my purpose.

I can post some other code if anybody have some suggestions on how to
proceed.

Thanks.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Claudio Poli

2008-Sep-01 12:08 UTC

head link

Re: assigning collection values and exceptions

FWIW and for posterity''s sake I workarounded with the not-yet-dry
code:

  def reason_attributes=(reason_attributes)
    reasons.clear
    reason_attributes.uniq.each do |attributes|
      reason = Reason.find_or_initialize_by_content(attributes)
      if self.new_record?
        # Entering create.
        reasons << reason # no matter if it''s invalid
      else
        # Entering update.
        if reason.valid?
          if Reason.exists?(reason)
            # Reason is valid and already exists, instead of
duplicating
            # a record just assing it.
            reasons << reason
          else
            # Reason is valid but doesn''t exists, build it.
            reasons.build(attributes)
          end
        else
          reasons.build(attributes) # workarounded to show error
messages
        end
      end
    end
  end

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Russell Norris

2008-Sep-01 12:11 UTC

head link

Re: assigning collection values and exceptions

i could be wrong but that seems functionally equivalent to
http://pastie.org/263949, right?

RSL

On Mon, Sep 1, 2008 at 8:08 AM, Claudio Poli
<masterkain-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
wrote:>
> FWIW and for posterity''s sake I workarounded with the not-yet-dry
> code:
>
>  def reason_attributes=(reason_attributes)
>    reasons.clear
>    reason_attributes.uniq.each do |attributes|
>      reason = Reason.find_or_initialize_by_content(attributes)
>      if self.new_record?
>        # Entering create.
>        reasons << reason # no matter if it''s invalid
>      else
>        # Entering update.
>        if reason.valid?
>          if Reason.exists?(reason)
>            # Reason is valid and already exists, instead of
> duplicating
>            # a record just assing it.
>            reasons << reason
>          else
>            # Reason is valid but doesn''t exists, build it.
>            reasons.build(attributes)
>          end
>        else
>          reasons.build(attributes) # workarounded to show error
> messages
>        end
>      end
>    end
>  end
>
> >
>
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Claudio Poli

2008-Sep-01 12:21 UTC

head link

Re: assigning collection values and exceptions

hi Russel,
no, they are different.
I must differentiate the fact that a owner object, self (which is a
Report), is a new record.

Because if it''s the case, then rails permits to push objects to a
collection even if they are not valid (via
validates_associated :reasons), and triggering validation only on
save, called from the create action of the controller.

if self already exists, and therefore self.new_record? returns false,
every assignment made through the use of push (<<) will bombs and
return instantly an ActiveRecord::InvalidRecord.

As I said this behaviour is best described directly into rails'' code
in association_collection.rb (<< method) and
has_many_through_association.rb (insert_record method).

So, to avoid the need to rescue those errors in controller, probably
losing my validation messages and due to the fact that this is an ajax
form with a dynamic number of fields on update I must check a couple
of things.

If the reason that I''m trying to pass passes validations
(with .valid?) then I must check if it already exists in database,
since I don''t want duplicates in Reason table, and if yes assign it to
the collection of self, which will not raise any exception since if
it''s in database it''s already valid, if not present just do a
build,
which will not trigger any exception but kist making update_attributes
in controller do its job.

if the reason is not valid, always on update, then I fake out a build
in any case, since the record will not pass, but at least I have now
my error messages right near the fields on a multimodel form, reusing
data from the collection.

On Sep 1, 2:11 pm, "Russell Norris"
<r...-ftMzyaTR+bHNyFkoKTPOtdi2O/JbrIOy@public.gmane.org>
wrote:> i could be wrong but that seems functionally equivalent
tohttp://pastie.org/263949, right?
>
> RSL
>
> On Mon, Sep 1, 2008 at 8:08 AM, Claudio Poli
<masterk...-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>
> > FWIW and for posterity''s sake I workarounded with the
not-yet-dry
> > code:
>
> >  def reason_attributes=(reason_attributes)
> >    reasons.clear
> >    reason_attributes.uniq.each do |attributes|
> >      reason = Reason.find_or_initialize_by_content(attributes)
> >      if self.new_record?
> >        # Entering create.
> >        reasons << reason # no matter if it''s invalid
> >      else
> >        # Entering update.
> >        if reason.valid?
> >          if Reason.exists?(reason)
> >            # Reason is valid and already exists, instead of
> > duplicating
> >            # a record just assing it.
> >            reasons << reason
> >          else
> >            # Reason is valid but doesn''t exists, build it.
> >            reasons.build(attributes)
> >          end
> >        else
> >          reasons.build(attributes) # workarounded to show error
> > messages
> >        end
> >      end
> >    end
> >  end--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Russell Norris

2008-Sep-01 12:28 UTC

head link

Re: assigning collection values and exceptions

you could still use your new_record? checks etc without using
reasons.build to recreate the same object you already have from the
find_or_initialize. that was my point.

RSL

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Russell Norris

2008-Sep-01 12:29 UTC

head link

Re: assigning collection values and exceptions

also, that in every single case you show here... you add the reason to
reasons. there''s no case [unless i''m missing it] that you do
something
besides add the new reason to the reasons collection.

RSL

On Mon, Sep 1, 2008 at 8:28 AM, Russell Norris
<rsl-ftMzyaTR+bHNyFkoKTPOtdi2O/JbrIOy@public.gmane.org>
wrote:> you could still use your new_record? checks etc without using
> reasons.build to recreate the same object you already have from the
> find_or_initialize. that was my point.
>
> RSL
>
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Claudio Poli

2008-Sep-01 12:42 UTC

head link

Re: assigning collection values and exceptions

you''re right, as I wrote this code wasn''t DRYed up yet, I
wrote it in
a hurry.

besides, thanks to your advice I''ve shortened this up to

   def reason_attributes=(reason_attributes)
     reasons.clear
     reason_attributes.uniq.each do |attributes|
       reason = Reason.find_or_initialize_by_content(attributes)
       if self.new_record?
         # Entering create.
         reasons << reason # no matter if it''s invalid
       else
         # Entering update.
         if reason.valid?
           reasons << reason
         else
           reasons.build(attributes) # workarounded to show error  
messages
         end
       end
     end
   end

do you have further optimisation in mind or I''m missing something  
obvious?

Il giorno 01/set/08, alle ore 14:29, Russell Norris ha scritto:
>
> also, that in every single case you show here... you add the reason to
> reasons. there''s no case [unless i''m missing it] that you
do something
> besides add the new reason to the reasons collection.
>
> RSL
>
> On Mon, Sep 1, 2008 at 8:28 AM, Russell Norris  
> <rsl-ftMzyaTR+bHNyFkoKTPOtdi2O/JbrIOy@public.gmane.org> wrote:
>> you could still use your new_record? checks etc without using
>> reasons.build to recreate the same object you already have from the
>> find_or_initialize. that was my point.
>>
>> RSL
>>
>
> >

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Claudio Poli

2008-Sep-01 12:52 UTC

head link

Re: assigning collection values and exceptions

another attempt

 def reason_attributes=(reason_attributes)
   reasons.clear
   reason_attributes.uniq.each do |attributes|
     reason = Reason.find_or_initialize_by_content(attributes)
     reasons << reason rescue reasons.build(attributes)
   end
 end
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Russell Norris

2008-Sep-01 13:06 UTC

head link

Re: assigning collection values and exceptions

>     reason = Reason.find_or_initialize_by_content(attributes)
>     reasons << reason rescue reasons.build(attributes)
doesn''t make sense to me. the first line already finds or creates a
reason. if it raises an exception adding it to the reasons collection,
i don''t see why building a new one with the exact same information
would solve anything. you''ve tried this?

RSL

On Mon, Sep 1, 2008 at 8:52 AM, Claudio Poli
<masterkain-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
wrote:>
> another attempt
>
>  def reason_attributes=(reason_attributes)
>   reasons.clear
>   reason_attributes.uniq.each do |attributes|
>     reason = Reason.find_or_initialize_by_content(attributes)
>     reasons << reason rescue reasons.build(attributes)
>   end
>  end
> >
>
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Claudio Poli

2008-Sep-01 13:22 UTC

head link

Re: assigning collection values and exceptions

I''ll add some view code.
It''s a kind of workaround to keep reasons collection with something  
filled in.
the new and edit view shares a partial which is from a collection:

# _report.erb
<div class="reason">
  <dl class="form">
   <% fields_for("report[reason_attributes][]", reason) do |r|
%>
    <dt class="required">Reason <%= r.error_message_on
:content %></dt>
     <dd>
      <%= r.text_field :content, { :index => nil,  :autocomplete =>  
''off'' } %>
     <%= link_to_function "remove",
"$(this).up(''.reason'').remove()" %>
    </dd>
   <% end %>
  </dl>
</div>

# edit.erb
<% form_for [:administration, @document, @report] do |f| -%>
...
<div id="reasons">
  <%= render :partial => ''reason'', :collection =>
@report.reasons %>
</div>
<p><%= add_reason_link "Add a reason" %></p>
<%= f.submit %>
<% end -%>

# controller update action uses only if  
@report.update_attributes(params[:report])

when you first access the edit page, _report is cycled through every  
reason it finds, and so it''s ok.
the moment I edit a reason field, putting in there an invalid  
attribute that doesn''t pass the validation I need to rescue the  
exception thrown by << somewhere.
I decided to rescue this exception with a reasons.build(attributes),  
which will instantiate a new reason on the invalid field, thus showing  
the error message.

Il giorno 01/set/08, alle ore 15:06, Russell Norris ha scritto:
>
>>    reason = Reason.find_or_initialize_by_content(attributes)
>>    reasons << reason rescue reasons.build(attributes)
>
> doesn''t make sense to me. the first line already finds or creates
a
> reason. if it raises an exception adding it to the reasons collection,
> i don''t see why building a new one with the exact same information
> would solve anything. you''ve tried this?
>
> RSL
>
> On Mon, Sep 1, 2008 at 8:52 AM, Claudio Poli
<masterkain-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> wrote:
>>
>> another attempt
>>
>> def reason_attributes=(reason_attributes)
>>  reasons.clear
>>  reason_attributes.uniq.each do |attributes|
>>    reason = Reason.find_or_initialize_by_content(attributes)
>>    reasons << reason rescue reasons.build(attributes)
>>  end
>> end
>>>
>>
>
> >

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Maybe Matching Threads

Search for more possibly parallel threads

Rails - Aug 2008 - assigning collection values and exceptions

assigning collection values and exceptions

Re: assigning collection values and exceptions

Re: assigning collection values and exceptions

Re: assigning collection values and exceptions

Re: assigning collection values and exceptions

Re: assigning collection values and exceptions

Re: assigning collection values and exceptions

Re: assigning collection values and exceptions

Re: assigning collection values and exceptions

Re: assigning collection values and exceptions

Maybe Matching Threads