search

MostTornBrain / pf2e-md-exporter

Foundry VTT markdown exporter optimized for PF2E content.
MIT License
8 stars 3 forks source link

Bug: Journal image attachments exported as broken webp images #9

Closed scooper4711 closed 2 months ago

scooper4711 commented 2 months ago

Journal attachment images for Handouts all come out as 16 byte broken webp images.

This is with both exporting both Cult of Cinders and Tomorrow Must Burn.

The non-forked md exporter exports the images just fine, but with a different naming convention.

MostTornBrain commented 2 months ago

Can you describe how you are exporting these? I want to make sure I'm reproducing the same issue. I do not own either of those modules, but assume it should happen with other modules as well.

scooper4711 commented 2 months ago
  1. I used the PDF Importer https://github.com/fryguy1013/pdftofoundry/
  2. I click on the Journal tab
  3. I select the folder for the adventure path
  4. I right click on the folder and select "Export to Markdown"
MostTornBrain commented 2 months ago

So something like this? image

This appears to export fine, including webp images: image

I'll check to see if I have any PDFs from Paizo I can import. Maybe it is something specific to how things are created in Foundry as part of the PDF import process...

MostTornBrain commented 2 months ago

Are you sure the pdf-to-foundry import process is creating the journal entry images correctly? I have tried several Paizo PDFs I own and a few of them after importing will contain broken webp images (of size 16 bytes) within Foundry itself specifically for some of the handout entries. This does not appear to be a PF2E Exporter issue - the source data the PDF import is creating appears to be incorrect in some instances. I see this issue with Fall of Plaguestone and Troubles in Otari as two examples:
image

scooper4711 commented 2 months ago

You nailed it.

Last time I did this (when the images worked), I believe I was on Foundry V10. With your new plugin, I'm re-doing everything and sure enough, those hand-out images are broken.

Closing this.