The latest update of Media mover module not working.After running the configuration, I get an warning message saying 'Parameter 4 to mm_dir_media_mover() expected to be a reference,value given in /includes/modules.inc on line 462' and the files are not harvested from the local directory and stored to the FTP Server.This is an issue I face after setting the configurations.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | refactor-for-php-5.3.718374.5.patch | 6.88 KB | aaron |
| #2 | media_mover.fix_reference.718374.2.patch | 1.02 KB | aaron |
Comments
Comment #1
aaron commentedI confirm similar behavior. Wondering if there's a new bug that's been introduced. Here's my bug:
In my case, apparently mm_cck_media_mover is being invoked with module_invoke during node_load, and that function requires a reference for &file. Unfortunately, you can't actually call module_invoke to a function requiring a reference, as that function passes everything as a value.
mm_dir_media_mover() has the same issue...
Comment #2
aaron commentedthis patch fixes the problem in my case.
Comment #3
aaron commentedok, looks like this will fix this possibly edge case while maintaining current functionality. i've committed the patch. re-open if you still have an issue, @Pratik.
Comment #4
aaron commentedThis is actually related to PHP 5.3. I had to refactor the code considerably to get it working properly now, as it was throwing other similar issues. Patch forthcoming.
Comment #5
aaron commentedpatch attached
Comment #6
aaron commentedok, that's in now.
Comment #8
ha_le commentedHi Aaron,
Many thanks for your help. I've tried to apply your patch but receive a message say it's outdated... I've downloaded the latest stable version of MM. Is there any other patches that I should apply before this patch. Thanks.
Comment #9
arthurf commentedYou need to apply the patch to the 6.1.x dev version