Basically the same as #2171901: Do not add "Powered by TCPDF" link to PDF files

Self-assigning, as I am going to take a look now.

CommentFileSizeAuthor
#4 views_pdf_2845979_3.patch1.12 KBwizonesolutions
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wizonesolutions created an issue. See original summary.

wizonesolutions’s picture

Ah. Actually, we are using an older versioon of TCPDF, so it seems like we have to use this fix: http://kreativschmiede.blogspot.com/2010/12/tcpdf-getting-rid-of-hidden-...

Going to try in our subclass.

wizonesolutions’s picture

Version: 7.x-2.x-dev » 7.x-1.x-dev
Assigned: wizonesolutions » Unassigned
Status: Active » Needs review

Here's the patch...oh, oops, it's against 7.x-1.5..well, hopefully it'll at least serve as a starting point if it doesn't apply to dev. Although...yeah, I'm gonna see if I can make it do so, I think.

wizonesolutions’s picture

FileSize
1.12 KB

Oops, forgot the patch (and this one is for 7.x-1.x-dev).

vegansupreme’s picture

@wizonesolutions,
What version of TCPDF are you using? I've never seen this link in any of my Views PDF output. Does this link only show up on the last page? Can you recommend any steps to reveal this link?

vegansupreme’s picture

Oh! I just saw it for the first time…

wizonesolutions’s picture

@vegansupreme Yep. You have to copy the PDF text and paste it into a plain text editor, but it's there. A client was using a provider that OCR'd the PDFs (or something) and they mentioned that that line was messing up their process.

killua99’s picture

Never is to late to commit a patch, is in my todo task

killua99’s picture

Status: Needs review » Closed (outdated)