Fixing failing of prepare_for_active_storage
method when attachment is of type ActiveStorage::Attached::One
#380
+21
−4
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.
What this does
Say there is a model
user
with one attachedprofile_document
, then following method call would fail resulting inNoMethodError
.To fix this error we need to modify switch statement in
prepare_for_active_storage
method i.e. forActiveStorage::Attached::One
we should returnattachment.blob
instead ofattachment.blobs
.Type of change
Scope check
Quality check
overcommit --install
and all hooks passmodels.json
,aliases.json
)API changes