Add exception message to avoid exposing class #115
Merged
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.
This exception class is added to the model errors and shown to the end-user as is (
Errors::NotIdentifiedByImageMagickError
), because there is no message attached to it.The exception is added as a processing error here:
kt-paperclip/lib/paperclip/attachment.rb
Lines 537 to 540 in 11abf22
And the the processing errors are transferred to the model here:
kt-paperclip/lib/paperclip/attachment.rb
Lines 574 to 578 in 91923a6