I'm using Print with webfm and private file downloads which is causing issues.
Essentially webfm obscures the URL for security purposes with the effect that _print_pdf_file_access_images() cannot get the file path from converting the URL.
This is similar to http://drupal.org/node/1333582 where you identified that without private file downloads the URL can be used (by not enabling use local paths), but this fails in my case as it will always convert private to local paths.

I can address this by checking for webfm paths, calling the file details and permissions check from webfm and then returning the correct file system path. I'm assuming however you're not going to want this in your codebase, so I was hoping you would be happy to enable a hook so that I can submit a patch to webfm to address the issue (or put in a custom module if necessary).

I've only really worked with using rather than invoking hooks before so I would appreciate it if you would advise where you would like it, then I'll have a crack at the development.

CommentFileSizeAuthor
#1 webfm.patch1.63 KBducklofty
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ducklofty’s picture

FileSize
1.63 KB

I've created a patch that resolves this issue, but this would be best implemented via a hook so the function in the attached patch can be added to the webfm module where I think it should reside.

Do you have a preference for where the hook should reside? Seems like where the if statement I've added is a sensible place.

ducklofty’s picture

Sorry, forgot to name the patch correctly.

jcnventura’s picture

Status: Active » Needs review
jcnventura’s picture

Issue summary: View changes
Status: Needs review » Closed (outdated)

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