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

Already on GitHub? Sign in to your account

Raise more readable error message #117

Merged
merged 1 commit into from Oct 5, 2012

Conversation

Projects
None yet
2 participants
Contributor

indrekj commented Oct 1, 2012

class User
  include Virtus
  attribute :adderss, Address
end

When I accidentally set a wrong attribute to the user address User.new.address = Object.new, then I'll get an error undefined methodeach' for nil:NilClass`. This doesn't tell much about what was wrong. I tried to make it a bit better with this pull request.

@solnic solnic added a commit that referenced this pull request Oct 5, 2012

@solnic solnic Merge pull request #117 from indrekj/error-message
Raise more readable error message
6b512b8

@solnic solnic merged commit 6b512b8 into solnic:master Oct 5, 2012

1 check passed

default The Travis build passed
Details
Owner

solnic commented Oct 5, 2012

Merged! Thanks for that :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment