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

Make missed association exception message more informative #12511

Conversation

pftg
Copy link
Contributor

@pftg pftg commented Oct 12, 2013

Add target class name, which should have missed association on preload,
into exception message to simplify detecting problem part.

@rafaelfranca
Copy link
Member

@pftg could you add a CHANGELOG entry?

Add target class name, which should have missed association on preload,
into exception message to simplify detecting problem part.
@pftg
Copy link
Contributor Author

pftg commented Oct 13, 2013

@rafaelfranca, I have updated PR with changelog entry.

rafaelfranca added a commit that referenced this pull request Oct 13, 2013
…for_incorrect_association

Make missed association exception message more informative
@rafaelfranca rafaelfranca merged commit c56dc75 into rails:master Oct 13, 2013
rafaelfranca added a commit that referenced this pull request Oct 13, 2013
…for_incorrect_association

Make missed association exception message more informative
Conflicts:
	activerecord/CHANGELOG.md
	activerecord/lib/active_record/associations/preloader.rb
@pftg pftg deleted the informative_raise_message_for_incorrect_association branch October 13, 2013 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants