Closed (fixed)
Project:
Media Mover
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Aug 2009 at 16:10 UTC
Updated:
17 Nov 2009 at 20:53 UTC
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...
| Comment | File | Size | Author |
|---|---|---|---|
| mm_api_deletefix.patch | 993 bytes | rusirius |
Comments
Comment #1
arthurf commentedHi- 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.
Comment #2
arthurf commentedclosed