Closed (won't fix)
Project:
Drupal core
Version:
7.x-dev
Component:
system.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
11 Feb 2009 at 02:20 UTC
Updated:
29 May 2009 at 21:06 UTC
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?