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

Monday, December 08 2008
1487 from_xml and from_json don't work was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 3:03 PM

    Looks like lighthouse has somewhat mangled what you wrote.

    Can you reduce this to a minimal example, where you - take a model - dump the xml - change the xml us...
1352 Enumerable#group_by is O(n^2) was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 11:26 AM

    As a very quick test I did this (in a rails 2.2 app)

    
module Enumerable
  def old_group_by
    groups = []
    inject({}) do |grouped, element|
      index = ...
1360 ActiveRecord test suite should SQLite by default was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 11:08 AM

    Can't you already do this with

    rake test_sqlite3
1520 image_tag has period appended to src attribute was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 10:59 AM

    Sounds like you've got it figured out, so why don't you create a patch/tests :-)
1529 scafffold, 1st column value is wrong was updated in Ruby on Rails
  • Frederick Cheung
    • State changed from new to invalid
    by Frederick Cheung at 10:47 AM

    Type is a "magic" column for activerecord, used for single table inheritance (and slightly magic for ruby too, since Object#type is a deprecated alias for Objec...
Sunday, December 07 2008
1518 Bug with dirty models in integer fields was updated in Ruby on Rails
1521 Strings That End in class Are Not Properly Singularized was updated in Ruby on Rails
  • Frederick Cheung
    • State changed from new to invalid
    by Frederick Cheung at 2:47 PM

    To quote inflector.rb:

    The Rails core team has stated patches for the inflections library will not be accepted in order to avoid breaking legacy applications ...
1522 render_to_string doesn't use :locals option was updated in Ruby on Rails
  • Frederick Cheung
    • State changed from new to invalid
    by Frederick Cheung at 12:59 PM

    I just did a quick experiment and this seemed to work ok (on 2.2.2) . Can you construct a failing test/example to show exactly how it isn't working for you?
Frederick Cheung Frederick Cheung was invited to Ruby on Rails at 12:32 PM
Thursday, October 16 2008
1185 Old-single-query-includes should be globally disableable was updated in Ruby on Rails
  • Frederick Cheung
    Frederick Cheung commented at 7:10 PM

    Even that can get hairy when table aliases are involved but at that point it's quite tricky to write the conditions anyway. Definitely worth a try

    On 16 Oct 200...