alias_method_chain instead of super

laserlemon / vestal_versions

Keep a DRY history of your ActiveRecord models' changes

MIT License

933 stars 229 forks source link

alias_method_chain instead of super #31

Open marcandre opened 14 years ago

marcandre commented 14 years ago

Hi! Just checking out vestal_versions, looks really good. I'm trying to modify it slightly for my needs, and I was wondering: any reason for all these alias_method_chain?

Have you thought of having a base module with trivial methods, and all the others could simply call super instead of method chaining?

I'd submit a patch if you want.

adamcooper commented 14 years ago

I've been looking through the source and was wondering the same thing?

Especially now that there is a the rich ActiveSupport::Concern in ActiveSupport 3.