Hi Arthur -- a list of errors thrown with the media mover port installed. Please contact me if you need more info...

With media mover api, ffmpeg media mover module and media mover cck active

1. When manually running cron

warning: constant() [function.constant]: Couldn't find constant MEDIA_MOVER_CRON_OFF_1 in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 290.

warning: constant() [function.constant]: Couldn't find constant MEDIA_MOVER_CRON_OFF_2 in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 290.

2. When harvesting from a cck field and processing with ffmpeg

Fatal error: Call to undefined function media_mover_api_cpu_check() in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 683

This is output at top of html on first run:

query: SELECT f.filepath, n.nid FROM files f
LEFT JOIN content_field_video c ON c.field_video_fid = f.fid
LEFT JOIN node n ON c.nid = n.nid
LEFT JOIN media_mover_files m ON m.fid = f.fid AND m.cid = 4
WHERE m.mmfid IS NULL AND n.changed >
ORDER BY n.nid DESC
error:You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ORDER BY n.nid DESC' at line 6

After first run this is output when there are no files to harvest:

query: SELECT f.filepath, n.nid FROM files f
LEFT JOIN content_field_video c ON c.field_video_fid = f.fid
LEFT JOIN node n ON c.nid = n.nid
LEFT JOIN media_mover_files m ON m.fid = f.fid AND m.cid = 3
WHERE m.mmfid IS NULL AND n.changed > 1224628453
ORDER BY n.nid DESC
error:

3. When the storage configuration attempts to save data to a cck field (which fails for both video and thumbnails):

# warning: Invalid argument supplied for foreach() in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 2921.
# warning: Invalid argument supplied for foreach() in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 2921.
# warning: Invalid argument supplied for foreach() in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 2921.
# warning: Invalid argument supplied for foreach() in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 2921.

5. When attempting to delete a node:

Fatal error: Cannot use string offset as an array in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 353

6. With media mover node installed attempting to add a configuration throws this error:

Fatal error: Call to undefined function locale_supported_languages() in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/mm_node/mm_node.module on line 603

7. With S3 media mover mdoule installed accessing media mover settings page throws this error:

Fatal error: require_once() [function.require]: Failed opening required 'Net/Socket.php' (include_path='.:/usr/local/lib/php') in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/contrib/mm_s3/drivers/Storage3/HTTP/Request.php on line 47

Comments

arthurf’s picture

arthurf commented

1. When manually running cron

warning: constant() [function.constant]: Couldn't find constant MEDIA_MOVER_CRON_OFF_1 in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 290.

Wow, yeah, that shouldn't happen. I changed the override settings to use the $conf array in settings.php

2. When harvesting from a cck field and processing with ffmpeg

Fatal error: Call to undefined function media_mover_api_cpu_check() in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 683

Fixed the function call name

This is output at top of html on first run:

query: SELECT f.filepath, n.nid FROM files f
LEFT JOIN content_field_video c ON c.field_video_fid = f.fid
LEFT JOIN node n ON c.nid = n.nid
LEFT JOIN media_mover_files m ON m.fid = f.fid AND m.cid = 4
WHERE m.mmfid IS NULL AND n.changed >
ORDER BY n.nid DESC
error:You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ORDER BY n.nid DESC' at line 6

Removed a queryd

After first run this is output when there are no files to harvest:

query: SELECT f.filepath, n.nid FROM files f
LEFT JOIN content_field_video c ON c.field_video_fid = f.fid
LEFT JOIN node n ON c.nid = n.nid
LEFT JOIN media_mover_files m ON m.fid = f.fid AND m.cid = 3
WHERE m.mmfid IS NULL AND n.changed > 1224628453
ORDER BY n.nid DESC
error:

This is the same issue as above- it's the queryd output


3. When the storage configuration attempts to save data to a cck field (which fails for both video and thumbnails):

# warning: Invalid argument supplied for foreach() in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 2921.

Added a catch for the foreach() on that line, need to investigate further as to why saving back to cck isn't working

5. When attempting to delete a node:

Fatal error: Cannot use string offset as an array in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 353

This was an issue which started with caching, but required quite a bit of fixing. This needs to get back ported to 5

6. With media mover node installed attempting to add a configuration throws this error:

Fatal error: Call to undefined function locale_supported_languages() in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/mm_node/mm_node.module on line 603

Function name changed, fixed now

7. With S3 media mover mdoule installed accessing media mover settings page throws this error:

Fatal error: require_once() [function.require]: Failed opening required 'Net/Socket.php' (include_path='.:/usr/local/lib/php') in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/contrib/mm_s3/drivers/Storage3/HTTP/Request.php on line 47

This maybe dependent on your PHP setup. This is a hard one to deal with, but I'll see what I can do about cleaning up the provided drivers

John Hodgins’s picture

John Hodgins commented

Hi Arthur,

I only got to look at this yesterday. Thanks for all your work!

I caught another media_mover_api_cpu_load_check() bug on line 477 and renamed it. And the CCK saving isn't working. But otherwise it looks great. I will test some of the other modules (S3, and directory) today and send you feedback.

Thanks,
John

John Hodgins’s picture

John Hodgins commented

Hi Arthur,

Still several critical bugs affecting the core media mover functionality:

- converts video twice on 1st and 2nd cron run. only happens when harvesting from CCK field, not regular file attachment.

- node deletion:
Fatal error: Cannot use object of type stdClass as array in /home/isuma/htdocs/2.0/sites/all/modules/media_mover/media_mover_api.module on line 2752
only happens when harvesting from CCK field, not regular file attachment.

- when adding a config:
* warning: Invalid argument supplied for foreach() in /home/isuma/htdocs/2.0/includes/form.inc on line 1194.
* warning: Invalid argument supplied for foreach() in /home/isuma/htdocs/2.0/includes/form.inc on line 1417.

- harvesting from file attachments: converts all previously uploaded videos whenever a new video node is added -- so the most recently added video node has all the video files from all previously created video nodes attached to it.

Leeteq’s picture

Leeteq commented

Subscribing.

John Hodgins’s picture

John Hodgins commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.