Skip to content

utf8-support #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

utf8-support #22

wants to merge 1 commit into from

Conversation

laserK3000
Copy link

Hi,

thanks for maintaining this project. It helped me a lot!

I had issues with forms that contained fields labeled with non-ascii characters. These fields were left blank after filling the form.
It turned out that this problem can be fixed be submitting the fields in the form of an xfdf to pdftk

Please consider the attached PR

@altaurog
Copy link
Owner

altaurog commented Feb 7, 2022

Thanks for your interest in pdfforms, @laserK3000 ! This is really interesting. I’ve wanted this functionality myself. Once we’re generating xml, I think I’d be more comfortable doing it with xml.etree. How difficult would that be? Also, is there any use case for which the original method would be preferred?

@laserK3000
Copy link
Author

I am afraid I can't give a qualified answer if that change breaks functionality. I would assume xfdf is a superset of the fdf functionality, but I am not sure.

@altaurog
Copy link
Owner

altaurog commented Mar 3, 2022

I would like to incorporate this; I will test this approach myself when I have time, but unfortunately this is a rather low-priority project for me at the moment. I hope to carve a window for it in the next few months.

@rameshjes
Copy link

@altaurog thanks for such a wonderful tool.

Do you have updates on this issue? I also have diacritics in my CSV which aren't properly filled in PDF. Thanks

@altaurog
Copy link
Owner

I tried this branch some time ago but got errors somewhere and I was not able at the time to resolve them. Unfortunately with the war here I am not able to work on this currently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants