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.

dira

dira

freelancer web · Not a member

dira's latest activity

Monday, October 12 2009
3225 AR.count - Using symbol in :group with :join fails to return proper column was updated in Ruby on Rails
  • dira
    dira commented at 2:10 PM

    Cool, thanks for the link!
  • dira
    dira commented at 12:56 PM

    Hi Christopher,

    thanks for your feedback. Indeed I tested only on MySQL as I did not have the other databases installed :|

    My Postgres installation is still in ...
Tuesday, October 06 2009
3225 AR.count - Using symbol in :group with :join fails to return proper column was updated in Ruby on Rails
  • dira
    • Tag changed from active_record, bugmash, count, group, group_field, join, table_name to active_record, bugmash, count, group, group_field, join, patch, table_name
    by dira at 11:17 PM

    I have attached a patch with tests and implementation.
Sunday, September 27 2009
2132 Confusing behavior with attr_readonly was updated in Ruby on Rails
  • dira
    dira commented at 11:55 AM

    +1 - this behavior is a lot less confusing as the previous one

    I attached a patch. The previous patch did not apply on master anymore; the attached one does.
2990 [PATCH] Non standard ID column on has_many :through was updated in Ruby on Rails
  • dira
    dira commented at 11:37 AM

    the master patch applies cleanly

    the 2.3.stable patch applies cleanly
2493 Eager Loading With :limit Returns Wrong Result Set (MySQL) was updated in Ruby on Rails
  • dira
    dira commented at 11:24 AM

    Verified, +1

    The preivious patch does not apply anymore. The attached one applies cleanly on master and demonstrates the bug
3235 default scoping and associations was updated in Ruby on Rails
  • dira
    dira commented at 11:07 AM

    I've attached a patch. This test applies on master and shows the bug.
  • dira
    dira commented at 11:02 AM

    +1 - applying the conditions of the scope is clearly a bug

    verified
3234 change_column carries over default when it can't be carried was updated in Ruby on Rails
  • dira
    dira commented at 10:37 AM

    Not reproducible.

    Tested in 2.3.stable and on master, by running the migration on the dev app & also in tests. The column is changed correctly and there is no e...
Monday, August 10 2009
2990 None standard ID column on has_many :through was updated in Ruby on Rails
  • dira
    dira commented at 9:27 AM

    Here is the correct patch - with tests and implementation.

    Thank you all for the feedback.