Needs work
Project:
Media Mover
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Dec 2009 at 12:37 UTC
Updated:
1 Jun 2010 at 17:52 UTC
Hi,
while reviewing the code, I found this:
function media_mover_ftp_extract_user($ftp_file, &$file, $configuration) {
// check the top level director for a user name
if ($configuration['mm_ftp_user_from_dir']) {
if ($account = user_load(array('name' => $ftp_file['user_name']))) {
$file['data']['user'] = $account;
return;
}
}
if ($user = media_mover_ftp_token_check($ftp_file['token'])) {
$file['data']['user'] = $account; // CHECK THIS
}
}
Check the assignment line (in bold). AFAIK, this won't work since $account doesn't exist there? Shouldn't this assign a $user object?
Comments
Comment #1
hatsch commentedsame issue here. with recent dev version as well as beta7
new nodes are created with the file from ftp attached, but the node owner is set to whatever is set in the mediamover config as default value.
have you figured out any solution?