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.

Sign up for GitHub

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

Jump to bottom

Allow the use of the :if argument to call local methods #12

Open

DangerDawson wants to merge 2 commits into brycesenz:master from DangerDawson:if_method

DangerDawson commented Oct 14, 2016

Previous to this commit the following would fail silently:

validates_hash_keys :hash do
  validates :key, presence: true, if: :if_method
end

def if_method
  false
end

This PR allows us to call a locally scoped method only for the if: argument.

If you are happy with this PR we should probably add documentation

David Dawson added 2 commits

October 14, 2016 18:10


          Allow the use of the :if argument to call local methods

467204a


          Remove commented out code

1e11590

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

Labels

None yet