Bug #10675

PDF derivatives not generating correctly on Ubuntu 16.04

Added by Steve Breker over 5 years ago. Updated over 5 years ago.

Status:VerifiedStart date:12/02/2016
Priority:MediumDue date:
Assignee:Dan Gillean% Done:

0%

Category:Digital object
Target version:Release 2.3.1
Google Code Legacy ID: Tested version:
Sponsored:No Requires documentation:

Description

When PDF digital objects are added to AtoM, derivatives created are entirely black if there are transparent layers in the PDF.

This issue only occurs under Ubuntu 16.04.

Community discussions describing the issue:

PDF derivatives issue: https://groups.google.com/d/msg/ica-atom-users/m2Hhs0i_xyw/76OXUCC2CAAJ
Same issue, previous thread: https://groups.google.com/d/topic/ica-atom-users/t5Xd-NM2IpA/discussion

TESTE.pdf (139 KB) Steve Breker, 12/14/2016 01:53 PM

History

#1 Updated by Steve Breker over 5 years ago

  • Copied from Bug #10633: Investigate potential Ubuntu 16.04 compatibility issues with CSV import added

#2 Updated by Steve Breker over 5 years ago

  • Copied from deleted (Bug #10633: Investigate potential Ubuntu 16.04 compatibility issues with CSV import)

#3 Updated by Steve Breker over 5 years ago

  • Status changed from New to Code Review
  • Assignee changed from Steve Breker to Nick Wilkinson

#4 Updated by Jesús García Crespo over 5 years ago

  • Status changed from Code Review to Feedback
  • Assignee changed from Nick Wilkinson to Steve Breker

#5 Updated by Steve Breker over 5 years ago

Sevein has given CR the thumbs up in Github.

#6 Updated by Steve Breker over 5 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from Steve Breker to Nick Wilkinson

Ready for QA. I have pulled this fix into the 16.04 qa/2.4.x test instance at:

http://198.199.111.25/

#7 Updated by Steve Breker over 5 years ago

The PDF posted in the user groups demonstrates the black derivative issue easily. I have attached it here.

#8 Updated by Dan Gillean over 5 years ago

  • Category set to Digital object
  • Status changed from QA/Review to Verified
  • Assignee changed from Nick Wilkinson to Steve Breker

#9 Updated by Steve Breker over 5 years ago

  • Assignee changed from Steve Breker to Nick Wilkinson

Cherry-picked to stable/2.3.x.

#10 Updated by Steve Breker over 5 years ago

While testing 10422 I discovered a bug in 10675 where the pdf mimetype was being examined before it was set to the class property variable. This change corrects this bug.

Code review required.

https://github.com/artefactual/atom/pull/489

#11 Updated by Steve Breker over 5 years ago

  • Status changed from Verified to Code Review

#12 Updated by Jesús García Crespo over 5 years ago

  • Status changed from Code Review to Feedback
  • Assignee changed from Nick Wilkinson to Steve Breker

LGTM

#13 Updated by Steve Breker over 5 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from Steve Breker to Nick Wilkinson

Merged to qa/2.4.x.

I have refreshed the test instance on the 16.04 test server and it is now running qa/2.4.x with this patch.

http://198.199.111.25/

Testing will need to be done using the file TESTE.pdf (linked above) to ensure that derivatives are not black squares, and with a very large .pdf file. Uploading a large pdf file via the WebUI should not cause the WebUI to time out - it should successfully complete and a number of seconds (10-20 seconds for really large ones, depending on size of file).

I was using this 1000+ page pdf to test speed (highlighted as problematic when we originally addressed this speed issue in 9918):
https://archivesdemontreal.ica-atom.org/index-noms-et-raisons-sociales-l

This will need to be tested and included in stable/2.3.x

#14 Updated by Nick Wilkinson over 5 years ago

  • Assignee changed from Nick Wilkinson to Dan Gillean

#15 Updated by Dan Gillean over 5 years ago

  • Target version changed from Release 2.3.1 to Release 2.3.2

#16 Updated by Dan Gillean over 5 years ago

  • Status changed from QA/Review to Verified

#17 Updated by Dan Gillean over 5 years ago

  • Target version changed from Release 2.3.2 to Release 2.3.1

Also available in: Atom PDF