Caught this after I noticed our configurations were not being run on cron/drupal_queue. Attached patch adds the configuration['enabled'] option to media_mover_ui_config_add_form and defaults it to TRUE so that new configurations are enabled unless otherwise specified.

Also adjusts the 'enabled' form item in media_mover_ui_config_edit_form to not be required, and adds logic to submit to convert the enabled true/false check box into the appropriate status (MMA_CONFIG_STATUS_ENABLED/MMA_CONFIG_STATUS_DISABLED)

Last but not least, adds $status to media_mover_configuration::load() so that it is available on the edit form.

CommentFileSizeAuthor
config_enabled_fix.patch2.93 KBcivicpixel

Comments

arthurf’s picture

arthurf commented

Patch is committed with slight code standardization. Ready for review!

civicpixel’s picture

civicpixel commented
Status: Needs review » Fixed

Tested on our development and live servers for a week, marking as fixed.

Status: Fixed » Closed (fixed)

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