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's latest activity

Saturday, September 26 2009
3035 auto_link in text_helper.rb doesn't handle some gmail URLS was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 2:51 AM

    Close this ticket, tried several urls and they all pass.

    attached a test to check this specific example, it passes on 2-3-stable

2989 Setting the id of a belongs_to object does not update the reference to the ob... was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 2:36 AM

    @Peter: mmm, do you see anywhere else it needs to be applied except for a foreign key assignment in a :belongs_to association?

3033 Regression on 2.3 branch: Allow delegating methods to nil was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 2:25 AM

    Bryan, can you supply a patch within a single file? please ? :)

3098 Metal names conflict with models was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 2:08 AM

    +1 verified. not sure that this is the right solution thu, maybe there's a way to scope the metal models under Metal:: ?

3218 default_scope is having a side affecct on create was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 2:03 AM

    Mmmm,
    by doing

    
    - self.attributes = attributes unless attributes.nil?
      self.class.send(:scope, :create).each { |att,value| self.send("#{att}=", value) } if se...
  • Elad Meidar
    Elad Meidar commented at 2:02 AM

    Mmmm,
    by doing

    -        self.attributes = attributes unless attributes.nil?

         self.class.send(:scope, :create).each { |att,value| self.send("#{at...
  • Elad Meidar
    • Tag changed from activerecord, bugmash, default_scope to activerecord, bugmash, bugmash-review, default_scope
    by Elad Meidar at 1:53 AM
2998 named_scope ignores :include option was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 1:43 AM

    +1 verified on 2-3-stable, i tried to follow up named_scope.rb and try to point to where exactly the :include option is dropped/ignored when set within a lambda...

Friday, September 25 2009
3027 Association Table Names Incorrectly Generated when set_table_name used was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 11:44 PM

    +1 verified

    if join_table is missing from the macro options, the join table name will be calculated on associations.rb#1596, by using undecorated_table_name (ht...

2989 Setting the id of a belongs_to object does not update the reference to the ob... was updated in Ruby on Rails
  • Elad Meidar
    Elad Meidar commented at 11:04 PM

    -1 current situation is expected and concurrent with other attribute behaviors. The reason for that is that there is no actual save upon the new association id ...