mm_cck_node_save attempts to return $file->filepath on completion, but field_file_save_file returns an array. This results in all sorts of strange behavior with files being duplicated, and step order getting messed up, patch attached is working for us.

CommentFileSizeAuthor
filefield_returns_array.patch584 bytescivicpixel

Comments

civicpixel’s picture

civicpixel commented
Status: Active » Needs review
arthurf’s picture

arthurf commented

The fix seemed to be a bit more significant- looks like I never cleaned up this function correctly from 1.x. As usual, I haven't really tested this yet, but it's on my list of cck stuff to go through.

darrick’s picture

darrick commented
Status: Needs review » Fixed

mm_cck no longer exists. Has been merged to mm_node. Not sure if this applied to mm_node_content_field_store or mm_node_save which do about the same thing. But that's a whole other issue.

istryker’s picture

istryker commented
Version: 6.x-2.x-dev » 6.x-1.0-beta9
Status: Fixed » Needs review

mm_cck still exist in 1.0.

arthurf’s picture

arthurf commented
Version: 6.x-1.0-beta9 » 6.x-2.x-dev
Status: Needs review » Fixed

@iStryker this issue was about the 2.x branch. If you are filing a bug against 1.x can you please open a separate issue? Since the codebases are dramatically different I don't want to track bugs for different versions in the same issue.

Status: Fixed » Closed (fixed)

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