Posted by rusirius on August 21, 2009 at 4:10pm
Jump to:
| Project: | Media Mover |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
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...
| Attachment | Size |
|---|---|
| mm_api_deletefix.patch | 993 bytes |
Comments
#1
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
closed