On 12 Apr, 05:25, Bob Smith <[email protected]> wrote: > Please help. I've tried all i know with no results... > > I have an app with a master class using has_many to another class. > When I create a new record with the params showing all fields for > master and the has_many class it seems to work OK. But doing an edit > on the record shows two copies of the has_many class, and checking > mysql shows the two copies. >
You're doing this in both cases > Visit.find_or_create_by_household_id_and_month_and_year(:household_id > => params[:id], :month => @today.month, :year => @today.year) Which will create a new visit unless there is one with all of those same parameters. Perhaps you only wanted find_or_create_by_household_id ? Also if you used the association methods (ie household.visit = etc.) then active record would take care of destroying / nulling the previous row Fred > end > > # POST /households > # POST /households.xml > def create > @today = Date.today > @household = Household.new(params[:household]) > [1,2,3,4,5,6,7,8,9,10,11,12].each do |month| > @visit = > Visit.find_or_create_by_household_id_and_month_and_year(:household_id > => params[:id], :month => @today.month, :year => @today.year) > end > > respond_to do |format| > # �[email protected] > if @household.update_attributes(params[:household]) > flash[:notice] = 'Household was successfully created.' > format.html { redirect_to(@household) } > format.xml { render :xml => @household, :status > => :created, :location => @household } > else > format.html { render :action => "new" } > format.xml { render :xml => @household.errors, :status > => :unprocessable_entity } > end > end > end > > # PUT /households/1 > # PUT /households/1.xml > def update > # debugger > @today = Date.today > @household = Household.find(params[:id]) > @v = > Visit.find_or_create_by_household_id_and_month_and_year(:household_id > => params[:id], :month => @today.month, :year => @today.year) > @v.update_attributes(params['visit']) > respond_to do |format| > if @household.update_attributes(params[:household]) > flash[:notice] = 'Household was successfully updated.' > format.html { redirect_to(@household) } > format.xml { head :ok } > else > format.html { render :action => "edit" } > format.xml { render :xml => @household.errors, :status > => :unprocessable_entity } > end > end > end > > # DELETE /households/1 > # DELETE /households/1.xml > def destroy > @household = Household.find(params[:id]) > @household.destroy > > respond_to do |format| > format.html { redirect_to(households_url) } > format.xml { head :ok } > end > end > end > > <h1>New household</h1> > > <% form_for(@household) do |f| %> > <%= f.error_messages %> > > <p> > <%= f.label :name %><br /> > <%= f.text_field :name %> > </p> > <%= render :partial => 'people', :object => @household %> > > <%= render :partial => 'visit', :object => @visit %> > <p> > <%= f.submit 'Create' %> > </p> > <% end %> > > <%= link_to 'Back', households_path %> > > <h4>People in household</h4> > <b> Sex</b> > <b> Birthday</b> > <table> > <div id="people"> > <%= render :partial => 'person', :collection => > @household.people > %> > </div> > </table> > <%= link_to_function "Add a Person" do |page| > page.insert_html :bottom, :people, :partial => 'person', > :object => > Person.new > end %> > > <div id="person"> > <% @household.build_person unless @household.people %> > <% fields_for "household[people_attributes][]", person do | > person_form| %> > <tr><td> <%= person_form.text_field :sex, :size => 1, :maxlength > =>1, :index => nil, :autocomplete => "off" %></td> > <td><%= person_form.text_field :month, :size => 2, :maxlength > =>2, :index => nil, :autocomplete => "off" %>/ > <%= person_form.text_field :day, :size => 2, :maxlength > =>2, :index => nil, :autocomplete => "off" %>/ > <%= person_form.text_field :year, :size => 4, :maxlength > =>4, :index => nil, :autocomplete => "off" %></td> > <td> <% unless person_form.object.new_record? %> > <%= person_form.hidden_field :id, :index => > nil %> > <% end %> > </tr> > <% @person = person %> > <% end %> > > </div> -- 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 [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en.
