search

farling42 / fvtt-pdf-pager

PDF Pager for Foundry
MIT License
6 stars 4 forks source link

Bug: Unable to open Purist Sheet in OSE 1.6.7 as character sheet #26

Closed justinthejaguar closed 1 year ago

justinthejaguar commented 1 year ago

Chrome node.js V10.288 Using AAC sheet https://necroticgnome.com/products/old-school-essentials-purist-character-sheet

Set up the sheet as a character sheet for this sheet, and

File shows corrupt "PDF.js v2.14.305 (build: eaaa8b4ad) Message: Invalid PDF structure."

farling42 commented 1 year ago

If you disable the PDF Pager module, and try to use the PDF with a journal PDF page do you get the same error?

farling42 commented 1 year ago

I just extracted the Old-School Essentials - Purist Character Sheet v1-2 - AAC.pdf from the zip file in your link, and the file loads perfectly inside Foundry. Are you sure that the file got uploaded to your Foundry server properly?

justinthejaguar commented 1 year ago

If you disable the PDF Pager module, and try to use the PDF with a journal PDF page do you get the same error?

No. I am able to open the PDF in a journal (even with PDF Pager)

I just extracted the Old-School Essentials - Purist Character Sheet v1-2 - AAC.pdf from the zip file in your link, and the file loads perfectly inside Foundry. Are you sure that the file got uploaded to your Foundry server properly?

-Out of curiosity are you running it in the OSE system. I am wondering if the conflict is the system. -The file is able to be opened through Samba just fine, so it is uploaded correctly. Here is the error I am receiving. image

Also to clarify, I updated to your latest version this morning before this testing.

farling42 commented 1 year ago

Hi, I just loaded the OSE game system and created a new world. Foundry 10.288 Old-School Essentials 1.6.7 Using the PDF document: Old-School Essentials - Purist Character Sheet v1-2 - AAC.pdf from the zip file downloaded in your link.

In the console window, I see a report about pdf.js version as: PDF 735c9e33bc515043badd17b6fffc355c [1.7 Adobe PDF Library 15.0 / Adobe InDesign 16.1 (Windows)] (PDF.js: 2.14.305)

I can see the PDF normally in a journal entry. I can configure the PDF to be used as an Actor sheet, and it gets displays as expected.

I see only one error reported in the app's console, which is about about a file not being found.

Where are you seeing the above screen shot within the app's window?

Are you seeing errors in the browser's main Console list of messages?

I opened up Firefox and logged into my local Foundry, and was still able to open the Actor showing the PDF character sheet.

justinthejaguar commented 1 year ago

The error appears in the console log for google chrome.

farling42 commented 1 year ago

I just started up my Node installation and the PDFs are still working. What do you have in the module settings box labelled "PDF to use for Actor 'character'" It is probably complaining about the file that has been specified in that box. I have my PDF at the toplevel of my userdata area, since my field simply contains Old-School%20Essentials%20-%20Purist%20Character%20Sheet%20v1-2%20-%20AAC.pdf

farling42 commented 1 year ago

The only message I see for viewer.html:1 is [DOM] Password field is not contained in a form: (More info: https://goo.gl/9p2vKq) <input type=​"password" id=​"password" class=​"toolbarField">​

I am using Chrome too. Could you try a different browser? To see if there is an extension or something else in your browser causing issues.

justinthejaguar commented 1 year ago

I have shut off all extensions in chrome and I still get the same error. Running Firefox I get the same error.

I have no modules active in the test aside from this one.

Attaching the file sheet from the server.

Old-School Essentials - Purist Character Sheet v1-2 - AAC.pdf

Screenshot of the sheet: image

farling42 commented 1 year ago

What do you have in the module settings box labelled "PDF to use for Actor 'character'"?

justinthejaguar commented 1 year ago

What do you have in the module settings box labelled "PDF to use for Actor 'character'"?

Oh my gosh. So obvious. I apologize for wasting your time. I had not filled this field...

farling42 commented 1 year ago

That's Ok. I am going to put into place something that will prevent being able to change the sheet to "PDF Sheet" if no PDF has been configured. This should prevent other users from encountering this same problem.

farling42 commented 1 year ago

Version 0.22.0 should prevent this issue happening (for other users) in the future, since the "PDF Sheet" option will only be presented if a PDF has been configured in the module settings.