On line 526 of migrate_ui.pages.inc, the /tmp directory is hard-coded for the destination of the log file: $log_file = '/tmp/' . $drush_arguments[0] . $log_suffix;. It might be better to use the temporary:// wrapper and drupal_realpath like this: $log_file = drupal_realpath('temporary://' . $drush_arguments[0] . $log_suffix);

Comments

mikeryan’s picture

Good catch, could you submit a patch?

Thanks.

mac_weber’s picture

StatusFileSize
new1 KB

Sending patch.

mac_weber’s picture

Status: Active » Needs review

  • mikeryan committed 54bc680 on 7.x-2.x authored by Mac_Weber
    Issue #2415977 by Mac_Weber, elephant.jim: Use temporary:// instead of /...
mikeryan’s picture

Status: Needs review » Fixed

Committed, thanks!

mikeryan’s picture

Issue tags: +Migrate 2.8

Status: Fixed » Closed (fixed)

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