PDF doesn't work with the d7-dev version as the output is sent, so the headers can't be sent and die() is called. Patch attached [#941050]

Simple patch attached which appears to solve the problem without adversely affecting other operations

Comment	File	Size	Author
#1	print_pdf_d7.diff	865 bytes	singularo
	print_pdf_d7.diff	527 bytes	singularo

Comments

Comment #1

singularo

Adelaide, AUS

commented 14 October 2010 at 00:55

Status	File	Size
new	print_pdf_d7.diff	865 bytes

Hmm, there was an error thats related sort of:

Warning: preg_match() expects parameter 2 to be string, array given in _print_generate_path() (line 597 of /projects/iseek/motorola/sites/all/modules/contrib/print/print.pages.inc).

drupal_get_http_header returns an array if no header item is specified, not a string.

However, when debugging this, the headers are not actually set to any value at all, but the body has MENU_NOT_FOUND set, which is the next thing in the code, and handles the issue.

So i've updated the patch to just remove that code, and set the changed variable which was missing in the dummy node and caused a token error.

Comment #2

shelleyp commented 28 January 2011 at 18:46

Just wanted to check before I filed another bug--this has to do with the fact that when I click the next page link and the next page opens, the header information isn't sent, and all I get is the body...correct?