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.

Frederick Cheung

Frederick Cheung

Member of 1 project: Ruby on Rails

Frederick Cheung's latest activity

Saturday, December 27 2008
732 Deprecation warning in xml_serializer.rb:308 was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 11:44 AM

    The join table ids are the only ones wrong because if you ask CmdbApplication 'What's the type of Cmdb_application_id' it says 'I don't know' because the corres...
1642 HasOneThroughAssociation should not be a child of HasManyThroughAssociation was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 11:14 AM

    Good call - that has always seemed a bit messy to me (and has been responsible for a fair few bugs). Haven't looked at patch in detail yet, but tests all pass
1631 Documentation patch for array :joins syntax was updated in Ruby on Rails
  • Frederick Cheung
    • State changed from new to invalid
    by Frederick Cheung at 1:35 AM

    I see you found lifo :-)
Friday, December 26 2008
732 Deprecation warning in xml_serializer.rb:308 was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 11:57 PM

    Figured this one out. If anything it's more of a habtm bug than xml_serialization: habtm does a join but (by default) does a select *, so you get the columns fr...
  • Frederick Cheung
    • State changed from incomplete to open
    by Frederick Cheung at 2:32 PM

    Ah ok, I mistakenly assumed that test was from your app
  • Frederick Cheung
    Frederick Cheung commented at 1:03 PM

    What do the corresponding tables/relations look like ?
1637 Accessing datetime via read_attribute loses timezone was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 10:26 PM

    Well currently the conversion happens at the level of the automatically generated attribute reader so it's not surprising that this happens.

    All you need to do ...
Thursday, December 25 2008
1633 has_many with :primary_key option bug was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 10:32 PM

    D'oh - read a little too quickly !
  • Frederick Cheung
    Frederick Cheung commented at 9:52 PM

    That's odd as foo.id always gets you the value of the primary key (whether it's called id or not) - I'm not sure the change in the patch accomplishes anything.
1111 String#parameterize should accept a custom separator was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 9:50 PM

    I don't think that's the point - The point is that String#parameterize doesn't allow you to specify the separator argument that is passed to Inflector.parameter...