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.

S. Brent Faulkner's latest activity

Friday, August 15 2008
791 Add empty_template option when rendering a collection was updated in Ruby on Rails
  • S. Brent Faulkner
    S. Brent Faulkner commented at 9:53 PM

    +1 for render_empty_collection_or_nil_returns_nil.diff

    definitely.

838 add support for selective resource routes was updated in Ruby on Rails
  • S. Brent Faulkner
    S. Brent Faulkner commented at 3:32 AM

    Hmmm... just remembered the addition of has_many and has_one...

    So, I guess I'm suggesting the following...

    
    map.resources :projects, :has_many => { :mileston...
  • S. Brent Faulkner
    S. Brent Faulkner commented at 3:22 AM

    How about using a notation similar to that used for ActiveRecord#find :includes

    We could then use...

    
    map.resources :projects => { :milestones => { :features ...
  • S. Brent Faulkner
    S. Brent Faulkner commented at 1:45 AM

    Actually that is the purpose of the selective routes.

    Jamis's proposed solution to the problem winds up doubly mapping the routes... so there is one route for ...

  • S. Brent Faulkner
    S. Brent Faulkner created the ticket at 1:23 AM

    UPDATED: I've edited this post to reflect the current patch... the original post is still included after this for the record.

    Attached is a patch for resource ...

Thursday, August 14 2008
825 test_include_query(EagerLoadPolyAssocsTest) fails for test_mysql when auto_in... was updated in Ruby on Rails
  • S. Brent Faulkner
    • Assigned user set to Michael Koziarski
    by S. Brent Faulkner at 5:27 AM
  • S. Brent Faulkner
    • Tag changed from activerecord, bug to activerecord, bug, patch, tests
    by S. Brent Faulkner at 5:04 AM

    a patch is attached to remove the dependency on auto incremented id values.

    as the original was implemented, the test would fail if the id values assigned to t...

  • S. Brent Faulkner
    S. Brent Faulkner commented at 5:02 AM

    to reproduce this problem you can either...

    a) edit your my.cnf to include auto_increment_increment=5 (so that sequential id values are missing)

    or

    b) run th...

  • S. Brent Faulkner
    S. Brent Faulkner created the ticket at 3:00 AM

    test/cases/associations/eager_load_nested_include_test.rb depends on specific primary values.

    In my case, when testing ActiveRecord using MySQL, since my MySQL...

823 test_atomic_write_preserves_default_file_permissions(AtomicWriteTest) fails w... was updated in Ruby on Rails
  • S. Brent Faulkner
    • Tag changed from bug to bug, patch
    by S. Brent Faulkner at 12:28 AM

    To reproduce the problem set your umask to 0002...

    $ umask 0002

    Then run your tests...

    $ rake test

    I've attached a patch to fix it.