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.

There was a problem

You must be a member of this account.

railsbob

railsbob

Not a member

railsbob's latest activity

Sunday, August 09 2009
1255 time.end_of_day not entirely accurate was updated in Ruby on Rails
  • railsbob
    railsbob commented at 1:28 PM

    +1 Hugo's patch applied cleanly on master, with ruby 1.8
1845 Finder methods ignore :include in has_many :through was updated in Ruby on Rails
  • railsbob
    railsbob commented at 1:08 PM

    verified

    This error is reproducible. The error is caused because options[:include] was assigned nil in has_many_through_association.rb.

    def construct_find_opti...
Saturday, August 08 2009
2934 association_collection#create acts as association_collection#create! was updated in Ruby on Rails
  • railsbob
    railsbob commented at 11:48 PM

    As discussed with lifo, the previous patch is modifying << method. I am attaching another patch, which removes << from create method.
  • railsbob
    railsbob commented at 9:04 PM

    Sorry, please ignore the previous patch (whitespace error). This one applies cleanly.
  • railsbob
    railsbob commented at 6:15 PM

    The following patch fixes the error.
  • railsbob
    railsbob commented at 6:01 PM

    I have attached a patch for the failing test case.
  • railsbob
    railsbob commented at 12:21 PM

    Verified

    I was able to reproduce this error with the help of mentioned example.
2493 Eager Loading With :limit Returns Wrong Result Set (MySQL) was updated in Ruby on Rails
  • railsbob
    railsbob commented at 3:34 AM

    Sorry for the formatting mess, the tables were supposed to be:

    
Data in Table 'as':

+----+------+
| id | name |
+----+------+
|  1 | A1   | 
|  2 | A2   | 
| ...
  • railsbob
    railsbob commented at 3:32 AM

    Verified

    I was able to test this using following scenario:

    
  class A < ActiveRecord::Base
    has_many :bs, :select => 'id, position, a_id'
  end

  class B <...
769 auto_link should apply html_options to email addresses too was updated in Ruby on Rails
  • railsbob
    railsbob commented at 1:24 AM

    Not Reproducible - It is working as expected.