Bug #10675
PDF derivatives not generating correctly on Ubuntu 16.04
| Status: | Verified | Start date: | 12/02/2016 | |
|---|---|---|---|---|
| Priority: | Medium | Due 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
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
Ready for CR.
#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:
#7 Updated by Steve Breker over 5 years ago
- File TESTE.pdf added
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.
#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.
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