Hi, thanks for this great module. I ran into a little problem with remotely located files.

I had to adjust the $temp_path for remote files so that it is within the Drupal temporary directory (./ isn't necessarily writable); moreover, freshly "copied" files were (at least in my case) immediately unlinked after the copying (because of a reference in drupal-core's file_copy()-function).

Please find a micro-patch attached (SVN).

CommentFileSizeAuthor
node_export_file_remote_file_urls.patch1.19 KBfrega

Comments

danielb’s picture

danielb commented
Component: Code » Node Export File
tech4him’s picture

tech4him commented

FYI. Patch worked for me also. Thanks!

danielb’s picture

danielb commented

not that important but thought I'd mention that
$temp_path = file_directory_temp().'/'.md5(mt_rand()) .'.txt';

should be

$temp_path = file_directory_temp() .'/'. md5(mt_rand()) .'.txt';

James Andres’s picture

James Andres commented
Status: Active » Fixed

This is already fixed in CVS (DRUPAL-6--2).

Status: Fixed » Closed (fixed)

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