Save document only once on set-fields

[lucasdf]

Description of your pull request

Change set-fields logic to save the document only once after all fields have been renamed. The current behavior calls .save after filling each field.

Why should it be considered?

The main motivation for this is to support using the function set-fields with a ByteArrayOutputStream.

The current logic calls .save after renaming each field, which appends the whole content of the PDF to the output stream, producing a bloated and potentially invalid PDF.

This is not a problem when passing a file since it just overwrites the file content each time, which works but seems inefficient.

Why am I asking this?

In order to support using this function to save the result to a ByteArray.

Pull request checklist

• The code is consistent with Clojure style guide.
• All code passes the linter (clj-kondo --lint src).
• You've added tests (if possible) to cover your change(s).
• All tests are passing.
• The commits are consistent with the Git commit style guide.
• You've updated the changelog (if adding/changing user-visible functionality).

Thanks for maintaining this lib!

[lucasdf]

adding this to run tests automatically using Github actions. seems useful, but I can remove it if you rather not add it. this will apply for next PRs after this one is merged.

[lucasdf]

using set-fields with either a file or a byte-array produces the same result

[dotemacs]

Thanks for the contribution @lucasdf!

I like it. I'll merge it soon.

Thanks again.

[rafaeldelboni]

@dotemacs any news on this? Do we need to do something else to get this merged? Also, it came to my attention that are still several open pull requests for a while, do you need any help evaluating them? :)

[dotemacs]

Hey @lucasdf

Thank you for doing this change.
I've merged it to master and I've updated the changelog.

Also, thank you for the patience in waiting for me to merge it

/cc @rafaeldelboni