I installed Media Mover 5.x-1.x-dev on a fresh site and get the following warning when visiting /admin/settings/media_mover:

warning: Missing argument 1 for media_mover_api_config_advanced_settings_form(), called in /var/www/drupaltest/drupal-5.7/sites/all/modules/media_mover/media_mover_api.module on line 1381 and defined in /var/www/drupaltest/drupal-5.7/sites/all/modules/media_mover/media_mover_api.module on line 1643.

The page is shown ok, and I can save configuration changes.

CommentFileSizeAuthor
#1 media_mover_api.patch599 bytesbrunodbo

Comments

brunodbo’s picture

brunodbo
he/him
commented
Status: Active » Needs review
StatusFileSize
new media_mover_api.patch599 bytes

I added the argument on line 1381. I changed: 

$form['admin'][] = media_mover_api_config_advanced_settings_form();

to:

$form['admin'][] = media_mover_api_config_advanced_settings_form($form);

The error's gone now. Is it correct to do it like this?

arthurf’s picture

arthurf commented
Status: Needs review » Fixed

I did a slightly different fix on this, but thank you for the error report! Fix is in CVS, will get committed to 1-0-alpha2

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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