Don’t simply test for the presence of the magic Paperclip attribute, it will return a paperclip Attachment object and thus always be true:
- if user.photo.present? # always true
= image_tag(user.photo.url)Use #photo? instead:
- if user.photo?
= image_tag(user.photo.url)
Glad to help :)
always surprised with RoR/gems simplicity and intuitive design
user.photo? didn't work for me.
user.photo.exists? works well.
user.photo? will tell you if the photo_file_name attribute is set in the database. user.photo.exists? will check that the file actually exists on the filesystem or S3, but is much slower.
But what if I want to know if a user.photo.url(:medium) exists ?
do it this way user.photo.exists?(:medium).
thx!
- if user.photo.present? # always true
= image_tag(user.photo.url)
That's not true! #present? checks if attachment was assigned and #exists? checks if file is on storage(S3 or whatever). Here is attachment code from paperclip:
# Returns true if a file has been assigned.
def file?
!original_filename.blank?
end
alias :present? :file?
def blank?
not present?
end
def exists?(style = default_style)
if original_filename
s3_object(style).exists?
else
false
end
rescue AWS::Errors::Base => e
false
end
That means #present? is much faster that #exists? and you rarely need second unless you have problems with your storage.
user.photo.exists? seems to actually be user.photo.exists?(:original). For some reason I didn't have an 'original' for some of my older attachments and this caused the logic to fail even though the thumbnails etc... were there.
just use the file_name column to check.
def thumbnail_path
thumbnail_file_name.present? ? thumbnail : nil
end
user.photo.exists? Always return me 'false', for both the cases.
It is showing me some message as, 'Digest::Digest is deprecated; use Digest', and then giving me 'false'.
user.photo_file_name.blank? is how i do it
That's exactly what I need, thank You!