The path set in filefield is still not taken into account while saving the file.

There is already an issue for that, with the same title, but for the 6.x-1.x version, which is quiet different and I don't want to hijack it: #704756: Filefield Path Support.

CommentFileSizeAuthor
#3 media_mover-filefield-path-1229210-2.patch5.08 KBosopolar
#1 media_mover-filefield-path-1229210-1.patch5.07 KBosopolar

Comments

osopolar’s picture

osopolar
Primary language German
Location 🇩🇪 GER 🌐
commented
Status: Needs review » Active
StatusFileSize
new media_mover-filefield-path-1229210-1.patch5.07 KB

There is a Helperfunction called mm_node_content_field_widget_files_directory. But the function filefield_widget_file_path from the filefield module does the same, even better. I suggest to use it. There was a comment before the first function:

@TODO Not sure that this is functioning as expected.

No it does not, because $file->data['user']->uid is not set before and module token is not necessary to get the right directory, only if token are in use.

osopolar’s picture

osopolar
Primary language German
Location 🇩🇪 GER 🌐
commented
Status: Active » Needs review
osopolar’s picture

osopolar
Primary language German
Location 🇩🇪 GER 🌐
commented
Status: Active » Needs review
StatusFileSize
new media_mover-filefield-path-1229210-2.patch5.08 KB

FIXED $dest to $destination_path in watchdog variables.