scinote-eln/scinote-web
1
1
Fork 0
mirror of https://github.com/scinote-eln/scinote-web.git synced 2025-11-08 07:21:03 +08:00
Code Issues Projects Releases Packages Wiki Activity

Handle situations when file preview was not yet generated for shared tasks [SCI-9183] (#6106)

Browse source
This commit is contained in:
Alex Kriuchykhin 2023-08-30 13:30:12 +02:00 • committed by GitHub
parent fb177765da
commit fd58a23e55
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
app/views/shareable_links/my_modules/step_attachments/_file_preview.html.erb
View file

@ -32,7 +32,7 @@
<div class="file-preview-container">
<% if asset.pdf_previewable? %>
<%= render partial: 'shared/pdf_viewer', locals: { asset: asset, report_document: false, shareable_document: true } %>
<% elsif asset.previewable? && asset.large_preview.image.attached? %>
<% elsif asset.previewable? && asset.large_preview&.image&.attached? %>
<%= image_tag asset.large_preview.url(expires_in: Constants::URL_SHORT_EXPIRE_TIME.minutes),
class: 'asset-image',
style: 'opacity: 0' %>
@ -50,7 +50,7 @@
class: "previous-asset shareable-gallery-switcher",
data: { id: previous_asset.id } do %>
<i class="sn-icon sn-icon-arrow-left"></i>
<% if previous_asset.previewable? && previous_asset.medium_preview.image.attached? %>
<% if previous_asset.previewable? && previous_asset.medium_preview&.image&.attached? %>
<%= image_tag previous_asset.medium_preview.url(expires_in: Constants::URL_SHORT_EXPIRE_TIME.minutes),
class: 'asset-image',
style: 'opacity: 0' %>
@ -67,7 +67,7 @@
<%= link_to '#',
class: "next-asset shareable-gallery-switcher",
data: { id: next_asset.id } do %>
<% if next_asset.previewable? && next_asset.medium_preview.image.attached? %>
<% if next_asset.previewable? && next_asset.medium_preview&.image&.attached? %>
<%= image_tag next_asset.medium_preview.url(expires_in: Constants::URL_SHORT_EXPIRE_TIME.minutes),
class: 'asset-image',
style: 'opacity: 0' %>

2
app/views/shareable_links/my_modules/step_attachments/_inline.html.erb
View file

@ -25,7 +25,7 @@
</div>
<% if asset.pdf_previewable? %>
<%= render partial: 'shared/pdf_viewer', locals: { asset: asset, report_document: false, shareable_document: true } %>
<% elsif asset.previewable? && asset.large_preview.image.attached? %>
<% elsif asset.previewable? && asset.large_preview&.image&.attached? %>
<div class="image-container">
<%= image_tag asset.large_preview.url(expires_in: Constants::URL_SHORT_EXPIRE_TIME.minutes),
class: 'asset-preview-image',

2
app/views/shareable_links/my_modules/step_attachments/_thumbnail.html.erb
View file

@ -7,7 +7,7 @@
id: asset.id
} do %>
<div class="attachment-preview <%= asset.file.attached? ? asset.file.metadata['asset_type'] : '' %>">
<% if asset.previewable? && asset.medium_preview.image.attached? %>
<% if asset.previewable? && asset.medium_preview&.image&.attached? %>
<%= image_tag asset.medium_preview.url(expires_in: Constants::URL_SHORT_EXPIRE_TIME.minutes),
class: 'asset-preview-image',
style: 'opacity: 0' %>