Let railtie auto-include ActiveRecord::Acts::Tree #4

Merged
merged 1 commit into from Apr 18, 2012

Conversation

Projects
None yet
2 participants
@kenn
Collaborator

kenn commented Apr 18, 2012

To make the public API consistent with other acts_as libraries, such as acts_as_list.

@amerine

This comment has been minimized.

Show comment
Hide comment
@amerine

amerine Apr 18, 2012

Owner

@kenn Thanks for the pull request.

I intentionally made the change to acts_as_tree to force uses to explicitly include the module in the models they want that support in. Rails 4 will no longer have the init.rb hook that previous versions did. You can see an example in the README.

I will merge this in for consistency purposes, but I expect that in a future change I'll force users to use the module. Thanks again for your help!

Owner

amerine commented Apr 18, 2012

@kenn Thanks for the pull request.

I intentionally made the change to acts_as_tree to force uses to explicitly include the module in the models they want that support in. Rails 4 will no longer have the init.rb hook that previous versions did. You can see an example in the README.

I will merge this in for consistency purposes, but I expect that in a future change I'll force users to use the module. Thanks again for your help!

amerine added a commit that referenced this pull request Apr 18, 2012

Merge pull request #4 from kenn/railtie_include
Let railtie auto-include ActiveRecord::Acts::Tree

@amerine amerine merged commit c94c2dd into amerine:master Apr 18, 2012

@amerine

This comment has been minimized.

Show comment
Hide comment
@amerine

amerine Apr 18, 2012

Owner
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment