scinote-eln/scinote-web
1
1
Fork 0
mirror of https://github.com/scinote-eln/scinote-web.git synced 2024-09-21 23:46:21 +08:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2028 from okriuchykhin/ok_SCI_3817

Browse source 
Make previewable document filter more strict [SCI-3817]
This commit is contained in:
Alex Kriuchykhin 2019-09-04 15:40:01 +02:00 committed by GitHub
parent fd99e49fac c65c0cf356
commit 77a1df0068
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
config/initializers/paperclip.rb
View file

@ -63,9 +63,11 @@ Paperclip::Attachment.class_eval do
end
def previewable_document?
previewable = Constants::PREVIEWABLE_FILE_TYPES.include?(content_type)
extensions = %w(.txt .pdf .xlsx .docx .pptx .xls .rtf .doc .ppt .odt .ods .odp)
previewable = Constants::PREVIEWABLE_FILE_TYPES.include?(content_type) &&
extensions.include?(File.extname(original_filename))
extensions = %w(.xlsx .docx .pptx .xls .doc .ppt)
# Mimetype sometimes recognizes Office files as zip files
# In this case we also check the extension of the given file
# Otherwise the conversion should fail if the file is being something else