avoid adding .pdf to Manuscript#pdf_url

scholastica · Apr 30, 2024 · dc0e12b · dc0e12b
1 parent 948e099
commit dc0e12b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/lib/arxiv/models/manuscript.rb b/lib/arxiv/models/manuscript.rb
@@ -51,8 +51,7 @@ def available_in_pdf?
     def pdf_url
       if available_in_pdf?
         url = links.find { |l| l.content_type == "application/pdf" }.url
-        url = StringScrubber.force_ssl_url(url)
-        "#{url}.pdf" unless url =~ /\.pdf$/
+        StringScrubber.force_ssl_url(url)
       end
     end
 

diff --git a/spec/arxiv/models/manuscript_spec.rb b/spec/arxiv/models/manuscript_spec.rb
@@ -93,7 +93,7 @@ module Arxiv
 
     describe "pdf_url" do
       it "should return the url to download the manuscript in PDF format" do
-        expect(@manuscript.pdf_url).to eql('https://arxiv.org/pdf/1202.0819v1.pdf')
+        expect(@manuscript.pdf_url).to eql('https://arxiv.org/pdf/1202.0819v1')
       end
     end