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.

Brian Hogan

Brian Hogan

Just another ninja · Not a member

Brian Hogan's latest activity

Friday, June 04 2010
4743 Session Cookie breaks if used with custom cookie in rails 2.3.8 was updated in Ruby on Rails
  • Brian Hogan
    Brian Hogan commented at 4:31 PM

    This seems to work for my apps as well. Can we get this looked at by core ASAP?
Thursday, April 09 2009
2463 Add first and last to OrderedHash was updated in Ruby on Rails
  • Brian Hogan
    Brian Hogan commented at 6:44 AM

    +1 just because of uniformity. Like .each or .collect.
Sunday, March 22 2009
2272 SQLite / SQLite3 ActiveRecord Adapter should quote table names when renaming ... was updated in Ruby on Rails
  • Brian Hogan
    Brian Hogan commented at 5:20 AM

    Thanks to tpope for noticing my tests sucked - Patch with better tests.
Friday, March 20 2009
2272 SQLite / SQLite3 ActiveRecord Adapter should quote table names when renaming ... was updated in Ruby on Rails
  • Brian Hogan
    • Title changed from ActiveRecord should quote table names to SQLite / SQLite3 ActiveRecord Adapter should quote table names when renaming tables
    by Brian Hogan at 1:47 PM

    Changed title to make this more descriptive
  • Brian Hogan
    Brian Hogan commented at 3:16 AM

    How about we try this again with a patch that uses the quote_table_name mechanism instead of just single quotes? :)
  • Brian Hogan
    • Tag set to 2.3.2, activerecord, patch, sql, sqlite, sqlite3
    by Brian Hogan at 3:00 AM

    Here's a patch with tests that resolves the issue. I've seen this come up for quite some time now and finally figured out how to write appropriate tests. Anyone...
Wednesday, December 31 2008
1623 Automatically invoke Update task after freezing to gems was updated in Ruby on Rails
  • Brian Hogan
    Brian Hogan commented at 4:30 AM

    +1

    It may not be necessary but it's inconsistent and that's what bugs me.
Monday, May 12 2008
168 [PATCH] Generators create tests with require 'test_helper' which doesn't work. was created in Ruby on Rails
  • Brian Hogan
    Brian Hogan created the ticket at 4:31 AM

    The generators all create tests that use

    require 'test_helper'

    Howegver, when you run the tests in TextMate or at the command-line via ruby test/units/foo_te...