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

In disable_table_options.rb, the behavior is different because the super is not called. #357

Merged
merged 1 commit into from
Jun 7, 2021

Conversation

pocari
Copy link
Contributor

@pocari pocari commented Jun 7, 2021

Since super was not called in the inherited method, ActiveSupport::DescendantsTracker's inherited was not called anymore and ActiveRecord::ConnectionAdapters:: AbstractAdapter.descendants was behaving incorrectly.

(Also, this is why the latest https://github.com/amatsuda/database_rewinder is not working anymore).

The steps to reproduce it are summarized in this repository.
https://github.com/pocari/mysql2-rails-DescendantsTracker

I have fixed it.

@winebarrel
Copy link
Collaborator

Thank you!

@winebarrel winebarrel merged commit 93760f6 into ridgepole:0.9 Jun 7, 2021
@pocari pocari deleted the fix-call-super-in-inherited branch June 7, 2021 15:13
@winebarrel
Copy link
Collaborator

v0.9.5 has been released.

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

2 participants