Brian Samson's profile - rails

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 Samson's latest activity

Thursday, March 10 2011

~~5023~~ ~~Preloading polymorphic association doesn't work with namespaced models~~ was updated in Ruby on Rails

Brian Samson commented at 6:45 PM

+1

This is still a bug in rails 2.3.11, as it has been since around 2.3.5

I've attached a patch for it in 2.3.11 that we use. I think it should be included in t...

Tuesday, March 30 2010

~~578~~ ~~Mime type lookup_by_extesion is case sensitive~~ was updated in Ruby on Rails

Brian Samson commented at 5:58 PM
Yes I totally agree, the entire system should be case insensitive. However for reasons I do not understand the committers seems to disagree with you, me, apache...

2627 has_many :dependent => :delete_all throws exception on models in modules was updated in Ruby on Rails

Brian Samson commented at 5:54 PM
I think that perhaps Andrew's patch would fix this problem, but it looks like that has been pushed off to 3.0. Basically my patch just makes this behavior ask f...

Wednesday, October 14 2009

2627 has_many :dependent => :delete_all throws exception on models in modules was updated in Ruby on Rails

Brian Samson commented at 10:48 PM

Good catch, Rizwan.

I was using double quotes in the eval which threw off sqlite's quoting. I changed it to %() which should work unless you know of a database ...

Monday, October 12 2009

2627 has_many :dependent => :delete_all throws exception on models in modules was updated in Ruby on Rails

Brian Samson commented at 8:14 PM
This bug still exists as far as I can tell. I updated this patch to apply cleanly to edge, if somebody could double-check that for me quick I'd appreciate it.

Saturday, May 09 2009

2627 has_many :dependent => :delete_all throws exception on models in modules was created in Ruby on Rails

Brian Samson created the ticket at 7:03 AM
If a model that is declared inside a module is associated via a has_many to another model in the same module, and :dependent is set to :delete_all or :nullify, ...

Friday, July 18 2008

~~578~~ ~~Mime type lookup_by_extesion is case sensitive~~ was updated in Ruby on Rails

Brian Samson commented at 7:46 AM

Ripta: Thanks for writing that test case, I appreciate it.

Pratik: Like I said in the original post, I noticed this issue mapping user-uploaded files to mime-...

Tuesday, July 08 2008

578 Mime type lookup_by_extesion is case sensitive was updated in Ruby on Rails

Brian Samson commented at 10:10 PM

Sorry that code snippet didn't come through right because of angle brackets.

>> Mime::Type.lookup_by_extension('JPG')

=> #<Mime::Type:0x18cd970 @synonyms=[],...
Brian Samson created the ticket at 10:07 PM
When reverse resolving extensions to mime types via Mime::Type.lookup_by_extension, the lookup fails unless the extension is of the same case as was registered...