When using this with the latest print_pdf dev module, i got an error while generating pdf.

it seem that print_pdf function has changed a bit so some changes is required in certificate.pages.inc

old

print_pdf_generate_html(array('node' => $node), $output, $node->title . '.pdf');

new

print_pdf_generate_html($output, array('node' => $node), $node->title . '.pdf');

Comments

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented

Confirmed...we'll just have to add a check somewhere like a function_exists to verify the version of Print.

tempo22’s picture

tempo22 commented

Well, my correction is incorrect, the function actually require more meta parameters.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Active » Fixed

Committed 6504c46: added wrapper for calling different versions of print_pdf_generate_html()

Status: Fixed » Closed (fixed)

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

bladwin’s picture

bladwin commented

Can someone mark this RTBC if this issue has in fact been resolved or committed to the latest dev release?

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented

Hi,

It was committed in my #3 comment, and marked as fixed.

Related issue #1892454: Call to undefined function drupal_get_installed_schema_version() was also fixed. Both are available in the dev release.