Updated pf1 mapping.

[arvoituksellinen]

Fixes due to breaking changes in Foundry vtt version 10 data model

[dyoung418]

@arvoituksellinen I have tried this new PF1 mapping with the most recent updated pdf-sheet scripts (updated this week) and I get the following errors:

foundry.js:68095 PDF Sheet | Invalid mapping JavaScript Object. See the <a href="https://github.com/arcanistzed/pdf-sheet/blob/main/README.md">README</a> for more info.

and later,

VM3345:3 Uncaught TypeError: Cannot read properties of undefined (reading 'alignment')
    at eval (eval at createForm (pdfsheet.js:264:4), <anonymous>:3:393)
    at Pdfconfig.createForm (pdfsheet.js:264:77)
    at Pdfconfig.onFileUpload (pdfsheet.js:393:8)
    at reader.onload (pdfsheet.js:192:31)

[dyoung418]

Please fix the errors that happen when trying to export a character using the latest version of the repository with your mapping.

[gioppoluca]

Just to have a better understanding is a mapping problem in pf1? or in PF2? Which version of foundry are you using? I do not use PF so cannot test it

[dyoung418]

This PR is a pf1 mapping. I tried testing it in PF1. I'm not too familiar with pf1, but I created a PC and tried to export it and it failed as noted. It likely worked when the original author first submitted it, but it needs to be modified now to work again.

…

On Fri, Aug 25, 2023 at 3:40 AM Luca Gioppo ***@***.***> wrote: Just to have a better understanding is a mapping problem in pf1? or in PF2? Which version of foundry are you using? I do not use PF so cannot test it — Reply to this email directly, view it on GitHub <#79 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAZVCVE4QGKZPQA43KYOLJTXXB6ITANCNFSM6AAAAAAXPKIKTM> . You are receiving this because you commented.Message ID: ***@***.***>

[gioppoluca]

This means that the content of this PR is not valid.
In this case we'll not approve it thanks.
Hope that someone will take a look at PF1 mapping and make it work again.

[imposeren]

Here is partially working mapping file. There are some issues like "med" instead of size modifier, but it's better than nothing and can be used to some degree:
pf1.mapping.txt

Tested on foundry version 11 build 308, PF1 version 9.4

[jpverhaeghe]

@imposeren Thanks for updating this. If I get time, I'll try to update it further, languages aren't coming through either. It is a good stop gap.

[spellmux]

@imposeren thanks for the mapping, it's a good starting point that is at least functional, with the caveats you mentioned.