Piers Cawley wrote:
> On 24/07/07, Ben Munat <[EMAIL PROTECTED]> wrote:
>> Anyway, I think I'll definitely check out Jonathan's plugin on my next
>> app... looks all nice and
>> "DSL-ish". :-)
>
> The problem with the plugin approach here is that it's monkeypatching
> a pretty fundamental method, and undocumented, method. And that makes
> it fragile; if the implementation of initialize ever changes (and
> there's no reason that it shouldn't), the plugin breaks.
>
> Still, if it's the only game in town...
>
Hmm, yeah that's a good point... always the risk with monkeypatching.
So, Piers, you're the one that started this whole thing... if it really is safe
to do:
class MyModel < ActiveRecord::Base
def initialize(attrs = {}, &block)
super
# all my initialization stuff here
end
end
and this is documented in the AR::Base docs, then this does kind of seem like
enough. I mean, it's
what I did all the time in my java life.
I'd just swear that someone told me the holy beasts of doom would swarm down
upon me if I ever tried
to override AR::Base#initialize.
Ben
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby
on Rails: Core" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---
