Suggestion: Use the Drupal libraries API to load TCPDF to reduce the ditribution size of this module and ease the code review. See Libraries.

Comments

karing’s picture

karing
Location 🇨🇦
commented

Would it go in /sites/all/libraries or possibly in sites/example.ca/libraries? We need to ensure Drupal doesn't find /sites/example.ca/modules/civicrm/packages/tcpdf first (that's only a very minimal install - v2.1 w/ 4.2.7 - we're using v3 with CiviCRM CDNTaxReceipts at the moment).

lolcode’s picture

lolcode commented
Status: Active » Fixed

It can go in /sites/all/libraries or in sites/example.ca/libraries. The location can be chosen from the admin settings. Changes committed on Jan 22nd.

jake.mw’s picture

jake.mw commented

Works great for me.

karing’s picture

karing
Location 🇨🇦
commented
Status: Fixed » Needs review

Not sure if this is in your ballpark - but:

When I go to:
/admin/config/civicrm_cdntaxreceipts/settings - to review my settings

I see:
PDF generation tool
one radio button option for sites/all/libraries/tcpdf/tcpdf.php
This option selects the PDF generation tool being used by this module to create the PDF output.

When I hit Save configuration at the bottom - I consistently get a
Notice: Use of undefined constant PRINT_PDF_PDF_TOOL_DEFAULT

lolcode’s picture

lolcode commented

Please try now. It should be fixed by commit e665320.