Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed useless utility classes inside HABTM association builder #33158

Merged
merged 1 commit into from
Jun 19, 2018

Conversation

bogdan
Copy link
Contributor

@bogdan bogdan commented Jun 19, 2018

Summary

This is yet another cleanup patch removing some useless classes inside HABTM association builder.
The additional supportive classes were probably build because of the need to load the table_name for the HABTM model lazily (covered with test). The same behavior can be achieved by using a simple lambda.

@rails-bot
Copy link

r? @eileencodes

(@rails-bot has picked a reviewer for you, use r? to override)

@kamipo kamipo merged commit ee21e05 into rails:master Jun 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants