knocked a PR to add support for `cfpdf action=extr...
# lucee
z
knocked a PR to add support for
cfpdf action=extractImages
https://github.com/lucee/extension-pdf/pull/53 also added a check for
overwrite=false
for extractThumbnail All with tests, additional eyes for a code review always welcome
👍 1
c
Good stuff. Just one incidental observation having tinkered with pdfBox extraction a little:
getXObjectNames()
seems to return a collection in reverse order of the objects' inclusion on the page. So if there are 2 images on the page, the second will be the first one in the returned collection. Probably fine most of the time, but might lead to unexpected results in cases where strict order is important.
z
that's quirky AF
let me extend that test case
yep, that's what it's doing, strange
image.png
PR updated with test case and fix
👍 1
published as PDF 1.2.0.6-SNAPSHOT
💥 1
metal 1