Re-enable libreoffice in derivative_pdf_url()

medusa-project · Dec 22, 2023 · 09619bd · 09619bd
1 parent 5c3684a
commit 09619bd
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/app/util/derivative_generator.rb b/app/util/derivative_generator.rb
@@ -100,8 +100,7 @@ def derivative_pdf_url
 
     key = derivative_pdf_key
     unless store.object_exists?(key: key)
-      # TODO: update libreoffice and remove this conditional
-      self.generate_pdf_derivative if format.derivative_generator != "libreoffice"
+      self.generate_pdf_derivative
     end
     store.presigned_download_url(key: key, expires_in: expiry)
   end

diff --git a/test/util/derivative_generator_test.rb b/test/util/derivative_generator_test.rb
@@ -93,7 +93,6 @@ class DerivativeGeneratorTest < ActiveSupport::TestCase
 
   test "derivative_pdf_url() returns the URL of a PDF for a bitstream that can
   be converted into PDF" do
-    skip # TODO: unskip this when openoffice is enabled again
     @bitstream = bitstreams(:southwest_unit1_collection1_item1_doc)
     @generator = DerivativeGenerator.new(@bitstream)
     url        = @generator.derivative_pdf_url