coryschires/impressionist — comment on commit 5386874eca1959ea84751b44a3332ac01f90089a
File: app/models/impressionist/impressionable.rb line 73
By johnmcaliley:

Thanks for submitting this patch! I have been meaning to implement this feature for months, but just never seem to find the time. I need to look a little closer before I release this to gemcutter, but I am planning on adding this pull this to the repo. I just had a question about the code above. Right now the engine.rb file extends this module and includes InstanceMethods, which is similar to what you are doing with self.included and AR:Base.send. I am just trying to understand why it was necessary to change this. Can you explain?

I am asking because I prefer to put all the includes and extends in the engine initializer, so all dynamic code modifications are defined in one place.

Thanks!