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.

Ben Orenstein

Ben Orenstein

Member of 1 project: Ruby on Rails

Ben Orenstein's latest activity

Monday, April 25 2011
6741 Rails 3: find_or_create_by gives ArgumentError (Unknown keys) with hash param... was updated in Ruby on Rails
Ben Orenstein Ben Orenstein was invited to Ruby on Rails at 4:15 AM
Thursday, April 07 2011
6683 Small refactoring: use ActiveSupport::Concern was created in Ruby on Rails
  • Ben Orenstein
    Ben Orenstein created the ticket at 5:27 PM

    I found two files in railties that weren't taking advantage of ActiveSupport::Concern.

    Attached patch refactors them.
Monday, April 04 2011
6659 Fix misspelling of 'lambda' in test was updated in Ruby on Rails
  • Ben Orenstein
    Ben Orenstein commented at 1:10 AM

    Rohit -- this isn't a documentation change though, it's a code change.

    I'm pretty sure docrails isn't the place for this. Could you take a second look?

    Thanks!
Thursday, March 31 2011
6659 Fix misspelling of 'lambda' in test was created in Ruby on Rails
  • Ben Orenstein
    Ben Orenstein created the ticket at 6:38 PM

    A test method misspelled 'local_lambda' as 'local_lamdba'.

    Attached patch fixes this.
Friday, March 25 2011
6625 Add tests for InheritableOptions class was created in Ruby on Rails
  • Ben Orenstein
    Ben Orenstein created the ticket at 7:52 PM

    Somehow, tests were never added for ActiveSupport::InheritableOptions

    Attached patch adds these tests. Comments welcomed.
Monday, February 28 2011
6483 ActiveSupport::OrderedOptions calls to_sym redundantly was updated in Ruby on Rails
  • Ben Orenstein
    Ben Orenstein commented at 3:56 PM

    Oops. That change snuck in there accidentally.

    Thanks for the catch, Josh.

    This new patch fixes that.
  • Ben Orenstein
    Ben Orenstein commented at 3:16 PM

    Thanks Jeff!

    If applied, this will by my first non-documentation, non-test-code patch :)