Generated PDF file name issue

That's all handled by the token module. Of course, those tokens are only set for nodes... I will try to filter everything between [] for non-nodes..

subscribe

any ideas on that?
How could be manage this issue?

any news?

I think this amounts to a bug, as far as I can tell, this also fails (this is a webform generated URL to generate PDFs from submissions):

http://www.example.com/printpdf/131/submission/99

This also has no access to node variables at all. What I'm a little unsure about is weather this is a token or print module issue. If someone can point me to where this is handled I'll be glad to submit a patch.

I'd vote for "token issue" but maybe there is someone out there who wants square-brackets in their otherwise token-replaced strings. See for example #733192: Tokens enclosed in [ ] are not recognized

Well, it turns out this one in particular is a print module issue.

The node, which is passed to tokens was undefined. All I needed was to load it before the call to token_replace(). Patch attached.

Oh, I think it's wrong to assign it to myself... unassigning...

I need to see this better.. If this were the solution, tokens would never have worked.

Calls to token_replace() should be setting the clear option.

Patch attached.

@pillarsdotnet: that would be for the D7 version right?

Yes, but the same syntax is used in the 6.x-1.x version of the token module.

6.x-compatible patch attached.

@LUTi: I think you have to be using the current dev of token module for that patch to be effective (I still haven't tested this)

I am getting blank file names when using [title] as well. Any progress on this issue?

This problem was introduced as part of splitting the print_pdf_controller() function in order to provide an API for PDF generation (#1044138: Allow print_pdf to be called from another module and return PDF as a string (eg to attach a PDF with mimemail)).

I've fixed this in D7 and D6.

Not sure it is the same issue, but using print-6.x-1.14 or the last dev generates a pdf name like this
book_export_html_165.pdf

where 165 is the node ID

Note:
- pdf file name is set to default "[site-name] - [title] - [mod-yyyy]-[mod-mm]-[mod-dd]"
- using tcpdf

here is my "not super clean" solution.
on print_pdf.pages.inc L35, replace

 elseif (ctype_digit($args[0])) {
    // normal nid
    $node = node_load($args[0]);
  }

by

	elseif($item = menu_get_item($path)){
		if($item['page_callback'] == 'book_export'){
			// get the node from book export arguments
			$node = node_load($item['page_arguments'][1]);
		}elseif (ctype_digit($args[0])) {
    	// normal nid
	    $node = node_load($args[0]);
	  }
	}

patch #25 works fine.
18 months issue, please commit!

Thanks

This patch works for me as well. Please commit.

FYI, Patch #25 worked for me as well using the Drupal 6 latest Dev version.