diff options
author | Edouard CHIN <edouard.chin@shopify.com> | 2019-07-09 14:40:57 +0200 |
---|---|---|
committer | Edouard CHIN <edouard.chin@shopify.com> | 2019-07-16 14:28:38 +0200 |
commit | b677adede027eefed41f6b37096df7bbc4eb3d38 (patch) | |
tree | 4721c73ed25b7445db1ffa394ac7b6b584919d7a /actionpack/test | |
parent | 5a9301ce47f1117c8d457cad2b850f932bf7f518 (diff) | |
download | rails-b677adede027eefed41f6b37096df7bbc4eb3d38.tar.gz rails-b677adede027eefed41f6b37096df7bbc4eb3d38.tar.bz2 rails-b677adede027eefed41f6b37096df7bbc4eb3d38.zip |
Move the `ActiveModel:Errors#full_message` method to the `Error` class:
- One regression introduced by the "AM errors as object" features is
about the `full_messages` method.
It's currently impossible to call that method if the `base` object
passed in the constructor of `AM::Errors` doesn't respond to the
`errors` method.
That's because `full_messages` now makes a weird back and forth trip
`AM::Errors#full_messages` -> `AM::Error#full_message` -> `AM::Errors#full_message`
Since `full_message` (singular) isn't needed by AM::Errors, I moved
it to the `AM::Error` (singular) class. This way we don't need to
grab the `AM::Errors` object from the base.
Diffstat (limited to 'actionpack/test')
0 files changed, 0 insertions, 0 deletions