Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Kevin Glowacz's latest activity

Monday, July 14 2008
327 nested fields_for with index should account for the index was updated in Ruby on Rails
  • Kevin Glowacz
    Kevin Glowacz commented at 8:04 PM

    updating patch to apply cleanly to edge. I think the only difference is in the hashes and line numbers, but I get an error with the old patch for some reason.

    ...

  • Kevin Glowacz
    Kevin Glowacz commented at 5:53 PM

    It is a good thing you had me write those tests. I discovered that nested auto_index didn't work. Now it does along with more tests to cover that the inner fie...

  • Kevin Glowacz
    Kevin Glowacz commented at 5:12 PM

    s/to/too

  • Kevin Glowacz
    Kevin Glowacz commented at 5:08 PM

    well those tests only showed the exact behaviour that changed, but no one ever complains about test being to explicit. More tests are on the way.

  • Kevin Glowacz
    Kevin Glowacz commented at 4:52 PM

    no it isn't in the patch. I just assumed it would be assumed that everything else continued to work. I suppose I should add it in.

1 earlier change…
  • Kevin Glowacz
    Kevin Glowacz commented at 4:42 PM

    This does work with indexes on the nested fields for. (It is actually what I use this for.)

    form_for(:post, @post, :index => 1) do |f|
      f.fields_for(:commen...
Monday, June 30 2008
518 index does not work on fields_for select was created in Ruby on Rails
  • Kevin Glowacz
    Kevin Glowacz created the ticket at 8:41 PM
    fields_for :post, @post, :index => 1 do |f|
      f.select(:category, %w( abe <mus> hest))
    end
    

    currently does not add the index to the id and name.

    This patch ...

327 nested fields_for with index should account for the index was updated in Ruby on Rails
  • Kevin Glowacz
    Kevin Glowacz commented at 8:35 PM

    the updated patch for real this time

  • Kevin Glowacz
    • Tag set to actionpack, patch, tested
    by Kevin Glowacz at 8:17 PM

    updating tests to work with rails edge and the _erbout to output_buffer change

Thursday, June 05 2008
327 nested fields_for with index should account for the index was updated in Ruby on Rails
  • Kevin Glowacz
    Kevin Glowacz commented at 4:49 PM

    new patch with modified regex that works a lil better for me.