Move the ActiveModel:Errors#full_message
method to the Error
class:
#36639
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Move the
ActiveModel:Errors#full_message
method to theError
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
objectpassed in the constructor of
AM::Errors
doesn't respond to theerrors
method.That's because
full_messages
now makes a weird back and forth tripAM::Errors#full_messages
->AM::Error#full_message
->AM::Errors#full_message
Since
full_message
(singular) isn't needed by AM::Errors, I movedit to the
AM::Error
(singular) class. This way we don't need tograb the
AM::Errors
object from the base.cc/ @casperisfine @rafaelfranca