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

Fix Errors#from_type use known_attributes #90

Merged
merged 2 commits into from
Jan 31, 2014
Merged

Fix Errors#from_type use known_attributes #90

merged 2 commits into from
Jan 31, 2014

Conversation

MorganShowman
Copy link

This commit replaces the use of #attributes.keys with #known_attributes
in Errors#from_array and Errors#from_hash. This is done so that when
ActiveResource grabs errors for attributes defined in the schema, but
don't yet exist as an attribute on the object, the error gets assigned
to the known attribute instead of the base object.

This commit replaces the use of #attributes.keys with #known_attributes
in Errors#from_array and Errors#from_hash. This is done so that when
ActiveResource grabs errors for attributes defined in the schema, but
don't yet exist as an attribute on the object, the error gets assigned
to the known attribute instead of the base object.
@MorganShowman
Copy link
Author

Job 100.4 (rvm: rbx-19mode) errored due to a timeout:

ERROR:  Could not find a valid gem 'bundler' (>= 0), here is why:
          Unable to download data from https://rubygems.org/ - Errno::ETIMEDOUT: Connection timed out - connect(2) (https://rubygems.org/latest_specs.4.8.gz)

@Fivell
Copy link
Contributor

Fivell commented Aug 27, 2013

+1

* master:
  Updating changelog.
  Adding `lib/activeresource.rb` to the project.
guilleiguaran added a commit that referenced this pull request Jan 31, 2014
…own-attributes

Fix Errors#from_type use known_attributes
@guilleiguaran guilleiguaran merged commit eb45a8f into rails:master Jan 31, 2014
guilleiguaran added a commit that referenced this pull request Nov 30, 2015
…own-attributes

Fix Errors#from_type use known_attributes
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.

3 participants