Permalink
Browse files

Merge pull request #26110 from aditya-kapoor/minor-doc-fix

Minor doc fix related to ActiveModel::SecurePassword [ci skip]
  • Loading branch information...
guilleiguaran committed Aug 11, 2016
2 parents 432222f + d61aec3 commit 5cb6b887d410fcd1ea821b32374a17a94acdc413
Showing with 5 additions and 2 deletions.
  1. +5 −2 guides/source/active_model_basics.md
@@ -416,7 +416,6 @@ the Active Model API.
```ruby
class Person
include ActiveModel::Model
end
```
@@ -467,7 +466,7 @@ In order to make this work, the model must have an accessor named `password_dige
The `has_secure_password` will add the following validations on the `password` accessor:
1. Password should be present.
2. Password should be equal to its confirmation.
2. Password should be equal to its confirmation (provided +password_confirmation+ is passed along).
3. The maximum length of a password is 72 (required by `bcrypt` on which ActiveModel::SecurePassword depends)
#### Examples
@@ -493,6 +492,10 @@ person.valid? # => false
person.password = person.password_confirmation = 'a' * 100
person.valid? # => false
# When only password is supplied with no password_confirmation.
person.password = 'aditya'
person.valid? # => true
# When all validations are passed.
person.password = person.password_confirmation = 'aditya'
person.valid? # => true

0 comments on commit 5cb6b88

Please sign in to comment.