[PATCH] fix for "HTML Input text controls doesn't print the value in PDF"

[ornati]

This little patch fixes:
wkhtmltopdf/wkhtmltopdf#2004
wkhtmltopdf/wkhtmltopdf#2005

There's also a typo "Parrent" vs "Parent" but I don't know if that affects anything ;)

diff --git a/src/gui/painting/qprintengine_pdf.cpp b/src/gui/painting/qprintengine_pdf.cpp
index 4988df1..e668fcf 100644
--- a/src/gui/painting/qprintengine_pdf.cpp
+++ b/src/gui/painting/qprintengine_pdf.cpp
@@ -278,9 +278,17 @@ void QPdfEngine::addTextField(const QRectF &r, const QString &text, const QStrin
     char buf[256];
     QRectF rr = d->pageMatrix().mapRect(r);
     if (d->formFieldList == -1) d->formFieldList = d->requestObject();
+    /*
+     * http://www.adobe.com/content/dam/Adobe/en/devnet/pdf/pdfs/PDF32000_2008.pdf
+     *
+     * 12.5.3 Annotation Flags - bit 3:
+     * (PDF 1.2) If set, print the annotation when the page is printed. If clear,
+     * never print the annotation, regardless of whether it is displayed on the screen.
+     */
     d->xprintf("<<\n"
                "/Type /Annot\n"
-               "/Parrent %d 0 R\n"
+               "/F 4\n" // printable flag
+               "/Parent %d 0 R\n"
                "/Rect[", d->formFieldList);
     d->xprintf("%s ", qt_real_to_string(rr.left(),buf));
     d->xprintf("%s ", qt_real_to_string(rr.top(),buf));

[ashkulz]

@ornati: thanks for the fix, will merge it this weekend.

[shineblu]

Hi,

Before merging please see additional comments there - https://groups.google.com/forum/#!topic/wkhtmltopdf-general/ebivj97OFbU

Thanks

[blittle]

Would love to see this merged and worked out.

[maxrb]

+1

[mchavarriagam]

Hi @ashkulz - Any update on this? I'm currently using 0.12.2.1 and ran into wkhtmltopdf/wkhtmltopdf#2005. Thanks! :)

[ashkulz]

Sorry for the long delay, but can someone create a patch for the /NeedAppearances true changes mentioned on the mailing list?

[ashkulz]

This issue has been fixed.