Skip to content

Fillable forms#89

Open
polyglot-jones wants to merge 5 commits intoclaird:masterfrom
polyglot-jones:fillable_forms
Open

Fillable forms#89
polyglot-jones wants to merge 5 commits intoclaird:masterfrom
polyglot-jones:fillable_forms

Conversation

@polyglot-jones
Copy link

Added sample code that copies a PDF, changing field values along the way (i.e. using a PDF with fillable fields as a template). This required modifying the PdfFileWriter.updatePageFormFieldValues() method (in pdf.py) to let you flag the filled-in field as read-only. This also required adding a new method to PdfFileWriter called have_viewer_render_fields(). It sets the /AcroForm/NeedAppearances flag. (FYI: The example PDF (fillable_form.pdf) was created in LibreOffice using the attached fillable_form.odt document.)

NOTE: updatePageFormFieldValues() didn't have a unit test that I could see. When I added one, I also had to fix two adjacent tests (to close the temp file before deleting it) to get test_pdf.py running again.

NOTE: I'm not set up for tox at the moment. I tested it using 3.8 only.

…way (i.e. using a PDF with fillable fields as a template).

This required modifying the PdfFileWriter.updatePageFormFieldValues() method (in pdf.py) to let you flag the filled-in field as read-only.
This also required adding a new method to PdfFileWriter called have_viewer_render_fields(). It sets the /AcroForm/NeedAppearances flag.
The example PDF (fillable_form.pdf) was created in LibreOffice using the attached fillable_form.odt document.

NOTE: updatePageFormFieldValues() didn't have a unit test that I could see.
When I added one, I also had to fix two adjacent tests (to close the temp file before deleting it) to get test_pdf.py running again.

NOTE: I'm not set up for tox at the moment. I tested it using 3.8 only.
…() method.

Corrected a minor detail in testUpdatePageFormFieldValues().
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.

1 participant