- <% unless resource.single_page_pdf? %>
-
The preview shows the first page only — download to get all pages.
- <% end %>
<%= link_to resource_download_path(resource), download: true, class: "btn btn-utility" do %>
Download
<% end %>
@@ -16,6 +14,4 @@
The preview below may take a moment to load.
-<%= render "assets/display_assets",
- resource: resource, file: resource.display_image,
- variant: :hero, link: true %>
+<%= render "assets/resource_display", resource: resource %>
diff --git a/app/views/resources/show.html.erb b/app/views/resources/show.html.erb
index a0f45a6d2c..5d6077bb14 100644
--- a/app/views/resources/show.html.erb
+++ b/app/views/resources/show.html.erb
@@ -47,7 +47,7 @@
- <%= render "assets/display_assets", resource: @resource, file: @resource.display_image, variant: :hero, link: true %>
+ <%= render "assets/resource_display", resource: @resource %>
diff --git a/spec/models/resource_spec.rb b/spec/models/resource_spec.rb
index cdd166248e..3e4b0a1671 100644
--- a/spec/models/resource_spec.rb
+++ b/spec/models/resource_spec.rb
@@ -56,15 +56,4 @@
end
end
end
-
- describe "#single_page_pdf?" do
- it "is true for the known single-page titles" do
- expect(build(:resource, title: "W-9").single_page_pdf?).to be(true)
- expect(build(:resource, title: "Letter to Supervisors").single_page_pdf?).to be(true)
- end
-
- it "is false for any other title" do
- expect(build(:resource, title: "AHA Moments").single_page_pdf?).to be(false)
- end
- end
end
diff --git a/spec/requests/events/registration_ticket_callouts_spec.rb b/spec/requests/events/registration_ticket_callouts_spec.rb
index 037b6d8ece..794dddc78a 100644
--- a/spec/requests/events/registration_ticket_callouts_spec.rb
+++ b/spec/requests/events/registration_ticket_callouts_spec.rb
@@ -72,6 +72,35 @@
end
end
+ context "when linked to a PDF resource" do
+ let(:resource) { create(:resource) }
+ let(:callout) { create(:registration_ticket_callout, event:, resource:, description: "") }
+
+ before { create(:downloadable_asset, owner: resource) }
+
+ it "shows the PDF in the browser's inline viewer" do
+ get event_registration_ticket_callout_path(event, callout)
+
+ expect(response).to have_http_status(:ok)
+ expect(response.body).to include("type=\"application/pdf\"")
+ expect(response.body).to include(rails_blob_path(resource.downloadable_asset.file, disposition: :inline))
+ end
+ end
+
+ context "when linked to a non-PDF resource" do
+ let(:resource) { create(:resource) }
+ let(:callout) { create(:registration_ticket_callout, event:, resource:, description: "") }
+
+ before { create(:downloadable_asset, :with_image, owner: resource) }
+
+ it "renders the preview instead of an inline PDF viewer" do
+ get event_registration_ticket_callout_path(event, callout)
+
+ expect(response).to have_http_status(:ok)
+ expect(response.body).not_to include("type=\"application/pdf\"")
+ end
+ end
+
context "when linked to a resource without a downloadable file" do
let(:resource) { create(:resource) }
let(:callout) do
diff --git a/spec/requests/events/registrations_spec.rb b/spec/requests/events/registrations_spec.rb
index abab55caf9..3775bcc4bc 100644
--- a/spec/requests/events/registrations_spec.rb
+++ b/spec/requests/events/registrations_spec.rb
@@ -292,24 +292,6 @@
expect(response.body).to include(resource_download_path(resource))
end
- it "prompts to download for all pages on a multi-page resource" do
- resource = create(:resource, title: "AHA Moments", kind: "Handout")
- create(:downloadable_asset, owner: resource)
-
- get registration_resource_path(registration.slug, resource)
-
- expect(response.body).to include("download to get all pages")
- end
-
- it "omits the all-pages prompt on a known single-page resource" do
- resource = create(:resource, title: "W-9", kind: "Form")
- create(:downloadable_asset, owner: resource)
-
- get registration_resource_path(registration.slug, resource)
-
- expect(response.body).not_to include("download to get all pages")
- end
-
it "is reachable by slug without logging in" do
resource = create(:resource, title: "AHA Moments", kind: "Handout")