Fatal error: Call to undefined function max_image_size_variable_settings_form_submit() in /var/www/html/scoutorg/docroot/includes/form.inc on line 1520

Comments

sushyl created an issue. See original summary.

sushyl’s picture

I think this patch could fix the bug.

steven.wichers’s picture

Priority: Normal » Minor
Status: Active » Postponed (maintainer needs more info)
Related issues: +#2693499: save settings: Call to undefined function

This is a duplicate of https://www.drupal.org/node/2693499

Nobody is able to provide directions to reproduce the issue on a clean install or point to specific modules causing the issue. Nothing in the code refers to max_image_size_variable_settings_form_submit, so it is an incompatibility with a contrib module.

The patch provided more works around the issue by not figuring out why an additional submit handler is breaking and instead just putting the code elsewhere.

pasqualle’s picture

Status: Postponed (maintainer needs more info) » Needs work

The bug is caused by the variable module

In variable_form_alter() the module alters the form submit function.
see: http://cgit.drupalcode.org/variable/tree/variable.module#n689

I am not sure if the variable_form_alter() function is correct, but changing the max_image_size_system_settings_form() function name (and its submit function) to something else should fix this issue.

pasqualle’s picture

Version: 7.x-1.4 » 7.x-1.x-dev
Status: Needs work » Needs review
StatusFileSize
new1.61 KB

Status: Needs review » Needs work

The last submitted patch, 5: 2826461-5-max_image_size_form_submit.patch, failed testing.

steven.wichers’s picture

Thanks Pasqualle, at first glance that does look like the culprit. I'll put some time on this a bit later to see if there's a better solution but if not I'll merge your patch.

anybody’s picture

Status: Needs work » Reviewed & tested by the community

RTBC. Please apply and create a new stable relase to fix the settings page. The patch is perfect.

  • Pasqualle authored 1b9ff4f on 7.x-1.x
    Issue #2826461 by sushyl, Pasqualle: Can not save configuration form due...
steven.wichers’s picture

Status: Reviewed & tested by the community » Fixed

I'm comfortable with renaming the function now that the root cause has been identified.

anybody’s picture

Thank you very much! :)

Status: Fixed » Closed (fixed)

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