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

Fixed ActiveModel::Model's inclusion chain #11004

Merged

Conversation

fredwu
Copy link
Contributor

@fredwu fredwu commented Jun 19, 2013

cc @josevalim @drogus

master version #11002

josevalim pushed a commit that referenced this pull request Jun 19, 2013
…-fix

Fixed ActiveModel::Model's inclusion chain
@josevalim josevalim merged commit 6ed17e2 into rails:4-0-stable Jun 19, 2013
@woto
Copy link
Contributor

woto commented Jun 19, 2013

Sorry for bad English. This commit will raise error if there no ancestors.

e3dc10f#commitcomment-3457605
In test case there is DefaultValue module included before ActiveModel::Model

@rafaelfranca
Copy link
Member

Yes, @fredwu could you check this

@fredwu
Copy link
Contributor Author

fredwu commented Jun 19, 2013

I'm already on it. :)

fredwu added a commit to fredwu/rails that referenced this pull request Jun 19, 2013
fredwu added a commit to fredwu/rails that referenced this pull request Jun 19, 2013
@fredwu
Copy link
Contributor Author

fredwu commented Jun 19, 2013

Fixed in #11010 and #11011. cc @rafaelfranca @woto

fredwu added a commit to fredwu/rails that referenced this pull request Jun 19, 2013
josevalim pushed a commit that referenced this pull request Jun 19, 2013
Signed-off-by: José Valim <jose.valim@plataformatec.com.br>
josevalim pushed a commit that referenced this pull request Jun 19, 2013
Signed-off-by: José Valim <jose.valim@plataformatec.com.br>
josevalim pushed a commit that referenced this pull request Jun 19, 2013
Signed-off-by: José Valim <jose.valim@plataformatec.com.br>
@woto
Copy link
Contributor

woto commented Jun 19, 2013

New problem with last commit on ActiveModel.
@fredwu Please look at this woto@9fc20cf
May be i am doing something wrong?

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