glutanimate / PDFMtEd

View and modify PDF metadata on Linux graphically
GNU General Public License v3.0
191 stars 22 forks source link

Added a Publisher field #24

Open ifohancroft opened 3 years ago

ifohancroft commented 3 years ago

I've added a Publisher field (XMP-dc Publisher and PDF-Info Producer). Closes #20

ifohancroft commented 2 years ago

I noticed yesterday that I didn't initially add the new UI field to the table in the README, so I updated the PR to include that change as well.

P.S. What is the idea behind var_unset() on line 136 in pdfmted-editor?

I noticed you unset some variables there and I thought I need to add the publisher variable in there as well, however, I noticed you don't unset all the variables that a document uses there. So, what is the idea behind it? Did you forget to add the other variables, or is there a reason only KEYWORDS, SUBJECT, AUTHOR, TITLE_FNN and UPDATED_NAME are being unset there?

Do I need to add PUBLISHER there as well?