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.

Elad Meidar

Elad Meidar

Not a member

Elad Meidar's latest activity

Thursday, October 14 2010
5796 Rails 3.0 generator inconsistency was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 3:33 PM

    Verified and Patch is working on stable,
    Just one thing: the patch adds a config.generators block, isn't there anything else that adds this block already and mi...
3565 Add limit functionality to find first and last was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 3:26 PM

    Makes sense that the Rails default would fetch all and select back n accordingly, proper SQL query should be implemented on the adapter level since it involves ...
5122 Add weeks_ago helper was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 3:16 PM

    I hate to be a party /smasher/ but shouldn't all those helpers be joined into one?, something like:

    DateTime.now.ago(10, :minutes)

    and if i think about it:

    Date...
Friday, June 18 2010
3984 rails does not handle columns that start with a number was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 11:07 AM

    I don't really think it makes any sense.

    Since columns are accessed via getter/setter ruby methods, the Ruby language restriction of not using method names that...
Tuesday, February 09 2010
3903 counter_cache not updating on update_attributes was created in Ruby on Rails
  • Elad Meidar
    Elad Meidar created the ticket at 4:56 AM

    When trying to use a simple case of cache_counter:

      class Registration < ActiveRecord::Base
    belongs_to :section, :counter_cache => :registrations_count
  ...
Monday, October 26 2009
3426 current child index on fields_for nested attributes was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 5:08 AM

    Ok, this patch makes more sense.

    I've added FormHelper#fields_for_with_index and some tests (duplicates of some fields_for tests).

    Going through the entire fiel...
  • Elad Meidar
    Elad Meidar commented at 2:57 AM

    Well, Pratik and i talked and he suggested a totally different way to solve this issue, fields_for_with_index.

    As far as i am concerned, this patch is invalid :...
Sunday, October 25 2009
3426 current child index on fields_for nested attributes was created in Ruby on Rails
  • Elad Meidar
    Elad Meidar created the ticket at 11:47 PM

    I tried to find a way to access the current child index while invoking fields_for on a collection, i needed that in order to provide meaningful CSS selectors fo...
Thursday, October 01 2009
2878 [patch] image_tag doesn't allow :escape => false option anymore was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 5:41 AM

    Maybe a "review" tag is going to help, but i am not sure.
Sunday, September 27 2009
3005 :counter_cache not updated if polymorphic => true when assigning association was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 9:53 PM

    +1 on solution idea, patches apply cleanly on 2-3-stable and master, i believe the implementation itself is mostly right.

    @Henry, as for the delete thing, i thi...