scinote-eln/scinote-web
1
1
Fork 0
mirror of https://github.com/scinote-eln/scinote-web.git synced 2024-11-10 17:36:33 +08:00
Code Issues Projects Releases Packages Wiki Activity

Don't include PDF files in PDF reports if no permission [SCI-6300]

Browse source
This commit is contained in:
Martin Artnik 2021-11-24 11:01:57 +01:00
parent 5a7d285264
commit a3fa63152e
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
app/jobs/reports/pdf_job.rb
View file

@ -5,6 +5,7 @@ module Reports
extend InputSanitizeHelper
include InputSanitizeHelper
include ReportsHelper
include Canaid::Helpers::PermissionsHelper
PDFUNITE_ENCRYPTED_PDF_ERROR_STRING = 'Unimplemented Feature: Could not merge encrypted files'
@ -75,7 +76,7 @@ module Reports
file = prepend_title_page(file, template, report, renderer)
file = append_result_asset_previews(report, file) if report.settings.dig(:task, :file_results_previews)
file = append_result_asset_previews(report, file, user) if report.settings.dig(:task, :file_results_previews)
report.pdf_file.attach(io: file, filename: 'report.pdf')
report.pdf_ready!
@ -98,9 +99,11 @@ module Reports
private
def append_result_asset_previews(report, report_file)
def append_result_asset_previews(report, report_file, user)
Dir.mktmpdir do |tmp_dir|
report.report_elements.my_module.each do |my_module_element|
next unless can_read_my_module?(user, my_module_element.my_module)
results = my_module_element.my_module.results
order_results_for_report(results, report.settings.dig(:task, :result_order)).each do |result|
next unless result.is_asset && PREVIEW_EXTENSIONS.include?(result.asset.file.blob.filename.extension)