I use the Filefield Paths module to keep my files directory organized, so attachments for different content types go in their own subfolders. However, attachments to nodes that come in via Mailsave all get dumped in the root files folder, creating a mess. Can we get these modules to play nicely together? It seems like it might be something as simple as firing a hook after saving the attachment letting other modules (a la FileField Paths) know that the node has been edited and an attachment added, so that they can do their thing (move the file into the appropriate directory).

Comments

yixiu’s picture

yixiu commented

Subscribing

WeRockYourWeb.com’s picture

WeRockYourWeb.com commented

Probably not the solution you're looking for, but here's a hack that got the job done for me (mailsave_to_imagefield.module) line 107 start following:

      else {
        $widget_image_path = $field_info['widget']['file_path'];
      }

...

$ptitle = str_replace(" ", "-", $node->title);
$stitle = strtolower($ptitle);
   
      	$directory = file_directory_path()."/".$widget_image_path; //was file_create_path($widget_image_path);
		module_load_include('inc','filefield','field_file');   // added
		field_file_check_directory($directory, FILE_CREATE_DIRECTORY);  // was file_check_directory($directory, FILE_CREATE_DIRECTORY)

      	$destination = file_destination(file_create_path($widget_image_path .'/'. $stitle .'-'. $file['filename']), FILE_EXISTS_RENAME);
      	file_move($file['filepath'], $destination);

...

Portions of this were borrowed from a previous hack. Again, I know hacks are not the way to go, but I'm not good enough yet to rewrite all the code.

You'll notice that I borrow the node title, which comes from the email subject, to create the filename. You can omit this part if you don't want this functionality.

yngens’s picture

yngens commented

Subscribing. Would be nice to have official version instead of hack.

danepowell’s picture

danepowell commented

Have you tried the 2.x version of Mailhandler to see if it works with Filefield Paths? It handles files 'natively', no dependency on Mailsave...