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.

sr.iniv.t's latest activity

Sunday, September 27 2009
3111 [PATCH] ActionController::Integration::Session#get does not set HTTP_REFERER was updated in Ruby on Rails
  • sr.iniv.t
    sr.iniv.t commented at 7:41 PM

    +1 verified. +1 for the idea too.

    Patch applies cleanly on 2-3-stable and all tests pass.

1472 ActiveResource errors.from_xml Can't handle symboled attribute keys because o... was updated in Ruby on Rails
  • sr.iniv.t
    sr.iniv.t commented at 7:34 PM

    +1 verified.

    The patch applies cleanly on 2-3-stable and tests pass. However, a different patch is needed for master because the activeresource test directory s...

3129 [patch] better cleaning of gem directories was updated in Ruby on Rails
  • sr.iniv.t
    sr.iniv.t commented at 7:24 PM

    +1 verified.

    The patch applies on 2-3-stable and all tests pass.

2536 .:format feature breaks URL compliance (rfc1738) - should be optional was updated in Ruby on Rails
  • sr.iniv.t
    sr.iniv.t commented at 7:15 PM

    +1

    I've had a few surprises trying to use version numbers in resource urls in the past.

3200 [PATCH] For Rails 2.3 stable. Add layout override option to ActionMailer was updated in Ruby on Rails
  • sr.iniv.t
    sr.iniv.t commented at 7:10 PM

    +1 verified.

    The idea is nice addition. The patch applies cleanly on 2-3-stable and tests pass.

3274 When an ActiveRecord has a column named 'field', bad behavior happens was updated in Ruby on Rails
  • sr.iniv.t
    sr.iniv.t commented at 7:03 PM

    +1 on rbxbx's patch (applies cleanly on master and all tests pass). Joe's patch applies, but the test test_should_allow_except_option_for_list_of_authors fails ...

3278 [PATCH] Allow PKG_NAME env variable to control the directory name used when ... was updated in Ruby on Rails
  • sr.iniv.t
    sr.iniv.t commented at 6:51 PM

    +1 verified.

    The patch applies cleanly on 2-3-stable and master. I've wished for something like this while doing bug verifications.

3188 [ActiveRecord] create(:association_id => @object.id) vs create(:association_i... was updated in Ruby on Rails
  • sr.iniv.t
    sr.iniv.t commented at 6:42 PM

    +1 verified.

    Elad's patch applies cleanly on master and 2-3-stable and the issue is reproducible.

3275 [PATCH] Optimization for dropping multiple columns in MySQL was updated in Ruby on Rails
  • sr.iniv.t
    sr.iniv.t commented at 6:35 PM

    +1 verified.

    The patch applies cleanly on 2-3-stable and master. But the patch lacks tests.

3130 [PATCH] add strip_attributes support to ActiveModel and Active Record was updated in Ruby on Rails
  • sr.iniv.t
    sr.iniv.t commented at 7:57 AM

    +1 verified.

    Great idea - i have run into situations where we needed this kind of behavior would have been helpful. The patch applies without any problems and a...