Media Mover does not delete files with Node deletes even with option selected...

rusirius - August 21, 2009 - 16:10
Project:Media Mover
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

I discovered an issue with Media Mover this morning. Even though we had the option enabled to delete media mover files with node deletion, I noticed it wasn't happening. The module would report back via the web interface that the deletes were taking place, but it was never actually doing it.

Looking at the code I found a small logic error with the media_mover_api.module. (I'm using 6.x-1.x-dev, but it possibly affects other versions?)

Line 3040 reads:
if (! $delete_item == $file['harvest_file']) {

and should be:
if ($delete_item != $file['harvest_file']) {

Here's a quick patch file...

AttachmentSize
mm_api_deletefix.patch993 bytes

#1

arthurf - September 15, 2009 - 05:08

Hi- thanks for the patch. I did some updates and removed some cruft from that function that I think should just make things a whole let better in that function- please give out the dev version a try and let me know how it works.

#2

arthurf - November 17, 2009 - 20:53
Status:active» closed

closed

 
 

Drupal is a registered trademark of Dries Buytaert.