Trying to use a hypehnation tex file with Views PDf, I get the following fatal arror (& WSOD):

PHP Fatal error: Call to undefined method PdfTemplate::getHyphenPatternsFromTEX()

The offending code is at line 428 of views_pdf_template.php:

$hyphen_patterns = $this->getHyphenPatternsFromTEX($patternFile);

A little research found this: http://sourceforge.net/p/tcpdf/bugs/782/

It appears TCPDF has recently changed the way to call the method getHyphenPatternsFromTEX(). I will attach a patch to update the method call.

H.

CommentFileSizeAuthor
#7 call-to-undefined-method-1999562-7.patch817 byteskillua99
#1 fixed_call_to_TCPDF_method_getHyphenPatternsFromTEX-1999562-1.patch625 bytesAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
FileSize
fixed_call_to_TCPDF_method_getHyphenPatternsFromTEX-1999562-1.patch625 bytes

Attach patch.

Simon Georges’s picture

Simon Georges CreditAttribution: Simon Georges commented
Status: Needs review » Needs work

We would then be forced to up the requirements to TCPDF version 6.0.001 or greater.
This bugs me a little, since a lot of current Views PDF users are using TCPDF version 5.
Maybe we should detect the version in the code to adapt the function call?

killua99’s picture

killua99 CreditAttribution: killua99 commented
Priority: Normal » Critical
Status: Needs work » Reviewed & tested by the community

Exist a patch to handle this issues and others. Please see #2032189: Multiple issue fixing in a single patch and review others issues patch.

This is RTBC, and ready to apply.

Simon Georges’s picture

Simon Georges CreditAttribution: Simon Georges commented
Priority: Critical » Normal
Status: Reviewed & tested by the community » Needs work

I'd still like to address the concerns of the comment in #2.

killua99’s picture

killua99 CreditAttribution: killua99 commented

For the next "stable" you can say the version 6 > it's required. Or do a patch to handle the diff. We can think about it.

killua99’s picture

killua99 CreditAttribution: killua99 commented
Priority: Normal » Major

Change status issue filter option

killua99’s picture

killua99 CreditAttribution: killua99 commented
Priority: Major » Normal
Status: Needs work » Needs review
FileSize
call-to-undefined-method-1999562-7.patch817 bytes

This could be the solution. I tested and seens like works.

killua99’s picture

killua99 CreditAttribution: killua99 commented
Status: Needs review » Fixed

And directly committed reopen in case.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.