6.x-1.19 causes this error at every visit to print page:

Missing argument 2 for variable_get(), called in sites/all/modules/print/print.pages.inc on line 187

in fact, it needs 2nd argument.

CommentFileSizeAuthor
#1 print-fix-missing-argument-2235565-1.patch723 bytesAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Active » Needs review
FileSize
print-fix-missing-argument-2235565-1.patch723 bytes

Hereby a patch with the default NULL argument added to the variable_get() function.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz commented
Status: Needs review » Reviewed & tested by the community

The issue and patch are pretty straight forward. I have replicated the issue, tested the patch, and it looks good.

webservant316’s picture

webservant316 CreditAttribution: webservant316 commented

is this committed to dev?

jelo’s picture

jelo CreditAttribution: jelo commented

Ran into the same issue, applied the patch and error is gone as expected.

Can this be committed please?

johnrosswvsu’s picture

johnrosswvsu CreditAttribution: johnrosswvsu commented

I can verify the issue. Patch fixes it.

jackalope’s picture

jackalope CreditAttribution: jackalope commented

That patch worked well for me as well. I hope it gets committed soon. Thanks demarcoz!

yan’s picture

yan CreditAttribution: yan commented

Patch works for me, too. I think it's RTBC.

jcnventura’s picture

jcnventura CreditAttribution: jcnventura at Wunder commented
Status: Reviewed & tested by the community » Closed (outdated)

Closing all issues related to the Drupal 6 version of the print module.