Description

AAAAAAUUUUUggggGGGGGHHHHHHhhhRRrrrr

So class, the lesson today is that if you have system_settings_form($form) form (meaning masic submit and creating variables is handled automatically), you CANNOT have a helper submit function with the same name plus _submit.

OK, you already knew that.

Additionally -- and here's where the screams come in -- you cannot have it with a similar name where the whole thing is prefixed with an underscore, either.

Oh, never mind. Agaric had *both* functions beginning with an underscore:

function _userreference_access_admin()

function _userreference_access_admin_submit($form_id, &$form_values)

Don't let this happen to you! Tell your children!

$form['#submit']['_userreference_access_admin_submit_helper'] = array();
function _userreference_access_admin_submit_helper($form_id, &$form_values)

That's better. Um, no, it's not.

OK. Apparently Drupal 5 has no way to use the drupal core system settings submit function and an additional custom one as a supplement, and I was delusional just to try, let alone to think it worked for a moment.

You can still name your own submit function, but understand it will override and you have to save all your values yourself, counting on

No free variable saving if you want to do anything else.

Now you know.

Resolution

If you want to take over part of submit handling for a system_settings_form handled form, you have to take over all the submit handling.

Or can you have element-level submit handlers? I guess so. But whatever. Using set_variable ain't that hard.