Currently, system_themes_form() does it's own variable_set()'s and variable_del()'s, when that handling could be done by using return system_settings_form($form);.
Currently, system_themes_form() does it's own variable_set()'s and variable_del()'s, when that handling could be done by using return system_settings_form($form);.
Comments
Comment #1
foxtrotcharlie commentedIs this just a matter of replacing
with
in the system_themes_form in modules/system/system.admin.inc, or have I got the wrong end of the stick here? I'm not sure what needs to be done regarding variable_set() and variable_del()?
Comment #2
JamesAn commentedI don't think that's correct.
system_settings_form_submit() simply walks through the submitted form values and sets them as a persistent variable.
system_themes_form_submit():
Yes/no?
Comment #3
JamesAn commentedIf I'm understanding the difference between the two submit functions, this won't be fixed. Right?