Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Trying to add a module from http://localhost/drupal-7.x-dev/#overlay=%3Fq%3Dadmin%2Fmodules%2Finstall I get a fatal error.
Fatal error: Call to undefined function ajax_process_form() in /home/clemens/htdocs/drupal-7.x-dev/includes/form.inc on line 1246
Adding includes/ajax.inc fixes this step.
Is this related with #656782: ajax_process_form() results in settings being returned for elements that aren't re-rendered as part of the AJAX request? Guess not.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | ajax_process_form-704656-2.patch | 502 bytes | clemens.tolboom |
Comments
Comment #1
clemens.tolboomComment #2
clemens.tolboomComment #3
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks!
Comment #5
dwwI'm not sure this is a good idea, and I'm not sure why we need ajax.inc at all once you land on authorize.php. At that point, there should be *no* AJAX happening anymore, period. I'd like to revisit this bug and figure out what's really going on, not just include more code at authorize.php. Every .inc file we load during authorize.php makes the Update manager more fragile and makes it harder to ever solve #606592: Allow updating core with the update manager. So, generally speaking, any "fix" that involves "just load another .inc file" is probably the wrong fix in authorize.php. ;) Anyway, I'll look at this more closely when I have a chance, but I think we should roll back this patch. We should be completely out of the overlay by the time we hit authorize.php...
Comment #6
clemens.tolboomI managed to run through the process by adding a 'FileTransferLocal' to system_filetransfer_backends just to see what happens (dunno how to conf ftpd or sshd). The process uses a batch and thus ajax?
Disabling javascript breaks the protocol form "authorize_filetransfer_form" completely too ;(
(my two cents)
Comment #7
dwwHrm, so it's a new requirement that batch.inc depends on ajax.inc? Drat. In that case, this is okay, I guess.
Re: "Disabling javascript breaks the protocol form "authorize_filetransfer_form" completely too ;(" -- yup, known bug:
#609772: Impossible to extend the FileTransfer class system in contrib
Comment #8
Dave ReidSounds like this should be marked back as fixed then.